fix: ultracode 通しレビューのフォローアップ(#138、v0.71.0)#140
Merged
Conversation
…、v0.71.0) - custom プロファイル update で agent-teams env が消える問題を修正(後発キーのデフォルト補完) - doc-blocker 拡張子ゲートの case-insensitive 化(SUMMARY.MD バイパス解消) - _strip_retired_hook_entries を /.claude/hooks/ にアンカー強化(ユーザー同名ディレクトリ保護) - agent-teams をウィザード・--hooks で切替可能に(#120 ギャップ)、GA トラッキング #139 起票 - effortLevel merge 2 ケース・spec-kit ゲート・retired 4 feature のテスト追加(unit 348) - CLAUDE.md 注入を delete モード化 + partial 区切り空行(体裁改善) - README ja/en・CLAUDE.md・hooks-reference・standard.conf のドキュメント残骸を掃除 Closes #138
There was a problem hiding this comment.
Pull request overview
v0.57.0〜v0.70.0 の累積差分レビュー(#138)で見つかった挙動バグと受け入れギャップを解消し、フック/注入/ドキュメント/テストを整合させるフォローアップPRです。主に「agent-teams の扱い」「CLAUDE.md 注入のクリーンアップ」「retired hook の誤マッチ防止」「doc-blocker の拡張子判定」を修正しています。
Changes:
- custom プロファイル update 時に
ENABLE_AGENT_TEAMSが欠落して env fragment が落ちる問題のデフォルト補完を追加 doc-blockerの拡張子ゲートを case-insensitive 化、retired hook strip のパス判定を/.claude/hooks/<feature>/にアンカー- CLAUDE.md 注入の未解決マーカー削除を delete モード化+各 partial 先頭の区切り空行追加、関連する回帰テスト/ドキュメント更新
Reviewed changes
Copilot reviewed 24 out of 24 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| wizard/wizard.sh | custom update 経路で ENABLE_AGENT_TEAMS のデフォルトを補完し、3-way merge によるキー削除を回避 |
| wizard/registry.sh | agent-teams を --hooks / wizard の選択肢として登録(keys/tokens/labels) |
| tests/unit/test-web-content-extraction.sh | spec-kit partial の INSTALL_COMMANDS ゲート回帰テストを追加 |
| tests/unit/test-retired-hooks.sh | retired hooks の一括 strip とユーザー同名ディレクトリ保護のテストを追加 |
| tests/unit/test-merge.sh | effortLevel ピン解除の 3-way merge 挙動(削除/保持)テストを追加 |
| README.md | プロファイル表/フック数/エディタ注記/--hooks 例を更新(agent-teams 反映) |
| README.en.md | Profiles 表から Memory 列を整理し、--hooks 例に agent-teams を追加 |
| profiles/standard.conf | Standard プロファイル説明コメントを現状に合わせて更新 |
| lib/update.sh | retired hook strip のマッチを /.claude/hooks/<feature>/ にアンカーして誤除去を防止 |
| lib/deploy.sh | spec-kit 注入コメント修正、未解決マーカーの削除を delete モードに変更 |
| i18n/ja/strings.sh | agent-teams のフック表示文言を追加 |
| i18n/ja/partials/web-content-extraction.md | partial 先頭に区切り空行を追加(注入時の体裁改善) |
| i18n/ja/partials/spec-kit.md | partial 先頭に区切り空行を追加(注入時の体裁改善) |
| i18n/ja/CLAUDE.md.base | marker 周辺の余分な空行を整理(注入体裁改善に追随) |
| i18n/en/strings.sh | agent-teams のフック表示文言を追加 |
| i18n/en/partials/web-content-extraction.md | partial 先頭に区切り空行を追加(注入時の体裁改善) |
| i18n/en/partials/spec-kit.md | partial 先頭に区切り空行を追加(注入時の体裁改善) |
| i18n/en/CLAUDE.md.base | marker 周辺の余分な空行を整理(注入体裁改善に追随) |
| features/doc-blocker/scripts/check-doc-write.sh | .md/.txt 判定を case-insensitive に変更 |
| features/codex-plugin/CLAUDE.md.partial.ja | partial 先頭に区切り空行を追加(注入時の体裁改善) |
| features/codex-plugin/CLAUDE.md.partial.en | partial 先頭に区切り空行を追加(注入時の体裁改善) |
| docs/GUIDES/hooks-reference.md | compaction 周辺の説明を現状(セッション状態はネイティブ委譲)に更新 |
| CLAUDE.md | Inline hooks 例・Doc Size Guard 説明を更新 |
| CHANGELOG.md | 0.71.0 の変更点を追記し、#138 の修正内容を記録 |
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
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.
概要
#121 完了後に実施した v0.57.0〜v0.70.0 累積差分の multi-agent 通しレビュー(8 次元レビュー + 14 Issue 受け入れ照合 + 敵対的 3 票検証、112 エージェント)で確認された findings と #120 の受け入れギャップをまとめて修正する。レビューでは critical / high はゼロ、acceptance は 14 Issue 中 13 件 OK だった。
挙動修正(3 件)
_fill_noninteractive_defaultsを通らず custom には conf が無いため、ENABLE_AGENT_TEAMS空 → fragment 不注入 → 3-way merge でキー削除となっていた(レビューの敵対的検証が session limit で部分故障し一度 rejected になった指摘を、本体側で再検証して実バグと確認)。_restore_config_from_manifestに後発キーの補完を追加SUMMARY.MD等が ask をバイパスしていた(CHANGELOG 0.62.0 の「大文字小文字不問」との不一致解消)_strip_retired_hook_entriesのアンカー強化:/hooks/<feature>/の substring マッチが~/dotfiles/hooks/memory-persistence/のようなユーザー自作 hook に誤一致し得たため/.claude/hooks/<feature>/に変更(保護テスト付き)#120 受け入れギャップ(3 件)
--hooksで切替可能に(i18n en/ja・README 例も追随)ドキュメント・体裁・テスト
検証
Closes #138