Repository navigation
docs(agent): mark spark-1 models deprecated in help text and README - #293
Conversation
There was a problem hiding this comment.
All reported issues were addressed across 3 files
Shadow auto-approve: would auto-approve with 1 open P3 issue. Docs-only update marking spark-1 agent models deprecated in CLI help, AgentOptions comment, and README, and switching the README example to --effort high. No behavior change: aliases still accepted, so the change is bounded and clearly beneficial.
Fix all with cubic | Re-trigger cubic
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would auto-approve. Docs-only update: CLI help text, AgentOptions comment, and README now present spark-2 as default with spark-1 aliases deprecated, and the README example uses --effort high. No behavior code changes, so risk is limited to documentation accuracy.
Re-trigger cubic
Part of removing the spark-1 agent models so spark-2 is the only generation left.
--modelhelp text and theAgentOptions.modeldoc comment now say spark-2 is the default andspark-1-mini/spark-1-proare deprecated and run spark-2.spark-1-minithe default. The "higher accuracy" example now uses--effort highinstead of--model spark-1-pro.Kept on purpose:
--model spark-1-pro/spark-1-miniare still accepted by the local validation and theAgentModeltype. The API accepts them as deprecated aliases that run spark-2, so scripts that pass them keep working instead of failing locally.pnpm testgives the same 41 failures onmainwithout this change (Alexandria discovery tests), so they're unrelated.Summary by cubic
Marks the spark-1 agent models as deprecated in the CLI help text,
AgentOptionsdoc comment, and README sospark-2is presented as the only current model. The README options table now documents the--effortoption, and the higher-accuracy example uses--effort highinstead of--model spark-1-pro.spark-1-miniandspark-1-proare still accepted as deprecated aliases that runspark-2, so existing scripts that pass them keep working.Written for commit 75ca31b. Summary will update on new commits.