@jiggai/recipes 0.3.10 → 0.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jiggai/recipes",
-  "version": "0.3.10",
+  "version": "0.4.0",
   "description": "ClawRecipes plugin for OpenClaw (markdown recipes -> scaffold agents/teams)",
   "main": "index.ts",
   "type": "commonjs",

package/recipes/default/_snippets/shared-context-ticket-flow.json

@@ -0,0 +1,10 @@
+{
+  "laneByOwner": {
+    "dev": "in-progress",
+    "devops": "in-progress",
+    "lead": "backlog",
+    "test": "testing",
+    "qa": "testing"
+  },
+  "defaultLane": "in-progress"
+}

package/recipes/default/development-team.md

@@ -9,14 +9,14 @@ cronJobs:
     name: "Lead triage loop"
     schedule: "*/30 7-23 * * 1-5"
     timezone: "America/New_York"
-    message: "Automated lead triage loop: triage inbox/tickets, assign work, and update notes/status.md. Anti-stuck: if lowest in-progress is HARD BLOCKED, advance the next unblocked ticket (or pull from backlog). If in-progress is stale (>12h no dated update), comment or move it back. Guardrail: run ./scripts/ticket-hygiene.sh each loop; if it fails, fix assignment stubs before proceeding."
+    message: "Automated lead triage loop: triage inbox/tickets, assign work, and update notes/status.md. Anti-stuck: if lowest in-progress is HARD BLOCKED, advance the next unblocked ticket (or pull from backlog). If in-progress is stale (>12h no dated update), comment or move it back. Guardrail: run ./scripts/ticket-hygiene.sh each loop; if it fails, fix lane/status/owner mismatches before proceeding (assignment stubs are deprecated)."
     enabledByDefault: true
 
   - id: execution-loop
     name: "Execution loop"
     schedule: "*/30 7-23 * * 1-5"
     timezone: "America/New_York"
-    message: "Automated execution loop: make progress on in-progress tickets, keep changes small/safe, and update notes/status.md. Guardrail: run ./scripts/ticket-hygiene-dev.sh each loop; if it fails, fix stubs/ownership/stage issues before proceeding."
+    message: "Automated execution loop: make progress on in-progress tickets, keep changes small/safe, and update notes/status.md. Guardrail: run ./scripts/ticket-hygiene-dev.sh each loop; if it fails, fix lane/status/owner mismatches before proceeding (assignment stubs are deprecated)."
     enabledByDefault: false
 
   - id: pr-watcher
@@ -70,78 +70,111 @@ agents:
       deny: []
 
 templates:
+  sharedContext.ticketFlow: |
+    {
+      "laneByOwner": {
+        "lead": "backlog",
+        "dev": "in-progress",
+        "devops": "in-progress",
+        "test": "testing",
+        "qa": "testing"
+      },
+      "defaultLane": "in-progress"
+    }
+
   lead.ticketHygiene: |
     #!/usr/bin/env bash
     set -euo pipefail
 
     # ticket-hygiene.sh
-    # Guardrail script used by lead triage cron.
-    # Fails if any assignment stub points at a non-existent or wrong-stage ticket path.
+    # Guardrail script used by lead triage + execution loops.
+    # Assignment stubs are deprecated.
+    #
+    # Checks (ACTIVE lanes only):
+    # - Ticket file location (lane) must match Status:
+    # - Ticket Owner should be in the expected lane per shared-context/ticket-flow.json (best-effort)
+    #
+    # Notes:
+    # - We intentionally do NOT enforce mapping for work/done/ because historical tickets may have old Owner/Status.
 
     cd "$(dirname "$0")/.."
 
-    stages=(backlog in-progress testing done)
-
     fail=0
-
-    for f in work/assignments/*-assigned-*.md; do
-      [[ -f "$f" ]] || continue
-
-      bn="$(basename "$f")"
-      num="${bn%%-*}"
-      if [[ ! "$num" =~ ^[0-9]{4}$ ]]; then
-        continue
+    flow="shared-context/ticket-flow.json"
+
+    lane_from_rel() {
+      # expects work/<lane>/<file>.md
+      echo "$1" | sed -E 's#^work/([^/]+)/.*$#
#'
+    }
+
+    field_from_md() {
+      local file="$1"
+      local key="$2"
+      # Extract first matching header line like: Key: value
+      local line
+      line="$(grep -m1 -E "^${key}:[[:space:]]*" "$file" 2>/dev/null || true)"
+      echo "${line#${key}:}" | sed -E 's/^\s+//'
+    }
+
+    expected_lane_for_owner() {
+      local owner="$1"
+      local currentLane="$2"
+
+      # If jq or the mapping file isn't available, do not block progress.
+      if [[ ! -f "$flow" ]]; then
+        echo "$currentLane"
+        return 0
       fi
-
-      canonical=""
-
-      if [[ "$bn" == *"pr-body"* ]]; then
-        for stage in "${stages[@]}"; do
-          cand="work/$stage/${num}-pr-body.md"
-          if [[ -f "$cand" ]]; then
-            canonical="$cand"
-            break
-          fi
-        done
+      if ! command -v jq >/dev/null 2>&1; then
+        echo "$currentLane"
+        return 0
       fi
 
-      if [[ -z "$canonical" ]]; then
-        for stage in "${stages[@]}"; do
-          match=( work/"$stage"/"$num"-*.md )
-          if [[ -e "${match[0]}" ]]; then
-            canonical="${match[0]}"
-            break
-          fi
-        done
+      local out
+      out="$(jq -r --arg o "$owner" '.laneByOwner[$o] // .defaultLane // empty' "$flow" 2>/dev/null || true)"
+      if [[ -n "$out" ]]; then
+        echo "$out"
+      else
+        echo "$currentLane"
       fi
+    }
 
-      if [[ -z "$canonical" ]]; then
-        echo "[FAIL] $f: no ticket file found for $num in work/{backlog,in-progress,testing,done}/" >&2
-        fail=1
-        continue
-      fi
+    check_ticket() {
+      local file="$1"
+      local rel="$file"
+      rel="${rel#./}"
 
-      ticket_line="$(awk 'BEGIN{flag=0} /^## Ticket$/{flag=1; next} /^## /{if(flag){exit}} {if(flag && $0!=""){print; exit}}' "$f" | tr -d '\r')"
-      if [[ -z "$ticket_line" ]]; then
-        echo "[FAIL] $f: missing Ticket path (expected: $canonical)" >&2
-        fail=1
-        continue
+      local lane
+      lane="$(lane_from_rel "$rel")"
+
+      # Ignore done lane for owner/status enforcement.
+      if [[ "$lane" == "done" ]]; then
+        return 0
       fi
 
-      if [[ "$ticket_line" != "$canonical" ]]; then
-        echo "[FAIL] $f: Ticket path mismatch" >&2
-        echo "  has: $ticket_line" >&2
-        echo "  want: $canonical" >&2
+      local owner status
+      owner="$(field_from_md "$file" "Owner")"
+      status="$(field_from_md "$file" "Status")"
+
+      if [[ -n "$status" && "$status" != "$lane" ]]; then
+        echo "[FAIL] $rel: Status mismatch (has: $status, lane: $lane)" >&2
         fail=1
-        continue
       fi
 
-      if [[ ! -f "$ticket_line" ]]; then
-        echo "[FAIL] $f: Ticket path does not exist on disk: $ticket_line" >&2
-        fail=1
-        continue
+      if [[ -n "$owner" ]]; then
+        local expected
+        expected="$(expected_lane_for_owner "$owner" "$lane")"
+        if [[ -n "$expected" && "$expected" != "$lane" ]]; then
+          echo "[FAIL] $rel: Owner '$owner' expects lane '$expected' per $flow (currently in '$lane')" >&2
+          fail=1
+        fi
       fi
+    }
 
+    shopt -s nullglob
+    for file in work/backlog/*.md work/in-progress/*.md work/testing/*.md work/done/*.md; do
+      [[ -f "$file" ]] || continue
+      check_ticket "$file"
     done
 
     if [[ "$fail" -ne 0 ]]; then
@@ -186,73 +219,94 @@ templates:
     set -euo pipefail
 
     # ticket-hygiene.sh
-    # Guardrail script used by lead triage cron.
-    # Fails if any assignment stub points at a non-existent or wrong-stage ticket path.
+    # Guardrail script used by lead triage + execution loops.
+    # Assignment stubs are deprecated.
+    #
+    # Checks (ACTIVE lanes only):
+    # - Ticket file location (lane) must match Status:
+    # - Ticket Owner should be in the expected lane per shared-context/ticket-flow.json (best-effort)
+    #
+    # Notes:
+    # - We intentionally do NOT enforce mapping for work/done/ because historical tickets may have old Owner/Status.
 
     cd "$(dirname "$0")/.."
 
-    stages=(backlog in-progress testing done)
-
     fail=0
-
-    for f in work/assignments/*-assigned-*.md; do
-      [[ -f "$f" ]] || continue
-
-      bn="$(basename "$f")"
-      num="${bn%%-*}"
-      if [[ ! "$num" =~ ^[0-9]{4}$ ]]; then
-        continue
+    flow="shared-context/ticket-flow.json"
+
+    lane_from_rel() {
+      # expects work/<lane>/<file>.md
+      echo "$1" | sed -E 's#^work/([^/]+)/.*$#
#'
+    }
+
+    field_from_md() {
+      local file="$1"
+      local key="$2"
+      # Extract first matching header line like: Key: value
+      local line
+      line="$(grep -m1 -E "^${key}:[[:space:]]*" "$file" 2>/dev/null || true)"
+      echo "${line#${key}:}" | sed -E 's/^\s+//'
+    }
+
+    expected_lane_for_owner() {
+      local owner="$1"
+      local currentLane="$2"
+
+      # If jq or the mapping file isn't available, do not block progress.
+      if [[ ! -f "$flow" ]]; then
+        echo "$currentLane"
+        return 0
       fi
-
-      canonical=""
-
-      if [[ "$bn" == *"pr-body"* ]]; then
-        for stage in "${stages[@]}"; do
-          cand="work/$stage/${num}-pr-body.md"
-          if [[ -f "$cand" ]]; then
-            canonical="$cand"
-            break
-          fi
-        done
+      if ! command -v jq >/dev/null 2>&1; then
+        echo "$currentLane"
+        return 0
       fi
 
-      if [[ -z "$canonical" ]]; then
-        for stage in "${stages[@]}"; do
-          match=( work/"$stage"/"$num"-*.md )
-          if [[ -e "${match[0]}" ]]; then
-            canonical="${match[0]}"
-            break
-          fi
-        done
+      local out
+      out="$(jq -r --arg o "$owner" '.laneByOwner[$o] // .defaultLane // empty' "$flow" 2>/dev/null || true)"
+      if [[ -n "$out" ]]; then
+        echo "$out"
+      else
+        echo "$currentLane"
       fi
+    }
 
-      if [[ -z "$canonical" ]]; then
-        echo "[FAIL] $f: no ticket file found for $num in work/{backlog,in-progress,testing,done}/" >&2
-        fail=1
-        continue
-      fi
+    check_ticket() {
+      local file="$1"
+      local rel="$file"
+      rel="${rel#./}"
 
-      ticket_line="$(awk 'BEGIN{flag=0} /^## Ticket$/{flag=1; next} /^## /{if(flag){exit}} {if(flag && $0!=""){print; exit}}' "$f" | tr -d '\r')"
-      if [[ -z "$ticket_line" ]]; then
-        echo "[FAIL] $f: missing Ticket path (expected: $canonical)" >&2
-        fail=1
-        continue
+      local lane
+      lane="$(lane_from_rel "$rel")"
+
+      # Ignore done lane for owner/status enforcement.
+      if [[ "$lane" == "done" ]]; then
+        return 0
       fi
 
-      if [[ "$ticket_line" != "$canonical" ]]; then
-        echo "[FAIL] $f: Ticket path mismatch" >&2
-        echo "  has: $ticket_line" >&2
-        echo "  want: $canonical" >&2
+      local owner status
+      owner="$(field_from_md "$file" "Owner")"
+      status="$(field_from_md "$file" "Status")"
+
+      if [[ -n "$status" && "$status" != "$lane" ]]; then
+        echo "[FAIL] $rel: Status mismatch (has: $status, lane: $lane)" >&2
         fail=1
-        continue
       fi
 
-      if [[ ! -f "$ticket_line" ]]; then
-        echo "[FAIL] $f: Ticket path does not exist on disk: $ticket_line" >&2
-        fail=1
-        continue
+      if [[ -n "$owner" ]]; then
+        local expected
+        expected="$(expected_lane_for_owner "$owner" "$lane")"
+        if [[ -n "$expected" && "$expected" != "$lane" ]]; then
+          echo "[FAIL] $rel: Owner '$owner' expects lane '$expected' per $flow (currently in '$lane')" >&2
+          fail=1
+        fi
       fi
+    }
 
+    shopt -s nullglob
+    for file in work/backlog/*.md work/in-progress/*.md work/testing/*.md work/done/*.md; do
+      [[ -f "$file" ]] || continue
+      check_ticket "$file"
     done
 
     if [[ "$fail" -ne 0 ]]; then
@@ -294,73 +348,94 @@ templates:
     set -euo pipefail
 
     # ticket-hygiene.sh
-    # Guardrail script used by lead triage cron.
-    # Fails if any assignment stub points at a non-existent or wrong-stage ticket path.
+    # Guardrail script used by lead triage + execution loops.
+    # Assignment stubs are deprecated.
+    #
+    # Checks (ACTIVE lanes only):
+    # - Ticket file location (lane) must match Status:
+    # - Ticket Owner should be in the expected lane per shared-context/ticket-flow.json (best-effort)
+    #
+    # Notes:
+    # - We intentionally do NOT enforce mapping for work/done/ because historical tickets may have old Owner/Status.
 
     cd "$(dirname "$0")/.."
 
-    stages=(backlog in-progress testing done)
-
     fail=0
-
-    for f in work/assignments/*-assigned-*.md; do
-      [[ -f "$f" ]] || continue
-
-      bn="$(basename "$f")"
-      num="${bn%%-*}"
-      if [[ ! "$num" =~ ^[0-9]{4}$ ]]; then
-        continue
+    flow="shared-context/ticket-flow.json"
+
+    lane_from_rel() {
+      # expects work/<lane>/<file>.md
+      echo "$1" | sed -E 's#^work/([^/]+)/.*$#
#'
+    }
+
+    field_from_md() {
+      local file="$1"
+      local key="$2"
+      # Extract first matching header line like: Key: value
+      local line
+      line="$(grep -m1 -E "^${key}:[[:space:]]*" "$file" 2>/dev/null || true)"
+      echo "${line#${key}:}" | sed -E 's/^\s+//'
+    }
+
+    expected_lane_for_owner() {
+      local owner="$1"
+      local currentLane="$2"
+
+      # If jq or the mapping file isn't available, do not block progress.
+      if [[ ! -f "$flow" ]]; then
+        echo "$currentLane"
+        return 0
       fi
-
-      canonical=""
-
-      if [[ "$bn" == *"pr-body"* ]]; then
-        for stage in "${stages[@]}"; do
-          cand="work/$stage/${num}-pr-body.md"
-          if [[ -f "$cand" ]]; then
-            canonical="$cand"
-            break
-          fi
-        done
+      if ! command -v jq >/dev/null 2>&1; then
+        echo "$currentLane"
+        return 0
       fi
 
-      if [[ -z "$canonical" ]]; then
-        for stage in "${stages[@]}"; do
-          match=( work/"$stage"/"$num"-*.md )
-          if [[ -e "${match[0]}" ]]; then
-            canonical="${match[0]}"
-            break
-          fi
-        done
+      local out
+      out="$(jq -r --arg o "$owner" '.laneByOwner[$o] // .defaultLane // empty' "$flow" 2>/dev/null || true)"
+      if [[ -n "$out" ]]; then
+        echo "$out"
+      else
+        echo "$currentLane"
       fi
+    }
 
-      if [[ -z "$canonical" ]]; then
-        echo "[FAIL] $f: no ticket file found for $num in work/{backlog,in-progress,testing,done}/" >&2
-        fail=1
-        continue
-      fi
+    check_ticket() {
+      local file="$1"
+      local rel="$file"
+      rel="${rel#./}"
 
-      ticket_line="$(awk 'BEGIN{flag=0} /^## Ticket$/{flag=1; next} /^## /{if(flag){exit}} {if(flag && $0!=""){print; exit}}' "$f" | tr -d '\r')"
-      if [[ -z "$ticket_line" ]]; then
-        echo "[FAIL] $f: missing Ticket path (expected: $canonical)" >&2
-        fail=1
-        continue
+      local lane
+      lane="$(lane_from_rel "$rel")"
+
+      # Ignore done lane for owner/status enforcement.
+      if [[ "$lane" == "done" ]]; then
+        return 0
       fi
 
-      if [[ "$ticket_line" != "$canonical" ]]; then
-        echo "[FAIL] $f: Ticket path mismatch" >&2
-        echo "  has: $ticket_line" >&2
-        echo "  want: $canonical" >&2
+      local owner status
+      owner="$(field_from_md "$file" "Owner")"
+      status="$(field_from_md "$file" "Status")"
+
+      if [[ -n "$status" && "$status" != "$lane" ]]; then
+        echo "[FAIL] $rel: Status mismatch (has: $status, lane: $lane)" >&2
         fail=1
-        continue
       fi
 
-      if [[ ! -f "$ticket_line" ]]; then
-        echo "[FAIL] $f: Ticket path does not exist on disk: $ticket_line" >&2
-        fail=1
-        continue
+      if [[ -n "$owner" ]]; then
+        local expected
+        expected="$(expected_lane_for_owner "$owner" "$lane")"
+        if [[ -n "$expected" && "$expected" != "$lane" ]]; then
+          echo "[FAIL] $rel: Owner '$owner' expects lane '$expected' per $flow (currently in '$lane')" >&2
+          fail=1
+        fi
       fi
+    }
 
+    shopt -s nullglob
+    for file in work/backlog/*.md work/in-progress/*.md work/testing/*.md work/done/*.md; do
+      [[ -f "$file" ]] || continue
+      check_ticket "$file"
     done
 
     if [[ "$fail" -ne 0 ]]; then
@@ -402,73 +477,94 @@ templates:
     set -euo pipefail
 
     # ticket-hygiene.sh
-    # Guardrail script used by lead triage cron.
-    # Fails if any assignment stub points at a non-existent or wrong-stage ticket path.
+    # Guardrail script used by lead triage + execution loops.
+    # Assignment stubs are deprecated.
+    #
+    # Checks (ACTIVE lanes only):
+    # - Ticket file location (lane) must match Status:
+    # - Ticket Owner should be in the expected lane per shared-context/ticket-flow.json (best-effort)
+    #
+    # Notes:
+    # - We intentionally do NOT enforce mapping for work/done/ because historical tickets may have old Owner/Status.
 
     cd "$(dirname "$0")/.."
 
-    stages=(backlog in-progress testing done)
-
     fail=0
-
-    for f in work/assignments/*-assigned-*.md; do
-      [[ -f "$f" ]] || continue
-
-      bn="$(basename "$f")"
-      num="${bn%%-*}"
-      if [[ ! "$num" =~ ^[0-9]{4}$ ]]; then
-        continue
+    flow="shared-context/ticket-flow.json"
+
+    lane_from_rel() {
+      # expects work/<lane>/<file>.md
+      echo "$1" | sed -E 's#^work/([^/]+)/.*$#
#'
+    }
+
+    field_from_md() {
+      local file="$1"
+      local key="$2"
+      # Extract first matching header line like: Key: value
+      local line
+      line="$(grep -m1 -E "^${key}:[[:space:]]*" "$file" 2>/dev/null || true)"
+      echo "${line#${key}:}" | sed -E 's/^\s+//'
+    }
+
+    expected_lane_for_owner() {
+      local owner="$1"
+      local currentLane="$2"
+
+      # If jq or the mapping file isn't available, do not block progress.
+      if [[ ! -f "$flow" ]]; then
+        echo "$currentLane"
+        return 0
       fi
-
-      canonical=""
-
-      if [[ "$bn" == *"pr-body"* ]]; then
-        for stage in "${stages[@]}"; do
-          cand="work/$stage/${num}-pr-body.md"
-          if [[ -f "$cand" ]]; then
-            canonical="$cand"
-            break
-          fi
-        done
+      if ! command -v jq >/dev/null 2>&1; then
+        echo "$currentLane"
+        return 0
       fi
 
-      if [[ -z "$canonical" ]]; then
-        for stage in "${stages[@]}"; do
-          match=( work/"$stage"/"$num"-*.md )
-          if [[ -e "${match[0]}" ]]; then
-            canonical="${match[0]}"
-            break
-          fi
-        done
+      local out
+      out="$(jq -r --arg o "$owner" '.laneByOwner[$o] // .defaultLane // empty' "$flow" 2>/dev/null || true)"
+      if [[ -n "$out" ]]; then
+        echo "$out"
+      else
+        echo "$currentLane"
       fi
+    }
 
-      if [[ -z "$canonical" ]]; then
-        echo "[FAIL] $f: no ticket file found for $num in work/{backlog,in-progress,testing,done}/" >&2
-        fail=1
-        continue
-      fi
+    check_ticket() {
+      local file="$1"
+      local rel="$file"
+      rel="${rel#./}"
 
-      ticket_line="$(awk 'BEGIN{flag=0} /^## Ticket$/{flag=1; next} /^## /{if(flag){exit}} {if(flag && $0!=""){print; exit}}' "$f" | tr -d '\r')"
-      if [[ -z "$ticket_line" ]]; then
-        echo "[FAIL] $f: missing Ticket path (expected: $canonical)" >&2
-        fail=1
-        continue
+      local lane
+      lane="$(lane_from_rel "$rel")"
+
+      # Ignore done lane for owner/status enforcement.
+      if [[ "$lane" == "done" ]]; then
+        return 0
       fi
 
-      if [[ "$ticket_line" != "$canonical" ]]; then
-        echo "[FAIL] $f: Ticket path mismatch" >&2
-        echo "  has: $ticket_line" >&2
-        echo "  want: $canonical" >&2
+      local owner status
+      owner="$(field_from_md "$file" "Owner")"
+      status="$(field_from_md "$file" "Status")"
+
+      if [[ -n "$status" && "$status" != "$lane" ]]; then
+        echo "[FAIL] $rel: Status mismatch (has: $status, lane: $lane)" >&2
         fail=1
-        continue
       fi
 
-      if [[ ! -f "$ticket_line" ]]; then
-        echo "[FAIL] $f: Ticket path does not exist on disk: $ticket_line" >&2
-        fail=1
-        continue
+      if [[ -n "$owner" ]]; then
+        local expected
+        expected="$(expected_lane_for_owner "$owner" "$lane")"
+        if [[ -n "$expected" && "$expected" != "$lane" ]]; then
+          echo "[FAIL] $rel: Owner '$owner' expects lane '$expected' per $flow (currently in '$lane')" >&2
+          fail=1
+        fi
       fi
+    }
 
+    shopt -s nullglob
+    for file in work/backlog/*.md work/in-progress/*.md work/testing/*.md work/done/*.md; do
+      [[ -f "$file" ]] || continue
+      check_ticket "$file"
     done
 
     if [[ "$fail" -ne 0 ]]; then
@@ -831,6 +927,10 @@ files:
   - path: NOTES.md
     template: notes
     mode: createOnly
+  - path: shared-context/ticket-flow.json
+    template: sharedContext.ticketFlow
+    mode: createOnly
+
 
   # Automation / hygiene scripts
   # NOTE: portable policy: we do NOT chmod automatically. After scaffold:

package/src/handlers/tickets.ts

@@ -6,7 +6,7 @@ import { findTicketFile, handoffTicket, takeTicket } from "../lib/ticket-workflo
 import { ticketStageDir } from "../lib/lanes";
 import { computeNextTicketNumber, TICKET_FILENAME_REGEX } from "../lib/ticket-finder";
 import { resolveTeamContext } from "../lib/workspace";
-import { DEFAULT_TICKET_NUMBER, VALID_ROLES, VALID_STAGES } from "../lib/constants";
+import { VALID_ROLES, VALID_STAGES } from "../lib/constants";
 
 export function patchTicketField(md: string, key: string, value: string): string {
   const lineRe = new RegExp(`^${key}:\\s.*$`, "m");
@@ -108,40 +108,20 @@ export async function handleMoveTicket(
   await fs.writeFile(srcPath, patched, "utf8");
   if (srcPath !== destPath) await fs.rename(srcPath, destPath);
 
-  // When a ticket is moved to done, archive any assignment stubs into work/assignments/archive/.
-  if (dest === "done") {
-    const filename = path.basename(destPath);
-    const m = filename.match(TICKET_FILENAME_REGEX);
-    const ticketNumStr = m?.[1] ?? null;
-    if (ticketNumStr) {
-      const assignmentsDir = ticketStageDir(teamDir, "assignments");
-      if (await fileExists(assignmentsDir)) {
-        const files = (await fs.readdir(assignmentsDir)).filter((f) => f.startsWith(`${ticketNumStr}-assigned-`) && f.endsWith(".md"));
-        if (files.length) {
-          const archiveDir = path.join(assignmentsDir, "archive");
-          await ensureDir(archiveDir);
-          for (const f of files) {
-            const from = path.join(assignmentsDir, f);
-            const to = path.join(archiveDir, f);
-            await fs.rename(from, to);
-          }
-        }
-      }
-    }
-  }
+  // Assignment stubs are deprecated; no archival behavior.
 
   return { ok: true, from: srcPath, to: destPath };
 }
 
 /**
- * Assign a ticket to an owner (writes assignment stub + updates Owner:).
+ * Assign a ticket to an owner (updates Owner: only; assignment stubs are deprecated).
  * @param api - OpenClaw plugin API
- * @param options - teamId, ticket, owner, optional overwrite, dryRun
+ * @param options - teamId, ticket, owner, optional dryRun
  * @returns Plan (dryRun) or ok with plan
  */
 export async function handleAssign(
   api: OpenClawPluginApi,
-  options: { teamId: string; ticket: string; owner: string; overwrite?: boolean; dryRun?: boolean },
+  options: { teamId: string; ticket: string; owner: string; dryRun?: boolean },
 ) {
   const teamId = String(options.teamId);
   const { teamDir } = await resolveTeamContext(api, teamId);
@@ -151,14 +131,8 @@ export async function handleAssign(
   }
   const ticketPath = await findTicketFile(teamDir, options.ticket);
   if (!ticketPath) throw new Error(`Ticket not found: ${options.ticket}`);
-  const filename = path.basename(ticketPath);
-  const m = filename.match(TICKET_FILENAME_REGEX);
-  const ticketNumStr = m?.[1] ?? DEFAULT_TICKET_NUMBER;
-  const slug = m?.[2] ?? (options.ticket.replace(/^\d{4}-?/, "") || "ticket");
-  const assignmentsDir = ticketStageDir(teamDir, "assignments");
-  await ensureDir(assignmentsDir);
-  const assignmentPath = path.join(assignmentsDir, `${ticketNumStr}-assigned-${owner}.md`);
-  const plan = { ticketPath, assignmentPath, owner };
+  // Previously parsed for assignment-stub ids; assignment stubs are deprecated.
+  const plan = { ticketPath, owner };
   if (options.dryRun) return { ok: true, plan };
   const patchOwner = (md: string) => {
     if (md.match(/^Owner:\s.*$/m)) return md.replace(/^Owner:\s.*$/m, `Owner: ${owner}`);
@@ -167,8 +141,7 @@ export async function handleAssign(
   const md = await fs.readFile(ticketPath, "utf8");
   const nextMd = patchOwner(md);
   await fs.writeFile(ticketPath, nextMd, "utf8");
-  const assignmentMd = `# Assignment — ${ticketNumStr}-${slug}\n\nAssigned: ${owner}\n\n## Ticket\n${path.relative(teamDir, ticketPath)}\n\n## Notes\n- Created by: openclaw recipes assign\n`;
-  await writeFileSafely(assignmentPath, assignmentMd, options.overwrite ? "overwrite" : "createOnly");
+  // Assignment stubs are deprecated; do not create/update work/assignments/*.md.
   return { ok: true, plan };
 }
 
@@ -269,7 +242,8 @@ export async function handleDispatch(
   if (!requestText) throw new Error("Request cannot be empty");
   const inboxDir = path.join(teamDir, "inbox");
   const backlogDir = ticketStageDir(teamDir, "backlog");
-  const assignmentsDir = ticketStageDir(teamDir, "assignments");
+  // Assignment stubs are deprecated; do not create work/assignments/*.
+  // Keep old stubs if they exist (migration will move them to work/assignments.__deprecated__/).
   const slugify = (s: string) =>
     s.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/(^-|-$)/g, "").slice(0, 60) || "request";
   const nowKey = () => {
@@ -283,27 +257,22 @@ export async function handleDispatch(
   const baseSlug = slugify(title);
   const inboxPath = path.join(inboxDir, `${nowKey()}-${baseSlug}.md`);
   const ticketPath = path.join(backlogDir, `${ticketNumStr}-${baseSlug}.md`);
-  const assignmentPath = path.join(assignmentsDir, `${ticketNumStr}-assigned-${owner}.md`);
   const receivedIso = new Date().toISOString();
-  const inboxMd = `# Inbox — ${teamId}\n\nReceived: ${receivedIso}\n\n## Request\n${requestText}\n\n## Proposed work\n- Ticket: ${ticketNumStr}-${baseSlug}\n- Owner: ${owner}\n\n## Links\n- Ticket: ${path.relative(teamDir, ticketPath)}\n- Assignment: ${path.relative(teamDir, assignmentPath)}\n`;
-  const ticketMd = `# ${ticketNumStr}-${baseSlug}\n\nCreated: ${receivedIso}\nOwner: ${owner}\nStatus: queued\nInbox: ${path.relative(teamDir, inboxPath)}\nAssignment: ${path.relative(teamDir, assignmentPath)}\n\n## Context\n${requestText}\n\n## Requirements\n- (fill in)\n\n## Acceptance criteria\n- (fill in)\n\n## Tasks\n- [ ] (fill in)\n\n## Comments\n- (use this section for @mentions, questions, decisions, and dated replies)\n`;
-  const assignmentMd = `# Assignment — ${ticketNumStr}-${baseSlug}\n\nCreated: ${receivedIso}\nAssigned: ${owner}\n\n## Goal\n${title}\n\n## Ticket\n${path.relative(teamDir, ticketPath)}\n\n## Notes\n- Created by: openclaw recipes dispatch\n`;
+  const inboxMd = `# Inbox — ${teamId}\n\nReceived: ${receivedIso}\n\n## Request\n${requestText}\n\n## Proposed work\n- Ticket: ${ticketNumStr}-${baseSlug}\n- Owner: ${owner}\n\n## Links\n- Ticket: ${path.relative(teamDir, ticketPath)}\n`;
+  const ticketMd = `# ${ticketNumStr}-${baseSlug}\n\nCreated: ${receivedIso}\nOwner: ${owner}\nStatus: queued\nInbox: ${path.relative(teamDir, inboxPath)}\n\n## Context\n${requestText}\n\n## Requirements\n- (fill in)\n\n## Acceptance criteria\n- (fill in)\n\n## Tasks\n- [ ] (fill in)\n\n## Comments\n- (use this section for @mentions, questions, decisions, and dated replies)\n`;
   const plan = {
     teamId,
     request: requestText,
     files: [
       { path: inboxPath, kind: "inbox", summary: title },
       { path: ticketPath, kind: "backlog-ticket", summary: title },
-      { path: assignmentPath, kind: "assignment", summary: owner },
     ],
   };
   if (options.dryRun) return { ok: true as const, plan };
   await ensureDir(inboxDir);
   await ensureDir(backlogDir);
-  await ensureDir(assignmentsDir);
   await writeFileSafely(inboxPath, inboxMd, "createOnly");
   await writeFileSafely(ticketPath, ticketMd, "createOnly");
-  await writeFileSafely(assignmentPath, assignmentMd, "createOnly");
   let nudgeQueued = false;
   try {
     const leadAgentId = `${teamId}-lead`;
@@ -312,7 +281,8 @@ export async function handleDispatch(
         `Dispatch created new intake for team: ${teamId}`,
         `- Inbox: ${path.relative(teamDir, inboxPath)}`,
         `- Backlog: ${path.relative(teamDir, ticketPath)}`,
-        `- Assignment: ${path.relative(teamDir, assignmentPath)}`,
+        // Assignment stubs are deprecated; no assignment artifact is created.
+
         `Action: please triage/normalize the ticket (fill Requirements/AC/tasks) and move it through the workflow.`,
       ].join("\n"),
       { sessionKey: `agent:${leadAgentId}:main` },
@@ -326,58 +296,6 @@ export async function handleDispatch(
   return { ok: true as const, wrote: plan.files.map((f) => f.path), nudgeQueued };
 }
 
-/**
- * Cleanup assignment stubs for tickets that are already closed (in work/done).
- *
- * Why: some automation/board views treat assignment stubs as active work signals.
- * If a ticket is manually moved to done (outside `openclaw recipes move-ticket`),
- * its `work/assignments/<num>-assigned-*.md` stubs may linger and resurface the ticket.
- *
- * This command archives any matching assignment stubs into `work/assignments/archive/`.
- */
-export async function handleCleanupClosedAssignments(
-  api: OpenClawPluginApi,
-  options: { teamId: string; ticketNums?: string[] }
-): Promise<{ ok: true; teamId: string; archived: Array<{ from: string; to: string }> }> {
-  const teamId = String(options.teamId);
-  const { teamDir } = await resolveTeamContext(api, teamId);
-
-  const assignmentsDir = ticketStageDir(teamDir, "assignments");
-  const archiveDir = path.join(assignmentsDir, "archive");
-  const doneDir = ticketStageDir(teamDir, "done");
-
-  const archived: Array<{ from: string; to: string }> = [];
-  if (!(await fileExists(assignmentsDir))) return { ok: true, teamId, archived };
-  await ensureDir(archiveDir);
-
-  const ticketNumsFilter = Array.isArray(options.ticketNums) && options.ticketNums.length
-    ? new Set(options.ticketNums.map((n) => String(n).padStart(4, "0")))
-    : null;
-
-  const doneFiles = (await fileExists(doneDir)) ? await fs.readdir(doneDir) : [];
-  const doneNums = new Set(
-    doneFiles
-      .map((f) => f.match(/^([0-9]{4})-/)?.[1])
-      .filter((x): x is string => !!x)
-  );
-
-  const files = (await fs.readdir(assignmentsDir)).filter((f) => f.endsWith(".md"));
-  for (const f of files) {
-    if (f === "archive") continue;
-    if (f.startsWith("archive" + path.sep)) continue;
-    const m = f.match(/^([0-9]{4})-assigned-.*\.md$/);
-    if (!m) continue;
-    const num = m[1];
-    if (ticketNumsFilter && !ticketNumsFilter.has(num)) continue;
-
-    // If the ticket number is present in done/, this assignment is considered closed.
-    if (!doneNums.has(num)) continue;
-
-    const from = path.join(assignmentsDir, f);
-    const to = path.join(archiveDir, f);
-    await fs.rename(from, to);
-    archived.push({ from, to });
-  }
-
-  return { ok: true, teamId, archived };
-}
+// Assignment stubs are deprecated and preserved only via migration to work/assignments.__deprecated__/.
+// This handler is intentionally removed to avoid continuing stub semantics.
+// (If you need to clean up legacy stubs, do it via a one-time migration script.)

package/src/lib/ticket-workflow.ts

@@ -1,19 +1,12 @@
 import fs from "node:fs/promises";
 import path from "node:path";
 
-import { fileExists } from "./fs-utils";
 import { ensureLaneDir } from "./lanes";
-import { DEFAULT_TICKET_NUMBER } from "./constants";
 import { findTicketFile as findTicketFileFromFinder } from "./ticket-finder";
-import { parseTicketFilename } from "./ticket-finder";
-
-async function ensureDir(p: string) {
-  await fs.mkdir(p, { recursive: true });
-}
 
 function patchTicketFields(
   md: string,
-  opts: { ownerSafe: string; status: string; assignmentRel: string }
+  opts: { ownerSafe: string; status: string }
 ): string {
   let out = md;
   if (out.match(/^Owner:\s.*$/m)) out = out.replace(/^Owner:\s.*$/m, `Owner: ${opts.ownerSafe}`);
@@ -22,9 +15,7 @@ function patchTicketFields(
   if (out.match(/^Status:\s.*$/m)) out = out.replace(/^Status:\s.*$/m, `Status: ${opts.status}`);
   else out = out.replace(/^(# .+\n)/, `$1\nStatus: ${opts.status}\n`);
 
-  if (out.match(/^Assignment:\s.*$/m)) out = out.replace(/^Assignment:\s.*$/m, `Assignment: ${opts.assignmentRel}`);
-  else out = out.replace(/^Owner:.*$/m, (line) => `${line}\nAssignment: ${opts.assignmentRel}`);
-
+  // Assignment stubs are deprecated; do not write/update Assignment: here.
   return out;
 }
 
@@ -47,34 +38,20 @@ export async function takeTicket(opts: { teamDir: string; ticket: string; owner?
   const filename = path.basename(srcPath);
   const destPath = path.join(inProgressDir, filename);
 
-  const parsed = parseTicketFilename(filename) ?? { ticketNumStr: opts.ticket.match(/^\d{4}$/) ? opts.ticket : DEFAULT_TICKET_NUMBER, slug: "ticket" };
-  const { ticketNumStr, slug } = parsed;
-
-  const assignmentsDir = path.join(teamDir, 'work', 'assignments');
-  await ensureDir(assignmentsDir);
-  const assignmentPath = path.join(assignmentsDir, `${ticketNumStr}-assigned-${ownerSafe}.md`);
-  const assignmentRel = path.relative(teamDir, assignmentPath);
+  // Previously parsed filename for assignment-stub ids; no longer needed.
 
   const alreadyInProgress = srcPath === destPath;
 
   const md = await fs.readFile(srcPath, 'utf8');
-  const nextMd = patchTicketFields(md, { ownerSafe, status: 'in-progress', assignmentRel });
+  const nextMd = patchTicketFields(md, { ownerSafe, status: 'in-progress' });
   await fs.writeFile(srcPath, nextMd, 'utf8');
 
   if (!alreadyInProgress) {
     await fs.rename(srcPath, destPath);
   }
 
-  const assignmentMd = `# Assignment — ${ticketNumStr}-${slug}\n\nAssigned: ${ownerSafe}\n\n## Ticket\n${path.relative(teamDir, destPath)}\n\n## Notes\n- Created by: openclaw recipes take\n`;
-
-  const assignmentExists = await fileExists(assignmentPath);
-  if (assignmentExists && !opts.overwriteAssignment) {
-    // createOnly
-  } else {
-    await fs.writeFile(assignmentPath, assignmentMd, 'utf8');
-  }
-
-  return { srcPath, destPath, moved: !alreadyInProgress, assignmentPath };
+  // Assignment stubs are deprecated; do not create/update work/assignments/*.md.
+  return { srcPath, destPath, moved: !alreadyInProgress };
 }
 
 export async function handoffTicket(opts: { teamDir: string; ticket: string; tester?: string; overwriteAssignment: boolean }) {
@@ -91,32 +68,18 @@ export async function handoffTicket(opts: { teamDir: string; ticket: string; tes
   const filename = path.basename(srcPath);
   const destPath = path.join(testingDir, filename);
 
-  const parsed = parseTicketFilename(filename) ?? { ticketNumStr: opts.ticket.match(/^\d{4}$/) ? opts.ticket : DEFAULT_TICKET_NUMBER, slug: "ticket" };
-  const { ticketNumStr, slug } = parsed;
-
-  const assignmentsDir = path.join(teamDir, 'work', 'assignments');
-  await ensureDir(assignmentsDir);
-  const assignmentPath = path.join(assignmentsDir, `${ticketNumStr}-assigned-${testerSafe}.md`);
-  const assignmentRel = path.relative(teamDir, assignmentPath);
+  // Previously parsed filename for assignment-stub ids; no longer needed.
 
   const alreadyInTesting = srcPath === destPath;
 
   const md = await fs.readFile(srcPath, 'utf8');
-  const nextMd = patchTicketFields(md, { ownerSafe: testerSafe, status: 'testing', assignmentRel });
+  const nextMd = patchTicketFields(md, { ownerSafe: testerSafe, status: 'testing' });
   await fs.writeFile(srcPath, nextMd, 'utf8');
 
   if (!alreadyInTesting) {
     await fs.rename(srcPath, destPath);
   }
 
-  const assignmentMd = `# Assignment — ${ticketNumStr}-${slug}\n\nAssigned: ${testerSafe}\n\n## Ticket\n${path.relative(teamDir, destPath)}\n\n## Notes\n- Created by: openclaw recipes handoff\n`;
-
-  const assignmentExists = await fileExists(assignmentPath);
-  if (assignmentExists && !opts.overwriteAssignment) {
-    // createOnly: leave as-is
-  } else {
-    await fs.writeFile(assignmentPath, assignmentMd, 'utf8');
-  }
-
-  return { srcPath, destPath, moved: !alreadyInTesting, assignmentPath };
+  // Assignment stubs are deprecated; do not create/update work/assignments/*.md.
+  return { srcPath, destPath, moved: !alreadyInTesting };
 }