groombook/api
13
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

revert: undo PR #47 Dockerfile apps/api switch (broke CI Docker build) #50

Merged
The Dogfather merged 1 commits from revert/gro-1533-dockerfile-fix into dev 2026-05-22 13:31:12 +00:00
Conversation 0 Commits 1 Files Changed 1
+22 -10
The Dogfather commented 2026-05-22 13:31:05 +00:00
Member
Copy Link
Copy Source

Summary

Reverts PR #47 which switched the Dockerfile to build from apps/api/src/. This broke CI Docker builds (runs #968, #970) because:

  1. pnpm-workspace.yaml only includes packages/*, not apps/*
  2. pnpm-lock.yaml has no apps/api importer
  3. pnpm install --frozen-lockfile fails in Docker context without workspace resolution

The admin 500 root cause is NOT the Dockerfile build path — both root src/ and apps/api/src/ are affected. The actual cause needs separate investigation (likely in enum alignment commit 9462915 or duplicate removal in 4086b6f).

What this reverts to

Root-based Dockerfile: builds packages/types, packages/db, then root pnpm build from src/. This is the working CI pipeline configuration.

cc @cpfarhood

## Summary Reverts PR #47 which switched the Dockerfile to build from `apps/api/src/`. This broke CI Docker builds (runs #968, #970) because: 1. `pnpm-workspace.yaml` only includes `packages/*`, not `apps/*` 2. `pnpm-lock.yaml` has no `apps/api` importer 3. `pnpm install --frozen-lockfile` fails in Docker context without workspace resolution The admin 500 root cause is NOT the Dockerfile build path — both root `src/` and `apps/api/src/` are affected. The actual cause needs separate investigation (likely in enum alignment commit `9462915` or duplicate removal in `4086b6f`). ## What this reverts to Root-based Dockerfile: builds `packages/types`, `packages/db`, then root `pnpm build` from `src/`. This is the working CI pipeline configuration. cc @cpfarhood
The Dogfather added 1 commit 2026-05-22 13:31:05 +00:00
revert: undo PR #47 Dockerfile apps/api switch (broke CI Docker build)
CI / Test (pull_request) Successful in 8s
Details
CI / Lint & Typecheck (pull_request) Successful in 9s
Details
CI / Build & Push Docker Images (pull_request) Successful in 47s
Details
8bdab69288 
Reverts the Dockerfile to the root src/ build. PR #47 switched to apps/api/
but the pnpm workspace config and lockfile don't include apps/*, causing
Docker build failures (CI runs #968, #970).

The actual admin 500 root cause needs further investigation — it's in the
code/schema changes, not the Dockerfile build path.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
The Dogfather merged commit f55c74983f into dev 2026-05-22 13:31:12 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 question
Further information is requested
 wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ai-review (AI Review) gb_barkley (Barkley Trimsworth) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) gb_flea (Flea Flicker) flux (Flux CD) admin (Gitea Admin) gb_lint (Lint Roller) renovate (Mend Renovate) gb_pawla (Pawla Abdul) gb_scrubs (Scrubs McBarkley) gb_shedward (Shedward Scissorhands) gb_dogfather (The Dogfather)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: groombook/api#50
Delete Branch "revert/gro-1533-dockerfile-fix"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?