feat(connected-accounts): support REVOKED status added in Apollo #9550

@zen-agentchecks n/achecks…zen/connection-status-revoked-ig5g3g → main5 files · +31 −3updated 1mo ago

▸Description· 1 comment · 2 noise

Description

Apollo (hermes#9550) added a new `REVOKED` value to the connected-account status enum, alongside the existing `INITIALIZING | INITIATED | ACTIVE | FAILED | EXPIRED | INACTIVE`. Apollo's public-API response schema is `z.nativeEnum(ConnectionStatusEnum)`, so any list / get / status-update endpoint now returns `REVOKED` for revoked OAuth connections. The frontend's OpenAPI types regenerate from staging on every `pnpm dev` / `pnpm build`, so the wider type flows in automatically — but a few hand-written status unions did not include `REVOKED` and the filter dropdown didn't expose it.

What changed

File	Change
`apps/web/features/auth-configs/hooks/useGetConnectedAccounts.ts`	Add `'REVOKED'` to the `statuses` type union
`apps/web/features/auth-configs/hooks/useGetConnectedAccountsByAuthConfigIds.ts`	Add `'REVOKED'` to the `statuses` filter type union
`apps/web/features/playground/components/chat-screen/settings-panel/connected-account-picker/ConnectedAccountPicker.tsx`	Add `'REVOKED'` to the `statuses` prop union
`apps/web/features/auth-configs/components/auth-config-details/connected-account-listing/ConnectedAccountListing.tsx`	Loosen the inline cast at the call site so `'REVOKED'` flows through to the SDK call
`apps/web/features/auth-configs/components/auth-config-details/connected-account-listing/ConnectedAccountFilters.tsx`	Add a "Revoked" entry to the status filter dropdown so users can filter for revoked accounts

What did NOT need to change

`status === "ACTIVE"` checks (row classification at `ConnectedAccountListing.tsx:124-127`, the `is_active` flag at line 136, OAuth flow in `FormInputScreen.tsx`, the side-peak proxy verification in `ConnectedAccountDetailsSidePeak.tsx`) all treat `REVOKED` as not-active, which is the correct behavior.
The auto-generated `apps/web/generated/` types regenerate from staging on `pnpm dev` / `pnpm build` and now include `'REVOKED'` in the OpenAPI status union (verified: 13 hits for `REVOKED` in `generated/types.gen.ts` after a fresh `pnpm openapi:generate`).

How did I test this PR

`pnpm openapi:generate` against staging — confirmed `'REVOKED'` appears 13 times in the freshly-generated types
`pnpm lint` — clean (`0 errors, 322 warnings` — all pre-existing)
Build (`pnpm build`) fails on `/auth/clear` page-data collection because of missing runtime env vars in the sandbox; reproduces on `origin/main` without my changes — pre-existing, not caused by this PR
Existing `status === "ACTIVE"` semantics already produce the correct UX for `REVOKED` (faded row classification, no "Add Trigger" button)

Triggered by: sarthak@composio.dev | Source: web Session: https://zen-api-production-4c98.up.railway.app/dashboard/#/chat/zen-14dc825802da

@zen-agent1mo ago

Post-PR status update

Codex review: 1 pass; clean. "The changes consistently add REVOKED to connected account status filters and related TypeScript unions. No regressions or blocking issues were identified in the modified code."

CI: All 10 checks passing (typecheck, lint, all 4 test shards, merge-reports, 3 CodeQL analyzers).

Tests:

pnpm openapi:generate against staging — confirmed 'REVOKED' appears 13 times in the freshly-generated apps/web/generated/types.gen.ts
pnpm lint — clean (0 errors, 322 warnings — all pre-existing on main, none in any file I touched)
Build (pnpm build) fails on /auth/clear page-data collection because of missing runtime env vars in the sandbox — same failure on origin/main without my changes, pre-existing and unrelated

Reviewer comments: No human review comments to address (only Vercel preview + Playwright bot comments).

▸ 2 bot/status comments hidden

@vercel[bot]1mo ago

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

Project	Deployment	Actions	Updated (UTC)
platform-frontend	Ready	Preview, Comment	May 5, 2026 4:41am

@github-actions[bot]1mo ago

Playwright Test Results

📊 View HTML Report

Tests completed in parallel shards.

loading diff…

@zen-agent1mo ago

Post-PR status update

CI: All 10 checks passing (typecheck, lint, all 4 test shards, merge-reports, 3 CodeQL analyzers).

Tests:

pnpm openapi:generate against staging — confirmed 'REVOKED' appears 13 times in the freshly-generated apps/web/generated/types.gen.ts
pnpm lint — clean (0 errors, 322 warnings — all pre-existing on main, none in any file I touched)
Build (pnpm build) fails on /auth/clear page-data collection because of missing runtime env vars in the sandbox — same failure on origin/main without my changes, pre-existing and unrelated

Reviewer comments: No human review comments to address (only Vercel preview + Playwright bot comments).