Repository navigation
Add --detach and remote status for unattended remote runs - #881
Open
rtibblesbot wants to merge 2 commits into
Open
rtibblesbot wants to merge 2 commits into
rtibblesbot wants to merge 2 commits into
Conversation
- `--detach` starts the run (or finds it live) and exits 0 without attaching. - `remote status` prints run state, Studio link and log paths; exits 0 ok, 3 running, 4 failed, 5 no run. - Link is read from the chef log past the byte offset recorded at chef start. Closes learningequality#880 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
rtibblesbot
force-pushed
the
issue-880-5d25ea
branch
from
October 27, 2026 23:16
c170dd8 to
f85a68b
Compare
rtibblesbot
marked this pull request as ready for review
October 27, 2026 23:17
rtibbles
requested changes
Oct 28, 2026
rtibbles
left a comment
Member
There was a problem hiding this comment.
Should fail fast at input rather than blocking.
Contributor
Author
|
Done in 5d5f357a: a
|
Contributor
Author
|
The fix is now d7a1944b (replaces 5d5f357a): |
- `--detach` runs the box chef with stdin on /dev/null, so input() raises EOFError. Co-Authored-By: Claude Opus 5.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
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.
Summary
--remote --detachstarts the run (or finds it live) and exits 0 without attaching; a non-TTY start used to exit 64 either way.python chef.py remote statusprints the run's state, start time, exit code, Studio link and log paths.remote statusexits 0 finished ok, 3 running, 4 finished failed, 5 no run./dev/null, so a token or Studio-version prompt fails the run instead of blocking;--detach --promptis refused.References
Closes #880.
Reviewer guidance
Suggested review: Full code review.
--detach --promptat parse time, and having a detached chef exit 1 with anEOFErrortraceback (in the session log only) at its firstinput(), is the fail-fast you asked for.--detachshould also get/dev/nullstdin; as on upstream, it blocks at a prompt until someone reattaches.test_detached_chef_fails_at_input_instead_of_blockingandtest_attached_chef_reads_typed_inputpass, since the agent's sandbox denies the pty they need under real tmux.~/.config/ricecooker/remote.toml, runpython chef.py --remote --detach < /dev/nullfrom a chef dir — it exits 0 and the chef runs on the box.python chef.py remote statuswhile the chef runs — it prints the start time and exits 3.remote statusafter a run that uploaded a channel finishes — it prints theDONE: Channel created atlink and both log paths, and exits 0.--remote --detachwith no Studio token on the laptop or the box —remote statusreports it finished failed (exit 4).Open risk:
getpassreads the tmux pane, not stdin, so a detached run still blocks there until attached.AI usage
Used Claude Code to plan, implement and test the change against a local box. Verified with the remote test suite, prek, and CI on a fork.
Written by rtibblesbot, an LLM-based coding agent.
🟡 Waiting for feedback
Last updated: 2026-10-28 02:46 UTC