Skip to content

fix(runtime): keep readiness diagnostic - #984

Merged
luokerenx4 merged 1 commit into
devfrom
codex/runtime-readiness-regressions
Aug 4, 2026
Merged

fix(runtime): keep readiness diagnostic#984
luokerenx4 merged 1 commit into
devfrom
codex/runtime-readiness-regressions

Conversation

@luokerenx4

@luokerenx4 luokerenx4 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Stop Chat and Workspace Manager from synchronously probing runtime readiness before an ordinary launch.
  • Preserve the hard structural gate only for runtimes that truly require a Workspace credential.
  • Add route, API, hook, and UI regressions for native login, empty or unreadable OpenAlice vault state, failed readiness snapshots, and retryable Session resume errors.
  • Document readiness as a diagnostic/background health signal rather than a launch gate.

Root cause

The credential requirement was corrected in PR #983, but the shared launch surfaces still called the active runtime probe before every first send. A stale, timed-out, or provider-failed readiness result could therefore block an otherwise valid native launch.

Validation

  • npx tsc --noEmit
  • cd ui && npx tsc -b
  • targeted readiness, route, API, Chat, Manager, and resume tests: 89 passed
  • pnpm test: 3,927 passed; 9 skipped
  • real pnpm dev browser check with Pi cached as failed/429: Send remained available and no probe or test Session was created
  • dev frontend and backend restarted and healthy

@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openalice-demo Ready Ready Preview Aug 4, 2026 5:34pm

Request Review

@luokerenx4 luokerenx4 added theme:reliability Failure recovery, retries, loading, or resilience area:workspace Workspace, Session, templates, or sidebar lifecycle review:deep Requires deliberate human review before merge labels Aug 4, 2026
@luokerenx4
luokerenx4 merged commit 54751c8 into dev Aug 4, 2026
12 checks passed
@luokerenx4
luokerenx4 deleted the codex/runtime-readiness-regressions branch August 4, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:workspace Workspace, Session, templates, or sidebar lifecycle review:deep Requires deliberate human review before merge theme:reliability Failure recovery, retries, loading, or resilience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant