agent-relay 3.1.16 → 3.1.17

package/packages/sdk/src/workflows/trajectory.ts

@@ -43,6 +43,13 @@ interface TrajectoryAgent {
   joinedAt: string;
 }
 
+interface StepParticipants {
+  role?: string;
+  owner?: string;
+  specialist?: string;
+  reviewer?: string;
+}
+
 interface TrajectoryFile {
   id: string;
   version: number;
@@ -230,16 +237,18 @@ export class WorkflowTrajectory {
   // ── Step events ────────────────────────────────────────────────────────────
 
   /** Record step started — captures intent, not just assignment. */
-  async stepStarted(step: WorkflowStep, agent: string): Promise<void> {
+  async stepStarted(step: WorkflowStep, agent: string, participants?: StepParticipants): Promise<void> {
     if (!this.enabled || !this.trajectory) return;
 
-    // Register agent if not seen
-    if (!this.trajectory.agents.some((a) => a.name === agent)) {
-      this.trajectory.agents.push({
-        name: agent,
-        role: step.agent ?? 'deterministic',
-        joinedAt: new Date().toISOString(),
-      });
+    await this.registerAgent(agent, participants?.role ?? step.agent ?? 'deterministic');
+    if (participants?.owner && participants.owner !== agent) {
+      await this.registerAgent(participants.owner, 'owner');
+    }
+    if (participants?.specialist) {
+      await this.registerAgent(participants.specialist, 'specialist');
+    }
+    if (participants?.reviewer) {
+      await this.registerAgent(participants.reviewer, 'reviewer');
     }
 
     // Capture the step's purpose: first non-empty sentence of the task
@@ -255,6 +264,78 @@ export class WorkflowTrajectory {
     await this.flush();
   }
 
+  async registerAgent(name: string, role: string): Promise<void> {
+    if (!this.enabled || !this.trajectory) return;
+    if (!this.trajectory.agents.some((a) => a.name === name)) {
+      this.trajectory.agents.push({
+        name,
+        role,
+        joinedAt: new Date().toISOString(),
+      });
+      await this.flush();
+    }
+  }
+
+  async stepSupervisionAssigned(
+    step: WorkflowStep,
+    supervised: { owner: { name: string }; specialist: { name: string }; reviewer?: { name: string } }
+  ): Promise<void> {
+    if (!this.enabled || !this.trajectory) return;
+
+    await this.registerAgent(supervised.owner.name, 'owner');
+    await this.registerAgent(supervised.specialist.name, 'specialist');
+    if (supervised.reviewer?.name) {
+      await this.registerAgent(supervised.reviewer.name, 'reviewer');
+    }
+
+    const reviewerNote = supervised.reviewer?.name ? `, reviewer=${supervised.reviewer.name}` : '';
+    this.addEvent(
+      'decision',
+      `"${step.name}" supervision assigned → owner=${supervised.owner.name}, specialist=${supervised.specialist.name}${reviewerNote}`,
+      'medium',
+      {
+        owner: supervised.owner.name,
+        specialist: supervised.specialist.name,
+        reviewer: supervised.reviewer?.name,
+      }
+    );
+    await this.flush();
+  }
+
+  async ownerMonitoringEvent(
+    stepName: string,
+    owner: string,
+    detail: string,
+    raw?: Record<string, unknown>
+  ): Promise<void> {
+    if (!this.enabled || !this.trajectory) return;
+
+    this.addEvent(
+      'note',
+      `"${stepName}" owner ${owner}: ${detail}`,
+      'medium',
+      raw ? { owner, ...raw } : { owner }
+    );
+    await this.flush();
+  }
+
+  async reviewCompleted(
+    stepName: string,
+    reviewerName: string,
+    decision: 'approved' | 'rejected',
+    reason?: string
+  ): Promise<void> {
+    if (!this.enabled || !this.trajectory) return;
+
+    this.addEvent('review-completed', `"${stepName}" review ${decision} by ${reviewerName}`, 'medium', {
+      stepName,
+      reviewer: reviewerName,
+      decision,
+      reason,
+    });
+    await this.flush();
+  }
+
   /** Record step completed — captures what was accomplished. */
   async stepCompleted(step: WorkflowStep, output: string, attempt: number): Promise<void> {
     if (!this.enabled || !this.trajectory) return;

package/packages/sdk/src/workflows/validator.ts

@@ -11,8 +11,37 @@ export interface ValidationIssue {
 export function validateWorkflow(config: RelayYamlConfig): ValidationIssue[] {
   const issues: ValidationIssue[] = [];
   const agentMap = new Map(config.agents.map((a) => [a.name, a]));
+  const hasReviewerAgent = config.agents.some((a) => {
+    const role = a.role?.toLowerCase() ?? '';
+    const name = a.name.toLowerCase();
+    return (
+      a.preset === 'reviewer' ||
+      role.includes('review') ||
+      role.includes('critic') ||
+      role.includes('verifier') ||
+      role.includes('qa') ||
+      name.includes('review')
+    );
+  });
 
   for (const workflow of config.workflows ?? []) {
+    const hasInteractiveAgentSteps = workflow.steps.some((step) => {
+      if (step.type === 'deterministic' || step.type === 'worktree') return false;
+      if (!step.agent) return false;
+      const raw = agentMap.get(step.agent);
+      if (!raw) return false;
+      return resolveForValidation(raw).interactive !== false;
+    });
+    if (hasInteractiveAgentSteps && !hasReviewerAgent) {
+      issues.push({
+        severity: 'warning',
+        code: 'NO_REVIEW_AGENT',
+        message: `Workflow "${workflow.name}" has interactive agent steps but no obvious reviewer agent. The runner can auto-fallback, but dedicated reviewers improve step hardening.`,
+        fix: `Add an agent with role/preset like \`reviewer\`, \`critic\`, or \`verifier\`.`,
+        location: `workflow:${workflow.name}`,
+      });
+    }
+
     for (const step of workflow.steps ?? []) {
       if (step.type === 'deterministic' || step.type === 'worktree') continue;
       if (!step.agent) continue;

package/packages/sdk-py/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "agent-relay-sdk"
-version = "3.1.16"
+version = "3.1.17"
 description = "Python SDK for Agent Relay workflows"
 readme = "README.md"
 license = "Apache-2.0"

package/packages/telemetry/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-relay/telemetry",
-  "version": "3.1.16",
+  "version": "3.1.17",
   "description": "Anonymous telemetry for Agent Relay usage analytics",
   "type": "module",
   "main": "dist/index.js",

package/packages/trajectory/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-relay/trajectory",
-  "version": "3.1.16",
+  "version": "3.1.17",
   "description": "Trajectory integration utilities (trail/PDERO) for Relay",
   "type": "module",
   "main": "dist/index.js",
@@ -22,7 +22,7 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@agent-relay/config": "3.1.16"
+    "@agent-relay/config": "3.1.17"
   },
   "devDependencies": {
     "@types/node": "^22.19.3",

package/packages/user-directory/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-relay/user-directory",
-  "version": "3.1.16",
+  "version": "3.1.17",
   "description": "User directory service for agent-relay (per-user credential storage)",
   "type": "module",
   "main": "dist/index.js",
@@ -22,7 +22,7 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@agent-relay/utils": "3.1.16"
+    "@agent-relay/utils": "3.1.17"
   },
   "devDependencies": {
     "@types/node": "^22.19.3",

package/packages/utils/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-relay/utils",
-  "version": "3.1.16",
+  "version": "3.1.17",
   "description": "Shared utilities for agent-relay: logging, name generation, command resolution, update checking",
   "type": "module",
   "main": "dist/cjs/index.js",
@@ -112,7 +112,7 @@
     "vitest": "^3.2.4"
   },
   "dependencies": {
-    "@agent-relay/config": "3.1.16",
+    "@agent-relay/config": "3.1.17",
     "compare-versions": "^6.1.1"
   },
   "publishConfig": {