Skip to content

chore(ai-slop): drop stale line-number citation from config comment - #107

Merged
kyle-sexton merged 1 commit into
mainfrom
claude/repo-code-tidying-simplify-62ck98
Aug 30, 2026
Merged

chore(ai-slop): drop stale line-number citation from config comment#107
kyle-sexton merged 1 commit into
mainfrom
claude/repo-code-tidying-simplify-62ck98

Conversation

@kyle-sexton

Copy link
Copy Markdown
Contributor

No related issue: behavior-preserving drift fix from a repo-wide simplification sweep.

Summary

A repo-mode /code-tidying:batch-simplify sweep covered all 33 files (9 survived the code-file filter across two groups: root/agent-adjacent config and GitHub templates/automation YAML). The single applied change: .claude/ai-slop.json's _comment cited "CLAUDE.md lines 15 and 24" for its reviewed appositive em dashes, but CLAUDE.md was rewritten after that comment was written (its em dashes now sit on lines 9, 15, 22, and 25), leaving the citation stale. Every other swept file was reviewed end to end with no changes needed.

Fix

Drop the line-number citation ("in CLAUDE.md lines 15 and 24," becomes "in CLAUDE.md,") rather than renumbering it, so it cannot re-stale on the next CLAUDE.md edit. The rest of the _comment and both semantic path arrays (excluded_paths, em_dash_allowed_paths) are byte-identical to main.

Verification

  • JSON validity via python3 -m json.tool; excluded_paths and em_dash_allowed_paths parse identical to main.
  • A fresh-context refutation pass ran the ai-slop plugin's detect.sh before/after in sandbox repos: byte-identical audit output; the plugin reads only semantic keys via targeted jq paths, never _comment.
  • typos, gitleaks (80 commits, no leaks), and editorconfig-checker clean repo-wide against this repo's own configs.
  • check-jsonschema against vendor.dependabot, vendor.github-issue-forms (3 forms), and vendor.github-issue-config: all pass (reviewed-unchanged Group 2 files).

Related

🤖 Generated with Claude Code

https://claude.ai/code/session_01N99bwGABw714WpH99BhhtA


Generated by Claude Code

Repo-mode batch-simplify wave 1 (groups: root/agent-adjacent config;
GitHub templates & automation YAML). The one applied change: the
_comment in .claude/ai-slop.json cited "CLAUDE.md lines 15 and 24" for
its reviewed appositive em dashes, but CLAUDE.md was rewritten after
that comment was written and its em dashes now sit on other lines. The
citation is dropped rather than renumbered so it cannot re-stale; the
rest of the comment and both path arrays are byte-identical. All other
swept files: reviewed, no changes.

Verified: JSON validity; byte-identical excluded_paths and
em_dash_allowed_paths vs HEAD; a fresh-context refutation pass ran the
ai-slop detector before/after with byte-identical output (the plugin
reads only the semantic keys, never _comment); typos, gitleaks, and
editorconfig-checker clean repo-wide; check-jsonschema against
vendor.dependabot, vendor.github-issue-forms, and
vendor.github-issue-config all pass.

No related issue: behavior-preserving drift fix from a repo-wide
simplification sweep.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01N99bwGABw714WpH99BhhtA
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 30, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-08-30T18:25:21.662844Z c421453 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@kyle-sexton
kyle-sexton merged commit 046c401 into main Aug 30, 2026
14 checks passed
@kyle-sexton
kyle-sexton deleted the claude/repo-code-tidying-simplify-62ck98 branch August 30, 2026 18:25
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.

2 participants