@tagma/sdk 0.3.0 → 0.3.2

package/README.md

@@ -326,6 +326,20 @@ Properties:
 - `runId` — engine-assigned run ID, available after the first `pipeline_start` event (`null` until then)
 - `status` — `'idle' | 'running' | 'done' | 'aborted'`
 
+### `TriggerBlockedError` / `TriggerTimeoutError`
+
+Typed error classes for trigger plugin error classification. The engine uses `instanceof` checks on these to set the correct task status (`blocked` or `timeout`) instead of matching on error message substrings.
+
+Built-in triggers (`manual`, `file`) throw these automatically. Third-party trigger plugins should throw `TriggerBlockedError` for user/policy rejections and `TriggerTimeoutError` for genuine wait timeouts. Plugins that throw plain `Error` still work — the engine falls back to string matching for backward compatibility, but typed errors are preferred to avoid misclassification from coincidental substrings.
+
+```ts
+import { TriggerBlockedError, TriggerTimeoutError } from '@tagma/sdk';
+
+// In a custom trigger plugin:
+throw new TriggerBlockedError('Access denied by policy');
+throw new TriggerTimeoutError('File did not appear within 30s');
+```
+
 ### `loadPlugins(names: string[]): Promise<void>`
 
 Dynamically loads and registers external plugin packages.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tagma/sdk",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "license": "MIT",
   "repository": {
     "type": "git",

package/src/dag.ts

@@ -84,7 +84,15 @@ export function buildDag(config: PipelineConfig): Dag {
         }
       }
       if (task.continue_from) {
-        const resolved = resolveRef(task.continue_from, track.id);
+        let resolved: string;
+        try {
+          resolved = resolveRef(task.continue_from, track.id);
+        } catch {
+          throw new Error(
+            `Task "${qid}": continue_from "${task.continue_from}" — no such task found. ` +
+            `Use a fully-qualified reference (trackId.taskId) or ensure the target task exists.`
+          );
+        }
         if (!deps.includes(resolved)) {
           deps.push(resolved); // continue_from implies dependency
         }

package/src/validate-raw.ts

@@ -241,11 +241,15 @@ function detectCycles(
   function dfs(id: string): void {
     if (inStack.has(id)) {
       const cycleStart = pathStack.indexOf(id);
-      const cycleNodes = [...pathStack.slice(cycleStart), id];
-      const key = [...cycleNodes].sort().join(',');
+      // Unique nodes in the cycle (without repeating the start node) for dedup.
+      // Previously the duplicate start node caused different sorted keys when
+      // the same cycle was discovered from different entry points.
+      const uniqueNodes = pathStack.slice(cycleStart);
+      const key = [...uniqueNodes].sort().join(',');
       if (!seenCycles.has(key)) {
         seenCycles.add(key);
-        errors.push({ path: 'tracks', message: `Circular dependency detected: ${cycleNodes.join(' → ')}` });
+        const display = [...uniqueNodes, id]; // include start for readable display
+        errors.push({ path: 'tracks', message: `Circular dependency detected: ${display.join(' → ')}` });
       }
       return;
     }