docs: record the round trip verified on real hardware - #30
Merged
Conversation
2026-08-21、実機で 1 往復が成立した。部屋の入力欄からの発言が channel notification としてセッションへ届き、セッションが say_to_room を呼び返し、 その内容が部屋のメッセージ一覧へ並んだ。CLI 側への追加入力は無し。 あわせて、複数の channel サーバを同時に有効化できることが実測された。部屋の サイドカーと github-webhook-mcp の双方が同一セッションで機能している。#90 の premise が確定させていたのは --channels 併記による二重登録の失敗のみで、この 点は扱っていなかった。「単独指定」は --channels を併記しないという意味であって server: エントリを一つに限る意味ではない、と読める形に注記した。 往復の所要時間は未計測。成立の可否のみを確認しており、秒数は取っていない。 docs と README の双方にそのまま記載した。参考値として #90 の実測では push から 返信 tool 呼び出しまで約 10 秒。 premise の「liplus-chat の現況」節が移植直後の記述のまま残っていたので、実装 状況節を参照する形へ置き換えた。 #14
smileygames
commented
Aug 21, 2026
smileygames
left a comment
Member
Author
There was a problem hiding this comment.
AI self-review (execution_mode = auto)
受け入れ条件の照合(issue #14 完了条件)
| 条件 | 結果 |
|---|---|
| 往復の成立可否が記録されている | pass |
| 成立した場合、所要時間が記録されている | 未達。未計測であることを明記して閉じる |
| 成立しなかった場合の記録 | 該当なし |
| 「未検証」節が実測結果に置き換わっている | pass(docs / README とも) |
所要時間だけが埋まっていない。#90 の参考値(約 10 秒)で埋めることもできたが、それは別の環境・別の実装の測定であり、今回の往復を測ったものではない。推定値を実測欄へ書くと、後から読む者には測ったものとして見える。未計測と書いた。
記載の出所
- 往復の成立と両 channel の動作は Master の実機報告。私が観測したのは途中経過の 1 行(
liplus-chat-room · マスター: ハロ~)のみで、折り返しの成立は報告に依る。docs にはそこを区別せず「成立した」と書いている。報告を疑う理由が無く、実装側の経路は自動テストで通っているため。 - 複数 channel の同時有効化は、#90 の premise が扱っていなかった点。premise の文面を書き換えず、扱っていなかったという形で注記した。あちらの実測は依然として正しく、範囲外だっただけであるため。
併せて直したもの
premise の「liplus-chat の現況」節が移植直後の記述(「フロントエンドはプレースホルダーであり、チャットルーム UI と channel 連携は未実装」)のまま残っていた。実装状況節を参照する形へ置き換えた。docs が source of truth である以上、二箇所で食い違ったまま置けない。
scope deviation
なし。
次の期待
execution_mode = auto のため人間レビューゲートなし。CI green(CI pass / check pass 1m38s)。self-review pass により squash merge へ進む。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #14
実測結果(2026-08-21、Master 実機)
1 往復が成立した。 部屋の入力欄からの発言が channel notification としてセッションへ届き、セッションが
say_to_roomを呼び返し、その内容が部屋のメッセージ一覧へ並んだ。CLI 側への追加入力は無し。途中経過として観測された行:
複数の channel サーバを同時に有効化できることが実測された。 部屋のサイドカー(
liplus-chat-room)とgithub-webhook-mcpの双方が同一セッションで機能した。これは #90 の premise が扱っていなかった点。あちらが確定させていたのは「
--channelsを併記すると二重登録で不通になる」ことのみで、同一フラグへの複数server:エントリは未検証だった。premise の「単独指定」は--channelsを併記しないという意味であって、server:エントリを一つに限る意味ではない——と読める形に注記した。完了条件の照合(#14)
docs/0-requirements.mdの「未検証」節が実測結果に置き換わっている所要時間だけが埋まっていない。計測していないものを推定値で埋めることはせず、未計測と明記した。参考値として #90 の実測では push から返信 tool 呼び出しまで約 10 秒(モデル思考時間込み)。
その他
premiseの「liplus-chat の現況」節が移植直後の記述(「フロントエンドはプレースホルダー」)のまま残っていたので、実装状況節を参照する形へ置き換えた。release type
patch — docs のみ。挙動への影響なし。