docs: changelog entry for the daily spend rail (#1546) - #1744
Open
santoshkumarradha wants to merge 1 commit into
Open
santoshkumarradha wants to merge 1 commit into
santoshkumarradha wants to merge 1 commit into
Conversation
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Assisted-by: CodeAF (glm-5.3) Co-Authored-By: CodeAF <267109073+agentfield-bot@users.noreply.github.com>
Member
Author
|
Pass 1 — ARCHITECTURE Checked: where the change sits and how it is shaped.
Found: nothing to change. drafted with CodeAF |
Member
Author
|
Pass 2 — SECURITY Checked: what the entry exposes and what it touches.
Found: nothing to change. drafted with CodeAF |
Member
Author
|
Pass 3 — CODE EFFICIENCY & DESIGN Checked: economy of the wording and truth of every claim.
Found: nothing to change. drafted with CodeAF |
santoshkumarradha
marked this pull request as ready for review
October 3, 2026 19:30
Member
Author
|
@ZeroPoint95 ready for review: all three review passes completed with no actionable findings, and final-head CI is green. This is a docs-only changelog change. |
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.
Closes #1546
What
The daily spend rail fix is already on
devvia #1627 (commitac84606f4): interactive turns enforce the day's budget, task workers and errands are exempt, and the tests are there too (TestSpendRailRefusesTurnWhenDailyBudgetExceeded,TestBudgetScopeAndLiveConversationRecovery). The takeover branch (fix/1546-daily-spend-rail) turned out to be a strict duplicate with older semantics — it was never pushed. The one piece that never landed is this changelog entry, which is all this PR adds.The entry
kind: fixed, surfaces[chat, engine],pr: 1546.internal/session/rail.go:daily limit reached · %s spent of %s · /budget day changes it.Gates
Docs-only diff: no video owed under the bar. No workloads ran locally (raw-ssh-Spark-only rule honored; none needed — GitHub CI validates). Three review passes are posted as separate comments. PR stays draft/[WIP] until CI is green; never merged by its author.
—
Drafted with CodeAF · reviewed and owned by the author