Skip to content

Extract 18 inline route handlers from registerReleaseRoutes()#781

Merged
selfcontained merged 1 commit into
mainfrom
agt_85a37d65d9af/job-debt-collector-76120549
Jul 16, 2026
Merged

Extract 18 inline route handlers from registerReleaseRoutes()#781
selfcontained merged 1 commit into
mainfrom
agt_85a37d65d9af/job-debt-collector-76120549

Conversation

@selfcontained

Copy link
Copy Markdown
Owner

Summary

What qualifies as tech debt

All business logic was defined inline within one 714-line function body. This made it hard to navigate, test, or understand individual route handlers in isolation.

Next up

The next debt-collector run will tackle: extracting sub-functions from the 233-line shutdown() method in apps/server/src/jobs/service.ts.

Test plan

  • pnpm run check — TypeScript clean
  • pnpm run test — 200/200 unit tests pass
  • pnpm run test:e2e — 122/123 pass (1 pre-existing flake in split-pane.spec.ts)

🤖 Generated with Claude Code

@selfcontained
selfcontained merged commit fce4348 into main Jul 16, 2026
1 check passed
@selfcontained
selfcontained deleted the agt_85a37d65d9af/job-debt-collector-76120549 branch July 16, 2026 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant