agents-can-communicate 0.5.3 → 0.5.5

package/node_modules/@agents-can-communicate/installer/src/apply.mjs

@@ -1,3 +1,4 @@
+import { applyServiceSetup } from "./service-setup.mjs";
 import { applyNativeActivation, deactivateNative } from "./native-activation.mjs";
 import { finalizeRemoval, missingArtifactParents, recordInstall,
   removeEmptyOwnedDirectories, removeOwnedArtifacts } from "./ownership.mjs";
@@ -68,7 +69,8 @@ export async function applyPlan({ plan, adapters, context, dataHome, dryRun = fa
         await recordInstall({ dataHome, adapterId: adapter.id,
           version: operation.clientVersion ?? null, accVersion,
           artifacts: operation.artifacts, createdDirectories,
-          deliveryPolicy: operation.livePolicy, nativeActivation: native });
+          deliveryPolicy: operation.livePolicy, deliveryDecision: operation.deliveryDecision,
+          nativeActivation: native });
         results.operations.push({ ...operation, applied: true, appendedRcBlock,
           needsAction: outcome.needsAction ?? [],
           changes: outcome.changes ?? [], diagnostics: [
@@ -77,6 +79,8 @@ export async function applyPlan({ plan, adapters, context, dataHome, dryRun = fa
             ...(outcome.diagnostics ?? []),
             ...notes,
           ] });
+        await applyServiceSetup({ adapter, context: installContext,
+          operation: results.operations.at(-1), results });
       } else {
         // Config ownership can veto removal. Ask before teardown or recorded
         // artifact deletion; an adapter-local check would run after those writes.
@@ -131,7 +135,7 @@ function describeTeardown(report) {
   for (const service of report.services) {
     lines.push(service.outcome === "stopped" ? `stopped the ${service.serviceId} service`
       : `retained the ${service.serviceId} service (${service.outcome === "retained_pre_existing"
-        ? "it existed before ACC" : "no vendor teardown exists"})`);
+        ? "it was reused at activation" : "no vendor teardown exists"})`);
   }
   return lines;
 }

package/node_modules/@agents-can-communicate/installer/src/detect.mjs

@@ -145,6 +145,17 @@ export async function detectInstallation({ adapters, context, probe = spawnProbe
         : unsupported(adapter.nativeDelivery === undefined
           ? "native_delivery_unsupported" : "version_unavailable");
 
+      if (entry.present && typeof adapter.inspectNativeServiceSetup === "function") {
+        try {
+          entry.nativeServiceSetup = await withTimeout(Promise.resolve().then(() =>
+            adapter.inspectNativeServiceSetup({ ...context, platform, clientVersion: entry.version })),
+          probeTimeoutMs, `${adapter.id} service setup probe`);
+        } catch {
+          entry.nativeServiceSetup = { state: "blocked", reasonCode: "service_inspection_failed",
+            diagnostic: `${adapter.displayName} service preparation could not be inspected; check the vendor service and retry install` };
+        }
+      }
+
       try {
         const detected = await adapter.detect({ ...context, clientVersion: entry.version,
           platform, nativeDelivery: entry.nativeDelivery });

package/node_modules/@agents-can-communicate/installer/src/ownership.mjs

@@ -110,7 +110,8 @@ async function saveOwnership({ dataHome, record }) {
  * runtime, and leaves the bundle inside the client exactly where it was.
  */
 export async function recordInstall({ dataHome, adapterId, version, accVersion = null,
-  artifacts, createdDirectories = [], deliveryPolicy, nativeActivation = null }) {
+  artifacts, createdDirectories = [], deliveryPolicy, deliveryDecision,
+  nativeActivation = null }) {
   const stamped = await Promise.all(artifacts.map(async artifact => ({
     path: artifact.path,
     kind: artifact.kind ?? "file",
@@ -120,6 +121,7 @@ export async function recordInstall({ dataHome, adapterId, version, accVersion =
   })));
   const record = await loadOwnership({ dataHome });
   const previous = record.installs.find(install => install.adapterId === adapterId);
+  const retainedDeliveryDecision = deliveryDecision ?? previous?.deliveryDecision;
   const directories = [...new Set([
     ...(previous?.createdDirectories ?? []), ...createdDirectories,
   ])].sort((left, right) => left.split(path.sep).length - right.split(path.sep).length
@@ -129,6 +131,9 @@ export async function recordInstall({ dataHome, adapterId, version, accVersion =
       { adapterId, version, accVersion, artifacts: stamped,
         ...(directories.length === 0 ? {} : { createdDirectories: directories }),
         ...(deliveryPolicy === undefined ? {} : { deliveryPolicy }),
+        ...(retainedDeliveryDecision === undefined ? {} : {
+          deliveryDecision: retainedDeliveryDecision,
+        }),
         // Present only for a consented native activation. A record without it
         // - every 0.2 install - reads as live policy off and is never rewritten
         // merely to add the field.

package/node_modules/@agents-can-communicate/installer/src/plan.mjs

@@ -14,7 +14,7 @@ import { LIVE_POLICIES, describeActivation, describeDeactivation, planActivation
  */
 export function planInstallation({ adapters, detected, context, action = "install",
   recorded = [], accVersion = null, allowDowngrade = false, requested = [],
-  deliveryByAdapter = {} }) {
+  deliveryByAdapter = {}, deliveryDecisionByAdapter = {}, allowServiceSetup = false }) {
   if (!["install", "uninstall"].includes(action)) {
     throw new AccError(EXIT.USAGE, `unknown installation action: ${action}`, { action });
   }
@@ -104,6 +104,9 @@ export function planInstallation({ adapters, detected, context, action = "instal
     // and so what will be removed. Asking the adapter instead would describe an
     // install for a machine this one no longer is.
     const delivery = deliveryByAdapter[entry.adapterId] ?? "off";
+    const deliveryDecision = deliveryDecisionByAdapter[entry.adapterId]
+      ?? recordedById.get(entry.adapterId)?.deliveryDecision
+      ?? { source: "legacy-unknown", completeSetup: false };
     const native = entry.nativeDelivery ?? null;
     const liveDeliverySupported = native?.state === "eligible"
       && native.activationPlan?.eligible === true;
@@ -122,8 +125,10 @@ export function planInstallation({ adapters, detected, context, action = "instal
       ? describeInstallDelivery(entry, delivery, effectiveLivePolicy) : null;
     const installContext = { ...context, requestedLivePolicy: delivery,
       livePolicy: configuredLivePolicy, clientVersion: entry.version, platform: entry.platform };
+    const nativeServiceSetup = action === "install" && allowServiceSetup && delivery !== "off"
+      && deliveryDecision.completeSetup === true ? entry.nativeServiceSetup : undefined;
     const setupNotes = action === "install" && delivery !== "off"
-      ? [entry.outgoingDelivery?.setup, entry.nativeSetup].filter(note => typeof note === "string") : [];
+      ? [entry.outgoingDelivery?.setup, nativeServiceSetup ? null : entry.nativeSetup].filter(note => typeof note === "string") : [];
     // A consented activation that this run keeps, activates, or takes back.
     // Only an explicit off or an uninstall removes one; an absent record never
     // creates one.
@@ -154,11 +159,13 @@ export function planInstallation({ adapters, detected, context, action = "instal
       alreadyInstalled: entry.installed === true,
       livePolicy: delivery,
       effectiveLivePolicy,
+      ...(action === "install" ? { deliveryDecision } : {}),
       ...(retainedActivation ? { configuredLivePolicy, retainedNativeActivation: retainedActivation } : {}),
       ...(deliveryDiagnostic === null ? {} : { deliveryDiagnostic }),
       ...(deliverySummary === null ? {} : { deliverySummary }),
       ...(setupNotes.length ? { setupNotes } : {}),
       ...(nativeActivation === null ? {} : { nativeActivation }),
+      ...(nativeServiceSetup ? { nativeServiceSetup } : {}),
       ...(deactivation === null ? {} : { deactivation }),
       artifacts,
       // Said in the operator's terms, not in paths: which files ACC creates
@@ -169,6 +176,7 @@ export function planInstallation({ adapters, detected, context, action = "instal
         ...(deliverySummary === null ? [] : [deliverySummary]),
         ...(retainedActivation ? ["keep existing native delivery setup; current readiness is unverified"] : []),
         ...setupNotes,
+        ...(nativeServiceSetup ? [nativeServiceSetup.diagnostic] : []),
         ...artifacts.filter(a => a.kind === "tree")
           .map(a => `${action === "install" ? "create" : "remove"} ${a.path}`),
         ...artifacts.filter(a => a.kind === "merge")

package/node_modules/@agents-can-communicate/installer/src/service-setup.mjs

@@ -0,0 +1,30 @@
+// Invoked only after adapter configuration and ownership/consent persistence.
+// A failed service command must not erase the successful adapter operation.
+export async function applyServiceSetup({ adapter, context, operation, results }) {
+  const plan = operation.nativeServiceSetup;
+  if (!plan || operation.livePolicy === "off" || operation.deliveryDecision?.completeSetup !== true) return;
+  let setup = plan;
+  if (["needed", "ready"].includes(plan.state)) {
+    try {
+      setup = await adapter.prepareNativeServiceSetup({ context, plan });
+      if (!setup || !["ready", "failed", "blocked"].includes(setup.state)) throw new Error("invalid setup result");
+    } catch {
+      setup = { state: "failed", started: false, reasonCode: "service_setup_failed",
+        diagnostic: `${adapter.displayName} service preparation failed; check the vendor service and retry acc install` };
+    }
+  }
+  operation.nativeServiceSetup = setup;
+  if (setup.state === "ready") {
+    const stale = new Set([operation.deliverySummary, operation.deliveryDiagnostic, plan.diagnostic]);
+    operation.summary = (operation.summary ?? []).filter(line => !stale.has(line));
+    operation.summary.push(setup.diagnostic);
+    operation.diagnostics = (operation.diagnostics ?? []).filter(line => !stale.has(line));
+    operation.deliverySummary = setup.diagnostic;
+    operation.deliveryDiagnostic = setup.diagnostic;
+  } else {
+    operation.needsAction.push(setup.diagnostic);
+    if (setup.state === "failed") results.failed.push({ adapterId: operation.adapterId,
+      error: setup.diagnostic, reasonCode: setup.reasonCode });
+  }
+  operation.diagnostics.push(setup.diagnostic);
+}

package/node_modules/@agents-can-communicate/mcp-server/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/mcp-server",
-  "version": "0.5.3",
+  "version": "0.5.5",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/protocol/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/protocol",
-  "version": "0.5.3",
+  "version": "0.5.5",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/storage-filesystem/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/storage-filesystem",
-  "version": "0.5.3",
+  "version": "0.5.5",
   "private": true,
   "type": "module",
   "exports": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-can-communicate",
-  "version": "0.5.3",
+  "version": "0.5.5",
   "accManagedUpdateProtocol": 2,
   "accStoreVersion": 6,
   "type": "module",