Skip to content

feat: add v2 build override APIs#199

Merged
vigneshrajsb merged 3 commits intomainfrom
vb/build-config-overrides-api
May 7, 2026
Merged

feat: add v2 build override APIs#199
vigneshrajsb merged 3 commits intomainfrom
vb/build-config-overrides-api

Conversation

@vigneshrajsb
Copy link
Copy Markdown
Contributor

Summary

  • Move PR comment override mutations into OverrideService as structured build/service override methods.
  • Expand PATCH /api/v2/builds/{uuid} for build-table config patches: UUID, static mode, default-branch tracking, and comment env overrides.
  • Add PATCH /api/v2/builds/{uuid}/services for batched service active/branch/external URL overrides.
  • Update OpenAPI docs and shift route-wrapper tests into focused OverrideService behavior coverage.

Testing

  • pnpm run lint
  • pnpm run ts-check fails on existing unrelated repo-wide type errors; no errors were in the new route/override/OpenAPI changes, and remaining filtered hits are pre-existing activityStream.ts nullability errors outside changed hunks.
  • pnpm test -- --runTestsByPath src/server/services/__tests__/override.test.ts src/server/services/__tests__/activityStream.test.ts src/server/lib/__tests__/comment.test.ts src/shared/openApiSpec.test.ts
  • BUILD_MODE=yes DATABASE_URL=postgres://lifecycle:lifecycle@127.0.0.1:5433/lifecycle APP_REDIS_HOST=127.0.0.1 APP_REDIS_PORT=6380 LIFECYCLE_MODE=web pnpm test

Local API Verification

  • Restarted Tilt from this worktree.
  • Verified PATCH /api/v2/builds/{uuid} for UUID changes, isStatic, trackDefaultBranches, env object replacement, empty env replacement, and same-UUID validation.
  • Verified PATCH /api/v2/builds/{uuid}/services for batched service updates, empty batch validation, missing-service validation, active state changes, branch overrides, external URL mutation, and redeploy queueing.

@vigneshrajsb vigneshrajsb marked this pull request as ready for review May 6, 2026 15:50
@vigneshrajsb vigneshrajsb requested a review from a team as a code owner May 6, 2026 15:50
@vigneshrajsb vigneshrajsb merged commit 5b61e0f into main May 7, 2026
1 check passed
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