Skip to content

docs(PLAN59): 再起動なしで VS Code だけ開き直す devbase open の要求仕様と設計 (#197) - #198

Merged
takemi-ohama merged 2 commits into
mainfrom
design/issue-197
Sep 18, 2026
Merged

takemi-ohama merged 2 commits into
mainfrom
design/issue-197

Conversation

@takemi-ohama

@takemi-ohama takemi-ohama commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary

issue #197(コンテナを止めずに VS Code の窓を開き直すコマンドの新設)の要求仕様と設計を載せる。実装は含まない。

ワークフローモードは standard。この Pull Request のマージを承認した後に実装へ進む。関連: #197

決めたこと

issues/PLAN59_editor-open-design.md

  • 決定 1: 起動中の判定は「dev インスタンスが 1 つ以上動いているか」で行い、index の検査と分ける
  • 決定 2: 状態を取得できないときは up へ委譲せずに止まる
  • 決定 3: 開く処理は _maybe_open_editor から切り出して共有し、有効判定を持たせない
  • 決定 4: 停止中は cmd_up(open_editor=True) へ委譲し、自前で開き直さない
  • 決定 5: opener.open_editor が skip を返したら終了コード 1 にする
  • 決定 6: TUI の open は index を尋ねない
  • 決定 7: container open も足す
  • 決定 8: 起動中の判定は env token の列挙を共有の場所へ移して使う

Test plan

  • 受け入れ条件の 20 件すべてが、設計のテスト設計の表に対応している
  • 決定の記録で採らなかった案(起動中の判定にコンテナ名の解決を使う方式、_maybe_open_editor へ open_flag=True を渡す方式)が、構成要素と処理の流れに残っていない
  • 設計で参照した既存コード(cmd_up・_maybe_open_editor・opener.open_editor・resolve_container_name・_RUNNING_OPS・bin/devbase の name 解決)を main @ 87f5978 で読んで確かめた

🤖 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 | kiro | COMMENT

設計・仕様の記述は既存コード(cmd_up / _maybe_open_editor / opener.open_editor の戻り値 launch|print_command|skip / resolve_container_name / _RUNNING_OPS / _BACK_TO_TOP_OPS / .docker-compose.scale.yml / --context の空 usage エラー)と整合していることを確認しました。1 点、既存の列挙ヘルパとの重複についてインラインで修正提案しています。

Comment thread issues/PLAN59_editor-open-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 | COMMENT

issues/PLAN59_editor-open-design.md の構成要素表における _add_open_subparser の責務記述と、決定 7 / 入出力契約表における container open の引数仕様([name] を持たない)の不整合について 1 件インラインで修正提案を行いました。

Comment thread issues/PLAN59_editor-open-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 | codex | APPROVE

要求仕様・設計と既存実装を照合し、修正が必要な指摘はありません。

@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 | APPROVE

docs のみの PR(PLAN59 の要求仕様と設計)。設計が参照する既存コード(cmd_up / _maybe_open_editor / _running_dev_containers / resolve_container_name / decide_action / _dispatch_lifecycle / _add_profile_subparser / _add_context_arg / SHORTCUTS / SUBCMD_MAP / bin/devbase の name 解決 2 リスト / TUI _RUNNING_OPS・_BACK_TO_TOP_OPS / bash・zsh 補完)を worktree で確認し、設計の記述・8 つの決定・受け入れ条件と矛盾する箇所は見つからなかった。受け入れ条件の番号飛び(19/20 → 12-18)は本文で追記順として明示済み、内部リンクも解決する。修正を要する指摘は無し。

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