refactor(editor): tasks.json 廃止で dead code 化した container_name 引数を削除

_maybe_place_terminal_task 削除により container_name の唯一の供給元が消え、
_maybe_open_editor / open_editor の container_name 引数は常に None となる
dead code になっていた。gemini レビュー指摘に従い以下を整理する。

- _maybe_open_editor: container_name 引数と docstring 説明・open_editor への伝播を削除
- open_editor: container_name 引数と docstring 説明を削除し
  container = resolve_container_name(...) に簡約
- test_open_editor_uses_given_container_name テストを削除

build_attach_uri / resolve_container_name / _query_container_name は別物のため不変更。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: takemi-ohama

lib/devbase/commands/container.py

@@ -416,7 +416,7 @@ def _resolve_open_index(open_index: Optional[int], scale: int) -> int:
 
 def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
                        open_index: Optional[int], scale: int,
-                       compose_file=None, container_name: Optional[str] = None) -> None:
+                       compose_file=None) -> None:
     """`up` 完了後に dev コンテナへ接続したエディタを開く ([6/6])。
 
     有効判定は ``open_flag`` (CLI ``--open``/``--no-open``) が優先、None なら env
@@ -429,9 +429,6 @@ def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
     ``compose_file`` は実コンテナ名問い合わせ用の override compose。``up`` 起動時と
     同じファイルを渡さないと ``{dev}-{index}`` サービスが見えず実名取得に失敗する。
     未指定なら ``.docker-compose.scale.yml`` が存在すればそれ、無ければ None。
-
-    ``container_name`` が渡されれば :func:`opener.open_editor` 内の
-    ``resolve_container_name`` (= ``docker compose ps``) をスキップして再利用する。
     """
     from devbase.editor import opener
 
@@ -456,7 +453,6 @@ def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
             workdir=workdir,
             index=open_index,
             compose_file=compose_file,
-            container_name=container_name,
         )
     except Exception as e:  # noqa: BLE001 - エディタ起動で up を倒さない
         logger.warning("エディタの自動オープンに失敗しましたがデプロイは成功しています: %s", e)

lib/devbase/editor/opener.py

@@ -462,7 +462,7 @@ def _launch(cmd: list, env: dict) -> None:
 
 
 def open_editor(*, project_name: str, dev_service_name: str, workdir: str,
-                index: int = 1, compose_file=None, container_name: Optional[str] = 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:
@@ -472,8 +472,6 @@ def open_editor(*, project_name: str, dev_service_name: str, workdir: str,
     握り潰して warning にし、``up`` 本体を絶対に失敗させない。``isatty`` /
     ``system`` は :func:`detect_context` への差し替え口 (テスト用)。``compose_file``
     は実コンテナ名問い合わせ時に起動と同じ override compose を ``-f`` で渡すため。
-    ``container_name`` が渡されれば :func:`resolve_container_name` (= ``docker compose
-    ps``) をスキップしてそれを使う (呼び出し側で解決済みの名前を使い回す)。
     """
     env = os.environ if environ is None else environ
     ctx = detect_context(env, isatty=isatty, system=system)
@@ -487,7 +485,7 @@ def open_editor(*, project_name: str, dev_service_name: str, workdir: str,
         logger.info("エディタの自動オープンをスキップ: %s", plan.reason)
         return "skip"
 
-    container = container_name or resolve_container_name(
+    container = resolve_container_name(
         dev_service_name, project_name, index, compose_file=compose_file)
     # SSH コンテキストでのみネスト authority (@ssh-remote+host) を組む。自動推測は
     # VS Code Remote-SSH 統合端末 (in_vscode) の時だけ有効にする — plain SSH (VS Code 外)

tests/editor/test_opener.py

@@ -508,25 +508,6 @@ def test_open_editor_flat_uri_when_ssh_host_unset(monkeypatch):
     assert "@ssh-remote" not in calls[0][2]
 
 
-def test_open_editor_uses_given_container_name(monkeypatch):
-    """container_name を渡すと resolve_container_name を呼ばずそれを使う。"""
-    monkeypatch.setattr(opener.shutil, "which", lambda c: "/usr/bin/code")
-
-    def boom(*a, **k):
-        raise AssertionError("resolve_container_name should not be called")
-
-    monkeypatch.setattr(opener, "resolve_container_name", boom)
-    calls = []
-    opener.open_editor(
-        project_name="carmo", dev_service_name="dev", workdir="/work/carmo",
-        container_name="preresolved-dev-1", environ={}, isatty=True,
-        launcher=lambda cmd, env: calls.append(cmd),
-    )
-    uri = calls[0][2]
-    hexpart = uri.split("attached-container+")[1].split("/work")[0]
-    assert json.loads(bytes.fromhex(hexpart).decode())["containerName"] == "/preresolved-dev-1"
-
-
 def test_open_editor_skip_when_no_editor(monkeypatch):
     monkeypatch.setattr(opener.shutil, "which", lambda c: None)
     calls = []