@aiquants/daily-report 0.18.2 → 0.20.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiquants/daily-report",
-  "version": "0.18.2",
+  "version": "0.20.0",
   "description": "Reusable daily-report feature package: shared types/schemas, React (Router v7 / v8) UI with virtual scrolling + optimistic updates + SSE sync, and a drizzle (mssql) server layer with DI ports (auth, user resolution, redis, external sources).",
   "sideEffects": [
     "**/*.css"
@@ -44,6 +44,7 @@
   "author": "AI Quants",
   "license": "MIT",
   "dependencies": {
+    "@aiquants/mssql": "^0.1.1",
     "@heroicons/react": "^2.2.0",
     "@radix-ui/react-label": "^2.1.8",
     "@radix-ui/react-slot": "^1.2.4",
@@ -55,6 +56,9 @@
     "tailwind-merge": "^3.5.0"
   },
   "devDependencies": {
+    "@aiquants/resize-panels": "^2.0.0",
+    "@aiquants/swipe-overlay": "^1.2.4",
+    "@aiquants/virtualscroll": "^3.5.0",
     "@tailwindcss/cli": "^4.1.18",
     "@testing-library/jest-dom": "^6.9.1",
     "@testing-library/react": "^16.3.2",
@@ -72,20 +76,17 @@
     "tsup": "^8.5.1",
     "typescript": "^5.9.3",
     "vitest": "^4.1.10",
-    "zod": "^3.25.76",
-    "@aiquants/resize-panels": "^1.9.0",
-    "@aiquants/swipe-overlay": "^1.2.4",
-    "@aiquants/virtualscroll": "^3.4.0"
+    "zod": "^3.25.76"
   },
   "peerDependencies": {
+    "@aiquants/resize-panels": "^2.0.0",
+    "@aiquants/swipe-overlay": "^1.2.4",
+    "@aiquants/virtualscroll": "^3.5.0",
     "drizzle-orm": ">=1.0.0-beta.4 <2.0.0",
     "react": ">=18",
     "react-dom": ">=18",
     "react-router": "^7.0.0 || ^8.0.0",
-    "zod": ">=3.25.0 <5.0.0",
-    "@aiquants/resize-panels": "^1.9.0",
-    "@aiquants/swipe-overlay": "^1.2.4",
-    "@aiquants/virtualscroll": "^3.4.0"
+    "zod": ">=3.25.0 <5.0.0"
   },
   "peerDependenciesMeta": {
     "drizzle-orm": {

package/src/client/components/daily-report-resize-panels.spec.tsx

@@ -151,7 +151,9 @@ describe("DailyReportList / リサイズパネル統合", () => {
             const handle = container.querySelector('[data-resize-handle-id="daily-report-list-handle"]')
             expect(handle).toBeInTheDocument()
             expect(handle).toHaveAttribute("aria-label")
-            expect(handle?.getAttribute("aria-label")).toContain("Resize")
+            // 支援技術が読み上げるのは、このパッケージが与えた名前でなければならない
+            // (@aiquants/resize-panels 1.x は利用者の aria-label を黙って捨て、自前の英語名を出していた)
+            expect(handle?.getAttribute("aria-label")).toBe("日報一覧と詳細ペインの境界リサイズ")
 
             // 視覚インジケーターおよび内部バーの存在検証
             const indicator = handle?.querySelector("[data-resize-handle-indicator]")

package/src/server/service.ts

@@ -4,6 +4,8 @@
  *
  * キャッシュキー・無効化順序・SSE publish 順序を含むデータアクセスロジックの中核。
  */
+
+import { runWithTransactionRetry, type TransactionRetryOptions } from "@aiquants/mssql"
 import { aliasedTable, and, asc, desc, eq, getColumns, inArray, isNull, or, sql } from "drizzle-orm"
 import { normalizeBusinessDateKey } from "../shared/business-date"
 import { compareIdsStreamOrder, isIdsStreamOrdered } from "../shared/ids-stream"
@@ -131,6 +133,15 @@ export type DailyReportServiceConfig = {
     cache: SqlResultCache
     /** クロスワーカー epoch ストア (facade が生成して注入)。 */
     epochs: EpochStore
+    /**
+     * Retry policy for transactions SQL Server chose as the deadlock victim. See `@aiquants/mssql`.
+     * サーバがロールバックしたトランザクションの再試行方針。
+     *
+     * 鍵の範囲が重なる並行操作はデッドロックしうる。犠牲に選ばれた側はサーバによって
+     * ロールバックされる — 何もコミットされていないので、書き直す再実行が正しい。既定で 3 回試す。
+     * `{ attempts: 1 }` で再試行しない。
+     */
+    transactionRetry?: TransactionRetryOptions
     /**
      * 下書きラベル名 (単一または配列で指定可能)。
      * 消費アプリ側で自 DB のラベル名や作成区分の候補名 ("下書き", "DRAFT" など) を注入できる。
@@ -219,6 +230,16 @@ export function createDailyReportService(config: DailyReportServiceConfig) {
     const draftLabelNames = config.draftLabelNames && config.draftLabelNames.length > 0 ? config.draftLabelNames : [config.draftLabelName ?? "下書き"]
     const draftLabelName = draftLabelNames[0] ?? "下書き"
     const logger = config.logger ?? createLogger(LogLevel.INFO, "[DailyReportService]")
+    /**
+     * 全トランザクションの唯一の入口。サーバがロールバックした場合だけやり直す。
+     *
+     * ⚠️ **閉包を DB の外へ波及させないこと。** 再実行は毎回はじめから走るため、ロールバックで元に
+     * 戻らない副作用 (キャッシュ無効化・epoch 更新・SSE 配信) を閉包へ入れてはならない。現在それらは
+     * すべてトランザクションの**後**に置かれている。
+     */
+    // 閉包を capture で包む。包まないと ORM が 1205 を EABORT へ置き換えた後しか見えず、
+    // デッドロックだと判定できなくなる
+    const inTransaction = <T>(fn: (tx: DailyReportDb) => Promise<T>): Promise<T> => runWithTransactionRetry((capture) => db.transaction((tx) => capture(() => fn(tx))), config.transactionRetry)
 
     const DAILY_REPORT_IDS_CACHE_KEY = "daily-report:ids"
     const DAILY_REPORT_IDS_TTL_MS = config.idsTtlMs ?? 180_000
@@ -1592,7 +1613,7 @@ export function createDailyReportService(config: DailyReportServiceConfig) {
         const userName = user?.displayName ?? encodeUserId(userId)
 
         try {
-            const result = await db.transaction(async (tx) => {
+            const result = await inTransaction(async (tx) => {
                 logger.info("Starting transaction")
                 // 1. Hub作成
                 const sourceId = `internal-temp-${Date.now()}-${Math.random()}` // 一時的なID
@@ -1817,7 +1838,7 @@ export function createDailyReportService(config: DailyReportServiceConfig) {
             throw new Error("Unauthorized")
         }
 
-        await db.transaction(async (tx) => {
+        await inTransaction(async (tx) => {
             if (data.title !== undefined) {
                 await updateDailyReportHub(tx, reportHubId, {
                     title: data.title,

package/src/server/transaction-retry.spec.ts

@@ -0,0 +1,63 @@
+/**
+ * @module transaction-retry.spec
+ * Tests that every transaction in this service goes through the retry seam.
+ * 本サービスの全トランザクションが再試行の接ぎ口を通っていることを検証するテストモジュール。
+ *
+ * The distinction between "rolled back by the server" and "might have committed" lives in
+ * `@aiquants/mssql` and is tested there. What can only be tested here is whether the
+ * service actually routes through it — a seam nobody calls is the same as no seam, and a per-call-site
+ * test cannot cover a site that does not exist yet.
+ * 「サーバがロールバックした」と「コミットされたかもしれない」の区別は `@aiquants/mssql`
+ * が持ち、そこで検証されている。ここでしか検証できないのは、サービスが実際にそれを通っているかで
+ * ある — 誰も呼ばない接ぎ口は接ぎ口が無いことと同じであり、呼び出し箇所ごとの検査では**まだ存在
+ * しない箇所**を覆えない。
+ */
+
+import { describe, expect, it } from "vitest"
+
+/*
+ * `import.meta.glob` は vite が解決する。型は `vite/client` が持つが、それを本パッケージの
+ * `tsconfig` へ足すと DOM 由来の型も一緒に入る。必要な 1 つだけを名指しする。
+ */
+const globSources = () =>
+    (import.meta as unknown as { glob: (pattern: string, options: { query: string; import: string; eager: boolean }) => Record<string, string> }).glob("./**/*.ts", {
+        query: "?raw",
+        import: "default",
+        eager: true,
+    })
+
+describe("トランザクションの入口は 1 つだけである", () => {
+    const sources = globSources()
+
+    it("`db.transaction(` を直に呼ぶのは接ぎ口の定義だけ", () => {
+        /*
+         * 呼び出し箇所で直に呼ぶ形に戻すと、方針を変えるたびに全箇所を触ることになり、いずれ
+         * 1 箇所だけが取り残される。取り残された箇所は、再試行されないことを誰にも告げない。
+         */
+        const offenders = Object.entries(sources)
+            .filter(([path]) => !path.endsWith(".spec.ts"))
+            .filter(([, source]) => {
+                // 接ぎ口自身の 1 行 (`runWithTransactionRetry((capture) => db.transaction(...))`) は除く
+                const direct = source.match(/\bdb\.transaction\(/g) ?? []
+                const inSeam = source.match(/runWithTransactionRetry\(\(capture\) => db\.transaction\(/g) ?? []
+                return direct.length > inSeam.length
+            })
+            .map(([path]) => path)
+
+        expect(offenders).toEqual([])
+    })
+
+    it("接ぎ口が再試行へ繋がり、原因を押さえている", () => {
+        /*
+         * `capture` を落とすと ORM が 1205 を `EABORT` へ置き換えた後しか見えず、デッドロックが
+         * 一度も再試行されなくなる — 例外は出ないので、静かに効かなくなる。
+         */
+        expect(sources["./service.ts"]).toContain("runWithTransactionRetry((capture) => db.transaction((tx) => capture(() => fn(tx))), config.transactionRetry)")
+    })
+
+    it("走査が実際に源を読めている (空振りで緑にならない)", () => {
+        // 走査対象が 0 件でも上の検査は通る。件数と、必ず存在する印を先に確かめる。
+        expect(Object.keys(sources).length).toBeGreaterThan(5)
+        expect(sources["./service.ts"]).toContain("createDailyReportService")
+    })
+})