設計(PLAN69): tmux のセッションを名指しで attach・調査・終了するコマンドと TUI(#234) - #260
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | REQUEST_CHANGES
修正が必要な指摘は 2 件です。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | agy | REQUEST_CHANGES
confirm-before における端末指定オプションの誤り(-c ではなく -t)およびプロンプトでの対象セッション名展開の不整合を修正し、tmux.conf 変更に伴う既存テスト test_conf_provides_windows_like_copy_bindings の追従修正および test_tmux_session.py のサーバ起動方式(TMUX_TMPDIR への統一)を設計へ反映してください。
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
🔧 /ndf:fix サマリ | round 1 | commit 35cbc98対応件数: critical=0 / major=2 / minor=3(合計 5 件) |
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | agy | REQUEST_CHANGES
PLAN69 の設計において、TUI メニューから「落とす」を実行した際に操作中の自セッションが終了できない仕様上の不整合を検出しました。修正を提案します。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | kiro | COMMENT
設計文書の技術的主張(confirm-before の -t/-c、capture-pane -S の挙動、prefix S 既定未割り当て、%%% + #{q:} の引用)は tmux(1) man と一致し、round2 の修正は妥当。1 点、決定 6 の CI ShellCheck severity 方針が実際の ci.yml と食い違うため要修正。
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
🔧 /ndf:fix サマリ | round 2 | commit e23509e対応件数: critical=0 / major=1 / minor=1(合計 2 件) |
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 3 | claude | COMMENT
設計の決定 2 に書いた template の形が、同じ文書の入出力の契約・tmux.conf の割り当てと食い違っている 1 点だけ直す。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 3 | kiro | COMMENT
設計・要求ともに実測と反証条件を伴い自己整合している。指摘は 1 件のみ。CI の ShellCheck で新規に検査対象へ加える tmux-first / tmux-clean の「0 件」検証が、CI の action の shellcheck 版・severity と一致していない点だけ、検証手段を補う余地がある。
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
🔧 /ndf:fix サマリ | commit efc879a対応件数: critical=0 / major=0 / minor=2(合計 2 件) |
Summary
#234 の要求と設計を載せる。実装は含まない。
Refs #234
決めたこと
issues/PLAN69_tmux-named-session-design.mdprefix Sに割り当て、choose-treeの template からmenuを呼ぶ~/.local/binと~/.tmux.confへ配る経路は作らず、手順を文書に書くtmux-firstから書き写し、共通化しないcontainers/base/tmux-*を検査するdisplay-popupに出し、Enter で閉じるTest plan
設計の段階で確かめたこと(実測は設計文書の「実測」の節にある)。
prefix Sが既定で空いていること:tmux -f /dev/nullのlist-keys -T prefix(ホスト 3.7b・devbase-base:latestの 3.6)choose-treeの template のrun-shell -t "%%%"と#{q:session_id}/#{q:client_name}が、'・"・$・;・~・\・空白を含む名前でも、選んだセッションと押した端末を指すこと(ptyで端末を attach した専用のソケットの tmux、3.7b と 3.6)tmux-first/tmux-cleanが base の shellcheck 0.11.0 で指摘 0 件(exit=0)🤖 Generated with Claude Code