Skip to content

v1.13.0.0 feat: add Claude outside-voice skill#1212

Merged
garrytan merged 7 commits into
mainfrom
garrytan/claude-codex-skill
Apr 25, 2026
Merged

v1.13.0.0 feat: add Claude outside-voice skill#1212
garrytan merged 7 commits into
mainfrom
garrytan/claude-codex-skill

Conversation

@garrytan
Copy link
Copy Markdown
Owner

@garrytan garrytan commented Apr 25, 2026

Summary

Adds /gstack-claude as the Claude outside-voice counterpart to /codex for non-Claude hosts. External hosts now get review, adversarial challenge, and read-only consult modes through claude -p without exposing nested Claude mutation tools.

Also fixes validation regressions found during the work: the gbrain config isolation test no longer depends on local machine config contents, the tracked-file-size check now warns instead of failing so large fixtures can remain in repo, and Claude review/challenge diff temp files now use mktemp instead of a $$ path.

Changes

  • Add claude/SKILL.md.tmpl with review/challenge/consult modes, temp-file stdin prompt transport, auth checks, JSON output parsing, --disable-slash-commands, tool-less review/challenge, and read-only consult flags.
  • Skip the Claude outside-voice skill only for the Claude host.
  • Extend generated-doc tests so all external hosts emit gstack-claude/SKILL.md and Claude host omits claude/SKILL.md.
  • Preserve real ~/.gstack/config.yaml contents in the GSTACK_HOME override test.
  • Convert oversized tracked file validation into [size-warning] advisory output.
  • Remove the restored Opus fanout overlay nudge and its hard-coded preflight/test requirement.
  • Bump release metadata to 1.13.0.0 and update CHANGELOG.md.

Test Coverage

Full suite before release metadata:

  • bun test — passed, exit 0

Targeted verification after release metadata:

  • bun test test/ship-version-sync.test.ts test/gen-skill-docs.test.ts test/skill-validation.test.ts test/brain-sync.test.ts test/model-overlay-opus-4-7.test.ts — 742 pass, 0 fail

Follow-up verification after Claude outside-voice review fixes:

  • bun run gen:skill-docs --host all
  • bun test test/gen-skill-docs.test.ts — 371 pass, 0 fail
  • bun test test/model-overlay-opus-4-7.test.ts test/gen-skill-docs.test.ts — 380 pass, 0 fail

Notes:

  • The fixture browse/test/fixtures/security-bench-haiku-responses.json remains in git.
  • The size check now logs [size-warning] tracked files over 2 MiB: browse/test/fixtures/security-bench-haiku-responses.json (27.5 MiB) instead of failing.
  • .context/attachments/image.png is ignored Conductor attachment state and is not part of this PR.

Pre-Landing Review

/gstack-claude review was run in tool-less mode against this branch. The actionable finding was the diff temp-file collision risk; it is fixed by using mktemp for DIFF_FILE.

Documentation

CHANGELOG.md updated for 1.13.0.0.

Test plan

  • Full bun test
  • Focused generated-doc, validation, gbrain, Opus overlay, and version-sync tests
  • /gstack-claude outside-voice review
  • Branch pushed to origin/garrytan/claude-codex-skill

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 25, 2026

E2E Evals: ✅ PASS

0/0 tests passed | $0 total cost | 12 parallel runners

Suite Result Status Cost

12x ubicloud-standard-2 (Docker: pre-baked toolchain + deps) | wall clock ≈ slowest suite

@garrytan garrytan merged commit 23c4d7b into main Apr 25, 2026
23 checks passed
gonnabe88 pushed a commit to gonnabe88/gstack that referenced this pull request May 9, 2026
* Add Claude outside-voice skill

* Fix gbrain config isolation test

* Restore Opus fanout overlay nudge

* Warn on oversized tracked files

* Release v1.13.0.0

* Fix Claude diff temp file handling

* Remove Opus fanout overlay nudge
RyotaKun pushed a commit to RyotaKun/gstack that referenced this pull request May 18, 2026
* Add Claude outside-voice skill

* Fix gbrain config isolation test

* Restore Opus fanout overlay nudge

* Warn on oversized tracked files

* Release v1.13.0.0

* Fix Claude diff temp file handling

* Remove Opus fanout overlay nudge
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