refactor(pipeline-view): migrate to adapter factory (MF-10) #318

Merged
code-lead merged 1 commit from feat/mf10-migrate-pipeline into main 2026-04-24 10:39:45 +00:00
Collaborator

Summary

  • Cleanup of MF-8 (#299) out-of-scope list — migrates pipeline.ts.
  • Replaces the five new ForgejoAdapter(token) callsites in apps/server/src/domain/views/pipeline.ts with createForgeAdapterForRepo(repo, token) so the /issues/pipeline (M19-1) and /pipeline/bounce-review (M19-6) paths route through the per-repo forge binding.
  • Ensures dashboards for github/gitlab-bound repos render the correct stage model instead of hard-wiring Forgejo.

Test plan

  • just qa — typecheck + Biome clean across all 4 workspaces.
  • bun test src/domain/views/pipeline*.test.ts — all 37 pipeline tests pass.
  • grep "ForgejoAdapter" apps/server/src/domain/views/pipeline.ts — no remaining references.
  • Smoke-check dashboard against a non-Forgejo repo once multi-forge is enabled end-to-end.

Ref: MF-10 cleanup follow-up from PR #313 (MF-8).

## Summary - Cleanup of MF-8 (#299) out-of-scope list — migrates `pipeline.ts`. - Replaces the five `new ForgejoAdapter(token)` callsites in `apps/server/src/domain/views/pipeline.ts` with `createForgeAdapterForRepo(repo, token)` so the `/issues/pipeline` (M19-1) and `/pipeline/bounce-review` (M19-6) paths route through the per-repo forge binding. - Ensures dashboards for github/gitlab-bound repos render the correct stage model instead of hard-wiring Forgejo. ## Test plan - [x] `just qa` — typecheck + Biome clean across all 4 workspaces. - [x] `bun test src/domain/views/pipeline*.test.ts` — all 37 pipeline tests pass. - [x] `grep "ForgejoAdapter" apps/server/src/domain/views/pipeline.ts` — no remaining references. - [ ] Smoke-check dashboard against a non-Forgejo repo once multi-forge is enabled end-to-end. Ref: MF-10 cleanup follow-up from PR #313 (MF-8).
refactor(pipeline-view): migrate to adapter factory (MF-10)
All checks were successful
qa / qa (pull_request) Successful in 3m56s
qa / dockerfile (pull_request) Successful in 10s
61b2e16702
Cleanup of MF-8 (#299) out-of-scope list — migrates the five
`new ForgejoAdapter(token)` callsites in `pipeline.ts` to
`createForgeAdapterForRepo(repo, token)` so the pipeline view
routes through the per-repo forge binding (github/gitlab/forgejo).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
code-lead deleted branch feat/mf10-migrate-pipeline 2026-04-24 10:39:46 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
charles/claude-hooks!318
No description provided.