Repository navigation
feat: a debug skill, a deslop pass, and the same flow on five harnesses - #37
Merged
Merged
Conversation
A failure with no known cause goes to the new debug skill: reproduce, root cause in one sentence, a failing test, a fix at the cause, and a stop after three fixes that did not hold. build loads it when a test fails. build removes what its own change did not need before the last test run, by the list in skills/build/deslop.md. review lists the same things under a new Slop heading, which does not touch the gate. Grok Build is a harness. The session-start hook tells Grok Build, Claude Code and Codex apart by the variable that each one sets for a plugin hook, so Codex no longer gets the Claude Code action map. The Cursor, Grok Build and opencode maps say what builder:<name> means there. A skill that runs the helper binary says where it is when no hook ran. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
…ta store or an agent tool plan runs the lens after the engineering lens, and review checks a change against the same file. It asks where the contract is written, whether every list has one shape, who owns each fact, where the customer boundary is enforced, what each agent tool can do, and which guard keeps each rule. The project's own rules win over it, and three questions have no default: the person answers them. Tenet 9 now says what a good guard does: it names the incident that made it, says what to do instead, and fails when it checks nothing. Co-Authored-By: Claude Fable 5.2 <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
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.
Stacked on #36. A fix at the cause, a change with nothing in it that the task did not need, and the same flow on five harnesses.
What a person can now do
debugskill reproduces the failure, states the root cause in one sentence, pins it with a failing test and fixes the cause. It changes one thing at a time and stops after three fixes that did not hold.buildloads it when a test fails.buildremoves what its own change did not need: a comment that restates the code, a guard for a case that cannot happen, a helper with one caller, an option nobody asked for, filler in prose.reviewlists the same things under a new "Slop" heading. Slop is not a finding and does not touch the gate.grokis a harness with its own action map and install steps.builder:<name>. On Cursor CLI, Grok Build and opencode a skill has its plain name, so the session start now says whatbuilder:<name>means there.planruns an architecture lens (skills/plan/architecture.md) andreviewchecks the change against the same questions. The project's own rules win over it. Tenet 9 now says what a good guard does.Proven
go test ./...,bash scripts/validate.shandbash tests/test_validate.shpass.PLUGIN_ROOTset the hook prints the Codex map, withGROK_PLUGIN_ROOTthe Grok Build map, with neither the Claude Code map.builder:<name>line of a map).debugfixture fails before any fix.Not proven
debugfixture is written and not graded.The design record is
docs/specs/2026-11-12-parity-design.md. Four of its rulings are proposed, not confirmed.🤖 Generated with Claude Code