@synth-deploy/server 1.0.6 → 1.2.0

package/tests/decision-diary.test.ts

@@ -127,16 +127,19 @@ async function testDeploy(
     opts.envVars ?? {},
   );
 
-  const trigger = {
+  const operationInput = {
+    type: "deploy" as const,
     artifactId: artifact.id,
     artifactVersionId: opts.version ?? "2.0.0",
+  };
+  const operationTrigger = {
     partitionId: opts.partitionId ?? partition.id,
     environmentId: env.id,
     triggeredBy: "user" as const,
     ...(opts.variables ? { variables: opts.variables } : {}),
   };
 
-  return agent.triggerDeployment(trigger);
+  return agent.triggerOperation(operationInput, operationTrigger);
 }
 
 // ---------------------------------------------------------------------------
@@ -169,7 +172,7 @@ describe("Decision Diary — entry specificity", () => {
       variables: { LOG_LEVEL: "error" },
     });
 
-    const entries = diary.getByDeployment(result.id);
+    const entries = diary.getByOperation(result.id);
     expect(entries.length).toBeGreaterThanOrEqual(5);
 
     for (const entry of entries) {
@@ -191,7 +194,7 @@ describe("Decision Diary — entry specificity", () => {
       variables: { LOG_LEVEL: "error" },
     });
 
-    const entries = diary.getByDeployment(result.id);
+    const entries = diary.getByOperation(result.id);
 
     // Reasoning must contain at least one concrete reference
     const genericPhrases = [
@@ -222,7 +225,7 @@ describe("Decision Diary — entry specificity", () => {
 
     expect(result.status).toBe("failed");
 
-    const entries = diary.getByDeployment(result.id);
+    const entries = diary.getByOperation(result.id);
     const failEntry = findDecisions(entries, "Deployment failed")[0];
 
     // Failure reasoning must contain recommended action
@@ -239,7 +242,7 @@ describe("Decision Diary — entry specificity", () => {
       variables: { LOG_LEVEL: "debug" },
     });
 
-    const entries = diary.getByDeployment(result.id);
+    const entries = diary.getByOperation(result.id);
     const conflictEntries = findDecisions(entries, "conflict");
     expect(conflictEntries.length).toBeGreaterThanOrEqual(1);
 
@@ -280,7 +283,7 @@ describe("Decision Diary — orchestration completeness", () => {
 
     const result = await testDeploy(agent, artifactStore, environmentStore, partitionStore);
 
-    const entries = diary.getByDeployment(result.id);
+    const entries = diary.getByOperation(result.id);
     const types = entries.map((e) => e.decisionType);
 
     expect(types).toContain("artifact-analysis");
@@ -297,7 +300,7 @@ describe("Decision Diary — orchestration completeness", () => {
 
     const result = await testDeploy(agent, artifactStore, environmentStore, partitionStore);
 
-    const entries = diary.getByDeployment(result.id);
+    const entries = diary.getByOperation(result.id);
     const types = entries.map((e) => e.decisionType);
 
     // Should have artifact analysis, plan, config, health check (retry), and failure
@@ -318,16 +321,12 @@ describe("Decision Diary — orchestration completeness", () => {
     const partition = partitionStore.create("Acme Corp", { DB_HOST: "prod-db.internal" });
     const env = environmentStore.create("staging", { DB_HOST: "staging-db.internal" });
 
-    const trigger = {
-      artifactId: artifact.id,
-      artifactVersionId: "2.0.0",
-      partitionId: partition.id,
-      environmentId: env.id,
-      triggeredBy: "user" as const,
-    };
-    const result = await agent.triggerDeployment(trigger);
+    const result = await agent.triggerOperation(
+      { type: "deploy", artifactId: artifact.id, artifactVersionId: "2.0.0" },
+      { partitionId: partition.id, environmentId: env.id, triggeredBy: "user" },
+    );
 
-    const entries = diary.getByDeployment(result.id);
+    const entries = diary.getByOperation(result.id);
     const types = entries.map((e) => e.decisionType);
 
     expect(types).toContain("variable-conflict");
@@ -394,8 +393,8 @@ describe("Decision Diary — retrieval dimensions", () => {
     const result1 = await testDeploy(agent, artifactStore, environmentStore, partitionStore, { version: "1.0.0" });
     const result2 = await testDeploy(agent, artifactStore, environmentStore, partitionStore, { version: "2.0.0" });
 
-    const entries1 = diary.getByDeployment(result1.id);
-    const entries2 = diary.getByDeployment(result2.id);
+    const entries1 = diary.getByOperation(result1.id);
+    const entries2 = diary.getByOperation(result2.id);
 
     // Each deployment has its own entries
     expect(entries1.length).toBeGreaterThanOrEqual(5);
@@ -403,10 +402,10 @@ describe("Decision Diary — retrieval dimensions", () => {
 
     // No cross-contamination
     for (const e of entries1) {
-      expect(e.deploymentId).toBe(result1.id);
+      expect(e.operationId).toBe(result1.id);
     }
     for (const e of entries2) {
-      expect(e.deploymentId).toBe(result2.id);
+      expect(e.operationId).toBe(result2.id);
     }
   });
 
@@ -421,14 +420,14 @@ describe("Decision Diary — retrieval dimensions", () => {
     const artifactA = seedArtifact(artifactStore, "app-a");
     const artifactB = seedArtifact(artifactStore, "app-b");
 
-    await agent.triggerDeployment({
-      artifactId: artifactA.id, artifactVersionId: "1.0.0",
-      partitionId: partA.id, environmentId: envA.id, triggeredBy: "user",
-    });
-    await agent.triggerDeployment({
-      artifactId: artifactB.id, artifactVersionId: "1.0.0",
-      partitionId: partB.id, environmentId: envB.id, triggeredBy: "user",
-    });
+    await agent.triggerOperation(
+      { type: "deploy", artifactId: artifactA.id, artifactVersionId: "1.0.0" },
+      { partitionId: partA.id, environmentId: envA.id, triggeredBy: "user" },
+    );
+    await agent.triggerOperation(
+      { type: "deploy", artifactId: artifactB.id, artifactVersionId: "1.0.0" },
+      { partitionId: partB.id, environmentId: envB.id, triggeredBy: "user" },
+    );
 
     const entriesA = diary.getByPartition(partA.id);
     const entriesB = diary.getByPartition(partB.id);
@@ -550,7 +549,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
   it("persists entries across close and reopen", () => {
     const entry = diary.record({
       partitionId: "partition-1",
-      deploymentId: "deploy-1",
+      operationId: "deploy-1",
       agent: "command",
       decisionType: "pipeline-plan",
       decision: "Planned deployment pipeline: resolve → execute → verify",
@@ -567,7 +566,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     expect(retrieved!.decision).toBe(entry.decision);
     expect(retrieved!.reasoning).toBe(entry.reasoning);
     expect(retrieved!.partitionId).toBe("partition-1");
-    expect(retrieved!.deploymentId).toBe("deploy-1");
+    expect(retrieved!.operationId).toBe("deploy-1");
     expect(retrieved!.decisionType).toBe("pipeline-plan");
     expect(retrieved!.context).toEqual({ artifactId: "web-app", version: "1.0.0" });
     diary2.close();
@@ -576,7 +575,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
   it("retrieval by deployment returns correct entries", () => {
     diary.record({
       partitionId: "t1",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "pipeline-plan",
       decision: "Planned pipeline for d1",
@@ -584,7 +583,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     });
     diary.record({
       partitionId: "t1",
-      deploymentId: "d2",
+      operationId: "d2",
       agent: "command",
       decisionType: "pipeline-plan",
       decision: "Planned pipeline for d2",
@@ -592,28 +591,28 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     });
     diary.record({
       partitionId: "t1",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "deployment-completion",
       decision: "Deployment d1 completed",
       reasoning: "All steps passed.",
     });
 
-    const d1Entries = diary.getByDeployment("d1");
+    const d1Entries = diary.getByOperation("d1");
     expect(d1Entries).toHaveLength(2);
     for (const e of d1Entries) {
-      expect(e.deploymentId).toBe("d1");
+      expect(e.operationId).toBe("d1");
     }
 
-    const d2Entries = diary.getByDeployment("d2");
+    const d2Entries = diary.getByOperation("d2");
     expect(d2Entries).toHaveLength(1);
-    expect(d2Entries[0].deploymentId).toBe("d2");
+    expect(d2Entries[0].operationId).toBe("d2");
   });
 
   it("retrieval by partition returns correct entries", () => {
     diary.record({
       partitionId: "acme",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "pipeline-plan",
       decision: "Acme deployment plan",
@@ -621,7 +620,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     });
     diary.record({
       partitionId: "beta",
-      deploymentId: "d2",
+      operationId: "d2",
       agent: "command",
       decisionType: "pipeline-plan",
       decision: "Beta deployment plan",
@@ -640,7 +639,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
   it("retrieval by decision type filters correctly", () => {
     diary.record({
       partitionId: "t1",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "health-check",
       decision: "Health check passed",
@@ -648,7 +647,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     });
     diary.record({
       partitionId: "t1",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "variable-conflict",
       decision: "LOG_LEVEL conflict resolved",
@@ -656,7 +655,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     });
     diary.record({
       partitionId: "t1",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "health-check",
       decision: "Post-flight check passed",
@@ -678,7 +677,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     const before = new Date();
     diary.record({
       partitionId: "t1",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "pipeline-plan",
       decision: "First entry",
@@ -686,7 +685,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     });
     diary.record({
       partitionId: "t1",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "deployment-completion",
       decision: "Second entry",
@@ -709,7 +708,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
     for (let i = 0; i < 5; i++) {
       diary.record({
         partitionId: "t1",
-        deploymentId: `d${i}`,
+        operationId: `d${i}`,
         agent: "command",
         decisionType: "pipeline-plan",
         decision: `Entry ${i}`,
@@ -731,7 +730,7 @@ describe("PersistentDecisionDebrief — SQLite backing store", () => {
   it("context round-trips through JSON correctly", () => {
     const entry = diary.record({
       partitionId: "t1",
-      deploymentId: "d1",
+      operationId: "d1",
       agent: "command",
       decisionType: "health-check",
       decision: "Health check with complex context",
@@ -800,14 +799,14 @@ describe("PersistentDecisionDebrief — integration with SynthAgent", () => {
     expect(result.status).toBe("succeeded");
 
     // Verify entries exist before close
-    const entriesBefore = diary.getByDeployment(result.id);
+    const entriesBefore = diary.getByOperation(result.id);
     expect(entriesBefore.length).toBeGreaterThanOrEqual(5);
 
     diary.close();
 
     // Reopen and verify persistence
     const diary2 = new PersistentDecisionDebrief(dbPath);
-    const entriesAfter = diary2.getByDeployment(result.id);
+    const entriesAfter = diary2.getByOperation(result.id);
     expect(entriesAfter).toHaveLength(entriesBefore.length);
 
     // Verify key decision types from the pipeline are present
@@ -832,18 +831,18 @@ describe("PersistentDecisionDebrief — integration with SynthAgent", () => {
     const artA = seedArtifact(artifactStore, "app-a");
     const artB = seedArtifact(artifactStore, "app-b");
 
-    const result1 = await agent.triggerDeployment({
-      artifactId: artA.id, artifactVersionId: "1.0.0",
-      partitionId: partA.id, environmentId: envA.id, triggeredBy: "user",
-    });
-    const result2 = await agent.triggerDeployment({
-      artifactId: artB.id, artifactVersionId: "1.0.0",
-      partitionId: partB.id, environmentId: envB.id, triggeredBy: "user",
-    });
+    const result1 = await agent.triggerOperation(
+      { type: "deploy", artifactId: artA.id, artifactVersionId: "1.0.0" },
+      { partitionId: partA.id, environmentId: envA.id, triggeredBy: "user" },
+    );
+    const result2 = await agent.triggerOperation(
+      { type: "deploy", artifactId: artB.id, artifactVersionId: "1.0.0" },
+      { partitionId: partB.id, environmentId: envB.id, triggeredBy: "user" },
+    );
 
     // By deployment
-    const acmeEntries = diary.getByDeployment(result1.id);
-    const betaEntries = diary.getByDeployment(result2.id);
+    const acmeEntries = diary.getByOperation(result1.id);
+    const betaEntries = diary.getByOperation(result2.id);
     expect(acmeEntries.length).toBeGreaterThanOrEqual(5);
     expect(betaEntries.length).toBeGreaterThanOrEqual(5);
 
@@ -879,7 +878,7 @@ describe("Decision Diary — human-readable format", () => {
       id: "abc-123-def-456",
       timestamp: new Date("2026-02-23T14:30:05.000Z"),
       partitionId: "partition-acme",
-      deploymentId: "deploy-789",
+      operationId: "deploy-789",
       agent: "command",
       decisionType: "health-check",
       decision: "Pre-flight health check passed",
@@ -905,7 +904,7 @@ describe("Decision Diary — human-readable format", () => {
       id: "sys-001",
       timestamp: new Date("2026-02-23T12:00:00.000Z"),
       partitionId: null,
-      deploymentId: null,
+      operationId: null,
       agent: "command",
       decisionType: "system",
       decision: "Command initialized with demo data",
@@ -925,7 +924,7 @@ describe("Decision Diary — human-readable format", () => {
         id: "e1",
         timestamp: new Date("2026-02-23T14:00:00.000Z"),
         partitionId: "t1",
-        deploymentId: "d1",
+        operationId: "d1",
         agent: "command",
         decisionType: "pipeline-plan",
         decision: "Entry one",
@@ -936,7 +935,7 @@ describe("Decision Diary — human-readable format", () => {
         id: "e2",
         timestamp: new Date("2026-02-23T14:01:00.000Z"),
         partitionId: "t1",
-        deploymentId: "d1",
+        operationId: "d1",
         agent: "command",
         decisionType: "deployment-completion",
         decision: "Entry two",

package/tests/diary-reader.test.ts

@@ -141,14 +141,10 @@ async function testDeploy(
   forceInsertEnvironment(stores.environments, e);
   forceInsertArtifact(stores.artifacts, DEFAULT_ARTIFACT_ID, "web-app");
 
-  return agent.triggerDeployment({
-    artifactId: DEFAULT_ARTIFACT_ID,
-    artifactVersionId: version,
-    environmentId: e.id,
-    partitionId: effectivePartition.id,
-    triggeredBy: "user",
-    variables: oldTrigger.variables,
-  });
+  return agent.triggerOperation(
+    { type: "deploy", artifactId: DEFAULT_ARTIFACT_ID, artifactVersionId: version },
+    { environmentId: e.id, partitionId: effectivePartition.id, triggeredBy: "user", variables: oldTrigger.variables },
+  );
 }
 
 // ---------------------------------------------------------------------------
@@ -190,7 +186,7 @@ describe("Simulated Postmortem — failed deployment read experience", () => {
 
     expect(deployment.status).toBe("failed");
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     // 1. Reviewer can identify WHAT was being deployed
@@ -241,7 +237,7 @@ describe("Simulated Postmortem — failed deployment read experience", () => {
 
     const deployment = await testDeploy(agent, stores, {});
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     // The postmortem should explain that DNS failures don't benefit from retry
@@ -284,7 +280,7 @@ describe("Simulated Postmortem — failed deployment read experience", () => {
 
     expect(deployment.status).toBe("failed");
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     // Should explain the configuration block
@@ -313,7 +309,7 @@ describe("Simulated Postmortem — failed deployment read experience", () => {
 
     expect(deployment.status).toBe("succeeded");
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     expect(postmortem.summary).toContain("SUCCEEDED");
@@ -333,7 +329,7 @@ describe("Simulated Postmortem — failed deployment read experience", () => {
 
     expect(deployment.status).toBe("succeeded");
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     // Timeline should show the retry decision chain
@@ -368,7 +364,7 @@ describe("Simulated Postmortem — failed deployment read experience", () => {
       makeEnvironment({ name: "staging" }),
     );
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     const text = postmortem.formatted;
@@ -724,7 +720,7 @@ describe("Postmortem report — structural guarantees", () => {
 
     const deployment = await testDeploy(agent, stores, {});
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     for (let i = 1; i < postmortem.timeline.length; i++) {
@@ -747,7 +743,7 @@ describe("Postmortem report — structural guarantees", () => {
 
     const deployment = await testDeploy(agent, stores, { variables: { LOG_LEVEL: "debug" } }, partition, env);
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     // LOG_LEVEL has three-way conflict (env → partition → trigger), total vars = 3
@@ -760,7 +756,7 @@ describe("Postmortem report — structural guarantees", () => {
 
     const deployment = await testDeploy(agent, stores, {});
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     expect(postmortem.failureAnalysis).toBeNull();
@@ -771,7 +767,7 @@ describe("Postmortem report — structural guarantees", () => {
 
     const deployment = await testDeploy(agent, stores, {});
 
-    const entries = diary.getByDeployment(deployment.id);
+    const entries = diary.getByOperation(deployment.id);
     const postmortem = generatePostmortem(entries, deployment);
 
     expect(postmortem.failureAnalysis).not.toBeNull();

package/tests/mcp-tools.test.ts

@@ -207,7 +207,7 @@ describe("get-deployment-status", () => {
     });
 
     expect(result.isError).toBe(true);
-    expect(result.content[0].text).toContain("Deployment not found");
+    expect(result.content[0].text).toContain("Operation not found");
     expect(result.content[0].text).toContain("does-not-exist");
   });
 });