agents-relay 2.0.127 → 2.0.129

package/dist/workflow/review-v2.js

@@ -20,7 +20,7 @@ export const reviewLifecycleV2 = new GraphRuntime(source, {
         reviewUnblocked: boolGuard('reviewUnblocked'),
     },
     actions: Object.fromEntries(['review.create', 'review.cancelStale', 'review.createFix', 'review.requestMerge'].map(id => [id, { id }])),
-    events: Object.fromEntries(['reconcile', 'review.started', 'review.approved', 'review.changes_required', 'review.blocked', 'review.retry', 'head.changed'].map(id => [id, { id }])),
+    events: Object.fromEntries(['reconcile', 'review.started', 'review.approved', 'review.changes_required', 'review.legacy_blocked', 'review.retry', 'head.changed'].map(id => [id, { id }])),
 });
 export function reviewGraphState(outcome) {
     if (!outcome)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-relay",
-  "version": "2.0.127",
+  "version": "2.0.129",
   "description": "Durable async agent jobs coordinated through GitHub pull requests",
   "repository": {
     "type": "git",

package/skills/agents-relay/agents/planner.agent.md

@@ -19,6 +19,15 @@ For every non-satisfied objective:
 3. Mark the points in that graph that deserve durable Task boundaries.
 4. Create only the next useful executable Task batch from those boundaries.
 
+### Objective decomposition and convergence
+
+- **Never bundle independently testable acceptance layers into one giant Task.** Break objectives into separately verifiable deliverables (for example, contract tests, real-provider compliance, scheduler-driven end-to-end scenarios, and integration evidence). Give each Task concrete acceptance checks and an observable output.
+- Split a large Task when its remaining work spans distinct implementation areas, independently testable failure paths, or more than one meaningful review/recovery cycle. Put shared prerequisites in earlier Tasks and express true dependencies explicitly; independent work may run in parallel.
+- For a failed Task with repeated deterministic acceptance gaps, do not reissue the identical broad assignment. Identify the *missing acceptance cases*, create focused same-Job repair Tasks with valid `same_job_dependency` resolutions, and specify how their evidence unblocks the original Task. Preserve the original failed Task and failure history.
+- For event-driven workflows, separate basic ingress/persistence checks from real scheduler-driven multi-stage scenarios. Require negative-path, multi-failure recovery and no-false-completion evidence before claiming full objective coverage.
+- In `assessment`, show the entire remaining objective as a dependency graph, clearly distinguishing already proven work, the executable batch, subsequent verification, and Relay-owned review/merge. Never report overall `satisfied` merely because one batch has passed.
+
+
 A Task is **not** just a workflow step. Create a Task when its outcome deserves an independent durable lifecycle because it may need to be:
 
 - discussed or referenced later;

package/workflow/review-lifecycle.mmd

@@ -5,6 +5,7 @@ pending["state: PENDING"]
 started["state: STARTED"]
 approved["state: APPROVED"]
 changes["state: CHANGES_REQUIRED"]
+%% BLOCKED is only for legacy records awaiting authoritative unblock.
 blocked["state: BLOCKED"]
 stale["state: STALE"]
 
@@ -91,6 +92,7 @@ pendingUnapproved -->|true| createPendingRetry
 pendingUnapproved -->|false| pending
 createPendingRetry --> pending
 
+pending -->|on: review.legacy_blocked| blocked
 pending -->|on: review.started| started
 pending -->|on: review.approved| autoMergePending
 autoMergePending -->|true| mergePending
@@ -98,7 +100,6 @@ mergePending --> approved
 autoMergePending -->|false| approved
 pending -->|on: review.changes_required| fixPending
 fixPending --> changes
-pending -->|on: review.blocked| blocked
 
 started -->|on: review.approved| autoMergeStarted
 autoMergeStarted -->|true| mergeStarted
@@ -106,7 +107,6 @@ mergeStarted --> approved
 autoMergeStarted -->|false| approved
 started -->|on: review.changes_required| fixStarted
 fixStarted --> changes
-started -->|on: review.blocked| blocked
 changes -->|on: review.changes_required| fixChanges
 fixChanges --> changes