Skip to content

feat(pstack): add opt-in Claude Mods bug-fix gates - #5

Merged
yxhta merged 4 commits into
mainfrom
feat/claude-mods-bugfix-gates
Oct 7, 2026
Merged

yxhta merged 4 commits into
mainfrom
feat/claude-mods-bugfix-gates

Conversation

@yxhta

@yxhta yxhta commented Oct 6, 2026 •

Copy link
Copy Markdown
Owner

Why

Make missing test and review evidence visible during a bug fix. Keep successful evidence tied to the code snapshot that produced it.

What changed

  • Add a separate, opt-in Claude Mods adapter for reproduction, fixes, verification, and two independent Thermos reviews.
  • Capture actual process exits and invalidate evidence after code changes, interruptions, or stale callbacks. Late tool failures cannot block a replacement pipeline.
  • Show the current stage, missing evidence, and failures; generate and validate the package from fork-owned inputs.

Scope

Bug-fix workflows on Claude Code 2.1.289 and Unix hosts. Existing pstack, Thermos, and Codex packages remain unchanged from current main. The adapter does not merge, deploy, or enable itself in an existing user session.

Tradeoffs

Require a clean committed reproduction baseline and reject unsupported Git state. Mods and bounded Stop reminders are workflow checks, not a universal security or completion guarantee.

Blast Radius

Only users who install the separate plugin and explicitly start a recipe enter the pipeline. CI gains pinned native Mods checks.

Verification

  • 131 Python tests and 89 official native Mods tests passed, including new current/stale reviewer Read and parent Edit failure regressions.
  • Strict native validation, generated TypeScript 5.9.3 checks, no-inference load, and both-runtime installer/source parity passed.
  • Fresh independent correctness and quality reviews found no remaining actionable issues. The remote tree and modes match the reviewed local tree; current main was merged without rewriting existing commits.

Authenticated real-model execution remains unverified. Native reviewer identities and results are stubbed in the test harness. See the original verification evidence and limits.

yxhta added 2 commits October 6, 2026 13:38
Capture process evidence and bind verification and independent Thermos reviews to the current code snapshot. Add bounded lifecycle guards, visible status, native tests, and generated Claude-only packaging.

Preserve upstream sources and existing portable pstack, Thermos, and Codex behavior. Authenticated model execution remains unverified.
Compare raw file bytes and modes against immutable baseline objects. Keep native type declarations outside generated package freshness checks.

Add regressions for cached timestamps, replacement refs, diff transitions, and native sidecars. All 126 Python and 85 native Mods tests pass; independent re-review has no remaining findings.
yxhta added 2 commits October 8, 2026 00:02
Merge current main and regenerate the plugin version for the updated upstream pin.
Associate each guarded tool call with its pipeline epoch. Preserve the host catch handler while preventing failures from canceled calls from blocking a replacement pipeline.
@yxhta
yxhta marked this pull request as ready for review October 7, 2026 21:39
@yxhta
yxhta merged commit 40433ea into main Oct 7, 2026
4 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