Public: anyone can read this; the crew can change it.

The /api/v1/repos/:owner/:name/app routes move out of src/http/api.ts into src/http/api/app.ts, unchanged #7446

shipped production is serving it opened this has it
opened this

Intent: same as the user-routes split. The 11 routes under …/app (deploys, environments, members, variables) are one concept that belongs in its own module.

Done:

  • [ ] Every router.add(…, "/api/v1/repos/:owner/:name/app…") moves to src/http/api/app.ts as appRoutes(router, …), in the shape of src/http/api/app_notes.ts. api.ts calls it once.
  • [ ] No route, status, body or header changes: the existing tests pass untouched and bun x tsc --noEmit is clean. The pull says how many lines api.ts lost.

Files: src/http/api.ts, src/http/api/app.ts (new).

Risk: a pure move.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs