@commonlyai/cli 0.1.172 → 0.1.173

package/README.md

@@ -33,11 +33,21 @@ Keep active-instance precedence separate from the newest-match fallback; back-to
 A hosted agent's local handoff card includes `--memory-from <hosted-agent>` and
 `--memory-instance <hosted-instance>` on the attach command. After login, attach
 copies that source's long-term memory into the local identity before returning
-its runnable configuration. Both identities must be accessible through the
-logged-in account's owner memory API. No hosted runtime credential is moved.
+its runnable configuration. The source and target must belong to the logged-in
+owner. No hosted runtime credential is moved.
 
 This is a snapshot, not a shared identity or continuing synchronization. Attach
 refuses to replace different existing local memory or copy into a non-private
-long-term section. A failed copy stops attach; correct the error and retry before
-running the local agent. A retry resumes only your matching active local installation,
+long-term section. The install request completes memory authorization and the
+revision-checked copy before joining the pod, creating an installation or posting
+a greeting. A refused copy leaves those effects absent (an existing installation
+is preserved). A later identity-creation or pod-join refusal can leave the copied
+memory without an installation; retries recognize the snapshot as already copied.
+The server claims the registry name before copying. On copy failure it releases
+only the new, unchanged claim when no installation uses it; a claim touched by
+another in-flight handoff is retained. Memory is not rolled back across later
+identity or pod-join operations. Correct the error and
+retry before running the local agent.
+The CLI checks server support before publishing or installing; deploy the backend
+with install-time handoff support before distributing the updated CLI. A retry resumes only your matching active local installation,
 without changing its configuration or scopes. Repeating a copy with identical memory is safe.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@commonlyai/cli",
-  "version": "0.1.172",
+  "version": "0.1.173",
   "license": "Apache-2.0",
   "description": "The Commonly CLI — connect agents, manage pods, iterate fast",
   "type": "module",

package/src/commands/agent.js

@@ -22,7 +22,6 @@ import { createHash } from 'crypto';
 
 import { requireSeatModel } from '../lib/adapters/seat-model.js';
 import { createClient } from '../lib/api.js';
-import { copyHandoffMemory } from '../lib/handoff-memory.js';
 import {
   buildSpawnId,
   createSpawnCredentialLease,
@@ -987,6 +986,13 @@ export const performAttach = async ({
   // server via legacy normalization).
   const runtimeType = adapter.runtimeType || adapter.name;
 
+  if (memoryFrom) {
+    const capability = await client.get('/api/registry/install/memory-handoff');
+    if (capability?.installTimeMemoryHandoff !== true) {
+      throw new Error('Server does not support install-time memory handoff; upgrade the server before attaching');
+    }
+  }
+
   // Idempotent publish — already-published is expected and fine. Install will
   // surface the real error if the manifest is truly unusable.
   try {
@@ -1010,7 +1016,9 @@ export const performAttach = async ({
   // on config.environment so cross-driver implementations can realize it
   // server-side without re-reading the user's file.
   const installResult = await client.post('/api/registry/install', {
-    ...(memoryFrom ? { resumeExisting: true } : {}),
+    ...(memoryFrom ? {
+      resumeExisting: true, memoryFrom: { agentName: memoryFrom, instanceId: memoryInstance },
+    } : {}),
     agentName,
     podId,
     displayName: displayName || agentName,
@@ -1034,10 +1042,8 @@ export const performAttach = async ({
   const instanceId = installation.instanceId || 'default';
 
   if (memoryFrom) {
-    const memoryCopy = await copyHandoffMemory(client, {
-      sourceAgentName: memoryFrom, sourceInstanceId: memoryInstance,
-      targetAgentName: installation.agentName, targetInstanceId: instanceId,
-    });
+    const memoryCopy = installResult.memoryCopy;
+    if (!memoryCopy) throw new Error('Server does not support install-time memory handoff; upgrade the server before attaching');
     log(memoryCopy.reason === 'source-empty'
       ? 'hosted agent has no long-term memory to copy'
       : 'hosted long-term memory is present in the local agent');

package/src/lib/handoff-memory.js

@@ -1,23 +0,0 @@
-/** Copy only long-term memory using the logged-in owner's revision-checked API. */
-export const copyHandoffMemory = async (client, {
-  sourceAgentName, sourceInstanceId = 'default', targetAgentName, targetInstanceId = 'default',
-}) => {
-  const endpoint = (name, instance) => `/api/agent-memory/${encodeURIComponent(name)}/${encodeURIComponent(instance)}/editable`;
-  const source = await client.get(endpoint(sourceAgentName, sourceInstanceId));
-  const content = source?.sections?.long_term?.content;
-  if (typeof content !== 'string') throw new Error('Hosted memory response is missing long_term content');
-  if (!content) return { copied: false, reason: 'source-empty' };
-  const targetPath = endpoint(targetAgentName, targetInstanceId);
-  const target = await client.get(targetPath);
-  const prior = target?.sections?.long_term;
-  if (typeof prior?.content !== 'string') throw new Error('Local memory response is missing long_term content');
-  if (prior.visibility !== 'private') throw new Error('Local long-term memory must be private before copying hosted memory');
-  if (prior.content === content) return { copied: false, reason: 'already-copied' };
-  if (prior.content) throw new Error('Local agent already has different memory; choose a new local name');
-  // A concurrent first write wins rather than being overwritten by this copy.
-  await client.patch(`${targetPath}/long_term`, {
-    copyFrom: { agentName: sourceAgentName, instanceId: sourceInstanceId },
-    expectedUpdatedAt: prior.updatedAt ?? null,
-  });
-  return { copied: true };
-};