Skip to content

fix(app): render stored keys from legacy-safe method labels - #4

Closed
AndreLCanada wants to merge 4 commits into
andre/prompt-queue-on-upstreamfrom
key-management-ui
Closed

AndreLCanada wants to merge 4 commits into
andre/prompt-queue-on-upstreamfrom
key-management-ui

Conversation

@AndreLCanada

Copy link
Copy Markdown
Owner

The generated SDK client was stripping the optional metadata fields from the ProviderAuthMethod schema before the UI received them. The connections array was empty by the time it reached the dialog component.

This removes the fragile dependency on connections entirely:

  • The UI derives the stored-key card from the stable method labels (Stored API key N (prefix...suffix))
  • The add-key method list filters those labels out
  • No reliance on optional schema fields that the legacy client can drop

Full 36-package typecheck passes.

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant