chore: release v0.4.0

.github/workflows/deployment.yaml

@@ -43,6 +43,18 @@ jobs:
           CLOUDFLARE_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}
           CLOUDFLARE_ACCOUNT_ID: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
 
+      # public/se/ は規約上リポジトリに置けないので、checkout 直後は空。
+      # ここで取らないと音源の無いアセットがそのまま公開される。
+      # キャッシュが当たれば fetch-se は既存ファイルを skip し、相手のサイトは叩かない。
+      - name: Cache sound effects
+        uses: actions/cache@v4
+        with:
+          path: public/se
+          key: se-${{ hashFiles('scripts/fetch-se.ts') }}
+
+      - name: Fetch sound effects
+        run: bun run se:fetch
+
       - name: Build
         run: bun run build
 

__tests__/api/validation.test.ts

@@ -1,5 +1,6 @@
 import { describe, expect, test } from 'bun:test'
 import app from '@/server/index'
+import { MAX_TOPIC_CHARS } from '@/shared/turns'
 
 /**
  * bun test は Workers ランタイムの外で動くので、バインディング（DO / KV / Hyperdrive）を
@@ -120,21 +121,21 @@ describe('POST /api/sessions/:id/ask', () => {
     expect(res.status).toBe(400)
   })
 
-  test('話題が501文字なら 400', async () => {
+  test('話題が上限を1文字でも超えれば 400', async () => {
     const res = await postJson(`/api/sessions/${SESSION_ID}/ask`, {
       sessionId: SESSION_ID,
       characterId: CHARACTER_ID,
-      topic: 'あ'.repeat(501),
+      topic: 'あ'.repeat(MAX_TOPIC_CHARS + 1),
     })
 
     expect(res.status).toBe(400)
   })
 
-  test('500文字ちょうどはバリデーションを通過する（境界の内側）', async () => {
+  test('上限ちょうどはバリデーションを通過する（境界の内側）', async () => {
     const res = await postJson(`/api/sessions/${SESSION_ID}/ask`, {
       sessionId: SESSION_ID,
       characterId: CHARACTER_ID,
-      topic: 'あ'.repeat(500),
+      topic: 'あ'.repeat(MAX_TOPIC_CHARS),
     })
 
     // バインディングが無いのでこの先は進めない。ここで見たいのは
@@ -151,6 +152,28 @@ describe('POST /api/sessions/:id/ask', () => {
 
     expect(res.status).toBe(400)
   })
+
+  test('調べられる場所も相手として受ける', async () => {
+    // 人物は uuid、遺体は victim、場所は作者が書いたローカルID。同じ口へ来る。
+    const res = await postJson(`/api/sessions/${SESSION_ID}/ask`, {
+      sessionId: SESSION_ID,
+      characterId: 'choba',
+      topic: '帳面を見てみる',
+    })
+
+    expect(res.status).not.toBe(400)
+  })
+
+  test('相手の形をしていない文字列は 400', async () => {
+    // uuid でも victim でも、場所の ID の形でもないもの。
+    const res = await postJson(`/api/sessions/${SESSION_ID}/ask`, {
+      sessionId: SESSION_ID,
+      characterId: '帳場',
+      topic: '帳面を見てみる',
+    })
+
+    expect(res.status).toBe(400)
+  })
 })
 
 describe('POST /api/sessions/:id/accuse', () => {

__tests__/client/chat-log.test.ts

@@ -15,53 +15,82 @@ const textsOf = (item: ReturnType<typeof groupTurns>[number]): string[] =>
   item.kind === 'topic' ? [item.text] : item.lines.map((line) => line.text)
 
 describe('groupTurns', () => {
-  test('空行で区切られた返答を行に割る', () => {
-    const items = groupTurns([
-      turn('a1', 'assistant', 'えっ、Yさんですか？\n\nそうですね。いい人でしたよ。'),
-    ])
+  test('返答を一文ずつに割る', () => {
+    const items = groupTurns(
+      [turn('a1', 'assistant', 'えっ、Yさんですか？\n\nそうですね。いい人でしたよ。')],
+      false,
+    )
 
     expect(items).toHaveLength(1)
-    expect(textsOf(items[0])).toEqual(['えっ、Yさんですか？', 'そうですね。いい人でしたよ。'])
+    expect(textsOf(items[0])).toEqual(['えっ、Yさんですか？', 'そうですね。', 'いい人でしたよ。'])
   })
 
   test('改行が続いても区切りは1つと数える', () => {
-    const items = groupTurns([turn('a1', 'assistant', '前半\n\n\n\n後半')])
+    const items = groupTurns([turn('a1', 'assistant', '前半\n\n\n\n後半')], false)
 
     expect(textsOf(items[0])).toEqual(['前半', '後半'])
   })
 
   test('割った行にはそれぞれ別の鍵が付く', () => {
-    const items = groupTurns([turn('a1', 'assistant', '一つ目\n\n二つ目')])
+    const items = groupTurns([turn('a1', 'assistant', '一つ目\n\n二つ目')], false)
     const item = items[0]
 
     expect(item.kind === 'block' ? item.lines.map((line) => line.id) : []).toEqual(['a1:0', 'a1:1'])
   })
 
   test('続けて喋った分は塊にまとまり、行が並ぶ', () => {
-    const items = groupTurns([
-      turn('a1', 'assistant', '一つ目\n\n二つ目'),
-      turn('a2', 'assistant', '三つ目'),
-    ])
+    const items = groupTurns(
+      [turn('a1', 'assistant', '一つ目\n\n二つ目'), turn('a2', 'assistant', '三つ目')],
+      false,
+    )
 
     expect(items).toHaveLength(1)
     expect(textsOf(items[0])).toEqual(['一つ目', '二つ目', '三つ目'])
   })
 
   test('話題を挟むと塊が分かれる', () => {
-    const items = groupTurns([
-      turn('a1', 'assistant', '前の返答'),
-      turn('t1', 'topic', '次の話題'),
-      turn('u1', 'user', '次の質問'),
-    ])
+    const items = groupTurns(
+      [
+        turn('a1', 'assistant', '前の返答'),
+        turn('t1', 'topic', '次の話題'),
+        turn('u1', 'user', '次の質問'),
+      ],
+      false,
+    )
 
     expect(items.map((item) => item.kind)).toEqual(['block', 'topic', 'block'])
   })
 
+  test('流れている最中は、書きかけの一文を出さない', () => {
+    const items = groupTurns([turn('a1', 'assistant', 'そうですね。いい人でし')], true)
+
+    expect(textsOf(items[0])).toEqual(['そうですね。'])
+  })
+
+  test('流れ終われば、句点で終わらない一文も出す', () => {
+    const items = groupTurns([turn('a1', 'assistant', 'そうですね。……雨でしたから')], false)
+
+    expect(textsOf(items[0])).toEqual(['そうですね。', '……雨でしたから'])
+  })
+
+  test('末尾より前の返答は、流れている最中でも全部出す', () => {
+    const items = groupTurns(
+      [turn('a1', 'assistant', '前の返答は書き終わっている'), turn('a2', 'assistant', '書きかけ')],
+      true,
+    )
+
+    expect(textsOf(items[0])).toEqual(['前の返答は書き終わっている'])
+  })
+
   test('返答待ちの空のターンは置かない', () => {
-    expect(groupTurns([turn('a1', 'assistant', '')])).toEqual([])
+    expect(groupTurns([turn('a1', 'assistant', '')], true)).toEqual([])
   })
 
   test('最初に届いたのが改行だけでも、まだ置かない', () => {
-    expect(groupTurns([turn('a1', 'assistant', '\n\n')])).toEqual([])
+    expect(groupTurns([turn('a1', 'assistant', '\n\n')], true)).toEqual([])
+  })
+
+  test('一文目が出来上がるまでは、名前も出さない', () => {
+    expect(groupTurns([turn('a1', 'assistant', 'そうですね')], true)).toEqual([])
   })
 })

__tests__/client/deadline.test.ts

@@ -0,0 +1,66 @@
+import { describe, expect, test } from 'bun:test'
+import { deadlineOf } from '@/client/lib/deadline'
+
+/**
+ * 被害者。死亡推定時刻は持っていない——あれは開示済みかどうかをサーバが判断して
+ * セッションの状態で運ぶもので、シナリオ詳細には最初から入ってこない
+ * （docs/design/deadline-window.md）。
+ */
+const VICTIM = {
+  name: '水野英治',
+  introduction: '青雨堂店主',
+  foundAt: '19:10',
+  foundIn: '店の奥',
+  hasEstimatedDeathAt: true,
+  investigable: true,
+}
+
+describe('deadlineOf', () => {
+  test('開示前は死亡推定が不明のまま', () => {
+    expect(deadlineOf(VICTIM, null)).toEqual({
+      foundAt: '19:10',
+      label: '死亡推定',
+      death: { kind: 'unknown' },
+    })
+  })
+
+  /*
+   * 「まだ見つけていない」と「最初から無い」は別のこと。点線と ? は「ここに探すものがある」
+   * という誘いなので、死亡推定時刻を持たない事件に出すと、無いものを探させることになる。
+   */
+  test('死亡推定時刻を持たない事件では、印そのものを出さない', () => {
+    expect(deadlineOf({ ...VICTIM, hasEstimatedDeathAt: false }, null)).toEqual({
+      foundAt: '19:10',
+      label: '死亡推定',
+      death: undefined,
+    })
+  })
+
+  test('開示されると死亡推定が確定して出る', () => {
+    expect(deadlineOf(VICTIM, '18:50')).toEqual({
+      foundAt: '19:10',
+      label: '死亡推定',
+      death: { kind: 'fixed', at: '18:50' },
+    })
+  })
+
+  test('発見時刻を持たない事件では、遺体発見の線を出さない', () => {
+    expect(deadlineOf({ ...VICTIM, foundAt: null }, null)).toEqual({
+      foundAt: undefined,
+      label: '死亡推定',
+      death: { kind: 'unknown' },
+    })
+  })
+
+  test('発見時刻が分かっていなくても、開示された死亡推定は出る', () => {
+    expect(deadlineOf({ ...VICTIM, foundAt: null }, '18:50')).toEqual({
+      foundAt: undefined,
+      label: '死亡推定',
+      death: { kind: 'fixed', at: '18:50' },
+    })
+  })
+
+  test('被害者のいない事件では刻限そのものが無い', () => {
+    expect(deadlineOf(null, '18:50')).toBeUndefined()
+  })
+})

__tests__/client/restore.test.ts

@@ -13,7 +13,15 @@ describe('restoreConversations', () => {
       history([
         {
           characterId: 'a',
-          exchanges: [{ question: '昨夜どこに？', answer: '書斎です', askedAt: 100, topic: null }],
+          exchanges: [
+            {
+              question: '昨夜どこに？',
+              answer: '書斎です',
+              askedAt: 100,
+              topic: null,
+              yielded: false,
+            },
+          ],
         },
       ]),
     )
@@ -32,32 +40,71 @@ describe('restoreConversations', () => {
         {
           characterId: 'a',
           exchanges: [
-            { question: '昨夜どこに？', answer: '書斎です', askedAt: 100, topic: 'アリバイ' },
-            { question: '何時まで？', answer: '日付が変わる頃まで', askedAt: 100, topic: null },
+            {
+              question: '昨夜どこに？',
+              answer: '書斎です',
+              askedAt: 100,
+              topic: 'アリバイ',
+              yielded: false,
+            },
+            {
+              question: '何時まで？',
+              answer: '日付が変わる頃まで',
+              askedAt: 100,
+              topic: null,
+              yielded: false,
+            },
           ],
         },
       ]),
     )
 
     expect(result.a).toEqual([
-      { id: '100:0', role: 'topic', text: 'アリバイ', askedAt: 100 },
+      { id: '100:0', role: 'topic', text: 'アリバイ', askedAt: 100, notable: false },
       { id: '100:1', role: 'user', text: '昨夜どこに？', askedAt: 100 },
       { id: '100:2', role: 'assistant', text: '書斎です', askedAt: 100 },
       { id: '100:3', role: 'user', text: '何時まで？', askedAt: 100 },
       { id: '100:4', role: 'assistant', text: '日付が変わる頃まで', askedAt: 100 },
     ])
   })
 
+  test('何かを引き出した話題には印が立つ', () => {
+    const result = restoreConversations(
+      history([
+        {
+          characterId: 'a',
+          exchanges: [
+            {
+              question: '傘は？',
+              answer: '差していません',
+              askedAt: 100,
+              topic: '雨',
+              yielded: true,
+            },
+          ],
+        },
+      ]),
+    )
+
+    expect(result.a?.[0]).toEqual({
+      id: '100:0',
+      role: 'topic',
+      text: '雨',
+      askedAt: 100,
+      notable: true,
+    })
+  })
+
   test('質問と答えは同じ時刻を持つ（NPCをまたいで並べ直すため）', () => {
     const result = restoreConversations(
       history([
         {
           characterId: 'a',
-          exchanges: [{ question: 'q1', answer: 'a1', askedAt: 300, topic: null }],
+          exchanges: [{ question: 'q1', answer: 'a1', askedAt: 300, topic: null, yielded: false }],
         },
         {
           characterId: 'b',
-          exchanges: [{ question: 'q2', answer: 'a2', askedAt: 200, topic: null }],
+          exchanges: [{ question: 'q2', answer: 'a2', askedAt: 200, topic: null, yielded: false }],
         },
       ]),
     )
@@ -71,21 +118,26 @@ describe('restoreConversations', () => {
       history([
         {
           characterId: 'a',
-          exchanges: [{ question: '聞きかけ', answer: '', askedAt: 1, topic: '話題' }],
+          exchanges: [
+            { question: '聞きかけ', answer: '', askedAt: 1, topic: '話題', yielded: false },
+          ],
         },
       ]),
     )
 
     expect(result.a).toEqual([
-      { id: '1:0', role: 'topic', text: '話題', askedAt: 1 },
+      { id: '1:0', role: 'topic', text: '話題', askedAt: 1, notable: false },
       { id: '1:1', role: 'user', text: '聞きかけ', askedAt: 1 },
     ])
   })
 
   test('一度も話していないNPCはキーごと作らない', () => {
     const result = restoreConversations(
       history([
-        { characterId: 'a', exchanges: [{ question: 'q', answer: 'a', askedAt: 1, topic: null }] },
+        {
+          characterId: 'a',
+          exchanges: [{ question: 'q', answer: 'a', askedAt: 1, topic: null, yielded: false }],
+        },
         { characterId: 'b', exchanges: [] },
       ]),
     )