test(worker,shared): unit tests for summarize, rate-limit, github-ip core logic (patch)

[liplus-lin-lay]

Closes #223

概要

webhook 本流の核心ロジックに単体テストを追加。これまでテストは OAuth (worker/oauth.ts) と mcp-server クライアント契約に偏っており、payload 要約・レート制限・IP 許可リストはテストの外だった。新インフラを足さず、既存ハーネス (tsx --test) で決定的にテストできる純ロジック層をカバーする。

影響スコープ

• テスト追加のみ（新規ファイル3本）＋ worker/src/github-ip.ts に visibility 追加の export 1行。ランタイム挙動の変更なし。
• ユーザー/システムから観測可能な変更なし → patch。

追加テスト（34本、全47本 pass）

ファイル	対象	主なケース
worker/test/summarize.test.ts	shared/summarize.ts	number/title/url のフォールバック階層、workflow_run 固有フィールドの type ゲート、空 payload の null 安全、各種 passthrough
worker/test/rate-limit.test.ts	worker/rate-limit.ts	sliding window (webhook 300 / api 120)、窓リセット、checkTenantQuota の 200/429/404 分岐、rateLimitResponse
worker/test/github-ip.test.ts	worker/github-ip.ts	CIDR 一致の境界・/32・/0・IPv6 CIDR skip・IPv6/不正IP passthrough、isGitHubWebhookIP の fallback 経路

リファクタ

github-ip.ts の内部純関数 ipMatchesCIDRs を export（CIDR 演算を直接単体テストするため）。ロジック不変。

スコープ外（別 issue 予定）

worker/src/store.ts / tenant.ts は Durable Object + ctx.storage.sql で tsx --test では実行不可。本格テストには vitest-pool-workers(Miniflare) 導入＝テストハーネス変更＋workers-types ピン留めが必要なため分離する。

確認

• npm test (worker): 47 pass / 0 fail
• npx wrangler deploy --dry-run: build 成功

[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-webhook-mcp	72d7c0d	Jun 21 2026, 04:03 AM

[liplus-lin-lay]

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

検証結果

• npm test (worker): 47 pass / 0 fail（新規34本込み）。CI gate test pass、CI gate pass、Workers Builds pass。
• wrangler deploy --dry-run: build 成功（github-ip.ts の export 追加が build を壊さないことを確認）。

妥当性（now: 問題なし）

• テストは実コードの分岐を反映：summarize の fallback 階層・workflow_run type ゲート、rate-limit の境界(300/120)・窓リセット・quota 200/429/404、github-ip の CIDR 境界/IPv6 passthrough。
• 決定性を確保：github-ip は global fetch を stub して FALLBACK 経路に固定（実ネット不使用）、rate-limit の窓リセットは Date.now を mock し finally で復元。
• ipMatchesCIDRs の export は visibility のみ、ロジック不変。
• ランタイム挙動の変更なし → patch 分類は妥当（observable impact 無し）。

留意（later: 別 issue）

• store.ts / tenant.ts（Durable Object + ctx.storage.sql）、index.ts の fetch handler、verifyGitHubSignature は tsx --test で実行不可。Miniflare(vitest-pool-workers)導入が要るため本 PR スコープ外。フォローアップ issue を起票予定。

受容済みトレードオフ (accepted)

• summarize.test.ts を shared 専用ハーネスではなく worker/test に配置。新インフラ回避のため（worker は summarize を実依存）。

self-review pass。auto mode に従い squash merge する。