gstack's /ship stops passing reviews that found bugs, and /cso runs on repos with submodules

1 hour ago

gstack for Claude Code: the /ship gate stops reading an outside review with findings as clean, a clean Codex review now passes, and /cso runs on repos with submodules, nested Git repos and long patch histories. Plus a saved "never ask" that sticks in /plan-eng-review, /context-restore checkpoint fixes and a first-deploy how-to. Covers gstack 1.91.151.0 to 1.91.153.0. Release notes: https://github.com/garrytan/gstack/blob/main/CHANGELOG.md 1.91.153.0: https://github.com/garrytan/gstack/blob/main/CHANGELOG.md#1911530---2026-10-19 1.91.152.0: https://github.com/garrytan/gstack/blob/main/CHANGELOG.md#1911520---2026-10-19 1.91.151.0: https://github.com/garrytan/gstack/blob/main/CHANGELOG.md#1911510---2026-10-19 Commits: https://github.com/garrytan/gstack/commits/main

Ask

Ask about this presentation

Answers are generated from this presentation.

Chapters

  1. 0:00False pass
  2. 0:15Codex
  3. 0:29Last word
  4. 0:45Nested repos
  5. 1:02Long history
  6. 1:16Fixes
  7. 1:32Your turn
Show transcript

False pass

gstack logo

/ship stops passing an outside review that found a bug

“The parser change has no new issues. However, … drops users_email_idx”
gate readscleanunverified
False passCodexLast wordNested reposLong historyFixesYour turn
garrytan/gstack · CHANGELOG.md · 1.91.153.0

gstack's /ship gate read an outside review as clean whenever no issues appeared anywhere in it, so a reply that flagged a dropped database index passed. Now that reply reads unverified.

Codex

A clean Codex review now reads clean

“I found no actionable regressions in the staged, unstaged, or untracked changes.”
gate readsunverifiedclean
False passCodexLast wordNested reposLong historyFixesYour turn
garrytan/gstack · CHANGELOG.md · 1.91.153.0

The fix works the other way too. A clean Codex review that says it found no actionable regressions used to read unverified. Regressions and defects now count as no-findings words, so that reply passes.

Last word

“No issues” only counts as the review’s last word

Counts: it opens the final sentence, with no “but”, “however” or “except”
Still reads clean: untagged findings, then “No issues found in the changed parser.”
False passCodexLast wordNested reposLong historyFixesYour turn
garrytan/gstack · CHANGELOG.md · 1.91.153.0

The phrase now has to open the review's final sentence, with no but or however attached. Untagged findings followed by no issues found in the changed parser still read clean, since without tags a finding looks like a summary.

Nested repos

/cso now starts on repos with a submodule or a nested Git repo

$ gstack-cso start
UNSAFE_PATH
inner/ excluded: nested Git repository
False passCodexLast wordNested reposLong historyFixesYour turn
garrytan/gstack · CHANGELOG.md · 1.91.152.0

/cso, the security audit, stopped with an unsafe path error on any repo holding a submodule or a nested Git repository. Now it lists that folder as excluded and never reads it. A symlink to a folder is still refused.

Long history

/cso finishes the history check on a long patch history

Before: patches over 1 MiB left history-inputs not assessed
Now: the newest whole commits that fit, never cut mid-commit
History coverage: partial, with the gap named
False passCodexLast wordNested reposLong historyFixesYour turn
garrytan/gstack · CHANGELOG.md · 1.91.152.0

A long patch history used to leave its history check unassessed. Now /cso keeps the newest whole commits that fit, never cutting inside one, and marks history coverage as partial.

Fixes

A saved “never ask” now sticks, plus two more fixes

/plan-eng-review: the structure question honors your saved preference
/context-restore: asks when recent checkpoints carry different titles
First deploy: a new how-to for when /setup-deploy finds nothing deployed
False passCodexLast wordNested reposLong historyFixesYour turn
garrytan/gstack · CHANGELOG.md · 1.91.151.0

/plan-eng-review now remembers a never ask you saved for its structure question. /context-restore asks you to choose when recent checkpoints carry different titles. A new guide covers a first deploy when /setup-deploy finds nothing deployed.

Your turn

The full notes are in gstack’s changelog

CHANGELOG.mdgstack release notes, October 19
Which agent runs your outside review before /ship?
False passCodexLast wordNested reposLong historyFixesYour turn
garrytan/gstack · CHANGELOG.md · 1.91.151.0 – 1.91.153.0

The test changes behind these fixes are listed in gstack's changelog. Which agent runs your outside review before you /ship?