Skip to content

Conversation

@wojtekmaj
Copy link
Contributor

@wojtekmaj wojtekmaj commented Feb 2, 2026

Due to a bug in vite-plus, every vite-plus command runs postinstall, consequently taking much longer than anticipated. The outcome of this is that commands like pnpm lint and perhaps even worse, git commit are about 4.5 seconds slower than they could. This PR fixes that by directly calling oxlint and oxfmt, circumventing the issue.

Copilot AI review requested due to automatic review settings February 2, 2026 22:28
@vercel
Copy link

vercel bot commented Feb 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs.npmx.dev Error Error Feb 2, 2026 10:36pm
npmx.dev Ready Ready Preview, Comment Feb 2, 2026 10:36pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Feb 2, 2026 10:36pm

Request Review

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the vite-plus dependency for linting and formatting operations to avoid a performance bug where postinstall scripts run on every vite-plus command. The change reduces command execution time by approximately 4.5 seconds.

Changes:

  • Replaced vite lint and vite fmt commands with direct calls to oxlint and oxfmt
  • Added explicit dependencies for oxlint and oxfmt packages
  • Updated lint-staged configuration to use the new command patterns
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Due to a bug in vite-plus, every vite-plus command runs postinstall, consequently taking much longer than anticipated. The outcome of this is that commands like `pnpm lint` and perhaps even worse, `git commit` are about 4.5 seconds slower than they could. This PR fixes that by directly calling oxlint and oxfmt, circumventing the issue.
@danielroe danielroe changed the title chore: use oxlint and oxfmt directly to avoid vite-plus bug chore: use oxlint and oxfmt directly Feb 2, 2026
@danielroe danielroe enabled auto-merge February 2, 2026 23:43
@danielroe danielroe added this pull request to the merge queue Feb 2, 2026
Merged via the queue into npmx-dev:main with commit 334baf7 Feb 2, 2026
13 of 21 checks passed
vinnymac pushed a commit to vinnymac/npmx.dev that referenced this pull request Feb 3, 2026
@wojtekmaj wojtekmaj deleted the faster-lint-staged branch February 3, 2026 09:08
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.

2 participants