feat(006/5): cleanup legacy — spec 006 complete

[zbigniewsobiecki]

Summary

Plan 006/5 of spec 006 — the final cleanup plan. Spec 006 is complete after this merges.

No operator-visible changes. Routing tables, webhook flows, trigger dispatch, wizard UX, save body shapes — everything behaves identically to `dev`.

What's landed

Backend

• `src/integrations/bootstrap.ts` — DELETED. Its three responsibilities split cleanly:
  • PM manifests register via `src/integrations/pm/index.ts` (already in place from 006/1–006/4; this plan extends that barrel to mirror manifests into `integrationRegistry` for the cross-category capability resolver + integration-validation).
  • SCM (GitHub) self-registers via new `src/github/register.ts`.
  • Alerting (Sentry) self-registers via new `src/sentry/register.ts`.
• `src/pm/registry.ts` — converted to a read-only adapter over `pmProviderRegistry`. `get`/`getOrNull`/`all`/`createProvider`/`resolveLifecycleConfig` all delegate. `register()` is a deprecation warn. The 9 unmigrated call sites (webhook handlers, manual runner, credential scope, lifecycle, GitHub adapter) keep working unchanged — they transparently read from the manifest registry now. Single source of truth = `pmProviderRegistry` with zero divergent registration.
• `src/router/index.ts` + `src/worker-entry.ts` — import the three new side-effect modules instead of the deleted bootstrap.
• `src/integrations/registry.ts` — header comment updated to reflect the new population topology.

Test wiring

• `tests/unit/integrations/bootstrap.test.ts` — rewritten to assert the same end-state invariants against the new wiring (file name retained for audit lineage).
• 8 other test files that imported `bootstrap.js` as a side effect are migrated to import the three new modules.

Docs

• `src/integrations/README.md` — full rewrite. Transitional note + "Legacy path" section removed. The README is now the single canonical author's guide for adding a new PM provider.
• `CLAUDE.md` — integration-abstraction pointer reflects the final state.
• `CHANGELOG.md` — entry per plan.

Spec status

`docs/specs/006-pm-integration-plug-and-play.md.done` — the spec is marked complete in a trailing commit.

Plan-divergence (both documented in the `.done` plan)

• AC chore: add .nvmrc for Node.js 22 #2 (delete `src/pm/registry.ts`) became "convert to a read-only delegate." Deleting the file outright would require migrating 9 call sites across webhook handlers, manual runner, credential scope, lifecycle, and the GitHub adapter — all out of spec 006 scope. The delegate preserves the spec's intent (single source of truth) without downstream churn. A follow-up PR can migrate the call sites to `pmProviderRegistry` directly and then delete the adapter.
• AC feat: add UpdateChecklistItem gadget and improve agent workflow #5 (consolidate `createXxxLabel` / `createXxxCustomField` tRPC endpoints into `pm.discovery.*`) deferred to a follow-up PR. `useTrelloLabelCreation`, `useJiraCustomFieldCreation`, `useLinearLabelCreation` still call the per-provider endpoints and work correctly. Purely additive cleanup; not required for any spec AC.

Tests

• 7809 / 7809 tests pass
• Conformance harness still runs 44 assertions (11 × TestProvider + Trello + JIRA + Linear)
• Lint + typecheck + backend build + web build all clean

Spec 006 outcome-level ACs — all satisfied

1. ✅ New PM provider = one backend folder + one frontend folder; no edits to shared registries or wizard routing. Conformance harness fails CI on missing contract surface.
2. ✅ Harness exercises every registered provider on every cross-cutting behavior (44 assertions across 4 providers).
3. ✅ Trello, JIRA, Linear behave identically — zero operator-visible regressions across 5 PRs.
4. ✅ Cross-cutting provider logic has one canonical implementation per concern. Divergent copies (Bearer-prefix auth, UUID resolver, Linear extractor branch) are physically impossible now.
5. ✅ Wizard renders every PM provider via the manifest registry — no per-provider branches in `pm-wizard.tsx`.
6. ✅ Migration rolled out with zero half-migrated states — each PR independently revertable.
7. ✅ `src/integrations/README.md` rewritten as the single canonical author's guide.

Post-merge

• Local: full `npm test` green (7809/7809); lint + typecheck + backend build + web build all clean
• After deploy-dev: regression-test the three PM wizards — save each, trigger each provider's webhooks. Byte-for-byte identical behavior expected.

🤖 Generated with Claude Code

[codecov]

Codecov Report

❌ Patch coverage is 77.77778% with 8 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/pm/registry.ts	60.00%	6 Missing ⚠️
src/router/index.ts	0.00%	2 Missing ⚠️

📢 Thoughts on this report? Let us know!