Skip to content

fix(setup): verify WSL before Local AI downloads - #1188

Closed
RomneyDa wants to merge 1 commit into
stack/pr-1178-local-aifrom
fix/local-ai-virtualization-preflight
Closed

fix(setup): verify WSL before Local AI downloads#1188
RomneyDa wants to merge 1 commit into
stack/pr-1178-local-aifrom
fix/local-ai-virtualization-preflight

Conversation

@RomneyDa

Copy link
Copy Markdown
Member

Summary

  • move the existing WSL install-and-verify stage ahead of Local AI runtime and model acquisition
  • fail before the 23 GB model download when WSL cannot start because hardware virtualization or Virtual Machine Platform is unavailable
  • keep the GPU compatibility check first, and align onboarding progress rows with the new execution order

Stacked on #1178 at head 145885614903b9c622197d542d7910eec5904d7c via the exact snapshot base branch stack/pr-1178-local-ai.

Validation

  • git diff --check - passed
  • .agents/skills/autoreview/scripts/autoreview --mode local - passed, no accepted/actionable findings
  • focused dotnet test - blocked locally because dotnet is unavailable on the macOS controller
  • required ./build.ps1, shared tests, and tray tests - blocked locally because no native Windows/Crabbox host is configured; PR Windows CI is the validation path

Real behavior proof

Not verified / blocked locally: this path requires a Windows host with WSL absent or hardware virtualization disabled. The pipeline ordering tests assert that EnsureWslPlatformStep precedes both Local AI download steps, and the progress-page contract asserts the same visible phase order.

Rubber-duck review

The reviewer found that the progress UI still grouped early WSL verification with later WSL networking. The implementation now splits those phases and removes the stale “Verify Local AI before WSL setup” claim.

@clawsweeper

clawsweeper Bot commented Aug 21, 2026

Copy link
Copy Markdown

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

@RomneyDa RomneyDa added the status: 🚢 actively landing A maintainer or agent is actively driving this item through implementation, validation, or merge. label Aug 21, 2026
@RomneyDa

Copy link
Copy Markdown
Member Author

Replaced by a direct PR into the contributor fork: joelagnel#4

@RomneyDa RomneyDa closed this Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: 🚢 actively landing A maintainer or agent is actively driving this item through implementation, validation, or merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant