Skip to content

Fix Windows Connect generation staging (4.3.0) - #3

Merged
cursor[bot] merged 2 commits into
mainfrom
cursor/windows-helper-staging-5371
Aug 18, 2026
Merged

Fix Windows Connect generation staging (4.3.0)#3
cursor[bot] merged 2 commits into
mainfrom
cursor/windows-helper-staging-5371

Conversation

@devlifeX

Copy link
Copy Markdown
Owner

Problem

Windows 4.2 Connect failed at register_runtime_generation with:

staged generation … is unreadable at C:\ProgramData\biflow\runtime\generations\… (os error 2)

NSIS perMachine uses SetShellVarContext all, so $LOCALAPPDATA expands to C:\ProgramData. The helper recorded that path in helper.toml while the desktop staged generations under the user’s %LOCALAPPDATA%\biflow\runtime\generations. SYSTEM never saw the files.

Fix (ADR 0064)

  • Packaged Windows staging is always C:\ProgramData\iran-split\staging (sibling of runtime, never nested inside it).
  • NSIS uses $PROGRAMDATA\iran-split\staging; in-app Install and WindowsPaths.generation_staging_dir use the same root.
  • The elevated installer ignores a mismatched --staging-dir and grants Builtin\Users modify via icacls.
  • Version 4.3.0.

A 4.3 NSIS install (or in-app Helper Install) rewrites helper.toml and the ACL. Then Connect stages where the helper already looks.

Open in Web Open in Cursor 

cursoragent and others added 2 commits August 18, 2026 16:29
NSIS perMachine expands $LOCALAPPDATA to C:\ProgramData, so 4.2 helper.toml
looked in C:\ProgramData\biflow\runtime\generations while the desktop wrote
under the user's LocalAppData. Stage at C:\ProgramData\iran-split\staging,
grant Users modify, and bump to 4.3.0.

Co-authored-by: Dariush vesal <dariush.vesal@gmail.com>
Keep helper source contracts free of LOCALAPPDATA literals and match the
ProgramData staging path in the Tauri installer tests.

Co-authored-by: Dariush vesal <dariush.vesal@gmail.com>
@devlifeX
devlifeX marked this pull request as ready for review August 18, 2026 16:37
@cursor
cursor Bot merged commit 777e9a9 into main Aug 18, 2026
4 of 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