Skip to content

docs(spec): add elephant-goldfish hand-off to allstack-spec - #12

Merged
decebal merged 1 commit into
mainfrom
docs/spec-egm-flow
Nov 9, 2026
Merged

decebal merged 1 commit into
mainfrom
docs/spec-egm-flow

Conversation

@decebal

@decebal decebal commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • allstack-spec gains an Elephant-Goldfish section: optional deeper review, started by a person because all seven EGM skills set disable-model-invocation: true.
  • EGM writes no receipt and egm-resolve edits the design, so allstack spec review still runs on the final bytes before spec approve. No CLI change.
  • EGM reports stay unstaged, and the skill's report line records whether EGM ran.
  • Evaluation report and glossary corrected: plan-ceo-review and autoplan are already installed for both hosts, so no port is planned, and EGM ships as the upstream plugin. The not-checked list states what was read.

Test plan

  • egm@egm 0.1.1 enabled in Claude Code and Codex (plugin list on each)
  • /egm:egm-review started in a headless Claude Code session and stopped at its prompt for a target
  • CI tests (Markdown only, not compiled locally: 5.4 GiB free and no build directory to seed from)
  • One real design through EGM, then spec review, approve, check
  • An EGM skill run in Codex

🤖 Generated with Claude Code

EGM is installed as an upstream plugin, but allstack-spec did not say how
it fits the approval gate. EGM writes reports without a receipt and
egm-resolve edits the design, so an EGM review cannot satisfy spec approve.

allstack-spec now names EGM as an optional deeper review that a person
starts, because all seven EGM skills set disable-model-invocation. It
requires spec review on the final bytes and keeps EGM reports unstaged.

The evaluation report and glossary are corrected: gstack plan-ceo-review
and autoplan are installed for both hosts, so no port is planned, and EGM
ships as the upstream plugin.

Markdown only. Not compiled locally; tests run in CI.
@decebal
decebal marked this pull request as ready for review November 9, 2026 01:50
@decebal
decebal merged commit 4e3c73a into main Nov 9, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant