feat(ask): add --inline-max-bytes per-run override - #1181
Merged
SamSaffron merged 1 commit intoSep 26, 2026
Merged
Conversation
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.
Adds
term-llm ask --inline-max-bytes <size>to overrideask.stdin_inline_max_bytesfor a single run.Why
Since
2cc3b291(structured ask input attachments), text from stdin /-flarger thanask.stdin_inline_max_bytes(default 10 KiB = 10240 bytes) is staged as a file and the model mustread_fileit. Scripted single-turn runs with deliberately large prompts break: the Jarvis daily digest passes a ~166 KB prompt with--max-turns 1and failed three days running withagentic loop exceeded max turns (1). The only fix was a global config change; this makes it per-invocation.Behaviour
4096,512K(512,000),2MiB(2,097,152).ask.stdin_max_bytes(default 20 MiB = 20,971,520); errors otherwise, e.g.50MiB→must not exceed ask.stdin_max_bytes (20971520).stdin_max_bytesis never changed.github.com/dustin/go-humanizemoves from indirect to direct (already in the module graph).Verification
1MiB.go test ./cmd,go build ./...,go vet ./cmd,make complexitypass.--max-turns 1:--inline-max-bytes 4K→agentic loop exceeded max turns (1)--inline-max-bytes 1MiB→PERSIMMON