Skip to content

fix(e2e): compare Windows sandbox identity - #11

Merged
abraxas914 merged 1 commit into
mainfrom
codex/windows-e2e-path-identity
Jul 18, 2026
Merged

fix(e2e): compare Windows sandbox identity#11
abraxas914 merged 1 commit into
mainfrom
codex/windows-e2e-path-identity

Conversation

@abraxas914

@abraxas914 abraxas914 commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • compare sandbox roots by canonical path or filesystem identity
  • handle Windows 8.3 short-path versus long-path aliases without weakening the sandbox boundary
  • reuse the same check in native smoke, bridge startup, and destructive bridge cleanup

Root cause

Nightly correctly launched the bridge from the isolated sandbox, but Node preserved the RUNNER~1 spelling for the harness path while Python reported runneradmin. Exact string comparison rejected the same directory.

Verification

  • regression contract uses two hard links with different path spellings
  • Vitest: 342/342
  • app and E2E TypeScript clean

Follow-up required by Nightly run 29635602999 before desktop-portable-v0.2.1 can be tagged.

@abraxas914
abraxas914 merged commit 961960c into main Jul 18, 2026
3 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