Feat/app update feedback ux b08b10 - #9
Merged
AdamXweb merged 4 commits intoAug 4, 2026
Conversation
…verhaul Graceful stops, working Docker checks, per-app targeting, interactive menu, and shellcheck CI
Offer Docker install when missing; fix install docs
Arrow-key menu navigation (fzf), system-update in menu, easy back
Leaving an app in auto mode and running `update` printed "Pulling images for 30 app(s), up to 3 at a time" and then went silent for the whole download - the spinner covered the phase with a single static label, so there was no way to tell which apps were downloading or how far along the run was. The pull phase now reports as it goes: each pull records an event when it finishes, the foreground prints a line per app as it lands or fails, and the spinner names the apps currently in flight and counts them off ([4/30]). The phase closes with a total. Without a terminal (cron, pipes) the same per-app lines are written as each pull finishes, instead of one label and silence until the end. Also replace the batch orchestrator with a sliding window. It waited for all three pulls in a batch before starting the next three, so one slow image idled the other slots - now visible in the new status line. Slots are tracked with the marker files the display already needed (`wait -n` would do this directly but isn't POSIX) and the marker is created before the fork so the accounting can't race. On an 8-app test workload the phase went 14s to 8s. Two fixes alongside: - PARALLEL_PULLS of 0, negative or non-numeric falls back to 3. It hung the orchestrator in a loop with no slot it could ever fill. - Drop pull_images(), the pre-parallel sequential pull, unreferenced since parallel pulls landed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.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.
No description provided.