chore(lint): tranzfer oxlint plugin catches stray re-exports and test debug markers#163

merged2
lint/2-oxlint-rules
lint/3-tranzfer-plugin
openedOct 13, 2026byAAviddaram
View on GitHubTwo-way sync enabledPR last imported 20d ago

Debug markers and redundant re-exports keep landing in PRs, and a convention in AGENTS.md does not stop them.

  • tools/oxlint/anti-slop/tranzfer/ is our own plugin, built like the vendored one.
  • tranzfer/no-import-then-reexport reports a module that imports a binding and also re-exports it from the same specifier, or imports it only to export { A }.
  • tranzfer/no-debug-markers-in-tests reports REPRO / REPRO1 identifiers, strings, template text and comments, plus console.log, in *.test.* and *.spec.*.
  • vitest/no-focused-tests is on for test files. The vitest plugin's other default rules are off because they misread @effect/vitest layer() blocks.
  • Each rule has a RuleTester file next to it, and vp run test runs them (31 cases).
  • AGENTS.md says vp check fails on warnings and where new rules go.

Existing hits: none for either rule, so there was nothing to fix.

vp check 0 errors 0 warnings, vp run test passed, web build ok.

Checks

4 successful · 2 skipped
All checks passed
fd24272

Added 1 commit

  • Add the tranzfer oxlint plugin with two rules; document where new rules godf83098
Synced from GitHub

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 59e26865-515d-472d-905d-cff6e594cd3b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Added 1 commit

  • Put the Auth and Mail doc comments back on what they describefd24272