Skip to content

docs(standards): drop the duplicate 'fix the class' bullet - #230

Merged
LukasWodka merged 1 commit into
developfrom
fix/dedupe-fix-the-class-bullet
Aug 12, 2026
Merged

docs(standards): drop the duplicate 'fix the class' bullet#230
LukasWodka merged 1 commit into
developfrom
fix/dedupe-fix-the-class-bullet

Conversation

@LukasWodka

@LukasWodkaLukasWodka commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Why

Bugbot flagged this on the develop→staging mirror #226, and it blocked the staging hop (NOT merged — Bugbot flagged 1 open finding: Duplicate standards bullet landed).

#220 and #224 both landed the same rule one minute apart (21:08 and 21:09), so org-standards.md now carries two near-identical "Fix the class, not the instance" bullets. Since this file is the synced canon, the duplicate would propagate into every repo's CLAUDE.md.

What

Removes the second bullet, keeping the line-25 wording.

Why that one:

  • Every other bullet in the file uses an unbolded lead-in — the removed one was the only bullet starting with bold.
  • It sat between the "stay on the PR" and "a finding that recurs" bullets, splitting that flow; the survivor follows "AI sessions end with evidence", which reads naturally.

Test plan

occurrences of the rule : 1 (was 2)
bullets starting with bold: 0 (was 1)

Note

There's an irony worth recording: the rule that got duplicated is "fix the class, not the instance", and it got duplicated because two drafts of the same fix both landed. That's the duplicate-work failure mode the rule is partly aimed at — a good argument for the claim-before-you-open convention in the hardening epic.

🤖 Generated with Claude Code


Note

Low Risk
Docs-only cleanup of a duplicate standards bullet; no code, auth, or behavioral changes.

Overview
Removes the second, near-identical "Fix the class, not the instance" bullet from the Quality bar section in org-standards.md, keeping the earlier wording so the synced canon no longer duplicates the rule.

Reviewed by Cursor Bugbot for commit 1745a1a. Bugbot is set up for automated code reviews on this repo. Configure here.

#220 and #224 both landed the same rule one minute apart, so org-standards.md
carried two near-identical 'Fix the class, not the instance' bullets. Because
this file is the synced canon, the duplicate would propagate into every repo's
CLAUDE.md (Bugbot, on the develop->staging mirror #226, which it blocked).
Keeps the line-25 wording: the neighbouring bullets all use unbolded lead-ins,
and the removed one was the only bullet in the file starting with bold. It also
sat between the post-PR and recurring-finding bullets, splitting that flow.
Verified: one occurrence of the rule remains, zero bullets start with bold.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@LukasWodka
LukasWodka merged commit 183f3ce into developAug 12, 2026
12 of 13 checks passed
@LukasWodka
LukasWodka deleted the fix/dedupe-fix-the-class-bullet branch August 14, 2026 13:53
Sign up for freeto 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.

1 participant

@LukasWodka