refactor: two craft skills stop explaining themselves to the reader - #123
Merged
Conversation
A reading of the craft tier with `de-slop` and `compressed-deliberation` open found each file spending about half its words on its own provenance and its own limits. Neither skill loses a rule here. What moved, moved to the source record beside the matrix, where the audit reader looks and the installed tree does not reach. `de-slop` keeps one clause of the reason it ships no word list, and the repository-governance case behind that decision goes to `source/craft/de-slop.md`. `What stands behind this shape` merges into `Boundary`, which already stated the same no-standard proposition. `compressed-deliberation` loses the seven bullets that described Claude Opus 5 and keeps the sentence a reader acts on. Those bullets move to `source/craft/compressed-deliberation.md`, beside the URLs, sections and read dates that already held them. So the matrix carries no `G` row, and it drops its `Source version` line, because ADR-0026 refuses that line in a matrix that cites nothing. Two headings over two bullets each become labelled bullets under `The shape of a reply`. Both matrices renumber in document order, and three files that named the retired `G` rows or the retired `SOURCE.md` now name what stands there.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 741446c8a4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Three review findings on PR #123. The retained evidence sentence claimed that files written to disk are the case the vendor documents most clearly, and attributed that to the source record. The record never said it. Nobody here has re-read `A1` to settle it, and no `G` row could carry it either, because the claim judges coverage across pages rather than citing a rule, so the `Source rule` cell would name an identifier that says no such thing. The clause leaves the installed skill and stands in the source record as unverified. What the skill keeps is its own claim, which is that these two cases are the ones this repository has tested least. The reproduction record said every row reads `unquoted`, which now describes no row. It states the absence of `G` rows instead. The folded conceding bullet dropped `and then stop`. That is a terminal condition and not a restatement, so it is back.
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.
These cuts execute an in-session slop review of this suite's own prose, read
with
de-slopandcompressed-deliberationopen, which found each of these twofiles spending about half its words on its own provenance and its own limits.
No issue records the review, so the findings it acted on are listed below.
wc -won the two skill files:skills/craft/de-slop/SKILL.mdskills/craft/compressed-deliberation/SKILL.mdThe review's own targets were about 650 and about 470. The enumerated findings
below do not reach them, and nothing beyond them was cut, so both files land
short of the target by roughly a hundred words. A second pass is a separate
decision.
What the review found, and what this does about it
de-slop:No part of this matches a wordspent 120 words arguing a repositorydecision. It keeps
This skill ships no word list.and one clause of reason.The rest is in
source/craft/de-slop.md.What stands behind this shapeandBoundarystated the same no-standardproposition twice. They are one
Boundarysection now.So this skill is the generative half on its own.restated the two sentencesabove it, and is gone.
One test settles most of them.announced that a point mattered, which isthe shape row
E-11of this skill's own matrix names as a defect. Theparagraph starts at
Remove the sentenceinstead.How this differs from the other skills herekeeps the repetitiondisagreement and the scope claims. The paragraph that duplicated
Boundarymerges into the first clause of the one that follows it, because
Both standardsneeds its antecedent.compressed-deliberation:What this correctsspent 211 words, seven of its bullets describing ClaudeOpus 5. The description moves to
source/craft/compressed-deliberation.md.The skill keeps the operative sentence, which is that the rules aim at the
whole stack rather than at the model alone.
It leads with
Apply the rules ... on their reasoning and not on our evidencenow.Conceding a correctionandSurfacing a decisionwere headings over twobullets each. They are labelled bullets under
The shape of a reply.The G-row deletion is the operator-approved decision here
Moving the seven vendor bullets out of the skill deletes all seven
Growsfrom
grounding/craft/compressed-deliberation.md. The operator approved thatdirectly. Three consequences, all of them mechanical once the rows go:
**Source version:**line. ADR-0026 refuses that linein a matrix with no
Grow, andground --checkenforces it by name.AuditedandSource textcells go empty throughout, which is the stategrounding/craft/de-slop.mdalready carries.**Quotation:** forbiddendeclaration stays, with its reason rewritten,because the pages it protects are still the pages behind this skill.
The provenance does not go anywhere.
source/craft/compressed-deliberation.mdcarries the seven statements whole, each beside the
A1,A3orA5sectionit came from, and it already held the URLs and the read dates. No ADR is added,
because no rule changed: ADR-0026 already decides the pin, ADR-0020 already
decides the declaration, and the skill still names its model build in
Purpose.Propagation
Three files named things that no longer exist, and now name what stands there:
grounding/craft/subagent-returns.mdcited the sevenGrows,source/craft/proportionate-execution.mdsaid the skill cites the vendorpages, and the
E-16row ingrounding/craft/de-slop.mdstill pointed at theretired
SOURCE.md. Front-matter descriptions are untouched. No editorialstamp and no audit date was written.
Checks
npm run checkexits 0. All nine: 901 tests pass,lint:docsclean,ground --check --allclean, docs metadata clean over 37 documents, probes,verdicts, resident, studies and editorial all clean.