@tagma/sdk 0.4.8 → 0.4.9

package/dist/templates.js

@@ -0,0 +1,93 @@
+// ═══ Template Discovery (F1) ═══
+//
+// Public helpers so editors / UIs can enumerate installed `@tagma/template-*`
+// packages in a workspace and read each template's declarative metadata
+// (name, description, params) without actually expanding the template.
+//
+// The legacy private `loadTemplate` in schema.ts uses Bun-specific APIs
+// (Bun.file, require.resolve). These helpers are Node-compatible because
+// the editor server runs on Node, not Bun.
+import { readFileSync, existsSync, readdirSync, statSync } from 'fs';
+import { join } from 'path';
+import yaml from 'js-yaml';
+/**
+ * Scan the workspace's `node_modules/@tagma/*` for packages whose name starts
+ * with `template-` and load each one's manifest. Packages without a valid
+ * `template.yaml` (or that fail to parse) are silently skipped.
+ *
+ * Returns an empty array when `workDir` doesn't exist or has no such packages.
+ */
+export function discoverTemplates(workDir) {
+    const out = [];
+    const scopeDir = join(workDir, 'node_modules', '@tagma');
+    if (!existsSync(scopeDir))
+        return out;
+    let entries = [];
+    try {
+        entries = readdirSync(scopeDir);
+    }
+    catch {
+        return out;
+    }
+    for (const entry of entries) {
+        if (!entry.startsWith('template-'))
+            continue;
+        const pkgDir = join(scopeDir, entry);
+        try {
+            const st = statSync(pkgDir);
+            if (!st.isDirectory())
+                continue;
+        }
+        catch {
+            continue;
+        }
+        const ref = `@tagma/${entry}`;
+        const manifest = loadTemplateManifestFromDir(pkgDir, ref);
+        if (manifest)
+            out.push(manifest);
+    }
+    // Sort alphabetically for deterministic UI rendering.
+    out.sort((a, b) => a.ref.localeCompare(b.ref));
+    return out;
+}
+/**
+ * Load a single template's manifest by its ref (e.g. `@tagma/template-review`)
+ * from the given workspace's `node_modules`. Returns `null` if the package
+ * isn't installed or its manifest can't be parsed.
+ */
+export function loadTemplateManifest(ref, workDir) {
+    // Only @tagma/template-* refs are supported (matches SDK validateTemplateRef).
+    const stripped = ref.replace(/@v\d+$/, '');
+    if (!stripped.startsWith('@tagma/template-'))
+        return null;
+    const pkgDir = join(workDir, 'node_modules', stripped);
+    if (!existsSync(pkgDir))
+        return null;
+    return loadTemplateManifestFromDir(pkgDir, stripped);
+}
+/**
+ * Resolve a template manifest from an absolute package directory. Tries
+ * `template.yaml` first (the documented convention), then a `template` export
+ * from `package.json`'s `main`. Returns `null` on any failure so discovery
+ * stays robust against malformed packages.
+ */
+function loadTemplateManifestFromDir(pkgDir, ref) {
+    const yamlPath = join(pkgDir, 'template.yaml');
+    if (existsSync(yamlPath)) {
+        try {
+            const content = readFileSync(yamlPath, 'utf-8');
+            const doc = yaml.load(content);
+            const tpl = (doc && typeof doc === 'object' && 'template' in doc
+                ? doc.template
+                : doc);
+            if (tpl && typeof tpl === 'object' && tpl.name && Array.isArray(tpl.tasks)) {
+                return { ...tpl, ref };
+            }
+        }
+        catch {
+            return null;
+        }
+    }
+    return null;
+}
+//# sourceMappingURL=templates.js.map

package/dist/templates.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"templates.js","sourceRoot":"","sources":["../src/templates.ts"],"names":[],"mappings":"AAAA,kCAAkC;AAClC,EAAE;AACF,8EAA8E;AAC9E,wEAAwE;AACxE,uEAAuE;AACvE,EAAE;AACF,wEAAwE;AACxE,yEAAyE;AACzE,2CAA2C;AAE3C,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,IAAI,CAAC;AACrE,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAC;AAC5B,OAAO,IAAI,MAAM,SAAS,CAAC;AAQ3B;;;;;;GAMG;AACH,MAAM,UAAU,iBAAiB,CAAC,OAAe;IAC/C,MAAM,GAAG,GAAuB,EAAE,CAAC;IACnC,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,EAAE,cAAc,EAAE,QAAQ,CAAC,CAAC;IACzD,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAO,GAAG,CAAC;IAEtC,IAAI,OAAO,GAAa,EAAE,CAAC;IAC3B,IAAI,CAAC;QACH,OAAO,GAAG,WAAW,CAAC,QAAQ,CAAC,CAAC;IAClC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,GAAG,CAAC;IACb,CAAC;IAED,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC5B,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,WAAW,CAAC;YAAE,SAAS;QAC7C,MAAM,MAAM,GAAG,IAAI,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAC;QACrC,IAAI,CAAC;YACH,MAAM,EAAE,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC;YAC5B,IAAI,CAAC,EAAE,CAAC,WAAW,EAAE;gBAAE,SAAS;QAClC,CAAC;QAAC,MAAM,CAAC;YACP,SAAS;QACX,CAAC;QAED,MAAM,GAAG,GAAG,UAAU,KAAK,EAAE,CAAC;QAC9B,MAAM,QAAQ,GAAG,2BAA2B,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;QAC1D,IAAI,QAAQ;YAAE,GAAG,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IACnC,CAAC;IAED,sDAAsD;IACtD,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;IAC/C,OAAO,GAAG,CAAC;AACb,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,oBAAoB,CAAC,GAAW,EAAE,OAAe;IAC/D,+EAA+E;IAC/E,MAAM,QAAQ,GAAG,GAAG,CAAC,OAAO,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC;IAC3C,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,kBAAkB,CAAC;QAAE,OAAO,IAAI,CAAC;IAC1D,MAAM,MAAM,GAAG,IAAI,CAAC,OAAO,EAAE,cAAc,EAAE,QAAQ,CAAC,CAAC;IACvD,IAAI,CAAC,UAAU,CAAC,MAAM,CAAC;QAAE,OAAO,IAAI,CAAC;IACrC,OAAO,2BAA2B,CAAC,MAAM,EAAE,QAAQ,CAAC,CAAC;AACvD,CAAC;AAED;;;;;GAKG;AACH,SAAS,2BAA2B,CAAC,MAAc,EAAE,GAAW;IAC9D,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,EAAE,eAAe,CAAC,CAAC;IAC/C,IAAI,UAAU,CAAC,QAAQ,CAAC,EAAE,CAAC;QACzB,IAAI,CAAC;YACH,MAAM,OAAO,GAAG,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;YAChD,MAAM,GAAG,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,CAAmD,CAAC;YACjF,MAAM,GAAG,GAAG,CAAC,GAAG,IAAI,OAAO,GAAG,KAAK,QAAQ,IAAI,UAAU,IAAI,GAAG;gBAC9D,CAAC,CAAE,GAAqC,CAAC,QAAQ;gBACjD,CAAC,CAAE,GAAsB,CAA+B,CAAC;YAC3D,IAAI,GAAG,IAAI,OAAO,GAAG,KAAK,QAAQ,IAAI,GAAG,CAAC,IAAI,IAAI,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC3E,OAAO,EAAE,GAAG,GAAG,EAAE,GAAG,EAAE,CAAC;YACzB,CAAC;QACH,CAAC;QAAC,MAAM,CAAC;YACP,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tagma/sdk",
-  "version": "0.4.8",
+  "version": "0.4.9",
   "description": "Local AI task orchestration engine — core SDK for tagma pipelines",
   "license": "MIT",
   "repository": {

package/src/dag.ts

@@ -178,22 +178,22 @@ export interface RawDag {
 }
 
 /**
- * Build a lightweight DAG from a raw (unresolved) pipeline config.
- * Unlike buildDag, this function:
- *   - Does not require a workDir or resolved PipelineConfig
- *   - Is lenient: missing or ambiguous refs are silently skipped
- *
- * Intended for the visual editor to render the flow graph before a pipeline is run.
- */
-export function buildRawDag(config: RawPipelineConfig): RawDag {
+ * Build a lightweight DAG from a raw (unresolved) pipeline config.
+ * Unlike buildDag, this function:
+ *   - Does not require a workDir or resolved PipelineConfig
+ *   - Is lenient: missing or ambiguous refs are silently skipped
+ *
+ * Intended for the visual editor to render the flow graph before a pipeline is run.
+ */
+export function buildRawDag(config: RawPipelineConfig): RawDag {
   const nodes = new Map<string, RawDagNode>();
   const bareToQualified = new Map<string, string>();
 
-  // 1. Register all concrete tasks
-  for (const track of config.tracks) {
-    for (const task of track.tasks) {
-      const qid = `${track.id}.${task.id}`;
-      if (nodes.has(qid)) continue; // skip duplicates silently
+  // 1. Register all concrete tasks
+  for (const track of config.tracks) {
+    for (const task of track.tasks) {
+      const qid = `${track.id}.${task.id}`;
+      if (nodes.has(qid)) continue; // skip duplicates silently
 
       if (bareToQualified.has(task.id)) {
         bareToQualified.set(task.id, '__ambiguous__');
@@ -216,10 +216,10 @@ export function buildRawDag(config: RawPipelineConfig): RawDag {
 
   const edges: { from: string; to: string }[] = [];
 
-  for (const track of config.tracks) {
-    for (const task of track.tasks) {
-      const qid = `${track.id}.${task.id}`;
-      const deps: string[] = [];
+  for (const track of config.tracks) {
+    for (const task of track.tasks) {
+      const qid = `${track.id}.${task.id}`;
+      const deps: string[] = [];
 
       for (const ref of task.depends_on ?? []) {
         const resolved = tryResolve(ref, track.id);

package/src/engine.ts

@@ -1,5 +1,5 @@
-import { resolve, dirname } from 'path';
-import { mkdir, readdir, rm } from 'fs/promises';
+import { resolve } from 'path';
+import { readdir, rm } from 'fs/promises';
 import type {
   PipelineConfig, TaskConfig, TrackConfig, TaskState, TaskStatus,
   TaskResult, DriverPlugin, TriggerPlugin, CompletionPlugin,
@@ -9,7 +9,7 @@ import type {
 import { buildDag, type Dag, type DagNode } from './dag';
 import { getHandler, hasHandler, loadPlugins } from './registry';
 import { runSpawn, runCommand } from './runner';
-import { parseDuration, nowISO, generateRunId, validatePath } from './utils';
+import { parseDuration, nowISO, generateRunId } from './utils';
 import {
   executeHook,
   buildPipelineStartContext, buildTaskContext,
@@ -79,7 +79,7 @@ function preflight(config: PipelineConfig, dag: Dag): void {
           const upstream = dag.nodes.get(upstreamId);
           if (upstream) {
             // A handoff is possible via session resume (already ruled out above),
-            // an output file, OR in-memory text injection through normalizedMap
+            // OR in-memory text injection through normalizedMap
             // (when the upstream driver implements parseResult and returns normalizedOutput).
             const upstreamDriverName = upstream.task.driver ?? upstream.track.driver
               ?? config.driver ?? 'claude-code';
@@ -88,12 +88,12 @@ function preflight(config: PipelineConfig, dag: Dag): void {
               : null;
             const canNormalize = typeof upstreamDriver?.parseResult === 'function';
 
-            if (!upstream.task.output && !canNormalize) {
+            if (!canNormalize) {
               errors.push(
                 `Task "${node.taskId}" uses continue_from: "${task.continue_from}", ` +
-                `but upstream task "${upstreamId}" has no "output" field and its driver ` +
+                `but upstream task "${upstreamId}" its driver ` +
                 `does not implement parseResult for text-injection handoff. ` +
-                `Add output to the upstream task, use a driver with parseResult, or remove continue_from.`
+                `Use a driver with parseResult, or remove continue_from.`
               );
             }
           }
@@ -297,7 +297,6 @@ export async function runPipeline(
   options.onEvent?.({ type: 'pipeline_start', runId, states: statesSnapshot });
 
   const sessionMap = new Map<string, string>();
-  const outputMap = new Map<string, string>();
   const normalizedMap = new Map<string, string>();
 
   // Pipeline timeout
@@ -407,7 +406,6 @@ export async function runPipeline(
       status: state.status,
       exit_code: state.result?.exitCode ?? null,
       duration_ms: state.result?.durationMs ?? null,
-      output_path: state.result?.outputPath ?? null,
       stderr_path: state.result?.stderrPath ?? null,
       session_id: state.result?.sessionId ?? null,
       started_at: state.startedAt,
@@ -585,7 +583,7 @@ export async function runPipeline(
           log.debug(`[task:${taskId}]`,
             `middleware chain: ${mws.map(m => m.type).join(' → ')}`);
           const mwCtx: MiddlewareContext = {
-            task, track, outputMap, workDir: task.cwd ?? workDir,
+            task, track, workDir: task.cwd ?? workDir,
           };
           for (const mwConfig of mws) {
             const before = prompt.length;
@@ -611,7 +609,7 @@ export async function runPipeline(
 
         // H1: hand the driver a continue_from that has already been
         // qualified by dag.ts. Without this, drivers like codex/opencode/
-        // claude-code do `outputMap.get(task.continue_from)` directly with
+        // claude-code look up maps directly with
         // the user's raw (possibly bare) string, which races whenever two
         // tracks share a task name. dag.ts has the only authoritative
         // resolver, so we use its precomputed answer here.
@@ -621,7 +619,7 @@ export async function runPipeline(
           continue_from: node.resolvedContinueFrom ?? task.continue_from,
         };
         const driverCtx: DriverContext = {
-          sessionMap, outputMap, normalizedMap, workDir: task.cwd ?? workDir,
+          sessionMap, normalizedMap, workDir: task.cwd ?? workDir,
         };
         const spec = await driver.buildCommand(enrichedTask, track, driverCtx);
         log.debug(`[task:${taskId}]`, `driver=${driverName}`);
@@ -635,27 +633,6 @@ export async function runPipeline(
         result = await runSpawn(spec, driver, runOpts);
       }
 
-      // 5. Write output file with RAW stdout (preserves driver output format).
-      // Done BEFORE the completion check so a `file_exists` completion pointing
-      // at `task.output` observes the AI-generated artefact. Writes happen
-      // regardless of exit code so failed/timed-out tasks still leave a
-      // debuggable artefact on disk.
-      if (task.output) {
-        // validatePath enforces no .. traversal and no absolute paths escaping workDir.
-        const outPath = validatePath(task.output, workDir);
-        await mkdir(dirname(outPath), { recursive: true });
-        await Bun.write(outPath, result.stdout);
-        result = { ...result, outputPath: outPath };
-        // H1: only write the fully-qualified taskId. The previous "also store
-        // bare id when not yet present" trick produced non-deterministic
-        // continue_from lookups when two tracks shared a task name —
-        // whichever finished first won the bare key. dag.ts now resolves
-        // continue_from to a qualified id (DagNode.resolvedContinueFrom),
-        // and the enrichedTask handed to drivers carries that qualified
-        // version, so bare keys are no longer needed.
-        outputMap.set(taskId, outPath);
-      }
-
       // 6. Determine terminal status (without emitting yet — result must be complete first)
       // H2: branch on failureKind so spawn errors no longer masquerade as
       // timeouts. Old runners that don't set failureKind still work — we
@@ -697,7 +674,6 @@ export async function runPipeline(
           ? result.normalizedOutput.slice(0, MAX_NORMALIZED_BYTES) +
             `\n[…clipped at ${MAX_NORMALIZED_BYTES} bytes]`
           : result.normalizedOutput;
-        // H1: qualified-only key (see comment near outputMap above).
         normalizedMap.set(taskId, clipped);
       }
 
@@ -708,7 +684,7 @@ export async function runPipeline(
       }
 
       if (result.sessionId) {
-        // H1: qualified-only key (see comment near outputMap above).
+        // H1: qualified-only key.
         sessionMap.set(taskId, result.sessionId);
       }
 
@@ -736,9 +712,6 @@ export async function runPipeline(
       if (result.sessionId) {
         log.debug(`[task:${taskId}]`, `sessionId: ${result.sessionId}`);
       }
-      if (result.outputPath) {
-        log.debug(`[task:${taskId}]`, `wrote output: ${result.outputPath}`);
-      }
       if (result.stderrPath) {
         log.debug(`[task:${taskId}]`, `wrote stderr: ${result.stderrPath}`);
       }
@@ -760,7 +733,7 @@ export async function runPipeline(
         exitCode: -1,
         stdout: '',
         stderr: errMsg,
-        outputPath: null, stderrPath: null, durationMs: 0,
+        stderrPath: null, durationMs: 0,
         sessionId: null, normalizedOutput: null,
         // H2: Engine-level pre-execution errors (driver throw, middleware
         // throw, getHandler 404) classify as spawn_error — the process never

package/src/hooks.ts

@@ -141,7 +141,6 @@ export interface TaskInfo {
   readonly status: string;
   readonly exit_code: number | null;
   readonly duration_ms: number | null;
-  readonly output_path: string | null;
   readonly stderr_path: string | null;
   readonly session_id: string | null;
   readonly started_at: string | null;