fix: implement deriver custom instructions#430
fix: implement deriver custom instructions#430adavyas wants to merge 3 commits intofix/prefix-based-cache-optimfrom
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip CodeRabbit can approve the review once all CodeRabbit's comments are resolved.Enable the |
Summary
Implements
reasoning.custom_instructionsfor the deriver on top of the prefix-cache prompt split from #425.What Changed
custom_instructionsin resolved reasoning configcustom_instructionsto the deriver user promptScope
This only implements
custom_instructionsfor the deriver.It does not add
custom_instructionssupport to:Why
The schema already exposed
reasoning.custom_instructions, but it was not used at runtime. This wires it into the deriver while preserving the cache-friendly split introduced in #425 by keeping the instructions in the user message rather than the stable system prompt.Testing
Verified locally with:
python -m py_compileon changed filesbasedpyrighton changed filestests/deriver/test_deriver_processing.pytests/deriver/test_prompts.pytests/utils/test_config_helpers.pyTargeted pytest result:
15 passed