Skip to content

Expand oxlint rules from 7 to 17#29

Merged
gricha merged 1 commit intomainfrom
feat/expand-oxlint-rules
Jan 7, 2026
Merged

Expand oxlint rules from 7 to 17#29
gricha merged 1 commit intomainfrom
feat/expand-oxlint-rules

Conversation

@gricha
Copy link
Copy Markdown
Owner

@gricha gricha commented Jan 7, 2026

Summary

  • Expand oxlint configuration from 7 rules to 17 rules
  • Add rules that catch common bugs: dead code, numeric precision, switch fallthrough, etc.
  • No violations found in current codebase

New rules added

  • no-self-compare - catches x === x mistakes
  • no-constant-condition - catches infinite loops
  • no-unreachable - catches dead code
  • no-loss-of-precision - catches numeric bugs
  • no-debugger - catches leftover debuggers
  • no-duplicate-case - catches switch statement bugs
  • no-fallthrough - catches missing breaks
  • no-unsafe-finally - catches finally block issues
  • no-sparse-arrays - catches array hole bugs
  • no-template-curly-in-string - catches template literal typos
  • valid-typeof - catches typeof comparison bugs

Test plan

  • bun run lint passes with no violations
  • bun run validate passes all tests

🤖 Generated with Claude Code

Add additional lint rules to catch common bugs:
- no-self-compare: catches x === x mistakes
- no-constant-condition: catches infinite loops
- no-unreachable: catches dead code
- no-loss-of-precision: catches numeric bugs
- no-debugger: catches leftover debuggers
- no-duplicate-case: catches switch statement bugs
- no-fallthrough: catches missing breaks
- no-unsafe-finally: catches finally block issues
- no-sparse-arrays: catches array hole bugs
- no-template-curly-in-string: catches template literal typos
- valid-typeof: catches typeof comparison bugs

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@gricha gricha merged commit 6fecf67 into main Jan 7, 2026
6 checks passed
@gricha gricha deleted the feat/expand-oxlint-rules branch January 7, 2026 17:57
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