fix(webhook): constant-time signature verification (close timing side-channel)

[liplus-lin-lay]

概要

src/webhook.ts の verifyGitHubSignature を 定数時間比較に修正（#167）。

旧実装は HMAC を再計算して expected === signature で文字列比較していた。=== は最初の不一致で早期終了するため、「先頭が何文字一致したか」が応答時間に滲み、理論上は署名を1バイトずつ当てる timing attack に晒される（タイミング側チャネル）。

変更

• crypto.subtle.verify("HMAC", ...) に置換。verify は HMAC を再計算して定数時間で比較するので、時間から情報が漏れない。
• 署名は事前に形式検証（sha256= prefix + 64 hex）。malformed は crypto に触れる前に一律 false。
• 挙動は不変（正しい署名→true / 不正・malformed→false）。webhook.test.ts の既存契約はそのまま green、non-hex ケースを1件追加。

影響範囲

• src/webhook.ts（worker 側）のみ。npm パッケージ（mcp-server/）は不変。修正は merge 時の Workers Builds で本番 Worker にデプロイされる。
• severity 低（ネットワークジッタ支配で実攻撃は非現実的）だが、セキュリティ境界のベストプラクティス遵守。挙動変更なし → patch。

Closes #167

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Updated (UTC)
✅ Deployment successful! View logs	github-rag-mcp	c8d70a1	Jun 21 2026, 03:20 AM

[liplus-lin-lay]

AI セルフレビュー (auto mode)

• ✅ CI 全 green: test（node 42 + workers 15）+ tsc --noEmit + wrangler dry-run + Workers Builds。ローカルでも確認済み。
• ✅ 修正は src/webhook.ts の verifyGitHubSignature のみ。crypto.subtle.verify による定数時間比較に置換し、=== のタイミング側チャネルを除去。
• ✅ 挙動不変: webhook.test.ts の契約（valid→true / 不正・malformed→false）はそのまま green、non-hex ケース1件追加。
• ✅ malformed 署名は crypto に触れる前に形式検証（prefix + 64 hex）で一律 false。
• リリース種別: 挙動変更なし → patch。

self-review pass。auto mode のため squash merge → #167 close。merge 時の Workers Builds で本番 Worker に定数時間版がデプロイされる。