[patrickbrandao]

Finishes the frontend leftovers of #75, #68 and #105. Backend parts landed in #153 and #157.

Changes

• Pendências da remoção do Autofix (#16) #68: removed the dropped auto_fixed field from the error-breakdown mocks in ErrorClassCard.preserved, AnalyticsChartSurfaces and GlobalOverviewFilter tests. No auto_fixed remains in packages/frontend.

Already done on main (verified, no change needed)

• Pendências da chave THINKER_CATALOG_SYNC (#39) #75 item 5: src/pages/Routing.tsx:343-347 shows a catalog-sync-off toast when sync_enabled === false (mentions THINKER_CATALOG_SYNC). Covered by tests/pages/Routing.test.tsx:888-902.
• Pendências da remoção do Autofix (#16) #68 fallback_error / superseded: MessageDetails.tsx:106 treats fallback_error or superseded === true as the recovered primary. Covered by MessageDetails.test.tsx:245 and MessageTable.test.tsx:577.
• Playground: headers do adaptador sempre vencem e pendências do #104 #105 item 4: RequestHeadersPopover.tsx:35-46 warns per row when a name is blocked, invalid, duplicated or over the cap, or when a value is invalid, and says the server will drop it. Covered by RequestHeadersPopover.test.tsx.
• Playground: headers do adaptador sempre vencem e pendências do #104 #105 item 5: playground-request-headers.ts:131-200 persists only valid, non-blocked header names, never values, and migrates and removes the legacy key that stored values.
• Legacy text sweep: no visible Manifest/mnfst text in src/, public/ or index.html. The only matches are the legacy storage-key migration helpers.

Tests (packages/frontend)

• lint: 0 errors (49 warnings, all pre-existing)
• typecheck: ok
• build: ok
• unit: 203 files, 4195 tests passed

Test-only change, so no changeset and no runtime change.

Closes #75
Closes #68
Closes #105

🤖 Generated with Claude Code

[patrickbrandao]

Review: #165 test(frontend): finish frontend leftovers of #75, #68, #105

Diff reviewed: git diff origin/main...fd8339b2024883122d3951b0fda3e739116a0491

Blockers: none.

Should-fix: none.

Nits: none.

Checks

• The diff only removes auto_fixed from the error-breakdown mocks in ErrorClassCard.preserved.test.tsx, AnalyticsChartSurfaces.test.tsx and GlobalOverviewFilter.test.tsx. git grep auto_fixed finds no remaining match in packages/frontend. The change is test-only, so no changeset or qa-docker is needed.

Closure evidence (spot-checked at fd8339b)

• Pendências da chave THINKER_CATALOG_SYNC (#39) #75 item 5: confirmed. src/pages/Routing.tsx:343-347 shows the THINKER_CATALOG_SYNC toast when sync_enabled === false. tests/pages/Routing.test.tsx:886-903 covers both the sync-off case and the sync-on case.
• Pendências da remoção do Autofix (#16) #68 fallback_error / superseded: confirmed. The handling is at src/components/MessageDetails.tsx:104-105; the PR body cites :106, a two-line drift. The test is at MessageDetails.test.tsx:245.
• Playground: headers do adaptador sempre vencem e pendências do #104 #105 item 4: confirmed. headerIssueMessage in src/components/playground/RequestHeadersPopover.tsx:33-47 has a message for each issue type: blocked, invalid name, control or unsupported characters, duplicate, and over the limit.
• Playground: headers do adaptador sempre vencem e pendências do #104 #105 item 5: confirmed. src/services/playground-request-headers.ts stores only valid, non-blocked header names (persistableNames, writeNames), never values, and removes the legacy key that stored values.

The closures of #75, #68 and #105 are justified.

CI on fd8339b: still running at review time. 3 of 18 runs are complete (Changeset check, Scripts, dependency-review, all success); the other 15 are in_progress. CI must finish green on this SHA before merge.

Head revisado: fd8339b
Veredito: aprovado