crewly 1.20.153 → 1.20.154

package/config/skills/agent/core/project-tickets/execute.sh

@@ -0,0 +1,182 @@
+#!/bin/bash
+# =============================================================================
+# project-tickets — a project's own backlog (<project>/.crewly/tickets/)
+#
+# Backed by /api/project-tickets (specs/2026-09-28-project-tickets.md §6).
+#
+# Usage:
+#   bash execute.sh list    [--project P] [--status ready]
+#   bash execute.sh show    --project P --id APP-12
+#   bash execute.sh create  --project P --title "…" [--description "…"] [--acceptance "…" …]
+#                           [--priority P1] [--labels a,b] [--team <teamId>] [--status ready]
+#                           [--source request:TKT-012] [--request-id <id>] [--owner-review]
+#   bash execute.sh update  --project P --id APP-12 [--title …] [--priority …] [--labels …]
+#                           [--description …] [--acceptance "…" …] [--status …] [--note "…"]
+#   bash execute.sh claim   --project P --id APP-12
+#   bash execute.sh release --project P --id APP-12 [--note "why"]
+#   bash execute.sh assign  --project P --id APP-12 --to <session> [--no-start]   (owner / orc / lead)
+#   bash execute.sh log     --project P --id APP-12 --note "progress"
+#   bash execute.sh '{"action":"create","project":"P","title":"…"}'
+#
+# P = project id, name, or absolute path.
+# =============================================================================
+set -euo pipefail
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+source "${SCRIPT_DIR}/../../_common/lib.sh"
+
+print_usage() {
+  cat <<'EOF_USAGE'
+Usage:
+  bash execute.sh list    [--project P] [--status ready]      Tickets of a project (or of all your teams' projects)
+  bash execute.sh show    --project P --id APP-12             One ticket with its full body
+  bash execute.sh create  --project P --title "…" [--description "…"] [--acceptance "…" …]
+                          [--priority P0-P3] [--labels a,b] [--team <teamId>] [--status backlog|ready]
+                          [--source request:TKT-012] [--request-id <id>] [--owner-review]
+  bash execute.sh update  --project P --id APP-12 [--title …] [--priority …] [--labels …]
+                          [--description …] [--acceptance "…" …] [--status …] [--note "…"]
+  bash execute.sh claim   --project P --id APP-12             Take a ready ticket (creates your WorkItem)
+  bash execute.sh release --project P --id APP-12 [--note …]  Give your ticket back (→ ready)
+  bash execute.sh assign  --project P --id APP-12 --to <session> [--no-start]
+                                                              Orchestrator / team lead: put someone on it
+  bash execute.sh log     --project P --id APP-12 --note "…"  Add a progress note to the ticket's Log
+
+P = project id, name or absolute path. Workers' new tickets start in backlog;
+the owner, the orchestrator or a team lead makes them ready.
+EOF_USAGE
+}
+
+ACTION=""; PROJECT=""; ID=""; TITLE=""; DESCRIPTION=""; PRIORITY=""; LABELS=""; TEAM=""
+STATUS=""; SOURCE=""; REQUEST_ID=""; NOTE=""; OWNER_REVIEW=""; ASSIGNEE=""; START="true"
+ACCEPTANCE_JSON="null"
+HAS_DESCRIPTION=0
+
+if [[ $# -gt 0 && ${1:0:1} == '{' ]]; then
+  J="$1"; shift
+  ACTION=$(printf '%s' "$J" | jq -r '.action // empty')
+  PROJECT=$(printf '%s' "$J" | jq -r '.project // .projectPath // .projectId // empty')
+  ID=$(printf '%s' "$J" | jq -r '.id // .ticket // empty')
+  TITLE=$(printf '%s' "$J" | jq -r '.title // empty')
+  if printf '%s' "$J" | jq -e 'has("description")' >/dev/null; then
+    DESCRIPTION=$(printf '%s' "$J" | jq -r '.description // ""'); HAS_DESCRIPTION=1
+  fi
+  PRIORITY=$(printf '%s' "$J" | jq -r '.priority // empty')
+  LABELS=$(printf '%s' "$J" | jq -r 'if (.labels|type) == "array" then (.labels|join(",")) else (.labels // empty) end')
+  TEAM=$(printf '%s' "$J" | jq -r '.team // empty')
+  STATUS=$(printf '%s' "$J" | jq -r '.status // empty')
+  SOURCE=$(printf '%s' "$J" | jq -r '.source // empty')
+  REQUEST_ID=$(printf '%s' "$J" | jq -r '.requestId // empty')
+  NOTE=$(printf '%s' "$J" | jq -r '.note // empty')
+  OWNER_REVIEW=$(printf '%s' "$J" | jq -r 'if .ownerReview == true then "true" else empty end')
+  ACCEPTANCE_JSON=$(printf '%s' "$J" | jq -c 'if (.acceptance|type) == "array" then .acceptance else null end')
+  ASSIGNEE=$(printf '%s' "$J" | jq -r '.to // .assignee // empty')
+  START=$(printf '%s' "$J" | jq -r 'if .start == false then "false" else "true" end')
+fi
+if [[ -z "$ACTION" && $# -gt 0 && ${1:0:1} != '-' ]]; then ACTION="$1"; shift; fi
+
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+    --project|-p)    [ $# -ge 2 ] || error_exit "--project requires a value";     PROJECT="$2"; shift 2 ;;
+    --id|--ticket)   [ $# -ge 2 ] || error_exit "--id requires a value";          ID="$2"; shift 2 ;;
+    --title)         [ $# -ge 2 ] || error_exit "--title requires a value";       TITLE="$2"; shift 2 ;;
+    --description)   [ $# -ge 2 ] || error_exit "--description requires a value"; DESCRIPTION="$2"; HAS_DESCRIPTION=1; shift 2 ;;
+    --acceptance)    [ $# -ge 2 ] || error_exit "--acceptance requires a value"
+                     ACCEPTANCE_JSON=$(jq -c --arg a "$2" 'if . == null then [$a] else . + [$a] end' <<<"$ACCEPTANCE_JSON"); shift 2 ;;
+    --priority)      [ $# -ge 2 ] || error_exit "--priority requires a value";    PRIORITY="$2"; shift 2 ;;
+    --labels)        [ $# -ge 2 ] || error_exit "--labels requires a value";      LABELS="$2"; shift 2 ;;
+    --team)          [ $# -ge 2 ] || error_exit "--team requires a value";        TEAM="$2"; shift 2 ;;
+    --status)        [ $# -ge 2 ] || error_exit "--status requires a value";      STATUS="$2"; shift 2 ;;
+    --source)        [ $# -ge 2 ] || error_exit "--source requires a value";      SOURCE="$2"; shift 2 ;;
+    --request-id)    [ $# -ge 2 ] || error_exit "--request-id requires a value";  REQUEST_ID="$2"; shift 2 ;;
+    --note)          [ $# -ge 2 ] || error_exit "--note requires a value";        NOTE="$2"; shift 2 ;;
+    --owner-review)  OWNER_REVIEW="true"; shift ;;
+    --to|--assignee) [ $# -ge 2 ] || error_exit "--to requires a value";         ASSIGNEE="$2"; shift 2 ;;
+    --no-start)      START="false"; shift ;;
+    --full)          shift ;;
+    --help|-h)       print_usage; exit 0 ;;
+    *) error_exit "Unknown option: $1" ;;
+  esac
+done
+
+[ -n "$ACTION" ] || { print_usage >&2; error_exit "Missing action: list | show | create | update | claim | release | assign | log"; }
+
+# URL-encode a path segment (project paths contain slashes).
+enc() { jq -rn --arg v "$1" '$v|@uri'; }
+
+# Compact one-line-per-ticket view for lists.
+TICKET_ROW='{id, title, status, priority, assignee, labels, workItemId}'
+
+case "$ACTION" in
+  list)
+    if [ -n "$PROJECT" ]; then
+      QS=""; [ -n "$STATUS" ] && QS="?status=$(enc "$STATUS")"
+      api_call GET "/project-tickets/$(enc "$PROJECT")${QS}" | jq "{success, project: .data.project, tickets: [.data.tickets[] | ${TICKET_ROW}], invalid: .data.invalid}"
+    else
+      QS=""; [ -n "$STATUS" ] && QS="?status=$(enc "$STATUS")"
+      api_call GET "/project-tickets${QS}" | jq "{success, projects: [.data[] | {project, tickets: [.tickets[] | ${TICKET_ROW}]}]}"
+    fi
+    ;;
+  show)
+    require_param "project" "$PROJECT"; require_param "id" "$ID"
+    api_call GET "/project-tickets/$(enc "$PROJECT")/$(enc "$ID")" | jq '{success, ticket: (.data | del(.extra))}'
+    ;;
+  create)
+    require_param "project" "$PROJECT"; require_param "title" "$TITLE"
+    BODY=$(jq -n --arg title "$TITLE" --arg description "$DESCRIPTION" --argjson hasDescription "$HAS_DESCRIPTION" \
+      --arg priority "$PRIORITY" --arg labels "$LABELS" --arg team "$TEAM" --arg status "$STATUS" \
+      --arg source "$SOURCE" --arg requestId "$REQUEST_ID" --arg ownerReview "$OWNER_REVIEW" --argjson acceptance "$ACCEPTANCE_JSON" \
+      '{title: $title}
+       + (if $hasDescription == 1 then {description: $description} else {} end)
+       + (if $acceptance != null then {acceptance: $acceptance} else {} end)
+       + (if $priority != "" then {priority: $priority} else {} end)
+       + (if $labels != "" then {labels: ($labels | split(",") | map(gsub("^\\s+|\\s+$"; "")))} else {} end)
+       + (if $team != "" then {team: $team} else {} end)
+       + (if $status != "" then {status: $status} else {} end)
+       + (if $source != "" then {source: $source} else {} end)
+       + (if $requestId != "" then {requestId: $requestId} else {} end)
+       + (if $ownerReview == "true" then {ownerReview: true} else {} end)')
+    api_call POST "/project-tickets/$(enc "$PROJECT")" "$BODY" | jq "{success, ticket: (.data | ${TICKET_ROW} + {fileName})}"
+    ;;
+  update)
+    require_param "project" "$PROJECT"; require_param "id" "$ID"
+    BODY=$(jq -n --arg title "$TITLE" --arg description "$DESCRIPTION" --argjson hasDescription "$HAS_DESCRIPTION" \
+      --arg priority "$PRIORITY" --arg labels "$LABELS" --arg team "$TEAM" --arg status "$STATUS" \
+      --arg note "$NOTE" --arg ownerReview "$OWNER_REVIEW" --argjson acceptance "$ACCEPTANCE_JSON" \
+      '{}
+       + (if $title != "" then {title: $title} else {} end)
+       + (if $hasDescription == 1 then {description: $description} else {} end)
+       + (if $acceptance != null then {acceptance: $acceptance} else {} end)
+       + (if $priority != "" then {priority: $priority} else {} end)
+       + (if $labels != "" then {labels: ($labels | split(",") | map(gsub("^\\s+|\\s+$"; "")))} else {} end)
+       + (if $team != "" then {team: $team} else {} end)
+       + (if $status != "" then {status: $status} else {} end)
+       + (if $note != "" then {note: $note} else {} end)
+       + (if $ownerReview == "true" then {ownerReview: true} else {} end)')
+    [ "$BODY" != "{}" ] || error_exit "Nothing to update"
+    api_call POST "/project-tickets/$(enc "$PROJECT")/$(enc "$ID")/update" "$BODY" | jq "{success, ticket: (.data | ${TICKET_ROW})}"
+    ;;
+  claim)
+    require_param "project" "$PROJECT"; require_param "id" "$ID"
+    api_call POST "/project-tickets/$(enc "$PROJECT")/$(enc "$ID")/claim" '{}' \
+      | jq "{success, claimed: .data.claimed, workItemId: .data.workItem.id, ticket: (.data.ticket | ${TICKET_ROW}),
+             next: \"Do the work; when done, complete this WorkItem (complete-task / report-status with workItemId). The ticket closes when it is verified.\"}"
+    ;;
+  release)
+    require_param "project" "$PROJECT"; require_param "id" "$ID"
+    BODY=$(jq -n --arg note "$NOTE" '{status: "ready"} + (if $note != "" then {note: $note} else {} end)')
+    api_call POST "/project-tickets/$(enc "$PROJECT")/$(enc "$ID")/transition" "$BODY" | jq "{success, ticket: (.data | ${TICKET_ROW})}"
+    ;;
+  assign)
+    require_param "project" "$PROJECT"; require_param "id" "$ID"; require_param "to" "$ASSIGNEE"
+    BODY=$(jq -n --arg to "$ASSIGNEE" --argjson start "$START" '{assignee: $to, start: $start}')
+    api_call POST "/project-tickets/$(enc "$PROJECT")/$(enc "$ID")/assign" "$BODY" \
+      | jq "{success, workItemId: (.data.workItem.id // null), ticket: (.data.ticket | ${TICKET_ROW})}"
+    ;;
+  log)
+    require_param "project" "$PROJECT"; require_param "id" "$ID"; require_param "note" "$NOTE"
+    BODY=$(jq -n --arg note "$NOTE" '{note: $note}')
+    api_call POST "/project-tickets/$(enc "$PROJECT")/$(enc "$ID")/log" "$BODY" | jq '{success, lastLog: (.data.log | last)}'
+    ;;
+  *)
+    error_exit "Unknown action: $ACTION (use list | show | create | update | claim | release | assign | log)"
+    ;;
+esac

package/config/skills/agent/core/project-tickets/execute.test.sh

@@ -0,0 +1,116 @@
+#!/bin/bash
+# Tests for project-tickets — run with: bash execute.test.sh
+# A python HTTP stub plays the backend; asserts the calls and the output.
+set -u
+HERE="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+EXEC="$HERE/execute.sh"
+PASS=0; FAIL=0
+check() {
+  local name="$1"; local got="$2"; local want="$3"
+  if [ "$got" = "$want" ]; then PASS=$((PASS+1)); else FAIL=$((FAIL+1)); echo "FAIL: $name"; echo "  want: $want"; echo "  got:  $got"; fi
+}
+
+PORT=18841
+STUB_LOG="$(mktemp)"; export STUB_LOG; export STUB_PORT="$PORT"
+python3 -u <<'PY' >/dev/null 2>&1 &
+from http.server import BaseHTTPRequestHandler, HTTPServer
+import json, os
+LOG = os.environ['STUB_LOG']; PORT = int(os.environ['STUB_PORT'])
+T = {'id': 'APP-1', 'title': 'Export CSV', 'status': 'ready', 'priority': 'P1', 'assignee': None,
+     'labels': ['ui'], 'workItemId': None, 'fileName': 'APP-1-export-csv.md', 'extra': {'x': 1},
+     'log': ['a · owner · created', 'b · dev · note']}
+class H(BaseHTTPRequestHandler):
+    def reply(self, code, obj):
+        self.send_response(code); self.send_header('Content-Type', 'application/json'); self.end_headers()
+        self.wfile.write(json.dumps(obj).encode())
+    def record(self, body=None):
+        open(LOG, 'w').write(json.dumps({'method': self.command, 'path': self.path, 'body': body, 'agent': self.headers.get('X-Agent-Session')}))
+    def do_GET(self):
+        self.record()
+        if self.path.startswith('/api/project-tickets/'):
+            if self.path.count('/') >= 4 and not self.path.split('?')[0].endswith('%2Fapp'):
+                return self.reply(200, {'success': True, 'data': T})
+            return self.reply(200, {'success': True, 'data': {'project': {'id': 'p1'}, 'tickets': [T], 'invalid': []}})
+        if self.path.startswith('/api/project-tickets'):
+            return self.reply(200, {'success': True, 'data': [{'project': {'id': 'p1'}, 'tickets': [T]}]})
+        return self.reply(404, {'success': False, 'error': 'nope'})
+    def do_POST(self):
+        n = int(self.headers.get('content-length', '0')); body = json.loads(self.rfile.read(n).decode() or '{}')
+        self.record(body)
+        if self.path.endswith('/claim'):
+            return self.reply(200, {'success': True, 'data': {'claimed': True, 'workItem': {'id': 'wi-9'}, 'ticket': dict(T, status='in_progress', assignee='dev-ann', workItemId='wi-9')}})
+        if 'forbidden' in self.path:
+            return self.reply(403, {'success': False, 'error': 'Not allowed'})
+        return self.reply(200, {'success': True, 'data': T})
+    def log_message(self, *a, **k): pass
+HTTPServer(('127.0.0.1', PORT), H).serve_forever()
+PY
+STUB_PID=$!
+disown "$STUB_PID" 2>/dev/null || true
+trap 'kill "$STUB_PID" >/dev/null 2>&1; rm -f "$STUB_LOG"' EXIT
+for i in $(seq 1 30); do curl -s -o /dev/null -m 0.2 "http://127.0.0.1:${PORT}/" 2>/dev/null && break; sleep 0.1; done
+
+run() { CREWLY_SESSION_NAME=dev-ann CREWLY_API_URL="http://127.0.0.1:${PORT}" bash "$EXEC" "$@" 2>/dev/null; }
+run_err() { CREWLY_SESSION_NAME=dev-ann CREWLY_API_URL="http://127.0.0.1:${PORT}" bash "$EXEC" "$@" 2>&1 >/dev/null; }
+last() { jq -r "$1" "$STUB_LOG"; }
+
+# --- list: one project (path is URL-encoded), and across my teams' projects ---
+OUT=$(run list --project /work/app --status ready)
+check "list: path" "$(last .path)" "/api/project-tickets/%2Fwork%2Fapp?status=ready"
+check "list: rows" "$(printf '%s' "$OUT" | jq -c '[.tickets[] | [.id, .status]]')" '[["APP-1","ready"]]'
+check "list: agent header" "$(last .agent)" "dev-ann"
+OUT=$(run list)
+check "list mine: path" "$(last .path)" "/api/project-tickets"
+check "list mine: grouped" "$(printf '%s' "$OUT" | jq -c '[.projects[] | .project.id]')" '["p1"]'
+
+# --- show drops the extra frontmatter noise ---
+OUT=$(run show --project p1 --id APP-1)
+check "show: path" "$(last .path)" "/api/project-tickets/p1/APP-1"
+check "show: no extra" "$(printf '%s' "$OUT" | jq -c '.ticket | has("extra")')" "false"
+
+# --- create with repeated acceptance and labels ---
+run create --project p1 --title "Export CSV" --description "why" --acceptance "header row" --acceptance "opens in Excel" --priority P1 --labels "ui, export" --source request:TKT-012 >/dev/null
+check "create: path" "$(last .path)" "/api/project-tickets/p1"
+check "create: body" "$(last '.body | tostring')" '{"title":"Export CSV","description":"why","acceptance":["header row","opens in Excel"],"priority":"P1","labels":["ui","export"],"source":"request:TKT-012"}'
+
+# --- create from JSON input ---
+run '{"action":"create","project":"p1","title":"From JSON","status":"ready","ownerReview":true}' >/dev/null
+check "create json: body" "$(last '.body | tostring')" '{"title":"From JSON","status":"ready","ownerReview":true}'
+
+# --- update only sends what was given ---
+run update --project p1 --id APP-1 --priority P0 --status ready --note groomed >/dev/null
+check "update: path" "$(last .path)" "/api/project-tickets/p1/APP-1/update"
+check "update: body" "$(last '.body | tostring')" '{"priority":"P0","status":"ready","note":"groomed"}'
+check "update: nothing" "$(run_err update --project p1 --id APP-1 | grep -c 'Nothing to update')" "1"
+
+# --- claim returns the WorkItem id and what to do next ---
+OUT=$(run claim --project p1 --id APP-1)
+check "claim: path" "$(last .path)" "/api/project-tickets/p1/APP-1/claim"
+check "claim: output" "$(printf '%s' "$OUT" | jq -c '[.claimed, .workItemId, .ticket.status]')" '[true,"wi-9","in_progress"]'
+
+# --- release = transition to ready ---
+run release --project p1 --id APP-1 --note "blocked on design" >/dev/null
+check "release: path" "$(last .path)" "/api/project-tickets/p1/APP-1/transition"
+check "release: body" "$(last '.body | tostring')" '{"status":"ready","note":"blocked on design"}'
+
+# --- assign (orchestrator / lead) ---
+run assign --project p1 --id APP-1 --to dev-bo >/dev/null
+check "assign: path" "$(last .path)" "/api/project-tickets/p1/APP-1/assign"
+check "assign: body" "$(last '.body | tostring')" '{"assignee":"dev-bo","start":true}'
+run '{"action":"assign","project":"p1","id":"APP-1","to":"Steve","start":false}' >/dev/null
+check "assign json no-start" "$(last '.body | tostring')" '{"assignee":"Steve","start":false}'
+check "assign: missing to" "$(run_err assign --project p1 --id APP-1 | grep -c 'to')" "1"
+
+# --- log ---
+OUT=$(run log --project p1 --id APP-1 --note "halfway")
+check "log: body" "$(last '.body | tostring')" '{"note":"halfway"}'
+check "log: last line" "$(printf '%s' "$OUT" | jq -r .lastLog)" "b · dev · note"
+
+# --- errors ---
+check "missing action" "$(run_err | grep -c 'Missing action')" "1"
+check "unknown action" "$(run_err explode | grep -c 'Unknown action')" "1"
+check "missing project" "$(run_err show --id APP-1 | grep -c 'project')" "1"
+check "server 403 surfaces" "$(run_err claim --project forbidden --id APP-1 >/dev/null; CREWLY_SESSION_NAME=dev-ann CREWLY_API_URL="http://127.0.0.1:${PORT}" bash "$EXEC" log --project forbidden --id APP-1 --note x 2>&1 >/dev/null | grep -c '403')" "1"
+
+echo "project-tickets: ${PASS} passed, ${FAIL} failed"
+[ "$FAIL" -eq 0 ]

package/config/skills/agent/core/recall/SKILL.md

@@ -55,7 +55,7 @@ Retrieve stored memories relevant to a given context or query. Use this to look
 | `--context` / `-c` | `context` | Yes | Search query (or pipe via stdin) |
 | `--scope` / `-s` | `scope` | No | Filter: `project`, `team`, or `global` |
 | `--limit` / `-l` | `limit` | No | Max number of results |
-| `--project` / `-p` | `projectPath` | No | Filter by project path |
+| `--project` / `-p` | `projectPath` | No | Filter by project path (default: `$CREWLY_PROJECT_PATH` when set) |
 
 ## Examples — CLI Flags (preferred)
 

package/config/skills/agent/core/recall/execute.sh

@@ -83,6 +83,13 @@ if [ -n "$INPUT_JSON" ]; then
   [ -z "$PROJECT_PATH" ] && PROJECT_PATH=$(printf '%s' "$INPUT" | jq -r '.projectPath // empty')
 fi
 
+# #816: Auto-inject projectPath from CREWLY_PROJECT_PATH when not provided,
+# the same default remember applies (#187). Without it a recall from an agent
+# shell searches agent memory only and never sees the project's knowledge.
+if [ -z "$PROJECT_PATH" ] && [ -n "${CREWLY_PROJECT_PATH:-}" ]; then
+  PROJECT_PATH="$CREWLY_PROJECT_PATH"
+fi
+
 require_param "agentId (--agent)" "$AGENT_ID"
 require_param "context (--context)" "$CONTEXT"
 

package/config/skills/agent/core/report-status/SKILL.md

@@ -41,7 +41,7 @@ execution:
 
 Proactively notify the orchestrator when a task is done, blocked, or failed. Use this skill to keep the orchestrator informed without waiting for a scheduled check-in.
 
-When `status` is `done` and a `taskPath` is provided, the task file is automatically moved from `in_progress/` to `done/` in the project's `.crewly/tasks/` directory.
+Pass the `workItemId` of the work you are reporting on (from your `[CREWLY-DISPATCH]` notice or `get-my-tasks`) so the right WorkItem is closed. When that WorkItem belongs to a project ticket, the ticket follows it (done once the WorkItem is verified). The old `taskPath` field is still accepted but no longer moves any file.
 
 ## Parameters
 
@@ -52,6 +52,7 @@ When `status` is `done` and a `taskPath` is provided, the task file is automatic
 | `--summary` / `-m` | `summary` | Yes | Brief description (or pipe via stdin) |
 | `--summary-file` | — | No | Read summary from a file path |
 | `--project` / `-p` | `projectPath` | No | Project path for auto-remember on completion |
+| `--role` / `-r` | `role` | No | Agent role recorded with the learning (default: `$CREWLY_ROLE`, else `agent`) |
 | `--task-path` | `taskPath` | No | Task file path; auto-moves to `done/` on completion |
 | `--task-id` | `taskId` | No | Task ID (for structured StatusReport format) |
 | `--progress` | `progress` | No | Progress percentage 0-100 |
@@ -112,7 +113,7 @@ bash execute.sh --session dev-1 --status done --summary-file /tmp/summary.txt --
 ## Examples — Legacy JSON (backward compatible)
 
 ```bash
-bash execute.sh '{"sessionName":"dev-1","status":"done","summary":"Finished implementing auth module","taskPath":"/path/.crewly/tasks/delegated/in_progress/implement_auth_1234.md"}'
+bash execute.sh '{"sessionName":"dev-1","status":"done","summary":"Finished implementing auth module","workItemId":"<your WorkItem id>"}'
 ```
 
 ## Output

package/config/skills/agent/core/report-status/execute.sh

@@ -31,6 +31,7 @@ Options:
   --summary  | -m   Status summary text (required unless piped via stdin)
   --summary-file    Read summary from file path
   --project  | -p   Project path for auto-remember on completion
+  --role     | -r   Agent role for the recorded learning (default: $CREWLY_ROLE, else "agent")
   --task-path       Task file path to auto-complete
   --task-id         Task ID (for structured StatusReport format)
   --progress        Progress percentage (0-100, for structured format)
@@ -45,6 +46,7 @@ SESSION_NAME=""
 STATUS=""
 SUMMARY=""
 PROJECT_PATH=""
+AGENT_ROLE=""
 TASK_PATH=""
 TASK_ID=""
 WORK_ITEM_ID=""
@@ -79,6 +81,10 @@ while [[ $# -gt 0 ]]; do
       PROJECT_PATH="$2"
       shift 2
       ;;
+    --role|-r)
+      AGENT_ROLE="$2"
+      shift 2
+      ;;
     --task-path)
       TASK_PATH="$2"
       shift 2
@@ -143,6 +149,7 @@ if [ -n "$INPUT_JSON" ]; then
   [ -z "$WORK_ITEM_ID" ] && WORK_ITEM_ID=$(printf '%s' "$INPUT" | jq -r '.workItemId // empty')
   [ -z "$PROGRESS" ] && PROGRESS=$(printf '%s' "$INPUT" | jq -r '.progress // empty')
   [ -z "$PROJECT_PATH" ] && PROJECT_PATH=$(printf '%s' "$INPUT" | jq -r '.projectPath // empty')
+  [ -z "$AGENT_ROLE" ] && AGENT_ROLE=$(printf '%s' "$INPUT" | jq -r '.role // .agentRole // empty')
   ARTIFACTS=$(printf '%s' "$INPUT" | jq -c '.artifacts // empty')
   BLOCKERS=$(printf '%s' "$INPUT" | jq -c '.blockers // empty')
   USE_STRUCTURED=$(printf '%s' "$INPUT" | jq -r '.structured // "false"')
@@ -156,6 +163,11 @@ require_param "sessionName (--session)" "$SESSION_NAME"
 require_param "status (--status)" "$STATUS"
 require_param "summary (--summary)" "$SUMMARY"
 
+# The agent's role labels its learnings. Agent sessions carry it in
+# CREWLY_ROLE (set by the backend at session creation); /memory/record-learning
+# rejects an empty agentRole, so fall back to a generic label.
+[ -z "$AGENT_ROLE" ] && AGENT_ROLE="${CREWLY_ROLE:-agent}"
+
 # Gate: milestone summaries must carry a real "WHAT shipped + WHAT it means
 # for the owner" — see config/sops/common/mid-flight-milestone-surface.md
 # (issue #435). Reject anything that looks like "task done" boilerplate
@@ -297,6 +309,28 @@ if [ "$STATUS" = "done" ]; then
   fi
 fi
 
+# record_task_learning <label> <summary>
+#
+# Records the report as a project learning via POST /memory/record-learning.
+# The controller requires {agentId, agentRole, projectPath, learning}; this
+# call used to send {agentId, content, type}, so every one 400'd and the
+# `2>/dev/null || true` hid it (#816). A failure is still non-fatal (the
+# status report itself already went out) but is now surfaced on stderr.
+record_task_learning() {
+  local label="$1" text="$2" body result
+  body=$(jq -n \
+    --arg agentId "$SESSION_NAME" \
+    --arg agentRole "$AGENT_ROLE" \
+    --arg projectPath "$PROJECT_PATH" \
+    --arg learning "${label}: ${text}" \
+    --arg relatedTask "${WORK_ITEM_ID:-$TASK_ID}" \
+    '{agentId: $agentId, agentRole: $agentRole, projectPath: $projectPath, learning: $learning}
+      + (if $relatedTask != "" then {relatedTask: $relatedTask} else {} end)')
+  if ! result=$(api_call POST "/memory/record-learning" "$body" 2>&1 >/dev/null); then
+    jq -n --arg err "$result" '{warning: "status reported, but recording the learning FAILED", error: $err}' >&2
+  fi
+}
+
 # Auto-persist key findings as project knowledge when task is done (#127, #219).
 if [ "$STATUS" = "done" ] && [ -n "$SUMMARY" ]; then
   auto_remember "$SESSION_NAME" "[COMPLETED] Task completed by ${SESSION_NAME}: ${SUMMARY}" "decision" "project" "$PROJECT_PATH"
@@ -306,14 +340,8 @@ fi
 # Uses existing APIs — no additional LLM calls. The agent's own summary
 # is the learning input; keyword extraction identifies growth areas.
 if [ "$STATUS" = "done" ] && [ -n "$SUMMARY" ] && [ -n "$PROJECT_PATH" ]; then
-  # Record as a structured learning entry (appends to what_worked.md)
-  LEARN_BODY=$(jq -n \
-    --arg agentId "$SESSION_NAME" \
-    --arg content "$SUMMARY" \
-    --arg projectPath "$PROJECT_PATH" \
-    --arg type "success" \
-    '{agentId: $agentId, content: $content, projectPath: $projectPath, type: $type}')
-  api_call POST "/memory/record-learning" "$LEARN_BODY" 2>/dev/null || true
+  # Record as a project learning (POST /memory/record-learning).
+  record_task_learning "Task completed" "$SUMMARY"
 
   # Extract growth areas from summary (keyword-based, no LLM)
   GROWTH_BODY=$(jq -n \
@@ -326,12 +354,6 @@ fi
 # Growth: record failures for learning when task fails or is blocked.
 if [ "$STATUS" = "failed" ] || [ "$STATUS" = "blocked" ]; then
   if [ -n "$SUMMARY" ] && [ -n "$PROJECT_PATH" ]; then
-    FAIL_BODY=$(jq -n \
-      --arg agentId "$SESSION_NAME" \
-      --arg content "$SUMMARY" \
-      --arg projectPath "$PROJECT_PATH" \
-      --arg type "failure" \
-      '{agentId: $agentId, content: $content, projectPath: $projectPath, type: $type}')
-    api_call POST "/memory/record-learning" "$FAIL_BODY" 2>/dev/null || true
+    record_task_learning "Task ${STATUS}" "$SUMMARY"
   fi
 fi

package/config/skills/agent/core/split-ticket/SKILL.md

@@ -0,0 +1,62 @@
+---
+name: Split Ticket
+description: Move an ask that landed in another ticket's discussion out into its own ticket, keeping the thread link.
+version: 1.0.0
+category: task-management
+skillType: claude-skill
+assignableRoles:
+  - developer
+  - qa
+  - tpm
+  - designer
+  - frontend-developer
+  - backend-developer
+  - fullstack-dev
+  - qa-engineer
+  - product-manager
+  - architect
+  - generalist
+  - sales
+  - support
+  - orchestrator
+  - team-leader
+triggers:
+  - split ticket
+  - new ask in thread
+  - separate ticket
+  - TKT
+tags:
+  - ticket
+  - intake
+execution:
+  type: script
+  script:
+    file: execute.sh
+    interpreter: bash
+    timeoutMs: 15000
+---
+
+# Split Ticket
+
+Every distinct ask from the owner should be its own ticket, so the owner can
+say it and move on. Intake opens a new ticket for a new ask said in an
+existing ticket's thread. When it is unsure, it **appends** instead, because a
+ticket per stray remark is worse than a missed split. If an ask landed in a
+ticket where it does not belong, split it out:
+
+```bash
+# See the ticket's follow-ups and their refs
+bash execute.sh --ticket TKT-039 --list
+
+# Move one of them out into its own ticket
+bash execute.sh --ticket TKT-039 --discussion-ref slackch-C0C2QCGE9K9-1790433421.1
+
+# Or open one from text (a pure information question: add --question)
+bash execute.sh --ticket TKT-039 --text "research how Opus makes video" --assignee <session>
+```
+
+- The new ticket stays in the same thread and records the old one as its
+  parent. Later follow-ups in that thread still find a ticket.
+- `--question` means a pure information question. It gets no acceptance step.
+- Split only a real new ask with its own deliverable. Answers, "好的",
+  clarifications and "send it as a PDF" belong to the ticket they were said in.

package/config/skills/agent/core/split-ticket/execute.sh

@@ -0,0 +1,87 @@
+#!/bin/bash
+# =============================================================================
+# split-ticket — move an ask out of a ticket into its own ticket (#827)
+#
+# Backed by GET /api/tickets/:tkt (list mode) and POST /api/tickets/:id/split
+# (specs/ticket-loop.md, "New asks in a thread").
+#
+# Usage:
+#   bash execute.sh --ticket TKT-039 --list
+#   bash execute.sh --ticket TKT-039 --discussion-ref <ref> [--title "…"] [--assignee <session>] [--question]
+#   bash execute.sh --ticket TKT-039 --text "research X" [--title "…"] [--assignee <session>] [--question]
+#   bash execute.sh '{"ticket":"TKT-039","discussionRef":"<ref>"}'
+# =============================================================================
+set -euo pipefail
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+source "${SCRIPT_DIR}/../../_common/lib.sh"
+
+print_usage() {
+  cat <<'EOF_USAGE'
+Usage:
+  bash execute.sh --ticket TKT-039 --list                       Show the ticket's follow-ups (with their refs)
+  bash execute.sh --ticket TKT-039 --discussion-ref <ref>       Move that follow-up out into its own ticket
+  bash execute.sh --ticket TKT-039 --text "the new ask"         Open a new ticket from text
+
+Options:
+  --ticket          TKT-039, 39 or the ticket id (required)
+  --list            List the follow-ups instead of splitting
+  --discussion-ref  ref of the follow-up to move out (from --list)
+  --text            Text of the new ask (when not moving a follow-up)
+  --title           Title for the new ticket
+  --assignee        Agent session that takes it (default: the source ticket's assignee)
+  --question        A pure information question: no acceptance step
+  --help | -h       Show this help
+EOF_USAGE
+}
+
+TICKET=""; LIST=""; DREF=""; TEXT=""; TITLE=""; ASSIGNEE=""; QUESTION="false"
+if [[ $# -gt 0 && ${1:0:1} == '{' ]]; then
+  TICKET=$(printf '%s' "$1" | jq -r '.ticket // .requestId // empty')
+  DREF=$(printf '%s' "$1" | jq -r '.discussionRef // empty')
+  TEXT=$(printf '%s' "$1" | jq -r '.text // empty')
+  TITLE=$(printf '%s' "$1" | jq -r '.title // empty')
+  ASSIGNEE=$(printf '%s' "$1" | jq -r '.assignee // empty')
+  QUESTION=$(printf '%s' "$1" | jq -r 'if .question == true then "true" else "false" end')
+  LIST=$(printf '%s' "$1" | jq -r 'if .list == true then "1" else empty end')
+  shift || true
+fi
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+    --ticket|--request-id) [ $# -ge 2 ] || error_exit "--ticket requires a value";         TICKET="$2";   shift 2 ;;
+    --list)                LIST="1"; shift ;;
+    --discussion-ref)      [ $# -ge 2 ] || error_exit "--discussion-ref requires a value"; DREF="$2";     shift 2 ;;
+    --text)                [ $# -ge 2 ] || error_exit "--text requires a value";           TEXT="$2";     shift 2 ;;
+    --title)               [ $# -ge 2 ] || error_exit "--title requires a value";          TITLE="$2";    shift 2 ;;
+    --assignee)            [ $# -ge 2 ] || error_exit "--assignee requires a value";       ASSIGNEE="$2"; shift 2 ;;
+    --question)            QUESTION="true"; shift ;;
+    --help|-h)             print_usage; exit 0 ;;
+    *) error_exit "Unknown option: $1" ;;
+  esac
+done
+
+require_param "ticket" "$TICKET"
+
+# List mode: the follow-ups, with the ref to pass to --discussion-ref.
+if [ -n "$LIST" ]; then
+  api_call GET "/tickets/${TICKET}" | jq '(.data.ticket // {}) as $t | {
+    tkt: (.data.board.tkt // null),
+    title: ($t.title // null),
+    followUps: [($t.discussion // [])[] | {ref, at, text: (.text | .[0:200])}]
+  } | . + {count: (.followUps | length)}'
+  exit 0
+fi
+
+[ -n "$DREF" ] || [ -n "$TEXT" ] || error_exit "Give --discussion-ref <ref> (move a follow-up out) or --text \"…\" (see --list)"
+
+BODY=$(jq -n --arg d "$DREF" --arg t "$TEXT" --arg title "$TITLE" --arg a "$ASSIGNEE" --argjson q "$QUESTION" \
+  '{question: $q}
+   + (if $d == "" then {} else {discussionRef: $d} end)
+   + (if $t == "" then {} else {text: $t} end)
+   + (if $title == "" then {} else {title: $title} end)
+   + (if $a == "" then {} else {assignee: $a} end)')
+api_call POST "/tickets/${TICKET}/split" "$BODY" | jq '{
+  success,
+  newTicket: (.data.ticket | if . then {id, ticketNumber, title, parentTicketId, threadRef: .origin.threadRef} else null end),
+  moved: (.data.moved // false),
+  error: (.error // null)
+}'