docs(cli): complete task and chat budget help - #1626
Closed
santoshkumarradha wants to merge 4 commits into
Closed
santoshkumarradha wants to merge 4 commits into
santoshkumarradha wants to merge 4 commits into
Conversation
santoshkumarradha
marked this pull request as ready for review
September 27, 2026 17:56
Member
Author
|
Superseded by the consolidated draft #1627. Exact reviewed head |
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
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.
The main help synopsis omitted supported task model/worker flags and chat budget flags, and its shared headless-work explanation contained a garbled sentence. The summary now names those options and explains that chat budgets require
--yolo.Part of #1558. The token-count correction is already present in #1604. The engine command's deliberate omission from public help and home-relative workspace semantics are unchanged. Includes the existing test-only model-pin fixture correction from #1615 so full CLI verification preserves its disabled load gates.
Validation: final head
72c8ff1a4f5cfd7a547d96ea4e6e9374e5db444bpassed Sparkmake pr-ready(exit 0; complete CLI package 191.015 seconds) and all GitHub checks. Built withmake build; actual--help,do --help, andchat --helpall exit 0 and include the missing options. Existing help layout tests pass with the original page/column limits unchanged. No model call is needed for this help-only change.Evidence:
extended/1558-ready.log/.exit,1558-check-host.txt,1558-check-revision.txt,1558-layout-pass.log,1558-final-cli-verification.jsonand the three1558-final-help-*.txtcaptures. Initial help-layout failures are preserved; the correction shortened existing prose instead of raising layout limits.