@aiquants/daily-report 0.1.1

package/src/client/hooks/use-daily-report-comments.ts

@@ -0,0 +1,73 @@
+/**
+ * Comment management hook merging server data with optimistic local updates.
+ * サーバーデータとローカルの楽観的更新を統合して日報コメントを管理するフック。
+ */
+import { useCallback, useMemo, useRef, useState } from "react"
+import { mergeComments, type UIComment } from "../../shared/comment-adapter"
+import type { DailyReportDetail } from "../../shared/types"
+import { useDailyReportActionContext } from "../contexts/daily-report-action-context"
+
+type User = { displayName?: string; name?: { familyName?: string; givenName?: string }; emails?: { value: string }[] }
+
+/**
+ * Manages daily report comments, integrating server data with optimistic local updates.
+ * サーバーデータとローカルの楽観的更新を統合して日報コメントを管理するフック。
+ */
+export const useDailyReportComments = (report: DailyReportDetail | null, user: User | null | undefined, userId?: string | null) => {
+    const { pendingAddCommentIds, pendingDeleteCommentIds, resolvedIdMap, version, addComment, deleteComment } = useDailyReportActionContext()
+    const [pendingDeleteId, setPendingDeleteId] = useState<number | string | null>(null)
+
+    const handleDeleteComment = useCallback((cId: number) => report && deleteComment(report.reportHubId, cId, report.date ?? ""), [report, deleteComment])
+    const handleAddComment = useCallback((content: string) => (report && user ? addComment(report.reportHubId, content, report.date ?? "") : Promise.resolve()), [report, user, addComment])
+
+    const prevUiCommentsRef = useRef<UIComment[]>([])
+
+    // UI用コメントリストの生成: サーバーデータ、レガシーコメント、楽観的更新（追加・削除）をマージして表示用データを構築する。
+    // 依存関係: [report, user, userId, pendingDeleteCommentIds, pendingAddCommentIds, resolvedIdMap, version]
+    // クリーンアップ: なし
+    // biome-ignore lint/correctness/useExhaustiveDependencies: version は ref (pending*) 変更時の再計算トリガーとして意図的に依存させる
+    const uiComments = useMemo(() => {
+        if (!report) return []
+
+        // 1. Base: Merge legacy and new comments
+        // 基本データ: レガシーコメントと新しい形式のコメントをマージ。
+        let comments = mergeComments(report.comments, report.commentItems, userId)
+
+        // 2. Filter: Remove pending deletions
+        // フィルタリング: 削除保留中のコメントを除外。
+        const deletedIds = pendingDeleteCommentIds.current.get(report.reportHubId)
+        if (deletedIds?.size) comments = comments.filter((c) => !deletedIds.has(c.id as number))
+
+        // 3. Add: Append pending optimistic additions
+        // 追加: 楽観的に追加されたコメントを末尾に追加。
+        const addedComments = pendingAddCommentIds.current.get(report.reportHubId)
+        if (addedComments?.size && user) {
+            const existingIds = new Set(comments.map((c) => c.id))
+            const authorName = user.displayName?.trim() || (user.name ? `${user.name.familyName ?? ""} ${user.name.givenName ?? ""}`.trim() : "") || user.emails?.[0]?.value || "Unknown"
+
+            for (const [tempId, content] of addedComments) {
+                if (deletedIds?.has(tempId)) continue
+                if (existingIds.has(tempId)) continue
+
+                // Skip if real ID resolved and already present
+                // 実IDが解決済みで既にリストに存在する場合はスキップ。
+                const realId = resolvedIdMap.current.get(tempId)
+                if (realId && existingIds.has(realId)) continue
+
+                // Skip if content duplicated (server data arrived before ID resolution)
+                // 内容が重複している場合（ID解決前にサーバーデータが到達した場合）はスキップ。
+                if (comments.some((c) => c.isMine && c.content === content && typeof c.id === "number" && c.id > 0)) continue
+
+                comments.push({ id: tempId, authorName, content, createdAt: new Date().toISOString(), isMine: true, isLegacy: false })
+            }
+        }
+
+        // Stable reference check
+        // 参照の安定性チェック: 内容に変更がなければ前回の参照を返すことで不要な再レンダリングを防ぐ。
+        if (JSON.stringify(comments) === JSON.stringify(prevUiCommentsRef.current)) return prevUiCommentsRef.current
+        prevUiCommentsRef.current = comments
+        return comments
+    }, [report, user, userId, pendingDeleteCommentIds, pendingAddCommentIds, resolvedIdMap, version])
+
+    return { uiComments, handleAddComment, handleDeleteComment, pendingDeleteId, setPendingDeleteId, resolvedIdMap: resolvedIdMap.current }
+}

package/src/client/hooks/use-daily-report-sse-connection.ts

@@ -0,0 +1,86 @@
+/**
+ * SSE connection lifecycle hook for daily-report real-time updates.
+ * 日報リアルタイム更新の SSE 接続ライフサイクルフック。
+ */
+import { useEffect } from "react"
+import { createLogger, LogLevel } from "../../shared/logger"
+import { type DailyReportSseMessage, dailyReportSseMessageSchema } from "../../shared/sse-schema"
+import { useDailyReportConfig } from "../config-context"
+
+const logger = createLogger(LogLevel.INFO, "daily-report-sse-connection")
+
+type UseDailyReportSseConnectionOptions = {
+    isSseEnabled: boolean
+    onMessage: (payload: Exclude<DailyReportSseMessage, { type: "connected" }>) => void
+}
+
+/**
+ * Manages Daily Report SSE connection lifecycle with reconnect handling.
+ * Handles open, message parse/dispatch, error logging, retry, and cleanup.
+ *
+ * 日報 SSE 接続のライフサイクルを管理する Hook。
+ * 接続開始、メッセージ解析と配送、エラー時再接続、クリーンアップを管理します。
+ */
+export const useDailyReportSseConnection = ({ isSseEnabled, onMessage }: UseDailyReportSseConnectionOptions) => {
+    const { ssePath } = useDailyReportConfig()
+
+    /**
+     * Maintains SSE connection while enabled and retries on error.
+     * isSseEnabled が true の間は接続を維持し、エラー時は再接続します。
+     */
+    useEffect(() => {
+        if (!isSseEnabled) return
+
+        let eventSource: EventSource | null = null
+        let reconnectTimer: ReturnType<typeof setTimeout> | null = null
+        let reconnectAttempt = 0
+        let lastReceivedEventId: string | undefined
+
+        const connect = () => {
+            eventSource?.close()
+            eventSource = new EventSource(`${ssePath}${lastReceivedEventId ? `?lastEventId=${lastReceivedEventId}` : ""}`)
+
+            eventSource.onopen = () => {
+                reconnectAttempt = 0
+                logger.info("SSE Connected")
+            }
+
+            eventSource.onmessage = (event) => {
+                if (event.lastEventId) {
+                    lastReceivedEventId = event.lastEventId
+                }
+                try {
+                    const raw = JSON.parse(event.data)
+                    const result = dailyReportSseMessageSchema.safeParse(raw)
+                    if (!result.success) {
+                        logger.error("SSE Validation Error", result.error.format())
+                        return
+                    }
+                    if (result.data.type === "connected") {
+                        logger.info("SSE Subscribe Ready")
+                        return
+                    }
+                    onMessage(result.data)
+                } catch (e) {
+                    logger.error("SSE Parse Error", e)
+                }
+            }
+
+            eventSource.onerror = (e) => {
+                eventSource?.close()
+                eventSource = null
+                reconnectAttempt++
+                const delayMs = Math.min(2000 * 2 ** (reconnectAttempt - 1), 30000)
+                logger.error(`SSE Error. Retrying in ${delayMs}ms (attempt ${reconnectAttempt})...`, e)
+                reconnectTimer = setTimeout(connect, delayMs)
+            }
+        }
+
+        connect()
+
+        return () => {
+            if (eventSource) eventSource.close()
+            if (reconnectTimer) clearTimeout(reconnectTimer)
+        }
+    }, [isSseEnabled, onMessage, ssePath])
+}

package/src/client/hooks/use-daily-report.spec.ts

@@ -0,0 +1,155 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"
+import type { DailyReportDetail } from "../../shared/types"
+
+/**
+ * Unit tests for DailyReportCache public API and getGlobal behavior.
+ * DailyReportCache のパブリック API と getGlobal の挙動を検証するユニットテスト。
+ */
+
+// テスト用のモック日報データを生成するヘルパー関数
+const makeMockReport = (overrides: Partial<DailyReportDetail> = {}): DailyReportDetail =>
+    ({
+        reportHubId: 1,
+        subject: "Test Report",
+        content: "Test Content",
+        date: "2026-01-01",
+        labels: [],
+        commentItems: [],
+        isRead: false,
+        isStar: false,
+        updatedAt: new Date().toISOString(),
+        ...overrides,
+    }) as DailyReportDetail
+
+describe("DailyReportCache public API", () => {
+    // vitest は jsdom 環境で動作するため window オブジェクトが存在する。
+    // vi.resetModules() + 動的 import でモジュールを再読み込みし、
+    // getGlobal がカスタム TTL を読み込むようにする。
+
+    let writeCache: typeof import("./use-daily-report").writeCache
+    let getCachedReport: typeof import("./use-daily-report").getCachedReport
+    let updateDailyReportCache: typeof import("./use-daily-report").updateDailyReportCache
+    let applyDailyReportServerUpdates: typeof import("./use-daily-report").applyDailyReportServerUpdates
+
+    beforeEach(async () => {
+        // モジュールキャッシュをリセットして、getGlobal がカスタム TTL を読み込むようにする
+        vi.resetModules()
+        // テスト用に短い TTL を設定
+        ;(window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_TTL__ = 5000
+        ;(window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_REVIVE_MS__ = 1000
+
+        // モジュールを動的にインポート（各テストで独立したキャッシュインスタンスを使用）
+        const mod = await import("./use-daily-report")
+        writeCache = mod.writeCache
+        getCachedReport = mod.getCachedReport
+        updateDailyReportCache = mod.updateDailyReportCache
+        applyDailyReportServerUpdates = mod.applyDailyReportServerUpdates
+    })
+
+    afterEach(() => {
+        // window グローバル変数をクリーンアップ
+        delete (window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_TTL__
+        delete (window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_REVIVE_MS__
+    })
+
+    describe("writeCache (DailyReportCache.set)", () => {
+        it("レポートを格納し、ID で取得できる", () => {
+            const report = makeMockReport()
+            writeCache(report, Date.now() + 5000)
+            const cached = getCachedReport(1)
+            expect(cached).toBeDefined()
+            expect(cached?.reportHubId).toBe(1)
+            expect(cached?.subject).toBe("Test Report")
+        })
+
+        it("同一 ID の set は完全置換される", () => {
+            writeCache(makeMockReport({ subject: "Original" }), Date.now() + 5000)
+            writeCache(makeMockReport({ subject: "Replaced", content: "New" }), Date.now() + 5000)
+            const cached = getCachedReport(1)
+            expect(cached?.subject).toBe("Replaced")
+            expect(cached?.content).toBe("New")
+        })
+
+        it("期限切れエントリは null を返す", () => {
+            writeCache(makeMockReport(), Date.now() - 1)
+            expect(getCachedReport(1)).toBeNull()
+        })
+    })
+
+    describe("updateDailyReportCache (DailyReportCache.update)", () => {
+        it("既存レポートの指定フィールドのみを更新し、他は保持する", () => {
+            writeCache(makeMockReport({ subject: "Original", content: "Original Content" }), Date.now() + 5000)
+            updateDailyReportCache(1, { subject: "Updated Title" } as Partial<DailyReportDetail>)
+            const cached = getCachedReport(1)
+            expect(cached?.subject).toBe("Updated Title")
+            expect(cached?.content).toBe("Original Content")
+        })
+
+        it("updatedAt が古い更新はスキップされる (stale guard)", () => {
+            writeCache(makeMockReport({ subject: "Current", updatedAt: "2026-01-02T00:00:00Z" }), Date.now() + 5000)
+            updateDailyReportCache(1, {
+                subject: "Stale",
+                updatedAt: "2026-01-01T00:00:00Z",
+            } as Partial<DailyReportDetail>)
+            expect(getCachedReport(1)?.subject).toBe("Current")
+        })
+
+        it("未キャッシュの ID を更新しても何も起きない", () => {
+            updateDailyReportCache(999, { subject: "Ghost" } as Partial<DailyReportDetail>)
+            expect(getCachedReport(999)).toBeNull()
+        })
+    })
+
+    describe("applyDailyReportServerUpdates", () => {
+        it("キャッシュ済みレポートを完全置換する", () => {
+            writeCache(makeMockReport({ subject: "Old", content: "Old" }), Date.now() + 1000)
+            applyDailyReportServerUpdates(1, makeMockReport({ subject: "Fresh", content: "Fresh" }))
+            const cached = getCachedReport(1)
+            expect(cached?.subject).toBe("Fresh")
+            expect(cached?.content).toBe("Fresh")
+        })
+
+        it("未キャッシュの ID でも新規エントリとして格納される", () => {
+            applyDailyReportServerUpdates(42, makeMockReport({ reportHubId: 42, subject: "New" }))
+            expect(getCachedReport(42)?.subject).toBe("New")
+        })
+    })
+})
+
+describe("getGlobal behavior (window グローバル変数の読み取り)", () => {
+    afterEach(() => {
+        delete (window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_TTL__
+        delete (window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_REVIVE_MS__
+    })
+
+    it("window に正の数値が設定されていれば、その値が使われる", async () => {
+        vi.resetModules()
+        ;(window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_TTL__ = 7777
+
+        const mod = await import("./use-daily-report")
+        const report = makeMockReport()
+        mod.writeCache(report, Date.now() + 7777)
+        expect(mod.getCachedReport(1)).toBeDefined()
+    })
+
+    it("window が undefined ならデフォルト値 (300000ms) が使われる", async () => {
+        vi.resetModules()
+        delete (window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_TTL__
+
+        const mod = await import("./use-daily-report")
+        const report = makeMockReport()
+        mod.writeCache(report, Date.now() + 300_000)
+        expect(mod.getCachedReport(1)).toBeDefined()
+    })
+
+    it("window に 0 が設定されていれば 0 が使われる (|| バグの修正確認)", async () => {
+        vi.resetModules()
+        ;(window as unknown as Record<string, unknown>).__DAILY_REPORT_CACHE_TTL__ = 0
+
+        const mod = await import("./use-daily-report")
+        const report = makeMockReport()
+        // TTL=0ms → expiresAt = Date.now() + 0 → 即時期限切れ → getCachedReport は null
+        mod.writeCache(report, Date.now() + 0)
+        expect(mod.getCachedReport(1)).toBeNull()
+    })
+})