fix(up): plain SSH コマンド提示を which 非依存化 + 実名問い合わせに compose file (-f) を渡す

cross-review round2 codex 指摘 C/D 対応。

C[正確性]: plain SSH の degrade (手元コマンド提示) は decide_action が
  editor_cmd None で先頭 skip するため、リモートに code が無いと print_command
  へ到達せず skip していた。提示コマンドは手元(ローカル)で実行する前提なので
  リモートの code 実在に依存させない。
  - resolve_editor_display() を追加 (which 非依存・必ず非None)
  - decide_action(ctx, editor_available: bool) へシグネチャ変更。SSH の
    print_command は editor_available に依存せず到達、launch 系経路のみ
    editor 不在で skip。
  - open_editor は launch=editor / print=display を使い分け。

D[正確性]: _query_container_name が docker compose ps に override compose を
  渡さず base compose.yml に無い {dev}-{index} を見てほぼ常にフォールバック
  していた。
  - _query_container_name / resolve_container_name / open_editor /
    _maybe_open_editor に compose_file を追加し起動時と同じ -f を伝播。
  - cmd_up は [6/6] で override_file を渡す。

テスト追加・更新 (766 passed)。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: takemi-ohama

lib/devbase/commands/container.py

@@ -356,7 +356,8 @@ def _auto_snapshot() -> None:
 
 
 def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
-                       open_index: Optional[int], scale: int) -> None:
+                       open_index: Optional[int], scale: int,
+                       compose_file=None) -> None:
     """`up` 完了後に dev コンテナへ接続したエディタを開く ([6/6])。
 
     有効判定は ``open_flag`` (CLI ``--open``/``--no-open``) が優先、None なら env
@@ -365,6 +366,10 @@ def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
     ``open_index`` は起動済みインスタンス範囲 ``1..scale`` 内である必要がある。
     0・負数・``scale`` 超過は存在しないコンテナ URI になり原因不明な起動失敗を招くため、
     警告を出して既定 (1) へフォールバックする。
+
+    ``compose_file`` は実コンテナ名問い合わせ用の override compose。``up`` 起動時と
+    同じファイルを渡さないと ``{dev}-{index}`` サービスが見えず実名取得に失敗する。
+    未指定なら ``.docker-compose.scale.yml`` が存在すればそれ、無ければ None。
     """
     from devbase.editor import opener
 
@@ -387,6 +392,11 @@ def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
         )
         open_index = 1
 
+    # 実コンテナ名問い合わせ用の compose file: 明示指定がなければ override が
+    # 存在すればそれを使う (起動時と同じ file を docker compose ps へ渡す)。
+    if compose_file is None and _SCALE_COMPOSE_FILE.exists():
+        compose_file = _SCALE_COMPOSE_FILE
+
     dev_service_name = get_dev_service_name()
     workdir = opener.resolve_workdir(os.environ, project_name)
     logger.info("[6/6] Opening editor attached to the dev container...")
@@ -396,6 +406,7 @@ def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
             dev_service_name=dev_service_name,
             workdir=workdir,
             index=open_index,
+            compose_file=compose_file,
         )
     except Exception as e:  # noqa: BLE001 - エディタ起動で up を倒さない
         logger.warning("エディタの自動オープンに失敗しましたがデプロイは成功しています: %s", e)
@@ -470,7 +481,8 @@ def cmd_up(project_name: str = None, scale: int = None,
         if deploy_script.exists() and deploy_script.is_file():
             _run_deploy_script_for_instances(deploy_script, range(1, scale + 1))
 
-        _maybe_open_editor(project_name, open_editor, open_index, scale)
+        _maybe_open_editor(project_name, open_editor, open_index, scale,
+                           compose_file=override_file)
 
         logger.info("=== Deploy completed successfully ===")
         return 0

lib/devbase/editor/opener.py

@@ -123,6 +123,26 @@ def resolve_editor_cmd(environ=None) -> Optional[list]:
     return None
 
 
+def resolve_editor_display(environ=None) -> list:
+    """コマンド提示 (print_command) 用のエディタ argv を解決する。
+
+    :func:`resolve_editor_cmd` と異なり ``shutil.which`` による実在チェックは
+    行わない。plain SSH では提示コマンドを実行するのは「ユーザの手元 (ローカル)」
+    であり、コマンドを実行している側 (リモート) に ``code`` が存在する必要は無い
+    ため、リモートの実在に依存せず必ず非 None を返す。
+
+    ``DEVBASE_EDITOR`` があればそれを (シェル風に分割して) 用い、無ければ既定の
+    ``["code"]`` を返す。
+    """
+    env = os.environ if environ is None else environ
+    explicit = env.get("DEVBASE_EDITOR")
+    if explicit:
+        parts = shlex.split(explicit)
+        if parts:
+            return parts
+    return ["code"]
+
+
 def build_attach_uri(container_name: str, workdir: str) -> str:
     """``vscode-remote://attached-container+<hex>/<workdir>`` を組む。
 
@@ -175,19 +195,28 @@ def _parse_compose_ps_name(stdout: str) -> Optional[str]:
 
 
 def _query_container_name(dev_service_name: str, index: int,
+                          compose_file=None,
                           runner: Optional[Callable] = None) -> Optional[str]:
     """実 docker へ問い合わせて dev インスタンスの実コンテナ名を取得する (保険)。
 
     scale 生成 compose ではサービス名が ``{dev}-{index}`` (例 ``dev-1``) になるため
     その service token を指定して ``docker compose ps --format json`` を実行する。
+    ``{dev}-{index}`` サービスは override compose (``.docker-compose.scale.yml``)
+    側にしか存在しないため、``compose_file`` が与えられた場合は起動時と同じ
+    ``-f <compose_file>`` を付与しないと base ``compose.yml`` には無いサービスを
+    見に行きほぼ常にフォールバックになる。
     取得できなければ None。docker 不在・非0・例外・空はすべて None に握り潰し、
     呼び出し側が決定的名へフォールバックできるようにする。
     """
     run = runner or subprocess.run
     service_token = f"{dev_service_name}-{index}"
+    cmd = ["docker", "compose"]
+    if compose_file is not None:
+        cmd += ["-f", str(compose_file)]
+    cmd += ["ps", "--format", "json", service_token]
     try:
         proc = run(
-            ["docker", "compose", "ps", "--format", "json", service_token],
+            cmd,
             capture_output=True, text=True, timeout=10,
         )
     except Exception:  # noqa: BLE001 - docker 不在等は保険なので握り潰す
@@ -201,6 +230,7 @@ def _query_container_name(dev_service_name: str, index: int,
 
 
 def resolve_container_name(dev_service_name: str, project_name: str, index: int = 1,
+                           compose_file=None,
                            runner: Optional[Callable] = None) -> str:
     """dev コンテナの実コンテナ名を返す。
 
@@ -213,7 +243,8 @@ def resolve_container_name(dev_service_name: str, project_name: str, index: int
     を全インスタンスへ設定する (volume/compose.py)。COMPOSE_PROJECT_NAME は
     project_name と一致するため、docker 問い合わせに失敗しても決定的に組み立てられる。
     """
-    queried = _query_container_name(dev_service_name, index, runner=runner)
+    queried = _query_container_name(dev_service_name, index,
+                                    compose_file=compose_file, runner=runner)
     if queried:
         return queried
     return f"{project_name}-{dev_service_name}-{index}"
@@ -229,24 +260,37 @@ def resolve_workdir(environ=None, project_name: Optional[str] = None) -> str:
     return f"/work/{repo}" if repo else "/work"
 
 
-def decide_action(ctx: EditorContext, editor_cmd: Optional[list]) -> OpenPlan:
-    """コンテキストとエディタ可用性から起動方針を決める (§2.4 マトリクス)。"""
-    if not editor_cmd:
-        return OpenPlan(
-            "skip",
-            "エディタ (code) が見つかりません。VS Code の `code` コマンドを PATH に "
-            "通すか DEVBASE_EDITOR を設定してください",
-        )
+_NO_EDITOR_REASON = (
+    "エディタ (code) が見つかりません。VS Code の `code` コマンドを PATH に "
+    "通すか DEVBASE_EDITOR を設定してください"
+)
+
+
+def decide_action(ctx: EditorContext, editor_available: bool) -> OpenPlan:
+    """コンテキストとエディタ可用性から起動方針を決める (§2.4 マトリクス)。
+
+    ``editor_available`` はローカルに launch 可能な ``code`` 系コマンドが実在するか
+    (``resolve_editor_cmd`` が非 None か) を表す。plain SSH の print_command 経路は
+    「ユーザの手元 (ローカル) でコマンドを実行する」前提のため、コマンドを実行して
+    いる側 (リモート) の editor 実在には依存させない (``editor_available`` を見ない)。
+    """
     if not ctx.is_tty:
         return OpenPlan("skip", "非対話 (非TTY/CI) 環境のため")
     if ctx.in_vscode:
         # VS Code 統合ターミナル (ローカル / WSL / Remote-SSH シム)。code が
-        # クライアント側へ委譲するため直接起動でよい。
+        # クライアント側へ委譲するため直接起動でよい。code シムが無いと委譲
+        # できないため editor が無ければ skip。
+        if not editor_available:
+            return OpenPlan("skip", _NO_EDITOR_REASON)
         return OpenPlan("launch", "VS Code 統合ターミナル経由")
     if ctx.is_ssh:
         # plain SSH (VS Code 外)。クライアントへ push する公式手段が無いため
-        # コマンドを提示する degrade。
+        # 手元で叩くコマンドを提示する degrade。提示先はローカルなのでリモートの
+        # editor 実在には依存しない。
         return OpenPlan("print_command", "SSH セッション (VS Code 外) のため")
+    # ローカル/WSL 端末。直接 launch するため editor が無ければ skip。
+    if not editor_available:
+        return OpenPlan("skip", _NO_EDITOR_REASON)
     return OpenPlan("launch", "ローカル/WSL 端末")
 
 
@@ -259,29 +303,34 @@ def _launch(cmd: list, env: dict) -> None:
 
 
 def open_editor(*, project_name: str, dev_service_name: str, workdir: str,
-                index: int = 1, environ=None,
+                index: int = 1, compose_file=None, environ=None,
                 isatty: Optional[bool] = None, system: Optional[str] = None,
                 launcher: Optional[Callable[[list, dict], None]] = None) -> str:
     """dev コンテナへ接続した VS Code を開く / コマンド提示 / スキップする。
 
     戻り値は実行された action ('launch' | 'print_command' | 'skip')。例外は
     握り潰して warning にし、``up`` 本体を絶対に失敗させない。``isatty`` /
-    ``system`` は :func:`detect_context` への差し替え口 (テスト用)。
+    ``system`` は :func:`detect_context` への差し替え口 (テスト用)。``compose_file``
+    は実コンテナ名問い合わせ時に起動と同じ override compose を ``-f`` で渡すため。
     """
     env = os.environ if environ is None else environ
     ctx = detect_context(env, isatty=isatty, system=system)
-    editor = resolve_editor_cmd(env)
-    plan = decide_action(ctx, editor)
+    editor = resolve_editor_cmd(env)        # launch 用 (which 込み・None あり得る)
+    display = resolve_editor_display(env)   # print 用 (必ず非 None)
+    plan = decide_action(ctx, editor_available=bool(editor))
 
-    container = resolve_container_name(dev_service_name, project_name, index)
+    container = resolve_container_name(dev_service_name, project_name, index,
+                                       compose_file=compose_file)
     uri = build_attach_uri(container, workdir)
 
     if plan.action == "skip":
         logger.info("エディタの自動オープンをスキップ: %s", plan.reason)
         return "skip"
 
-    quoted = " ".join(shlex.quote(c) for c in editor)
     if plan.action == "print_command":
+        # 提示コマンドは手元 (ローカル) で実行する前提。ローカルに code が無くても
+        # 提示できるよう display (which 非依存) を用いる。
+        quoted = " ".join(shlex.quote(c) for c in display)
         logger.info("SSH セッションを検出しました (%s)。", plan.reason)
         logger.info(
             "手元の VS Code で次を実行するか、VS Code の Remote-SSH 統合ターミナルから "
@@ -290,6 +339,7 @@ def open_editor(*, project_name: str, dev_service_name: str, workdir: str,
         logger.info("  %s --folder-uri '%s'", quoted, uri)
         return "print_command"
 
+    quoted = " ".join(shlex.quote(c) for c in editor)
     logger.info("[editor] %s を起動します (%s)", quoted, plan.reason)
     cmd = [*editor, "--folder-uri", uri]
     try:

tests/cli/test_project_dispatch.py

@@ -417,3 +417,17 @@ def test_maybe_open_editor_valid_index_within_scale(monkeypatch):
                         lambda **kw: called.append(kw) or 'launch')
     container._maybe_open_editor('carmo', True, 2, 3)
     assert called[0]['index'] == 2
+
+
+def test_maybe_open_editor_forwards_compose_file(monkeypatch):
+    """compose_file 引数が open_editor まで伝播する (実コンテナ名問い合わせ用)。"""
+    from devbase.commands import container
+    from devbase.editor import opener
+    monkeypatch.setattr(opener, 'is_open_enabled', lambda environ=None: True)
+    monkeypatch.setattr(container, 'get_dev_service_name', lambda: 'dev')
+    called = []
+    monkeypatch.setattr(opener, 'open_editor',
+                        lambda **kw: called.append(kw) or 'launch')
+    container._maybe_open_editor('carmo', True, 1, 1,
+                                 compose_file='override.yml')
+    assert called[0]['compose_file'] == 'override.yml'