feat(coding-agent): walk one case step by step in explain-until-understood - #81
Merged
Conversation
…stood Three observed gaps in how the skill lands with a real reader: - A worked example beat a general description every time, but the skill only reached for numbered steps as ladder attempt 2, after a miss. Promoted to step 5, placed after the picture so the diagram is the map and the walk is one journey across it; old steps 5-7 become 6-8. - "Assume the reader knows nothing" left project-local vocabulary uncovered — the repo, the surrounding work, an internal acronym. - "Draw in ASCII" never said which characters, so box-drawing and arrow glyphs passed. A failed glyph breaks the picture exactly where the picture carries the meaning. Making a walk the default collides with the escalation ladder, whose attempt 2 is numbered steps: disambiguated in prose rather than by deleting the rung, which was the only step between prose and pictures. Also drops the plugin README claim that model auto-invocation is disabled for this skill; that stopped being true in c626f88. Generated with [Claude Code](https://claude.com/claude-code) by CEH
The skill required "one new idea per sentence" and was itself written in eight-to-thirteen-line paragraphs carrying four rules each. Every procedure step now opens with one bold claim and drops its qualifiers into sub-bullets, which is the form the skill tells you to use. No rule changed. Verified by word-diff: every deleted phrase reappears in the rewritten sentence that replaced it. The Rules section is a shared block with explain-codebase per docs/CROSS_REFERENCES.md and was left byte-identical. Longest unbroken prose block drops from 13 lines (step 1) to 5; the 13-line block that remains is the anti-pattern bullet list. No version bump: 3.1.2 is unreleased and already covers this branch. Generated with [Claude Code](https://claude.com/claude-code) by CEH
Ships a new procedure step for explain-until-understood: walk one real input through the mechanism, one hop per numbered line, as the default form rather than the ladder-attempt-2 recovery it used to be. Alongside it, the reader floor becomes someone on their first day on the project, the ASCII charset for diagrams is spelled out so box-drawing and arrow glyphs stop passing, and the skill body is rewritten against its own "one new idea per sentence" rule. - Bump: PATCH — content and description changes only; no new skill, agent, or dependency edge, so neither the plugin nor the repo tag earns MINOR. - Manifests: plugins/ceh-coding-agent/.claude-plugin/plugin.json and .claude-plugin/marketplace.json, 3.1.1 -> 3.1.2 (landed in 4c37e28). - Docs: CHANGELOG.md v5.0.2 entry added here; root README and plugins/ceh-coding-agent/README.md rows updated in 4c37e28; CLAUDE.md needs no update — the skill count and the 6-of-77 flagged-skill count are both unchanged. Generated with [Claude Code](https://claude.com/claude-code) by CEH
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.
What
explain-until-understoodgets a new procedure step — walk one real input through the mechanism,one hop per numbered line — plus a first-day reader floor, a spelled-out ASCII charset for diagrams,
and a readability rewrite of the whole body. Released as repo tag v5.0.2 /
ceh-coding-agent3.1.2.Why
Three observations from using the skill:
steps as ladder attempt 2 — a recovery after an explanation had already missed.
repo, the work that led here, an internal acronym everyone says out loud.
that fails to render breaks the picture exactly where the picture carries the meaning.
The skill also required "one new idea per sentence" while being written in eight-to-thirteen-line
paragraphs carrying four rules each.
How
The walk becomes step 5, placed after the drawing step so the picture is the map and the walk is
one journey across it; old steps 5-7 shift to 6-8 and all internal step references were re-checked.
The non-obvious decision is the collision with the escalation ladder, whose attempt 2 is numbered
steps: making a walk the default means attempt 2 is no longer a change of representation. Rejected
deleting that rung — it was the only step between prose and pictures. Disambiguated in prose
instead: a numbered walk inside prose is still attempt 1; attempt 2 is the whole explanation
rebuilt as steps.
The density pass restructures each procedure step as one bold claim plus sub-bullets. The
Rulessection is a shared block with
explain-codebaseperdocs/CROSS_REFERENCES.mdand was leftbyte-identical.
Testing
python tools/validate-plugins/validate.py— green (this is the CI gate,validate.yml).git diff --word-difffiltered todeletions: every removed phrase reappears in the sentence that replaced it.
yaml.safe_loadto confirm the folded>-description still folds to one string.ceh-evaluationwouldmeasure that and was not run.
Checklist
any/@ts-ignore/# type: ignoreintroduced.agents_workspace/DECISION_LOG.mdentry 84Generated with Claude Code by CEH