chore: trigger CI from uat for GRO-1754 #81

Merged

Flea Flicker merged 38 commits from fix/gro-1754-uat-ci into main 2026-05-25 23:23:15 +00:00

Conversation 0 Commits 38 Files Changed 16

+295 -12

Flea Flicker commented 2026-05-25 23:22:11 +00:00

Member

Copy Link

Copy Source

Trigger CI on main to produce dated image tag for GRO-1754 UAT bump

Trigger CI on main to produce dated image tag for GRO-1754 UAT bump

Flea Flicker added 38 commits 2026-05-25 23:22:12 +00:00

fix(GRO-1441): remove duplicate coatType/petSizeCategory from buildPet ... 9f2809e89b

Lines 108-109 were duplicates of lines 102-103 from the PR #12 merge.
Removing the duplicate pair resolves the TS1117 error on dev.

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

fix(GRO-1544): register health endpoint at /api/health not /health ... 55894c6ff2

The health check was registered on `app` at `/health`, but the HTTPRoute
routes `/api/*` to the API pod. Since auth middleware protects the /api
basePath, GET /api/health fell through to authMiddleware → 401.

Now registered on `api` before auth middleware at /api/health.

Updated UAT_PLAYBOOK.md §GRO-1485 — new health endpoint path.

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

docs(api): update UAT_PLAYBOOK.md §4.0 — new health endpoint path ... 7b2b533c16

Added TC-API-0.1 for GET /api/health (unauthenticated).
Corrected path from /health to /api/health (GRO-1544).

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

fix(GRO-1533): add missing default_buffer_minutes migration ... 3609087980

Adds 0033_add_services_default_buffer_minutes.sql with idempotent
ALTER TABLE to ensure services.default_buffer_minutes exists.

Also fixes _journal.json by adding missing 0031_buffer_rules entry (idx 31)
and 0032_staff_read_at entry (idx 32) that were absent from the journal.

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

fix(GRO-1533): fix snapshot id in 0033_snapshot.json ... ce0739b3ba

Fixes id from "0026_stripe_payment" to "0033_add_services_default_buffer_minutes".

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

Merge pull request 'fix(GRO-1533): add missing default_buffer_minutes migration' (#53) from fix/gro-1533-missing-migration-journal into dev ... f9c679b392

Merge fix/gro-1533-missing-migration-journal into dev: add missing default_buffer_minutes migration (GRO-1533)

Merge pull request 'chore: promote dev → uat (GRO-1533 migration fix)' (#54) from dev into uat ... 122d32d635

Promote dev → uat: GRO-1533 migration fix (PR #53)

fix(GRO-1533): add missing 0032_staff_read_at.sql migration file ... 002e6575ba

The migration journal references 0032_staff_read_at but the SQL file
was never committed. drizzle-kit migrate fails with "No file
./migrations/0032_staff_read_at.sql found" which blocks all subsequent
migrations including the 0033 default_buffer_minutes fix.

Added as a no-op since the staff table schema has no readAt column.

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

Merge pull request 'fix(GRO-1533): add missing 0032_staff_read_at.sql migration file' (#55) from fix/gro-1533-missing-migration-0032 into dev ... 9fe6e15012

fix(GRO-1533): add missing 0032_staff_read_at.sql migration file

Merged by CTO after QA approval (Review #3513).
Unblocks UAT migration pipeline.

Merge pull request 'chore: promote dev → uat (GRO-1533 migration fix)' (#56) from dev into uat ... 190c39f905

chore: promote dev → uat (GRO-1533 migration fix)

Promotes 0032_staff_read_at.sql migration file to uat branch.
Unblocks UAT migration pipeline.

fix(GRO-1533): add missing coat_type/pet_size_category columns in migration 0031 ... 174d1c667b

Migration 0031 tries to ALTER the coat_type and pet_size_category columns
on the pets table to use new enum types, but no prior migration adds
these columns. On a fresh DB (after the reset CronJob wiped all tables),
this causes the entire migration chain to fail and roll back.

Added ADD COLUMN IF NOT EXISTS before the ALTER TYPE so the migration
works both on fresh databases and existing ones with the columns.

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

Merge pull request 'fix(GRO-1533): fix migration 0031 for empty databases' (#57) from fix/gro-1533-migration-0031-coat-type into dev ... 68df697cf3

fix(GRO-1533): fix migration 0031 for empty databases (#57)

Adds ADD COLUMN IF NOT EXISTS for coat_type and pet_size_category before ALTER TYPE casts, making migration safe for both fresh and existing databases.

Reviewed-by: gb_lint (QA)
Approved-by: CTO

Merge pull request 'promote: dev → uat (migration 0031 fix, GRO-1533)' (#58) from dev into uat ... d847343090

promote: dev → uat — migration 0031 fix (GRO-1533) (#58)

Merge pull request 'fix(GRO-1544): register health endpoint at /api/health not /health' (#52) from fix/gro-1544-api-health-endpoint into dev ... 62dfc7776b

fix(GRO-1544): register health endpoint at /api/health not /health

Merge PR #52 to dev. Unblocks GRO-1485 UAT regression chain.

Merge pull request 'chore: promote dev → uat (GRO-1544 health endpoint fix)' (#59) from dev into uat ... f4561b539f

chore: promote dev → uat (GRO-1544 health endpoint fix)

Merge dev → uat. CI auto-deploys to UAT environment.

fix(GRO-1544): restore /health alongside /api/health endpoint ... 49f70eb74b

The previous GRO-1544 PR changed /health to /api/health but removed
the /health endpoint entirely. This breaks:
- Dockerfile HEALTHCHECK (curl -f http://localhost:3000/health)
- K8s readinessProbe/livenessProbe (httpGet: path: /health, port: 3000)

Both paths are registered before auth middleware so both remain
publicly accessible without authentication.

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

fix(GRO-1566): bypass auth for /api/health endpoint on UAT ... 3b9c72c2c4

The /api/health endpoint returns 401 on UAT because authMiddleware
was not skipping it — the health check was registered on the Hono app
instance (not the api sub-router), placing it below authMiddleware on
the base app. The fix adds /api/health to the auth skip list alongside
/api/auth/.

The /health endpoint (registered at app level, above all middleware)
correctly returns 200. The /api/health endpoint must also be public
since the task requires confirming it returns 200.

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

Merge pull request 'fix(GRO-1566): bypass auth for /api/health endpoint on UAT' (#61) from fix/gro-1566-api-health-auth-bypass into dev ea28095434

Merge pull request 'Promote dev to UAT: GRO-1566 auth bypass fix' (#62) from dev into uat 00c6a36021

fix: add network=host to buildx driver-opts for DinD DNS resolution ... 4e8c66f3ca

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

fix(GRO-1576): add provenance: false to all build-push-action steps ... e26d960046

Docker Buildx v6 defaults to OCI attestation manifests (--attest
type=provenance,mode=max). These hit a Gitea registry bug when image
layers are pre-existing (blob mount), causing "unknown" errors on manifest
list push. API image succeeds because it pushes new layers; migrate/seed/
reset fail because their layers already exist.

Disabling provenance attestation on all four build-push-action steps
resolves the push failures. Addresses GRO-1575.

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

Merge pull request 'fix(GRO-1576): add provenance: false to all build-push-action steps' (#64) from fix/gro-1576-ci-provenance-false into dev ... 33aa63b10f

fix(GRO-1576): add provenance: false to all build-push-action steps (#64)

Disables OCI attestation manifest generation that was hitting a Gitea registry bug when image layers are pre-existing.

Reviewed-by: Lint Roller (QA)
Approved-by: The Dogfather (CTO)

Merge pull request 'Promote dev → uat: provenance: false CI fix' (#65) from dev into uat ... 2380698128

Promote dev → uat: provenance: false CI fix (#65)

Includes fix(GRO-1576): add provenance: false to all build-push-action steps.

Approved-by: The Dogfather (CTO)

Merge pull request 'fix(GRO-1544): restore /health alongside /api/health endpoint' (#60) from fix/gro-1544-api-health-endpoint into dev ... 06d72b5baf

fix(GRO-1544): restore /health alongside /api/health endpoint (#60)

Merge pull request 'fix(GRO-1441): remove duplicate coatType/petSizeCategory from buildPet' (#35) from fix/gro-1441-remove-duplicate-coat-props into dev ... b5a08a2c7e

fix(GRO-1441): remove duplicate coatType/petSizeCategory from buildPet (#35)

fix(api): add timeouts for OIDC discovery fetch and DB connection (#66) b486c44a82

Merge pull request 'chore: promote dev → uat (GRO-1678 TCP resilience + backlog fixes)' (#67) from dev into uat ... 8d9a9d8dba

chore: promote dev → uat (GRO-1678 TCP resilience + backlog fixes)

feat(gro-1743): add UAT customer and pets to admin seed endpoint ... a1466b44c9

Add UAT Customer (uat-customer@groombook.dev) with two pets (Bella and Max)
to the idempotent admin seed endpoint for portal UAT testing.

- Client: UAT Customer, email: uat-customer@groombook.dev, phone: 555-0100, status: active
- Pet 1: Bella, Dog, Poodle, coatType: curly
- Pet 2: Max, Dog, Labrador Retriever, coatType: short

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

Merge pull request 'feat(gro-1743): add UAT customer and pets to admin seed endpoint' (#69) from fix/gro-1743-uat-seed-data into dev ... 040ff4a253

Merge PR #69: feat(gro-1743): add UAT customer and pets to admin seed endpoint

Merge pull request 'chore: promote dev → uat (GRO-1743 seed data)' (#70) from dev into uat ... baeff6c4f5

Merge PR #70: chore: promote dev → uat (GRO-1743 seed data)

fix(gro-1749): sync UAT seed data to root src and fix route path (#71) ... a03771f7e7

Co-authored-by: Flea Flicker <flea@groombook.dev>
Co-committed-by: Flea Flicker <flea@groombook.dev>

Merge pull request 'chore: promote dev → uat (GRO-1749 seed data sync)' (#72) from promo/gro-1749-uat into uat e5f16a5fe5

fix(schema): add missing extended pet profile fields to packages/db (#73) ad1b210de1

fix(db): add missing extended pet profile fields to buildPet factory ... 74da042d13

Lint Roller (QA) flagged that buildPet in factories.ts was missing the
4 fields added to the pets table schema, causing TS2739 in the Docker
build job (run 1701, job 3717).

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

trigger CI again cc09a8e1e8

Merge pull request 'fix(db): add missing extended pet profile fields to buildPet factory' (#78) from fix/gro-1752-factories-v2 into dev 0a2259b67f

Merge pull request 'promote(dev→uat): add missing extended pet profile fields (GRO-1752)' (#79) from dev into uat ... aa9670d4dc

promote(dev→uat): add missing extended pet profile fields (GRO-1752)

chore: trigger CI from uat for GRO-1754 dd83f29736

Flea Flicker merged commit ba95088653 into main 2026-05-25 23:23:15 +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#81