@cohortapp/agent-sdk 2.3.0

package/scripts/local-triggers/generate-plists.sh

@@ -0,0 +1,432 @@
+#!/bin/bash
+# =============================================================================
+# generate-plists.sh — Generate launchd plist files from templates
+# =============================================================================
+#
+# Reads config/agent.ts to extract the agent's first name and generates
+# properly configured plist files for all scheduled triggers and the daemon.
+#
+# Architecture marker:
+#   MAESTRO_PLIST_ARCH=cadence-bus
+#
+# This generator emits the *cadence-bus* plist architecture (introduced in
+# maestro 1.8). Scheduled cadence ticks invoke a lightweight Node enqueue
+# script (scripts/cadence/enqueue-cadence-tick.mjs) that drops an event onto
+# state/cadence-bus/. The persistent maestro-daemon.mjs consumes the bus and
+# decides — per cadence — whether to handle the tick inline or escalate to a
+# managed sub-session. Launchd NEVER spawns a fresh Claude Code session for
+# routine cadence ticks anymore.
+#
+# Doctor verifies the `MAESTRO_PLIST_ARCH=cadence-bus` marker is present in
+# installed plists, and flags any plist that still runs run-trigger.sh
+# directly as the legacy spawn-per-tick pattern.
+#
+# Usage:
+#   ./scripts/local-triggers/generate-plists.sh
+#
+# Called by: init-agent.sh (Step 5) and by `maestro upgrade` migration.
+# =============================================================================
+
+set -e
+
+# Plist architecture marker — embedded as a comment in every generated plist.
+MAESTRO_PLIST_ARCH="cadence-bus"
+MAESTRO_PLIST_ARCH_VERSION="1"
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+AGENT_DIR="$(cd "$SCRIPT_DIR/../.." && pwd)"
+PLIST_DIR="$SCRIPT_DIR/plists"
+
+# Resolve the agent's first name. Order of preference:
+#   1. config/agent.json  — single source of truth (SOT) introduced when
+#      maestro split identity out of TypeScript into JSON. Use jq if
+#      available, otherwise a small awk fallback.
+#   2. config/agent.ts    — legacy path; only works if firstName is
+#      defined inline (string literal), not as a type-only declaration
+#      like `firstName: string;`.
+#   3. Directory name     — last-resort fallback (e.g. ~/ravi-ai → ravi).
+AGENT_JSON="$AGENT_DIR/config/agent.json"
+AGENT_TS="$AGENT_DIR/config/agent.ts"
+AGENT_FIRST=""
+
+if [ -z "$AGENT_FIRST" ] && [ -f "$AGENT_JSON" ]; then
+    if command -v jq >/dev/null 2>&1; then
+        AGENT_FIRST=$(jq -r '.firstName // empty' "$AGENT_JSON" 2>/dev/null | tr '[:upper:]' '[:lower:]')
+    else
+        # Awk fallback — handle "firstName": "value" with optional whitespace
+        AGENT_FIRST=$(awk -F'"' '/"firstName"[[:space:]]*:/ { print tolower($4); exit }' "$AGENT_JSON")
+    fi
+fi
+
+if [ -z "$AGENT_FIRST" ] && [ -f "$AGENT_TS" ]; then
+    # Only accept a quoted inline value — `firstName: string;` (type
+    # declaration) must not match. The grep requires a quote on the line.
+    AGENT_FIRST=$(grep "firstName:[[:space:]]*['\"]" "$AGENT_TS" \
+        | head -1 \
+        | sed "s/.*firstName:[[:space:]]*['\"]//; s/['\"].*//" \
+        | tr '[:upper:]' '[:lower:]')
+fi
+
+# Fall back to directory name if config not set or UNCONFIGURED.
+if [ -z "$AGENT_FIRST" ] || [ "$AGENT_FIRST" = "unconfigured" ]; then
+    AGENT_FIRST=$(basename "$AGENT_DIR" | sed 's/-ai$//')
+fi
+
+AGENT_UPPER=$(echo "$AGENT_FIRST" | tr '[:lower:]' '[:upper:]')
+AGENT_USER=$(whoami)
+NODE_PATH=$(which node 2>/dev/null || echo "/opt/homebrew/bin/node")
+
+echo "[generate-plists] Agent: $AGENT_FIRST"
+echo "[generate-plists] Directory: $AGENT_DIR"
+echo "[generate-plists] User: $AGENT_USER"
+
+mkdir -p "$PLIST_DIR"
+
+# Clear old plists before generating — prevents stale plists from a previous
+# agent identity (e.g. an old AGENT_LAUNCHD_PREFIX) from being installed
+# alongside new ones
+rm -f "$PLIST_DIR"/*.plist 2>/dev/null
+
+# ── Helper: generate a single plist ──────────────────────────────────────────
+
+generate_plist() {
+    local LABEL="$1"
+    local PROGRAM_ARGS="$2"  # The command to run (array elements separated by |)
+    local SCHEDULE="$3"      # Optional: StartCalendarInterval dict content
+    local INTERVAL="$4"      # Optional: StartInterval in seconds
+    local KEEP_ALIVE="$5"    # Optional: true for daemon-style
+
+    local FILE="$PLIST_DIR/$LABEL.plist"
+
+    cat > "$FILE" << PLIST_HEADER
+<?xml version="1.0" encoding="UTF-8"?>
+<!-- maestro-plist-arch: $MAESTRO_PLIST_ARCH v$MAESTRO_PLIST_ARCH_VERSION -->
+<!-- generated-by: scripts/local-triggers/generate-plists.sh -->
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+    <key>Label</key>
+    <string>$LABEL</string>
+
+    <key>ProgramArguments</key>
+    <array>
+PLIST_HEADER
+
+    # Split program args by |
+    IFS='|' read -ra ARGS <<< "$PROGRAM_ARGS"
+    for arg in "${ARGS[@]}"; do
+        echo "        <string>$arg</string>" >> "$FILE"
+    done
+
+    cat >> "$FILE" << PLIST_MID
+    </array>
+
+    <key>WorkingDirectory</key>
+    <string>$AGENT_DIR</string>
+PLIST_MID
+
+    if [ "$KEEP_ALIVE" = "true" ]; then
+        cat >> "$FILE" << 'PLIST_KEEPALIVE'
+
+    <key>KeepAlive</key>
+    <dict>
+        <key>SuccessfulExit</key>
+        <false/>
+        <key>Crashed</key>
+        <true/>
+    </dict>
+
+    <key>RunAtLoad</key>
+    <true/>
+
+    <key>ThrottleInterval</key>
+    <integer>30</integer>
+PLIST_KEEPALIVE
+    fi
+
+    if [ -n "$SCHEDULE" ]; then
+        cat >> "$FILE" << PLIST_SCHED
+
+    <key>StartCalendarInterval</key>
+    <dict>
+$SCHEDULE
+    </dict>
+PLIST_SCHED
+    fi
+
+    if [ -n "$INTERVAL" ]; then
+        cat >> "$FILE" << PLIST_INTERVAL
+
+    <key>StartInterval</key>
+    <integer>$INTERVAL</integer>
+PLIST_INTERVAL
+    fi
+
+    # ── EnvironmentVariables block ──────────────────────────────────────────
+    # Exports for every spawned process:
+    #   PATH                  — homebrew + standard paths so node/python/etc are findable
+    #   CLAUDE_CODE_TMPDIR    — redirect Claude Code per-cwd temp to external SSD if mounted
+    #   AGENT_ROOT            — used by maestro's singleton lock and other helpers
+    cat >> "$FILE" << PLIST_ENV
+
+    <key>EnvironmentVariables</key>
+    <dict>
+        <key>PATH</key>
+        <string>/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin</string>
+        <key>HOME</key>
+        <string>$HOME</string>
+        <key>USER</key>
+        <string>$AGENT_USER</string>
+        <key>AGENT_ROOT</key>
+        <string>$AGENT_DIR</string>
+PLIST_ENV
+
+    if [ -d "/Volumes/4TB-SSD" ]; then
+        cat >> "$FILE" << PLIST_SSD
+        <key>CLAUDE_CODE_TMPDIR</key>
+        <string>/Volumes/4TB-SSD/maestro/${AGENT_FIRST}/claude-tmp</string>
+PLIST_SSD
+    fi
+
+    cat >> "$FILE" << 'PLIST_ENV_END'
+    </dict>
+PLIST_ENV_END
+
+    # Resolve symlinked log path to a real path so launchd can write to it.
+    # If logs/ is a symlink (e.g. when redirected to an external SSD), launchd's
+    # StandardErrorPath/StandardOutPath sometimes fail with EX_CONFIG (78).
+    LOG_REAL=$(cd -P "$AGENT_DIR/logs/daemon" 2>/dev/null && pwd -P || echo "$AGENT_DIR/logs/daemon")
+
+    cat >> "$FILE" << PLIST_FOOTER
+
+    <key>StandardOutPath</key>
+    <string>$LOG_REAL/launchd-stdout.log</string>
+
+    <key>StandardErrorPath</key>
+    <string>$LOG_REAL/launchd-stderr.log</string>
+</dict>
+</plist>
+PLIST_FOOTER
+
+    echo "  Generated: $LABEL"
+}
+
+# ── Helper: trigger plist (enqueues a cadence tick) ──────────────────────────
+#
+# Cadence-bus architecture: launchd invokes the lightweight Node enqueue
+# script (scripts/cadence/enqueue-cadence-tick.mjs) which drops a JSON event
+# onto state/cadence-bus/inbox/. The persistent maestro-daemon.mjs consumes
+# the bus and either handles the tick inline or escalates to a managed
+# sub-session.
+#
+# This deliberately does NOT spawn Claude Code per tick anymore. The legacy
+# run-trigger.sh is preserved for manual one-shot invocations only.
+
+generate_trigger_plist() {
+    local TRIGGER_NAME="$1"
+    local SCHEDULE="$2"
+    local INTERVAL="$3"
+    local CATCHUP_SECONDS="$4"   # Optional: catch-up interval (seconds) for
+                                 # calendar-scheduled cadences that have no
+                                 # StartInterval. Used by the bus's at-most-once
+                                 # gate (WS4) for stale fast-forward after an
+                                 # outage. For StartInterval cadences the
+                                 # interval is taken from $INTERVAL directly.
+
+    local LABEL="ai.maestro.${AGENT_FIRST}-${TRIGGER_NAME}"
+    local ENQUEUE="$AGENT_DIR/scripts/cadence/enqueue-cadence-tick.mjs"
+    local WRAPPER="$AGENT_DIR/scripts/cadence/launchd-cadence-wrapper.sh"
+
+    # The cadence interval (seconds) handed to the enqueue script so the bus can
+    # enforce at-most-once delivery + stale fast-forward. Prefer the explicit
+    # StartInterval; otherwise the caller-supplied calendar catch-up interval.
+    local TICK_INTERVAL="$INTERVAL"
+    [ -z "$TICK_INTERVAL" ] && TICK_INTERVAL="$CATCHUP_SECONDS"
+
+    local ENQUEUE_ARGS="$WRAPPER|$ENQUEUE|$TRIGGER_NAME|--source=launchd|--quiet"
+    if [ -n "$TICK_INTERVAL" ]; then
+        ENQUEUE_ARGS="$ENQUEUE_ARGS|--interval=$TICK_INTERVAL"
+    fi
+
+    generate_plist "$LABEL" \
+        "$ENQUEUE_ARGS" \
+        "$SCHEDULE" \
+        "$INTERVAL" \
+        ""
+}
+
+# ── Generate all plists ─────────────────────────────────────────────────────
+
+echo ""
+echo "Generating launchd plist files..."
+
+# 1. Main daemon (KeepAlive) — uses wrapper to bootstrap env (HOME, PATH, etc).
+#    Wrapper is at scripts/daemon/launchd-wrapper.sh. This avoids passing
+#    EnvironmentVariables in the plist, which has been observed to cause
+#    EX_CONFIG (78) failures on macOS when the agent's logs/state dirs are
+#    symlinked to an external SSD.
+generate_plist "ai.maestro.${AGENT_FIRST}-daemon" \
+    "${AGENT_DIR}/scripts/daemon/launchd-wrapper.sh" \
+    "" "" "true"
+
+# 2. Slack events relay polling job (every 5 sec).
+#    The local Mac mini polls the Railway-hosted webhook relay for buffered
+#    Slack events. This replaces the older local slack-events-server.mjs which
+#    needed an inbound tunnel from the Mac. See docs/guides/webhook-relay-setup.md.
+#    Routed through the generic wrapper so its output goes to the SSD.
+generate_plist "ai.maestro.${AGENT_FIRST}-poll-relay" \
+    "${AGENT_DIR}/scripts/daemon/launchd-wrapper-generic.sh|${AGENT_DIR}/scripts/poll-slack-events.sh" \
+    "" "5" ""
+
+# 2b. Slack Socket Mode listener (persistent WebSocket — KeepAlive).
+#     Replaces the 60s slack-poller cycle for DMs + @mentions with a
+#     realtime WSS connection to Slack. Requires SLACK_APP_LEVEL_TOKEN
+#     in .env (the operator generates this via api.slack.com → App-Level
+#     Tokens with connections:write scope). The wrapper at
+#     scripts/cadence/launchd-socket-mode-wrapper.sh bootstraps env +
+#     execs scripts/poller/slack-socket-mode.mjs.
+#
+#     KeepAlive uses the same SuccessfulExit:false / Crashed:true semantics
+#     as the daemon plist so .emergency-stop (clean exit 0) refuses to
+#     restart, but unexpected crashes do trigger a restart treadmill.
+generate_plist "ai.maestro.${AGENT_FIRST}-slack-socket" \
+    "${AGENT_DIR}/scripts/cadence/launchd-socket-mode-wrapper.sh" \
+    "" "" "true"
+
+# 3. Standard cadence triggers — derived from the canonical STANDARD_CADENCES
+#    in lib/cadences.mjs (the single source of truth), NOT a second hardcoded
+#    list. Previously these were hand-maintained here and drifted from the SoT
+#    (e.g. org-pulse was declared standard but never got a plist). The node
+#    emitter prints one pipe-delimited line per cadence:
+#       <id>|<interval-or-empty>|<base64(StartCalendarInterval-xml)-or-empty>|<catchup-seconds-or-empty>
+#    consumed by the same generate_trigger_plist used for archetype cadences.
+#
+#    Resolution order for the cadences module: the agent's own lib/ (framework
+#    checkout), then the installed @cohortapp/agent-sdk package. If neither resolves
+#    (e.g. a stripped-down repo) we FALL BACK to the legacy hardcoded standard
+#    set so the generator never breaks.
+CADENCES_MJS=""
+for cand in \
+    "$AGENT_DIR/lib/cadences.mjs" \
+    "$AGENT_DIR/node_modules/@cohortapp/agent-sdk/lib/cadences.mjs" \
+    "${MAESTRO_ROOT:-}/lib/cadences.mjs"; do
+    if [ -n "$cand" ] && [ -f "$cand" ]; then CADENCES_MJS="$cand"; break; fi
+done
+
+STD_CADENCE_LINES=""
+if [ -n "$CADENCES_MJS" ]; then
+    STD_CADENCE_LINES=$(node --input-type=module -e "
+import { STANDARD_CADENCES } from '$CADENCES_MJS';
+// StartCalendarInterval dict-content XML, mirroring generate-cadences.mjs.
+function xml(cal) {
+  let x = '';
+  if (cal.weekday != null) x += '        <key>Weekday</key>\n        <integer>' + cal.weekday + '</integer>\n';
+  if (cal.day != null) x += '        <key>Day</key>\n        <integer>' + cal.day + '</integer>\n';
+  x += '        <key>Hour</key>\n        <integer>' + (cal.hour ?? 9) + '</integer>\n        <key>Minute</key>\n        <integer>' + (cal.minute ?? 0) + '</integer>';
+  return x;
+}
+const DAILY = 86400, MONTHLY = 2592000;
+for (const c of STANDARD_CADENCES) {
+  if (typeof c.interval === 'number') {
+    // StartInterval cadence: interval is both schedule and catch-up.
+    process.stdout.write(c.id + '|' + c.interval + '||\n');
+  } else if (c.calendar) {
+    // Calendar cadence: monthly catch-up if it's pinned to a Day, else daily.
+    const catchup = c.calendar.day != null ? MONTHLY : DAILY;
+    const b64 = Buffer.from(xml(c.calendar), 'utf-8').toString('base64');
+    process.stdout.write(c.id + '||' + b64 + '|' + catchup + '\n');
+  }
+  // Cadences with neither interval nor calendar (none today) are skipped — they
+  // have no launchd trigger.
+}
+" 2>/dev/null) || STD_CADENCE_LINES=""
+fi
+
+if [ -n "$STD_CADENCE_LINES" ]; then
+    while IFS='|' read -r CID CINT CSCHED64 CCATCHUP; do
+        [ -z "$CID" ] && continue
+        if [ -n "$CINT" ]; then
+            generate_trigger_plist "$CID" "" "$CINT"
+        else
+            CSCHED=$(printf '%s' "$CSCHED64" | base64 -D 2>/dev/null || printf '%s' "$CSCHED64" | base64 -d 2>/dev/null)
+            generate_trigger_plist "$CID" "$CSCHED" "" "$CCATCHUP"
+        fi
+    done <<< "$STD_CADENCE_LINES"
+    echo "[generate-plists] emitted standard cadences from lib/cadences.mjs (SoT)"
+else
+    # ── Legacy fallback: hardcoded standard set (used only when lib/cadences.mjs
+    #    cannot be resolved). Kept in sync with STANDARD_CADENCES. ──
+    echo "[generate-plists] WARN: lib/cadences.mjs not resolvable; using hardcoded standard set"
+    generate_trigger_plist "inbox-processor" "" "300"
+    generate_trigger_plist "backlog-executor" "" "600"
+    generate_trigger_plist "meeting-prep" "" "900"
+    generate_trigger_plist "meeting-action-capture" "" "1800"
+    generate_trigger_plist "daily-morning-brief" \
+        "        <key>Hour</key>
+        <integer>6</integer>
+        <key>Minute</key>
+        <integer>30</integer>" \
+        "" "86400"
+    generate_trigger_plist "daily-midday-sweep" \
+        "        <key>Hour</key>
+        <integer>12</integer>
+        <key>Minute</key>
+        <integer>0</integer>" \
+        "" "86400"
+    generate_trigger_plist "daily-evening-wrap" \
+        "        <key>Hour</key>
+        <integer>18</integer>
+        <key>Minute</key>
+        <integer>0</integer>" \
+        "" "86400"
+    generate_trigger_plist "org-pulse" \
+        "        <key>Hour</key>
+        <integer>7</integer>
+        <key>Minute</key>
+        <integer>30</integer>" \
+        "" "86400"
+    generate_trigger_plist "quarterly-self-assessment" \
+        "        <key>Day</key>
+        <integer>1</integer>
+        <key>Hour</key>
+        <integer>9</integer>
+        <key>Minute</key>
+        <integer>0</integer>" \
+        "" "2592000"
+    generate_trigger_plist "skill-curator" \
+        "        <key>Hour</key>
+        <integer>4</integer>
+        <key>Minute</key>
+        <integer>0</integer>" \
+        "" "86400"
+fi
+
+# NOTE: Weekly/monthly archetype cadences (legacy weekly-hiring/priorities/
+# execution/engineering-health/strategic-memo, plus function-specific ones) are
+# ARCHETYPE-DRIVEN. They come from config/.cadence-plists.tsv, written by
+# scripts/setup/generate-cadences.mjs from the agent's function × altitude, and
+# are emitted by the loop further below — so a compliance officer and a founder
+# get different schedules instead of one hardcoded exec-office rhythm.
+
+# ── Archetype-specific cadences (function + altitude) ────────────────────────
+# Emitted from config/.cadence-plists.tsv (written by generate-cadences.mjs).
+# Each line is pipe-delimited: <cadence-id>|<interval-seconds-or-empty>|<base64(StartCalendarInterval-xml)-or-empty>.
+# Pipe (not tab) so bash preserves the empty middle field; base64/ids never contain '|'.
+# Reuses generate_trigger_plist so these get the identical cadence-bus plist
+# structure as the standard set. No-op when the file is absent (legacy / framework).
+CADENCE_TSV="$AGENT_DIR/config/.cadence-plists.tsv"
+if [ -f "$CADENCE_TSV" ]; then
+    while IFS='|' read -r CID CINT CSCHED64; do
+        [ -z "$CID" ] && continue
+        if [ -n "$CINT" ]; then
+            generate_trigger_plist "$CID" "" "$CINT"
+        else
+            CSCHED=$(printf '%s' "$CSCHED64" | base64 -D 2>/dev/null || printf '%s' "$CSCHED64" | base64 -d 2>/dev/null)
+            generate_trigger_plist "$CID" "$CSCHED" ""
+        fi
+    done < "$CADENCE_TSV"
+    echo "[generate-plists] emitted archetype cadences from config/.cadence-plists.tsv"
+fi
+
+echo ""
+echo "Generated $(ls "$PLIST_DIR"/*.plist 2>/dev/null | wc -l | tr -d ' ') plist files in $PLIST_DIR"