feat(cctrace): rename to Context Causality Trace, add model/effort presets - #3
Merged
Merged
Conversation
…sets SKILL.ja.md の関西弁を標準語に修正し、cctrace の正式名称として Context Causality Trace を両言語版のタイトルに併記。Claude 側は model エイリアス(sonnet-4.6/sonnet-5)、Codex 側は codex_preset(pattern1=Luna/high, pattern2=Terra/low)で GPT-5.6 モデル+reasoning effort を選べるよう config.json スキーマを拡充した。 hook-utils.nu にエイリアス解決・プリセット解決ロジックを追加し、summarize() が -c model_reasoning_effort を渡せるよう変更。実機の codex exec で -m/-c の組み立てを検証済み。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
📝 WalkthroughWalkthroughClaude と Codex のモデル設定を分離しました。Codex は ChangesCodex プリセット対応
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant precompact-auto.nu
participant hook-utils.nu
participant summarize
participant Codex CLI
precompact-auto.nu->>hook-utils.nu: codex_preset から effort を取得
precompact-auto.nu->>summarize: モデルと effort を渡す
summarize->>Codex CLI: model_reasoning_effort を付けて実行
Codex CLI-->>summarize: 要約結果を返す
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@cc-trace/SKILL.ja.md`:
- Around line 261-267: Update the Codex command examples in cc-trace/SKILL.ja.md
lines 261-267 and cc-trace/SKILL.md lines 262-268 to construct arguments
conditionally: omit both -m and model_reasoning_effort when codex_preset is
unset, while including them when a preset is configured. Keep the examples
executable and apply the same behavior consistently in both files.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 20b23a51-d013-4b28-a6d7-917c0d7e1903
📒 Files selected for processing (5)
cc-trace/SKILL.ja.mdcc-trace/SKILL.mdcc-trace/hooks/hook-utils.nucc-trace/hooks/precompact-auto.nucc-trace/tests/test_hooks_nu.sh
Comment on lines
261
to
+267
| **Codex** — `--output-last-message` はサマリーを**ファイル**に書くので、後から読み直すこと。 | ||
| モデルが未設定なら `-m` ごと省略する: | ||
| `codex_preset` が未設定なら `-m` と `-c model_reasoning_effort` の両方を省略する: | ||
| ```bash | ||
| tmp=$(mktemp -t cctrace-summary) | ||
| python3 <skill-dir>/scripts/process_transcript.py --host codex "<transcript>" "<cwd>" "<detail>" \ | ||
| | codex exec --skip-git-repo-check -C "<cwd>" -s read-only -m <model> -o "$tmp" - | ||
| | codex exec --skip-git-repo-check -C "<cwd>" -s read-only -m <model> \ | ||
| -c model_reasoning_effort="<effort>" -o "$tmp" - |
Contributor
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
未設定 preset 用のコマンド例を実行可能にしてください。
codex_preset が null の場合も、現在のコード例は -m <model> と -c model_reasoning_effort を含みます。直前の省略規則に従う利用者は、このコード例をそのまま実行できません。preset の有無に応じて引数を組み立ててください。
cc-trace/SKILL.ja.md#L261-L267: preset 未設定時に-mと-cを渡さない条件付きの Codex コマンド例へ変更してください。cc-trace/SKILL.md#L262-L268: 同じ条件付きの Codex コマンド例へ変更してください。
修正例
tmp=$(mktemp -t cctrace-summary)
+codex_args=()
+if [ -n "$model" ]; then
+ codex_args=(-m "$model" -c "model_reasoning_effort=$effort")
+fi
python3 <skill-dir>/scripts/process_transcript.py --host codex "<transcript>" "<cwd>" "<detail>" \
- | codex exec --skip-git-repo-check -C "<cwd>" -s read-only -m <model> \
- -c model_reasoning_effort="<effort>" -o "$tmp" -
+ | codex exec --skip-git-repo-check -C "<cwd>" -s read-only "${codex_args[@]}" -o "$tmp" -📍 Affects 2 files
cc-trace/SKILL.ja.md#L261-L267(this comment)cc-trace/SKILL.md#L262-L268
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@cc-trace/SKILL.ja.md` around lines 261 - 267, Update the Codex command
examples in cc-trace/SKILL.ja.md lines 261-267 and cc-trace/SKILL.md lines
262-268 to construct arguments conditionally: omit both -m and
model_reasoning_effort when codex_preset is unset, while including them when a
preset is configured. Keep the examples executable and apply the same behavior
consistently in both files.
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.
背景・課題
cctrace は Codex CLI 対応が済んだが、正式名称が旧来の「Claude Code Trace」的な想定のままで、Codex 対応を反映していなかった。また
SKILL.ja.mdに関西弁が混入しており、ドキュメントとして不適切だった。加えて、
.cctrace/config.jsonのcodex_modelは生のモデルID文字列を直書きする方式で、reasoning effort を指定する手段が一切なかった。Claude 側のmodelも生モデルIDの直書きで、ユーザーが選びやすいエイリアスがなかった。方針・変更内容
cc/cctraceは維持）とし、SKILL.md/SKILL.ja.mdのタイトル行に併記SKILL.ja.mdの関西弁を標準語に修正.cctrace/config.jsonのスキーマを変更:model: 生モデルIDではなく Claude 用エイリアス（sonnet-4.6/sonnet-5）codex_modelを廃止しcodex_preset（pattern1= Luna/gpt-5.6-luna/high,pattern2= Terra/gpt-5.6-terra/low）に置き換え。プリセットの中身はhooks/hook-utils.nuに固定テーブルとして定義し、必要なら config.json を直接編集して上書き可能hook-utils.nuにclaude-model-alias/config-codex-presetを追加し、summarize()が-c model_reasoning_effort=<level>を渡せるよう拡張~/.agents/skills/cctrace/（Claude Code / Codex CLI 双方が参照）にも変更を同期却下した代替: pattern1/pattern2 それぞれ2案（例: pattern1 = Luna/high or Terra/medium）が候補にあったが、選択メニューを複雑にしすぎないため固定の組み合わせ1つをデフォルト採用とし、別の組み合わせが欲しい場合は config.json 直編集で対応する設計にした。
影響範囲
cc-trace/スキルのみ。video-frame-reader/ci-monitoringには影響しない.cctrace/ディレクトリ構造・アーカイブ命名規則は変更しないcodex_modelキーとの後方互換は持たせていない（config.json に手動で書かれていた場合は再設定が必要）検証
cd cc-trace && uv run --with pytest python -m pytest tests/ -q— 80件パスbash cc-trace/tests/test_hooks_nu.sh— 52件パスcodex exec（codex-cli 0.147.0）で-m gpt-5.6-luna -c model_reasoning_effort=high/-m gpt-5.6-terra -c model_reasoning_effort=lowを実行し、起動バナーのmodel:/reasoning effort:表示で反映を確認関連 Issue
Closes #
メモ
このPRには、リポジトリ用の
CONTRIBUTING.mdと.github/pull_request_template.mdの追加（tukuyomil032/embed-previewを土台にこのリポジトリ向けへ最適化したもの）も含まれる。既にmainへ直接コミット・pushしてあり、このブランチにはマージ済み。このPR自体の説明も、新設したテンプレートの形式に従って書いている。🤖 Generated with Claude Code
Summary by CodeRabbit
新機能
ドキュメント