Skip to content

tooling: stricter oxlint rules - #55

Merged
Octember merged 3 commits into
mainfrom
chore/stricter-oxlint
Aug 24, 2026
Merged

tooling: stricter oxlint rules#55
Octember merged 3 commits into
mainfrom
chore/stricter-oxlint

Conversation

@Octember

Copy link
Copy Markdown
Owner

Summary

  • Turn on nested-ternary, duplicate-import, import/first, no-use-before-define, prefer-destructuring, no-array-for-each, prefer-global-this, prefer-code-point, no-array-callback-reference, and inline type imports.
  • Fix the handful of sites those rules catch (merged imports, hoisted helpers, flattened status ternaries, for…of on the site).

Test plan

  • bun run lint
  • bun run fmt
  • bun run typecheck
  • bun test (44)

Made with Cursor

Inline type imports, merge duplicates, hoist helpers, and flatten nested ternaries so the stricter set is green.

Co-authored-by: Cursor <cursoragent@cursor.com>

@exe-dev-github-integration exe-dev-github-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM ✅

Named capture groups where we actually use the match; non-capturing otherwise. The rest of the new rules were already clean.

Co-authored-by: Cursor <cursoragent@cursor.com>

@exe-dev-github-integration exe-dev-github-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

still ✅

exactOptionalPropertyTypes, verbatimModuleSyntax, noImplicitOverride, and the leftover unused-label / unreachable / side-effect-import flags. Skip index-signature and erasableSyntaxOnly — they typecheck agent-tools source.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Octember
Octember merged commit d94300f into main Aug 24, 2026
2 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