Commit f45090f
fix: clean windows onboarding with reviewed fixes
Cherry-pick the windows onboarding flow from AnrokX PRs #796 and #791
with all review issues fixed:
Features ported:
- First-run dependency wizard (Git, Node, npm, gh, Claude, Codex)
- Setup action service with PowerShell command runner
- GitHub CLI device-flow login with code detection
- Git identity configuration (name/email)
- Diagnostics panel enhancements
- Toast notification CSS rewrite
- Workspace startup race condition fix (workspaceSystemReady gate)
- Desktop launch trace support in bootstrap page
Issues fixed from review:
- Remove dead `guidance` variable (~60 lines never rendered in template)
- Add retention limit (50 runs) to setupActionService Maps (memory leak)
- Disable updater plugin instead of empty pubkey (prevents silent update failures)
- Keep stderr on non-Windows platforms for debugging (was nulled everywhere)
- Document autoCreate behavioral change in workspaceManager
- Exclude binary artifacts (zip files) and patch script from cherry-pick
Based on work by AnrokX in PRs #796 and #791.
Co-Authored-By: AnrokX <anrokx@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>1 parent a9a6c7b commit f45090f
File tree
12 files changed
+6338
-2785
lines changed- client
- styles
- server
- src-tauri
- src
12 files changed
+6338
-2785
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
538 | 538 | | |
539 | 539 | | |
540 | 540 | | |
| 541 | + | |
| 542 | + | |
| 543 | + | |
| 544 | + | |
| 545 | + | |
| 546 | + | |
| 547 | + | |
| 548 | + | |
| 549 | + | |
| 550 | + | |
| 551 | + | |
541 | 552 | | |
542 | 553 | | |
0 commit comments