fix: default Terra reasoning to medium - #109
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3d66340050
ℹ️ 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".
|
|
||
| export const DEFAULT_AGENT_REASONING = Object.freeze({ | ||
| terra: "high", | ||
| terra: "medium", |
There was a problem hiding this comment.
Keep the default profile out of hierarchy warnings
With a fresh install and no reasoning flags, this changes Terra to medium while Luna remains xhigh; reasoningWarnings() treats that two-rank gap as a reversed plan/execute hierarchy, so every default installation—and the newly documented example—now emits warning: Luna is materially stronger than Terra. If this is the supported default profile, update the warning policy alongside it so users are not told that the package's own defaults are misconfigured.
Useful? React with 👍 / 👎.
Summary
--terra-reasoningvalue fromhightomediumValidation
npm test(109 passed)npm run checknpm run test:packageThe primary-model
--set-defaultbehavior remains unchanged.