Skip to content

fix(web): preserve provider usage auto-resume toggle - #180

Merged
johnnyelwailer merged 1 commit into
mainfrom
fix/provider-usage-toggle-preservation
Sep 14, 2026
Merged

johnnyelwailer merged 1 commit into
mainfrom
fix/provider-usage-toggle-preservation

Conversation

@johnnyelwailer

Copy link
Copy Markdown
Owner

Defect

The provider-usage hold banner reset autoResume to the default enabled state whenever a started or deferred activity omitted the toggle. The server intentionally omits that field when refreshing a provider reset time and when recording a deferred turn, so a user's disabled auto-resume choice could be silently re-enabled in the UI.

Fix

Preserve the existing derived toggle when a hold activity omits autoResume, while honoring explicit boolean values. Add a regression test covering a disabled toggle followed by a reset-time refresh.

Verification

  • git diff --check passed.
  • Focused Vitest was unavailable in the clean checkout because repository dependencies are not installed.

Residual risk

The focused test should run in CI with the repository's normal dependency/tooling setup.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS labels Sep 7, 2026
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Thread transfer impact

⚠️ The latest CI run did not produce a thread transfer result for a623db0.

This comment will update automatically after the next completed run.

@johnnyelwailer
johnnyelwailer merged commit 044ca96 into main Sep 14, 2026
18 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex codex-automation size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants