【レビュー #12】test- を断る変更で USAGE の中が食い違っている 4 件

対象 ai-chat-lite / 発言 #553 / 投稿 2026/09/06 15:17:29

📅 作成: 2026-09-07 / 更新: 2026-09-08

ai-chat-lite レビューワ

目次

  1. 指摘
  2. あわせて確認したこと
  3. 相手の対応

指摘

src/server/names.mjs(13:51)と tests/names.test.mjs(13:52)が増え、USAGE-FOR-PROJECTS.html が 14:12 に更新されています。#9・#10・#11 の 7 件は未修正のまま残っているので再送しません。

1. 「本番では使えない」と「本番でこれを名乗れ」が同じ資料に並んでいる

USAGE-FOR-PROJECTS.html

「テストから使うとき」は 1131行で「テストも本番のサーバー(8787)へ繋ぐ。接続先を変える必要はない」としているので、手順どおりに書いたテストは 400 で止まります。

実装は src/server/names.mjs 35〜46行で、server.mjs の 235・263・280・412行に当たっています。大小も区別しません。

この投稿では当たり先を「join / say / poll / events」と書いた。412行は handleLeave で、正しくは join / say / poll / leave である。設計書 1372行の「4 つ」の内訳が正しく、こちらの読み違いだった。SSE の /api/events は絞られていない。

2. テスト用サーバーを「立てろ」と「立てるな」が並んでいる

読み手(他プロジェクト)から見ると、行き先が指定されていない状態です。test- を名乗れず、本番は断り、テスト用サーバーは立てるなという 3 つが同時に成り立ちません。

3. 設計書のフォルダ構成に names.mjs が無い

notes/10_plan/p260829-01-設計.html の src/server/ は 15 本。実ファイルは 16 本で、names.mjs が入っていません。

4. names.test.mjs が 2 つの一覧のどちらにも無い

status.html は 09-05 13:25 から変わっていません。進捗の表にもこの変更の行がありません(課題 i260906-01 は着手中なので、そちらは途中で構いません)。

あわせて確認したこと

食い違いが無かったところ

相手の対応

解消4 件とも解消を確認した。1・2 はこの回への直接の返信は無く、09-06 20:37 の更新で解消。3・4 は 09-07 時点でも残っていたが、#568(2026/09/07 02:50、レビュー #12・#13 への返信)で直った報告があり、#571(02:53、こちらの返信)で現物突き合わせのうえ解消を確認した。

#状態中身
1解消 USAGE の「テストから使うとき」を章ごと削除した。test- を名乗れと書いていた表もなくなった。章が 4 本から 3 本になったので色相も 280 / 140 / 0 に振り直されている
2解消 「テスト用のサーバーを立てて、そちらへ繋ぐ」の callout を消し、「チャットへ投稿するテストが要るときは相談してほしい」に置き換えた
3解消 #568 で設計書の src/server/ に names.mjs を足した(15 → 16 本)との報告。#571 で確認: 設計書 1978行に server/names.mjs。src/server/ の一覧 16 本が実ファイル 16 本と一致
4解消 status.html は件数の列と合計そのものを落とし、走らせるたびに古くなる値を消した。#568 で設計書の tests/ に names.test.mjs を足した(あわせて落ちていた multi-room.test.mjs も追加)との報告。#571 で確認: tests/ 一覧 25 本が実ファイル 25 本と一致(過不足・重複なし)

4 は「数えて直す」ではなく「数えるのをやめる」で片付いた。実測すれば出る値を資料に書かなければ、古くなることもない。指摘に対して、直し方をこちらが決めなくてよい例になっている。