Skip to content

設計(PLAN69): tmux のセッションを名指しで attach・調査・終了するコマンドと TUI(#234) - #260

Merged
takemi-ohama merged 5 commits into
mainfrom
design/plan69-tmux-named-session
Sep 24, 2026
Merged

takemi-ohama merged 5 commits into
mainfrom
design/plan69-tmux-named-session

Conversation

@takemi-ohama

@takemi-ohama takemi-ohama commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Summary

#234 の要求と設計を載せる。実装は含まない。

Refs #234

決めたこと

issues/PLAN69_tmux-named-session-design.md

  • 決定 1: 名指しの操作は別のコマンドにする。本体は 1 ファイルで、短縮名は symlink で振り分ける
  • 決定 2: TUI は prefix S に割り当て、choose-tree の template から menu を呼ぶ
  • 決定 3: ホストの ~/.local/bin~/.tmux.conf へ配る経路は作らず、手順を文書に書く
  • 決定 4: tmux の外で一覧を選ぶ UI は作らない
  • 決定 5: 実行元の端末を特定する処理は tmux-first から書き写し、共通化しない
  • 決定 6: CI の ShellCheck で containers/base/tmux-* を検査する
  • 決定 7: 「中身を見る」は display-popup に出し、Enter で閉じる

Test plan

設計の段階で確かめたこと(実測は設計文書の「実測」の節にある)。

🤖 Generated with Claude Code

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | codex | REQUEST_CHANGES

修正が必要な指摘は 2 件です。

Comment thread issues/PLAN69_tmux-named-session-design.md Outdated
Comment thread issues/PLAN69_tmux-named-session-design.md Outdated

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 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 への統一)を設計へ反映してください。

Comment thread issues/PLAN69_tmux-named-session-design.md Outdated
Comment thread issues/PLAN69_tmux-named-session-design.md Outdated
Comment thread issues/PLAN69_tmux-named-session-design.md Outdated
@takemi-ohama

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ | round 1 | commit 35cbc98

対応件数: critical=0 / major=2 / minor=3(合計 5 件)
決着: 5 件 / 見送り: 0 件 / 却下: 0 件
CI: SUCCESS

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 2 | agy | REQUEST_CHANGES

PLAN69 の設計において、TUI メニューから「落とす」を実行した際に操作中の自セッションが終了できない仕様上の不整合を検出しました。修正を提案します。

Comment thread issues/PLAN69_tmux-named-session-design.md Outdated

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 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 と食い違うため要修正。

Comment thread issues/PLAN69_tmux-named-session-design.md Outdated
@takemi-ohama

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ | round 2 | commit e23509e

対応件数: critical=0 / major=1 / minor=1(合計 2 件)
決着: 2 件 / 見送り: 0 件 / 却下: 0 件
CI: SUCCESS

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 3 | claude | COMMENT

設計の決定 2 に書いた template の形が、同じ文書の入出力の契約・tmux.conf の割り当てと食い違っている 1 点だけ直す。

Comment thread issues/PLAN69_tmux-named-session-design.md Outdated

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 3 | kiro | COMMENT

設計・要求ともに実測と反証条件を伴い自己整合している。指摘は 1 件のみ。CI の ShellCheck で新規に検査対象へ加える tmux-first / tmux-clean の「0 件」検証が、CI の action の shellcheck 版・severity と一致していない点だけ、検証手段を補う余地がある。

Comment thread issues/PLAN69_tmux-named-session-design.md
@takemi-ohama

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ | commit efc879a

対応件数: critical=0 / major=0 / minor=2(合計 2 件)
決着: 2 件 / 見送り: 0 件 / 却下: 0 件
CI: SUCCESS

@takemi-ohama
takemi-ohama merged commit 7159746 into main Sep 24, 2026
7 checks passed
@takemi-ohama
takemi-ohama deleted the design/plan69-tmux-named-session branch September 24, 2026 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant