Skip to content

fix(desktop): atomically publish staged providers - #30

Merged
BrianInAz merged 2 commits into
mainfrom
fix/desktop-provider-staging-atomic
Aug 11, 2026
Merged

fix(desktop): atomically publish staged providers#30
BrianInAz merged 2 commits into
mainfrom
fix/desktop-provider-staging-atomic

Conversation

@BrianInAz

Copy link
Copy Markdown
Owner

Summary

  • copy provider bytes into an unpublished candidate path
  • sync, protect, and close the candidate before atomically publishing the executable pathname
  • preserve immutable-byte and Windows sharing contracts while eliminating Linux ETXTBSY execution races

TDD evidence

  • Red: the new publication contract failed to compile until candidate-copy and atomic-publish helpers existed; PR fix(ci): bootstrap Sprig rolling release #29 job 93664535019 supplied the runtime ETXTBSY Red
  • Green: candidate publication contract and all 8 provider-focused tests pass
  • Green: exact former failing deployment test passes 20 consecutive runs
  • Green: cargo fmt and clippy with warnings denied
  • Green: full Desktop Tauri suite (2,116 passed, 15 expected ignored; 3 diagnostics passed)
  • Green: full just ci including 1,098 mobile tests with 1 expected skip

Tracking

No sleeps, retries, ignored tests, platform skips, workflow narrowing, or Sprig changes were introduced.

Signed-off-by: Brian Charbonneau <github@briancharbonneau.com>
@cursor

cursor Bot commented Aug 11, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

…staging-atomic

Signed-off-by: Brian Charbonneau <github@briancharbonneau.com>
@BrianInAz
BrianInAz merged commit 4ff4064 into main Aug 11, 2026
23 checks passed
@BrianInAz
BrianInAz deleted the fix/desktop-provider-staging-atomic branch August 11, 2026 07:45
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