Vp migrate test#1
Conversation
There was a problem hiding this comment.
Code Review
This pull request integrates Vite+ as the unified toolchain for the project, replacing standalone oxfmt and oxlint tools. It updates package scripts, pre-commit hooks, VS Code settings, and adds documentation for the new toolchain. The feedback recommends formatting the single-line lint configuration in vite.config.ts into a multi-line object literal to improve readability.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| "*": "vp check --fix" | ||
| }, | ||
| fmt: {}, | ||
| lint: {"jsPlugins":[{"name":"vite-plus","specifier":"vite-plus/oxlint-plugin"}],"rules":{"vite-plus/prefer-vite-plus-imports":"error"},"options":{"typeAware":true,"typeCheck":true}}, |
There was a problem hiding this comment.
The lint configuration object is currently written as a single-line JSON-like string, which is difficult to read and maintain. It should be formatted as a standard, multi-line TypeScript object literal.
lint: {
jsPlugins: [
{
name: 'vite-plus',
specifier: 'vite-plus/oxlint-plugin'
}
],
rules: {
'vite-plus/prefer-vite-plus-imports': 'error'
},
options: {
typeAware: true,
typeCheck: true
}
},
No description provided.