Skip to content

[codex] guard settings API key entries#321

Draft
chaochaoweb3 wants to merge 1 commit into
CJackHwang:mainfrom
chaochaoweb3:codex/guard-api-key-objects
Draft

[codex] guard settings API key entries#321
chaochaoweb3 wants to merge 1 commit into
CJackHwang:mainfrom
chaochaoweb3:codex/guard-api-key-objects

Conversation

@chaochaoweb3

Copy link
Copy Markdown

Summary

  • normalize API key entries before rendering the settings key list
  • support both string keys and object entries with key/value fields
  • use the normalized string for display and delete confirmation

Fixes #302

Validation

  • npm run typecheck
  • npm exec -- vitest run src/components/settings/SettingsPage.test.ts
  • npm exec -- eslint src/components/settings/SettingsPage.tsx src/components/settings/SettingsPage.test.ts src/components/settings/apiKeys.ts
  • git diff --check

Note: full npm run lint still fails on existing unrelated files: ChatPanel.tsx, PortConfig.tsx, ProxySettings.tsx, and several context exports.

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.

添加keys之后,设置页无法打开

1 participant