docs: de-slop the org-wide prose and drop every ai-slop exclusion - #130
Merged
Merged
Conversation
Repo-wide /ai-slop:audit fix pass over the ten tracked markdown files (CODE_OF_CONDUCT.md excluded by config as verbatim Contributor Covenant). The deterministic detector returned zero findings before and after; every change below is a judgment-rubric finding, each confirmed by two blind readers and passed through a blind semantic-diff check. - profile/README.md: replace the two mechanism-free "How we work" claims with what actually happens (standards states the bar as configs and conventions; repos enforce it by calling the same SHA-pinned actions). - SECURITY.md: name the channel the acknowledgement and progress updates arrive on (the advisory thread, or email reply), without adding a response-time commitment the policy never made. - .claude/source-control.md and README.md: drop the "seam" metaphor in favour of the plain "layered config" / "convention"; name the two files that actually resolve a *.local.* overlay instead of "config surfaces". - .github/PULL_REQUEST_TEMPLATE.md: rewrite the Verification hint so it names the two kinds of evidence instead of a three-beat cadence whose middle item entailed its first. Section headings are unchanged. .claude/rules/pr-body-contract.md was judged clean and is sync-managed from standards, so it was not edited. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01E6FUnft3N2ZDycTtDLbo2g
The ai-slop config had excluded CODE_OF_CONDUCT.md outright and exempted seven files from the em-dash rule. Both lists are now empty, with the config comment stating the standing rule: only text this repository does not own may be excluded, and nothing currently qualifies. Every finding the exclusions had hidden is fixed here, each rewrite passed through a blind semantic-diff check, with two hunks reverted or revised when the check found a dropped category or a drifted referent. - Em dashes (30 across CLAUDE.md, README.md, profile/README.md, SUPPORT.md, GOVERNANCE.md, CONTRIBUTING.md, PULL_REQUEST_TEMPLATE.md): bold list labels take a colon; appositive dashes become a comma pair, a sentence break, or a restructured sentence. No en dashes or parentheses were substituted. - CODE_OF_CONDUCT.md: restate the "not just for us, but for the community" line without the construction; split the enforcement sentence; name community leaders as the actor who reviews complaints; say the correction-tier warning explains what the violation was. The pledge's "diverse, inclusive" adjectives and the "opinions, viewpoints, and experiences" triad stay, because dropping either loses a commitment. The Contributor Covenant attribution still holds; the text was already declared "adapted from" v2.1 under CC BY 4.0. - Re-examined first-pass declines: README's opening colon chain and the dependabot "it" referent, GOVERNANCE's title-restating opener, the profile page's "consistent ... contributions" tail, and SUPPORT's "provided templates". .claude/rules/pr-body-contract.md is sync-managed from standards and was judged clean, so nothing is filed upstream. The PR template's comment prose is authored here (standards' policy.json carries only section names, keywords, and markers), so its fixes stay local. Template headings and the source-control section list are byte-identical. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01E6FUnft3N2ZDycTtDLbo2g
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
PR body contract — issue linkage This PR body conforms to the issue-linkage contract. Nothing to do. |
kyle-sexton
changed the title
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
2 participants
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.
No related issue: repo-wide prose hygiene pass requested directly; it changes only the tracked markdown and the ai-slop config, with no tracked work item behind it.
Summary
A repo-wide
/ai-slop:audit fixover every tracked markdown file. The first commit applies the judgment-rubric findings (mechanism-free claims, metaphor jargon, one rhythm triad). The second removes both exclusion lists from.claude/ai-slop.jsonon the standing rule that only text this repository does not own may be excluded, and fixes everything those exclusions had hidden: 30 em dashes across seven files and the Code of Conduct, which had been excluded outright.Two files are deliberately not edited.
.claude/rules/pr-body-contract.mdis sync-managed fromstandardsand was judged clean by two independent readers, so nothing is filed upstream. The PR template's comment prose is authored here (the standards policy file carries only section names, keywords, and markers, and the lockstep check reads headings and two marker strings), so its fixes stay local; its##headings are byte-identical.Fix
.claude/ai-slop.json:excluded_pathsandem_dash_allowed_pathsare now empty; the comment states the rule and why nothing qualifies.profile/README.md: the two "How we work" bullets now state the mechanism (standards states the bar as configs and conventions; repos enforce it by calling the same SHA-pinned actions); thestandardsbullet's purpose tail replaced with the same fact; the marketplaces bullet restructured so its colon introduces the marketplaces.SECURITY.md: the acknowledgement promise names the channel updates arrive on; no response-time commitment added.CODE_OF_CONDUCT.md: the "not just for us, but for the community" line restated; the enforcement sentence split; community leaders named as the actor who reviews complaints; the correction-tier warning "explains what the violation was". The pledge's "diverse, inclusive" and the "opinions, viewpoints, and experiences" triad stay, because dropping either loses a commitment. Attribution already read "adapted from" v2.1 under CC BY 4.0.README.md,CLAUDE.md,.claude/source-control.md,GOVERNANCE.md,SUPPORT.md,CONTRIBUTING.md,.github/PULL_REQUEST_TEMPLATE.md: plain-word replacements for "seam" and "config surfaces", an ambiguous "it" resolved to "that pin", a title-restating opener cut, and the Verification hint rewritten as the two kinds of evidence it meant.Verification
detect.sh) over all 11 tracked markdown files: 0 findings, 0 declined, no exclusions in effect (was 31 findings once exclusions were removed)..cursor/check.sh): markdownlint, typos, editorconfig, gitleaks, lychee, actionlint, jsonschema, shellcheck, and eol-renormalize pass. Thepr-section-driftlane's unit tests pass; its live fetch returned HTTP 401 from the sandbox, so its two invariants were checked by diffing the template##headings and thepr_body_required_sectionslist againstmain(both identical).Related
melodic-software/standardscomponents/pr-convention-policy/policy.jsonandlockstep-drift.mjs, consulted to establish that the PR template's prose is owned here..work/reviews/directory of the session checkout.🤖 Generated with Claude Code
https://claude.ai/code/session_01E6FUnft3N2ZDycTtDLbo2g