vp migrate beta test#9
Conversation
There was a problem hiding this comment.
Code Review
This pull request integrates Vite+ into the project, updating VS Code settings, package configurations, and git hooks. The review feedback highlights three main areas of improvement: resolving potential conflicts or redundancies between the existing prek.toml hooks and the new Vite+ hooks, removing the temporary registry bridge in .npmrc before merging, and narrowing down the staged file pattern in vite.config.ts from * to specific code extensions to prevent running checks on non-code assets.
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.
| { id = "check", name = "vp check", entry = "vp check", language = "system", pass_filenames = false, priority = 10 }, | ||
| { id = "typecheck", name = "vp run typecheck", entry = "vp run typecheck", language = "system", pass_filenames = false, priority = 20 }, |
There was a problem hiding this comment.
Vite+ の導入に伴い .vite-hooks/pre-commit(vp staged)が追加されましたが、既存の prek.toml にも vp check と vp run typecheck が定義されています。\n\nもし Vite+ が Git フックを上書きして .vite-hooks/pre-commit のみを実行するようになる場合、prek.toml に定義されている typecheck が実行されなくなり、型エラーのあるコードがコミットされてしまう可能性があります。\n逆に、両方のフックツールが同時に動作する場合、vp check が重複して実行されるため非効率です。\n\n型チェック(typecheck)を vite.config.ts の staged タスクに移行するか、フックツールの統合方針(prek を使い続けるか、Vite+ のフック管理に一本化するか)を整理することを推奨します。
| @@ -0,0 +1,2 @@ | |||
| # pkg.pr.new registry bridge (added by test-pkg-pr-new-migrate.sh) | |||
| registry=https://pkg-pr-registry-bridge.void.app/ | |||
|
|
||
| export default defineConfig({ | ||
| staged: { | ||
| "*": "vp check --fix", |
There was a problem hiding this comment.
staged の対象として *(すべてのファイル)が指定されています。\n\nこの設定では、画像などのバイナリファイルやロックファイル、その他の非コード資産に対しても vp check が実行されてしまい、エラーが発生したり、コミット時の処理が不必要に遅くなったりする可能性があります。\n\n以下のように、対象とするファイルの拡張子(例: *.{ts,tsx,js,jsx,json,md})を明示的に指定することを推奨します。
| "*": "vp check --fix", | |
| "*.{js,jsx,ts,tsx,json,md}": "vp check --fix", |
No description provided.