Skip to content

fix(rules): correct 2 real stale pointers surfaced by audit-rule-cross-refs#3558

Merged
AceHack merged 1 commit into
mainfrom
fix/rules-stale-pointers-naming-expert-wwjd-keeps-grey-otto-cli-2026-05-15
May 15, 2026
Merged

fix(rules): correct 2 real stale pointers surfaced by audit-rule-cross-refs#3558
AceHack merged 1 commit into
mainfrom
fix/rules-stale-pointers-naming-expert-wwjd-keeps-grey-otto-cli-2026-05-15

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 15, 2026

Summary

Surfaced via `bun tools/hygiene/audit-rule-cross-refs.ts` — the sibling at-substrate-scope of B-0533's lint. Two genuine stale pointers; both classifiable via the 9-variant taxonomy in tick 1920Z.

Fix 1 — `m-acc-multi-oracle-end-user-moral-invariants.md`

`.claude/rules/naming-expert.md` → `.claude/skills/naming-expert/SKILL.md`

The real artifact is a SKILL, not a rule. 9-variant taxonomy: alternative-location.

Fix 2 — `default-to-both.md`

Two refs to `memory/feedback_aaron_wwjd_keeps_the_grey_in_aaron_honest_devil_lives_in_the_grey_in_numbers_2026_05_12.md` removed. The file never existed:

  • `ls memory/ | grep keeps_the_grey` → no matches
  • `git log --all -- "keeps_the_grey"` → empty

WWJD-as-discipline-grounding content preserved via the `wwjd_tedium` segment of the kestrel autonomous-arrival memo (already cited in same section).

9-variant taxonomy: anti-pattern (rule referenced aspirational content that never materialized).

Test plan

  • 2 files, 4 line-edits (4 insertions, 8 deletions)
  • Verification: file existence checked via `ls` + git log archaeology
  • CI green
  • Auto-merge arms

🤖 Generated with Claude Code

…s-refs

Surfaced via `bun tools/hygiene/audit-rule-cross-refs.ts` (the sibling
of B-0533's lint at substrate scope).

Fix 1 — `.claude/rules/m-acc-multi-oracle-end-user-moral-invariants.md`:

  The rule cited `.claude/rules/naming-expert.md` but the real artifact
  is a SKILL at `.claude/skills/naming-expert/SKILL.md`. Updated path.

Fix 2 — `.claude/rules/default-to-both.md`:

  Two references to a file that never existed:
  `memory/feedback_aaron_wwjd_keeps_the_grey_in_aaron_honest_devil_lives_in_the_grey_in_numbers_2026_05_12.md`
  Verified via `ls memory/ | grep keeps_the_grey` (no matches) +
  `git log --all -- "*keeps_the_grey*"` (empty). Removed both
  references; the WWJD-as-discipline-grounding content is preserved
  via the wwjd_tedium segment of the kestrel autonomous-arrival memo
  (also cited in same section).

The first WWJD reference (line 143) also gained a `memory/` prefix
for repo-root-relative consistency with other refs in the same rule.

Both classes of stale-pointer caught by the audit-rule-cross-refs
9-variant taxonomy:
- Fix 1: alternative-location variant (correct file exists at different path)
- Fix 2: anti-pattern variant (file never existed; rule referenced aspirational content)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 15, 2026 19:07
@AceHack AceHack enabled auto-merge (squash) May 15, 2026 19:07
@AceHack AceHack merged commit f656f4e into main May 15, 2026
23 checks passed
@AceHack AceHack deleted the fix/rules-stale-pointers-naming-expert-wwjd-keeps-grey-otto-cli-2026-05-15 branch May 15, 2026 19:09
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes two stale cross-references surfaced by tools/hygiene/audit-rule-cross-refs.ts, keeping .claude/rules/** pointers aligned with the current repo layout.

Changes:

  • Update an m/acc rule cross-ref from a non-existent rule path to the actual naming-expert SKILL location.
  • Remove references in default-to-both.md to a memory file that never existed, and consolidate WWJD grounding to an existing cited memo.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.claude/rules/m-acc-multi-oracle-end-user-moral-invariants.md Updates cross-ref to naming-expert from rules path to skills path.
.claude/rules/default-to-both.md Removes nonexistent memory reference and keeps WWJD grounding via an existing memory citation.

Comment on lines +134 to +135
- `.claude/skills/naming-expert/SKILL.md` convention applies if m/acc
goes public-surface (Ilyana review)
- `.claude/rules/naming-expert.md` convention applies if m/acc goes
public-surface (Ilyana review)
- `.claude/skills/naming-expert/SKILL.md` convention applies if m/acc
goes public-surface (Ilyana review)
AceHack added a commit that referenced this pull request May 15, 2026
…ters; fixed in PR #3558 (#3559)

- PR #3557 (1855Z) merged
- Ran audit-rule-cross-refs.ts: 48 candidates → 2 genuine bugs (9-variant taxonomy)
- Fix 1: naming-expert.md path (rule → skill location)
- Fix 2: removed 2 refs to file that never existed
- 10-tick parallel-substantive arc continues

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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