roadmap: prune low-value items (close #213, #214; mark #223 done; clarify #230/#231) - #233
Merged
Merged
Conversation
Contributor
Claude Code Review SummaryStatus: Skipped (trivial changes) This PR only contains documentation or configuration changes. Code review skipped to save API costs. To force a review, add the |
BaseInfinity
added a commit
that referenced
this pull request
Apr 24, 2026
* release: v1.37.0 Bundles two Codex-certified PRs: - PR #235 (ROADMAP #231 Phase 1): delete monthly-research.yml — 519 lines + 4 claude-code-action steps removed, zero merged artifacts in 30d, $11-23/mo API burn eliminated. Stubbed 17 test_monthly_* assertions (#231 Phase 1 pattern), marked 5 live-doc sections REMOVED/historical. Codex 3-round: 9/10. - PR #236 (ROADMAP #217): loud WARNING below xhigh effort — max preferred, xhigh floor, single source of truth. Dupe effort check removed from instructions-loaded-check.sh. 2 TDD tests + 1 regression test. Codex 3-round: 10/10. Version parity: - package.json, SDLC.md metadata + table, CLAUDE_CODE_SDLC_WIZARD.md (2 refs), .claude-plugin/plugin.json, .claude-plugin/marketplace.json → 1.37.0 - skills/update/SKILL.md Step 3 example refreshed - CHANGELOG entry "[1.37.0] - 2026-04-24" ROADMAP: - #232: /update-wizard should mimic 'claude update' UX - #233: mixed-mode coder+reviewer tier (Sonnet coder + Opus reviewer for simple repos, inspired by codex-sdlc-wizard) - #234: GitHub About populated (description, homepage, 8 topics) Release review (Codex round 1 → 2): - REL-001 fixed: commit now exists on release branch (was empty) - REL-002 fixed: '2 claude-code-action blocks' corrected to '4 steps' - REL-003 fixed: .reviews/handoff.json + untracked .claude/hooks excluded * release(v1.37.0): fix CHANGELOG line 26 count (2 → 4 claude-code-action steps) Codex round-2 recheck caught that the 'Removed' section bullet still said '2 API blocks' — line 11 was fixed, line 26 was not. Align with reality: pre-delete workflow had 4 uses: anthropics/claude-code-action@v1 steps.
4 tasks
BaseInfinity
added a commit
that referenced
this pull request
Apr 25, 2026
* fix(ci): wire #233 + #224 tests into validate (orphan-test fix) Codex round 1 review of v1.39.0 surfaced that v1.38.0's new tests weren't wired into ci.yml/CONTRIBUTING.md, so test-workflow-triggers fails on this branch (164/1). - ci.yml: append test-repo-complexity.sh (#233) + test-prompt-hook- fires-once.sh (#224) after test-local-shepherd.sh - CONTRIBUTING.md: add same two entries to the local-test command list (must mirror ci.yml validate steps; test-workflow-triggers enforces this) Verified: bash tests/test-workflow-triggers.sh → 165/165 PASS. * feat(#233): mixed-mode tier (Sonnet coder + Opus reviewer) — v1.38.0 Adds repo-complexity heuristic + 3-way setup prompt so simple repos can opt into Sonnet 4.6 in-session while keeping cross-model review at the flagship tier. - New cli/lib/repo-complexity.js + `npx agentic-sdlc-wizard complexity` subcommand. Classifies via filesystem signals (LOC, tests, hooks, workflows) plus stakes flag (.env / secrets/ / credentials/ at any depth forces complex regardless of size). - Setup Step 9.5 expanded from binary y/N to 3-way (no-pin / mixed / flagship), with the heuristic suggesting a default per repo. - Cross-model review section in skills/sdlc/SKILL.md explicitly notes reviewer always at flagship regardless of coder pin. - Update skill Step 7.5 recognizes sonnet[1m] as a valid mixed-mode pin (no migration prompt). - Reconciles with #198: opt-in per-project; no-pin remains default. Tests: 11/11 in tests/test-repo-complexity.sh including boundary (29 tests/4 hooks/4 workflows/~9K LOC → simple, 30 tests = high threshold → complex) and nested stakes (config/.env, app/secrets/). 128/128 hook regression, 22/22 doc consistency, 70/70 cli, 25/25 plugin, 10/10 compliance. Codex 2-round review: round 1 5/10 NOT CERTIFIED (4 findings — un- runnable doc command, root-only stakes, threshold mismatch with docs, stale 1.37.1 example), round 2 9/10 CERTIFIED. * feat(#224): prompt-hook-fires-once instrumentation (bundled with v1.38.0) CC 2.1.118 shipped a fix for prompt hooks double-firing when an agent-hook verifier subagent itself made tool calls. We can't unit- test CC's behavior, but we can ship instrumentation so the maintainer can verify the fix holds in real sessions. - hooks/sdlc-prompt-check.sh: opt-in via SDLC_HOOK_FIRE_LOG env var. Each post-dedupe invocation appends one tab-separated record (<ts>\t<pid>\tsdlc-prompt-check). Unwritable paths fail silently. - tests/test-prompt-hook-fires-once.sh: 6 tests asserting counter increment, opt-in semantics, log shape, byte-identical output diff (instrumented vs uninstrumented), and unwritable-path tolerance. Test harness cd's into an isolated workspace so the result is independent of caller cwd. - CLAUDE_CODE_SDLC_WIZARD.md: new "Verifying Prompt-Hook-Fires-Once" subsection with maintainer procedure (set env, run session, count log lines, compare to prompt count). Codex 2-round (rounds 3+4 of the v1.38.0 review): round 3 8/10 (test harness false-green from caller cwd, weak output assertion), round 4 10/10 CERTIFIED.
User audit: "with all the opus fixes i dont think we need to do adaptive thinking test anymore... im confused why does that [5.5 calibration] need API and replay harness huh this... we should audit [weekly/monthly] when replacing them if we even need it." Closed: - #214 adaptive thinking A/B: moot given xhigh/max floor mandate. Saves $12 + spares API burn. - #213 ship DISABLE_ADAPTIVE_THINKING default: same logic. xhigh floor > default band-aid env var. Keeps as opt-in for power users. - #223 GPT-5.5 in review tier: already shipped via Codex config (model = "gpt-5.5" default). Formal calibration was academic. Updated: - #230 shepherd baseline/candidate: marked P3 nice-to-have with user quote "just get feature parity once." Not urgent. - #231 weekly/monthly cleanup: audit outcome embedded. weekly-update actively useful (migrate its 9 API blocks). monthly-research mostly fails on cron + rarely run manually (last success 2026-03-27) + 519 lines of low-value research-issue creation → proposed deletion instead of migration. No code changes. doc-consistency green.
2 tasks
BaseInfinity
added a commit
that referenced
this pull request
Apr 27, 2026
Mixed-mode tier (Sonnet coder + Opus reviewer) docs already shipped in v1.38.0: - skills/sdlc/SKILL.md:233 reviewer-at-flagship invariant - CLAUDE_CODE_SDLC_WIZARD.md:1027-1066 "Mixed-Mode Tier" section What's NOT shipped: setup auto-detection of repo complexity + wizard auto-write of model pin + Prove-It Gate (3+ simple repos pair-tested). Deferring (a)/(b)/Prove-It with rationale captured in ROADMAP entry. Short version: primary maintainer is Max-only, model pin disables auto- mode, repo-complexity heuristic is fragile, Prove-It costs real API $$ for a feature the maintainer won't use, manual opt-in already works for API-cost users. Research evidence (2026-04-26): - Opus 4.7 ≈ 1.67× Sonnet 4.6 ($5/$25 vs $3/$15), ~2.25× factoring Opus tokenizer 35% inflation - 8 pt SWE-bench Verified gap (79.6% vs 87.6%); ~30% on hard agentic - Pattern is Anthropic-blessed (Advisor Strategy) + matches Aider architect/editor mode Trigger to revisit: API-cost user files issue OR maintainer API canary restored and burn becomes measurable.
9 tasks
BaseInfinity
added a commit
that referenced
this pull request
May 5, 2026
* release: v1.37.0 Bundles two Codex-certified PRs: - PR #235 (ROADMAP #231 Phase 1): delete monthly-research.yml — 519 lines + 4 claude-code-action steps removed, zero merged artifacts in 30d, $11-23/mo API burn eliminated. Stubbed 17 test_monthly_* assertions (#231 Phase 1 pattern), marked 5 live-doc sections REMOVED/historical. Codex 3-round: 9/10. - PR #236 (ROADMAP #217): loud WARNING below xhigh effort — max preferred, xhigh floor, single source of truth. Dupe effort check removed from instructions-loaded-check.sh. 2 TDD tests + 1 regression test. Codex 3-round: 10/10. Version parity: - package.json, SDLC.md metadata + table, CLAUDE_CODE_SDLC_WIZARD.md (2 refs), .claude-plugin/plugin.json, .claude-plugin/marketplace.json → 1.37.0 - skills/update/SKILL.md Step 3 example refreshed - CHANGELOG entry "[1.37.0] - 2026-04-24" ROADMAP: - #232: /update-wizard should mimic 'claude update' UX - #233: mixed-mode coder+reviewer tier (Sonnet coder + Opus reviewer for simple repos, inspired by codex-sdlc-wizard) - #234: GitHub About populated (description, homepage, 8 topics) Release review (Codex round 1 → 2): - REL-001 fixed: commit now exists on release branch (was empty) - REL-002 fixed: '2 claude-code-action blocks' corrected to '4 steps' - REL-003 fixed: .reviews/handoff.json + untracked .claude/hooks excluded * release(v1.37.0): fix CHANGELOG line 26 count (2 → 4 claude-code-action steps) Codex round-2 recheck caught that the 'Removed' section bullet still said '2 API blocks' — line 11 was fixed, line 26 was not. Align with reality: pre-delete workflow had 4 uses: anthropics/claude-code-action@v1 steps.
BaseInfinity
added a commit
that referenced
this pull request
May 5, 2026
* fix(ci): wire #233 + #224 tests into validate (orphan-test fix) Codex round 1 review of v1.39.0 surfaced that v1.38.0's new tests weren't wired into ci.yml/CONTRIBUTING.md, so test-workflow-triggers fails on this branch (164/1). - ci.yml: append test-repo-complexity.sh (#233) + test-prompt-hook- fires-once.sh (#224) after test-local-shepherd.sh - CONTRIBUTING.md: add same two entries to the local-test command list (must mirror ci.yml validate steps; test-workflow-triggers enforces this) Verified: bash tests/test-workflow-triggers.sh → 165/165 PASS. * feat(#233): mixed-mode tier (Sonnet coder + Opus reviewer) — v1.38.0 Adds repo-complexity heuristic + 3-way setup prompt so simple repos can opt into Sonnet 4.6 in-session while keeping cross-model review at the flagship tier. - New cli/lib/repo-complexity.js + `npx agentic-sdlc-wizard complexity` subcommand. Classifies via filesystem signals (LOC, tests, hooks, workflows) plus stakes flag (.env / secrets/ / credentials/ at any depth forces complex regardless of size). - Setup Step 9.5 expanded from binary y/N to 3-way (no-pin / mixed / flagship), with the heuristic suggesting a default per repo. - Cross-model review section in skills/sdlc/SKILL.md explicitly notes reviewer always at flagship regardless of coder pin. - Update skill Step 7.5 recognizes sonnet[1m] as a valid mixed-mode pin (no migration prompt). - Reconciles with #198: opt-in per-project; no-pin remains default. Tests: 11/11 in tests/test-repo-complexity.sh including boundary (29 tests/4 hooks/4 workflows/~9K LOC → simple, 30 tests = high threshold → complex) and nested stakes (config/.env, app/secrets/). 128/128 hook regression, 22/22 doc consistency, 70/70 cli, 25/25 plugin, 10/10 compliance. Codex 2-round review: round 1 5/10 NOT CERTIFIED (4 findings — un- runnable doc command, root-only stakes, threshold mismatch with docs, stale 1.37.1 example), round 2 9/10 CERTIFIED. * feat(#224): prompt-hook-fires-once instrumentation (bundled with v1.38.0) CC 2.1.118 shipped a fix for prompt hooks double-firing when an agent-hook verifier subagent itself made tool calls. We can't unit- test CC's behavior, but we can ship instrumentation so the maintainer can verify the fix holds in real sessions. - hooks/sdlc-prompt-check.sh: opt-in via SDLC_HOOK_FIRE_LOG env var. Each post-dedupe invocation appends one tab-separated record (<ts>\t<pid>\tsdlc-prompt-check). Unwritable paths fail silently. - tests/test-prompt-hook-fires-once.sh: 6 tests asserting counter increment, opt-in semantics, log shape, byte-identical output diff (instrumented vs uninstrumented), and unwritable-path tolerance. Test harness cd's into an isolated workspace so the result is independent of caller cwd. - CLAUDE_CODE_SDLC_WIZARD.md: new "Verifying Prompt-Hook-Fires-Once" subsection with maintainer procedure (set env, run session, count log lines, compare to prompt count). Codex 2-round (rounds 3+4 of the v1.38.0 review): round 3 8/10 (test harness false-green from caller cwd, weak output assertion), round 4 10/10 CERTIFIED.
BaseInfinity
added a commit
that referenced
this pull request
May 5, 2026
Mixed-mode tier (Sonnet coder + Opus reviewer) docs already shipped in v1.38.0: - skills/sdlc/SKILL.md:233 reviewer-at-flagship invariant - CLAUDE_CODE_SDLC_WIZARD.md:1027-1066 "Mixed-Mode Tier" section What's NOT shipped: setup auto-detection of repo complexity + wizard auto-write of model pin + Prove-It Gate (3+ simple repos pair-tested). Deferring (a)/(b)/Prove-It with rationale captured in ROADMAP entry. Short version: primary maintainer is Max-only, model pin disables auto- mode, repo-complexity heuristic is fragile, Prove-It costs real API $$ for a feature the maintainer won't use, manual opt-in already works for API-cost users. Research evidence (2026-04-26): - Opus 4.7 ≈ 1.67× Sonnet 4.6 ($5/$25 vs $3/$15), ~2.25× factoring Opus tokenizer 35% inflation - 8 pt SWE-bench Verified gap (79.6% vs 87.6%); ~30% on hard agentic - Pattern is Anthropic-blessed (Advisor Strategy) + matches Aider architect/editor mode Trigger to revisit: API-cost user files issue OR maintainer API canary restored and burn becomes measurable.
This was referenced May 24, 2026
BaseInfinity
added a commit
that referenced
this pull request
May 24, 2026
…l stale items, add demand-signal gate (#349) Applies the Codex prioritization from .reviews/roadmap-prio-codex.md (2026-05-24). Excised (tracked in sibling repos): - #9 OpenCode port → BaseInfinity/opencode-sdlc-wizard - #82 Domain DLCs → Stefan's separate track - #91 Multi-Agent Adapter umbrella → per-adapter sibling repos - Back Burner: Agent-agnostic SDLC → adapter/sibling strategy Killed (no demand signal + months stale): - #85 Phase 2 (1.7 mo, killed by #231 zero-cron philosophy) - #233 automation subitems (1 mo, Max-user footgun, no API-cost user) - Back Burner: Chaos/Resilience Testing (1.8 mo, no concrete failure) - Back Burner: Subagent Model Compliance Audit (3+ mo, prototype already deleted) New: Demand-Signal-First entry gate added at top of ROADMAP. New items require one of: maintainer pain event with repro, second external user signal, dated platform deadline, or low-cost cleanup. Everything else goes to a Research Parking Lot with 30/60-day expiry. Row states: Actionable now / Waiting on trigger / Tracked elsewhere / DONE / KILLED. Net: backlog shrinks, sibling repos own their work, and the gate stops 'NO for now' from accumulating as permanent cognitive debt.
4 tasks
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
1 participant
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.
User audit 2026-04-24. Closes redundant gates, marks already-shipped work as done, clarifies scope on the two #212 follow-ups.
Closed:
Refined:
Doc-only. No code. 22/0 doc-consistency.