Merge pull request #3 from AgentWorkforce/fix/cli-version-and-dry-run-labels

fix(cli): dynamic --version + label deterministic steps in dry-run

Author: willwashburn

memory/workspace/.relay/state.json

@@ -0,0 +1 @@
+{"workspaceId":"rw_7ccfea89","remoteRoot":"/memory/workspace","localRoot":"/home/daytona/workspace/memory/workspace","mode":"poll","intervalMs":5000,"lastReconcileAt":"2026-06-05T17:42:35.095091918Z","lastSuccessfulReconcileAt":"2026-06-05T17:42:35.095091918Z","staleAfter":"2026-06-05T17:42:45.095091918Z","status":"ready","states":{"stale":false,"offline":false,"hasConflicts":false,"hasPendingWriteback":false},"pendingWriteback":0,"pendingConflicts":0,"deniedPaths":0,"counters":{"snapshotDeleteBlocked":95},"circuit":{"open":false,"openedAt":"0001-01-01T00:00:00Z","windowMs":60000,"cooldownMs":30000,"threshold":5,"nextRetry":"0001-01-01T00:00:00Z"}}

packages/cli/src/cli.ts

@@ -1,4 +1,5 @@
 #!/usr/bin/env node
+import { createRequire } from 'node:module';
 import path from 'node:path';
 import { Command } from 'commander';
 import {
@@ -45,12 +46,19 @@ async function runYamlWorkflow(
   return result;
 }
 
+// Read the version from package.json at runtime so it stays in sync with the
+// version the publish workflow bumps. `../package.json` resolves to the package
+// root from both src/cli.ts (dev) and dist/cli.js (published). createRequire
+// avoids a static JSON import, which would fall outside tsconfig's rootDir.
+const require = createRequire(import.meta.url);
+const { version } = require('../package.json') as { version: string };
+
 const program = new Command();
 
 program
   .name('relayflows')
   .description('Run Agent Relay workflows from the command line')
-  .version('0.1.0');
+  .version(version);
 
 program
   .command('run <file>')

packages/core/src/__tests__/dry-run-format.test.ts

@@ -0,0 +1,51 @@
+import { describe, it, expect } from 'vitest';
+
+import { formatDryRunReport } from '../dry-run-format.js';
+import type { DryRunReport } from '../types.js';
+
+function makeReport(overrides: Partial<DryRunReport> = {}): DryRunReport {
+  return {
+    valid: true,
+    errors: [],
+    warnings: [],
+    name: 'demo',
+    pattern: 'dag',
+    agents: [{ name: 'builder', cli: 'claude', stepCount: 1 }],
+    waves: [
+      {
+        wave: 1,
+        steps: [
+          { name: 'implement', agent: 'builder', dependsOn: [] },
+          // Agent-less step => deterministic (shell/gate) step.
+          { name: 'test-gate', dependsOn: ['implement'] },
+        ],
+      },
+    ],
+    totalSteps: 2,
+    estimatedWaves: 1,
+    ...overrides,
+  };
+}
+
+describe('formatDryRunReport', () => {
+  it('labels agent-less steps as deterministic, not "undefined"', () => {
+    const out = formatDryRunReport(makeReport());
+
+    expect(out).toContain('implement (builder)');
+    expect(out).toContain('test-gate (deterministic)');
+    // Regression guard: the literal "undefined" must never reach the plan.
+    expect(out).not.toContain('(undefined)');
+  });
+
+  it('still renders the agent name when a step has one', () => {
+    const out = formatDryRunReport(
+      makeReport({
+        waves: [{ wave: 1, steps: [{ name: 'review', agent: 'reviewer', dependsOn: [] }] }],
+        totalSteps: 1,
+      })
+    );
+
+    expect(out).toContain('review (reviewer)');
+    expect(out).not.toContain('deterministic');
+  });
+});

packages/core/src/dry-run-format.ts

@@ -52,7 +52,10 @@ export function formatDryRunReport(report: DryRunReport): string {
       for (let i = 0; i < wave.steps.length; i++) {
         const step = wave.steps[i];
         const prefix = i === 0 ? `  Wave ${String(wave.wave).padStart(2)}:` : '          ';
-        lines.push(`${prefix}  ${step.name} (${step.agent})`);
+        // Steps without an agent are deterministic (shell/gate) steps; label
+        // them as such rather than printing the literal "undefined".
+        const actor = step.agent ?? 'deterministic';
+        lines.push(`${prefix}  ${step.name} (${actor})`);
       }
     }
     lines.push('');