@firatcand/roster 0.1.0 → 1.0.0

package/templates/scaffold/scripts/new-agent.sh

@@ -0,0 +1,416 @@
+#!/usr/bin/env bash
+# new-agent.sh — scaffolds a new global agent under a function category
+#
+# Usage:
+#   bash scripts/new-agent.sh <function> <agent-name>
+#   bash scripts/new-agent.sh --slash-only <function> <agent-name>
+#
+# --slash-only is the recovery flag invoked by chief-of-staff's guided-mode
+# atomic-write transaction when the slash command file fails to land after
+# the agent tree has already been written (see ROS-52). It writes only
+# .claude/commands/<agent>.md, with strict no-clobber, and requires the
+# agent tree to exist.
+
+set -euo pipefail
+
+print_usage() {
+  cat >&2 <<EOF
+Usage:
+  $0 <function> <agent-name>
+  $0 --slash-only <function> <agent-name>
+
+Function: any slug registered in .config/functions.yaml
+Example:  $0 gtm content-agent
+
+--slash-only: recovery flag. Writes only .claude/commands/<agent-name>.md.
+  Requires the agent tree at <function>/<agent-name>/ to already exist.
+  Exits non-zero (no clobber) if the slash command file already exists.
+EOF
+}
+
+SLASH_ONLY=0
+if [ "${1:-}" = "--slash-only" ]; then
+  SLASH_ONLY=1
+  shift
+fi
+
+if [ $# -ne 2 ]; then
+  print_usage
+  exit 1
+fi
+
+FN="$1"
+NAME="$2"
+ROOT="$(cd "$(dirname "$0")/.." && pwd)"
+TARGET="$ROOT/$FN/$NAME"
+
+source "$ROOT/scripts/lib/functions.sh"
+
+# write_slash_command <abs-file-path> <function> <agent-name>
+# Canonical writer for .claude/commands/<agent>.md. Both the full-install
+# path and --slash-only call this so the two paths cannot drift over time
+# (see docs/learnings/2026-Q2/installer-mutations-need-drift-detector-mirror.md).
+write_slash_command() {
+  local file="$1" fn="$2" name="$3"
+  # ROS-62: quote the description so YAML-special characters in $fn or the
+  # placeholder body don't trip the I4 YAML parser when invariants run.
+  cat > "$file" <<EOF
+---
+name: $name
+description: "$fn agent — TODO: fill in description"
+---
+
+# /$name
+
+You are operating the \`$fn/$name\` agent. Load \`$fn/$name/agent.md\` and the workspace's relevant context.
+
+The user request is: \$ARGUMENTS
+
+## Routing logic
+
+Parse the user request:
+
+1. **If it matches \`run <plan-name>\`**:
+   - Load \`$fn/$name/plans/<plan-name>.yaml\`. If it doesn't exist, list available plans and ask user to pick.
+   - Load \`$fn/$name/config.yaml\` and resolve env via \`resolveAgentEnv\` (\`$fn/$name/.env\` overrides workspace \`/.env\`).
+   - Load workspace guidelines referenced under \`config.yaml\` \`guideline_refs:\` (e.g., \`/guidelines/voice.md\`, \`/guidelines/icps/\`, \`/guidelines/messaging.md\`).
+   - Validate that all required tool bindings have non-empty env vars. Abort with a clear message if not.
+   - Execute the plan steps. Substitute \`\${tools.X.env_var}\`, \`\${inputs.X}\`, \`\${config.X}\`.
+   - Log to \`$fn/$name/logs/runs/<YYYY-MM>/<YYYY-MM-DD-HHMM>.md\`.
+   - Surface HITL approvals per the plan's \`approval_channel\`.
+
+2. **If no plan is named**:
+   - List available plans from \`$fn/$name/plans/\` with descriptions. Ask user to pick.
+
+3. **For ad-hoc strategic work**: suggest invoking \`$fn/EXPERT.md\` instead.
+
+## Constraints
+
+- Only run plans that exist as files in \`$fn/$name/plans/\`.
+- Don't bypass approval gates.
+- File writes go to \`$fn/$name/logs/runs/\` unless the plan explicitly writes elsewhere.
+EOF
+}
+
+if ! is_valid_function "$FN"; then
+  echo "ERROR: '$FN' is not a registered function." >&2
+  echo "Registered functions:" >&2
+  read_functions | sed 's/^/  - /' >&2
+  echo "" >&2
+  echo "To add a new function: bash scripts/create-function.sh $FN" >&2
+  exit 1
+fi
+
+if ! [[ "$NAME" =~ ^[a-z][a-z0-9-]*$ ]]; then
+  echo "ERROR: Agent name must be lowercase, alphanumeric + hyphens." >&2
+  exit 1
+fi
+
+# --slash-only branch: write only the slash command file and exit.
+# Requires agent tree to exist; refuses to clobber an existing slash command.
+if [ "$SLASH_ONLY" = "1" ]; then
+  if [ ! -d "$TARGET" ]; then
+    echo "ERROR: agent tree '$FN/$NAME' does not exist at $TARGET." >&2
+    echo "  --slash-only is a recovery flag — create the agent tree first with:" >&2
+    echo "    bash scripts/new-agent.sh $FN $NAME" >&2
+    exit 1
+  fi
+  COMMANDS_DIR="$ROOT/.claude/commands"
+  SLASH_CMD_FILE="$COMMANDS_DIR/$NAME.md"
+  if [ -f "$SLASH_CMD_FILE" ]; then
+    echo "ERROR: slash command already exists at .claude/commands/$NAME.md. Refusing to clobber." >&2
+    exit 1
+  fi
+  mkdir -p "$COMMANDS_DIR"
+  write_slash_command "$SLASH_CMD_FILE" "$FN" "$NAME"
+  echo "Created: .claude/commands/$NAME.md"
+  exit 0
+fi
+
+if [ -d "$TARGET" ]; then
+  echo "ERROR: Agent '$FN/$NAME' already exists at $TARGET" >&2
+  exit 1
+fi
+
+echo "Creating agent: $FN/$NAME"
+
+mkdir -p "$TARGET"/{subagents,playbook,pending,logs/runs,logs/feedback,.claude/skills,.claude/plugins}
+
+# agent.md stub
+cat > "$TARGET/agent.md" << EOF
+# $NAME
+
+## Purpose
+
+<One paragraph: what this agent does, why it exists.>
+
+## Inputs
+
+The orchestrator expects per-plan inputs (declared in each plan's \`inputs:\` block).
+
+Read at runtime:
+
+- \`agent.md\` (this file)
+- \`config.yaml\` (workspace-root-relative guideline refs + tool bindings)
+- Workspace guidelines referenced under \`config.yaml\` \`guideline_refs:\` (e.g., \`/guidelines/voice.md\`, \`/guidelines/icps/\`, \`/guidelines/messaging.md\`)
+- \`playbook/\` — validated lessons (single playbook per agent)
+
+Env resolution: \`<this-agent>/.env\` overrides workspace \`/.env\`. Required tool env vars validated before the plan runs.
+
+## Plans
+
+Named plans this agent runs (files in \`plans/<plan>.yaml\`). One-line description per plan.
+No default plan — invoking without a plan triggers an interactive "which plan?" prompt.
+
+- <plan-name>: <one-liner>
+
+## Subagents
+
+- <subagent-name>.md — <one-liner>
+
+## Outputs
+
+Run file at \`logs/runs/<YYYY-MM>/<YYYY-MM-DD-HHMM>.md\`. See \`conventions.md\` § "Run file format".
+Per-plan output schemas live in each plan's \`outputs:\` block.
+
+## Approval
+
+\`approval_channel: auto\` — in-session if interactive, Slack \`#${FN}\` if cron (resolved via \`SLACK_HITL_CHANNEL_$(echo "$FN" | tr '[:lower:]-' '[:upper:]_')\` in workspace \`/.env\`).
+
+## Lessons protocol
+
+Log candidate lessons inline in run output under \`## Candidate lessons\`. Don't write to \`playbook/\` directly during runs — that's the dreamer's job.
+
+## Failure modes
+
+- <known failure mode>: <handling>
+EOF
+
+# README
+cat > "$TARGET/README.md" << EOF
+# $NAME
+
+<One-line description.>
+
+## Files
+
+- \`agent.md\` — orchestrator contract (behavioral prompt, plans list, tool bindings schema)
+- \`config.yaml\` — guideline refs + tool bindings (workspace-root paths)
+- \`.env\` — agent-scoped env overrides (gitignored, 0600 — optional, inherits from workspace \`/.env\`)
+- \`plans/\` — named workflows (\`<plan>.yaml\`)
+- \`subagents/\` — specialized roles
+- \`playbook/\` — validated lessons (single playbook per agent)
+- \`pending/\` — HITL items awaiting approval
+- \`logs/runs/\`, \`logs/feedback/\` — run outputs + mirrored feedback
+- \`asset-references.md\` — which workspace assets this agent uses (thin pointer)
+- \`.claude/\` — agent-scoped Claude Code config (skills, plugins, settings)
+- \`.mcp.json\` — agent-scoped MCPs
+
+## Invocation
+
+From the workspace root:
+
+\`\`\`bash
+claude
+> /$NAME run <plan-name>
+\`\`\`
+
+Or via natural language:
+
+\`\`\`
+"Run $FN/$NAME using the <plan-name> plan"
+\`\`\`
+
+From cron: see ADR-0001 (subscription-billed scheduling via native local schedulers). Install via \`roster schedule install $FN/$NAME <plan> --cron "<expr>" --tool claude|codex\`.
+
+## Configuration
+
+\`config.yaml\` (this agent) — guideline refs + tool bindings.
+Workspace \`/.env\` (root) + optional \`<this-agent>/.env\` for agent-scoped overrides.
+
+## Outputs
+
+\`logs/runs/<YYYY-MM>/<YYYY-MM-DD-HHMM>.md\` — one file per invocation.
+EOF
+
+# .mcp.json stub
+cat > "$TARGET/.mcp.json" << EOF
+{
+  "_comment": "Agent-scoped MCPs for $NAME. Available when working in this agent's tree. Add MCPs this agent specifically needs. Universal MCPs (Slack, Google Drive) are inherited from the workspace .mcp.json.",
+  "mcpServers": {}
+}
+EOF
+
+# .claude/settings.json
+cat > "$TARGET/.claude/settings.json" << EOF
+{
+  "_comment": "Agent-scoped Claude Code settings for $NAME."
+}
+EOF
+
+# Subagent template
+cat > "$TARGET/subagents/_template.md" << 'EOF'
+# <Subagent Name>
+
+## Role
+<One paragraph: narrow job, single responsibility.>
+
+## Inputs
+<What the orchestrator passes in.>
+
+## Output
+<Structured output the orchestrator can parse.>
+
+## Tools
+<Named tools this subagent uses.>
+
+## Boundaries
+<What this subagent does NOT do.>
+
+## Quality bar
+<Specific criteria for acceptable output.>
+EOF
+
+# config.yaml — agent config (guideline refs + tool bindings)
+# Minimal stub. Until the Phase 2 env-merge loader lands, bindings are
+# hand-edited: copy the ## Tools and bindings YAML block from agent.md
+# into the tools: mapping below, then fill the corresponding env vars
+# in workspace /.env (or this agent's .env).
+cat > "$TARGET/config.yaml" << EOF
+agent: $FN/$NAME
+plans_dir: ./plans/
+
+# Workspace-root-relative refs. Loader rejects literal absolute fs paths
+# like /Users/, /home/, /etc/, /var/, /tmp/, /opt/.
+guideline_refs:
+  voice: /guidelines/voice.md
+  icps: /guidelines/icps/
+  messaging: /guidelines/messaging.md
+  # add more as needed:
+  # brand_book: /guidelines/brand-book.md
+  # do_and_dont: /guidelines/do-and-dont.md
+  # compliance: /guidelines/compliance.md
+  # competitors: /guidelines/competitors.md
+
+# Tool bindings. Each tool entry names the env var, required flag, and a
+# short description. Env vars themselves live in workspace /.env (or are
+# overridden in this agent's .env).
+tools: {}
+EOF
+
+# asset-references.md — thin pointer at agent root
+cat > "$TARGET/asset-references.md" << EOF
+# Asset references — $FN/$NAME
+
+This agent uses these assets from \`guidelines/asset-links.md\`:
+
+- <e.g., specific asset by name>
+EOF
+
+touch "$TARGET/playbook/.gitkeep"
+touch "$TARGET/pending/.gitkeep"
+touch "$TARGET/logs/runs/.gitkeep"
+touch "$TARGET/logs/feedback/.gitkeep"
+touch "$TARGET/.claude/skills/.gitkeep"
+touch "$TARGET/.claude/plugins/.gitkeep"
+
+# === Tool bindings prompt (added by tool-bindings workflow) ===
+# Ask the user whether to define tools now. If yes, collect tool names and scaffold
+# a stub `## Tools and bindings` section in the new agent.md.
+
+echo ""
+NEW_AGENT_MD="$ROOT/$FN/$NAME/agent.md"
+
+# Skip prompt entirely if non-interactive (no TTY) or AGENT_TEAM_NO_CONFIRM=1
+if [ -t 0 ] && [ "${AGENT_TEAM_NO_CONFIRM:-0}" != "1" ]; then
+  read -r -p "Define tools and bindings now? (y/N): " DEFINE_TOOLS
+  DEFINE_TOOLS="${DEFINE_TOOLS:-N}"
+
+  if [[ "$DEFINE_TOOLS" =~ ^[Yy]$ ]]; then
+    echo ""
+    echo "Which tools is this agent using?"
+    echo "  Enter comma-separated names (e.g., gmail, drive, attio, heyreach)."
+    echo "  Or press Enter to skip."
+    read -r -p "  > " TOOLS_LINE
+
+    if [ -n "$TOOLS_LINE" ]; then
+      # Normalize: split by comma, trim whitespace, lowercase
+      TOOL_NAMES=()
+      IFS=',' read -ra RAW_TOOLS <<< "$TOOLS_LINE"
+      for raw in "${RAW_TOOLS[@]}"; do
+        clean="$(echo "$raw" | sed 's/^[[:space:]]*//; s/[[:space:]]*$//' | tr '[:upper:]' '[:lower:]')"
+        if [[ -n "$clean" && "$clean" =~ ^[a-z][a-z0-9_-]*$ ]]; then
+          TOOL_NAMES+=("$clean")
+        elif [ -n "$clean" ]; then
+          echo "  WARN: skipping invalid tool name '$clean' (must be lowercase, start with letter, only [a-z0-9_-])" >&2
+        fi
+      done
+
+      if [ ${#TOOL_NAMES[@]} -gt 0 ]; then
+        {
+          echo ""
+          echo "## Tools and bindings"
+          echo ""
+          echo "Tool bindings expected by this agent. Until the Phase 2 env-merge loader ships, configure them by hand: mirror the YAML block below into \`<agent>/config.yaml\` under a \`tools:\` key, and put the env var values in workspace \`/.env\` (or \`<agent>/.env\` for agent-scoped overrides)."
+          echo ""
+          echo "Fill in each tool's bindings below. Schema per conventions.md § \"Tool bindings\": each tool has a \`env_var\` (the env-var name the runtime reads), a \`required\` flag (true/false), and a one-line \`description\`."
+          echo ""
+          echo '```yaml'
+          for tool in "${TOOL_NAMES[@]}"; do
+            tool_env=$(echo "$tool" | tr '[:lower:]-' '[:upper:]_')
+            echo "$tool:"
+            echo "  env_var: ${tool_env}_API_KEY  # TODO: confirm env var name"
+            echo "  required: true               # TODO: confirm required vs optional"
+            echo "  description: \"\"             # TODO: one-line description"
+          done
+          echo '```'
+        } >> "$NEW_AGENT_MD"
+
+        echo ""
+        echo "✓ Added '## Tools and bindings' to $NEW_AGENT_MD with stubs for: ${TOOL_NAMES[*]}"
+        echo "  Edit agent.md to fill in env_var names + descriptions, then mirror the tools: block into <agent>/config.yaml by hand."
+        echo "  (Phase 2 will add an env-merge loader + automated config.yaml/.env wiring; until then this step is manual.)"
+      else
+        echo "  No valid tool names provided. Skipping section."
+      fi
+    else
+      echo "  Empty input. Skipping section."
+    fi
+  else
+    echo "(Skipped tool definition. Add a '## Tools and bindings' section manually later if needed.)"
+  fi
+else
+  # Non-interactive: skip the prompt entirely. User can add the section manually.
+  :
+fi
+# === End tool bindings prompt ===
+
+# === Plans directory ===
+PLANS_DIR="$TARGET/plans"
+mkdir -p "$PLANS_DIR"
+touch "$PLANS_DIR/.gitkeep"
+echo "Created: $FN/$NAME/plans/"
+
+# === Slash command file ===
+COMMANDS_DIR="$ROOT/.claude/commands"
+mkdir -p "$COMMANDS_DIR"
+SLASH_CMD_FILE="$COMMANDS_DIR/$NAME.md"
+
+if [ ! -f "$SLASH_CMD_FILE" ]; then
+  write_slash_command "$SLASH_CMD_FILE" "$FN" "$NAME"
+  echo "Created: .claude/commands/$NAME.md"
+fi
+
+echo ""
+echo "✓ Agent '$FN/$NAME' created at $TARGET"
+echo ""
+echo "Next steps:"
+echo "  1. Fill in $TARGET/agent.md (purpose, inputs, plans, subagents, tools, outputs)"
+echo "  2. Fill in $TARGET/config.yaml (guideline_refs, tools)"
+echo "  3. Add subagents: cp $TARGET/subagents/_template.md $TARGET/subagents/<name>.md and fill in"
+echo "  4. Add agent-scoped MCPs to $TARGET/.mcp.json if needed (HeyReach, Apollo, etc.)"
+echo "  5. Update $TARGET/README.md with a real description"
+echo "  6. Add at least one plan to $TARGET/plans/ (e.g., $TARGET/plans/<plan-name>.yaml)"
+echo "  7. Edit .claude/commands/$NAME.md to fill in the description"
+echo ""
+echo "Reference: see gtm/sdr/ for a complete example."

package/templates/scaffold/scripts/rename-agent.sh

@@ -0,0 +1,91 @@
+#!/usr/bin/env bash
+# rename-agent.sh — renames a global agent everywhere it appears
+# Usage: bash scripts/rename-agent.sh <function> <old> <new>
+
+set -euo pipefail
+
+if [ $# -ne 3 ]; then
+  echo "Usage: $0 <function> <old> <new>"
+  exit 1
+fi
+
+FN="$1"
+OLD="$2"
+NEW="$3"
+ROOT="$(cd "$(dirname "$0")/.." && pwd)"
+
+if ! [[ "$NEW" =~ ^[a-z][a-z0-9-]*$ ]]; then
+  echo "ERROR: New slug must be lowercase, alphanumeric + hyphens, starting with a letter."
+  exit 1
+fi
+
+OLD_DIR="$ROOT/$FN/$OLD"
+NEW_DIR="$ROOT/$FN/$NEW"
+
+if [ ! -d "$OLD_DIR" ]; then
+  echo "ERROR: Agent '$FN/$OLD' not found at $OLD_DIR"
+  exit 1
+fi
+
+if [ -d "$NEW_DIR" ]; then
+  echo "ERROR: Agent '$FN/$NEW' already exists at $NEW_DIR"
+  exit 1
+fi
+
+TIMESTAMP=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
+
+mv "$OLD_DIR" "$NEW_DIR"
+echo "  Moved: $FN/$OLD/ → $FN/$NEW/"
+
+SED_INPLACE=(-i)
+if [[ "$(uname)" == "Darwin" ]]; then
+  SED_INPLACE=(-i '')
+fi
+
+# Broad replace on prose files only — these are markdown stubs where the
+# old slug appears in headings / paths and we want every reference updated.
+for f in "$NEW_DIR"/agent.md "$NEW_DIR"/README.md "$NEW_DIR"/asset-references.md; do
+  [ -f "$f" ] && sed "${SED_INPLACE[@]}" "s|$OLD|$NEW|g" "$f"
+done
+
+# config.yaml: only the identity field. A broad replace would corrupt env-var
+# names, comments, and any value that legitimately contains $OLD.
+CFG="$NEW_DIR/config.yaml"
+if [ -f "$CFG" ]; then
+  sed "${SED_INPLACE[@]}" "s|^agent: $FN/$OLD$|agent: $FN/$NEW|" "$CFG"
+fi
+
+while IFS= read -r f; do
+  case "$f" in
+    *"_archive/"*) continue ;;
+    *"/logs/"*) continue ;;
+    *"/log/runs/"*) continue ;;
+    *"/log/feedback/"*) continue ;;
+    *"/playbook/"*) continue ;;
+    *) sed "${SED_INPLACE[@]}" "s|$FN/$OLD|$FN/$NEW|g" "$f" ;;
+  esac
+done < <(grep -rl "$FN/$OLD" --include='*.md' --include='*.yaml' --include='*.json' --include='*.sh' --include='*.txt' "$ROOT" 2>/dev/null)
+
+OLD_CMD="$ROOT/.claude/commands/$OLD.md"
+NEW_CMD="$ROOT/.claude/commands/$NEW.md"
+if [ -f "$OLD_CMD" ]; then
+  mv "$OLD_CMD" "$NEW_CMD"
+  sed "${SED_INPLACE[@]}" "s|$OLD|$NEW|g" "$NEW_CMD"
+  echo "  Renamed slash command: .claude/commands/$OLD.md → $NEW.md"
+fi
+
+LOG_DIR="$ROOT/chief-of-staff/logs/$(date +%Y-%m)"
+mkdir -p "$LOG_DIR"
+LOG_FILE="$LOG_DIR/operations-$(date +%Y-%m-%d).md"
+{
+  echo ""
+  echo "## $TIMESTAMP — rename-agent: $FN/$OLD → $FN/$NEW"
+} >> "$LOG_FILE"
+
+echo ""
+echo "Files mentioning '$OLD' that were NOT auto-updated (review manually):"
+grep -rln "$OLD" "$ROOT" 2>/dev/null | grep -v "_archive\|/logs/\|/log/runs/\|/log/feedback/\|/playbook/" | sed "s|$ROOT/|  - |" || echo "  (none found)"
+
+echo ""
+echo "✓ Rename complete."
+echo "  Operation log: $LOG_FILE"

package/templates/scaffold/scripts/save-state.sh

@@ -0,0 +1,32 @@
+#!/usr/bin/env bash
+# save-state.sh — write workspace state.md
+#
+# In v1, workspace = project. state.md lives at workspace root.
+# Format: see conventions.md § "State file format". Normally Claude writes
+# state.md directly when asked. This script exists for external invocation
+# or scripted updates.
+#
+# Usage: bash scripts/save-state.sh "<state notes (multiline OK)>"
+
+set -euo pipefail
+
+if [ $# -lt 1 ]; then
+  echo "Usage: $0 \"<state notes>\""
+  exit 1
+fi
+
+NOTES="$*"
+ROOT="$(cd "$(dirname "$0")/.." && pwd)"
+STATE_FILE="$ROOT/state.md"
+TIMESTAMP=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
+
+cat > "$STATE_FILE" << EOF
+---
+updated: $TIMESTAMP
+---
+
+$NOTES
+EOF
+
+echo "✓ Saved workspace state"
+echo "  $STATE_FILE"

package/agents/critic.md

@@ -1,74 +0,0 @@
----
-name: critic
-description: "Reviews an outreach draft for tone, accuracy, brand fit, risk, compliance, and do-and-don't violations. Returns pass/fail with specific feedback. Used by the sdr skill. Does not rewrite drafts — the writer iterates."
-version: "0.1.0"
-owner_skill: sdr
----
-
-# Critic
-
-## Role
-
-Review a draft for tone, accuracy, brand fit, risk, compliance, and do-and-don't violations. Returns pass/fail with specific feedback. Does not rewrite — that's the writer's job on next iteration.
-
-## Inputs
-
-- `draft` (object): output from the writer
-- `prospect` (object): the enriched prospect
-- `voice` (markdown): project's voice doc
-- `icps` (markdown): all relevant persona docs
-- `do_and_dont` (markdown, optional)
-- `compliance` (markdown, optional)
-- `competitors` (markdown, optional)
-- `lessons` (markdown): relevant project + global lessons
-- `iteration` (int): current iteration count (orchestrator caps at 2)
-
-## Output
-
-```yaml
-verdict: pass | fail
-score: 0-10
-issues:
-  - severity: blocker | major | minor
-    category: voice | accuracy | risk | cta | length | personalization | compliance | do-and-dont | competitor-handling
-    description: "..."
-    suggested_fix: "..."
-voice_match: 0-10
-personalization: 0-10
-risk_flags: []
-```
-
-`verdict: pass` requires:
-- Zero blocker issues
-- Zero major issues
-- voice_match ≥ 7
-- personalization ≥ 6
-- No risk flags
-- Zero do-and-dont violations
-- Zero compliance violations
-
-Otherwise `fail`. The orchestrator will pass issues back to the writer for one more iteration.
-
-## Tools
-
-None. Pure review from inputs.
-
-## Boundaries
-
-- Do NOT rewrite the draft. List issues; the writer rewrites.
-- Do NOT invent facts to test against. Use only inputs.
-- Be strict on accuracy, risk, compliance, and do-and-dont; less strict on style preferences (those go in `minor`).
-- A "blocker" would embarrass the user. A "major" hurts effectiveness. A "minor" is polish.
-
-## Risk categories to flag
-
-- Unverifiable claims about the prospect or company
-- Aggressive, manipulative, or pressuring language
-- Misrepresentation of sender's relationship to prospect
-- Compliance issues (GDPR, CAN-SPAM, platform ToS — see `compliance.md` if provided)
-- Anything that would damage the project's brand if seen publicly
-- Improper handling of competitor mentions (see `competitors.md` if provided)
-
-## Quality bar
-
-A pass means: I, the user, would be willing to put my name on this and send it. No exceptions.

package/agents/enricher.md

@@ -1,56 +0,0 @@
----
-name: enricher
-description: "Fills in missing fields on existing prospects (recent posts, company news, mutual signals) via Apollo, HeyReach, and web search. Used by the sdr skill before drafting outreach. Does not score, does not filter, does not contact."
-version: "0.1.0"
-owner_skill: sdr
----
-
-# Enricher
-
-## Role
-
-Take an existing prospect list and fill in missing fields needed for personalized outreach. Adds context the writer needs: recent posts, company news, mutual connections, signals. Does not contact, does not score.
-
-## Inputs
-
-- `prospects` (array): list from prospector or external source
-- `required_fields` (array): which fields must be filled — e.g., `[recent_post, company_news, role_tenure]`
-- `enrichment_depth` (string): `light` (search results only) | `deep` (web fetch + multi-source)
-
-## Output
-
-Same prospects array, with new fields added. Mark unfillable fields explicitly:
-
-```yaml
-prospects:
-  - name: "Alice Example"
-    role: "Head of Growth"
-    company: "ExampleCo"
-    enrichment:
-      recent_post:
-        url: "..."
-        snippet: "..."
-        date: "2026-04-22"
-      company_news:
-        - { headline: "...", url: "...", date: "..." }
-      role_tenure_months: 8
-      mutual_signals: []
-    enrichment_status: complete   # complete | partial | failed
-    enrichment_notes: ""
-```
-
-## Tools
-
-- Apollo.io MCP: `apollo_people_match`, `apollo_organizations_enrich`, `apollo_organizations_job_postings`
-- Web search + web_fetch: recent posts, news, signals
-- HeyReach MCP: `get_lead` for LinkedIn URL-based lookup
-
-## Boundaries
-
-- Do NOT score or filter — return everything, even partially enriched.
-- Do NOT make assumptions about missing fields. Mark explicitly.
-- Cap web_fetch at 3 sources per prospect for deep enrichment.
-
-## Quality bar
-
-A prospect with `enrichment_status: complete` must have all required fields. Otherwise `partial` with notes on what's missing.