crewly 1.6.4 → 1.6.5

package/dist/backend/backend/src/controllers/task-pool/task-pool.controller.d.ts

@@ -36,11 +36,35 @@ export declare function maybeEnforceContract(body: Record<string, unknown>): Pro
 /**
  * Adds a WorkItem to the Task Pool.
  *
- * This is the HTTP entry point for the V3 pull-mode task path.
- * Used by delegate-task and other orchestration flows to queue
- * execution-ready WorkItems.
+ * HTTP entry point for the V3 pull-mode task path. Used by delegate-task
+ * and other orchestration shell skills to queue execution-ready WorkItems.
  *
- * Request body: a WorkItem object (id, type, owner, title, status='queued', etc.)
+ * Accepts two body shapes:
+ *
+ * 1. **Minimal `CreateWorkItemInput`** (preferred) — `{type, owner, title,
+ *    target?, description?, briefMarkdown?, priority?, requestId?, ...}`.
+ *    The server fills `id` (uuid), `status` (`'queued'` or `'blocked'`
+ *    if `dependsOn` set), `createdAt`, `retryCount=0`, `maxRetries`,
+ *    `inputTokens=0`, `outputTokens=0`, `cost=0`.
+ *
+ * 2. **Legacy full `WorkItem`** — body carries `id` AND `status` AND
+ *    `createdAt`. The server passes through unchanged (subject to the
+ *    same shape validation as before). Preserved for callers that
+ *    construct WIs locally with deterministic ids (e.g. break-down-request,
+ *    decomposition pipelines that need to record id-references upfront).
+ *
+ * **Why both shapes.** 2026-05-12 dogfood: orc-side `delegate-task` shell
+ * skill and 3 sibling skills (`agent/core/create-task`,
+ * `team-leader/decompose-goal`, `team-leader/delegate-task`) all send
+ * the minimal shape. They were silently 400'd by the strict validator
+ * because they omit `id` / `status` / `createdAt` / `retryCount` etc.
+ * Result: every orc delegation failed — `task-pool/add` returned
+ * `WorkItem.id is required and must be a string`, the skill exited 1,
+ * the orc thought it had delegated but no WI ever entered the pool.
+ *
+ * Fix the endpoint, not 4 skills — the canonical creation primitive
+ * `createWorkItem(input)` already exists for internal callers; the
+ * HTTP surface should expose the same shape.
  *
  * @param req - Express request with WorkItem body
  * @param res - Express response
@@ -89,12 +113,49 @@ export declare function releaseItem(req: Request, res: Response): Promise<void>;
 /**
  * Marks a running WorkItem as completed ('done').
  *
- * Request body:
+ * ## Canonical body shape (STRICT)
+ *
  * ```json
- * { "agentId": "crewly-product-leo-member-n", "tokenUsage": { ... }, "result": { ... } }
+ * {
+ *   "agentId": "crewly-product-leo-member-n",
+ *   "result": { "summary": "what I produced …", "prNumber": 525, "...": "..." },
+ *   "tokenUsage": { "inputTokens": 0, "outputTokens": 0, "totalCost": 0 }
+ * }
  * ```
  *
- * @param req - Express request with workItemId param
+ * Validation rules (in order — first failure short-circuits):
+ * 1. `workItemId` URL param — required (path; never empty).
+ * 2. `agentId` body field — required, non-empty string. Identifies the
+ *    completing agent for token attribution and audit trail.
+ * 3. `result.summary` body field — required, non-empty string after trim.
+ *    Enforces the Request Contract's Done Definition: workers report
+ *    what they produced (artifact, decision, verified result), not just
+ *    "done".
+ *
+ * Any field beyond `summary` inside `result` (e.g. `prNumber`, `links`)
+ * is preserved into `WorkItem.output` via spread merge, so downstream
+ * verifiers can read the proof-of-work without digging through chat logs.
+ *
+ * ## Hygiene #4 (PR ?) — strict-shape lock
+ *
+ * Prior to Hygiene #4, three skill callers (`config/skills/agent/core/{
+ * report-status,complete-task}/execute.sh`, `config/skills/orchestrator/
+ * complete-task/execute.sh`) and two TS callers (`tool-registry.ts`
+ * §complete_task + §report_status auto-complete) emitted top-level
+ * `{summary}` instead of the canonical `{agentId, result:{summary}}`.
+ * That shape failed both validators (missing agentId + result.summary),
+ * forcing every worker into a direct-curl workaround that Quinn
+ * surfaced on the #499 verify-WI dogfood.
+ *
+ * Decision (locked T+9min on Hygiene #4 by Quinn): keep the controller
+ * STRICT — fix all 5 callsites in the same PR rather than ship a
+ * backward-compat resolver. Rationale: all 5 callers are in-tree, OSS
+ * is the only consumer (Crewly Pro extends via Plugin System and does
+ * NOT fork core skill or task-pool callers per workspace CLAUDE.md),
+ * and a compat layer would add permanent ambiguity for zero downstream
+ * benefit.
+ *
+ * @param req - Express request with `workItemId` param + canonical body
  * @param res - Express response
  */
 export declare function completeItem(req: Request, res: Response): Promise<void>;
@@ -177,4 +238,95 @@ export declare function scanExpired(req: Request, res: Response): Promise<void>;
  * @param res - Express response
  */
 export declare function revokeAndRelease(req: Request, res: Response): Promise<void>;
+/**
+ * Returns every WorkItem currently in the pool — including cancelled,
+ * done_by_worker, failed, etc. — so admin / cleanup tooling can audit the
+ * full state without filtering by claimability.
+ *
+ * Distinct from `/api/task-pool` (and its `/all` alias) which return ONLY
+ * the available subset (queued + unclaimed). The existing alias was
+ * naming-confusing for cleanup workflows; rather than rename and break
+ * downstream consumers, this endpoint adds the audit-shaped surface that
+ * the bulk-DELETE script needs.
+ *
+ * Response: `{ success: true, data: WorkItem[], count }`.
+ *
+ * @param req - Express request
+ * @param res - Express response
+ */
+export declare function listAllItems(req: Request, res: Response): Promise<void>;
+/**
+ * Removes a WorkItem from the pool entirely. Powers the bulk-cleanup
+ * script and any future operator workflow that needs to drop stale or
+ * misplanned items.
+ *
+ * Request:
+ *   DELETE /api/task-pool/:workItemId[?force=1]
+ *
+ * Response shapes:
+ *   - 200 `{ success: true, removed: true,  workItem: WorkItem,  hadActiveClaim }`
+ *   - 200 `{ success: true, removed: false, reason: 'not_found' }`
+ *     (idempotent — repeated calls on a missing id do not error)
+ *   - 409 `{ success: false, error: '...', code: 'work_item_claimed',
+ *           workItemId, claimId, claimedBy }`
+ *     (active claim + no force; pass `?force=1` to override)
+ *   - 400 `{ success: false, error: 'workItemId param is required' }`
+ *
+ * @param req - Express request with `workItemId` route param and
+ *   optional `?force=1` query flag.
+ * @param res - Express response
+ */
+export declare function deleteItem(req: Request, res: Response): Promise<void>;
+/**
+ * Fetches a single WorkItem by id.
+ *
+ * Replaces `POST /task-management/read-task` and `POST /task-management/get-output`.
+ * Both v1 endpoints walked the project's `.crewly/tasks/` filesystem to load
+ * the `.md` body and `<id>.output.json`. With v1 retired, the entire
+ * task body, brief, and worker output live on the WorkItem itself —
+ * one fetch returns everything.
+ *
+ * @param req - Express request, `:workItemId` route param
+ * @param res - 200 `{ success, data: WorkItem }` | 404 if not found
+ */
+export declare function getItem(req: Request, res: Response): Promise<void>;
+/**
+ * Stores worker-supplied structured output on a WorkItem.
+ *
+ * Replaces v1's `<taskId>.output.json` filesystem write. The TL
+ * `verify-output` skill reads this back via `GET /api/task-pool/items/:id`.
+ *
+ * Body: `{ output: Record<string, unknown> }` — output may be any JSON shape;
+ * the schema is task-specific.
+ *
+ * @param req - Express request, `:workItemId` route param + body.output
+ * @param res - 200 `{ success, data: WorkItem }` | 404 | 400
+ */
+export declare function setItemOutput(req: Request, res: Response): Promise<void>;
+/**
+ * Reassigns a WorkItem to a different agent.
+ *
+ * Replaces `POST /task-management/handoff` (which moved the `.md` file
+ * between agent task directories). With v1 retired, handoff is a single
+ * `target` field flip on the WorkItem plus an audit-trail note.
+ *
+ * Body: `{ newTarget: string, fromAgent?: string, reason?: string }`.
+ *
+ * @param req - Express request
+ * @param res - 200 `{ success, data: WorkItem }` | 404 | 400 | 409 (terminal)
+ */
+export declare function handoffItem(req: Request, res: Response): Promise<void>;
+/**
+ * Appends a working-state note to a WorkItem.
+ *
+ * Replaces `POST /task-management/sync` and `POST /task-management/save-working-notes`,
+ * both of which appended progress lines to the `.md` task body. With v1
+ * retired, notes accumulate under `WorkItem.metadata.notes[]`.
+ *
+ * Body: `{ author: string, note: string }`.
+ *
+ * @param req - Express request
+ * @param res - 200 `{ success, data: WorkItem }` | 404 | 400
+ */
+export declare function appendItemNote(req: Request, res: Response): Promise<void>;
 //# sourceMappingURL=task-pool.controller.d.ts.map

package/dist/backend/backend/src/controllers/task-pool/task-pool.controller.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"task-pool.controller.d.ts","sourceRoot":"","sources":["../../../../../../backend/src/controllers/task-pool/task-pool.controller.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AA2DjD;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAsB,oBAAoB,CACxC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAC5B,OAAO,CACN,IAAI,GACJ;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;CAAE,CACvD,CAiEA;AAMD;;;;;;;;;;;GAWG;AACH,wBAAsB,OAAO,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA+CxE;AAMD;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAQ9E;AAMD;;;;;;;;;;;;;GAaG;AACH,wBAAsB,SAAS,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAqC1E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,WAAW,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAiB5E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,YAAY,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA2C7E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,SAAS,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAmC1E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,eAAe,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA8BhF;AAMD;;;;;GAKG;AACH,wBAAsB,QAAQ,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAOzE;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,SAAS,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA0B1E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,WAAW,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA0B5E;AAMD;;;;;;;;;GASG;AACH,wBAAsB,WAAW,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAe5E;AAMD;;;;;;;;;;;GAWG;AACH,wBAAsB,gBAAgB,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAsBjF"}
+{"version":3,"file":"task-pool.controller.d.ts","sourceRoot":"","sources":["../../../../../../backend/src/controllers/task-pool/task-pool.controller.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AAmEjD;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAsB,oBAAoB,CACxC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAC5B,OAAO,CACN,IAAI,GACJ;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;CAAE,CACvD,CAiEA;AAMD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAmCG;AACH,wBAAsB,OAAO,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAgFxE;AAMD;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAQ9E;AAMD;;;;;;;;;;;;;GAaG;AACH,wBAAsB,SAAS,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAqC1E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,WAAW,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAiB5E;AAMD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+CG;AACH,wBAAsB,YAAY,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA0F7E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,SAAS,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAmC1E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,eAAe,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA8BhF;AAMD;;;;;GAKG;AACH,wBAAsB,QAAQ,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAOzE;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,SAAS,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA0B1E;AAMD;;;;;;;;;;GAUG;AACH,wBAAsB,WAAW,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA0B5E;AAMD;;;;;;;;;GASG;AACH,wBAAsB,WAAW,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAe5E;AAMD;;;;;;;;;;;GAWG;AACH,wBAAsB,gBAAgB,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAsBjF;AAMD;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,YAAY,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAO7E;AAOD;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAsB,UAAU,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAwB3E;AAOD;;;;;;;;;;;GAWG;AACH,wBAAsB,OAAO,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAgBxE;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAqB9E;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,WAAW,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CA4B5E;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,cAAc,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,CAsB/E"}