@plot-pm/board 0.16.2 → 0.16.3

package/plot-state-receipt.sh

@@ -165,11 +165,17 @@ _action_receipt_dir() {
 # adapter — a script name in a controller is a boundary crossing no spawn
 # counter can see. So the shared filename is the action word, and mapping a
 # script back to it happens HERE, on the side that reads command lines.
+#
+# `plot-deliver.sh` maps to `deliver` here, same as always — `release` is a
+# SEPARATE action sharing that script, and only `plot-controller-gate.sh` can
+# tell the two calls apart (by whether `--release` rode along), so it passes
+# the action word directly rather than the script name for that one case.
 _action_of() { # $1=script name or action word → the action, or nothing
   case "${1##*/}" in
     plot-dispatch.sh|dispatch) printf 'dispatch\n' ;;
     plot-approve.sh|approve)   printf 'approve\n' ;;
     plot-deliver.sh|deliver)   printf 'deliver\n' ;;
+    release)                   printf 'release\n' ;;
   esac
 }
 
@@ -231,7 +237,7 @@ if [ "${BASH_SOURCE[0]}" = "${0}" ]; then
     action_reason="$*"
     if [ -z "$action_script" ] || [ -z "$action_subject" ] || [ -z "$action_reason" ]; then
       echo "plot-state-receipt: --unowned-action needs an action, a subject and a reason." >&2
-      echo "  usage: plot-state-receipt.sh --unowned-action <dispatch|approve|deliver> <slug> <reason>" >&2
+      echo "  usage: plot-state-receipt.sh --unowned-action <dispatch|approve|deliver|release> <slug> <reason>" >&2
       echo "" >&2
       echo "  The reason is required because each use names a routing gap, and a gap" >&2
       echo "  nobody wrote down is one nobody closes. Where the board IS running, the" >&2
@@ -240,13 +246,17 @@ if [ "${BASH_SOURCE[0]}" = "${0}" ]; then
     fi
     # The action word, or the script name — an operator reaching for this has
     # just read a refusal naming a script, so both spellings are accepted.
+    # `release` has no script of its own (`plot-deliver.sh --release` shares
+    # `plot-deliver.sh`, which already names `deliver`), so only the bare word
+    # is accepted for it.
     case "$action_script" in
       dispatch|plot-dispatch.sh) action_script="dispatch" ;;
       approve|plot-approve.sh)   action_script="approve" ;;
       deliver|plot-deliver.sh)   action_script="deliver" ;;
+      release)                   action_script="release" ;;
       *)
         echo "plot-state-receipt: '$action_script' is not a controller-owned action." >&2
-        echo "  The three are: dispatch, approve, deliver." >&2
+        echo "  The four are: dispatch, approve, deliver, release." >&2
         exit 2
         ;;
     esac

package/plot-tmp.sh

@@ -59,22 +59,50 @@ _plot_tmp_register() {
 # unlisted. While `__plot_tmp_busy` is set, the signal handler records the
 # signal in `__plot_tmp_pending` and returns; `_plot_tmp_create` handles it once
 # the path is registered.
+#
+# A SUBSHELL HOLDS THE SIGNAL ITSELF. bash resets caught traps in `$(…)` and
+# `( … )`, so there the handler above is not installed and a TERM between
+# `mktemp` and the registration ends the subshell with the path unlisted. The
+# owner's cleanup then runs and cannot remove it. `plot-host.sh`'s
+# `pr_list_call` creates `plot-host-prlist-err.*` this way, inside
+# `_raw="$(pr_list_call …)"`, and CI run 37052327801 left one when the board
+# test teardown sent its process group TERM. Inside a subshell
+# (`BASH_SUBSHELL` > 0; bash 3.2 has no `BASHPID`) the creation installs
+# handlers that only record the signal, and after the registration it restores
+# the defaults and exits with 128 + the signal's number. It does not run the
+# cleanup: the registry belongs to the owner, which received the same group
+# signal and removes the path when the subshell has ended.
 __plot_tmp_busy=''
 __plot_tmp_pending=''
 
 # _plot_tmp_create VAR prefix [-d] — create, register, assign to VAR.
 _plot_tmp_create() {
-  local __plot_tmp_new __plot_tmp_rc
+  local __plot_tmp_new __plot_tmp_rc __plot_tmp_sub=''
   __plot_tmp_busy=1
+  if [ "${BASH_SUBSHELL:-0}" -gt 0 ]; then
+    __plot_tmp_sub=1
+    trap '__plot_tmp_pending="INT 2"' INT
+    trap '__plot_tmp_pending="TERM 15"' TERM
+    trap '__plot_tmp_pending="PIPE 13"' PIPE
+  fi
   if [ "${3:-}" = -d ]; then
     __plot_tmp_new=$(mktemp -d "${TMPDIR:-/tmp}/plot-$2.XXXXXX")
   else
     __plot_tmp_new=$(mktemp "${TMPDIR:-/tmp}/plot-$2.XXXXXX")
   fi
   __plot_tmp_rc=$?
-  [ "$__plot_tmp_rc" -eq 0 ] && _plot_tmp_register p "$__plot_tmp_new"
+  # THE PATH IS REGISTERED WHEN IT EXISTS, whatever the status says. bash 3.2
+  # reports 143 for `$(mktemp …)` when a trap ran during it, although `mktemp`
+  # created the path and printed it; a status-only test leaves that path
+  # unlisted.
+  if [ -n "$__plot_tmp_new" ] && { [ -e "$__plot_tmp_new" ] || [ "$__plot_tmp_rc" -eq 0 ]; }; then
+    _plot_tmp_register p "$__plot_tmp_new"
+  fi
   __plot_tmp_busy=''
-  if [ -n "$__plot_tmp_pending" ]; then
+  if [ -n "$__plot_tmp_sub" ]; then
+    trap - INT TERM PIPE
+    [ -z "$__plot_tmp_pending" ] || exit $((128 + ${__plot_tmp_pending#* }))
+  elif [ -n "$__plot_tmp_pending" ]; then
     # shellcheck disable=SC2086 # "<name> <number>", split on purpose
     _plot_tmp_on_signal $__plot_tmp_pending
   fi

package/plot-transcript-quiet.sh

@@ -1,6 +1,8 @@
 #!/usr/bin/env bash
 # The ONE answer to "how long has this worktree's agent been quiet?" — sourced,
-# not run, by `plot-worker-monitor.sh` and `plot-worker-loop.sh`.
+# not run, by `plot-worker-loop.sh` for its own watcher (`plot-worker-state.sh`'s
+# `plot_worker_idle_watch_pass`) and for its own ending message. Sourced by the
+# now-deleted `plot-worker-monitor.sh` until `bug/the-loop-reports-idle`.
 #
 # It reads the AGENT rather than the machine. A `claude -p` session appends a
 # timestamped line to its transcript for every model turn, tool call and tool