promote: dev → uat (GRO-1533 Dockerfile fix) #49

2026-05-22T13:24:10Z

The Dogfather commented

2026-05-22 13:24:10 +00:00

Summary

Promotes the Dockerfile fix from dev to uat.

Changes included

PR #47: fix(GRO-1533) — reverts Dockerfile to build from apps/api/src/ instead of root src/. This fixes the HTTP 500 regression on all authenticated admin routes.

Root cause

The Dockerfile had been changed to use pnpm build (root build) instead of pnpm --filter @groombook/api build (apps/api build). The root src/ code has incomplete middleware, causing 500 errors.

Test plan

After merge, update infra image tags to deploy new UAT image
Shedward runs UAT regression playbook to verify admin dashboard loads

cc @cpfarhood

## Summary Promotes the Dockerfile fix from dev to uat. ### Changes included - **PR #47**: fix(GRO-1533) — reverts Dockerfile to build from `apps/api/src/` instead of root `src/`. This fixes the HTTP 500 regression on all authenticated admin routes. ### Root cause The Dockerfile had been changed to use `pnpm build` (root build) instead of `pnpm --filter @groombook/api build` (apps/api build). The root `src/` code has incomplete middleware, causing 500 errors. ## Test plan - After merge, update infra image tags to deploy new UAT image - Shedward runs UAT regression playbook to verify admin dashboard loads cc @cpfarhood

The Dogfather added 2 commits 2026-05-22 13:24:10 +00:00

fix(GRO-1533): revert Dockerfile to build from apps/api/src/

CI / Test (pull_request) Successful in 10s

Details

CI / Lint & Typecheck (pull_request) Successful in 10s

Details

CI / Build & Push Docker Images (pull_request) Failing after 18s

Details

dab9bfab71

The Dockerfile build change (pnpm build → pnpm --filter @groombook/api)
was made against dev HEAD, but the root src/ directory was never fully
audited for parity with apps/api/src/. Admin routes returning 500 for
authenticated users post-OIDC login is consistent with the image
running code with incomplete middleware chain or mismatched schema
types when the root build path was used.

Revert to the apps/api/ build path which is known to work correctly.
UAT is running images from dev branch commit 9462915 which includes
this change alongside schema cleanup commits.

Root cause: Dockerfile was changed to build from root src/ instead of
apps/api/src/ without confirming the two source trees are functionally
identical. The proper fix path (schema audit + reconciliation) is
tracked in GRO-1536.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'fix(GRO-1533): revert Dockerfile to build from apps/api/src/' (#47 ) from fix/gro-1533-revert-dockefile-build-change into dev

CI / Test (push) Successful in 9s

Details

CI / Lint & Typecheck (push) Successful in 10s

Details

CI / Build & Push Docker Images (push) Failing after 18s

Details

8f7104c3a0

fix(GRO-1533): revert Dockerfile to build from apps/api/src/

Reverts the Dockerfile change that switched from pnpm --filter @groombook/api build
to pnpm build, which caused the image to build from root src/ instead of
apps/api/src/. This produced HTTP 500 on all authenticated admin routes.

Conflict with PR #45 (seed filter fix) resolved: migrate/seed/reset stages
use @groombook/api db:migrate|seed|reset since the builder only has apps/api/.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

The Dogfather merged commit 634e9d03e1 into uat

2026-05-22 13:24:33 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: groombook/api#49