promote: dev → uat (pnpm-lock.yaml fix + CI/enum fixes + seed Docker fix) #48

Merged

The Dogfather merged 15 commits from dev into uat 2026-05-22 13:18:13 +00:00

Conversation 0 Commits 15 Files Changed 12

+84 -71

The Dogfather commented 2026-05-22 13:14:03 +00:00

Member

Copy Link

Copy Source

dev → uat promotion

Promotes commits from dev to uat, including:

• fix(seed): use --filter @groombook/db for seed/migrate/reset scripts (PR #45, GRO-1531)
• fix(deps): update pnpm-lock.yaml for better-auth ^1.5.6 (GRO-1538)
• fix(ci): Gitea CI workflow with all 4 image targets + Node 22 (GRO-1522)
• fix: align root src/ enum values with packages/db schema
• fix: remove duplicate bufferRules table and duplicate properties
• fix: use root pnpm build for Dockerfile

cc @cpfarhood

## dev → uat promotion Promotes commits from dev to uat, including: - **fix(seed):** use --filter @groombook/db for seed/migrate/reset scripts (PR #45, GRO-1531) - **fix(deps):** update pnpm-lock.yaml for better-auth ^1.5.6 (GRO-1538) - **fix(ci):** Gitea CI workflow with all 4 image targets + Node 22 (GRO-1522) - **fix:** align root src/ enum values with packages/db schema - **fix:** remove duplicate bufferRules table and duplicate properties - **fix:** use root pnpm build for Dockerfile cc @cpfarhood

The Dogfather added 13 commits 2026-05-22 13:14:03 +00:00

fix(deps): update pnpm-lock.yaml for better-auth ^1.5.6 ... 0cab1522cf

ERR_PNPM_OUTDATED_LOCKFILE — specifiers in lockfile did not match
packages/db/package.json after better-auth upgrade.

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

Merge branch 'dev' of https://git.farh.net/groombook/api into dev c9e176f08c

fix(ci): build all service images + upgrade Node 22 + pin packageManager (GRO-1522) ... 2b78fcf731

- Upgrade CI jobs (lint-typecheck, test, build) to Node 22
- Dockerfile uses node:22-alpine for base and runner stages
- Root package.json gets packageManager field for corepack pin
- Docker build already targets all 4 stages (api/migrate/seed/reset)

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

fix: resolve pre-existing TypeScript errors for CI compliance ... 59893908e2

- petsExtendedFields.test.ts: import and/eq/exists/or from db/index.js
  (avoids mock scope collision with TypeScript closures)
- petsExtendedFields.test.ts: add non-null assertion on petRows[0]
  in makeDeleteChainable (petRows always has at least one element)
- factories.ts buildPet: add missing extended pet fields to defaults
  (coatType, temperamentScore, temperamentFlags, medicalAlerts,
  preferredCuts) so the inferred PetRow type is satisfied

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

fix: resolve pre-existing test and TypeScript errors for CI compliance ... ce9fcfb362

- Fix CLIENT_ID/PET_ID in petsExtendedFields.test.ts to valid UUIDs so
  createPetSchema validation (z.string().uuid()) passes in tests
- Replace top-level imports of and/eq/exists/or with vi.fn() stubs in
  petsExtendedFields.test.ts mock to avoid vi.mock hoisting ReferenceError
- Add impersonationAuditLogs proxy + insert() chain to portal.test.ts mock
  to fix audit-log write failures
- Add 5 missing extended fields to buildPet factory defaults
- Add non-null assertion on petRows[0] in makeDeleteChainable

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

fix(ci): add Gitea CI workflow with all 4 image targets + Node 22 (GRO-1522) ... da913d600f

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

merge: resolve conflicts with dev (keep Node 22 + add Gitea CI images) ... 1ea319e122

- Dockerfile: keep node:22-alpine for both base and runner stages
- package.json: keep dev's full content + add packageManager field
- .gitea/workflows/ci.yml: keep fixed version with all 4 image targets
- petsExtendedFields.test.ts: keep dev UUIDs + PR's vi.fn() mocks

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

Merge pull request 'fix(ci): build all service images + upgrade Node 22 + pin packageManager (GRO-1522)' (#44) from fix/gro-1522-ci-images-node22 into dev ... 3fddf80fac

fix(ci): build all service images + upgrade Node 22 + pin packageManager (GRO-1522)

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

fix(db): remove duplicate petSizeCategoryEnum/coatTypeEnum declarations (GRO-1522) ... 3bec5d095a

The schema file had two sets of these enum declarations with different values.
The first (stale) set broke all tests importing @groombook/db via the vitest alias,
causing CI to fail and blocking the docker build job.

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

fix(ci): use monorepo-filtered commands in Gitea CI (GRO-1522) ... d7d98791c7

The root pnpm scripts typecheck/lint/test the stale src/ directory.
Use pnpm --filter @groombook/api to target the correct apps/api/ path,
matching the GitHub Actions CI configuration.

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

fix: remove duplicate bufferRules table and duplicate properties blocking Docker build ... 4086b6f5c0

CI Run 942 Docker build fails on TS2451 (duplicate bufferRules at lines
190 & 653 of schema.ts), TS1117 (duplicate defaultBufferMinutes in
services table, duplicate coatType/petSizeCategory in factories.ts),
and TS2322 (null vs number for defaultBufferMinutes in factories.ts).

Keep the newer, more complete bufferRules declaration (with comments and
index) and the .notNull().default(0) variant of defaultBufferMinutes.

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

fix: use root pnpm build instead of --filter for api in Dockerfile ... 46f134a294

pnpm --filter @groombook/api doesn't match the workspace root package
because pnpm-workspace.yaml only includes packages/*. The root
package.json already has a build script that compiles src/ → dist/,
which is what the Dockerfile copies.

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

fix: align root src/ enum values with packages/db schema ... 9462915a66

Root src/routes/ used stale enum values (xlarge→extra_large,
smooth→silky, missing short/medium/silky from coatType) and
sizeCategory→petSizeCategory field name mismatch with the
pets table column.

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

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

fix(seed): use --filter @groombook/db to invoke seed/migrate/reset scripts ... 40422a14f0

The seed/migrate/reset Dockerfile targets specified `pnpm db:migrate`
etc., but those scripts are defined in packages/db/package.json, not
at the workspace root. pnpm workspace filtering is required to route
the command to the correct package.

- migrate: pnpm --filter @groombook/db migrate
- seed:    pnpm --filter @groombook/db seed
- reset:   pnpm --filter @groombook/db reset

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

Merge pull request 'fix(seed): use --filter @groombook/db for seed/migrate/reset scripts' (#45) from flea-flicker/gro-1531-seed-db-filter into dev ... 70bc946a0d

fix(seed): use --filter @groombook/db for seed/migrate/reset scripts

Merge PR #45 to dev. QA approved (review #3506), CTO reviewed.

The Dogfather changed title from promote: dev → uat (pnpm-lock.yaml fix + CI/enum fixes) to promote: dev → uat (pnpm-lock.yaml fix + CI/enum fixes + seed Docker fix) 2026-05-22 13:18:07 +00:00

The Dogfather merged commit 974dade8f7 into uat 2026-05-22 13:18:13 +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#48