Repository navigation
Fix developing-story response budgets and request context - #108
Merged
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.
Why
The first live developing-story preview stopped halfway through JSON at the inherited 900-token output ceiling. It also exposed ambiguous citation/action instructions. Live acceptance of the first fix then found that realistic source metadata and the revision payload exceeded the existing input limit.
Closes #106.
Changes
Validation
git diff --check: passed.Live preview result
The final candidate modules were loaded only into a separate no-send preview process. Deployed files and the running bot were unchanged. The source bytes match this commit.
The draft completed normally (
finish_reason=stop) at 1,638 output tokens under the 4,096 ceiling, with 5,040 prompt tokens. This resolves the observed truncation. However, the model omitted two required keys, emitted eleven claims, left change uncited, and mixed eight papers under one publish story ID. Strict parsing rejected it before independent review. This is not an accepted draft or editorial hold.The preview spent 6,678 tokens and $0.0011592 through ordinary usage accounting. Before/after comparison confirmed unchanged story, decision, delivery, seen, observation, lease and Curated metadata state. No message was sent.
Issue #107 owns the remaining model-adherence and focused-reporting failure. Story activation remains blocked on a later complete, independently reviewed live result.
Rollout boundary
No merge, deployment, configuration change or Discord post is part of this PR preparation. Story mode remains disabled; the current Digest channel and daily settings remain configured. A merge releases an image to the existing deployment timer and requires explicit approval. Story-mode and stream activation remain separate decisions.
The larger output allowance is a ceiling, not a target. Budget checks still happen before each request, so a call can cross the daily threshold and prevent a later review/revision. Unusually large contexts still fail closed at the unchanged input limit. A revision is limited to evidence selected or cited by the rejected report and can hold when that evidence is insufficient.