Skip to content

fix(context): keep context state and send behavior in sync#306

Merged
sudo-tee merged 2 commits intosudo-tee:mainfrom
disrupted:fix/context-sync-and-delta
Mar 6, 2026
Merged

fix(context): keep context state and send behavior in sync#306
sudo-tee merged 2 commits intosudo-tee:mainfrom
disrupted:fix/context-sync-and-delta

Conversation

@disrupted
Copy link
Contributor

@disrupted disrupted commented Mar 4, 2026

I noticed a few discrepancies between context internal state, UI (context bar/completion menu) and what gets actually sent.

  • current_file could leak into follow-up prompts even when disabled (stale state now gets cleared)
  • context bar/completion state could lag behind real state because some context mutations did not emit state.context_updated_at
  • typo in context bar subscription (opencode_focused vs is_opencode_focused)

tbh I did not know about the current_file delta behavior and why the context bar sometimes shows it dimmed 😄 I've updated the readme with a short description about this feature

@sudo-tee
Copy link
Owner

sudo-tee commented Mar 5, 2026

@disrupted

These are great fixes :)

Thanks for this.

@sudo-tee
Copy link
Owner

sudo-tee commented Mar 5, 2026

@disrupted is this still in draft or ready to review/merge?

@disrupted
Copy link
Contributor Author

@disrupted is this still in draft or ready to review/merge?

I wanna test it a bit more. Haven't had the chance to make sure it's stable. I'll keep it a draft for now and let you know when it's ready

@disrupted disrupted marked this pull request as ready for review March 6, 2026 10:17
@disrupted
Copy link
Contributor Author

@sudo-tee should be ready. I didn't notice any issues

@sudo-tee
Copy link
Owner

sudo-tee commented Mar 6, 2026

Looks all good to me. Thanks

@sudo-tee sudo-tee merged commit 42ae5e2 into sudo-tee:main Mar 6, 2026
5 checks passed
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