@pennyfarthing/core 11.0.0 → 11.1.1

package/pennyfarthing-dist/agents/handoff.md

@@ -1,250 +0,0 @@
----
-name: handoff
-description: Workflow-driven handoff that reads phase requirements from workflow definition
-tools: Bash, Read, Edit, Grep
-model: haiku
----
-
-<arguments>
-| Argument | Required | Description |
-|----------|----------|-------------|
-| `STORY_ID` | Yes | e.g., "31-10" |
-| `WORKFLOW` | Yes | "tdd", "trivial", etc. |
-| `CURRENT_PHASE` | Yes | "red", "green", "review" |
-| `REPOS` | Yes | Repository name |
-| `VERDICT` | No | For review: "approved"/"rejected" |
-| `TEST_RESULT` | No | "RED" or "GREEN" |
-| `ASSESSMENT_SECTION` | No | e.g., "TEA Assessment" |
-| `PR_NUMBER` | No | For green→review |
-</arguments>
-
-<critical>
-**Marker generation happens in the CALLING agent, not here.**
-This subagent does gate checks and session updates only.
-Return `HANDOFF_RESULT` with the next agent name - the calling agent runs `handoff-marker.sh` as their last action.
-</critical>
-
----
-
-## Gate Types
-
-<gate>
-### tests_fail (TEA → Dev)
-- [ ] Tests committed
-- [ ] Tests are RED (failing)
-- [ ] Assessment exists
-
-**STOP if tests GREEN** - TEA must verify tests exercise new code.
-</gate>
-
-<gate>
-### tests_pass (Dev → Reviewer)
-- [ ] Quality checks pass (run: `.pennyfarthing/scripts/workflow/check.sh`)
-- [ ] Git working tree clean
-- [ ] Changes pushed to remote
-- [ ] Local PR exists and is open
-- [ ] Assessment exists
-
-**STOP if any check fails.**
-</gate>
-
-<gate>
-### approval (Reviewer → SM/Dev)
-- [ ] Reviewer Assessment exists
-- [ ] Contains APPROVED or REJECTED
-- [ ] Verdict matches VERDICT parameter
-- [ ] PR pushed to remote
-
-**If VERDICT=approved:** Status → approved, ready for SM finish
-**If VERDICT=rejected:** Return to Dev with issues
-</gate>
-
-<gate>
-### manual (SM setup)
-No automated checks. Always passes.
-</gate>
-
----
-
-## Workflow
-
-1. **Find phase and gate type:**
-   ```bash
-   ./scripts/handoff-cli.sh find-phase --workflow {WORKFLOW} --phase {CURRENT_PHASE}
-   ```
-
-2. **Verify assessment exists** (if ASSESSMENT_SECTION provided)
-
-3. **Run gate-specific checks** (see above)
-
-4. **Determine next phase and check for tandem partner:**
-   ```bash
-   ./scripts/handoff-cli.sh next-phase --workflow {WORKFLOW} --phase {CURRENT_PHASE}
-   ```
-
-   After determining the next phase, check for tandem partner:
-   ```bash
-   TANDEM_PARTNER=$(yq eval ".workflow.phases[] | select(.name == \"$NEXT_PHASE\") | .tandem.partner // \"\"" "$PROJECT_ROOT/.pennyfarthing/workflows/${WORKFLOW}.yaml" 2>/dev/null)
-   TANDEM_SCOPE=$(yq eval ".workflow.phases[] | select(.name == \"$NEXT_PHASE\") | .tandem.scope // \"\"" "$PROJECT_ROOT/.pennyfarthing/workflows/${WORKFLOW}.yaml" 2>/dev/null)
-   ```
-
-   If tandem partner exists, add `**Tandem:** {partner} ({scope})` to session metadata.
-
-5. **Update session file using Edit tool:**
-
-   First, get timestamps and calculate duration:
-   ```bash
-   PHASE_STARTED=$(grep "^\*\*Phase Started:\*\*" .session/{STORY_ID}-session.md | sed 's/\*\*Phase Started:\*\* //')
-   NOW=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
-   DURATION=$(./scripts/handoff-cli.sh calculate-duration --started-at "$PHASE_STARTED" --ended-at "$NOW")
-   ```
-
-   **5a. Update `**Phase:**` field** - Use Edit tool:
-   ```
-   file_path: .session/{STORY_ID}-session.md
-   old_string: "**Phase:** {CURRENT_PHASE}"
-   new_string: "**Phase:** {NEXT_PHASE}"
-   ```
-
-   **5b. Update `**Phase Started:**` field** - Use Edit tool:
-   ```
-   file_path: .session/{STORY_ID}-session.md
-   old_string: "**Phase Started:** {PHASE_STARTED}"
-   new_string: "**Phase Started:** {NOW}"
-   ```
-
-   **5c. Update Phase History table** - Use Edit tool to add end timestamp and duration:
-   Find the row for current phase (has `| - | - |` at end) and update:
-   ```
-   file_path: .session/{STORY_ID}-session.md
-   old_string: "| {CURRENT_PHASE} | {PHASE_STARTED} | - | - |"
-   new_string: "| {CURRENT_PHASE} | {PHASE_STARTED} | {NOW} | {DURATION} |"
-   ```
-
-   Phase History table format:
-   ```
-   | Phase | Started | Ended | Duration |
-   |-------|---------|-------|----------|
-   ```
-
-   **5d. Add Handoff History row** - Use Edit tool to append to Handoff History section:
-   If `### Handoff History` section doesn't exist, create it first.
-   Then append a row:
-   ```
-   | {CURRENT_PHASE} ({CURRENT_AGENT}) | {NEXT_PHASE} ({NEXT_AGENT}) | {GATE_TYPE} | PASSED | {NOW} |
-   ```
-
-   Handoff History table format:
-   ```
-   | From | To | Gate | Status | Timestamp |
-   |------|-----|------|--------|-----------|
-   ```
-
-6. **Return HANDOFF_RESULT** (see Output Format below)
-
----
-
-<output>
-## Output Format
-
-Return a `HANDOFF_RESULT` block:
-
-### Success
-```
-HANDOFF_RESULT:
-  status: success
-  next_agent: {NEXT_AGENT}
-  next_phase: {NEXT_PHASE}
-  gate: {GATE_TYPE}
-  story_id: {STORY_ID}
-  tandem_partner: {PARTNER or "none"}
-  tandem_scope: {SCOPE or "none"}
-
-  next_steps:
-    - "Handoff complete. Run handoff-marker.sh as ABSOLUTE LAST ACTION."
-    - "Command: .pennyfarthing/scripts/core/handoff-marker.sh {next_agent}"
-    - "Output marker result verbatim, then EXIT. Nothing after."
-```
-
-### Example (TEA → Dev, with tandem)
-```
-HANDOFF_RESULT:
-  status: success
-  next_agent: dev
-  next_phase: green
-  gate: tests_fail
-  story_id: 46-5
-  tandem_partner: ux-designer
-  tandem_scope: file-watch
-
-  next_steps:
-    - "Handoff complete. Run handoff-marker.sh as ABSOLUTE LAST ACTION."
-    - "Command: .pennyfarthing/scripts/core/handoff-marker.sh dev"
-    - "Output marker result verbatim, then EXIT. Nothing after."
-```
-
-### Example (Dev → Reviewer)
-```
-HANDOFF_RESULT:
-  status: success
-  next_agent: reviewer
-  next_phase: review
-  gate: tests_pass
-  story_id: 46-5
-  tandem_partner: pm
-  tandem_scope: file-watch
-
-  next_steps:
-    - "Handoff complete. Run handoff-marker.sh as ABSOLUTE LAST ACTION."
-    - "Command: .pennyfarthing/scripts/core/handoff-marker.sh reviewer"
-    - "Output marker result verbatim, then EXIT. Nothing after."
-```
-
-### Blocked
-```
-HANDOFF_RESULT:
-  status: blocked
-  error: "{description}"
-  fix: "{recommended action}"
-  gate: {GATE_TYPE}
-  failed_check: "{specific check that failed}"
-
-  next_steps:
-    - "Handoff blocked at gate '{gate}': {error}"
-    - "Required action: {fix}"
-    - "Do NOT run handoff-marker.sh. Resolve issue first."
-```
-</output>
-
----
-
-## Agent Command Mapping
-
-| Agent | Command |
-|-------|---------|
-| TEA | `/tea` |
-| Dev | `/dev` |
-| Reviewer | `/reviewer` |
-| SM | `/sm` |
-| Architect | `/architect` |
-| DevOps | `/devops` |
-| Tech-Writer | `/tech-writer` |
-| UX-Designer | `/ux-designer` |
-| PM | `/pm` |
-
----
-
-## Calling Agent Exit Sequence
-
-When you receive `HANDOFF_RESULT`:
-
-1. If `status: blocked` → Report the error, do NOT proceed
-2. If `status: success` → Continue to exit sequence below
-
-**CRITICAL: The calling agent MUST run this as their ABSOLUTE LAST ACTION:**
-
-```bash
-.pennyfarthing/scripts/core/handoff-marker.sh {next_agent}
-```
-
-Then output the script's result verbatim and EXIT. Nothing else after.

package/pennyfarthing-dist/agents/sm-handoff.md

@@ -1,152 +0,0 @@
----
-name: sm-handoff
-description: Complete handoff bookkeeping when SM work is done
-tools: Bash, Read, Edit
-model: haiku
----
-
-<critical>
-**NEVER mark acceptance criteria as complete.** This subagent only:
-1. Verifies prerequisites exist
-2. Updates the Workflow Tracking section for phase transition
-
-AC checkboxes are marked ONLY by the agent that does the work.
-</critical>
-
-<critical>
-**Marker generation happens in the CALLING agent, not here.**
-This subagent verifies prerequisites and updates session file only.
-Return `HANDOFF_RESULT` with the next agent name - SM runs `handoff-marker.sh` as their last action.
-</critical>
-
-<arguments>
-| Argument | Required | Description |
-|----------|----------|-------------|
-| `STORY_ID` | Yes | Story identifier, e.g., "31-10" |
-| `NEXT_AGENT` | Yes | Target agent: `tea`, `dev`, `ux-designer`, `orchestrator` |
-| `NEXT_PHASE` | Yes | Target phase from workflow YAML: `red`, `design`, `implement` |
-| `WORKFLOW` | Yes | Workflow type: "tdd", "tdd-tandem", "bdd", "bdd-tandem", "trivial", etc. |
-</arguments>
-
-<tandem-awareness>
-## Tandem Partner Detection
-
-After verifying prerequisites, check if the next phase has a tandem partner:
-
-```bash
-TANDEM_PARTNER=$(yq eval ".workflow.phases[] | select(.name == \"$NEXT_PHASE\") | .tandem.partner // \"\"" "$PROJECT_ROOT/.pennyfarthing/workflows/${WORKFLOW}.yaml" 2>/dev/null)
-TANDEM_SCOPE=$(yq eval ".workflow.phases[] | select(.name == \"$NEXT_PHASE\") | .tandem.scope // \"\"" "$PROJECT_ROOT/.pennyfarthing/workflows/${WORKFLOW}.yaml" 2>/dev/null)
-```
-
-If `TANDEM_PARTNER` is non-empty, include it in the HANDOFF_RESULT and add a note to the session file:
-
-```markdown
-**Tandem:** {TANDEM_PARTNER} ({TANDEM_SCOPE})
-```
-
-This tells the receiving agent they can spawn their tandem partner for consultation.
-</tandem-awareness>
-
-<gate>
-## Handoff Checklist
-
-- [ ] Session file exists with story context
-- [ ] Acceptance criteria defined (count, don't mark)
-- [ ] Feature branches created
-- [ ] Jira story claimed (if applicable)
-- [ ] Update Workflow Tracking section:
-  - `**Phase:**` → `{NEXT_PHASE}`
-  - `**Phase Started:**` → `{NOW}`
-  - Add Phase History row
-- [ ] Report status summary
-</gate>
-
-## Phase Transition Update
-
-Edit `## Workflow Tracking`:
-
-```markdown
-## Workflow Tracking
-**Workflow:** {WORKFLOW}
-**Phase:** {NEXT_PHASE}
-**Phase Started:** {NOW}
-
-### Phase History
-| Phase | Started | Ended | Duration |
-|-------|---------|-------|----------|
-| setup | {SM_STARTED} | {NOW} | {DURATION} |
-| {NEXT_PHASE} | {NOW} | - | - |
-```
-
-**Duration:** Subtract SM Started from {NOW}, format as `Xm` or `Xh Ym`.
-
-<output>
-## Output Format
-
-Return a `HANDOFF_RESULT` block:
-
-### Success
-```
-HANDOFF_RESULT:
-  status: success
-  next_agent: {NEXT_AGENT}
-  next_phase: {NEXT_PHASE}
-  story_id: {STORY_ID}
-  tandem_partner: {PARTNER or "none"}
-  tandem_scope: {SCOPE or "none"}
-  summary: "{what was done}"
-
-  next_steps:
-    - "Handoff complete. Run handoff-marker.sh as ABSOLUTE LAST ACTION."
-    - "Command: .pennyfarthing/scripts/core/handoff-marker.sh {next_agent}"
-    - "Output marker result verbatim, then EXIT. Nothing after."
-```
-
-### Example (SM → TEA)
-```
-HANDOFF_RESULT:
-  status: success
-  next_agent: tea
-  next_phase: red
-  story_id: MSSCI-12274
-  tandem_partner: architect
-  tandem_scope: file-watch
-  summary: "Session updated (setup → red), branch verified, 7 AC defined. Tandem: architect available."
-
-  next_steps:
-    - "Handoff complete. Run handoff-marker.sh as ABSOLUTE LAST ACTION."
-    - "Command: .pennyfarthing/scripts/core/handoff-marker.sh tea"
-    - "Output marker result verbatim, then EXIT. Nothing after."
-```
-
-### Blocked
-```
-HANDOFF_RESULT:
-  status: blocked
-  error: "{description}"
-  fix: "{recommended action}"
-  failed_check: "{which gate check failed}"
-
-  next_steps:
-    - "Handoff blocked: {error}"
-    - "Required action: {fix}"
-    - "Do NOT run handoff-marker.sh. Resolve issue first."
-```
-</output>
-
----
-
-## Calling Agent Exit Sequence
-
-When SM receives `HANDOFF_RESULT`:
-
-1. If `status: blocked` → Report the error, do NOT proceed
-2. If `status: success` → Continue to exit sequence below
-
-**CRITICAL: SM MUST run this as their ABSOLUTE LAST ACTION:**
-
-```bash
-.pennyfarthing/scripts/core/handoff-marker.sh {next_agent}
-```
-
-Then output the script's result verbatim and EXIT. Nothing else after.

package/pennyfarthing-dist/scripts/core/handoff-marker.sh

@@ -1,112 +0,0 @@
-#!/bin/bash
-# Generate complete AGENT_COMMAND block with pre-rendered marker
-# Usage: handoff-marker.sh <next-agent> [--error <message>]
-#
-# Outputs a complete AGENT_COMMAND block that subagents can emit verbatim.
-# Handles IS_CYCLIST and USE_TIREPUMP detection automatically.
-#
-# Single source of truth for marker format - both agents and quick-actions.js
-# should reference this script's output format.
-
-set -euo pipefail
-
-NEXT_AGENT=""
-IS_ERROR=false
-ERROR_MSG=""
-
-# Parse args
-while [[ $# -gt 0 ]]; do
-  case "$1" in
-    --error)
-      IS_ERROR=true
-      ERROR_MSG="${2:-Error occurred}"
-      shift 2
-      ;;
-    *)
-      if [[ -z "$NEXT_AGENT" ]]; then
-        NEXT_AGENT="$1"
-      fi
-      shift
-      ;;
-  esac
-done
-
-if [[ -z "$NEXT_AGENT" && "$IS_ERROR" != "true" ]]; then
-  echo "Usage: handoff-marker.sh <next-agent> [--error <message>]" >&2
-  echo "" >&2
-  echo "Examples:" >&2
-  echo "  handoff-marker.sh dev               # Normal handoff to dev" >&2
-  echo "  handoff-marker.sh tea               # Normal handoff to tea" >&2
-  echo "  handoff-marker.sh --error 'Tests failing'  # Error case" >&2
-  exit 1
-fi
-
-# Get environment from check-context.sh
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-eval "$("$SCRIPT_DIR/check-context.sh" 2>/dev/null)" || true
-
-# Default values if check-context.sh failed
-IS_CYCLIST="${IS_CYCLIST:-false}"
-USE_TIREPUMP="${USE_TIREPUMP:-false}"
-RELAY_MODE="${RELAY_MODE:-false}"
-
-# Generate marker based on environment
-if [[ "$IS_ERROR" == "true" ]]; then
-  # Error case - no marker
-  cat <<EOF
----
-AGENT_COMMAND:
-  marker: ""
-  fallback: "$ERROR_MSG"
-  error: true
----
-EOF
-elif [[ "$IS_CYCLIST" != "true" ]]; then
-  # Not in Cyclist - no marker, include context info
-  # Get context percentage for warning
-  CONTEXT_PCT="${CONTEXT_USABLE_PERCENT:-unknown}"
-  if [[ "$CONTEXT_PCT" != "unknown" ]] && [[ "$CONTEXT_PCT" -ge 60 ]] 2>/dev/null; then
-    CONTEXT_WARNING=" (context: ${CONTEXT_PCT}% - consider /clear before continuing)"
-  elif [[ "$CONTEXT_PCT" != "unknown" ]]; then
-    CONTEXT_WARNING=" (context: ${CONTEXT_PCT}%)"
-  else
-    CONTEXT_WARNING=""
-  fi
-  cat <<EOF
----
-AGENT_COMMAND:
-  marker: ""
-  fallback: "Run \`/${NEXT_AGENT}\` to continue${CONTEXT_WARNING}"
-  relay_mode: ${RELAY_MODE}
-  context_percent: ${CONTEXT_PCT}
----
-EOF
-elif [[ "$RELAY_MODE" != "true" ]]; then
-  # Cyclist + Relay OFF - emit question marker for manual handoff confirmation
-  cat <<EOF
----
-AGENT_COMMAND:
-  marker: "<!-- CYCLIST:QUESTION:yesno -->"
-  question: "Ready to hand off to /${NEXT_AGENT}?"
-  fallback: "Run \`/${NEXT_AGENT}\` to continue"
----
-EOF
-elif [[ "$USE_TIREPUMP" == "true" ]]; then
-  # Cyclist + Relay ON + high context - context clear marker
-  cat <<EOF
----
-AGENT_COMMAND:
-  marker: "<!-- CYCLIST:CONTEXT_CLEAR:/${NEXT_AGENT} -->"
-  fallback: "Run \`/${NEXT_AGENT}\` to continue"
----
-EOF
-else
-  # Cyclist + Relay ON + low context - handoff marker
-  cat <<EOF
----
-AGENT_COMMAND:
-  marker: "<!-- CYCLIST:HANDOFF:/${NEXT_AGENT} -->"
-  fallback: "Run \`/${NEXT_AGENT}\` to continue"
----
-EOF
-fi