Skip to content

Fix WASM initialization retries - #61

Merged
melonamin merged 1 commit into
masterfrom
agent/fix-wasm-init-retry
Aug 12, 2026
Merged

Fix WASM initialization retries#61
melonamin merged 1 commit into
masterfrom
agent/fix-wasm-init-retry

Conversation

@melonamin

Copy link
Copy Markdown
Member

What changed

  • use one coherent WASM initialization lifecycle instead of a second analyzer-level rejected-promise cache
  • allow a later call to retry after transient initialization failure
  • guard reset and concurrent initialization so stale completions cannot repopulate cleared state
  • add regression coverage for retry, concurrency, reset, and analyzer calls

Why

A failed first initialization was cleared by the loader but remained cached by the analyzer, permanently poisoning later analysis, completion, and export calls in the process.

Validation

  • mise x node@20.20.2 yarn@1.22.19 -- yarn workspace @pondpilot/flowscope-core test
  • mise x node@20.20.2 yarn@1.22.19 -- just check
  • Codex review: no actionable findings

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@melonamin
melonamin marked this pull request as ready for review August 12, 2026 16:14
@melonamin
melonamin merged commit 6a6341d into master Aug 12, 2026
9 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.

1 participant