| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
✅ Deploy Preview for viteplus-staging canceled.
|
Sorry, something went wrong.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Sorry, something went wrong.
Remove the 874-line hand-written oxlint-config.ts and import OxlintConfig directly from the oxlint package, which already exports the same type. Clean up related lint overrides and typos exclusions for the deleted file.
oxlint's TypeScript checker (tsgolint) does not support baseUrl in tsconfig.json, causing lint failures after migration. Guard both migration and lint --init paths to omit typeAware/typeCheck defaults when baseUrl is present.
When tsconfig.json has baseUrl and we skip injecting typeAware/typeCheck, log a warning with actionable advice to run `npx @andrewbranch/ts5to6 --fixBaseUrl .` to remove baseUrl. Added warning in both migration and lint --init paths. Updated snap test fixture with .oxlintrc.json to capture the warning output.
…hed package The static import of init-config.js in bin.ts transitively loads detect-indent (a devDependency), causing ERR_MODULE_NOT_FOUND when vite-plus is installed as a dependency. Change to dynamic import so it's only loaded for local commands that actually need it. Fixes frm-stack E2E test failure.
The previous lazy-import fix was insufficient — init-config.js is still loaded for all non-global commands. The actual root cause is that these packages were in devDependencies but are imported at runtime by tsc-compiled dist/utils/json.js. Move them to dependencies so they're available when vite-plus is installed in other projects.
…t-indent dependency chain Move hasBaseUrlInTsconfig from utils/json.ts (which imports detect-indent) to a new utils/tsconfig.ts that uses only Node.js builtins. This breaks the transitive dependency chain from init-config.ts → json.ts → detect-indent, allowing detect-indent/detect-newline to remain devDependencies since they are only used by rolldown-bundled global CLI code.
…nto vite.config.ts The $schema annotation in .oxlintrc.json/.oxfmtrc.json causes TS2769 type errors when merged into vite.config.ts because it's not part of OxlintConfig/OxfmtConfig types. Strip it before merging.
Merge activity |
Sorry, something went wrong.
The `--no-type-check` flag was removed from `vp check` in voidzero-dev#739, but the help examples section still referenced it. Replace with `--no-lint` which is a valid flag. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…#875) ## Summary - The `--no-type-check` flag was removed from `vp check` in #739, but the help examples section still referenced it - Replace with `--no-lint` which is a valid flag Fixes #879 One-line change in `crates/vite_global_cli/src/help.rs`. ## Context The Options section was correctly cleaned up in #739, but the Examples block ~18 lines below it was missed: ```diff - vp check --no-type-check src/index.ts + vp check --no-lint src/index.ts ``` --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: MK <fengmk2@gmail.com>
If tsconfig has comments, JSON.parse fails and we can't detect nor remove baseUrl! This PR resolves this. Related: #739
| Back | FazBrowse Home | New Git URL |
oxlint natively reads options.typeAware and options.typeCheck from
its config file, so there's no need for vp check to inject these as
CLI flags when calling vp lint.
vp lint --init, vp migrate, and project root vite.config.ts
config automatically)