@davidorex/pi-workflows 0.2.0 → 0.3.1

package/src/workflow-discovery.test.ts

@@ -1,105 +0,0 @@
-import { describe, it } from "node:test";
-import assert from "node:assert";
-import { discoverWorkflows, findWorkflow } from "./workflow-discovery.ts";
-import fs from "node:fs";
-import path from "node:path";
-import os from "node:os";
-
-// Use a nonexistent dir as builtinDir so tests don't pick up real demo workflows
-const NO_BUILTINS = path.join(os.tmpdir(), "wf-no-builtins-nonexistent");
-
-describe("discoverWorkflows", () => {
-  it("discovers workflows from project directory", () => {
-    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wf-test-"));
-    const wfDir = path.join(tmpDir, ".workflows");
-    fs.mkdirSync(wfDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(wfDir, "test.workflow.yaml"),
-      "name: test\nsteps:\n  s:\n    agent: a",
-    );
-
-    const specs = discoverWorkflows(tmpDir, NO_BUILTINS);
-    assert.strictEqual(specs.length, 1);
-    assert.strictEqual(specs[0].name, "test");
-    assert.strictEqual(specs[0].source, "project");
-
-    fs.rmSync(tmpDir, { recursive: true });
-  });
-
-  it("returns empty array when no workflow directories exist", () => {
-    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wf-test-"));
-    const specs = discoverWorkflows(tmpDir, NO_BUILTINS);
-    assert.strictEqual(specs.length, 0);
-    fs.rmSync(tmpDir, { recursive: true });
-  });
-
-  it("skips invalid specs with warning (no throw)", () => {
-    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wf-test-"));
-    const wfDir = path.join(tmpDir, ".workflows");
-    fs.mkdirSync(wfDir, { recursive: true });
-    fs.writeFileSync(path.join(wfDir, "bad.workflow.yaml"), "not: valid: workflow");
-    fs.writeFileSync(
-      path.join(wfDir, "good.workflow.yaml"),
-      "name: good\nsteps:\n  s:\n    agent: a",
-    );
-
-    const specs = discoverWorkflows(tmpDir, NO_BUILTINS);
-    assert.strictEqual(specs.length, 1);
-    assert.strictEqual(specs[0].name, "good");
-
-    fs.rmSync(tmpDir, { recursive: true });
-  });
-
-  it("project specs shadow user specs with same name", () => {
-    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wf-test-"));
-    const projectDir = path.join(tmpDir, ".workflows");
-    fs.mkdirSync(projectDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(projectDir, "shared.workflow.yaml"),
-      "name: shared\ndescription: project version\nsteps:\n  s:\n    agent: a",
-    );
-
-    const specs = discoverWorkflows(tmpDir, NO_BUILTINS);
-    assert.strictEqual(specs.length, 1);
-    assert.strictEqual(specs[0].source, "project");
-
-    fs.rmSync(tmpDir, { recursive: true });
-  });
-
-  it("discovers builtin demo workflows", () => {
-    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wf-test-"));
-    const demoDir = path.resolve(import.meta.dirname, "..", "workflows");
-
-    const specs = discoverWorkflows(tmpDir, demoDir);
-    assert.ok(specs.length > 0, "should find at least one demo workflow");
-
-    const names = specs.map((s) => s.name);
-    assert.ok(names.includes("typed-analysis"), "should find typed-analysis demo");
-
-    fs.rmSync(tmpDir, { recursive: true });
-  });
-});
-
-describe("findWorkflow", () => {
-  it("finds a workflow by name", () => {
-    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wf-test-"));
-    const wfDir = path.join(tmpDir, ".workflows");
-    fs.mkdirSync(wfDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(wfDir, "bugfix.workflow.yaml"),
-      "name: bugfix\nsteps:\n  s:\n    agent: a",
-    );
-
-    const spec = findWorkflow("bugfix", tmpDir, NO_BUILTINS);
-    assert.ok(spec);
-    assert.strictEqual(spec.name, "bugfix");
-
-    fs.rmSync(tmpDir, { recursive: true });
-  });
-
-  it("returns undefined for unknown workflow", () => {
-    const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wf-test-"));
-    assert.strictEqual(findWorkflow("nonexistent", tmpDir, NO_BUILTINS), undefined);
-    fs.rmSync(tmpDir, { recursive: true });
-  });
-});

package/src/workflow-discovery.ts

@@ -1,114 +0,0 @@
-import fs from "node:fs";
-import path from "node:path";
-import os from "node:os";
-import type { WorkflowSpec } from "./types.ts";
-import { parseWorkflowSpec } from "./workflow-spec.ts";
-import { WORKFLOWS_DIR } from "./workflows-dir.ts";
-
-/**
- * Discover all workflow specs from project, user, and builtin directories.
- *
- * Scans (highest priority first):
- *   1. .workflows/                (project-level, source: "project")
- *   2. ~/.pi/agent/workflows/     (user-level, source: "user")
- *   3. <package>/workflows/       (builtin workflows, source: "user")
- *
- * Higher-priority specs shadow lower-priority specs with the same name.
- *
- * @param cwd - current working directory (project root)
- * @param builtinDir - optional path to builtin workflows (defaults to workflows/ relative to package root)
- * @returns Array of parsed WorkflowSpec objects. Specs that fail parsing are
- *          skipped with a warning (logged to stderr), not thrown.
- */
-export function discoverWorkflows(cwd: string, builtinDir?: string): WorkflowSpec[] {
-  const projectDir = path.join(cwd, WORKFLOWS_DIR);
-  const userDir = path.join(os.homedir(), ".pi", "agent", "workflows");
-  const demoDir = builtinDir ?? path.resolve(import.meta.dirname, "..", "workflows");
-
-  const projectSpecs = scanDirectory(projectDir, "project");
-  const userSpecs = scanDirectory(userDir, "user");
-  const builtinSpecs = scanDirectory(demoDir, "user");
-
-  // Deduplicate: project > user > builtin (last write wins, so add lowest priority first)
-  const byName = new Map<string, WorkflowSpec>();
-
-  for (const spec of builtinSpecs) {
-    byName.set(spec.name, spec);
-  }
-  for (const spec of userSpecs) {
-    byName.set(spec.name, spec);
-  }
-  for (const spec of projectSpecs) {
-    byName.set(spec.name, spec);
-  }
-
-  // Return sorted by name
-  const result = Array.from(byName.values());
-  result.sort((a, b) => a.name.localeCompare(b.name));
-  return result;
-}
-
-/**
- * Find a workflow by name from discovered workflows.
- * Returns undefined if not found.
- */
-export function findWorkflow(name: string, cwd: string, builtinDir?: string): WorkflowSpec | undefined {
-  const specs = discoverWorkflows(cwd, builtinDir);
-  return specs.find((s) => s.name === name);
-}
-
-/**
- * Scan a directory for *.workflow.yaml files and parse them.
- * Non-recursive. Returns parsed specs; logs warnings for parse failures.
- */
-function scanDirectory(dirPath: string, source: "user" | "project"): WorkflowSpec[] {
-  if (!fs.existsSync(dirPath)) {
-    return [];
-  }
-
-  let entries: string[];
-  try {
-    entries = fs.readdirSync(dirPath);
-  } catch {
-    return [];
-  }
-
-  const specs: WorkflowSpec[] = [];
-
-  for (const entry of entries) {
-    if (!entry.endsWith(".workflow.yaml")) {
-      continue;
-    }
-
-    const filePath = path.join(dirPath, entry);
-
-    // Only process files, not directories
-    let stat: fs.Stats;
-    try {
-      stat = fs.statSync(filePath);
-    } catch {
-      continue;
-    }
-    if (!stat.isFile()) {
-      continue;
-    }
-
-    let content: string;
-    try {
-      content = fs.readFileSync(filePath, "utf-8");
-    } catch {
-      console.error(`[pi-workflows] Warning: skipping ${filePath}: could not read file`);
-      continue;
-    }
-
-    try {
-      const spec = parseWorkflowSpec(content, filePath, source);
-      specs.push(spec);
-    } catch (err) {
-      const msg = err instanceof Error ? err.message : String(err);
-      console.error(`[pi-workflows] Warning: skipping ${filePath}: ${msg}`);
-    }
-  }
-
-  return specs;
-}