Skip to content

fix(4261): Codex P1 — remove name attribution from holding rule#4264

Merged
AceHack merged 1 commit into
mainfrom
fix/4261-codex-p1-remove-aaron-name-attribution-from-rule-2026-05-18
May 18, 2026
Merged

fix(4261): Codex P1 — remove name attribution from holding rule#4264
AceHack merged 1 commit into
mainfrom
fix/4261-codex-p1-remove-aaron-name-attribution-from-rule-2026-05-18

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 18, 2026

Codex P1 finding on #4261: my rule edit introduced 'Aaron's PERSONAL INVARIANT' in a current-state .claude/rules surface, violating docs/AGENT-BEST-PRACTICES.md 'No name attribution in code, docs, or skills' convention (same project-rule that flagged the earlier #4231#4235 rule rename).

Fix: rephrase composes-with reference to use pathname only. Body now references the rule by filename without personal-name attribution.

…1 per docs/AGENT-BEST-PRACTICES.md 'No name attribution in code, docs, or skills' convention); rephrase composes-with reference to use pathname only — same project-convention catch as #4231#4235 rule rename
Copilot AI review requested due to automatic review settings May 18, 2026 23:45
@AceHack AceHack enabled auto-merge (squash) May 18, 2026 23:45
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

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

This PR removes personal-name attribution from a current-state Claude rule by replacing the link label with the target rule filename.

Changes:

  • Rephrases the “Composes with” reference to use the rule pathname instead of a personal-name label.

@AceHack AceHack merged commit e8d7dc3 into main May 18, 2026
28 of 30 checks passed
@AceHack AceHack deleted the fix/4261-codex-p1-remove-aaron-name-attribution-from-rule-2026-05-18 branch May 18, 2026 23:47
AceHack added a commit that referenced this pull request May 18, 2026
…in rule body); fix via #4264 same pattern as #4231#4235 rule rename; convention violation caught twice today → internalize (#4265)
AceHack added a commit that referenced this pull request May 18, 2026
… + allow #6 next; deeper saturation pattern (cycle-of-cycles meta-decomposition same-shape) emerging but holding (#4269)
AceHack added a commit that referenced this pull request May 18, 2026
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