feat: 配布設定の世代依存ピンを整理（#120、v0.70.0）#137
Conversation
….70.0） - CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS を settings-base から agent-teams feature へ切り出し（全プロファイル既定 true・文書化・GA 時の除去経路確保） - settings-base の effortLevel: high 固定を削除（CLI の世代別 effort 管理に委譲、ユーザー上書きは merge で保持） - auto-update.sh の MIN_ASYNC_VERSION デッド変数を 2 箇所同時削除 - legacy hook 経路の退役マイルストーンを Issue #136 として起票 Closes #120
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 244693af1f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ENABLE_BIOME_HOOKS="false" | ||
| ENABLE_PR_CREATION_LOG="false" | ||
| ENABLE_GHOSTTY_SETUP="false" | ||
| ENABLE_AGENT_TEAMS="true" |
There was a problem hiding this comment.
For existing installs saved with PROFILE=custom, this default is not applied on update or saved-config reuse: _restore_config_from_manifest calls load_profile_config "$PROFILE", and fill_missing_profile_defaults custom also routes through load_profile_config, but there is no profiles/custom.conf, so old custom configs that lack ENABLE_AGENT_TEAMS leave it empty. In that scenario build_settings_file skips the new agent-teams fragment and the 3-way merge removes CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS from users who had the previous unconditional setting, changing behavior despite the intended default being true.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Pull request overview
配布設定に含まれていた世代依存・実験フラグの「常時ピン」を整理し、既存挙動を維持しつつ設定の可搬性/将来追従性を上げる変更です。CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS=1 の feature 化と、effortLevel: "high" の静的ピン撤去、auto-update のデッド変数削除を通して、配布物が将来の CLI デフォルトを黙って上書きしない方向に寄せています。
Changes:
CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS=1をagent-teamsfeature（ENABLE_AGENT_TEAMS）へ切り出し、wizard/profiles/docs に配線config/settings-base.jsonからenvとeffortLevelのピンを削除し、テストで担保- auto-update の
MIN_ASYNC_VERSIONデッド変数を削除し、detach 経路の env 伝播も合わせて整理
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| wizard/wizard.sh | ENABLE_AGENT_TEAMS の wizard グローバル初期化を追加 |
| wizard/steps.sh | 確認表示に Agent Teams を追加し、非対話 defaults を追加 |
| wizard/registry.sh | wizard の config allowlist/save 対象に ENABLE_AGENT_TEAMS を追加 |
| wizard/defaults.conf | custom profile 用 defaults に ENABLE_AGENT_TEAMS=true を追加 |
| tests/unit/test-content-cleanup.sh | settings-base から世代依存ピンが消えたこと等の静的テストを追加 |
| profiles/standard.conf | ENABLE_AGENT_TEAMS=true を追加 |
| profiles/minimal.conf | ENABLE_AGENT_TEAMS=true を追加 |
| profiles/full.conf | ENABLE_AGENT_TEAMS=true を追加 |
| lib/features.sh | feature registry に agent-teams を登録し merge 順序へ追加 |
| i18n/ja/strings.sh | confirm 表示用ラベルを追加 |
| i18n/en/strings.sh | confirm 表示用ラベルを追加 |
| features/auto-update/scripts/auto-update.sh | MIN_ASYNC_VERSION の定義/伝播を削除 |
| features/agent-teams/hooks.json | env 注入フラグメントを新規追加 |
| features/agent-teams/feature.json | Agent Teams feature メタデータを新規追加 |
| docs/wizard-config-mapping.md | ENABLE_AGENT_TEAMS をマッピング表に追記 |
| docs/wizard-config-mapping.en.md | ENABLE_AGENT_TEAMS をマッピング表に追記 |
| config/settings-base.json | env と effortLevel の静的ピンを削除 |
| CHANGELOG.md | v0.70.0 として変更点を記録 |
| "description": "Enable Claude Code experimental agent teams via CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS=1", | ||
| "category": "setting", | ||
| "default": true, |
| { | ||
| test_name="content-cleanup: settings-base has no generation-dependent pins (env / effortLevel)" | ||
| if ! jq -e 'has("env") or has("effortLevel")' "$PROJECT_DIR/config/settings-base.json" >/dev/null \ | ||
| && jq -e '.env.CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS == "1"' "$PROJECT_DIR/features/agent-teams/hooks.json" >/dev/null \ | ||
| && grep -q '\[agent-teams\]=ENABLE_AGENT_TEAMS' "$PROJECT_DIR/lib/features.sh"; then | ||
| pass "$test_name" | ||
| else | ||
| fail "$test_name" | ||
| fi | ||
| } |
概要
LLM 性能監査計画 #121 の最終 Issue P4-14（#120）。これで P1-P4 全 14 Issue が完了。
変更内容
CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS=1を、no-flicker と同型の feature（ENABLE_AGENT_TEAMS、全プロファイル既定 true）へ切り出し。既存ユーザーの動作は変わらず（settings-base から消えても fragment で再注入）、ウィザードでオフ可能になり、docs/wizard-config-mapping ja/en に文書化。GA 時はフラグメントからキーを削除するだけで全ユーザーから除去できる（fix: 3-way merge がキー削除の代わりに literal JSON null を書き込む問題を修正 #75 の kit-remove 経路 — Issue 原文の「除去経路がない」前提は不採用）検証
Closes #120