promote: dev → uat (GRO-1395 drizzle-orm root dep fix) #31

Merged

The Dogfather merged 23 commits from dev into uat 2026-05-21 04:11:30 +00:00

Conversation 0 Commits 23 Files Changed 115

+27434 -338

The Dogfather commented 2026-05-21 04:11:19 +00:00

Member

Copy Link

Copy Source

Promote dev → uat

Includes PR #29 (fix/gro-1395-drizzle-orm-root-dep → dev): adds drizzle-orm and postgres to root package.json so CI test runs resolve drizzle-orm/postgres-js.

QA

• GRO-1405: QA approved (Lint Roller, review #3402)
• CI run 622: Lint & Typecheck ✅, 505/505 tests pass
• No user-facing behaviour change

## Promote dev → uat Includes PR #29 (`fix/gro-1395-drizzle-orm-root-dep` → `dev`): adds `drizzle-orm` and `postgres` to root `package.json` so CI test runs resolve `drizzle-orm/postgres-js`. ### QA - [GRO-1405](/GRO/issues/GRO-1405): QA approved (Lint Roller, review #3402) - CI run 622: Lint & Typecheck ✅, 505/505 tests pass - No user-facing behaviour change

The Dogfather added 23 commits 2026-05-21 04:11:20 +00:00

Initial commit 4d7baec939

Extract groombook/api from monorepo with CI workflow ... abac9dfe6c

- Add source code from apps/api
- Add packages/db and packages/types workspace dependencies
- Add GitHub Actions CI workflow (lint, typecheck, test, docker)
- Generate pnpm-lock.yaml
- Add .gitignore

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

fix: correct vitest alias paths for workspace packages ... 9ed28f8bab

- Fix @groombook/db and @groombook/db/factories alias paths
- Change from ../../packages to ./packages (workspace packages are at root)

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

fix: update Dockerfile for standalone repo structure ... d9ee14b17e

- Change apps/api/ to src/ (api package is now at root)
- Update COPY paths for new structure
- Change CMD from apps/api/dist/index.js to dist/index.js
- Remove api package.json copy (now at root)

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

chore: migrate CI workflow to .gitea/workflows d8f2135506

chore: remove legacy .github/workflows 4f981bbebd

fix(ci): use REGISTRY_TOKEN for Docker push auth ... 539ef21d89

Use the org-level REGISTRY_TOKEN secret instead of gitea.token for
authenticating to the Gitea Container Registry. The gitea.token
does not have packages:write scope.

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

feat(GRO-1174): persist petSizeCategory and petCoatType from booking ... faf7def77d

- Add petSizeCategory and petCoatType to bookingSchema zod validator (optional)
- Save coatType to pets row on booking creation
- Add coatType and petSizeCategory columns to pets DB schema
- Add coatType and petSizeCategory to Pet interface in @groombook/types

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

feat(GRO-1174): add MedicalAlert/CoatType/AlertSeverity types to @groombook/types ... f1258023ac

Sync api packages/types with web workspace — add MedicalAlert, AlertSeverity,
CoatType, preferredCuts, medicalAlerts, temperamentScore, temperamentFlags.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Save petSizeCategory to pet record on booking creation ... 9c5e470737

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

fix(GRO-1350): use explicit tsconfig path in packages/types build ... 1403517067

tsc without --project flag fails to find tsconfig.json when run from
a nested package directory inside a Docker COPY layer that overlays
files after deps install. Use explicit --project . to ensure tsc
finds the local tsconfig.json regardless of working directory context.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

fix(GRO-1350): add missing coatType and petSizeCategory to buildPet defaults ... d9bfed4424

PetRow (pets.$inferSelect) now includes these nullable columns after
the GRO-1174 migration, but buildPet's defaults were never updated.
Adding null defaults fixes the typecheck failure in CI.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix(docker): use explicit tsconfig in api build command ... 43f17dc612

tsc without --project flag picks up tsconfig.json from the workspace
root, which lacks the packages/* paths needed for the monorepo build.
Explicit --project . ensures tsc uses the local tsconfig.json.

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

fix(docker): use explicit tsconfig in db package build ... 01069f8c6c

tsc without --project traverses up to workspace root, which has a
different tsconfig.json that lacks package-local paths. Fix both
@groombook/types and @groombook/db scripts consistently.

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

fix(docker): cd into packages/db before building ... a205fe1138

pnpm --filter runs in the workspace root where tsc finds the root
tsconfig.json instead of packages/db/tsconfig.json. Change into the
package directory so tsc picks up the correct local tsconfig.

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

fix(docker): use -p flag for explicit tsconfig path ... c3c99ad6c4

Both -p . and --project . should be equivalent, but the Docker build
appears to resolve them differently. Use -p for consistency.

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

fix(docker): use absolute tsconfig path for api build ... fc82e24ead

When pnpm --filter runs the api package build, tsc cannot find the
tsconfig.json. Use an absolute path to avoid any ambiguity about the
working directory context.

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

fix(docker): use absolute tsconfig.json path for tsc ... e417d8f6a7

tsc -p /app does not resolve to tsconfig.json at /app/tsconfig.json
without an explicit filename. Pass the full path.

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

fix(docker): use pnpm --filter for all monorepo package builds ... 467b85abc7

Use pnpm --filter consistently for all three package builds in the
Dockerfile instead of mixing filter and cd approaches. Also set
--project . explicitly on tsc invocations to ensure tsconfig resolution
from the package directory rather than workspace root.

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

Merge dev into gitea/migrate-workflows (allow-unrelated-histories) ... 90b3811577

Merges the dev branch history into gitea/migrate-workflows to resolve
PR #24. The two branches had unrelated git histories due to the Gitea
migration. Conflict resolution favors gitea/migrate-workflows for
packages/, src/, .gitea/ structure and dev for apps/, .github/ content.

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

Merge pull request 'fix(ci): use REGISTRY_TOKEN for Docker push auth' (#24) from gitea/migrate-workflows into dev ... f36a3626a8

Merge gitea/migrate-workflows into dev: fix(ci) REGISTRY_TOKEN, Dockerfile, schema updates

fix(GRO-1395): add drizzle-orm and postgres to root package.json ... 4204bea2b3

Add drizzle-orm ^0.38.4 and postgres ^3.4.5 to root package.json
dependencies so that pnpm --frozen-lockfile install in CI makes
them available at the root node_modules level.

apps/api is not a pnpm workspace member (workspace declares
packages: ["apps/*"]), so CI's pnpm install does not reach into
apps/api/node_modules/. Adding these deps to the root package.json
fixes the ERR_MODULE_NOT_FOUND error that Vitest encountered when
running tests under pnpm --frozen-lockfile.

Also moves drizzle-orm and postgres from apps/api/devDependencies
to dependencies per the issue spec.

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

Merge pull request 'fix(GRO-1395): add drizzle-orm and postgres to root package.json' (#29) from fix/gro-1395-drizzle-orm-root-dep into dev ... 51b45b529d

fix(GRO-1395): add drizzle-orm and postgres to root package.json (#29)

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

The Dogfather merged commit 5c9cac7a28 into uat 2026-05-21 04:11:30 +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

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: groombook/api#31