fbfd31a042
CI / check (pull_request) Successful in 2m27s
BREAKING: All workflow YAML files must update $START._ to $START.new + $START.resume. The resume edge prompt replaces the previously hardcoded resume message. - evaluate.ts: remove START_ROLE/START_STATUS special case, use $status like all nodes - thread.ts: resolveEvaluateArgs passes 'new', cmdThreadResume passes 'resume' - validate.ts: reject '_' everywhere (no longer valid) - validate-semantic.ts: require 'new' and 'resume' edges on $START - All workflow YAMLs and test fixtures updated Fixes #101
185 lines
5.9 KiB
TypeScript
185 lines
5.9 KiB
TypeScript
import { execFileSync } from "node:child_process";
|
|
import { mkdir, mkdtemp, rm, writeFile } from "node:fs/promises";
|
|
import { tmpdir } from "node:os";
|
|
import { dirname, join } from "node:path";
|
|
import { fileURLToPath } from "node:url";
|
|
import { putSchema } from "@ocas/core";
|
|
import { openStore } from "@ocas/fs";
|
|
import type { CasRef, StepNodePayload, ThreadId } from "@united-workforce/protocol";
|
|
import { afterEach, beforeEach, describe, expect, test } from "vitest";
|
|
import { cmdThreadShow } from "../commands/thread.js";
|
|
import { registerUwfSchemas } from "../schemas.js";
|
|
import { seedThreads } from "./thread-test-helpers.js";
|
|
|
|
const OUTPUT_SCHEMA = {
|
|
type: "object" as const,
|
|
properties: {
|
|
$status: { type: "string" as const },
|
|
question: { type: "string" as const },
|
|
},
|
|
required: ["$status"],
|
|
additionalProperties: false,
|
|
};
|
|
|
|
let tmpDir: string;
|
|
|
|
beforeEach(async () => {
|
|
tmpDir = await mkdtemp(join(tmpdir(), "cli-uwf-suspend-step-test-"));
|
|
});
|
|
|
|
afterEach(async () => {
|
|
await rm(tmpDir, { recursive: true, force: true });
|
|
});
|
|
|
|
describe("suspend step CAS chain and threads.yaml metadata", () => {
|
|
test("thread exec records suspend step in CAS and suspend metadata in threads.yaml", async () => {
|
|
const casDir = join(tmpDir, "cas");
|
|
await mkdir(casDir, { recursive: true });
|
|
const originalCasDir = process.env.OCAS_HOME;
|
|
process.env.OCAS_HOME = casDir;
|
|
|
|
try {
|
|
const store = await openStore(casDir);
|
|
const schemas = await registerUwfSchemas(store);
|
|
|
|
const outputSchemaHash = await putSchema(store, OUTPUT_SCHEMA);
|
|
|
|
const workflowHash = await store.cas.put(schemas.workflow, {
|
|
name: "test-suspend-step",
|
|
description: "suspend step integration test",
|
|
roles: {
|
|
worker: {
|
|
description: "Worker role",
|
|
goal: "Work",
|
|
capabilities: [],
|
|
procedure: "work",
|
|
output: "result",
|
|
frontmatter: outputSchemaHash,
|
|
},
|
|
},
|
|
graph: {
|
|
$START: {
|
|
new: { role: "worker", prompt: "Start work", location: null },
|
|
resume: { role: "worker", prompt: "Resume work", location: null },
|
|
},
|
|
worker: {
|
|
needs_input: {
|
|
role: "$SUSPEND",
|
|
prompt: "Please clarify: {{{question}}}",
|
|
location: null,
|
|
},
|
|
},
|
|
},
|
|
});
|
|
|
|
const startHash = await store.cas.put(schemas.startNode, {
|
|
workflow: workflowHash,
|
|
prompt: "Test suspend task",
|
|
cwd: tmpDir,
|
|
});
|
|
|
|
const threadId = "01SUSPENDSTEPTEST0000000" as ThreadId;
|
|
await seedThreads(tmpDir, { [threadId]: startHash });
|
|
|
|
const outputHash = await store.cas.put(outputSchemaHash, {
|
|
$status: "needs_input",
|
|
question: "Which API?",
|
|
});
|
|
const detailHash = await store.cas.put(schemas.text, "mock detail");
|
|
|
|
const startedAtMs = 1716600000000;
|
|
const completedAtMs = 1716600001500;
|
|
|
|
const stepHash = await store.cas.put(schemas.stepNode, {
|
|
start: startHash,
|
|
prev: null,
|
|
role: "worker",
|
|
output: outputHash,
|
|
detail: detailHash,
|
|
agent: "uwf-mock",
|
|
edgePrompt: "Start work",
|
|
startedAtMs,
|
|
completedAtMs,
|
|
cwd: tmpDir,
|
|
assembledPrompt: null,
|
|
});
|
|
|
|
const mockAgentPath = join(tmpDir, "mock-agent.sh");
|
|
const adapterJson = JSON.stringify({
|
|
stepHash,
|
|
detailHash,
|
|
role: "worker",
|
|
frontmatter: { $status: "needs_input", question: "Which API?" },
|
|
body: "",
|
|
startedAtMs,
|
|
completedAtMs,
|
|
});
|
|
await writeFile(mockAgentPath, `#!/bin/sh\necho '${adapterJson}'\n`, { mode: 0o755 });
|
|
|
|
const configPath = join(tmpDir, "config.yaml");
|
|
await writeFile(
|
|
configPath,
|
|
`defaultAgent: uwf-hermes\ndefaultModel: test-model\nagentOverrides: null\nagents: {}\nproviders: {}\nmodels: {}\n`,
|
|
);
|
|
|
|
const cliPath = join(dirname(fileURLToPath(import.meta.url)), "..", "..", "dist", "cli.js");
|
|
const stdout = execFileSync(
|
|
process.execPath,
|
|
[cliPath, "thread", "exec", threadId, "--agent", mockAgentPath],
|
|
{
|
|
encoding: "utf8",
|
|
stdio: ["ignore", "pipe", "pipe"],
|
|
env: {
|
|
...process.env,
|
|
UWF_HOME: tmpDir,
|
|
OCAS_HOME: casDir,
|
|
},
|
|
cwd: tmpDir,
|
|
timeout: 30000,
|
|
},
|
|
);
|
|
|
|
const cliOutput = JSON.parse(stdout.trim());
|
|
expect(cliOutput.status).toBe("suspended");
|
|
expect(cliOutput.head).toBe(stepHash);
|
|
expect(cliOutput.suspendedRole).toBe("worker");
|
|
expect(cliOutput.suspendMessage).toBe("Please clarify: Which API?");
|
|
|
|
const storeAfter = await openStore(casDir);
|
|
const stepNode = storeAfter.cas.get(cliOutput.head as CasRef);
|
|
expect(stepNode).not.toBeNull();
|
|
const payload = stepNode!.payload as StepNodePayload;
|
|
expect(payload.role).toBe("worker");
|
|
expect(payload.output).toBe(outputHash);
|
|
|
|
const outputNode = storeAfter.cas.get(outputHash);
|
|
expect(outputNode?.payload).toEqual({
|
|
$status: "needs_input",
|
|
question: "Which API?",
|
|
});
|
|
|
|
const { createUwfStore, getThread } = await import("../store.js");
|
|
const uwf = await createUwfStore(tmpDir);
|
|
const threadEntry = getThread(uwf.varStore, threadId);
|
|
expect(threadEntry).toEqual({
|
|
head: stepHash,
|
|
status: "suspended",
|
|
suspendedRole: "worker",
|
|
suspendMessage: "Please clarify: Which API?",
|
|
completedAt: null,
|
|
});
|
|
|
|
const showResult = await cmdThreadShow(tmpDir, threadId);
|
|
expect(showResult.status).toBe("suspended");
|
|
expect(showResult.suspendMessage).toBe("Please clarify: Which API?");
|
|
expect(showResult.suspendedRole).toBe("worker");
|
|
} finally {
|
|
if (originalCasDir === undefined) {
|
|
delete process.env.OCAS_HOME;
|
|
} else {
|
|
process.env.OCAS_HOME = originalCasDir;
|
|
}
|
|
}
|
|
});
|
|
});
|