feat(up): devbase up 時の .vscode/tasks.json 自動配置を廃止#73
Merged
Conversation
統合ターミナル自動表示用の 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>
takemi-ohama
commented
Jun 20, 2026
takemi-ohama
left a comment
takemi-ohama
left a comment
Contributor
Author
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | APPROVE
修正必須の指摘はありません。
takemi-ohama
commented
Jun 20, 2026
takemi-ohama
left a comment
takemi-ohama
left a comment
Contributor
Author
There was a problem hiding this comment.
🤖 cross-review | round 1 | gemini | COMMENT
_maybe_place_terminal_task の削除に伴い、同機能の二重実行回避のために導入されていたコンテナ名の事前解決・引き回しコードが不要になっています。以下の該当箇所も併せて削除し、デッドコードを整理することを提案します。
lib/devbase/commands/container.py:_maybe_open_editorのcontainer_name引数および docstringlib/devbase/editor/opener.py:open_editorのcontainer_name引数および docstringtests/editor/test_opener.py:test_open_editor_uses_given_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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
devbase up後に統合ターミナルを自動表示する目的で.vscode/tasks.json(folderOpen) を dev コンテナのワークスペースへ自動配置していた挙動を廃止する。実運用では余計な動作だったため削除する。VS Code 自動オープン (PR #69 / #70) はそのまま維持する。関連 Issue
変更点
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: 関連テストを削除動作確認
tests/cli/test_project_dispatch.py,tests/editor/test_opener.py) 118 件 green補足
VS Code 自動オープン機能自体は維持され、ターミナル自動表示用の
tasks.json配置のみが無くなる。