Skip to content

fix: serialize embedded view refreshes - #194

Merged
vycdev merged 2 commits into
vycdev:developfrom
vycdev2:fix/serialize-embedded-refreshes
Aug 10, 2026
Merged

fix: serialize embedded view refreshes#194
vycdev merged 2 commits into
vycdev:developfrom
vycdev2:fix/serialize-embedded-refreshes

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 9, 2026

Copy link
Copy Markdown

Summary

  • Serialize per-file embedded view refreshes so filesystem and open-document events cannot update a provider concurrently.
  • Keep the refresh queue usable after a rejected refresh while preserving the original error for the caller.
  • Add a regression test and document the user-visible fix in the 5.3.0 changelog.

Verification

  • npm install — passed; npm reported existing deprecation and vulnerability warnings.
  • npm test — passed (135 tests).
  • npm run compile — passed.
  • npm run format:check — passed.
  • git diff --check upstream/develop...HEAD — passed.
  • The new test was run before implementation and failed because enqueueEmbeddedRefresh was not exported; it passed after the implementation.

Risk

  • Low: only the embedded view's targeted refresh scheduling changes; full refresh behavior and provider error handling remain unchanged.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

@vycdev
vycdev merged commit 6f81c0c into vycdev:develop Aug 10, 2026
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