Skip to content

fix: resolve all oxlint warnings across the monorepo#381

Open
tlgimenes wants to merge 2 commits intomainfrom
tlgimenes/fix-lint-issues
Open

fix: resolve all oxlint warnings across the monorepo#381
tlgimenes wants to merge 2 commits intomainfrom
tlgimenes/fix-lint-issues

Conversation

@tlgimenes
Copy link
Copy Markdown
Contributor

@tlgimenes tlgimenes commented Apr 15, 2026

Summary

  • Fixed all 38 oxlint warnings across 23 files in the monorepo (unused variables/imports/parameters, useless try/catch wrappers, unnecessary spreads, empty destructuring patterns, useless escapes)
  • Prefixed intentionally unused parameters with _, removed truly unused declarations, unwrapped no-op try/catch blocks, and added inline suppression for intentional control-character regexes in content-scraper
  • Lint now passes cleanly with 0 warnings and 0 errors

Test plan

  • bun run lint passes with 0 warnings and 0 errors
  • bun run fmt passes (pre-commit hook verified)

🤖 Generated with Claude Code


Summary by cubic

Clears all oxlint warnings across the monorepo and silences TypeScript 5.9 baseUrl deprecation to unblock CI. No functional changes intended; lint and TS checks now pass cleanly.

  • Refactors

    • Prefixed intentionally unused params with _; removed unused imports/types/vars.
    • Simplified error handling and removed defensive/no-op code in loggers and tools across discord-read, slack-mcp, multi-channel-inbox, registry, vtex, and virtual-try-on.
    • Added targeted oxlint-disable-next-line no-control-regex in content-scraper sanitization; fixed VTEX masked-card regex.
  • Bug Fixes

    • Added "ignoreDeprecations": "5.0" to tsconfig.json in content-scraper, hyperdx, multi-channel-inbox, registry, and virtual-try-on to silence TS5101 (baseUrl) in TS 5.9 and pass CI.

Written for commit 36bf7be. Summary will update on new commits.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 23 files

Add ignoreDeprecations: "5.0" to tsconfig.json for 5 MCPs that use
baseUrl for path aliases, fixing the TS5101 CI check failures.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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