fix(web): use static import for vault store in __E2E helpers

[FSM1]

Summary

The dynamic import('./stores/vault.store').then() in main.tsx created a race condition — window.__E2E was undefined when conflict-detection tests called page.evaluate() in beforeAll because the async import hadn't resolved yet.

Switched to a static import. Vite dead-code-eliminates the entire VITE_E2E block (including the unused import) in production builds where the flag isn't set.

Test plan

• Conflict-detection E2E tests pass (__E2E.getAccessToken() no longer throws)
• Production build doesn't include vault store (verify with VITE_E2E unset)

🤖 Generated with Claude Code

Summary by CodeRabbit

• Refactor
  • Updated internal import handling for improved initialization flow. No user-facing changes.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

The change converts a dynamic import of useVaultStore inside an E2E conditional block to a static top-level import while maintaining the runtime guard. This shifts initialization from asynchronous callback-based to synchronous execution when the VITE_E2E condition is met.

Changes

Cohort / File(s)	Summary
E2E Store Initialization apps/web/src/main.tsx	Replaced conditional dynamic import with static top-level useVaultStore import; converted async Promise-based window.__E2E initialization to synchronous assignment while preserving the VITE_E2E runtime guard.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• chore(e2e): expose Zustand stores in all builds for E2E test access #302: Modifies the same file's E2E export logic and changes how useVaultStore and window.__E2E are exposed.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly describes the main change: replacing a dynamic import with a static import for the vault store in E2E helpers, which is the core fix addressing the race condition issue.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/e2e-static-import-race

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}