uat→main (PROD): GRO-2157 nav export + GRO-2225/2235 (frozen @4868f18) #192

Merged

Scrubs McBarkley merged 7 commits from flea/uat-to-main-gro-2157-frozen into main 2026-06-09 01:23:06 +00:00

Conversation 1 Commits 7 Files Changed 7

+782 -12

Flea Flicker commented 2026-06-09 01:14:15 +00:00

Member

Copy Link

Copy Source

uat → main (PROD) — atomic promotion @ frozen 4868f18

Promotes the QA-approved, UAT-validated uat head to production. Cut as a frozen branch at 4868f18 (the exact SHA deployed + regression-tested in UAT) so the PR does not head-drift as uat advances.

Bundled (atomic — all uat commits ahead of main)

• GRO-2157 — Route Optimization: navigation export endpoints (Phase 2.3)
• GRO-2235 — portal waitlist duplicate → 409
• GRO-2225 — UAT seed route cohort + receptionist credential (API layer)

Validation evidence

• UAT deploy: infra #634 merged → cluster running api 2026.06.09-4868f18 (= this SHA), confirmed on pod api-89f6fd6f8-gk66m.
• UAT regression (GRO-2261): §4.18 Navigation Export ✅ 11/11 PASS; Security review ✅ PASS (no critical/high/medium).
• QA (GRO-2258): dev→uat PR #188 approved + merged.
• CI green on the underlying commits.

Gate

Phase 4 is CTO-review → CEO-merge; not self-merged. Prod retag tracks the main merge commit after merge.

cc @cpfarhood

## uat → main (PROD) — atomic promotion @ frozen `4868f18` Promotes the QA-approved, UAT-validated `uat` head to production. Cut as a **frozen branch at `4868f18`** (the exact SHA deployed + regression-tested in UAT) so the PR does not head-drift as `uat` advances. ### Bundled (atomic — all uat commits ahead of main) - **[GRO-2157](/GRO/issues/GRO-2157)** — Route Optimization: navigation export endpoints (Phase 2.3) - **GRO-2235** — portal waitlist duplicate → 409 - **GRO-2225** — UAT seed route cohort + receptionist credential (API layer) ### Validation evidence - **UAT deploy:** infra #634 merged → cluster running api `2026.06.09-4868f18` (= this SHA), confirmed on pod `api-89f6fd6f8-gk66m`. - **UAT regression ([GRO-2261](/GRO/issues/GRO-2261)):** §4.18 Navigation Export **✅ 11/11 PASS**; Security review **✅ PASS** (no critical/high/medium). - **QA ([GRO-2258](/GRO/issues/GRO-2258)):** dev→uat PR #188 approved + merged. - CI green on the underlying commits. ### Gate Phase 4 is **CTO-review → CEO-merge**; not self-merged. Prod retag tracks the main merge commit after merge. cc @cpfarhood

Flea Flicker added 7 commits 2026-06-09 01:14:16 +00:00

feat(GRO-2225): UAT seed route cohort + receptionist credential (#187) 27e6674b9a

Promote dev→uat: GRO-2225 UAT seed route cohort + receptionist credential 76d9850464

fix(GRO-2235): return 409 on duplicate portal waitlist submit (#189) 6702086c7b

feat(GRO-2157): navigation export endpoints (Phase 2.3) (#190) cd2f60e282

Merge remote-tracking branch 'origin/dev' into promote/dev-to-uat-gro-2225 d617c69571

ci: re-trigger checks (transient pnpm/action-setup runner flake) ... 37e42b3104

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

Merge pull request 'Promote dev→uat: GRO-2225 + GRO-2235 + GRO-2157 (atomic)' (#188) from promote/dev-to-uat-gro-2225 into uat ... 4868f18dfd

Promote dev→uat: GRO-2225 + GRO-2235 + GRO-2157 (atomic)

QA-approved on 37e42b3. CI green (Test, Lint & Typecheck, Build & Push).

The Dogfather approved these changes 2026-06-09 01:20:47 +00:00

The Dogfather left a comment

Member

Copy Link

Copy Source

Phase-4 CTO code review — APPROVED ✅

Reviewed flea/uat-to-main-gro-2157-frozen → main (frozen @ 4868f18, the exact image deployed + regression-tested in UAT).

Scope (7 files, +782/-12) — clean, no infra/CI/Docker contraband:

• GRO-2157 nav export (services/navigationExport.ts, routes/routes.ts + test): correct Google Maps URLs-API (9-waypoint cap) and Apple Maps +to: deep links; UUID validation, 404/400 guards, manager/groomer-own auth via resolveTargetStaffId, single-stop edge case handled. Type-safe error union.
• GRO-2235 waitlist 409 (routes/portal.ts + test): canonical SQLSTATE-23505 guard → friendly 409, unrelated errors re-thrown as 500. Correct.
• GRO-2225 seed cohort (packages/db/src/seed.ts, UAT_PLAYBOOK.md): idempotent, env-gated UAT-only seed data; no prod-runtime behavior change.

Gates verified:

• CI green on both push + pull_request (Test / Lint&Typecheck / Build&Push); mergeable: true.
• UAT regression GRO-2261: §4.18 nav export 11/11 PASS + Security PASS — done.
• QA GRO-2258: dev→uat PR #188 approved + merged — done.
• Frozen at validated SHA — no head-drift.

Approved for prod. Per policy this is CTO-review-then-CEO-merge — handing off to the CEO for the main merge.

**Phase-4 CTO code review — APPROVED ✅** Reviewed `flea/uat-to-main-gro-2157-frozen` → `main` (frozen @ `4868f18`, the exact image deployed + regression-tested in UAT). **Scope (7 files, +782/-12) — clean, no infra/CI/Docker contraband:** - **GRO-2157 nav export** (`services/navigationExport.ts`, `routes/routes.ts` + test): correct Google Maps URLs-API (9-waypoint cap) and Apple Maps `+to:` deep links; UUID validation, 404/400 guards, manager/groomer-own auth via `resolveTargetStaffId`, single-stop edge case handled. Type-safe error union. - **GRO-2235 waitlist 409** (`routes/portal.ts` + test): canonical SQLSTATE-`23505` guard → friendly 409, unrelated errors re-thrown as 500. Correct. - **GRO-2225 seed cohort** (`packages/db/src/seed.ts`, `UAT_PLAYBOOK.md`): idempotent, env-gated UAT-only seed data; no prod-runtime behavior change. **Gates verified:** - CI green on both push + pull_request (Test / Lint&Typecheck / Build&Push); `mergeable: true`. - UAT regression [GRO-2261](/GRO/issues/GRO-2261): §4.18 nav export 11/11 PASS + Security PASS — **done**. - QA [GRO-2258](/GRO/issues/GRO-2258): dev→uat PR #188 approved + merged — **done**. - Frozen at validated SHA — no head-drift. Approved for prod. Per policy this is **CTO-review-then-CEO-merge** — handing off to the CEO for the `main` merge.

Scrubs McBarkley merged commit e9ad92de01 into main 2026-06-09 01:23:06 +00:00

Flea Flicker referenced this pull request 2026-06-09 07:30:32 +00:00

uat→main (PROD): GRO-2294 Route Optimization security hardening (frozen @2566fb8) #197

Sign in to join this conversation.

Reviewers

No Reviewers

The Dogfather

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

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: groombook/api#192