Skip to content

fix(tui): auto-start init health checks#1454

Merged
Aaronontheweb merged 2 commits into
netclaw-dev:devfrom
Aaronontheweb:healthcheck-autostart
Jun 19, 2026
Merged

fix(tui): auto-start init health checks#1454
Aaronontheweb merged 2 commits into
netclaw-dev:devfrom
Aaronontheweb:healthcheck-autostart

Conversation

@Aaronontheweb

Copy link
Copy Markdown
Collaborator

Summary

  • start the final init health check automatically when the step opens
  • render pending health-check rows with the shared animated spinner
  • update the native init-wizard smoke tape and onboarding spec for the new flow

Verification

  • dotnet test src/Netclaw.Cli.Tests/Netclaw.Cli.Tests.csproj --filter FullyQualifiedName~InitWizardPageTests
  • dotnet test src/Netclaw.Cli.Tests/Netclaw.Cli.Tests.csproj --no-restore
  • ./scripts/smoke/run-smoke.sh init-wizard
  • dotnet slopwatch analyze
  • pwsh ./scripts/Add-FileHeaders.ps1 -Verify
  • git diff --check
  • dotnet test --no-restore

@Aaronontheweb Aaronontheweb added tui Terminal UI (Termina) issues UX/DX UI / UX / DX friction issue or user-facing annoyances. labels Jun 19, 2026

@Aaronontheweb Aaronontheweb left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

var (icon, color) = item.Passed switch
if (item.Passed is null)
{
lines.Add(SpinnerViews.Labeled(item.Label, Color.Yellow));

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add spinners to each of the health check stages

{
if (!healthStep.IsRunning.Value && !healthStep.IsComplete.Value)
healthStep.StartWithOrchestrator(_orchestrator);
StartHealthCheckIfReady(healthStep);

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just automatically run the health checks

@Aaronontheweb
Aaronontheweb merged commit 8bcef59 into netclaw-dev:dev Jun 19, 2026
15 checks passed
@Aaronontheweb
Aaronontheweb deleted the healthcheck-autostart branch June 19, 2026 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tui Terminal UI (Termina) issues UX/DX UI / UX / DX friction issue or user-facing annoyances.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant