feat(up): devbase up 時の .vscode/tasks.json 自動配置を廃止

統合ターミナル自動表示用の folderOpen tasks.json をユーザのワークスペースへ
自動配置する挙動は余計だったため削除する。VS Code 自動オープン (PR #69/#70)
はそのまま維持する。

- opener: build_folder_open_tasks_json() / is_open_terminal_enabled() を削除
- container: _maybe_place_terminal_task() と cmd_up/dispatch の open_terminal 引数を削除
- cli: --open-terminal / --no-open-terminal フラグを削除
- keys: DEVBASE_OPEN_TERMINAL 定数を削除
- docs: DEVBASE_OPEN_TERMINAL の記述を削除
- tests: 関連テストを削除

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

Author: takemi-ohama

docs/user/environment-variables.md

@@ -151,20 +151,8 @@ devbase はホストマシンの認証情報を自動収集し、コンテナ内
 | `DEVBASE_OPEN_INDEX` | scale 時に開く dev インスタンス番号（既定: `1`） |
 | `DEVBASE_EDITOR_SSH_HOST` | Remote-SSH 跨ホスト構成での ssh-remote ホスト名（例 `mac2`）。**通常は `~/.vscode-server` から自動検出**され不要。検出が外れる場合のみ明示。下記「跨ホスト」参照 |
 | `DEVBASE_EDITOR_DOCKER_CONTEXT` | 跨ホスト時に ssh 先で使う docker context（既定: ホストの `docker context show`） |
-| `DEVBASE_OPEN_TERMINAL` | 真で `up` 後に folderOpen ターミナル用 `.vscode/tasks.json` を配置（**既定: ON**） |
 
-都度の上書きは CLI フラグで行います: `devbase up --open` / `--no-open` / `--open-index N` / `--open-terminal` / `--no-open-terminal`（env より優先）。
-
-### 起動時に統合ターミナルを自動表示（`DEVBASE_OPEN_TERMINAL`）
-
-`up` 時に、開く dev コンテナのワークスペース（`/work/$GIT_REPO`）へ folderOpen タスク（`.vscode/tasks.json`）を `docker exec` で配置します（既存があれば変更しません）。VS Code はフォルダを開いた時にこのタスクで統合ターミナルを前面表示します。
-
-VS Code 公式には「起動時にターミナルを開く」専用設定が無く、folderOpen タスクが唯一の方法です。なお自動実行には次の 2 つの **VS Code クライアント側ユーザー設定**が関わり、devbase からは制御できません（いずれも application/user スコープ専用）:
-
-- **Workspace Trust**: 信頼していないフォルダではタスクは自動実行されません（初回は「フォルダを信頼」が必要）。
-- **`task.allowAutomaticTasks`**: 既定 `off` ではフォルダごとに 1 回「自動タスクを許可」を尋ねます。`on` にするとプロンプト無しで実行されます。
-
-→ 実際は「初回のみ信頼（＋許可）クリック、以降は自動でターミナルが開く」挙動になります。無効化は `DEVBASE_OPEN_TERMINAL=0` または `devbase up --no-open-terminal`。
+都度の上書きは CLI フラグで行います: `devbase up --open` / `--no-open` / `--open-index N`（env より優先）。
 
 ### 実行コンテキスト別の挙動
 

lib/devbase/cli.py

@@ -112,13 +112,6 @@ def _add_open_args(parser):
     parser.add_argument('--open-index', dest='open_index', type=int, default=None,
                         metavar='N',
                         help='Container index to open (default: 1)')
-    parser.add_argument('--open-terminal', dest='open_terminal', action='store_true',
-                        default=None,
-                        help='Place .vscode/tasks.json so the integrated terminal '
-                             'auto-opens on folder open (overrides DEVBASE_OPEN_TERMINAL)')
-    parser.add_argument('--no-open-terminal', dest='open_terminal', action='store_false',
-                        help='Do not place the folderOpen terminal tasks.json '
-                             '(overrides DEVBASE_OPEN_TERMINAL)')
     return parser
 
 

lib/devbase/commands/container.py

@@ -327,8 +327,7 @@ def _dispatch_lifecycle(args) -> int:
         'up':    lambda: cmd_up(project_name=project_name,
                                 scale=getattr(args, 'scale', None),
                                 open_editor=getattr(args, 'open_editor', None),
-                                open_index=getattr(args, 'open_index', None),
-                                open_terminal=getattr(args, 'open_terminal', None)),
+                                open_index=getattr(args, 'open_index', None)),
         'down':  lambda: cmd_down(),
         'login': lambda: cmd_login(index=getattr(args, 'index', '1')),
         'ps':    lambda: cmd_ps(all_containers=getattr(args, 'all', False)),
@@ -397,8 +396,8 @@ def _resolve_open_index(open_index: Optional[int], scale: int) -> int:
     """開く dev インスタンス番号を解決する (CLI 引数 → env ``DEVBASE_OPEN_INDEX`` → 既定 1)。
 
     ``1..scale`` の範囲外 (0・負数・``scale`` 超過) は存在しないコンテナを指し原因不明な
-    起動失敗を招くため、警告を出して 1 へフォールバックする。:func:`_maybe_place_terminal_task`
-    と :func:`_maybe_open_editor` で共有し env フォールバック・範囲チェックの重複を避ける。
+    起動失敗を招くため、警告を出して 1 へフォールバックする。:func:`_maybe_open_editor`
+    で env フォールバック・範囲チェックを共有する。
     """
     if open_index is None:
         raw = os.environ.get('DEVBASE_OPEN_INDEX')
@@ -432,8 +431,7 @@ def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
     未指定なら ``.docker-compose.scale.yml`` が存在すればそれ、無ければ None。
 
     ``container_name`` が渡されれば :func:`opener.open_editor` 内の
-    ``resolve_container_name`` (= ``docker compose ps``) をスキップして再利用する
-    (``_maybe_place_terminal_task`` が既に解決済みの名前を使い回し二重実行を避ける)。
+    ``resolve_container_name`` (= ``docker compose ps``) をスキップして再利用する。
     """
     from devbase.editor import opener
 
@@ -464,69 +462,9 @@ def _maybe_open_editor(project_name: str, open_flag: Optional[bool],
         logger.warning("エディタの自動オープンに失敗しましたがデプロイは成功しています: %s", e)
 
 
-def _maybe_place_terminal_task(project_name: str, open_flag: Optional[bool],
-                               open_index: Optional[int], scale: int,
-                               compose_file=None) -> Optional[str]:
-    """`up` 後、開く dev コンテナの作業ディレクトリへ folderOpen ターミナル tasks.json を配置。
-
-    フォルダを開いた時に統合ターミナルを自動表示するための ``.vscode/tasks.json`` を、
-    対象 dev インスタンスのワークスペース (``/work/$GIT_REPO``) に置く。作業ディレクトリは
-    コンテナ内 (named volume) のためホストから直接書けず、起動済みコンテナへ ``docker exec``
-    で書き込む。**既存の ``.vscode/tasks.json`` があれば一切触らない**。
-
-    有効判定は ``open_flag`` (CLI ``--open-terminal``/``--no-open-terminal``) が優先、None なら
-    env ``DEVBASE_OPEN_TERMINAL`` (既定 ON)。配置失敗は warning に握り潰し ``up`` を倒さない。
-    ``open_index`` は開くインスタンスに合わせる (範囲外は 1 へフォールバック)。
-
-    解決した実コンテナ名を返す (無効時は None)。直後の :func:`_maybe_open_editor` へ渡して
-    ``resolve_container_name`` (= ``docker compose ps``) の二重実行を避けるため。
-    """
-    from devbase.editor import opener
-
-    enabled = open_flag if open_flag is not None else opener.is_open_terminal_enabled()
-    if not enabled:
-        return None
-
-    open_index = _resolve_open_index(open_index, scale)
-
-    if compose_file is None and _SCALE_COMPOSE_FILE.exists():
-        compose_file = _SCALE_COMPOSE_FILE
-
-    dev_service_name = get_dev_service_name()
-    container = opener.resolve_container_name(dev_service_name, project_name,
-                                              open_index, compose_file=compose_file)
-    workdir = opener.resolve_workdir(os.environ, project_name)
-    content = opener.build_folder_open_tasks_json()
-
-    # 既存があれば書かず、無ければ stdin から書き込む (冪等)。workdir は引数で渡し
-    # シェル内クォートを避ける ($1)。
-    script = (
-        'set -e; d="$1/.vscode"; mkdir -p "$d"; '
-        'if [ -e "$d/tasks.json" ]; then echo keep; '
-        'else cat > "$d/tasks.json"; echo placed; fi'
-    )
-    try:
-        proc = subprocess.run(
-            ["docker", "exec", "-i", container, "sh", "-c", script, "_", workdir],
-            input=content, text=True, capture_output=True, timeout=15,
-        )
-    except Exception as e:  # noqa: BLE001 - 配置失敗で up を倒さない
-        logger.warning("ターミナル用 tasks.json の配置に失敗しましたが続行します: %s", e)
-        return container  # 名前解決は済んでいるのでエディタ側で再利用させる
-    if proc.returncode != 0:
-        logger.warning("ターミナル用 tasks.json の配置に失敗しましたが続行します: %s",
-                       (proc.stderr or "").strip())
-        return container
-    if (proc.stdout or "").strip() == "placed":
-        logger.info("[6/6] 統合ターミナル自動表示用 tasks.json を配置: %s/.vscode/tasks.json",
-                    workdir)
-    return container
-
-
 def cmd_up(project_name: str = None, scale: int = None,
            open_editor: Optional[bool] = None,
-           open_index: Optional[int] = None,
-           open_terminal: Optional[bool] = None) -> int:
+           open_index: Optional[int] = None) -> int:
     """Deploy containers with specified scale"""
     if project_name is None:
         project_name = get_project_name()
@@ -593,12 +531,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))
 
-        # エディタを開く前に tasks.json を置く (開いた瞬間に folderOpen が効くように)。
-        # 解決済みコンテナ名を editor 側へ渡し docker compose ps の二重実行を避ける。
-        dev_container = _maybe_place_terminal_task(project_name, open_terminal, open_index,
-                                                   scale, compose_file=override_file)
         _maybe_open_editor(project_name, open_editor, open_index, scale,
-                           compose_file=override_file, container_name=dev_container)
+                           compose_file=override_file)
 
         logger.info("=== Deploy completed successfully ===")
         return 0

lib/devbase/editor/opener.py

@@ -128,56 +128,6 @@ def is_open_enabled(environ=None) -> bool:
     return value.strip().lower() in _TRUTHY
 
 
-def is_open_terminal_enabled(environ=None) -> bool:
-    """``DEVBASE_OPEN_TERMINAL`` env が真か (**未設定は True = 既定 ON**)。
-
-    ``DEVBASE_OPEN_EDITOR`` (既定 OFF) と既定が逆である点に注意。up 時の tasks.json 配置は
-    暴発リスクが低く、ユーザ要望で既定 ON とする (PLAN31_3)。
-    """
-    env = os.environ if environ is None else environ
-    value = env.get("DEVBASE_OPEN_TERMINAL")
-    if value is None:
-        return True
-    return value.strip().lower() in _TRUTHY
-
-
-def build_folder_open_tasks_json() -> str:
-    """フォルダを開いた時に統合ターミナルを表示する folderOpen タスク (.vscode/tasks.json)。
-
-    VS Code 公式には「起動時にターミナルを開く」単独設定が無く (``hideOnStartup`` は復元
-    された永続セッションを隠すか否かに過ぎず新規生成はしない)、``runOn: folderOpen`` の
-    タスクが新規ターミナルを出せる唯一の方法 (docs/terminal/*, docs/debugtest/tasks)。
-    ``reveal: always`` でパネルを前面に出し対話シェルを起動する。``type: process`` で
-    ``/bin/sh -lc 'exec "${SHELL:-/bin/sh}"'`` を直接起動し、``$SHELL`` 未設定のコンテナでも
-    ``/bin/sh`` にフォールバックして必ずシェルが立ち上がるようにする (``type: shell`` +
-    ``${env:SHELL}`` だと未設定時に command が空になりタスクが即失敗する)。
-
-    .. note:: 自動実行には2つの user 設定ゲートがあり devbase からは制御できない:
-       Workspace Trust (信頼済みフォルダのみ自動実行) と ``task.allowAutomaticTasks``
-       (既定 off = フォルダ毎に1回許可確認)。いずれも application/user スコープ専用。
-    """
-    tasks = {
-        "version": "2.0.0",
-        "tasks": [
-            {
-                "label": "devbase: open terminal",
-                "type": "process",
-                "command": "/bin/sh",
-                "args": ["-lc", 'exec "${SHELL:-/bin/sh}"'],
-                "isBackground": True,
-                "problemMatcher": [],
-                "presentation": {
-                    "reveal": "always",
-                    "panel": "dedicated",
-                    "focus": True,
-                },
-                "runOptions": {"runOn": "folderOpen"},
-            }
-        ],
-    }
-    return json.dumps(tasks, indent=2, ensure_ascii=False) + "\n"
-
-
 def resolve_editor_cmd(environ=None) -> Optional[list]:
     """起動に使うエディタコマンド (argv list) を解決する。
 

lib/devbase/env/keys.py

@@ -61,4 +61,3 @@ def gcp_credentials_key(profile: str) -> str:
 # Remote-SSH 跨ホスト構成 (Windows VS Code → ssh → Mac のコンテナ) 用。
 DEVBASE_EDITOR_SSH_HOST = "DEVBASE_EDITOR_SSH_HOST"            # 任意。ssh-remote ホスト名 (例 mac2)。通常は ~/.vscode-server から自動検出
 DEVBASE_EDITOR_DOCKER_CONTEXT = "DEVBASE_EDITOR_DOCKER_CONTEXT"  # 任意。ssh 先 docker context (既定: docker context show)
-DEVBASE_OPEN_TERMINAL = "DEVBASE_OPEN_TERMINAL"  # 真偽。up 後に folderOpen ターミナル tasks.json を配置 (既定 ON)

tests/cli/test_project_dispatch.py

@@ -340,19 +340,17 @@ def test_up_parser_open_flags_tri_state():
 
 
 def test_lifecycle_propagates_open_args_to_cmd_up(monkeypatch):
-    """up の open_editor / open_index / open_terminal が cmd_up まで伝播する。"""
+    """up の open_editor / open_index が cmd_up まで伝播する。"""
     from devbase.commands import container
     captured = {}
     monkeypatch.setattr(
         container, 'cmd_up',
-        lambda project_name=None, scale=None, open_editor=None, open_index=None,
-        open_terminal=None: captured.update(
-            open_editor=open_editor, open_index=open_index,
-            open_terminal=open_terminal) or 0)
-    args = _args(subcommand='up', scale=None, open_editor=True, open_index=2,
-                 open_terminal=False)
+        lambda project_name=None, scale=None, open_editor=None,
+        open_index=None: captured.update(
+            open_editor=open_editor, open_index=open_index) or 0)
+    args = _args(subcommand='up', scale=None, open_editor=True, open_index=2)
     assert container._dispatch_lifecycle(args) == 0
-    assert captured == {'open_editor': True, 'open_index': 2, 'open_terminal': False}
+    assert captured == {'open_editor': True, 'open_index': 2}
 
 
 def test_maybe_open_editor_disabled_by_default(monkeypatch):
@@ -423,79 +421,6 @@ def test_maybe_open_editor_valid_index_within_scale(monkeypatch):
     assert called[0]['index'] == 2
 
 
-class _DockerProc:
-    """subprocess.run 互換スタブ (docker exec 用)。"""
-    def __init__(self, returncode=0, stdout="placed", stderr=""):
-        self.returncode = returncode
-        self.stdout = stdout
-        self.stderr = stderr
-
-
-def test_maybe_place_terminal_task_disabled(monkeypatch):
-    """DEVBASE_OPEN_TERMINAL 無効時は docker exec を呼ばない。"""
-    from devbase.commands import container
-    from devbase.editor import opener
-    monkeypatch.setattr(opener, 'is_open_terminal_enabled', lambda environ=None: False)
-    calls = []
-    monkeypatch.setattr(container.subprocess, 'run',
-                        lambda *a, **k: calls.append(a) or _DockerProc())
-    container._maybe_place_terminal_task('carmo', None, None, 1)
-    assert calls == []
-
-
-def test_maybe_place_terminal_task_flag_off_overrides_env(monkeypatch):
-    """open_flag=False なら env が ON でも置かない。"""
-    from devbase.commands import container
-    from devbase.editor import opener
-    monkeypatch.setattr(opener, 'is_open_terminal_enabled', lambda environ=None: True)
-    calls = []
-    monkeypatch.setattr(container.subprocess, 'run',
-                        lambda *a, **k: calls.append(a) or _DockerProc())
-    container._maybe_place_terminal_task('carmo', False, None, 1)
-    assert calls == []
-
-
-def test_maybe_place_terminal_task_runs_docker_exec(monkeypatch):
-    """既定 ON で docker exec -i <container> へ tasks.json を stdin 投入する。"""
-    from devbase.commands import container
-    from devbase.editor import opener
-    monkeypatch.setattr(opener, 'is_open_terminal_enabled', lambda environ=None: True)
-    monkeypatch.setattr(opener, 'resolve_container_name', lambda *a, **k: 'carmo-dev-1')
-    monkeypatch.setattr(opener, 'resolve_workdir', lambda *a, **k: '/work/carmo')
-    monkeypatch.setattr(container, 'get_dev_service_name', lambda: 'dev')
-    captured = {}
-
-    def fake_run(cmd, **kw):
-        captured['cmd'] = cmd
-        captured['input'] = kw.get('input')
-        return _DockerProc(returncode=0, stdout="placed")
-
-    monkeypatch.setattr(container.subprocess, 'run', fake_run)
-    result = container._maybe_place_terminal_task('carmo', None, 1, 1)
-    cmd = captured['cmd']
-    assert cmd[:4] == ['docker', 'exec', '-i', 'carmo-dev-1']
-    assert cmd[-1] == '/work/carmo'  # workdir は $1 として末尾に渡す
-    assert '"runOn": "folderOpen"' in captured['input']
-    # 解決済みコンテナ名を返し editor 側で再利用させる (docker compose ps 二重実行回避)
-    assert result == 'carmo-dev-1'
-
-
-def test_maybe_place_terminal_task_failure_does_not_raise(monkeypatch):
-    """docker exec が例外でも up を倒さない (握り潰す)。"""
-    from devbase.commands import container
-    from devbase.editor import opener
-    monkeypatch.setattr(opener, 'is_open_terminal_enabled', lambda environ=None: True)
-    monkeypatch.setattr(opener, 'resolve_container_name', lambda *a, **k: 'carmo-dev-1')
-    monkeypatch.setattr(opener, 'resolve_workdir', lambda *a, **k: '/work/carmo')
-    monkeypatch.setattr(container, 'get_dev_service_name', lambda: 'dev')
-
-    def boom(*a, **k):
-        raise OSError("docker missing")
-
-    monkeypatch.setattr(container.subprocess, 'run', boom)
-    container._maybe_place_terminal_task('carmo', None, 1, 1)  # 例外が出なければ OK
-
-
 def test_maybe_open_editor_forwards_compose_file(monkeypatch):
     """compose_file 引数が open_editor まで伝播する (実コンテナ名問い合わせ用)。"""
     from devbase.commands import container

tests/editor/test_opener.py

@@ -290,35 +290,6 @@ def boom(cmd, **kw):
     assert opener.resolve_docker_context({}, runner=boom) is None
 
 
-# ---------------------------------------------------------------------------
-# is_open_terminal_enabled (既定 ON)
-# ---------------------------------------------------------------------------
-
-@pytest.mark.parametrize("value,expected", [
-    (None, True), ("", False), ("0", False), ("false", False), ("no", False),
-    ("1", True), ("true", True), ("on", True), ("YES", True),
-])
-def test_is_open_terminal_enabled(value, expected):
-    env = {} if value is None else {"DEVBASE_OPEN_TERMINAL": value}
-    assert opener.is_open_terminal_enabled(env) is expected
-
-
-# ---------------------------------------------------------------------------
-# build_folder_open_tasks_json
-# ---------------------------------------------------------------------------
-
-def test_build_folder_open_tasks_json_is_valid_folderopen_task():
-    data = json.loads(opener.build_folder_open_tasks_json())
-    assert data["version"] == "2.0.0"
-    task = data["tasks"][0]
-    assert task["runOptions"]["runOn"] == "folderOpen"
-    assert task["presentation"]["reveal"] == "always"
-    # SHELL 未設定でも /bin/sh にフォールバックして必ずシェルを起動する (空 command 回避)
-    assert task["type"] == "process"
-    assert task["command"] == "/bin/sh"
-    assert "SHELL:-/bin/sh" in " ".join(task["args"])
-
-
 # ---------------------------------------------------------------------------
 # resolve_container_name / resolve_workdir
 # ---------------------------------------------------------------------------