feat(web): group Authorize repository's GitHub picker by account - #2511
Conversation
With whole installations offered beside repositories, the picker was one flat list: every account's "All repositories" choice on top, then every repository from every account. It now groups by account, in installation order: each group is headed by the account, opens with that account's "All repositories" choice when it can still be granted, and then lists the account's repositories. Search still filters both. Part of #2398. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Approved. The account grouping preserves the picker’s existing search and selection behavior, and I found no blocking issue in this PR. git diff --check passed. Web tests and typecheck could not run here because dependencies and pnpm are unavailable.
sent by review-bot (Codex · gpt-6-sol) · open in session
|
The code review still passes at sent by |
|
unexpected status 401 Unauthorized: Incorrect API key provided: sk-svcac***********************************************************************************************************************************************************fvMA. You can find your API key at https://platform.openai.com/account/api-keys., url: https://chatgpt.com/backend-api/codex/responses, cf-ray: a40db0681b50ec9c-SEA, request id: 80b9d93c-9295-4dd3-b79d-78be2ff3b57e sent by |
There was a problem hiding this comment.
Approved at 74b843f. This review generation targets the same verified revision reviewed earlier. The account grouping preserves the picker’s existing search and selection behavior, and I found no blocking regression. Web tests and typecheck could not run in this checkout because dependencies and pnpm are unavailable.
sent by review-bot (Codex · gpt-6-sol) · open in session
Summary
Part of #2398. #2509 added whole installations to Authorize repository's GitHub picker, but the list was flat: every account's "All repositories in
<account>" at the top, then every repository from every account mixed together. The picker now groups by account.Changes
AddAgentRepoModal.tsx: the GitHub picker renders one group per account, in the order of the organization's installations. An account with no installation comes after them, alphabetically.<account>" choice, when the installation can still be granted, followed by the account's repositories.agent-multi-repo-authorization.md: the Web section describes the grouped picker.Tests
EditWorkspaceModal.installations.test.tsx: two accounts, one already granted. The groups come out in installation order, and each lists the installation choice, if still grantable, before its repositories.pnpm --dir packages/web exec vitest run --maxWorkers=2 src/components/console/modals/ src/components/console/WorkspaceCard.test.tsx: 25 files, 207 tests passed.pnpm --filter @agentconnect.md/web typecheckpasses, and prettier and eslint are clean on the changed files.🤖 Generated with Claude Code . Opus 5.5