Skip to content

fix: prevent setup-status hang with timeout/retry + retry UI#457

Merged
0xNyk merged 1 commit intomainfrom
fix/setup-status-timeout-retry
Mar 19, 2026
Merged

fix: prevent setup-status hang with timeout/retry + retry UI#457
0xNyk merged 1 commit intomainfrom
fix/setup-status-timeout-retry

Conversation

@0xNyk
Copy link
Member

@0xNyk 0xNyk commented Mar 19, 2026

Summary

Fixes the setup page hang scenario where users can get stuck on "Checking Setup Status..." indefinitely.

What changed

  • Added fetchSetupStatusWithRetry helper (src/lib/setup-status.ts) with:
    • bounded timeout per request
    • bounded retry attempts with small backoff
    • response shape validation
  • Updated src/app/setup/page.tsx to use timeout+retry helper for initial setup check
  • Replaced non-actionable fallback (return null) with a visible error state + Retry button
  • Added unit tests for retry helper (src/lib/__tests__/setup-status.test.ts)

Why

Issue #456 reported setup flow hanging at "Checking Setup Status...". The prior code used a bare fetch('/api/setup') with no timeout/retry, and on failed check could also fall into a blank UI path.

Test plan

  • pnpm exec tsc --noEmit
  • pnpm exec vitest run src/lib/__tests__/setup-status.test.ts
  • Manually reviewed setup error path now shows retry UI instead of empty screen

Fixes #456

- add bounded timeout+retry helper for /api/setup checks
- show actionable setup error state with Retry button
- avoid blank-screen fallback when setup status check fails
- add unit tests for retry helper

Fixes #456
@0xNyk 0xNyk added the bug Something isn't working label Mar 19, 2026
@github-actions github-actions bot added the screenshot-drift UI changed — README screenshots may need updating label Mar 19, 2026
@github-actions
Copy link

📸 Screenshot Drift Check

This PR modifies UI source files. Please verify whether the README screenshots need refreshing:

  • docs/mission-control-overview.png — main dashboard
  • docs/mission-control-agents.png — agents panel
  • docs/mission-control-memory-graph.png — memory graph
Changed UI files
src/app/setup/page.tsx

See docs/SCREENSHOT-GUIDE.md for instructions on capturing and optimising screenshots.

This comment is posted automatically and can be dismissed if no visual changes occurred.

@0xNyk 0xNyk merged commit 69e89a9 into main Mar 19, 2026
1 of 2 checks passed
@0xNyk 0xNyk deleted the fix/setup-status-timeout-retry branch March 19, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working screenshot-drift UI changed — README screenshots may need updating

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Checking Setup Status Hanging

1 participant