@pennyfarthing/core 11.1.0 → 11.2.0

package/pennyfarthing-dist/scripts/core/dialogue-manager.sh

@@ -0,0 +1,322 @@
+#!/usr/bin/env bash
+# dialogue-manager.sh — Dialogue file management for tandem agent consultation
+#
+# Creates, appends to, and archives dialogue files recording consultation exchanges
+# between tandem agents. Format defined in ADR-0012 (lines 156-195).
+#
+# Usage:
+#   dialogue-manager.sh init <story-id> <workflow> <leader> <partner>
+#   dialogue-manager.sh append <story-id> <question> <recommendation> <confidence>
+#   dialogue-manager.sh outcome <story-id> <exchange-num> <applied|deferred|rejected> [note]
+#   dialogue-manager.sh summarize <story-id>
+#   dialogue-manager.sh archive <story-id> [jira-key]
+
+set -euo pipefail
+
+# Self-locate and set up PROJECT_ROOT
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)"
+source "$SCRIPT_DIR/../lib/find-root.sh"
+
+SESSION_DIR="$PROJECT_ROOT/.session"
+ARCHIVE_DIR="$PROJECT_ROOT/sprint/archive"
+SUMMARY_MARKER="## Summary"
+
+# =============================================================================
+# Helpers
+# =============================================================================
+
+usage() {
+  echo "Usage:"
+  echo "  dialogue-manager.sh init <story-id> <workflow> <leader> <partner>"
+  echo "  dialogue-manager.sh append <story-id> <question> <recommendation> <confidence>"
+  echo "  dialogue-manager.sh outcome <story-id> <exchange-num> <applied|deferred|rejected> [note]"
+  echo "  dialogue-manager.sh summarize <story-id>"
+  echo "  dialogue-manager.sh archive <story-id> [jira-key]"
+  exit 1
+}
+
+dialogue_path() {
+  echo "$SESSION_DIR/${1}-dialogue.md"
+}
+
+now_iso() {
+  date -u +"%Y-%m-%dT%H:%M:%SZ"
+}
+
+now_hhmm() {
+  date -u +"%H:%M"
+}
+
+# Count existing exchanges in a dialogue file
+count_exchanges() {
+  local file="$1"
+  grep -c '^## Exchange ' "$file" 2>/dev/null || echo "0"
+}
+
+# =============================================================================
+# Commands
+# =============================================================================
+
+cmd_init() {
+  local story_id="${1:?Missing story-id}"
+  local workflow="${2:?Missing workflow}"
+  local leader="${3:?Missing leader}"
+  local partner="${4:?Missing partner}"
+  local file
+  file="$(dialogue_path "$story_id")"
+
+  if [[ -f "$file" ]]; then
+    echo "Dialogue file already exists: $file"
+    exit 0
+  fi
+
+  mkdir -p "$SESSION_DIR"
+
+  cat > "$file" <<EOF
+# Tandem Dialogue: ${story_id}
+
+**Workflow:** ${workflow}
+**Leader:** ${leader} | **Partner:** ${partner}
+**Started:** $(now_iso)
+
+---
+
+${SUMMARY_MARKER}
+- **Total exchanges:** 0
+- **Key decisions:** None
+- **Time in tandem:** 0m
+EOF
+
+  echo "Created dialogue file: $file"
+}
+
+cmd_append() {
+  local story_id="${1:?Missing story-id}"
+  local question="${2:?Missing question}"
+  local recommendation="${3:?Missing recommendation}"
+  local confidence="${4:?Missing confidence}"
+  local file
+  file="$(dialogue_path "$story_id")"
+
+  # Auto-init if file missing
+  if [[ ! -f "$file" ]]; then
+    echo "Error: Dialogue file not found. Run 'init' first: $file" >&2
+    exit 1
+  fi
+
+  local num
+  num=$(( $(count_exchanges "$file") + 1 ))
+  local ts
+  ts="$(now_hhmm)"
+
+  # Extract leader and partner from header
+  local leader partner
+  leader=$(grep '^\*\*Leader:\*\*' "$file" | sed 's/.*\*\*Leader:\*\* \([^ ]*\).*/\1/' | head -1)
+  partner=$(grep '^\*\*Leader:\*\*' "$file" | sed 's/.*\*\*Partner:\*\* \([^ ]*\).*/\1/' | head -1)
+
+  # Fallback if parsing fails
+  leader="${leader:-leader}"
+  partner="${partner:-partner}"
+
+  local exchange_block
+  exchange_block="## Exchange ${num}
+**[${ts}] ${leader} → ${partner}**
+
+> ${question}
+
+**[${ts}] ${partner}:**
+
+${recommendation}
+
+**Confidence:** ${confidence}
+
+**Outcome:** _pending_
+
+---
+"
+
+  # Insert before summary marker
+  if grep -q "^${SUMMARY_MARKER}" "$file"; then
+    # Use awk to insert before summary
+    local tmpfile
+    tmpfile="$(mktemp)"
+    awk -v block="$exchange_block" -v marker="$SUMMARY_MARKER" '
+      $0 == marker { printf "%s\n\n", block }
+      { print }
+    ' "$file" > "$tmpfile"
+    mv "$tmpfile" "$file"
+  else
+    echo "" >> "$file"
+    echo "$exchange_block" >> "$file"
+  fi
+
+  echo "Appended exchange #${num} to $file"
+}
+
+cmd_outcome() {
+  local story_id="${1:?Missing story-id}"
+  local exchange_num="${2:?Missing exchange-num}"
+  local outcome="${3:?Missing outcome (applied|deferred|rejected)}"
+  local note="${4:-}"
+  local file
+  file="$(dialogue_path "$story_id")"
+
+  if [[ ! -f "$file" ]]; then
+    echo "Error: Dialogue file not found: $file" >&2
+    exit 1
+  fi
+
+  # Validate outcome
+  case "$outcome" in
+    applied|deferred|rejected) ;;
+    *) echo "Error: outcome must be applied, deferred, or rejected" >&2; exit 1 ;;
+  esac
+
+  local outcome_text
+  if [[ -n "$note" ]]; then
+    outcome_text="**Outcome:** ${outcome} - ${note}"
+  else
+    outcome_text="**Outcome:** ${outcome}"
+  fi
+
+  # Find the exchange and update its outcome line
+  local in_target=false
+  local found=false
+  local tmpfile
+  tmpfile="$(mktemp)"
+
+  while IFS= read -r line; do
+    if [[ "$line" =~ ^##\ Exchange\ ([0-9]+) ]]; then
+      if [[ "${BASH_REMATCH[1]}" == "$exchange_num" ]]; then
+        in_target=true
+      else
+        in_target=false
+      fi
+    fi
+
+    if $in_target && [[ "$line" =~ ^\*\*Outcome:\*\* ]]; then
+      echo "$outcome_text" >> "$tmpfile"
+      found=true
+      in_target=false
+    else
+      echo "$line" >> "$tmpfile"
+    fi
+  done < "$file"
+
+  if $found; then
+    mv "$tmpfile" "$file"
+    echo "Updated exchange #${exchange_num} outcome to: ${outcome}"
+  else
+    rm -f "$tmpfile"
+    echo "Error: Exchange #${exchange_num} not found" >&2
+    exit 1
+  fi
+}
+
+cmd_summarize() {
+  local story_id="${1:?Missing story-id}"
+  local file
+  file="$(dialogue_path "$story_id")"
+
+  if [[ ! -f "$file" ]]; then
+    echo "Error: Dialogue file not found: $file" >&2
+    exit 1
+  fi
+
+  local total
+  total="$(count_exchanges "$file")"
+
+  # Collect applied decisions
+  local decisions=""
+  local in_exchange=false
+  local current_outcome=""
+  local current_note=""
+
+  while IFS= read -r line; do
+    if [[ "$line" =~ ^\*\*Outcome:\*\*\ applied ]]; then
+      local note_part="${line#*applied}"
+      note_part="${note_part# - }"
+      if [[ -n "$note_part" ]]; then
+        decisions="${decisions}  - ${note_part}\n"
+      fi
+    fi
+  done < "$file"
+
+  if [[ -z "$decisions" ]]; then
+    decisions="None"
+  fi
+
+  # Calculate time span from first and last exchange timestamps
+  local first_ts last_ts duration="0m"
+  first_ts=$(grep -m 1 '^\*\*\[' "$file" | sed 's/.*\*\*\[\([0-9]*:[0-9]*\)\].*/\1/' || echo "")
+  last_ts=$(grep '^\*\*\[' "$file" | tail -1 | sed 's/.*\*\*\[\([0-9]*:[0-9]*\)\].*/\1/' || echo "")
+
+  if [[ -n "$first_ts" && -n "$last_ts" ]]; then
+    local first_mins last_mins
+    first_mins=$(( 10#${first_ts%%:*} * 60 + 10#${first_ts##*:} ))
+    last_mins=$(( 10#${last_ts%%:*} * 60 + 10#${last_ts##*:} ))
+    local diff=$(( last_mins - first_mins ))
+    if (( diff > 0 )); then
+      duration="${diff}m"
+    fi
+  fi
+
+  # Build new summary
+  local new_summary="${SUMMARY_MARKER}
+- **Total exchanges:** ${total}
+- **Key decisions:**
+$(echo -e "$decisions")- **Time in tandem:** ${duration}"
+
+  # Replace summary section (everything from marker to end)
+  if grep -q "^${SUMMARY_MARKER}" "$file"; then
+    local tmpfile
+    tmpfile="$(mktemp)"
+    awk -v marker="$SUMMARY_MARKER" '
+      $0 == marker { found=1; next }
+      !found { print }
+    ' "$file" > "$tmpfile"
+    echo "$new_summary" >> "$tmpfile"
+    mv "$tmpfile" "$file"
+  else
+    echo "" >> "$file"
+    echo "$new_summary" >> "$file"
+  fi
+
+  echo "Summary refreshed for $file (${total} exchanges)"
+}
+
+cmd_archive() {
+  local story_id="${1:?Missing story-id}"
+  local jira_key="${2:-}"
+  local file
+  file="$(dialogue_path "$story_id")"
+
+  if [[ ! -f "$file" ]]; then
+    echo "No dialogue file to archive: $file"
+    exit 0
+  fi
+
+  mkdir -p "$ARCHIVE_DIR"
+
+  local prefix="${jira_key:-$story_id}"
+  local dest="$ARCHIVE_DIR/${prefix}-dialogue.md"
+
+  cp "$file" "$dest"
+  echo "Archived dialogue to: $dest"
+}
+
+# =============================================================================
+# Dispatch
+# =============================================================================
+
+CMD="${1:-}"
+shift || true
+
+case "$CMD" in
+  init)      cmd_init "$@" ;;
+  append)    cmd_append "$@" ;;
+  outcome)   cmd_outcome "$@" ;;
+  summarize) cmd_summarize "$@" ;;
+  archive)   cmd_archive "$@" ;;
+  *)         usage ;;
+esac

package/pennyfarthing-dist/scripts/git/README.md

@@ -1,23 +1,33 @@
 # Git Scripts
 
-Scripts for Git operations, branching, and release management.
+Scripts for Git operations, branching, and worktree management.
 
-## Scripts
+## CLI Commands (preferred)
 
-| Script | Purpose |
-|--------|---------|
-| `create-feature-branches.sh` | Create feature branches in repos |
-| `git-status-all.sh` | Show git status across all repos |
-| `worktree-manager.sh` | Manage git worktrees for parallel work |
-| `install-git-hooks.sh` | Install git hooks in project |
-| `release.sh` | Release workflow (merge develop to main) |
+All git operations are available via the `pf git` CLI:
 
-## Usage
+| Command | Purpose |
+|---------|---------|
+| `pf git status [--brief]` | Show git status across all repos |
+| `pf git branches <name> [--repos all\|api\|ui]` | Create feature branches in repos |
+| `pf git worktree create <name> <branch>` | Create worktrees for parallel work |
+| `pf git worktree remove <name>` | Remove worktree and clean up |
+| `pf git worktree list` | List all active worktrees |
+| `pf git worktree status` | Show detailed worktree status |
+| `pf git install-hooks` | Install git hooks with .d/ dispatcher |
+| `pf git cleanup` | Organize changes into commits/branches |
 
-```bash
-.pennyfarthing/scripts/git/create-feature-branches.sh feat/my-feature
-.pennyfarthing/scripts/git/release.sh
-```
+## Legacy Scripts (deprecated shims)
+
+These scripts now forward to `pf git` commands:
+
+| Script | Forwards to |
+|--------|-------------|
+| `git-status-all.sh` | `pf git status` |
+| `create-feature-branches.sh` | `pf git branches` |
+| `worktree-manager.sh` | `pf git worktree` |
+| `install-git-hooks.sh` | `pf git install-hooks` |
+| `release.sh` | Release workflow (use `/pf-workflow start release`) |
 
 ## Ownership
 

package/pennyfarthing-dist/scripts/git/create-feature-branches.sh

@@ -1,267 +1,6 @@
-#!/usr/bin/env zsh
-# Create or checkout feature branches in Pennyfarthing repos
-# Idempotent, worktree-aware, branches from develop
-
+#!/usr/bin/env bash
+# DEPRECATED: Use `pf git branches` instead.
+# This shim forwards to the Python CLI.
 set -e
-
-# Load environment
-if [ -f .env ]; then
-    set -a; source .env; set +a
-elif [ -f ../.env ]; then
-    set -a; source ../.env; set +a
-fi
-
-show_help() {
-    cat << EOF
-Create Feature Branches - Idempotent branch creation/checkout
-
-Usage: ./create-feature-branches.sh <branch-name> [repos]
-
-Arguments:
-  branch-name    Branch name (e.g., feat/11-2-file-preview)
-  repos          Which repos to branch (default: all)
-                 Options: api, ui, all
-
-Repo Options:
-  api     - Pennyfarthing-api only
-  ui      - Pennyfarthing-ui only
-  all     - both repos (default)
-
-Examples:
-  # All repos (default)
-  ./create-feature-branches.sh feat/cross-repo-feature
-
-  # Pennyfarthing UI only
-  ./create-feature-branches.sh feat/11-2-ui-component ui
-
-  # Pennyfarthing API only
-  ./create-feature-branches.sh fix/validation-bug api
-
-Behavior:
-  - Idempotent: Checks out existing branches or creates new ones
-  - Worktree-aware: Detects main checkout vs worktree
-  - Always branches from develop
-  - Fetches latest develop before branching
-
-EOF
-}
-
-if [ -z "$1" ] || [ "$1" = "-h" ] || [ "$1" = "--help" ]; then
-    show_help
-    exit 0
-fi
-
-BRANCH_NAME="$1"
-REPOS="${2:-all}"
-
-# Validate repos parameter
-if [[ ! "$REPOS" =~ ^(api|ui|all)$ ]]; then
-    echo "❌ Error: repos must be one of: api, ui, all"
-    echo "   Got: $REPOS"
-    echo ""
-    echo "Options:"
-    echo "  api     - Pennyfarthing-api only"
-    echo "  ui      - Pennyfarthing-ui only"
-    echo "  all     - both repos (default)"
-    exit 1
-fi
-
-# Detect if we're in a worktree
-CURRENT_DIR=$(pwd)
-if [[ "$CURRENT_DIR" == *"/worktrees/"* ]]; then
-    # Extract worktree name from path
-    WORKTREE_NAME=$(echo "$CURRENT_DIR" | sed -E 's|.*/worktrees/([^/]+).*|\1|')
-    REPO_BASE="$PROJECT_ROOT/worktrees/$WORKTREE_NAME"
-    echo "📂 Detected worktree: $WORKTREE_NAME"
-else
-    REPO_BASE="$PROJECT_ROOT"
-    echo "📂 Using main checkout"
-fi
-
-# Array to track processed repos for verification
-declare -a PROCESSED_REPOS
-
-# Function to create or checkout branch in a repo
-create_or_checkout_branch() {
-    local repo_path=$1
-    local repo_name=$2
-
-    if [ ! -d "$repo_path" ]; then
-        echo "⚠️  Skipping $repo_name: directory not found at $repo_path"
-        return
-    fi
-
-    cd "$repo_path"
-
-    echo ""
-    echo "🔧 Processing $repo_name..."
-
-    # Fetch latest from remote
-    echo "   Fetching from origin..."
-    git fetch origin --quiet
-
-    # Check if branch exists locally
-    if git show-ref --verify --quiet "refs/heads/$BRANCH_NAME"; then
-        echo "   ✅ Branch exists locally, checking out: $BRANCH_NAME"
-        git checkout "$BRANCH_NAME"
-    # Check if branch exists on remote
-    elif git show-ref --verify --quiet "refs/remotes/origin/$BRANCH_NAME"; then
-        echo "   ✅ Branch exists on remote, checking out and tracking: $BRANCH_NAME"
-        git checkout -b "$BRANCH_NAME" "origin/$BRANCH_NAME"
-    else
-        # Branch doesn't exist, create from develop
-        echo "   🌿 Creating new branch from develop: $BRANCH_NAME"
-
-        # Make sure we have latest develop
-        git fetch origin develop:develop --quiet 2>/dev/null || true
-
-        # Checkout develop first
-        if git show-ref --verify --quiet "refs/heads/develop"; then
-            git checkout develop --quiet
-            git pull origin develop --quiet
-        else
-            echo "   ⚠️  Local develop doesn't exist, fetching from remote"
-            git fetch origin develop:develop --quiet
-            git checkout develop --quiet
-        fi
-
-        # Create new branch
-        git checkout -b "$BRANCH_NAME"
-        echo "   ✅ Created: $BRANCH_NAME"
-    fi
-
-    # Show current status
-    CURRENT_BRANCH=$(git branch --show-current)
-    echo "   📍 Now on: $CURRENT_BRANCH"
-
-    # Track this repo for verification
-    PROCESSED_REPOS+=("$repo_path:$repo_name")
-}
-
-echo "🌿 Creating/checking out feature branches..."
-echo "   Branch: $BRANCH_NAME"
-echo "   Repos: $REPOS"
-
-# Process repos based on selection
-# When processing multiple repos, run in parallel for faster network I/O
-case "$REPOS" in
-    api)
-        create_or_checkout_branch "$REPO_BASE/Pennyfarthing-api" "Pennyfarthing-api"
-        ;;
-    ui)
-        create_or_checkout_branch "$REPO_BASE/Pennyfarthing-ui" "Pennyfarthing-ui"
-        ;;
-    all)
-        # Parallel execution for both repos
-        tmpdir=$(mktemp -d)
-        trap "rm -rf '$tmpdir'" EXIT
-        HAD_ERRORS=false
-
-        # Run both in parallel, capturing output
-        # Write to separate files to avoid race condition on shared file
-        (
-            create_or_checkout_branch "$REPO_BASE/Pennyfarthing-api" "Pennyfarthing-api"
-            echo "$REPO_BASE/Pennyfarthing-api:Pennyfarthing-api" > "$tmpdir/api.processed"
-        ) > "$tmpdir/api.out" 2>&1 &
-        pid_api=$!
-
-        (
-            create_or_checkout_branch "$REPO_BASE/Pennyfarthing-ui" "Pennyfarthing-ui"
-            echo "$REPO_BASE/Pennyfarthing-ui:Pennyfarthing-ui" > "$tmpdir/ui.processed"
-        ) > "$tmpdir/ui.out" 2>&1 &
-        pid_ui=$!
-
-        # Wait for both and capture exit codes
-        wait $pid_api; rc_api=$?
-        wait $pid_ui; rc_ui=$?
-
-        # Show output in order
-        [ -f "$tmpdir/api.out" ] && cat "$tmpdir/api.out"
-        [ -f "$tmpdir/ui.out" ] && cat "$tmpdir/ui.out"
-
-        # Rebuild PROCESSED_REPOS from separate files (avoids race condition)
-        [ -f "$tmpdir/api.processed" ] && PROCESSED_REPOS+=("$(cat "$tmpdir/api.processed")")
-        [ -f "$tmpdir/ui.processed" ] && PROCESSED_REPOS+=("$(cat "$tmpdir/ui.processed")")
-
-        # Check for failures
-        if [ $rc_api -ne 0 ] || [ $rc_ui -ne 0 ]; then
-            echo "⚠️  Some repos had errors"
-            HAD_ERRORS=true
-        fi
-        ;;
-esac
-
-echo ""
-echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
-echo "🔍 Verification Summary"
-echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
-
-# Verify each processed repo
-for repo_info in "${PROCESSED_REPOS[@]}"; do
-    repo_path="${repo_info%%:*}"
-    repo_name="${repo_info##*:}"
-
-    if [ -d "$repo_path" ]; then
-        cd "$repo_path"
-
-        echo ""
-        echo "📦 $repo_name"
-        echo "   Path: $repo_path"
-
-        # Current branch
-        CURRENT_BRANCH=$(git branch --show-current)
-        echo "   Branch: $CURRENT_BRANCH"
-
-        # Check if branch matches expected
-        if [ "$CURRENT_BRANCH" = "$BRANCH_NAME" ]; then
-            echo "   ✅ Branch matches: $BRANCH_NAME"
-        else
-            echo "   ⚠️  Branch mismatch! Expected: $BRANCH_NAME, Got: $CURRENT_BRANCH"
-        fi
-
-        # Latest commit
-        COMMIT_INFO=$(git log -1 --format="%h - %s" 2>/dev/null)
-        echo "   Commit: $COMMIT_INFO"
-
-        # Check remote tracking
-        UPSTREAM=$(git rev-parse --abbrev-ref --symbolic-full-name @{u} 2>/dev/null || echo "none")
-        if [ "$UPSTREAM" != "none" ]; then
-            echo "   Remote: $UPSTREAM"
-
-            # Check if up to date with remote
-            git fetch origin "$CURRENT_BRANCH" --quiet 2>/dev/null || true
-            LOCAL=$(git rev-parse @ 2>/dev/null)
-            REMOTE=$(git rev-parse @{u} 2>/dev/null || echo "none")
-
-            if [ "$LOCAL" = "$REMOTE" ]; then
-                echo "   Status: ✅ Up to date with remote"
-            elif [ "$REMOTE" = "none" ]; then
-                echo "   Status: ⚠️  No remote branch yet (push needed)"
-            else
-                BEHIND=$(git rev-list --count HEAD..@{u} 2>/dev/null || echo "0")
-                AHEAD=$(git rev-list --count @{u}..HEAD 2>/dev/null || echo "0")
-                if [ "$AHEAD" -gt 0 ] && [ "$BEHIND" -gt 0 ]; then
-                    echo "   Status: ⚠️  Diverged ($AHEAD ahead, $BEHIND behind)"
-                elif [ "$AHEAD" -gt 0 ]; then
-                    echo "   Status: ⬆️  $AHEAD commit(s) ahead of remote"
-                elif [ "$BEHIND" -gt 0 ]; then
-                    echo "   Status: ⬇️  $BEHIND commit(s) behind remote"
-                fi
-            fi
-        else
-            echo "   Remote: none (local branch only)"
-            echo "   Status: ⚠️  Not tracking remote (push with -u needed)"
-        fi
-    fi
-done
-
-echo ""
-echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
-if [ "$HAD_ERRORS" = true ]; then
-    echo "⚠️  Done with errors. Check output above."
-    echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
-    exit 1
-else
-    echo "✅ Done! All branches verified and ready."
-    echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
-fi
+echo "DEPRECATED: create-feature-branches.sh — use 'pf git branches' instead" >&2
+exec pf git branches "$@"