Skip to content

fix(ui): keep group and kind on typed references - #3607

Open
codyhartsook wants to merge 1 commit into
codyhartsook/ownedRefsfrom
codyhartsook/ownedRefsUI
Open

codyhartsook wants to merge 1 commit into
codyhartsook/ownedRefsfrom
codyhartsook/ownedRefsUI

Conversation

@codyhartsook

@codyhartsook codyhartsook commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Description

Stacked on #3604, which adds optional group and kind to model and MCP server references. This PR shows only the UI commit.

The UI rebuilt references from the name alone, so saving a form dropped group and kind. That points a reference to another kind at the same-named default object, which #3604 exists to prevent.

Editing a template now keeps group and kind on model and MCP server references. A server picked in the form still saves as a plain RemoteMCPServer reference.

Change Type

/kind fix

Changelog

The UI keeps `group` and `kind` on model and MCP server references when it saves them.

Validation

  • yarn typecheck, yarn test and yarn lint (0 errors).
  • Unit tests: a reference with group: example.com, kind: Foo survives a template edit.

@github-actions github-actions Bot added the bug Something isn't working label Nov 9, 2026
@codyhartsook
codyhartsook added this pull request to stack #3608 November 9, 2026 20:22
@codyhartsook
codyhartsook force-pushed the codyhartsook/ownedRefsUI branch 2 times, most recently from 60b6d70 to ad0d241 Compare November 10, 2026 18:50
@codyhartsook
codyhartsook force-pushed the codyhartsook/ownedRefsUI branch from ad0d241 to be8d132 Compare November 10, 2026 19:07
@github-actions github-actions Bot added bug Something isn't working and removed bug Something isn't working labels Nov 10, 2026
@blacksmith-sh

This comment has been minimized.

shashankram
shashankram previously approved these changes Nov 10, 2026
@codyhartsook
codyhartsook marked this pull request as ready for review November 10, 2026 20:06
@codyhartsook
codyhartsook force-pushed the codyhartsook/ownedRefsUI branch from be8d132 to daf10d2 Compare November 10, 2026 20:21
timflannagan
timflannagan previously approved these changes Nov 10, 2026
Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
codyhartsook force-pushed the codyhartsook/ownedRefsUI branch from daf10d2 to 11c092b Compare November 10, 2026 22:12
@codyhartsook
codyhartsook removed this pull request from stack #3608 November 11, 2026 18:54
@codyhartsook
codyhartsook added this pull request to stack #3648 November 11, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants