Skip to content

feat(oxc): lint results inspecteor#432

Draft
yuyinws wants to merge 6 commits into
mainfrom
chore/clean-code
Draft

feat(oxc): lint results inspecteor#432
yuyinws wants to merge 6 commits into
mainfrom
chore/clean-code

Conversation

@yuyinws

@yuyinws yuyinws commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Close #433

@yuyinws yuyinws changed the title chore/clean code chore: clean code Jul 16, 2026
@pkg-pr-new

pkg-pr-new Bot commented Jul 16, 2026

Copy link
Copy Markdown

Open in StackBlitz

@vitejs/devtools

npm i https://pkg.pr.new/vitejs/devtools/@vitejs/devtools@432

@vitejs/devtools-kit

npm i https://pkg.pr.new/vitejs/devtools/@vitejs/devtools-kit@432

@vitejs/devtools-oxc

npm i https://pkg.pr.new/vitejs/devtools/@vitejs/devtools-oxc@432

@vitejs/devtools-rolldown

npm i https://pkg.pr.new/vitejs/devtools/@vitejs/devtools-rolldown@432

@vitejs/devtools-vite

npm i https://pkg.pr.new/vitejs/devtools/@vitejs/devtools-vite@432

@vitejs/devtools-vitest

npm i https://pkg.pr.new/vitejs/devtools/@vitejs/devtools-vitest@432

commit: 4583b98

@yuyinws
yuyinws marked this pull request as ready for review July 16, 2026 09:01
@yuyinws yuyinws changed the title chore: clean code refactor(oxc): remove lint log cli Jul 16, 2026
Comment thread packages/oxc/package.json
"@clack/prompts": "catalog:inlined",
"@vitejs/devtools-kit": "workspace:*",
"ansis": "catalog:deps",
"cac": "catalog:deps",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am actually ok with keeping gunshi for a bit of diversity

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

cac is more lightweight than gunshi. Given that the current CLI is relatively simple, I think it's a better fit for now. We can consider switching to gunshi if the CLI grows more complex in the future.

@antfu

antfu commented Jul 17, 2026

Copy link
Copy Markdown
Member
CleanShot 2026-07-17 at 11 39 24@2x I am seeing this - should we have a button to run/rerun the lint? Do we still need the concept of sessions or it's always the live result?

@yuyinws

yuyinws commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator Author

The complete implementation of the lint inspector will be submitted in a separate PR.

The current proposal is:

  1. Trigger linting via a button.
  2. Persist lint results to local files to enable historical snapshots.
  3. Allow users to delete lint history directly from the UI.

@antfu

antfu commented Jul 17, 2026

Copy link
Copy Markdown
Member

Would you like to directly implement it in this PR? Right now this PR only remove the cli which makes this devtools' feature incomplete

@yuyinws
yuyinws marked this pull request as draft July 17, 2026 03:33
@yuyinws yuyinws changed the title refactor(oxc): remove lint log cli feat(oxc): lint results inspecteor Jul 17, 2026
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.

feat(oxc): lint results inspector

2 participants