docs(changelog): add CHANGELOG and CONTRIBUTORS entries for #7525
CI / build (pull_request) Successful in 1m17s
CI / helm (pull_request) Successful in 30s
CI / push-validation (pull_request) Successful in 30s
CI / lint (pull_request) Successful in 1m44s
CI / security (pull_request) Successful in 1m51s
CI / typecheck (pull_request) Successful in 1m59s
CI / quality (pull_request) Successful in 1m57s
CI / integration_tests (pull_request) Successful in 5m28s
CI / e2e_tests (pull_request) Successful in 5m26s
CI / unit_tests (pull_request) Failing after 6m48s
CI / docker (pull_request) Has been skipped
CI / coverage (pull_request) Successful in 9m49s
CI / status-check (pull_request) Failing after 3s
CI / build (pull_request) Successful in 1m17s
CI / helm (pull_request) Successful in 30s
CI / push-validation (pull_request) Successful in 30s
CI / lint (pull_request) Successful in 1m44s
CI / security (pull_request) Successful in 1m51s
CI / typecheck (pull_request) Successful in 1m59s
CI / quality (pull_request) Successful in 1m57s
CI / integration_tests (pull_request) Successful in 5m28s
CI / e2e_tests (pull_request) Successful in 5m26s
CI / unit_tests (pull_request) Failing after 6m48s
CI / docker (pull_request) Has been skipped
CI / coverage (pull_request) Successful in 9m49s
CI / status-check (pull_request) Failing after 3s
Restore CHANGELOG.md and CONTRIBUTORS.md entries that were dropped during a prior rebase/force-push on this branch. - CHANGELOG.md: Added entry under [Unreleased] > Fixed for the budget warning re-trigger fix (issue #7525). - CONTRIBUTORS.md: Added specific contribution entry for HAL 9000 documenting the budget warning re-trigger fix (#7525). ISSUES CLOSED: #7525
This commit is contained in:
@@ -7,6 +7,15 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
|
||||
|
||||
### Fixed
|
||||
|
||||
- **Budget Warning Re-trigger After Session Reconfiguration** (#7525): Fixed a bug
|
||||
where `CostBudgetService._check_warning()` would never re-fire after a session's
|
||||
budget was reconfigured via `configure_session_budget()`. The root cause was that
|
||||
`configure_session_budget()` did not clear `_warned_sessions` when updating
|
||||
`max_cost_usd`, so the session remained permanently suppressed. Fix: added
|
||||
`self._warned_sessions.discard(session_id)` inside the `with self._lock:` block
|
||||
at the end of `configure_session_budget()`, so warning state is reset on every
|
||||
reconfiguration.
|
||||
|
||||
- **TDD Non-AssertionError Guard Visibility** (#8294): `apply_tdd_inversion` in
|
||||
`features/environment.py` now emits its non-assertion exception guard warning to
|
||||
both the structured logger and `stderr` via a new `_warning_with_stderr` helper.
|
||||
|
||||
@@ -21,3 +21,4 @@ Below are some of the specific details of various contributions.
|
||||
* HAL 9000 has contributed the plugin entry point security hardening fix (#7476): enforced entry point allowlist validation before importing plugin modules to prevent malicious plugin loading.
|
||||
* This project was made possible thanks to considerable donation of time, money, and resources by CleverThis, Inc.
|
||||
* HAL 9000 has contributed automated bug fixes, CLI output formatting improvements, and ongoing maintenance as part of the CleverAgents automation system.
|
||||
* HAL 9000 has contributed the budget warning re-trigger fix (#7525): added `_warned_sessions.discard(session_id)` inside the lock in `configure_session_budget()` so budget warnings can re-fire after session reconfiguration.
|
||||
|
||||
Reference in New Issue
Block a user