Skip to content

Remove last weak AI-signal rules from default pack#26

Merged
benvinegar merged 1 commit intomainfrom
chore/remove-last-weak-rules
Apr 26, 2026
Merged

Remove last weak AI-signal rules from default pack#26
benvinegar merged 1 commit intomainfrom
chore/remove-last-weak-rules

Conversation

@benvinegar
Copy link
Copy Markdown
Member

Summary

  • remove defensive.async-noise and structure.duplicate-function-signatures from the default pack only
  • keep both rule implementations/docs in the repo, but stop counting them in default scans
  • update README and regression/config tests to match the slimmer default pack

Why

These were the last remaining weak default-pack candidates from the per-rule benchmark review. Both behave more like generic maintainability/lint smells than strong AI-slop defaults, and both underperform on the pinned AI-vs-mature-OSS benchmark.

Validation

  • bun test tests/heuristics.test.ts tests/fixtures-regression.test.ts tests/benchmark.test.ts tests/generic-record-casts.test.ts tests/config.test.ts
  • bun run lint

Published benchmark/report artifacts are unchanged in this PR.

This PR description was generated by Pi using GPT-5 Codex

@benvinegar benvinegar merged commit 326869c into main Apr 26, 2026
3 checks passed
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.

1 participant