@aiquants/daily-report 0.7.0 → 0.9.0

package/src/server/handlers.ts

@@ -7,8 +7,10 @@
  * - sse.loader: Redis Streams ベースのリアルタイム更新 SSE エンドポイント
  */
 import { normalizeBusinessDateKey } from "../shared/business-date"
+import { type DailyReportIdsStreamChunkLine, type DailyReportIdsStreamCursor, findIdsStreamResumeIndex } from "../shared/ids-stream"
 import { createLogger, type DailyReportLogger, LogLevel } from "../shared/logger"
 import { dailyReportSseMessageSchema } from "../shared/sse-schema"
+import type { DailyReportItem } from "../shared/types"
 import type { DailyReportAttachmentFailure, DailyReportAuthenticate, DailyReportEncodeUserId, DailyReportIdCodec, DailyReportReadAttachment, DailyReportRedisProvider } from "./ports"
 import { jsonResponseWithETag } from "./response"
 import type { DailyReportService } from "./service"
@@ -24,6 +26,220 @@ const jsonData = (payload: unknown, init?: { status?: number }): Response =>
         headers: { "Content-Type": "application/json" },
     })
 
+// ---------------- ids NDJSON ストリームの定数とヘルパー (純粋・モジュールスコープ) ----------------
+
+/**
+ * First-chunk size, tuned for time-to-first-render.
+ * 初回チャンクのアイテム数。最初の描画までの時間を優先して小さく保つ。
+ *
+ * 一覧の可視行 + オーバースキャンを一度に満たせる件数であればよい。
+ */
+const IDS_STREAM_FIRST_CHUNK_SIZE = 200
+
+/**
+ * 2 チャンク目以降のアイテム数。1 アイテムが数十バイト (id/日付/種別のみ) と軽いため
+ * 大きめに運ぶ。実データは 20 万件超に達するため、小さすぎると行数 (= pull 往復と
+ * クライアント側 publish 回数) が肥大する (4000 件 ≈ 240 KB/行、20 万件で約 50 行)。
+ */
+const IDS_STREAM_NEXT_CHUNK_SIZE = 4000
+
+/**
+ * Sanitized message for mid-stream failures (no internal details).
+ * ストリーム送出中の失敗をクライアントへ伝える定型文 (内部情報は載せない)。
+ */
+const IDS_STREAM_ERROR_CLIENT_MESSAGE = "stream interrupted by server error"
+
+/** 再開カーソルの営業日として受け付ける唯一の形式 (正規化済み YYYY-MM-DD)。 */
+const IDS_STREAM_DATE_PATTERN = /^\d{4}-\d{2}-\d{2}$/
+
+/**
+ * Heartbeat interval while the next chunk is pending (ms).
+ * 次チャンク待ちの間に空行ハートビートを送る間隔 (ms)。
+ *
+ * 全量クエリ (20 万件超) の確定待ちで無音が続くと、クライアントの stall ウォッチドッグ
+ * (既定 30s) が生きた接続を half-open と誤認する。それより十分短い間隔で空行を送り、
+ * read を解決させて誤検知を防ぐ。空行はクライアント側で読み飛ばされる (`trim() === ""`)。
+ */
+const IDS_STREAM_HEARTBEAT_MS = 15_000
+
+/** next() 待ちがハートビート間隔に負けたことを表す番兵。 */
+const IDS_STREAM_HEARTBEAT = Symbol("ids-stream-heartbeat")
+
+/**
+ * Parses and validates resume-cursor query params (fail-closed).
+ * 再開カーソルのクエリパラメータを検証付きで解釈する処理 (不正は fail-closed)。
+ *
+ * - `cursor_report_hub_id` 単独 = 「営業日なし領域」のカーソル (businessDate: null)。
+ * - `cursor_business_date` は `cursor_report_hub_id` 無しでは意味を持たないため 400。
+ *
+ * @param url Request URL. リクエスト URL。
+ * @returns Cursor (null = 先頭から) か、検証失敗の印。
+ */
+const parseIdsStreamCursor = (url: URL): { cursor: DailyReportIdsStreamCursor | null } | { invalid: true } => {
+    const rawId = url.searchParams.get("cursor_report_hub_id")
+    const rawDate = url.searchParams.get("cursor_business_date")
+
+    if (rawId === null) {
+        // 営業日だけのカーソルは並び順上の位置を決められない
+        return rawDate === null ? { cursor: null } : { invalid: true }
+    }
+
+    const reportHubId = Number.parseInt(rawId, 10)
+    if (!Number.isSafeInteger(reportHubId) || String(reportHubId) !== rawId || reportHubId <= 0) {
+        return { invalid: true }
+    }
+
+    if (rawDate === null) {
+        return { cursor: { businessDate: null, reportHubId } }
+    }
+    if (!IDS_STREAM_DATE_PATTERN.test(rawDate)) {
+        return { invalid: true }
+    }
+    return { cursor: { businessDate: rawDate, reportHubId } }
+}
+
+/**
+ * Emits a finalized id array as chunked NDJSON lines from a resume position.
+ * 確定済み ID 配列を再開位置からチャンク行として送出するジェネレーター。
+ *
+ * `total` は最初に yield する行へ載せる (= 接続内で全量が確定した最初の行。
+ * キャッシュヒットの新規接続と再開接続では先頭行、高速先頭ページ経路では 2 行目)。
+ *
+ * @param items Stream-ordered full array. ストリーム順の全量配列。
+ * @param resumeKey Resume position (null = 先頭から)。
+ * @param firstChunkSize このジェネレーターの最初のチャンクサイズ。
+ */
+async function* streamLinesFromFullArray(items: DailyReportItem[], resumeKey: DailyReportIdsStreamCursor | null, firstChunkSize = IDS_STREAM_FIRST_CHUNK_SIZE): AsyncGenerator<DailyReportIdsStreamChunkLine> {
+    let position = resumeKey ? findIdsStreamResumeIndex(items, resumeKey) : 0
+    let isFirstYield = true
+    while (true) {
+        const chunkItems = items.slice(position, position + (isFirstYield ? firstChunkSize : IDS_STREAM_NEXT_CHUNK_SIZE))
+        position += chunkItems.length
+        const isLast = position >= items.length
+        const lastItem = chunkItems[chunkItems.length - 1]
+        yield {
+            items: chunkItems,
+            nextCursor: isLast || !lastItem ? null : { businessDate: lastItem.businessDate, reportHubId: lastItem.reportHubId },
+            isLast,
+            ...(isFirstYield ? { total: items.length } : {}),
+        }
+        isFirstYield = false
+        if (isLast) return
+    }
+}
+
+/**
+ * Wraps a line iterator into a pull-based NDJSON streaming Response.
+ * 行イテレーターを pull 駆動の NDJSON ストリーミング Response へ包む処理。
+ *
+ * - 切断検知時はイテレーターを畳んで (`return()`) DB 待ちを解放し、close する
+ *   (素の return は desiredSize > 0 のまま pull が再帰する)。
+ * - イテレーターの throw は内部情報を含まない定型エラー行 + 正常クローズへ倒す
+ *   (クライアントは「isLast 無し終端」をカーソル再開のシグナルとして扱う)。
+ */
+const buildIdsStreamResponse = (cookie: string | null, lines: AsyncGenerator<DailyReportIdsStreamChunkLine>, request: Request, logger: DailyReportLogger): Response => {
+    const encoder = new TextEncoder()
+    // cancel() の lines.return() は「pull が DB 待ちの next() を保留中」だと先取りできない。
+    // 待ちが明けた後の enqueue が閉じたストリームへ落ちて catch に入るため、切断は
+    // フラグで決定的に識別し、通常イベントとして静かに畳む (ERROR ログにしない —
+    // 全量クエリ中の離脱は日常イベントで、本物の送出失敗が雑音に埋もれる)
+    let cancelled = false
+    // next() の解決はハートビートを跨いで保持する。ハートビートが先着したら空行を出し、
+    // 同じ next() を次の pull で await し直す (毎回 next() を呼び直すと二重取得になる)。
+    let pendingNext: Promise<IteratorResult<DailyReportIdsStreamChunkLine>> | null = null
+    const stream = new ReadableStream<Uint8Array>({
+        async pull(controller) {
+            if (cancelled || request.signal.aborted) {
+                await lines.return(undefined).catch(() => {})
+                try {
+                    controller.close()
+                } catch {
+                    // cancel 済みコントローラーの close は失敗してよい
+                }
+                return
+            }
+            let heartbeatTimer: ReturnType<typeof setTimeout> | null = null
+            try {
+                if (pendingNext === null) {
+                    pendingNext = lines.next()
+                }
+                // 次チャンクの確定待ちが長引く間、接続の生存を示す空行を送る
+                const heartbeatPromise = new Promise<typeof IDS_STREAM_HEARTBEAT>((resolve) => {
+                    heartbeatTimer = setTimeout(() => resolve(IDS_STREAM_HEARTBEAT), IDS_STREAM_HEARTBEAT_MS)
+                })
+                const raced = await Promise.race([pendingNext, heartbeatPromise])
+                if (heartbeatTimer !== null) {
+                    clearTimeout(heartbeatTimer)
+                    heartbeatTimer = null
+                }
+                if (raced === IDS_STREAM_HEARTBEAT) {
+                    // next() はまだ保留中。空行だけ送って pending は次の pull へ持ち越す
+                    if (cancelled || request.signal.aborted) {
+                        await lines.return(undefined).catch(() => {})
+                        try {
+                            controller.close()
+                        } catch {
+                            // cancel 済みコントローラーの close は失敗してよい
+                        }
+                        return
+                    }
+                    controller.enqueue(encoder.encode("\n"))
+                    return
+                }
+                pendingNext = null
+                const { value, done } = raced
+                if (done) {
+                    controller.close()
+                    return
+                }
+                // 保留中の next() の間に切断された場合はここで検知して静かに終える
+                if (cancelled || request.signal.aborted) {
+                    await lines.return(undefined).catch(() => {})
+                    try {
+                        controller.close()
+                    } catch {
+                        // cancel 済みコントローラーの close は失敗してよい
+                    }
+                    return
+                }
+                controller.enqueue(encoder.encode(`${JSON.stringify(value)}\n`))
+            } catch (error) {
+                if (heartbeatTimer !== null) {
+                    clearTimeout(heartbeatTimer)
+                }
+                // next() が throw した = このイテレーターは畳む。保留参照を残さない
+                pendingNext = null
+                if (cancelled || request.signal.aborted) {
+                    // 切断起因の enqueue/close 失敗はエラーではない
+                    return
+                }
+                logger.error(`ids-stream emission failed: ${error instanceof Error ? error.message : String(error)}`)
+                try {
+                    controller.enqueue(encoder.encode(`${JSON.stringify({ error: IDS_STREAM_ERROR_CLIENT_MESSAGE })}\n`))
+                    controller.close()
+                } catch {
+                    // 既に切断済みなら何もできない
+                }
+            }
+        },
+        cancel() {
+            cancelled = true
+            void lines.return(undefined).catch(() => {})
+        },
+    })
+
+    const headers = new Headers({
+        "Content-Type": "application/x-ndjson; charset=utf-8",
+        // no-transform: compression ミドルウェア/中間プロキシによるバッファリングを禁止する (SSE と同じ理由)
+        "Cache-Control": "private, no-cache, no-transform",
+        "X-Content-Type-Options": "nosniff",
+    })
+    if (cookie) {
+        headers.append("Set-Cookie", cookie)
+    }
+    return new Response(stream, { status: 200, headers })
+}
+
 // ---------------- 添付配信の定数とヘルパー (純粋・モジュールスコープ) ----------------
 
 /** 添付 1 件あたりの既定上限。サービス側のワイヤ上限 (既定 64 MiB) より必ず低く保つ。 */
@@ -302,6 +518,13 @@ export type DailyReportHandlersConfig = {
     streamKey: string
     /** 未ログイン時のリダイレクト先 (index.loader 用、既定 "/auth/login")。 */
     loginRedirectPath?: string
+    /**
+     * `intent=clearCache` (全サーバーキャッシュの一括破棄) を許可するか。既定 false。
+     * 開発用アフォーダンスであり、有効化すると認証済みユーザーなら誰でも全ワーカーの
+     * キャッシュを無制限に破棄できるため、本番では必ず無効のままにする
+     * (消費アプリは開発環境でのみ true を注入する)。
+     */
+    enableDevCacheClear?: boolean
     /** ロガー (既定は console ベース)。 */
     logger?: DailyReportLogger
 }
@@ -370,13 +593,88 @@ export function createDailyReportHandlers(config: DailyReportHandlersConfig) {
             return jsonResponseWithETag(request, cookie, { businessDate: normalizedBusinessDate, reports }, 200)
         },
         /**
-         * Retrieves a list of all daily report IDs.
-         * 全ての日報 ID の一覧を取得する。
+         * Streams the full daily report id list as NDJSON chunks.
+         * 日報 ID 一覧全件を NDJSON チャンクとして逐次配信する。
+         *
+         * 全量クエリ (`SqlResultCache` の TTL / Redis epoch / in-flight コアレッシングを
+         * 素通しで維持) は常に即時に開始する。全量確定を待ってから送出するのは再開接続のみで、
+         * その場合の DB エラーはストリーム開始前に throw されて apiLoader の catch が
+         * 500 JSON を返す。新規接続はキャッシュ無しの TOP 200 高速先頭ページと競争させ、
+         * キャッシュミス時は全量確定前に最初のチャンクを送出する (最初の描画までの時間が
+         * 全体データ量に依存しない)。キャッシュヒット時は全量が即勝ちし、従来どおり
+         * 確定済み配列のメモリスライス送出になる。
          */
-        ids: async (url, cookie, request, user) => {
+        "ids-stream": async (url, cookie, request, user) => {
+            const parsed = parseIdsStreamCursor(url)
+            if ("invalid" in parsed) {
+                return jsonResponseWithETag(request, cookie, { error: { message: "Invalid cursor" } }, 400)
+            }
             const forceRefresh = url.searchParams.get("forceRefresh") === "true"
-            const dailyReportIds = await service.getDailyReportIdsByExternalId(user.id, { forceRefresh })
-            return jsonResponseWithETag(request, cookie, { ids: dailyReportIds }, 200)
+
+            // 全量 (SqlResultCache / epoch / コアレッシング維持) は常に即時に走らせ始める
+            const fullArrayPromise = service.getDailyReportIdsByExternalId(user.id, { forceRefresh })
+            // 拒否はイテレーターの await 側で扱う (ここで未処理拒否警告を出さない)
+            fullArrayPromise.catch(() => {})
+
+            if (parsed.cursor) {
+                // 再開接続: クライアントは高速先頭を受信済みなので全量確定を待つ (エラーは 500)。
+                // 全量クエリはアプリ側の requestTimeout (既定 60s) で境界づけられ、確定後は
+                // スライスだけなので、接続確立後の無音はハートビート対象にならない
+                return buildIdsStreamResponse(cookie, streamLinesFromFullArray(await fullArrayPromise, parsed.cursor), request, apiLogger)
+            }
+
+            /**
+             * 新規接続: 「高速先頭ページ (TOP n・キャッシュ無し)」と「全量 (キャッシュ経由)」を
+             * 競争させ、先に確定した方で最初のチャンクを出す。キャッシュヒット時は全量が即勝ち
+             * (従来どおり先頭行に total)。キャッシュミス時は先頭ページが先に届き、最初の描画は
+             * 全量クエリの確定を待たない。total は全量確定後の最初の行に載る。
+             */
+            // キャッシュヒット時もこの TOP 200 クエリは走って結果が捨てられる (二重クエリ)。
+            // インデックス順の部分走査で ms 級と実測見積もりされており、ヒット判定のための
+            // キャッシュ同期 peek API を足す複雑さに見合わないため意図的に許容する
+            const firstPagePromise = service.getDailyReportIdsFirstPageByExternalId(user.id, IDS_STREAM_FIRST_CHUNK_SIZE)
+            firstPagePromise.catch(() => {})
+
+            const lineIterator = (async function* (): AsyncGenerator<DailyReportIdsStreamChunkLine> {
+                const winner = await Promise.race([
+                    fullArrayPromise.then(
+                        (items) => ({ kind: "full" as const, items }),
+                        (error) => ({ kind: "full-error" as const, error }),
+                    ),
+                    firstPagePromise.then(
+                        (items) => ({ kind: "page" as const, items }),
+                        () => ({ kind: "page-error" as const }),
+                    ),
+                ])
+
+                let resumeKey: DailyReportIdsStreamCursor | null = null
+                let fullItems: DailyReportItem[] | null = null
+
+                if (winner.kind === "full") {
+                    fullItems = winner.items
+                } else if (winner.kind === "page" || winner.kind === "full-error") {
+                    // 全量が先に死んでいても先頭ページが生きていれば、まずそれを届ける
+                    const pageItems = winner.kind === "page" ? winner.items : await firstPagePromise
+                    if (pageItems.length < IDS_STREAM_FIRST_CHUNK_SIZE) {
+                        // 先頭ページが上限未満 = データセット全量。全量クエリを待たずに完結する
+                        yield { items: pageItems, nextCursor: null, isLast: true, total: pageItems.length }
+                        return
+                    }
+                    const lastItem = pageItems[pageItems.length - 1]
+                    resumeKey = { businessDate: lastItem.businessDate, reportHubId: lastItem.reportHubId }
+                    // total は未知 (全量確定後の行に載せる)
+                    yield { items: pageItems, nextCursor: resumeKey, isLast: false }
+                }
+                // page-error は全量へフォールバック。full-error は上の分岐でページを出してから、ここで throw する
+                if (fullItems === null) {
+                    fullItems = await fullArrayPromise
+                }
+                // 高速先頭ページ経路 (resumeKey あり) はクライアントが既に 200 件を描画済み
+                // なので、続きは大きいチャンクで運ぶ。キャッシュヒット経路は従来どおり 200 件先頭
+                yield* streamLinesFromFullArray(fullItems, resumeKey, resumeKey ? IDS_STREAM_NEXT_CHUNK_SIZE : IDS_STREAM_FIRST_CHUNK_SIZE)
+            })()
+
+            return buildIdsStreamResponse(cookie, lineIterator, request, apiLogger)
         },
         /**
          * Retrieves details for a specific daily report.
@@ -460,6 +758,11 @@ export function createDailyReportHandlers(config: DailyReportHandlersConfig) {
         const clientTempId = formData.get("clientTempId") as string | null
 
         if (intent === "clearCache") {
+            // 開発専用アフォーダンス。既定は封鎖 (認証済みなら誰でも全ワーカーの
+            // キャッシュを無制限破棄できてしまうため、注入で明示的に開けた環境のみ許可)
+            if (!config.enableDevCacheClear) {
+                return jsonData({ error: "Invalid intent" }, { status: 400 })
+            }
             await service.clearCache()
             return jsonData({ status: "OK", intent: "clearCache" })
         }
@@ -567,9 +870,22 @@ export function createDailyReportHandlers(config: DailyReportHandlersConfig) {
                 if (!content) {
                     return jsonData({ error: "Content required" }, { status: 400 })
                 }
-                const newComment = await service.addComment(userId, reportHubId, content, businessDate, clientTempId)
-                const safeComment = { ...newComment, userId: newComment.userId }
-                return jsonData({ status: "OK", intent: "addComment", newComment: safeComment, reportHubId: String(reportHubId), clientTempId })
+                try {
+                    const newComment = await service.addComment(userId, reportHubId, content, businessDate, clientTempId)
+                    const safeComment = { ...newComment, userId: newComment.userId }
+                    return jsonData({ status: "OK", intent: "addComment", newComment: safeComment, reportHubId: String(reportHubId), clientTempId })
+                } catch (e: unknown) {
+                    if (e instanceof Error) {
+                        // 削除直後の日報への投稿レース等。500 ではなく意味のある状態コードへ写像する
+                        if (e.message === "Not Found") {
+                            return jsonData({ error: "Report not found" }, { status: 404 })
+                        }
+                        if (e.message.startsWith("Comments are restricted")) {
+                            return jsonData({ error: "Comments are restricted for this report" }, { status: 403 })
+                        }
+                    }
+                    throw e
+                }
             }
             case "deleteComment": {
                 const commentId = Number(formData.get("commentId"))

package/src/server/service.spec.ts

@@ -55,10 +55,14 @@ type RecordedUpdate = {
  * @param rows Rows every query resolves to. すべてのクエリが返す行。
  * @returns The db mock and the per-query recordings. db モックとクエリごとの記録。
  */
-const makeDb = (rows: unknown[]): { db: DailyReportDb; queries: RecordedQuery[]; updates: RecordedUpdate[] } => {
+const makeDb = (rows: unknown[], options: { selectQueue?: unknown[][] } = {}): { db: DailyReportDb; queries: RecordedQuery[]; updates: RecordedUpdate[] } => {
     const queries: RecordedQuery[] = []
     const updates: RecordedUpdate[] = []
+    // selectQueue があれば select 呼び出しごとに 1 要素ずつ消費する (枯れたら rows へフォールバック)。
+    // 複数 select が異なる結果を返す必要のあるフロー (draft 判定など) の検証に使う
+    const selectQueue = options.selectQueue ? [...options.selectQueue] : null
     const select = () => {
+        const result = selectQueue && selectQueue.length > 0 ? (selectQueue.shift() as unknown[]) : rows
         const recorded: RecordedQuery = { whereSql: null, whereParams: [], leftJoins: 0, joinParams: [] }
         queries.push(recorded)
         const chain: Record<string, unknown> = {
@@ -76,11 +80,11 @@ const makeDb = (rows: unknown[]): { db: DailyReportDb; queries: RecordedQuery[];
                 return chain
             },
             top: () => chain,
-            orderBy: () => Promise.resolve(rows),
+            orderBy: () => Promise.resolve(result),
             // `.where()` / `.orderBy()` のどちらで終わっても await できるようにする。
             // thenable であること自体が目的なので、この規則はここでは意図的に外す
             // biome-ignore lint/suspicious/noThenProperty: 意図的な thenable (クエリビルダの終端を模擬する)
-            then: (resolve: (value: unknown) => unknown) => resolve(rows),
+            then: (resolve: (value: unknown) => unknown) => resolve(result),
         }
         return chain
     }
@@ -99,7 +103,17 @@ const makeDb = (rows: unknown[]): { db: DailyReportDb; queries: RecordedQuery[];
         }
         return chain
     }
-    return { db: { select, update } as unknown as DailyReportDb, queries, updates }
+    // INSERT: values をそのまま id 付きでエコーする (addComment の inserted 行を模擬)
+    const insert = () => {
+        const chain: Record<string, unknown> = {
+            output: () => chain,
+            values: (values: Record<string, unknown>) => Promise.resolve([{ id: 501, ...values }]),
+        }
+        return chain
+    }
+    // DELETE: 記録のみ (deleteComment 用)
+    const del = () => ({ where: () => Promise.resolve({ rowsAffected: [1] }) })
+    return { db: { select, update, insert, delete: del } as unknown as DailyReportDb, queries, updates }
 }
 
 const baseHub = (over: Partial<DailyReportHubRow>): DailyReportHubRow => ({
@@ -120,10 +134,20 @@ const baseHub = (over: Partial<DailyReportHubRow>): DailyReportHubRow => ({
     ...over,
 })
 
-const makeService = (rows: unknown[], withAttachmentCodec = false) => {
+const makeService = (rows: unknown[], withAttachmentCodec = false, options: { selectQueue?: unknown[][] } = {}) => {
     const epochs = createEpochStore(undefined)
     const cache = new SqlResultCache({ defaultTtlMs: 60_000 }, epochs)
-    const { db, queries, updates } = makeDb(rows)
+    const { db, queries, updates } = makeDb(rows, { selectQueue: options.selectQueue })
+    // SSE publish (redis xAdd) の捕捉。宛先フィルタ (recipientRawUserId) の検証に使う
+    const sseMessages: Record<string, unknown>[] = []
+    const redis = {
+        getClient: async () => ({
+            xAdd: async (_key: string, _id: string, fields: { data: string }) => {
+                sseMessages.push(JSON.parse(fields.data) as Record<string, unknown>)
+                return "1-1"
+            },
+        }),
+    }
     const service = createDailyReportService({
         db,
         tables,
@@ -132,12 +156,13 @@ const makeService = (rows: unknown[], withAttachmentCodec = false) => {
         encodeUserId: (id) => `ENC${id}`, // テスト用の識別可能な難読化
         cache,
         epochs,
+        redis: redis as unknown as Parameters<typeof createDailyReportService>[0]["redis"],
         draftLabelName: "draft", // 必須注入 (ドラフト可視性フィルタ用のラベル名。テストは中立値で十分)
         // 既定では未注入のまま。既存テストが縮退動作 (添付は常に空) で通ることも同時に確認する。
         // ユーザー ID 用 (`ENC`) とは別の接頭辞にして ID 空間の分離を表現する。
         ...(withAttachmentCodec ? { attachmentIdCodec: { encode: (id: number) => `ATT${id}`, decode: (t: string) => (t.startsWith("ATT") ? Number(t.slice(3)) : null) } } : {}),
     })
-    return Object.assign(service, { __queries: queries, __updates: updates })
+    return Object.assign(service, { __queries: queries, __updates: updates, __sse: sseMessages })
 }
 
 describe("mapHubRecord audit-actor masking (security)", () => {
@@ -361,3 +386,54 @@ describe("実体存在の記録 (三値の書き込み側)", () => {
         expect(updates).toHaveLength(0)
     })
 })
+
+describe("コメント操作の整合性チェック", () => {
+    it("addComment は存在しない日報に対して Not Found を投げる (孤児コメントを作らない)", async () => {
+        // hub 取得が空 = 存在しない日報。以前は hub 未取得でもフォールスルーして INSERT していた
+        const svc = makeService([])
+        await expect(svc.addComment(8, 999, "本文", "2099-01-05", "tmp")).rejects.toThrow("Not Found")
+    })
+
+    it("addComment は論理削除済みの日報を拒否する", async () => {
+        const svc = makeService([{ sourceType: "Internal", deletedAt: "2099-01-06 00:00:00", userId: 8 }])
+        await expect(svc.addComment(8, 1, "本文", "2099-01-05", "tmp")).rejects.toThrow("Not Found")
+    })
+
+    it("deleteComment は reportHubId がコメントの実所属と一致しないと Not Found を投げる", async () => {
+        // コメントは hubId=100 に属するのに、呼び出しは reportHubId=999 を渡している。
+        // 不一致のまま進むと無関係な日報のキャッシュ無効化と SSE 配信を起こすため拒否する
+        const svc = makeService([{ id: 5, hubId: 100, userId: 8, body: "x", createdAt: "2099-01-05 00:00:00", createdBy: "8", updatedAt: "2099-01-05 00:00:00", updatedBy: "8" }])
+        await expect(svc.deleteComment(8, 999, 5, "2099-01-05", "tmp")).rejects.toThrow("Not Found")
+    })
+
+    it("addComment: 公開済み日報のコメントは全員配信 (recipientRawUserId 無し)", async () => {
+        const svc = makeService([], false, {
+            selectQueue: [
+                [{ sourceType: "Internal", deletedAt: null, userId: 8 }], // hub 取得
+                [{ id: 10 }], // getDraftLabelJoinIds (下書きラベル定義)
+                [], // 下書き関連 (該当なし = 公開済み)
+                [{ displayName: "山田" }], // コメント投稿者名
+            ],
+        })
+        await svc.addComment(8, 1, "本文", "2099-01-05", "tmp")
+        const sse = (svc as unknown as { __sse: Record<string, unknown>[] }).__sse
+        const add = sse.find((m) => m.type === "comment-add")
+        expect(add).toBeDefined()
+        expect(add?.recipientRawUserId).toBeUndefined()
+    })
+
+    it("addComment: 下書き日報のコメントは所有者だけに配信 (本文の全員配信を防ぐ)", async () => {
+        const svc = makeService([], false, {
+            selectQueue: [
+                [{ sourceType: "Internal", deletedAt: null, userId: 8 }], // hub (所有者 8)
+                [{ id: 10 }], // getDraftLabelJoinIds
+                [{ hubId: 1 }], // 下書き関連あり = 下書き
+                [{ displayName: "山田" }], // 投稿者名
+            ],
+        })
+        await svc.addComment(8, 1, "秘密の下書きコメント", "2099-01-05", "tmp")
+        const sse = (svc as unknown as { __sse: Record<string, unknown>[] }).__sse
+        const add = sse.find((m) => m.type === "comment-add")
+        expect(add?.recipientRawUserId).toBe(8)
+    })
+})