v5.0.3 — Opus 4.7 vocabulary alignment + mermaid diagrams - #11
JosephOIbrahim wants to merge 2 commits into
Conversation
Add DEPTH_TO_EFFORT mapping aligning Orchestra's depth tiers with
Anthropic's `output_config.effort` levels. On Opus 4.7, the previous
`thinking.budget_tokens` API returns 400; `effort` (low/medium/high/xhigh)
is the correct knob. Additive, non-breaking: DEPTH_BUDGETS stays exported,
ThinkDepth enum values unchanged, anchor format unchanged.
Soften all-caps imperatives in claude_code_hook expert_guidance
("EMPATHY FIRST" → "Lead with empathy", etc.) for Opus 4.7's stricter
literal-instruction-following calibration. Routing behavior unchanged.
Add "Scope of Determinism" section to THINKINGMACHINES_COMPLIANCE.md
distinguishing Orchestra's batch-invariant routing (deterministic per
He2025) from Claude API output (not deterministic, especially under
adaptive thinking which is the only on-mode on Opus 4.7).
Retroactive CHANGELOG entries for 5.0.1 + 5.0.2 (entries were missing
since the version bumps shipped without notes).
Verification: 797 tests pass + 3 new TestDepthToEffort assertions.
He2025 batch-invariance tests intact. Anchor format unchanged.
Zero new semgrep findings.
Deferred to v5.1.0: MCP server anthropic SDK integration with prompt
caching, duplicate guidance-dict consolidation, framework_orchestrator
temperature audit.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Replace the 5-phase NEXUS pipeline ASCII art with a mermaid flowchart using a two-tone palette (charcoal #1F2937 for processing phases, warm cream #F5F1EA for I/O boundaries). Renders correctly on GitHub light and dark themes. Add a new "Burnout Escalation" subsection between Safety Gating and CLI Commands, with a mermaid stateDiagram-v2 showing GREEN → YELLOW → ORANGE → RED transitions and recovery paths. Same two-tone palette (cream = calm, charcoal = alert) reinforces the visual language. Recovery from RED is annotated as body-first protocol — matches the existing CLAUDE.md guidance and product philosophy. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
📝 WalkthroughWalkthroughVersion 5.0.3 introduces a Changesv5.0.3 Release: DEPTH_TO_EFFORT and Opus 4.7 Alignment
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
PR Analysis SummaryChanged files: 7 Checks
ThinkingMachines [He2025] Compliance
|
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/orchestra/parameter_locker.py (1)
432-436:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winAdd
DEPTH_TO_EFFORTto__all__.
DEPTH_TO_EFFORTis imported and re-exported from__init__.py(line 359) and included in the top-level__all__(line 659), but it's missing from this module's__all__export list. For consistency and discoverability, add it here alongsideDEPTH_BUDGETS.📦 Proposed fix to add DEPTH_TO_EFFORT to __all__
__all__ = [ 'ThinkDepth', 'Paradigm', 'LockStatus', 'LockedParams', 'LockResult', 'ParameterLocker', - 'DEPTH_BUDGETS', 'create_locker' + 'DEPTH_BUDGETS', 'DEPTH_TO_EFFORT', 'create_locker' ]🤖 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 `@src/orchestra/parameter_locker.py` around lines 432 - 436, The module-level __all__ is missing DEPTH_TO_EFFORT — update the __all__ list in parameter_locker.py to include 'DEPTH_TO_EFFORT' alongside 'DEPTH_BUDGETS' so the symbol is exported; locate the __all__ definition that currently lists 'DEPTH_BUDGETS' and add 'DEPTH_TO_EFFORT' to that list.
🤖 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.
Outside diff comments:
In `@src/orchestra/parameter_locker.py`:
- Around line 432-436: The module-level __all__ is missing DEPTH_TO_EFFORT —
update the __all__ list in parameter_locker.py to include 'DEPTH_TO_EFFORT'
alongside 'DEPTH_BUDGETS' so the symbol is exported; locate the __all__
definition that currently lists 'DEPTH_BUDGETS' and add 'DEPTH_TO_EFFORT' to
that list.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 941fb8cd-445d-4d26-9d02-4f472eec6e10
📒 Files selected for processing (7)
CHANGELOG.mdREADME.mdTHINKINGMACHINES_COMPLIANCE.mdsrc/orchestra/__init__.pysrc/orchestra/claude_code_hook.pysrc/orchestra/parameter_locker.pytests/test_parameter_locker.py
Summary
Two-commit PR migrating Orchestra's internal vocabulary from
thinking.budget_tokens(Opus 4.5/4.6) tooutput_config.effort(Opus 4.7), plus a docs refresh replacing ASCII diagrams with mermaid.Commit 1 —
feat: v5.0.3 — Opus 4.7 vocabulary alignment(fb97d7c)DEPTH_TO_EFFORTmapping inparameter_locker.py(MINIMAL→"low",STANDARD→"medium",DEEP→"high",ULTRADEEP→"xhigh"). Exported fromorchestratop-level. On Opus 4.7,thinking.budget_tokensreturns 400;effortis the correct knob.claude_code_hook.pyexpert_guidance("EMPATHY FIRST."→"Lead with empathy.", etc.) for Opus 4.7's stricter literal-instruction-following. Routing behavior unchanged.THINKINGMACHINES_COMPLIANCE.mddistinguishing Orchestra's batch-invariant routing (deterministic per He2025) from Claude API output (not deterministic under adaptive thinking).TestDepthToEffort.Non-breaking:
DEPTH_BUDGETSstays exported,ThinkDepthenum values unchanged, anchor format unchanged, MCP schema unchanged.Commit 2 —
docs: replace ASCII pipeline with mermaid + add burnout state diagram(be587d3)Verification
[EXEC:checksum|expert|paradigm|altitude|depth](5 fields) preserved6bb68dDEPTH_BUDGETS+DEPTH_TO_EFFORT+ThinkDepthall importableTest plan
test-statuscheckpip install -e .from clean checkout worksorchestra statusrunsDeferred to v5.1.0
anthropicSDK integration with prompt caching +effortmappingexpert_guidancedicts (hooks/cognitive_hook.py↔claude_code_hook.py)framework_orchestrator.pyDeferred to v5.0.4 cleanup
CONTRIBUTING.md,CITATIONS.md,docs/QUICKSTART.md,ADVANCEMENT_ROADMAP.md,docs/USD_COGNITIVE_SUBSTRATE_V5.md(still say 5.0.1)docs/(root is canonical)StatusView.jsxlabel semantics (budget: '1K'→effort: 'low')🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
DEPTH_TO_EFFORTparameter mapping for specifying model effort levels across depth tiers.Documentation
Improvements