garrytan / garrytan/gstack

"Land & Deploy report" recap says "Shipped" — terminology is inconsistent and "land" isn't standard

Open
#1,146 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
133k
Forks
19.9k
Avg merge
18h 46m
Merged PRs (30d)
26

Description

In the chat excerpt below, the "LAND & DEPLOY REPORT" ends with a recap that says "Shipped...":

```
● All 4 jobs succeeded (test, build-and-push, deploy-production, smoke-test). The in-CI
smoke-test already hit https://XXXXXX post-deploy per your project's Deploy
Configuration, so the site is verified healthy.

Skipping the gstack canary — this change is to a local dev-sync script that doesn't
run in production, so hitting the site gives zero signal about the fix. The real
verification is running ./dev.sh --sync on your machine; CI + smoke-test already
covered prod health.

LAND & DEPLOY REPORT
════════════════════
PR: #XXX — fix(scripts): skip self-referential CREATE TABLE LIKE
in loadFromRemoteDb
Branch: fix/load-remote-db-self-like → master
Merged: 2026-04-22 (squash, branch deleted)
Merge SHA: XXXXXXX

CI (pre-merge): ✓ test, ✓ GitGuardian
Deploy workflow: ✓ test, ✓ build-and-push, ✓ deploy-production, ✓ smoke-test
Canary: skipped (dev script, zero prod surface)

Reviews addressed:
Copilot (nit): SOURCE_DB constant extracted (XXXXXXX)
Codex (P1): existence check replaces target==source guard (XXXXX)

VERDICT: DEPLOYED AND VERIFIED (via in-CI smoke-test)

The fix is live. To confirm end-to-end, run ./dev.sh --sync locally — it should
finish cleanly without the MySQL 1066 error.

✻ Worked for 17m 7s

※ recap: Shipped a fix for the `./dev.sh --sync` MySQL 1066 error via PR #XXX, which
is now merged and deployed to prod with CI smoke-test green. Next: run `./dev.sh
--sync` locally to confirm it finishes cleanly. (disable recaps in /config)
```

The skill clearly just ran land-and-deploy. Shipping happened earlier. But the recap says "Shipped." When I come back to this terminal hours later and read "shipped," I think I still need to run land-and-deploy — but I already did.

The inverse happens after `/ship`: the recap says "Next: merge it" when it really means "run `/land-and-deploy`."

Summary:
- Run `/ship` → recap says "next: merge" when it means land-and-deploy
- Run `/land-and-deploy` → recap says "shipped" when it means land-and-deploy

My guess: the model's training has very little concept of "land and deploy," but it knows "ship" and "merge," so it biases toward the familiar terms. Tellingly, the LAND & DEPLOY REPORT itself uses "merge" 4 times and never says "land" or "landed."

This is a signal we probably shouldn't use "land and deploy." ChatGPT calls "land" a niche term — used mainly at Google/Meta to mean merging into main, non-standard elsewhere where people say "merge" or "commit."

**Recommendation:** rename to `/merge-and-deploy`. The "land" term is problematic.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.