fix(web): preserve running state for queued prompts - #364
Merged
tt-a1i merged 4 commits intoSep 5, 2026
Conversation
Signed-off-by: seekskyworld <djh1813553759@gmail.com>
This was referenced Sep 4, 2026
tt-a1i
added a commit
to 6iKUN6/openpi
that referenced
this pull request
Sep 5, 2026
prompt_accepted and the HTTP receipt were resetting an active turn to preparing. Preserve running unless the command already settled, matching the openpi-dev#364 live-phase rule without copying legacy app.js.
tt-a1i
added a commit
that referenced
this pull request
Sep 7, 2026
* docs(web): define React migration MVP Record the frozen PR #352 behavior baseline, Pi-native ownership boundaries, selected frontend stack, static delivery contract, and validation matrix for the local migration experiment. * docs(web): define development port conflict handling Document default port fallback, strict explicit ports, fast startup diagnostics, and TUI error projection for the local React validation branch. * feat(web): migrate workbench UI to React - replace the imperative browser UI with typed React and Zustand projections - add Vite HMR, production asset builds, and bounded port conflict handling - cover the Web store, packaged host, responsive browser flows, and artifact drift in CI * fix(packaging): stabilize Git source installs - disable npm peer resolution for Pi-provided host packages - guard the Git install contract and record the production smoke rationale * fix(web): address React MVP review feedback * fix(ci): provide Pi host peers to source smoke * docs(web): record React MVP review validation * fix(web): keep backend readiness deadline alive Use an explicitly referenced timer for each readiness probe so Node 22 does not let the test process or launcher exit while the fetch is still pending. Clear the timer after every probe. * docs(web): record Node 22 readiness validation Advance the design evidence boundary to the readiness timer fix and record the focused Node 22 and Node 24 regression runs. * docs(web): record green full test run Replace the superseded load-sensitive test note with the successful Node 24 full-suite result observed after the readiness fix. * fix(web): install shutdown handlers before ready output Ensure the CLI cannot advertise a ready Web Host before its SIGINT and SIGTERM cleanup handlers are installed. * docs(web): record CLI shutdown race validation Advance the implementation boundary and distinguish focused CLI evidence from unrelated full-suite timing failures. * fix(web): keep running phase for queued follow-up prompts prompt_accepted and the HTTP receipt were resetting an active turn to preparing. Preserve running unless the command already settled, matching the #364 live-phase rule without copying legacy app.js. --------- Co-authored-by: tt-a1i <tt-a1i@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Closes the state-regression slice of #363. When a follow-up prompt was accepted during an active turn, the Web UI unconditionally changed the status from running back to preparing.
Value
Long workflow/subagent turns retain an accurate running indicator while queued prompts wait for admission.
Approach
Centralize prompt-accepted state handling and preserve
runningwhen already established; only new work enterspreparing, and settled prompts return to idle.Validation
node --test --experimental-strip-types tests/web/app-render.test.tsgit diff --checkImpact