Skip to content

feat(web): rework secrets UI, drop unused budgets, clean up copy - #166

Merged
SumonMSelim merged 1 commit into
mainfrom
feat/secrets-ui-settings-cleanup
Aug 4, 2026
Merged

feat(web): rework secrets UI, drop unused budgets, clean up copy#166
SumonMSelim merged 1 commit into
mainfrom
feat/secrets-ui-settings-cleanup

Conversation

@SumonMSelim

Copy link
Copy Markdown
Collaborator

Summary

  • Secrets tab: official backend icons (Vault/AWS marks), default-first ordering, stored/encrypted badge for backend bootstrap creds, AWS region dropdown, dropped irrelevant chain-default note, multi-field auth methods on their own row
  • Removed unused Agent context-budget and Spend-budgets cards from Features tab (backend untouched)
  • Removed Bedrock STS session-token support end to end (unused)
  • Analytics latency table: human-readable durations instead of raw ms
  • Routes list: shows full ordered chain instead of a bare count
  • Swept every em dash from user-visible UI copy app-wide

Test plan

  • npm run build
  • npm run lint (0 errors, pre-existing warnings only)
  • npm test (394 tests passing)

Secrets tab gets backend icons (official Vault/AWS marks, matching
provider cards), default-first ordering, a stored/encrypted badge for
backend bootstrap credentials instead of a bare configured flag, and a
proper AWS region dropdown for Secrets Manager. Chain-only backends no
longer show an irrelevant "nothing stored" note, and multi-input auth
methods (named profile, access keys) now lay out on their own row.

Agent context-budget and spend-budget cards are removed from the
Features tab: neither backs a feature in active use yet, and their
presence implied more configurability than the product currently
offers. Backend endpoints are untouched.

Removes Bedrock's STS session-token support end to end (state, JSON
payload field, and UI) since it was unused, and swaps raw millisecond
latency figures in the analytics table for human-readable durations
(745ms, 8.9s, 2m 22s) via the existing formatDuration helper.

Routes list now renders the full ordered provider/model chain instead
of a bare count, so the chain composition is visible without opening
each route.

Also removes every em dash from user-visible UI copy across the app,
per the project's no-emdash convention.
Copilot AI lite review requested due to automatic review settings August 4, 2026 17:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@SumonMSelim
SumonMSelim enabled auto-merge (squash) August 4, 2026 17:39
@SumonMSelim
SumonMSelim merged commit bdb528e into main Aug 4, 2026
12 checks passed
@SumonMSelim
SumonMSelim deleted the feat/secrets-ui-settings-cleanup branch August 4, 2026 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants