@aiquants/daily-report 0.8.0 → 0.9.0

package/src/client/streaming/daily-report-ids-stream-client.ts

@@ -50,6 +50,13 @@ export type DailyReportIdsStreamClientOptions = {
     /** タイマー実装 (既定 setTimeout / clearTimeout)。 */
     setTimeoutFn?: (handler: () => void, ms: number) => ReturnType<typeof setTimeout>
     clearTimeoutFn?: (id: ReturnType<typeof setTimeout>) => void
+    /**
+     * 1 回の read が無音でいられる上限 (ms)。0 以下で無効。
+     * これを超えたら接続を half-open とみなして中断し、retryable として
+     * カーソル再開へ倒す。サーバーのハートビート間隔より必ず長く取る
+     * (生きているが遅いだけの全量クエリ待ちを誤って切らないため)。
+     */
+    readStallTimeoutMs?: number
     /** 警告ロガー (既定 console.warn)。 */
     warn?: (...args: unknown[]) => void
     /**
@@ -73,6 +80,16 @@ const MAX_RETRY_DELAY_MS = 30_000
 /** 連続失敗の上限。到達で failed へ落ち、手動 retry 待ちになる。 */
 const MAX_CONSECUTIVE_FAILURES = 6
 
+/**
+ * 1 回の read が無音でいられる既定の上限 (ms)。
+ * サーバーのハートビート間隔 (15s) の 2 倍を取り、生きた接続を誤って切らない。
+ * 超過した read は half-open 接続とみなし retryable に倒す。
+ */
+const DEFAULT_READ_STALL_TIMEOUT_MS = 30_000
+
+/** read が stall ウォッチドッグに捕まったことを表す番兵。 */
+const READ_STALL = Symbol("read-stall")
+
 /** 既定の API ベースパス (config-context の既定と一致させる)。 */
 const DEFAULT_API_BASE_PATH = "/daily_report/api"
 
@@ -89,6 +106,8 @@ export class DailyReportIdsStreamClient {
     private readonly setTimeoutFn: NonNullable<DailyReportIdsStreamClientOptions["setTimeoutFn"]>
     private readonly clearTimeoutFn: NonNullable<DailyReportIdsStreamClientOptions["clearTimeoutFn"]>
     private readonly warn: NonNullable<DailyReportIdsStreamClientOptions["warn"]>
+    /** 1 回の read が無音でいられる上限 (ms)。0 以下で無効。 */
+    private readonly readStallTimeoutMs: number
 
     /** reportHubId → アイテム。挿入順を保ち、再開の重複到着を吸収する。 */
     private readonly itemMap = new Map<number, DailyReportItem>()
@@ -148,6 +167,7 @@ export class DailyReportIdsStreamClient {
         this.setTimeoutFn = options.setTimeoutFn ?? ((handler, ms) => setTimeout(handler, ms))
         this.clearTimeoutFn = options.clearTimeoutFn ?? ((id) => clearTimeout(id))
         this.warn = options.warn ?? ((...args) => console.warn(...args))
+        this.readStallTimeoutMs = options.readStallTimeoutMs ?? DEFAULT_READ_STALL_TIMEOUT_MS
         this.publishCoalesceMs = options.publishCoalesceMs ?? DEFAULT_PUBLISH_COALESCE_MS
         if (typeof window !== "undefined") {
             // オフライン復帰でバックオフ待機を即時解除する
@@ -402,11 +422,32 @@ export class DailyReportIdsStreamClient {
         const abortController = new AbortController()
         this.activeAbortController = abortController
         try {
-            const response = await this.fetchFn(this.buildStreamUrl(forceRefresh), {
+            // ヘッダー到着前 (再開経路は全量クエリ確定までヘッダーを返さない) にはハートビートが
+            // 流れないため、read より緩い上限 (3 倍 = 既定 90s。サーバーの requestTimeout 60s より
+            // 長い) で fetch 自体も見張る。ここで切れた場合も retryable としてカーソル再開する
+            const fetchPromise = this.fetchFn(this.buildStreamUrl(forceRefresh), {
                 signal: abortController.signal,
                 headers: { Accept: "application/x-ndjson" },
                 cache: "no-store",
             })
+            let headerTimer: ReturnType<typeof setTimeout> | null = null
+            const response =
+                this.readStallTimeoutMs <= 0
+                    ? await fetchPromise
+                    : await Promise.race([
+                          fetchPromise,
+                          new Promise<typeof READ_STALL>((resolve) => {
+                              headerTimer = this.setTimeoutFn(() => resolve(READ_STALL), this.readStallTimeoutMs * 3)
+                          }),
+                      ]).finally(() => {
+                          if (headerTimer !== null) this.clearTimeoutFn(headerTimer)
+                      })
+            if (response === READ_STALL) {
+                abortController.abort()
+                // 破棄した接続の解決/拒否を未処理のまま残さない
+                fetchPromise.catch(() => {})
+                return { kind: "retryable", error: new Error("Stream stalled: no response headers within timeout") }
+            }
             if (response.status === 401 || response.status === 403) {
                 return { kind: "auth-required" }
             }
@@ -421,7 +462,15 @@ export class DailyReportIdsStreamClient {
             let sawIsLast = false
 
             while (true) {
-                const { done, value } = await reader.read()
+                const chunk = await this.readWithStallGuard(reader)
+                if (chunk === READ_STALL) {
+                    // 無音が続いた half-open 接続。中断してカーソル再開へ倒す
+                    // (finally が activeAbortController を掃除する)
+                    abortController.abort()
+                    void reader.cancel().catch(() => {})
+                    return { kind: "retryable", error: new Error("Stream stalled: no data within timeout") }
+                }
+                const { done, value } = chunk
                 if (done) break
                 // ネットワークチャンクは行境界と無関係に割れるため、末尾の欠け行を持ち越す
                 const lines = (partialLine + decoder.decode(value, { stream: true })).split("\n")
@@ -486,6 +535,35 @@ export class DailyReportIdsStreamClient {
         }
     }
 
+    /**
+     * Reads one chunk, racing the read against a stall watchdog.
+     * 1 チャンクを読む処理。読み取りを stall ウォッチドッグと競争させる。
+     *
+     * サーバーが無音のまま接続を half-open で放置しても (NAT/プロキシのアイドル切断・
+     * VPN 断・スリープ)、`reader.read()` は永久に保留したままになる。これを検知するため
+     * read とタイマーを競争させ、無音が上限を超えたら番兵を返す。生きているが遅いだけの
+     * 全量クエリ待ちはサーバーのハートビート (空行) が read を解決するので誤検知しない。
+     * `readStallTimeoutMs <= 0` のときは素の read (タイマーを張らない・テスト用)。
+     *
+     * @returns 読み取り結果、または stall を表す番兵。read の reject はそのまま伝播する。
+     */
+    private async readWithStallGuard(reader: ReadableStreamDefaultReader<Uint8Array>): Promise<ReadableStreamReadResult<Uint8Array> | typeof READ_STALL> {
+        if (this.readStallTimeoutMs <= 0) {
+            return reader.read()
+        }
+        let stallTimer: ReturnType<typeof setTimeout> | null = null
+        const stallPromise = new Promise<typeof READ_STALL>((resolve) => {
+            stallTimer = this.setTimeoutFn(() => resolve(READ_STALL), this.readStallTimeoutMs)
+        })
+        try {
+            return await Promise.race([reader.read(), stallPromise])
+        } finally {
+            if (stallTimer !== null) {
+                this.clearTimeoutFn(stallTimer)
+            }
+        }
+    }
+
     /**
      * Merges chunk items into the dedupe map.
      * チャンクのアイテム群を重複排除 Map へ取り込む処理。

package/src/client.ts

@@ -16,6 +16,7 @@ export * from "./client/components/daily-report-resolved-content"
 export * from "./client/components/unread-indicator"
 export * from "./client/config-context"
 export * from "./client/contexts/daily-report-action-context"
+export * from "./client/contexts/daily-report-error-context"
 export * from "./client/hooks/use-animated-number"
 export * from "./client/hooks/use-daily-report"
 export * from "./client/hooks/use-daily-report-comments"

package/src/server/handlers.action.spec.ts

@@ -0,0 +1,44 @@
+/**
+ * Action-endpoint tests: the dev-only clearCache gate.
+ * action エンドポイントのテスト: 開発専用 clearCache の封鎖ゲート。
+ */
+import { describe, expect, it } from "vitest"
+import { createDailyReportHandlers } from "./handlers"
+import { makeHandlersFixture } from "./test-helpers/handlers-config"
+
+/** POST /action の引数を組み立てる (FormData ボディ付き)。 */
+const makeActionArgs = (intent: string) => {
+    const body = new FormData()
+    body.set("intent", intent)
+    return {
+        request: new Request("http://localhost/daily_report/api/action", { method: "POST", body }),
+        params: { endpoint: "action" } as Record<string, string | undefined>,
+    }
+}
+
+describe("action / clearCache の封鎖ゲート", () => {
+    it("既定 (未注入) では clearCache を 400 で拒否し、service.clearCache を呼ばない", async () => {
+        // 認証済みなら誰でも全ワーカーのキャッシュを無制限破棄できてしまうため既定封鎖
+        const { config, calls } = makeHandlersFixture()
+        const { api } = createDailyReportHandlers(config)
+        const res = await api.action(makeActionArgs("clearCache"))
+        expect(res.status).toBe(400)
+        expect(calls.clearCache).toBe(0)
+    })
+
+    it("enableDevCacheClear:true のときだけ clearCache を実行する", async () => {
+        const { config, calls } = makeHandlersFixture({ enableDevCacheClear: true })
+        const { api } = createDailyReportHandlers(config)
+        const res = await api.action(makeActionArgs("clearCache"))
+        expect(res.status).toBe(200)
+        expect(calls.clearCache).toBe(1)
+    })
+
+    it("未認証は clearCache 以前に 401 で弾く", async () => {
+        const { config, calls } = makeHandlersFixture({ user: null, enableDevCacheClear: true })
+        const { api } = createDailyReportHandlers(config)
+        const res = await api.action(makeActionArgs("clearCache"))
+        expect(res.status).toBe(401)
+        expect(calls.clearCache).toBe(0)
+    })
+})

package/src/server/handlers.ids-stream.spec.ts

@@ -12,7 +12,7 @@
  * - **`Cache-Control: no-transform`** — compression ミドルウェアのバッファリング防止が
  *   ストリーミングの成立条件そのもの。
  */
-import { describe, expect, it } from "vitest"
+import { describe, expect, it, vi } from "vitest"
 import type { DailyReportIdsStreamChunkLine, DailyReportIdsStreamLine } from "../shared/ids-stream"
 import type { DailyReportItem } from "../shared/types"
 import { createDailyReportHandlers } from "./handlers"
@@ -360,3 +360,55 @@ describe("ids-stream / 中断", () => {
         expect(await response.text()).toBe("")
     })
 })
+
+describe("ids-stream / ハートビート", () => {
+    it("次チャンク待ち (全量クエリ確定待ち) が長引く間は空行ハートビートを送る", async () => {
+        vi.useFakeTimers()
+        try {
+            // 全量クエリを保留させ、先頭ページ (200 件) 描画後に無音区間を作る
+            let releaseFull: () => void = () => {}
+            const fullGate = new Promise<void>((resolve) => {
+                releaseFull = resolve
+            })
+            const items = makeItems(400)
+            const { config } = makeHandlersFixture({
+                idsStreamItems: items,
+                idsStreamFirstPageItems: items.slice(0, 200),
+                idsStreamFullGate: fullGate,
+            })
+            const { api } = createDailyReportHandlers(config)
+            const response = await api.loader(makeApiArgs("ids-stream"))
+            const reader = (response.body as ReadableStream<Uint8Array>).getReader()
+            const decoder = new TextDecoder()
+
+            // 先頭ページの最初のチャンク (200 件、isLast 無し)
+            const first = await reader.read()
+            const firstLine = JSON.parse(decoder.decode(first.value).trim()) as DailyReportIdsStreamChunkLine
+            expect(firstLine.items).toHaveLength(200)
+            expect(firstLine.isLast).toBe(false)
+
+            // 全量ゲート未解放のまま 15s 経過 → 接続の生存を示す空行
+            const heartbeatRead = reader.read()
+            await vi.advanceTimersByTimeAsync(15_000)
+            const heartbeat = await heartbeatRead
+            expect(decoder.decode(heartbeat.value)).toBe("\n")
+
+            // 全量を解放すると残りが流れて isLast で閉じる (ハートビートが本体を壊していない)
+            releaseFull()
+            let sawLast = false
+            for (let i = 0; i < 50; i++) {
+                const { done, value } = await reader.read()
+                if (done) break
+                const text = decoder.decode(value).trim()
+                if (text === "") continue
+                for (const raw of text.split("\n")) {
+                    const line = JSON.parse(raw) as DailyReportIdsStreamChunkLine
+                    if (line.isLast) sawLast = true
+                }
+            }
+            expect(sawLast).toBe(true)
+        } finally {
+            vi.useRealTimers()
+        }
+    })
+})

package/src/server/handlers.ts

@@ -52,6 +52,19 @@ 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)。
@@ -131,6 +144,9 @@ const buildIdsStreamResponse = (cookie: string | null, lines: AsyncGenerator<Dai
     // フラグで決定的に識別し、通常イベントとして静かに畳む (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) {
@@ -142,8 +158,36 @@ const buildIdsStreamResponse = (cookie: string | null, lines: AsyncGenerator<Dai
                 }
                 return
             }
+            let heartbeatTimer: ReturnType<typeof setTimeout> | null = null
             try {
-                const { value, done } = await lines.next()
+                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
@@ -160,6 +204,11 @@ const buildIdsStreamResponse = (cookie: string | null, lines: AsyncGenerator<Dai
                 }
                 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
@@ -469,6 +518,13 @@ export type DailyReportHandlersConfig = {
     streamKey: string
     /** 未ログイン時のリダイレクト先 (index.loader 用、既定 "/auth/login")。 */
     loginRedirectPath?: string
+    /**
+     * `intent=clearCache` (全サーバーキャッシュの一括破棄) を許可するか。既定 false。
+     * 開発用アフォーダンスであり、有効化すると認証済みユーザーなら誰でも全ワーカーの
+     * キャッシュを無制限に破棄できるため、本番では必ず無効のままにする
+     * (消費アプリは開発環境でのみ true を注入する)。
+     */
+    enableDevCacheClear?: boolean
     /** ロガー (既定は console ベース)。 */
     logger?: DailyReportLogger
 }
@@ -561,7 +617,9 @@ export function createDailyReportHandlers(config: DailyReportHandlersConfig) {
             fullArrayPromise.catch(() => {})
 
             if (parsed.cursor) {
-                // 再開接続: クライアントは高速先頭を受信済みなので全量確定を待つ (エラーは 500)
+                // 再開接続: クライアントは高速先頭を受信済みなので全量確定を待つ (エラーは 500)。
+                // 全量クエリはアプリ側の requestTimeout (既定 60s) で境界づけられ、確定後は
+                // スライスだけなので、接続確立後の無音はハートビート対象にならない
                 return buildIdsStreamResponse(cookie, streamLinesFromFullArray(await fullArrayPromise, parsed.cursor), request, apiLogger)
             }
 
@@ -700,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" })
         }
@@ -807,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)
+    })
+})

package/src/server/service.ts

@@ -1153,15 +1153,50 @@ export function createDailyReportService(config: DailyReportServiceConfig) {
         return updatedStatus
     }
 
+    /**
+     * Resolves the SSE recipient for comment events on a possibly-draft report.
+     * 下書きの可能性がある日報のコメントイベントについて SSE の宛先を解決する処理。
+     *
+     * 下書きは所有者にしか見えないため、下書きラベルが付いている間はコメント本文も
+     * 所有者だけに配信する (無指定だと本文が全ユーザーへブロードキャストされる)。
+     * 公開済み (下書きラベル無し) は全員配信 = undefined を返す。
+     *
+     * @param hubId Parent hub id. 親日報の hub ID。
+     * @param ownerUserId Hub owner's raw user id. 所有者の内部ユーザー ID。
+     * @returns Recipient raw user id, or undefined to broadcast. 宛先 (全員配信は undefined)。
+     */
+    const resolveCommentRecipient = async (hubId: number, ownerUserId: number | null): Promise<number | undefined> => {
+        // 所有者不明 (共有日報など) は従来どおり全員配信
+        if (ownerUserId === null) return undefined
+        const draftIds = await getDraftLabelJoinIds()
+        const [rel] = await db
+            .select<{ hubId: number }>({ hubId: DailyReportHub_Label.hubId })
+            .top(1)
+            .from(DailyReportHub_Label)
+            .where(and(eq(DailyReportHub_Label.hubId, hubId), inArray(DailyReportHub_Label.labelId, draftIds)))
+        return rel ? ownerUserId : undefined
+    }
+
     /**
      * Adds a comment to a daily report.
      * 日報にコメントを追加する。
      */
     const addComment = async (userId: number, reportHubId: number, content: string, businessDate: string | null, clientTempId: string): Promise<DailyReportCommentItem> => {
-        const [hub] = await db.select<{ sourceType: string }>({ sourceType: DailyReportHub.sourceType }).top(1).from(DailyReportHub).where(eq(DailyReportHub.id, reportHubId))
-        if (hub && hub.sourceType.toLowerCase() !== "internal") {
+        const [hub] = await db
+            .select<{ sourceType: string; deletedAt: Date | string | null; userId: number | null }>({ sourceType: DailyReportHub.sourceType, deletedAt: DailyReportHub.deletedAt, userId: DailyReportHub.userId })
+            .top(1)
+            .from(DailyReportHub)
+            .where(eq(DailyReportHub.id, reportHubId))
+        // 存在しない / 論理削除済みの日報にはコメントを付けない (以前は hub 未取得でも
+        // フォールスルーして孤児コメントを INSERT していた)
+        if (!hub || hub.deletedAt) {
+            throw new Error("Not Found")
+        }
+        if (hub.sourceType.toLowerCase() !== "internal") {
             throw new Error("Comments are restricted for external daily report sources")
         }
+        // 下書きなら SSE を所有者宛に絞る (本文の全員配信を防ぐ)
+        const recipientRawUserId = await resolveCommentRecipient(reportHubId, hub.userId)
 
         const [inserted] = await db
             .insert<DailyReportCommentRow>(DailyReportCommentModel)
@@ -1206,6 +1241,7 @@ export function createDailyReportService(config: DailyReportServiceConfig) {
                 reportHubId: reportHubId,
                 comment: commentItem,
                 clientTempId,
+                recipientRawUserId,
             }),
             "addComment",
         )
@@ -1233,6 +1269,15 @@ export function createDailyReportService(config: DailyReportServiceConfig) {
         if (comments[0].userId !== userId) {
             throw new Error("Unauthorized")
         }
+        // 渡された reportHubId がコメントの実際の所属と一致することを確認する。
+        // 不一致のまま進むと、無関係な日報のキャッシュ無効化と SSE 配信を引き起こす
+        if (comments[0].hubId !== reportHubId) {
+            throw new Error("Not Found")
+        }
+
+        // 下書きなら削除イベントも所有者宛に絞る (comment-add と対称)
+        const [hubForDelete] = await db.select<{ userId: number | null }>({ userId: DailyReportHub.userId }).top(1).from(DailyReportHub).where(eq(DailyReportHub.id, reportHubId))
+        const recipientRawUserId = await resolveCommentRecipient(reportHubId, hubForDelete?.userId ?? null)
 
         await db.delete(DailyReportCommentModel).where(eq(DailyReportCommentModel.id, commentId))
 
@@ -1254,6 +1299,7 @@ export function createDailyReportService(config: DailyReportServiceConfig) {
                 reportHubId: reportHubId,
                 commentId,
                 clientTempId,
+                recipientRawUserId,
             }),
             "deleteComment",
         )
@@ -1504,9 +1550,20 @@ export function createDailyReportService(config: DailyReportServiceConfig) {
             }
         })
 
-        // キャッシュ無効化
+        // キャッシュ無効化: 一覧プレビューは詳細キャッシュから引くため、詳細を全プロセス横断で落とす。
+        // 以前は読み取りに使われないプレフィックスを無効化し epoch も上げていなかったため、
+        // 編集後の件名/本文が他ワーカーで古いまま配信されていた
+        sqlResultCache.invalidate(`daily-report:detail:${reportHubId}`)
         sqlResultCache.invalidatePrefix(`daily-report:detail:${reportHubId}:user:`)
         sqlResultCache.invalidatePrefix(DAILY_REPORT_BUSINESS_DATE_CACHE_PREFIX)
+        await incrementRedisEpoch(`${DAILY_REPORT_DETAIL_EPOCH_PREFIX}${reportHubId}`)
+        // 編集内容は全ユーザーの一覧に映るため、営業日別一覧は全ユーザー分 + 全ワーカー横断で
+        // 無効化する (addComment と同じ流儀。編集者本人のキーだけでは他者へ古い件名が残る)
+        const editedBusinessDate = normalizeBusinessDateKey(formatDateValue(report[0].businessDate, "YYYY-MM-DD"))
+        if (editedBusinessDate) {
+            sqlResultCache.invalidatePrefix(`daily-report:date:${editedBusinessDate}:user:`)
+            await incrementRedisEpoch(`${DAILY_REPORT_DATE_EPOCH_PREFIX}${editedBusinessDate}`)
+        }
 
         const fullDetail = await getDailyReportDetailById(reportHubId, userId, { forceRefresh: true })
         if (fullDetail) {