Repository navigation
lorenzo/start kw integration pr 6 - #204
Draft
lorenzoberts wants to merge 6 commits into
Draft
lorenzoberts wants to merge 6 commits into
lorenzoberts wants to merge 6 commits into
Conversation
This commit subscribes AppActor to KwHandle::watch_status once at startup and redraws when the snapshot changes, so a running job is visible without waiting for a keypress. The nav bar shows a compact running indicator; restore availability and cancelled log paths are published on the snapshot so the UI does not have to infer them. This commit is part of the kw integration's step 6. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit adds a typed confirmation popup so Esc on the mailing list cannot tear down a running kw job by accident. Wait is the highlighted default; cancel-and-quit requests KwHandle::cancel and then leaves, including the harmless race where the job already finished. This commit is part of the kw integration's step 6. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit adds an explicit KwOps build dashboard from Patchset Details so a running job can be started, cancelled, and restored without blocking the TUI. The branch field is prefilled from HEAD, and extras that would hang, wipe the tree, or mutate git are stripped rather than forwarded to kw. This commit is part of the kw integration's step 6. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit reads a bounded tail of the kw job log so KwOps can show live build output without loading the whole file or blocking the TUI. The redraw tick runs only while KwOps is visible and a job is running, and a missing log is treated as "waiting" rather than an error. This commit is part of the kw integration's step 6. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit keeps the visible log on the newest output so a failed build's error is on screen, and moves log-tail and readiness work off the actor task so they cannot stall the TUI. Start no longer races a second keypress or stays stuck on "starting" when the status watch is missing: accept applies the current status immediately, and keyboard-only mode polls while a job is in flight without rereading the log off-screen. This commit completes the kw integration's step 6. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit drops the Step 7 boot-once note and "build-only" wording from comments that describe the screen as it exists in this change. This commit is part of the kw integration's step 6. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
lorenzoberts
added this pull request to stack #200
September 20, 2026 14:02
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
1 participant
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.
Stack created with GitHub Stacks CLI • Give Feedback 💬