@tagma/sdk 0.1.9 → 0.2.1

package/README.md

@@ -117,7 +117,7 @@ Options:
   - `pipeline_start` — pipeline began; includes `states: ReadonlyMap<taskId, TaskState>` (initial snapshot of all tasks at `waiting`)
   - `task_status_change` — a task changed status; includes `state: TaskState` (complete snapshot at the time of change: `startedAt` is populated before the `running` event; `result` and `finishedAt` are populated before any terminal-status event)
   - `pipeline_end` — pipeline finished; includes `success: boolean`
-- `maxLogRuns` -- number of per-run log directories to keep under `<workDir>/logs/` (default: 20)
+- `maxLogRuns` -- number of per-run log directories to keep under `<workDir>/.tagma/logs/` (default: 20)
 
 ### `PipelineRunner`
 
@@ -260,4 +260,4 @@ Use `buildDag` instead when you have a fully resolved `PipelineConfig` and need
 
 ## License
 
-MIT
+MIT

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tagma/sdk",
-  "version": "0.1.9",
+  "version": "0.2.1",
   "type": "module",
   "workspaces": [
     "plugins/*"

package/src/config-ops.ts

@@ -140,6 +140,12 @@ export function removeTask(
 
   const qualId = `${trackId}.${taskId}`;
 
+  // After deletion, can a bare ref "taskId" still resolve to some other task globally?
+  // It can if any track in the post-deletion config still contains a task with that bare id.
+  const bareIdSurvivesGlobally = withoutTask.tracks.some(t =>
+    t.tasks.some(tk => tk.id === taskId),
+  );
+
   return {
     ...withoutTask,
     tracks: withoutTask.tracks.map(t => {
@@ -149,15 +155,19 @@ export function removeTask(
 
       // Resolve whether a ref in THIS track points to the deleted task:
       //   - Fully-qualified ref ("trackId.taskId") — always points to the deleted task.
-      //   - Bare ref ("taskId") from the same track — always pointed to the deleted task
-      //     (same-track lookup takes priority, and the task was in this track).
-      //   - Bare ref from a different track — points to the deleted task only if this
-      //     track has no task with that same id (no local task to shadow it).
+      //   - Bare ref ("taskId") from the SAME track as the deleted task — always pointed
+      //     to the deleted task (same-track lookup takes priority).
+      //   - Bare ref from a DIFFERENT track:
+      //       1. If this track has a local task with that id → ref resolves locally, not removed.
+      //       2. Else if some other track still has a task with that id → ref will resolve
+      //          there after deletion, not removed.
+      //       3. Else → ref is dangling, remove it.
       const isRemovedFrom = (ref: string): boolean => {
         if (ref === qualId) return true;
         if (ref === taskId) {
           if (t.id === trackId) return true;            // same track — was pointing here
-          return !remainingIds.has(taskId);             // cross-track — only if no local override
+          if (remainingIds.has(taskId)) return false;   // local task shadows — ref is fine
+          return !bareIdSurvivesGlobally;               // remove only if truly dangling
         }
         return false;
       };
@@ -236,4 +246,4 @@ export function transferTask(
   };
   if (!task) return config;
   return upsertTask(afterRemove, toTrackId, task);
-}
+}

package/src/engine.ts

@@ -29,7 +29,10 @@ function preflight(config: PipelineConfig, dag: Dag): void {
     const track = node.track;
     const driverName = task.driver ?? track.driver ?? config.driver ?? 'claude-code';
 
-    if (!hasHandler('drivers', driverName)) {
+    // Pure command tasks don't use a driver — skip driver registration check.
+    const isCommandOnly = task.command && !task.prompt;
+
+    if (!isCommandOnly && !hasHandler('drivers', driverName)) {
       errors.push(`Task "${node.taskId}": driver "${driverName}" not registered`);
     }
 
@@ -117,7 +120,7 @@ export type PipelineEvent =
 export interface RunPipelineOptions {
   readonly approvalGateway?: ApprovalGateway;
   /**
-   * Maximum number of per-run log directories to retain under `<workDir>/logs/`.
+   * Maximum number of per-run log directories to retain under `<workDir>/.tagma/logs/`.
    * Oldest directories are deleted after each run. Defaults to 20. Set to 0 to disable cleanup.
    */
   readonly maxLogRuns?: number;
@@ -207,7 +210,7 @@ export async function runPipeline(
       runId,
       logPath: log.path,
       summary: { total: dag.nodes.size, success: 0, failed: 0, skipped: 0, timeout: 0, blocked: 0 },
-      states,
+      states: freezeStates(states),
     };
   }
 
@@ -697,7 +700,7 @@ export async function runPipeline(
   console.log(`  Log: ${log.path}`);
 
   emit({ type: 'pipeline_end', runId, success: allSuccess });
-  return { success: allSuccess, runId, logPath: log.path, summary, states };
+  return { success: allSuccess, runId, logPath: log.path, summary, states: freezeStates(states) };
 
   } finally {
     // Prune old per-run log directories on every exit path (normal, blocked, or thrown).
@@ -741,3 +744,19 @@ function isTerminal(status: TaskStatus): boolean {
   return status === 'success' || status === 'failed' || status === 'timeout'
     || status === 'skipped' || status === 'blocked';
 }
+
+/** Return a deep-copied, caller-safe snapshot of the states map. */
+function freezeStates(states: Map<string, TaskState>): ReadonlyMap<string, TaskState> {
+  const copy = new Map<string, TaskState>();
+  for (const [id, s] of states) {
+    copy.set(id, {
+      config: { ...s.config },
+      trackConfig: { ...s.trackConfig },
+      status: s.status,
+      result: s.result ? { ...s.result } : null,
+      startedAt: s.startedAt,
+      finishedAt: s.finishedAt,
+    });
+  }
+  return copy;
+}

package/src/pipeline-runner.ts

@@ -108,9 +108,8 @@ export class PipelineRunner {
    * Returns null only if the pipeline has never started.
    */
   getStates(): ReadonlyMap<string, TaskState> | null {
-    if (this._states) return this._states;
-    // Return a snapshot copy so callers cannot mutate SDK-internal state.
-    if (this._statesMirror.size > 0) return new Map([...this._statesMirror]);
+    if (this._states) return snapshotStates(this._states);
+    if (this._statesMirror.size > 0) return snapshotStates(this._statesMirror);
     return null;
   }
 
@@ -124,3 +123,19 @@ export class PipelineRunner {
     return () => this._handlers.delete(handler);
   }
 }
+
+/** Deep-copy a states map so callers cannot mutate SDK internals. */
+function snapshotStates(src: ReadonlyMap<string, TaskState>): ReadonlyMap<string, TaskState> {
+  const copy = new Map<string, TaskState>();
+  for (const [id, s] of src) {
+    copy.set(id, {
+      config: { ...s.config },
+      trackConfig: { ...s.trackConfig },
+      status: s.status,
+      result: s.result ? { ...s.result } : null,
+      startedAt: s.startedAt,
+      finishedAt: s.finishedAt,
+    });
+  }
+  return copy;
+}