feat: /adr skill — architectural decision records with decision gate#737
Open
ljagged wants to merge 12 commits intogarrytan:mainfrom
Open
feat: /adr skill — architectural decision records with decision gate#737ljagged wants to merge 12 commits intogarrytan:mainfrom
ljagged wants to merge 12 commits intogarrytan:mainfrom
Conversation
…hestration (v1.1.0) Two modes: brief (autonomous competitive intelligence via WebSearch + browse) and session (interactive Rumelt's kernel diagnosis with framework selection). Produces versioned strategy documents with inline citations, milestone-gated execution plans, and change tracking. Integrates with the gstack skill network. v1.1 improvements from dogfood: broad market scan for competitor discovery, browse-aggressive research, coherent actions with mutual support structure, milestone-gated execution (not calendar months), brief amendment step, Codex second opinion option, and "what did I miss?" validation. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Both review skills now discover *-strategy-*.md files from /strategist alongside design docs. Strategy docs provide competitive context and guiding policy to inform scope and architecture decisions during reviews. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
CLAUDE.md: add /strategist to available skills list. TODOS.md: add 3 items — CSO/retro artifact ingestion, autoplan integration, and E2E eval test (all post-dogfood). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Standardize on gstack-slug everywhere (was mixing remote-slug and gstack-slug, causing Mode 1 write / Mode 2 read path mismatch) - Add SLUG derivation to bash blocks that used it without re-deriving - Fix description: "milestone-gated execution plans" not "90-day action plans" - Add auto-chain retry guard: if Mode 1 fails, stop instead of retrying Found by adversarial review (Claude subagent, 15 findings). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
feat: add /strategist skill — competitive strategy with framework orchestration (v0.11.22.0)
Resolved conflicts in VERSION, CHANGELOG.md, README.md. Kept upstream version (0.12.11.0), upstream CHANGELOG entries, and upstream's --single-branch --depth 1 clone command. Preserved /strategist in skill lists from feat-strategist merge. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…(v0.1.0)
New skill with 5 modes (create, list, check, revisit, litmus-test) plus
utilities (accept, renumber, skipped). Decision gate in shared preamble
detects architectural decisions in-flight and pauses before implementation.
ADRs stored in docs/adr/ with YAML frontmatter. Skip log in .skipped.jsonl.
Includes {{ADR_DISCOVERY}} resolver consumed by /plan-eng-review, /review,
/plan-ceo-review, and /investigate. ADR count in review readiness dashboard.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
# Conflicts: # README.md # SKILL.md # SKILL.md.tmpl # TODOS.md # canary/SKILL.md # cso/SKILL.md # design-shotgun/SKILL.md # document-release/SKILL.md # investigate/SKILL.md # retro/SKILL.md # scripts/resolvers/index.ts # scripts/resolvers/preamble.ts # setup-deploy/SKILL.md
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/adrskill with 5 modes (create, list, check, revisit, litmus-test) plus utilities (accept, renumber, skipped){{ADR_DISCOVERY}}resolver consumed by/plan-eng-review,/review,/plan-ceo-review, and/investigatedocs/adr/(repo, not~/.gstack/) with YAML frontmatterdocs/adr/.skipped.jsonl(JSONL, CI-friendly)docs/adr/.config(not global gstack-config)Design decisions
docs/adr/): ADRs are project artifacts. Full visibility of drafts matters as much as finalized records./adritself to prevent recursion.docs/adr/exists. Zero overhead for non-ADR projects./adr renumberfor post-merge cleanup./adr accept Nfor explicit transition.Test plan
bun run gen:skill-docsgenerates cleanly (adr: 869 lines, ~10049 tokens)bun testpasses (602 pass, 4 fail — all pre-existing in strategist)🤖 Generated with Claude Code