Skip to content

feat: Broad-Side re-submits truncated slices automatically (#133) - #151

Merged
TheAmericanMaker merged 2 commits into
mainfrom
feat/133-broadside-truncation-resubmit
Aug 24, 2026
Merged

feat: Broad-Side re-submits truncated slices automatically (#133)#151
TheAmericanMaker merged 2 commits into
mainfrom
feat/133-broadside-truncation-resubmit

Conversation

@TheAmericanMaker

Copy link
Copy Markdown
Member

Closes the remaining half of #133 (truncation detection shipped earlier in #144).

  • Submit persists every request body to .codecarto/broadside/<run>/requests.json — so collect can recover a truncated slice without re-walking the repo (also survives a crash between submit and collect).
  • Collect re-submits each truncated result once with a doubled max_tokens (bounded by the model's completion ceiling), rewrites the recovered JSON/markdown, clears the truncation flag, and reports ↻ N recovered in the summary.
  • Still-truncated slices stay flagged (⚠ still truncated after retry); opt out with retry_truncated: false.

3 new tests (request persistence, resubmit flow asserting the doubled cap, opt-out path). 38 broadside tests, 398 total, all passing.

Note: this branch inserts a bullet at the top of the CHANGELOG 'Added' section, as do #149 and #150 — trivial list-merge conflicts expected on whichever merges later; keep all bullets.

Submit persists every request body to .codecarto/broadside/<run>/
requests.json so collect can recover coverage lost to a max_tokens
cutoff without re-walking the repo. Collect re-submits each truncated
result once with a doubled output cap (bounded by the model's
completion ceiling), rewrites the JSON/markdown for recovered slices,
clears their truncation flag, and reports the recovery in the collect
summary. Still-truncated slices stay flagged. Opt out with
retry_truncated: false.

Closes the remaining half of #133. 3 new tests (request persistence,
resubmit flow with a doubled cap, opt-out); 38 broadside tests, 398
total, all passing.
# Conflicts:
#	CHANGELOG.md
#	ROADMAP.md
#	tests/broadside.test.mjs
@TheAmericanMaker
TheAmericanMaker merged commit f2ed4e6 into main Aug 24, 2026
5 checks passed
@TheAmericanMaker
TheAmericanMaker deleted the feat/133-broadside-truncation-resubmit branch August 24, 2026 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants