docs(requirements): record the 2026-08-28 real-device verification of the sidebar toggles - #124
Merged
Merged
Conversation
… the sidebar toggles サイドバーの開閉(#118)について「全項目が実機未確認」と書いたまま置いていた記述を、2026-08-28 の操作で確かめた分だけ実機側へ移す。 - 「実機で確認済み」へ `#### 2026-08-28(AI が操作して確認)` を立てた。`main`(`97920a0`)を焼いて操作した分である。`panels` 欄を持たない既存設定が両方開いた状態で読まれアカウント 3 件も残ったこと(`#[serde(default)]` の移行経路)、アイコンが鏡像で出て高さが `<select>` と `端末` ボタンに揃っていること、畳んだ面が幅 0 で帯を残さないこと、畳んだ状態でも左右のボタンがヘッダに残ること、`Ctrl+B` が左の面だけを開閉すること、発言欄に焦点がある状態でも効き入力中の文字が壊れないこと、端末に焦点がある間は面が動かないこと、`history: true` / `participants: false` の非対称な状態が再起動後も復元されること、面の開閉を保存した後もアカウント 3 件が残ること(#122 の `saveConfig` 化)を項目ごとに書いた。 - 同節の末尾に未計測として、シェル側が `^B` をどう受けたかを残した。当日の CLI は起動直後の確認プロンプトで止まっており、そこでは `Ctrl+B` が何も描かない。この半分は xterm の `attachCustomKeyEventHandler` の性質であって、#118 が書いた側ではない。 - 「未実装」節のサイドバーの項目は落とさず、残る未計測だけを残す形へ書き替えた。確認済みの六点は日付付きで上節への参照に変え、項目そのものはシェル側の計測として残っている。「譲っていることは不在でしか見えない」という理由は既に本文に在ったのでそのまま活かした。 端末に焦点がある間に面が動かないことは、それだけではガードの発火を意味しない形にも読める。キーの購読は `window` に付いており、素の `Ctrl+B` が途中で降りる道は端末ガードの一本しかないため、面が動かないことがガードの発火である——発言欄に焦点がある場合との対照で、違いはフォーカスの位置だけである。この導出を項目の本文へ書いた。 docs は source of truth であり、確かめたものを未確認と書いたまま置けば、次に読んだ者が同じ計測をやり直す。逆に、確かめていないシェル側まで確認済みへ移せば、実機で見ていない保護を「在る」と読ませる。 #123 Co-authored-by: Claude Lin & Lay <eightborder@gmail.com> Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
smileygames
commented
Aug 28, 2026
smileygames
left a comment
Member
Author
There was a problem hiding this comment.
AI セルフレビュー(execution_mode = auto、#123 に対して)。
受け入れ基準
本文の制約を一つずつ diff で確認した。すべて満たしている。
- 新節の位置と様式 —
#### 2026-08-28(AI が操作して確認)が 2026-08-27 節の直後、### 未実装の直前に入っている。太字の見出し語+issue 参照の箇条書き、末尾に未計測:の段落という形は 2026-08-27 節と同じであり、新しい様式を持ち込んでいない。 - 九項目 — 本文の観測 1〜9 と一対一で対応している。各項目が実機で見た事実の範囲に収まっており、決定番号への対応づけ(決定1 / 決定2 / 決定3 / 決定4 / AI 判断5 / AI 判断6)も取り違えていない。
- 未計測の扱い — シェル側が
^Bをどう受けたかは確認済みの側へ移っていない。新節末尾の未計測:に残り、その半分が xterm のattachCustomKeyEventHandlerの性質であって #118 が書いた側ではない、という帰属も正しい。 - 未実装 bullet — 丸ごと消していない。確認済みの六点を日付付きの参照へ畳み、残る計測(シェル側の受け)だけを項目として残している。「譲っていることは不在でしか見えない」という既存の理由もそのまま生きている。
- 導出の記載 — 「面が動かない=ガードの発火」を、キーの購読が
windowに付いていること、素のCtrl+Bが降りる道が端末ガード一本であること、発言欄との対照でフォーカス位置だけが違うこと、の三点で支えている。観測から結論への段が飛んでいない。
逸脱
なし。変更は docs/0-requirements.md の 1 ファイル(+17 / −1)で、本文の target files と一致する。実装には触れていない。
CI
check / CI ともに success(a1ca370)。mergeStateStatus = CLEAN。
版数
patch。docs のみでユーザ/システムから観測できる変化が無い。
申し送り
push で保存済み認証情報(Windows Credential Manager)が HTTP 403「Your account is suspended」を返した。実装者はコマンド単位の上書きで通しており、git config へは何も永続化していない。認証情報そのものの差し替えはこの PR の範囲外であり、別途扱う。
次: 自己レビュー通過につき、auto モードの直接マージへ進む。
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 #123
サイドバーの開閉(#118)について、
docs/0-requirements.mdが全項目を「実機未確認」と書いたままであった。2026-08-28 に AI が操作して確かめた分を実機側へ移し、残る未計測だけを「未実装」へ残す。変更
#### 2026-08-28(AI が操作して確認)を追加した。main(97920a0)を焼いて操作した 9 項目である。panels欄を持たない既存設定の読み込み(#[serde(default)]の移行経路)、鏡像のアイコン、幅 0 で畳むこと、畳んだ状態のボタン、Ctrl+Bが左の面だけを開閉すること、発言欄に焦点がある状態でのCtrl+B、端末に焦点がある間は面が動かないこと、再起動後の保持、面の開閉を保存した後もアカウント 3 件が残ること(feat(ui): collapse either sidebar from a header toggle (config.rs, index.html, main.ts, styles.css, docs) #122)。^Bをどう受けたかであり、当日の CLI は確認プロンプトで止まっていた。確認していないこと
シェル側が
^Bをどう受けたかは実機で見ていないため、確認済みの側へは移していない。docs のみで、実装には触れていない。
Co-authored-by: Claude Lin & Lay eightborder@gmail.com
Co-Authored-By: Claude Opus 5 noreply@anthropic.com