[ujiro99]

No description provided.

[claude]

レビュー結果(Dev 1.4.0)

196 ファイル / +5782 −3677 と大規模なため、新規・影響の大きいロジック(offscreen clipboard、PostCSS プラグイン、manifest プラグイン、shared uuid、manifest.json)を中心に確認しました。全体として設計意図がコメントで丁寧に説明されており、テストも追加されています。以下は指摘事項です。

要確認・バグの可能性

1. packages/extension/src/lib/vite-plugin-manifest.ts:64-103 — assertNoRemInCss が dev/watch ビルドで失敗する恐れ
   content_scripts が無い場合に throw するよう変更され(L69)、その後 contentScriptCssFiles.length === 0 なら必ず例外になります(L103、assertNoRemInCss 内)。writeBundle が yarn dev(crxjs の dev/watch)でも呼ばれる場合、CSS が manifest に含まれないケースでビルドが落ちます。yarn dev と yarn build:e2e で動作確認をお願いします。必要なら production ビルド時のみ検査する(config.command === "build" 等)ことを検討してください。
   また L97-99 の catch は console.error のみで握りつぶすため、manifest 処理失敗の原因が「rem 検出失敗」という別メッセージに隠れます。エラーを保持して assertNoRemInCss のメッセージに含めると原因追跡しやすくなります。

2. packages/extension/src/lib/vite-plugin-manifest.ts:6-17 — コメントの位置ずれ
   injectContentScriptCssFiles の説明コメント(L6-12)の直後に assertNoRemInCss の説明が連結されており、前者のコメントが injectContentScriptCssFiles から離れています。assertNoRemInCss のコメントを関数直前へ分離してください。

3. packages/extension/src/lib/postcss-shadow-dom-properties.ts:70 — root.prepend の @layer 位置
   @charset / @import を含むスタイルシートでは、先頭に @layer を挿入すると @import が無効になる(@import は @charset と @layer 宣言以外より前に置く必要がある)可能性があります。現状 Tailwind v4 の出力で問題がないか、@import を含む CSS でのテストを 1 件追加すると安心です。

4. packages/extension/src/services/offscreenClipboard.ts:33 — エラーメッセージ文字列への依存
   /single offscreen document/i で既存ドキュメントを判定していますが、Chrome のメッセージ変更に弱いです。失敗時にもう一度 getContexts で存在確認する方が堅牢です。また readWithRetry は権限エラーなど恒久的エラーも 3 回リトライします(遅延 50ms なので影響は小さい)。

5. packages/extension/manifest.json — 権限とバージョン

   • offscreen 権限追加と minimum_chrome_version: "116" は offscreen API(Chrome 109+、getContexts は 116+)に整合しています。ただし clipboardRead を残している点は execCommand("paste") に必要なため妥当です。ストア審査向けに justification(offscreen_clipboard.ts の理由文)が十分か確認してください。
   • version が 1.3.2 のままです。ブランチ名 dev-1.4.0 なので、リリース前に更新が必要か確認してください。

軽微・改善提案

• packages/extension/src/offscreen_clipboard.ts:13 — document.getElementById("clipboard") as HTMLTextAreaElement が null の場合、try の外で textarea.value を触る finally が TypeError になります。null チェックを try の前に入れてください(finally 内の参照が別例外を生みます)。
• packages/extension/src/offscreen_clipboard.ts:4-8 — sender.id 検証は良いですが、return false で同期応答しているので問題ありません。ただしクリップボード内容がメモリ上に残らないよう finally で空にしている点は良い対応です。
• packages/shared/src/utils/uuid.ts — cmd2uuid の削除は、リポジトリ内に参照が無いことを確認済みです(旧 hub 側のみで使用されていたため)。shared の公開 API 削除なので、CHANGELOG/リリースノートに記載しておくと良いです。
• .claude/skills/check-japanese-in-src/ の削除 — AGENTS.md では「コード内コメントは英語」とされており、このチェック用スキルを削除する理由(別の仕組みへ移行したか)を PR 説明に書いてください。
• PR の説明(body)が空です。Tailwind v4 移行、onboarding ABテスト、offscreen clipboard、hub 縮小など複数テーマが混在しているため、概要と動作確認項目(特に Shadow DOM での shadow/ring/transform/gradient の見た目、rem 不使用の確認)を記載してください。

パフォーマンス / セキュリティ

• 重大な問題は見つかりませんでした。offscreen ドキュメントを再利用し、読み取りを直列化している点(offscreenClipboard.ts:67-71)は適切です。
• 依存関係更新(Radix、tailwind-merge 3.x、Tailwind 4)は見た目の回帰リスクがあるため、yarn test / yarn build / e2e の結果(CI)を確認のうえマージしてください。

テスト

• offscreenClipboard、postcss プラグイン、findRemUsages、onboarding 実験など新規ロジックにテストが追加されており良好です。
• 追加推奨: vite-plugin-manifest.ts の assertNoRemInCss(空リスト時の失敗・rem 検出時の失敗)と、offscreen_clipboard.ts のメッセージハンドラ(フォーカス失敗/paste 失敗)のユニットテスト。

※ 差分全体(特に UI コンポーネントのスタイル変更)を網羅的には確認していません。