@mmerterden/multi-agent-pipeline 20.4.0 → 20.6.0

package/pipeline/scripts/update-check.sh

@@ -26,7 +26,8 @@
 # Usage:
 #   bash pipeline/scripts/update-check.sh                 # auto-detect local version
 #   bash pipeline/scripts/update-check.sh --local 10.8.0  # explicit local version
-#   bash pipeline/scripts/update-check.sh --ttl-hours 24  # cache window (default 24)
+#   bash pipeline/scripts/update-check.sh --ttl-hours 24  # cache window (default: prefs
+#                                                         # updateCheck.ttlHours, else 24)
 #   bash pipeline/scripts/update-check.sh --force         # ignore cache
 #   bash pipeline/scripts/update-check.sh --print-required  # emit the floor only
 #
@@ -41,7 +42,8 @@ set -euo pipefail
 PKG="@mmerterden/multi-agent-pipeline"
 REGISTRY_URL="https://registry.npmjs.org/${PKG/\//%2F}"
 CACHE_FILE="${UPDATE_CHECK_CACHE:-$HOME/.claude/logs/multi-agent/.update-check}"
-TTL_HOURS=24
+TTL_HOURS=""
+FAIL_TTL_SECONDS=3600
 LOCAL_VERSION=""
 FORCE=0
 PRINT_REQUIRED=0
@@ -49,37 +51,65 @@ PRINT_REQUIRED=0
 while [ $# -gt 0 ]; do
   case "$1" in
     --local) LOCAL_VERSION="${2:-}"; shift; [ $# -gt 0 ] && shift ;;
-    --ttl-hours) TTL_HOURS="${2:-24}"; shift; [ $# -gt 0 ] && shift ;;
+    --ttl-hours) TTL_HOURS="${2:-}"; shift; [ $# -gt 0 ] && shift ;;
     --force) FORCE=1; shift ;;
     --print-required) PRINT_REQUIRED=1; shift ;;
     *) shift ;;
   esac
 done
 
-# Local version: explicit arg, else read from the pipeline repo clone.
+# Local version: explicit arg, else the marker the installer stamps. The
+# marker is what is installed and what /multi-agent:update rewrites, so it is
+# read before any repo clone; a clone is only a fallback for a machine that
+# runs the pipeline from source without installing it.
 if [ -z "$LOCAL_VERSION" ]; then
-  for candidate in "$HOME/multi-agent-pipeline" "$HOME/dev/multi-agent-pipeline" "$HOME/projects/multi-agent-pipeline"; do
-    if [ -f "$candidate/package.json" ]; then
-      LOCAL_VERSION=$(node -p "require('$candidate/package.json').version" 2>/dev/null || true)
+  for marker in "$HOME/.claude/.pipeline-version" "$HOME/.copilot/.pipeline-version" \
+                "$HOME/.codex/.pipeline-version"; do
+    if [ -f "$marker" ]; then
+      LOCAL_VERSION=$(head -1 "$marker" 2>/dev/null | tr -d '[:space:]')
       [ -n "$LOCAL_VERSION" ] && break
     fi
   done
 fi
-# npx-only installs have no repo clone; the installer stamps the version here.
 if [ -z "$LOCAL_VERSION" ]; then
-  for marker in "$HOME/.claude/.pipeline-version" "$HOME/.copilot/.pipeline-version" \
-                "$HOME/.codex/.pipeline-version"; do
-    if [ -f "$marker" ]; then
-      LOCAL_VERSION=$(head -1 "$marker" 2>/dev/null | tr -d '[:space:]')
+  for candidate in "$HOME/multi-agent-pipeline" "$HOME/dev/multi-agent-pipeline" "$HOME/projects/multi-agent-pipeline"; do
+    if [ -f "$candidate/package.json" ]; then
+      LOCAL_VERSION=$(node -p "require('$candidate/package.json').version" 2>/dev/null || true)
       [ -n "$LOCAL_VERSION" ] && break
     fi
   done
 fi
 [ -z "$LOCAL_VERSION" ] && exit 0  # cannot determine local version -> silent no-op
 
-# Highest of two dotted versions; ties resolve to the second argument.
-highest_of() {
-  printf '%s\n%s\n' "$1" "$2" | sort -t. -k1,1n -k2,2n -k3,3n | tail -1
+# Cache lifetime: --ttl-hours, else prefs global.updateCheck.ttlHours, else 24.
+# Anything that is not a whole number falls back to 24.
+if [ -z "$TTL_HOURS" ]; then
+  PREFS_FILE="${MULTI_AGENT_PREFS:-$HOME/.claude/multi-agent-preferences.json}"
+  if [ -f "$PREFS_FILE" ] && command -v jq >/dev/null 2>&1; then
+    TTL_HOURS=$(jq -r '.global.updateCheck.ttlHours // empty' "$PREFS_FILE" 2>/dev/null || true)
+  fi
+fi
+case "$TTL_HOURS" in (""|*[!0-9]*) TTL_HOURS=24 ;; esac
+
+# 0 when $1 sorts strictly below $2. Semver order: numeric major.minor.patch,
+# and a prerelease (1.2.3-beta) below its release (1.2.3). A leading v is ignored.
+version_lt() {
+  local a="${1#v}" b="${2#v}" ac ap bc bp i x y
+  ac="${a%%-*}"; bc="${b%%-*}"
+  ap=""; bp=""
+  [ "$ac" != "$a" ] && ap="${a#*-}"
+  [ "$bc" != "$b" ] && bp="${b#*-}"
+  for i in 1 2 3; do
+    x=$(printf '%s' "$ac" | cut -d. -f"$i"); y=$(printf '%s' "$bc" | cut -d. -f"$i")
+    case "$x" in (""|*[!0-9]*) x=0 ;; esac
+    case "$y" in (""|*[!0-9]*) y=0 ;; esac
+    [ "$x" -lt "$y" ] && return 0
+    [ "$x" -gt "$y" ] && return 1
+  done
+  [ -n "$ap" ] && [ -z "$bp" ] && return 0
+  [ -z "$ap" ] && [ -n "$bp" ] && return 1
+  [ -n "$ap" ] && [ "$(printf '%s\n%s\n' "$ap" "$bp" | sort | head -1)" = "$ap" ] && [ "$ap" != "$bp" ] && return 0
+  return 1
 }
 
 now=$(date +%s)
@@ -92,6 +122,8 @@ if [ "$FORCE" -eq 0 ] && [ -f "$CACHE_FILE" ]; then
   cached_latest=$(cut -d'|' -f2 "$CACHE_FILE" 2>/dev/null || echo "")
   cached_required=$(cut -d'|' -f3 "$CACHE_FILE" 2>/dev/null || echo "")
   case "$cached_epoch" in (*[!0-9]*|"") cached_epoch=0 ;; esac
+  # A stamp in the future is a clock or copy mistake, not a fresh answer.
+  [ "$cached_epoch" -gt "$now" ] && cached_epoch=0
   if [ $((now - cached_epoch)) -lt $((TTL_HOURS * 3600)) ] && [ -n "$cached_latest" ]; then
     latest="$cached_latest"
     required="$cached_required"
@@ -100,6 +132,16 @@ fi
 
 # Stale or missing cache -> one bounded registry call (silent on any failure).
 # The abbreviated packument carries dist-tags at a quarter of the full document.
+# A failed read is remembered for FAIL_TTL_SECONDS in "<cache>.fail", so an
+# offline machine does not wait out the timeout on every call.
+if [ -z "$latest" ] && [ "$FORCE" -eq 0 ] && [ -f "$CACHE_FILE.fail" ]; then
+  failed_at=$(head -1 "$CACHE_FILE.fail" 2>/dev/null | tr -d '[:space:]')
+  case "$failed_at" in (*[!0-9]*|"") failed_at=0 ;; esac
+  if [ "$failed_at" -le "$now" ] && [ $((now - failed_at)) -lt "$FAIL_TTL_SECONDS" ]; then
+    exit 0
+  fi
+fi
+
 if [ -z "$latest" ]; then
   tags=$(curl -sm 3 -H 'Accept: application/vnd.npm.install-v1+json' "$REGISTRY_URL" 2>/dev/null \
     | { if command -v jq >/dev/null 2>&1; then
@@ -110,14 +152,25 @@ if [ -z "$latest" ]; then
   latest="${tags%%|*}"
   required="${tags#*|}"
   [ "$required" = "$tags" ] && required=""
-  [ -z "$latest" ] && exit 0
   mkdir -p "$(dirname "$CACHE_FILE")" 2>/dev/null || exit 0
-  printf '%s|%s|%s\n' "$now" "$latest" "$required" > "$CACHE_FILE" 2>/dev/null || true
+  if [ -z "$latest" ]; then
+    printf '%s\n' "$now" > "$CACHE_FILE.fail" 2>/dev/null || true
+    exit 0
+  fi
+  rm -f "$CACHE_FILE.fail" 2>/dev/null || true
+  # Written through a temp file and a rename, so a concurrent reader never
+  # sees a half-written line.
+  tmp_cache=$(mktemp "$CACHE_FILE.XXXXXX" 2>/dev/null) || exit 0
+  if printf '%s|%s|%s\n' "$now" "$latest" "$required" > "$tmp_cache" 2>/dev/null; then
+    mv -f "$tmp_cache" "$CACHE_FILE" 2>/dev/null || rm -f "$tmp_cache"
+  else
+    rm -f "$tmp_cache"
+  fi
 fi
 
 # A floor above latest is a publisher mistake (the `required` tag was moved to a
 # version that `latest` no longer covers). Clamp rather than brick every user.
-if [ -n "$required" ] && [ "$(highest_of "$required" "$latest")" != "$latest" ]; then
+if [ -n "$required" ] && version_lt "$latest" "$required"; then
   required="$latest"
 fi
 
@@ -127,8 +180,7 @@ if [ "$PRINT_REQUIRED" -eq 1 ]; then
 fi
 
 # Forced update: local sorts strictly BELOW the required floor.
-if [ -n "$required" ] && [ "$LOCAL_VERSION" != "$required" ] \
-   && [ "$(highest_of "$LOCAL_VERSION" "$required")" = "$required" ]; then
+if [ -n "$required" ] && version_lt "$LOCAL_VERSION" "$required"; then
   printf '%s|%s|force\n' "$LOCAL_VERSION" "$latest"
   exit 0
 fi
@@ -137,7 +189,7 @@ fi
 
 # Update available only when latest sorts strictly ABOVE local (a dev machine
 # running ahead of the registry must not see an "update" prompt).
-if [ "$(highest_of "$LOCAL_VERSION" "$latest")" = "$latest" ]; then
+if version_lt "$LOCAL_VERSION" "$latest"; then
   printf '%s|%s\n' "$LOCAL_VERSION" "$latest"
 fi
 exit 0

package/pipeline/scripts/website-deploy-commit.sh

@@ -7,8 +7,9 @@
 # refuses to build a commit whose author is not a contributor on the project. The
 # push still succeeds, the deployment is created and sits at UNKNOWN with a 0ms
 # build (API `readyState: BLOCKED`), and the live site keeps serving the previous
-# version. v16.4.0 and v16.5.0 were both pushed that way and neither was built,
-# while the sync reported the website as synced both times.
+# version. A successful push is therefore not evidence of a deploy, so the script
+# pins the author before committing and reads the deployment state back after
+# pushing instead of reporting the website as synced on the push alone.
 #
 # Usage:
 #   website-deploy-commit.sh <identity-name> <identity-email> <version> [repo-dir]

package/pipeline/skills/.skill-manifest.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": "1.0.0",
-  "generatedAt": "2026-09-25T15:14:42Z",
+  "generatedAt": "2026-09-26T13:39:28Z",
   "skillCount": 218,
   "entries": [
     {
@@ -29,7 +29,7 @@
     },
     {
       "path": "shared/core/multi-agent-autopilot-on/SKILL.md",
-      "sha256": "ca671ffe46809e3b0c825a6b5c19e8ea00e529874531eb480a0987428366d2f4"
+      "sha256": "d4f23f6f7bb493a4efa2cfa252fab50b22b6506aeaf31985db4498b5f010c8c8"
     },
     {
       "path": "shared/core/multi-agent-autopilot-status/SKILL.md",

package/pipeline/skills/shared/core/multi-agent-autopilot-on/SKILL.md

@@ -74,7 +74,7 @@ The same name works on both sides and both mechanisms already exist:
 
 | Source | Query | Extra condition |
 |---|---|---|
-| GitHub | `gh issue list --repo <r> --label <label> --state open` | none - the label is the whole gate |
+| GitHub | `gh issue list --repo <r> --label <label> --assignee @me --state open` | **assigned to you** (`githubAssignee`, `"*"` for any), so a label an issue form applies is not enough |
 | Jira | `assignee = currentUser() AND labels = "<label>" AND resolution = EMPTY AND status not in (Done, Closed, Cancelled)` | **assigned to you**, so a label somebody else adds is not enough |
 
 Two Jira caveats worth saying out loud when Jira is selected: labels are one