mirror of
https://github.com/garrytan/gstack.git
synced 2026-11-01 22:45:02 +01:00
Garry Tanandcapy-ai[bot]
148d28ea81
* wip(GSTA-131): review depth, fix-check transitions, dispatch counters, carry pregate, autoplan round policy Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * wip(GSTA-131): reviewers stay at full effort; audits light Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * feat(ship): Step 9 dispatches what gstack-review-plan plans; fix-check after below-floor fixes; cap two full passes plus one fix-check Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * test: review-dispatch-compare has a free lane; review-depth fixtures registered free; resumed max_rounds case off the old default Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * fix(GSTA-131): ship review pass-1 repairs: fix-check binds under its own skill, Step 11.5 accepts a fix-check or a planned adversarial skip, delta carry with a pregate, carry --pregate pinned to the checkout, quiet-after-cap revise regates, skill-dir markdown and rule docs classed code, gate signals from tests class up, plan fails closed on git errors, per-class compare ratios Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * fix(GSTA-131): ship review pass-2 repairs: Step 11 follows the last full-pass plan, Important Rules restored, ship goldens regenerated, true per-class medians, review-policy transitions and once-per-invocation voices, docs/** assets stay docs-class, CLI and fix-check binding tests Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * docs(changelog): GSTA-131 review depth, fix-check, delta /ship and autoplan rounds Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * docs: ship-policy cap and counters, review-policy /review cycle cap, unattended passes (docs audit) Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * fix(review): 400-word findings cap fits the /review size pins (five host pins lowered by 2); CHANGELOG max_rounds wording Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * docs(changelog): gate integrity names the size-ledger entry and the lowered /review pins Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * test: the round-budget case asserts the default through ROUNDS output only (pregate literals heuristic) Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * test: curate ship-receipt-carry-pregate out of the Windows lane like its --pregate siblings Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * docs(changelog): end the Windows curation sentence instead of a mid-sentence colon Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * fix(ship): one decision each for the review-loop exits, the planned adversarial skip and Step 16's TODOS rerun (screen judge repair) Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> * fix(ship): fix-check repairs: pin the renamed Step 11 heading, keep the below-floor exit at the cap, name the plan run a repeat reuses, print the native skip on outside-only plans Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> --------- Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
221 lines
8.2 KiB
JSON
221 lines
8.2 KiB
JSON
{
|
|
"source": "Step and phase headings of the judged workflow skills, derived on 2026-10-04 from the generated Claude SKILL.md and sections (templates at the integration head). qa-only lists its template's own top-level headings. C1 calibration showed the workflow judges pass a bundle with a required step removed, so missing-step coverage lives here.",
|
|
"skills": {
|
|
"qa": {
|
|
"SKILL.md": [
|
|
"## Step 0: Detect platform and base branch",
|
|
"## Phases 1-6: QA Baseline",
|
|
"## Phase 7: Triage",
|
|
"## Phase 8: Fix Loop",
|
|
"### 8a. Diagnose and reproduce",
|
|
"### 8a.5. Regression test before repair",
|
|
"### 8b. Fix",
|
|
"### 8c. Re-test",
|
|
"### 8d. Commit verified work",
|
|
"### 8e. Classify",
|
|
"### 8e.5. Regression Test record",
|
|
"### 8f. Self-Regulation (STOP AND EVALUATE)",
|
|
"## Phase 9: Final QA",
|
|
"## Phase 10: Report",
|
|
"## Phase 11: TODOS.md Update"
|
|
],
|
|
"sections/exploratory.md": [
|
|
"## 1. Charter and preflight",
|
|
"## 2. Probe loop",
|
|
"## 3. Parent handoff",
|
|
"## 4. Final report"
|
|
],
|
|
"sections/qa-patterns.md": [
|
|
"### Phase 1: Initialize",
|
|
"### Phase 2: Authenticate (if needed)",
|
|
"### Phase 3: Orient",
|
|
"### Phase 4: Explore",
|
|
"### Phase 5: Document",
|
|
"### Phase 6: Wrap Up"
|
|
]
|
|
},
|
|
"qa-only": {
|
|
"SKILL.md": [
|
|
"## Request Parameters",
|
|
"## Test Plan Context",
|
|
"## Select Surfaces and Isolation",
|
|
"## Prepare Report Artifacts",
|
|
"## Browser Setup (conditional)",
|
|
"## Run the Selected Checks",
|
|
"## Output",
|
|
"## Additional Rules (qa-only specific)"
|
|
],
|
|
"sections/exploratory.md": [
|
|
"## 0. Preparation gate",
|
|
"## 1. Charter and preflight",
|
|
"## 2. Probe loop",
|
|
"## 3. Parent handoff",
|
|
"## 4. Final report"
|
|
],
|
|
"sections/reporting.md": [
|
|
"## 1. Establish each finding once",
|
|
"## 2. Fill timing fields from their actual boundaries",
|
|
"## 3. Assemble and check every repetition before writing",
|
|
"## 4. Capture permitted notes, then write the report"
|
|
]
|
|
},
|
|
"review": {
|
|
"SKILL.md": [
|
|
"## Step 0: Detect platform and base branch",
|
|
"## Step 1: Check branch",
|
|
"## Step 1.5: Scope Drift Detection",
|
|
"## Step 2: Read the checklist",
|
|
"## Step 2.5: Check for Greptile review comments",
|
|
"## Step 3: Get the diff",
|
|
"## Step 3.4: Workspace-aware queue status (advisory)",
|
|
"## Step 3.5: Diff scans",
|
|
"## Step 3.6: Gather review context",
|
|
"## Step 4: Critical pass (core review)",
|
|
"### Step 4.7: Exploratory QA (before Fix-First)",
|
|
"## Step 5: Fix-First Review",
|
|
"### Step 5.0: Cross-review finding dedup",
|
|
"### Step 5a: Classify each finding",
|
|
"### Step 5b: Auto-fix all AUTO-FIX items",
|
|
"### Step 5c: Batch-ask about ASK items",
|
|
"### Step 5d: Apply user-approved fixes",
|
|
"## Step 5.8: Persist Eng Review result",
|
|
"### 1. Re-review after edits",
|
|
"### 2. Fill the record"
|
|
],
|
|
"sections/adversarial.md": [
|
|
"## Step 4.8: Adversarial review (always-on)"
|
|
],
|
|
"sections/review-army.md": [
|
|
"## Step 4.5: Review Army — Specialist Dispatch",
|
|
"### Step 4.6: Collect and merge findings",
|
|
"#### 1. Parse outputs",
|
|
"#### 2. Validate severity",
|
|
"#### 3. Identify and merge",
|
|
"#### 4. Lead judgment",
|
|
"#### 5. Apply specialist confidence gates",
|
|
"#### 6. Score and present specialists",
|
|
"#### 7. Save specialist activity",
|
|
"#### 8. Hand off to Fix-First"
|
|
]
|
|
},
|
|
"ship": {
|
|
"SKILL.md": [
|
|
"## Step 0: Detect platform and base branch",
|
|
"## Step 0.9: Apple target detection",
|
|
"## Step 1: Pre-flight",
|
|
"## Step 2: Distribution Pipeline Check",
|
|
"## Step 3: Merge the base branch (BEFORE tests)",
|
|
"## Step 11.5: Bind the reviews",
|
|
"## Step 12: Version bump (auto-decide)",
|
|
"## Step 14: TODOS.md (auto-update)",
|
|
"## Step 14.5: Documentation audit (every ship)",
|
|
"## Step 15: Commit (bisectable chunks)",
|
|
"## Step 16: Verification Gate",
|
|
"### 1. Finish writers and prepare outputs",
|
|
"### 2. Choose the change route",
|
|
"### 3. Resolve documentation freshness",
|
|
"### 4. Verify the frozen candidate",
|
|
"### 5. Report, then push",
|
|
"## Step 17: Push",
|
|
"## Step 18: Prepare publication metadata",
|
|
"## Step 20: Persist ship metrics",
|
|
"## Step 21: Plan-tune discoverability nudge (first-successful-ship only)",
|
|
"## Step 22: Terminal line (every run)"
|
|
],
|
|
"sections/adversarial.md": [
|
|
"## Step 11: Adversarial review"
|
|
],
|
|
"sections/changelog.md": [
|
|
"## Step 13: CHANGELOG (auto-generate)"
|
|
],
|
|
"sections/greptile-early.md": [
|
|
"## Step 6.5: Early Greptile PR (after the free tests pass)"
|
|
],
|
|
"sections/greptile.md": [
|
|
"## Step 10: Address Greptile review comments (if PR exists)"
|
|
],
|
|
"sections/plan-completion.md": [
|
|
"## Step 8: Plan Completion Audit",
|
|
"## Step 8.1: Plan Verification",
|
|
"## Step 8.2: Scope Drift Detection"
|
|
],
|
|
"sections/pr-body.md": [
|
|
"## Step 19: Create PR/MR"
|
|
],
|
|
"sections/review-army.md": [
|
|
"## Step 9: Pre-Landing Review",
|
|
"## Step 9.1: Review Army — Specialist Dispatch",
|
|
"### Step 9.2: Collect and merge findings",
|
|
"#### 1. Parse outputs",
|
|
"#### 2. Validate severity",
|
|
"#### 3. Identify and merge",
|
|
"#### 4. Lead judgment",
|
|
"#### 5. Apply specialist confidence gates",
|
|
"#### 6. Score and present specialists",
|
|
"#### 7. Save specialist activity",
|
|
"#### 8. Hand off to Fix-First",
|
|
"### Step 9.2.1: Exploratory QA (before Fix-First)",
|
|
"### Step 9.3: Cross-review finding dedup",
|
|
"## Step 9.4: Fix-First and persistence"
|
|
],
|
|
"sections/test-coverage.md": [
|
|
"## Step 7: Test Coverage Audit"
|
|
],
|
|
"sections/tests.md": [
|
|
"## Step 4: Test Framework Bootstrap",
|
|
"## Step 5: Run tests (on merged code)",
|
|
"## Step 6: Eval Suites (conditional)"
|
|
]
|
|
},
|
|
"plan-ceo-review": {
|
|
"SKILL.md": [
|
|
"## Step 0: Detect platform and base branch",
|
|
"## Step 0: Nuclear Scope Challenge + Mode Selection",
|
|
"### 0A. Premise Challenge",
|
|
"### 0B. Existing Code Leverage",
|
|
"### 0C. Dream State Mapping",
|
|
"### 0D. Alternatives (reusable decision procedure)",
|
|
"### 0E. Mode Selection",
|
|
"### 0F. Expansion Framing (shared by EXPANSION and SELECTIVE EXPANSION)",
|
|
"### 0G. Mode-Specific Analysis",
|
|
"### 0H. Persist CEO Plan (EXPANSION and SELECTIVE EXPANSION only)",
|
|
"### 0I. Temporal Interrogation (EXPANSION, SELECTIVE EXPANSION, and HOLD modes)"
|
|
],
|
|
"sections/review-sections.md": [
|
|
"### Section 1: Architecture Review",
|
|
"### Section 2: Error & Rescue Map",
|
|
"### Section 3: Security & Threat Model",
|
|
"### Section 4: Data Flow & Interaction Edge Cases",
|
|
"### Section 5: Code Quality Review",
|
|
"### Section 6: Test Review",
|
|
"### Section 7: Performance Review",
|
|
"### Section 8: Observability & Debuggability Review",
|
|
"### Section 9: Deployment & Rollout Review",
|
|
"### Section 10: Long-Term Trajectory Review",
|
|
"### Section 11: Design & UX Review (skip if no UI scope detected)"
|
|
]
|
|
},
|
|
"plan-eng-review": {
|
|
"SKILL.md": [
|
|
"### Step 0: Scope Challenge"
|
|
],
|
|
"sections/review-sections.md": [
|
|
"### 1. Architecture review",
|
|
"### 2. Code quality review",
|
|
"### 3. Test review",
|
|
"### 4. Performance review"
|
|
]
|
|
},
|
|
"setup-deploy": {
|
|
"SKILL.md": [
|
|
"### Step 1: Check existing configuration",
|
|
"### Step 2: Detect platform",
|
|
"### Step 3: Platform-specific setup",
|
|
"### Step 4: Write configuration",
|
|
"### Step 5: Verify",
|
|
"### Step 6: Summary"
|
|
]
|
|
}
|
|
}
|
|
}
|