Skip to content

fix(client): show runtime adapter account details - #177

Merged
NWYLZW merged 1 commit into
mainfrom
codex/fix-adapter-account-detail-fallback
Jul 28, 2026
Merged

fix(client): show runtime adapter account details#177
NWYLZW merged 1 commit into
mainfrom
codex/fix-adapter-account-detail-fallback

Conversation

@NWYLZW

@NWYLZW NWYLZW commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Route adapter account detail URLs to the existing account manager before the generic unknown-adapter JSON fallback.
  • Preserve the JSON editor for ordinary unknown adapters and avoid creating source configuration merely by viewing a runtime account.

Changelog

  • Changelog path: changelog/0.1.0-beta.10/client.md

Screenshots

Runtime adapter account detail with personal identifiers redacted

Validation

  • git diff --check
  • env DPRINT_CACHE_DIR=/private/tmp/dprint-cache pnpm exec dprint check apps/client/src/components/config/ConfigSectionForm.tsx apps/client/__tests__/config-schema-form.spec.tsx changelog/0.1.0-beta.10/client.md
  • pnpm exec vitest run apps/client/__tests__/config-schema-form.spec.tsx apps/client/__tests__/adapter-accounts-manager.spec.tsx
  • pnpm typecheck
  • Independent browser review: runtime account facts, quota, and account settings rendered with the local source still uncreated; no JSON fallback or viewport overflow; console warnings/errors were empty.

Experience Review

  • 已判断是否需要沉淀经验
  • 如需要,已运行 $post-task-experience-review
  • reviewer PASS 后才进入 merge

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
oneworks-dev Ready Ready Preview, Comment Jul 28, 2026 6:15pm

@github-actions github-actions Bot 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.

Experience Review

Before merge, make sure the PR body contains this completed checklist:

## Experience Review
- [x] 已判断是否需要沉淀经验
- [x] 如需要,已运行 `$post-task-experience-review`
- [x] reviewer PASS 后才进入 merge

Checklist to confirm:

  • Decide whether this PR produced reusable project experience.
  • If experience should be captured, run $post-task-experience-review.
  • Merge only after the independent reviewer reports PASS.

@NWYLZW
NWYLZW merged commit 82a2397 into main Jul 28, 2026
11 checks passed
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.

1 participant