Skip to content

vp migrate beta test#9

Draft
fengmk2 wants to merge 1 commit into
developfrom
vp-migrate-test
Draft

vp migrate beta test#9
fengmk2 wants to merge 1 commit into
developfrom
vp-migrate-test

Conversation

@fengmk2

@fengmk2 fengmk2 commented Jun 29, 2026

Copy link
Copy Markdown
Owner

No description provided.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Comment thread prek.toml
Comment on lines +19 to +20
{ 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 },

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

Vite+ の導入に伴い .vite-hooks/pre-commitvp staged)が追加されましたが、既存の prek.toml にも vp checkvp run typecheck が定義されています。\n\nもし Vite+ が Git フックを上書きして .vite-hooks/pre-commit のみを実行するようになる場合、prek.toml に定義されている typecheck が実行されなくなり、型エラーのあるコードがコミットされてしまう可能性があります。\n逆に、両方のフックツールが同時に動作する場合、vp check が重複して実行されるため非効率です。\n\n型チェック(typecheck)を vite.config.tsstaged タスクに移行するか、フックツールの統合方針(prek を使い続けるか、Vite+ のフック管理に一本化するか)を整理することを推奨します。

Comment thread .npmrc
@@ -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/

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

一時的なレジストリブリッジ(https://pkg-pr-registry-bridge.void.app/)が .npmrc に設定されています。\n\nこの設定が残ったままメインブランチにマージされると、他の開発者の環境やCI環境でのパッケージインストールがこのブリッジの稼働状況に依存してしまい、ビルドの不安定化やセキュリティ上のリスクにつながる可能性があります。\n\nベータテスト用の設定である場合は、マージ前に削除するか、特定のスコープのみに限定するなどの対策を検討してください。

Comment thread vite.config.ts

export default defineConfig({
staged: {
"*": "vp check --fix",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

staged の対象として *(すべてのファイル)が指定されています。\n\nこの設定では、画像などのバイナリファイルやロックファイル、その他の非コード資産に対しても vp check が実行されてしまい、エラーが発生したり、コミット時の処理が不必要に遅くなったりする可能性があります。\n\n以下のように、対象とするファイルの拡張子(例: *.{ts,tsx,js,jsx,json,md})を明示的に指定することを推奨します。

Suggested change
"*": "vp check --fix",
"*.{js,jsx,ts,tsx,json,md}": "vp check --fix",

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.

1 participant