feat: add UI support for flag metadata

[devin-ai-integration]

Implements UI components for flag metadata CRUD operations.

Closes #2682

Link to Devin run: https://app.devin.ai/sessions/d62abd8bc3cb467dba175d847f5b8f07

Metadata entries can be of type primitive, array, or object.
If array or object we render the JSONEditor.
If primitive the user selects a subtype of 'string, number, or boolean'

Handling Errors

If there is an error rendering the MetadataForm we use an ErrorBoundary to render a more simplified version of the metadata which just uses the JSONEditor component:

TODO

• test with more complicated metadata
• Fix UI ITs

Robot Summary

This pull request introduces several changes across multiple files to enhance the handling of JSON editors, metadata in flags, and to update dependencies. The most significant changes include renaming and updating the JSON editor component, adding metadata support to flags, and updating test cases to reflect these changes.

JSON Editor Updates:

• Renamed ContextEditor to JsonEditor and updated its implementation to support additional props such as value, disabled, strict, and height. The linter function was also updated to handle strict mode. (ui/src/components/json/JsonEditor.tsx, ui/src/components/json/JsonEditor.module.css, ui/src/components/json/lint.ts) [1] [2] [3] [4] [5]

Metadata Support in Flags:

• Added metadata support to the FlagForm component, including validation and UI changes to handle metadata input. (ui/src/components/flags/forms/FlagForm.tsx, ui/src/app/flags/flagsApi.ts, ui/src/types/Flag.ts) [1] [2] [3] [4] [5] [6] [7]

Test Case Updates:

• Added new test cases to verify the creation, updating, and deletion of flags with metadata, as well as ensuring metadata cannot be edited in read-only mode. (ui/tests/flags.spec.ts) [1] [2]

Dependency Updates:

• Removed an outdated dependency from go.work.sum. (go.work.sum)

Miscellaneous:

• Added coverage to the .gitignore file to exclude coverage reports from version control. (ui/.gitignore)
• Minor UI adjustments in table-skeleton.tsx and updated a helper function in helpers.ts. (ui/src/components/ui/table-skeleton.tsx, ui/src/utils/helpers.ts) [1] [2]

[erka]

The object could be included in the metadata. This is an example what people have https://discord.com/channels/960634591000014878/1309109822657921034/1309120286142107668.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.84%. Comparing base (1c01075) to head (73a08b4).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3717   +/-   ##
=======================================
  Coverage   64.84%   64.84%           
=======================================
  Files         171      171           
  Lines       17391    17391           
=======================================
  Hits        11277    11277           
  Misses       5418     5418           
  Partials      696      696           

Flag	Coverage Δ
unittests	64.84% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[markphelps]

The object could be included in the metadata. This is an example what people have https://discord.com/channels/960634591000014878/1309109822657921034/1309120286142107668.

im updating it to support nested objects and other types such as arrays

[erka]

Great work @markphelps.

I think we have a very strict rules for keys right npw. Probably many people would like to use - and _ in the key name.
Another thing, I could have few records with the same key while adding. Is it by design?

[markphelps]

Great work @markphelps.

I think we have a very strict rules for keys right npw. Probably many people would like to use - and _ in the key name. Another thing, I could have few records with the same key while adding. Is it by design?

Cheers @erka ! I went ahead and reverted the change to the stringAsKey helper and added validation to check for duplicate keys and a test.

[erka]

nice!