fix(tui): dispatch 委譲層で CWD/環境変数を実行後に復元 (PR #55 round1 major)

TUI から project 操作を実行すると _resolve_project_name の
os.chdir / env 反映 / COMPOSE_PROJECT_NAME 上書きが同一プロセスに
残留し、トップメニュー復帰後の env get 等が直前プロジェクトの
CWD・環境変数 (PWD 含む) を参照していた。

委譲チョークポイントの tui.dispatch に _preserve_cwd_env コンテキスト
マネージャを追加し、dispatch_lifecycle / dispatch_group の前後で
CWD と os.environ を保存・復元する (例外時も finally で保証)。
回帰テスト 3 件を追加。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: takemi-ohama

lib/devbase/tui/dispatch.py

@@ -16,22 +16,48 @@
 
 from __future__ import annotations
 
+import contextlib
+import os
 import types
 from pathlib import Path
 from typing import Callable
 
 
+@contextlib.contextmanager
+def _preserve_cwd_env():
+    """ハンドラ実行前後で CWD と ``os.environ`` を保存・復元する。
+
+    CLI 経路は 1 コマンド = 1 プロセスのため、``_resolve_project_name`` が行う
+    ``os.chdir`` / env 反映 / ``COMPOSE_PROJECT_NAME`` 上書きはプロセス終了で消える。
+    一方 TUI は同一プロセスでトップメニューへ復帰し操作を続行するため、復元しないと
+    直前プロジェクトの CWD / 環境変数 (PWD 含む) を後続操作 (env get 等) が参照して
+    しまう (PR #55 round1 codex/gemini major 指摘)。委譲チョークポイントである本層で
+    一括復元し、各 actions_* / 共有ハンドラへ復元処理を散らさない。
+    """
+    old_cwd = os.getcwd()
+    old_env = os.environ.copy()
+    try:
+        yield
+    finally:
+        with contextlib.suppress(OSError):
+            os.chdir(old_cwd)
+        os.environ.clear()
+        os.environ.update(old_env)
+
+
 def dispatch_lifecycle(subcommand: str, name: str | None = None, **attrs) -> int:
     """``project <subcommand> [name]`` を共有ハンドラ ``cmd_project`` 経由で起動する。
 
     ``name`` が指定されると ``_dispatch_lifecycle`` が対象ディレクトリへ chdir して
     から実行する。``up`` の ``scale`` など各サブコマンド固有の属性は ``attrs`` で渡す
-    (未指定でも getattr の既定で吸収される)。
+    (未指定でも getattr の既定で吸収される)。chdir / env 変更は TUI セッションへ
+    残留させないよう ``_preserve_cwd_env`` で実行後に復元する。
     """
     from devbase.commands.container import cmd_project
 
     ns = types.SimpleNamespace(subcommand=subcommand, name=name, **attrs)
-    return cmd_project(ns)
+    with _preserve_cwd_env():
+        return cmd_project(ns)
 
 
 def dispatch_group(handler: Callable[[Path, object], int], devbase_root: Path,
@@ -40,7 +66,9 @@ def dispatch_group(handler: Callable[[Path, object], int], devbase_root: Path,
 
     env / plugin / snapshot の各 ``cmd_*`` は ``(devbase_root, args)`` を取り、
     ``args.subcommand`` で分岐する。TUI はサブコマンドと属性を ``SimpleNamespace`` に
-    詰めてそのまま委譲する (PR3 以降の actions_* が利用)。
+    詰めてそのまま委譲する (PR3 以降の actions_* が利用)。現行ハンドラは CWD /
+    environ を変更しないが、lifecycle 側と契約を揃えるため同じく復元境界を張る。
     """
     ns = types.SimpleNamespace(subcommand=subcommand, **attrs)
-    return handler(devbase_root, ns)
+    with _preserve_cwd_env():
+        return handler(devbase_root, ns)

tests/cli/tui/test_dispatch.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import os
 from pathlib import Path
 
 from devbase.tui import dispatch
@@ -34,6 +35,80 @@ def test_dispatch_lifecycle_name_optional(monkeypatch):
     assert captured == {"name": None}
 
 
+def test_dispatch_lifecycle_restores_cwd_and_env(monkeypatch, tmp_path):
+    """ハンドラが chdir / 環境変数変更したまま戻っても呼び出し前の状態へ復元する。
+
+    PR #55 round1 major 回帰テスト: TUI は同一プロセスで継続するため、
+    `_resolve_project_name` 相当の chdir / env 反映 / COMPOSE_PROJECT_NAME 上書きが
+    トップメニュー復帰後の操作 (env get 等) へ残留してはならない。
+    """
+    from devbase.commands import container as container_mod
+
+    other = tmp_path / "projects" / "carmo"
+    other.mkdir(parents=True)
+
+    def mutating_handler(args):
+        os.chdir(other)                                  # chdir 残留を模擬
+        os.environ["COMPOSE_PROJECT_NAME"] = "carmo"     # 上書き残留を模擬
+        os.environ["DEV_SERVICE_NAME"] = "leaked"        # 新規キー残留を模擬
+        os.environ.pop("DEVBASE_TEST_KEEP", None)        # 既存キー削除を模擬
+        return 0
+
+    monkeypatch.setattr(container_mod, "cmd_project", mutating_handler)
+    monkeypatch.chdir(tmp_path)
+    monkeypatch.setenv("DEVBASE_TEST_KEEP", "orig")
+    monkeypatch.delenv("DEV_SERVICE_NAME", raising=False)
+    monkeypatch.setenv("COMPOSE_PROJECT_NAME", "before")
+
+    rc = dispatch.dispatch_lifecycle("up", "carmo", scale=None)
+
+    assert rc == 0
+    assert Path.cwd() == tmp_path                        # CWD 復元
+    assert os.environ["COMPOSE_PROJECT_NAME"] == "before"  # 上書き復元
+    assert "DEV_SERVICE_NAME" not in os.environ          # 漏えいキー除去
+    assert os.environ["DEVBASE_TEST_KEEP"] == "orig"     # 削除キー復元
+
+
+def test_dispatch_lifecycle_restores_state_on_exception(monkeypatch, tmp_path):
+    """ハンドラが例外を投げても CWD / 環境変数は復元される (try/finally 保証)。"""
+    import pytest
+
+    from devbase.commands import container as container_mod
+
+    def raising_handler(args):
+        os.chdir(tmp_path)
+        os.environ["DEV_SERVICE_NAME"] = "leaked"
+        raise RuntimeError("boom")
+
+    monkeypatch.setattr(container_mod, "cmd_project", raising_handler)
+    old_cwd = Path.cwd()
+    monkeypatch.delenv("DEV_SERVICE_NAME", raising=False)
+
+    with pytest.raises(RuntimeError):
+        dispatch.dispatch_lifecycle("up", "carmo", scale=None)
+
+    assert Path.cwd() == old_cwd
+    assert "DEV_SERVICE_NAME" not in os.environ
+
+
+def test_dispatch_group_restores_cwd_and_env(tmp_path, monkeypatch):
+    """dispatch_group も lifecycle と同じ復元境界を張る (契約整合)。"""
+
+    def mutating_handler(devbase_root, args):
+        os.chdir(tmp_path)
+        os.environ["DEV_SERVICE_NAME"] = "leaked"
+        return 0
+
+    monkeypatch.delenv("DEV_SERVICE_NAME", raising=False)
+    old_cwd = Path.cwd()
+
+    rc = dispatch.dispatch_group(mutating_handler, Path("/devbase"), "init")
+
+    assert rc == 0
+    assert Path.cwd() == old_cwd
+    assert "DEV_SERVICE_NAME" not in os.environ
+
+
 def test_dispatch_group_builds_namespace_and_calls_handler():
     """dispatch_group は (devbase_root, args) 形式のハンドラへ委譲する。"""
     captured = {}