Skip to content

Run long commands so that waiting ends with the command (for discussion) #56

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
anakanemison wants to merge 1 commit into from
Open

Run long commands so that waiting ends with the command (for discussion) #56

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -204,6 +204,20 @@ the reference build or bazel.
> repo root** (the Bazel workspace). Only the `$SKILL_DIR/scripts/*.py` helpers
> live elsewhere.
>
> **Long commands.** Run a build or test suite in the foreground, with a
> `timeout` long enough for it, when your command tool allows that long (its
> maximum timeout is in its description; 10 minutes by default in Claude Code):
> it returns the moment the command ends. If it may
> take longer, start it in the background as
> `cmd > /tmp/cmd.log 2>&1 & echo $! > /tmp/cmd.pid` (outside the workspace, so
> the log is not migrated) and wait in slices that end as soon as it does,
> repeating until it has ended:
> `for i in $(seq 36); do kill -0 "$(cat /tmp/cmd.pid)" 2>/dev/null || break; sleep 15; done; tail -5 /tmp/cmd.log`.
> Never wait with `pgrep -f <name>`: the waiting command's own command line
> contains the name, so it always finds itself. Don't sleep a fixed long time
> on a guess, either. A second Bazel command in the same workspace waits for
> the first. Run built programs with `</dev/null` and a `timeout`.
>
> **`<repo_root>` placeholder:** the project's source/workspace root. It MUST be
> **identical** in the reference extractor (`extract_<frontend>.py`) and
> `extract_bazel.py` calls: both key translation units by their path relative
Expand Down
Loading