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
@Aaronontheweb Aaronontheweb mentioned this pull request Aug 26, 2026
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