explicit mapping in labels

[skull8888888]

Important

Refactor label value handling by changing valueMap to Record<string, number> and update related components and styles.

• Behavior:
  • Change valueMap from string[] to Record<string, number> in LabelClass and SpanLabel in types.ts.
  • Update label value handling in AddLabel, AddLabelInstance, and SpanLabels to use the new valueMap structure.
  • Ensure duplicate numerical values are not allowed in AddLabel.
• UI Components:
  • Modify EvaluatorEditor, AddLabelPopover, and AddLabel to map over Object.entries(labelClass.valueMap) instead of labelClass.valueMap.
  • Update UI to display label keys instead of indices in EvaluatorEditor and AddLabelPopover.
• Styles:
  • Adjust color variables in globals.css for secondary-foreground, muted-foreground, and destructive.

^{This description was created by for 515ac9a. It will automatically update as commits are pushed.}

[ellipsis-dev]

❌ Changes requested. Reviewed everything up to 515ac9a in 1 minute and 13 seconds

More details

• Looked at 366 lines of code in 6 files
• Skipped 0 files when reviewing.
• Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_2pwvsERHuMvKDMU9

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.