Skip to content

style: アンインストールURL周りのコメントを削減 - #463

Merged
ujiro99 merged 1 commit into
dev-1.3.0from
style/trim-uninstall-comments
Sep 22, 2026
Merged

ujiro99 merged 1 commit into
dev-1.3.0from
style/trim-uninstall-comments

Conversation

@ujiro99

@ujiro99 ujiro99 commented Sep 22, 2026

Copy link
Copy Markdown
Owner

概要

#462 のレビュー指摘(コメントが多すぎる)を受けたコメントのみの修正です。動作の変更はありません。

ファイル Before After
background_script.ts 4行 3行
services/analytics.ts 4行 2行

どちらも「アンインストール時点でService Workerが破棄済みのため拡張からは送信できない」という同じ経緯を繰り返し説明していました。この経緯は ujiro99/selection-command-hub#275 に記載済みのため、コード側は参照のみに留めています。

Hub側の同種の削減(69行 → 26行)は ujiro99/selection-command-hub#276 に含めています。

Test plan

  • yarn workspace @selection-command/extension test:run src/background_script.test.ts(14件 pass)
  • prettier --check 差分なし

🤖 Generated with Claude Code

https://claude.ai/code/session_01HtiybZS7tYtWXK78B89PQJ

Both notes explained at length why the extension can't send the event
itself, which the linked issue already covers.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HtiybZS7tYtWXK78B89PQJ
@claude

claude Bot commented Sep 22, 2026

Copy link
Copy Markdown

レビュー結果

コメントの記述量削減のみの変更で、動作の変更はありません。内容を確認しましたが問題ありません。

確認したポイント

  • packages/extension/src/background_script.ts:500-502: 短縮後も「Hub がアンインストール時に GA4 イベントを送る」という要点と参照先 issue(selection-command-hub#275)は保持されており、情報の欠落はありません。直後の try/catch の意図を説明するコメント(L503-504)とも整合しています。
  • packages/extension/src/services/analytics.ts:57-58: 同様に「Service Worker が破棄済みのため uninstall イベントを Extension 側では送れない」という非自明な理由(AGENTS.md の「コメントは必要な箇所へ」の方針に合致)が残されており、経緯の重複のみが削除されています。
  • 両ファイルのコメントは同じ経緯を指しており、外部 issue への参照のみに一本化する方針は妥当です。矛盾や記述漏れは見当たりません。

コード品質・スタイル

  • TypeScript のコーディング規約(コメント英語表記)に準拠しています。
  • diff は最小限(+3/-6行)で、意図と一致しています。

バグ・パフォーマンス・セキュリティ

  • ロジックの変更が一切ないため、これらの観点で新たに指摘する点はありません。

テストカバレッジ

  • PR 本文記載の background_script.test.ts(14件 pass)で十分です。コメントのみの変更なので追加テストは不要と判断します。

以上、特に修正が必要な指摘事項はありません。LGTM です。

@ujiro99
ujiro99 merged commit 443df22 into dev-1.3.0 Sep 22, 2026
2 checks passed
@ujiro99
ujiro99 deleted the style/trim-uninstall-comments branch September 22, 2026 10:47
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