impel-cli 0.14.0 → 0.14.2

package/README.md

@@ -630,8 +630,8 @@ every managed profile. This catalog is separate from automatic specialists: it
 may include truthfully declared write-capable agents, but those agents require
 explicit user selection before the server will start a run.
 
-- Claude Code: `<CLAUDE_CONFIG_DIR>/agents/impel-managed/*.md`
-- Codex / ChatGPT Codex mode: `<CODEX_HOME>/agents/impel-managed/*.toml`
+- Claude Code / Claude Desktop: `<CLAUDE_CONFIG_DIR>/agents/<agent-name>.md`
+- Codex / ChatGPT Codex mode: `<CODEX_HOME>/agents/impel-<tenant>-<agent-id>.toml`
 
 Each generated agent is bound to one exact `agentId` and
 `scopeParam`. It verifies that the exact agent is still available, starts one
@@ -643,8 +643,8 @@ when invoked.
 
 Visible agent names come from the registry title. Codex keeps that title as the
 custom-agent name; Claude uses the lowercase, hyphenated form required by its
-native agent schema. The generated filenames remain stable tenant-and-agent
-identifiers so title changes update in place without colliding with user files.
+native agent schema and Desktop's `@` picker requires the Markdown filename to
+match that name. Codex filenames remain stable tenant-and-agent identifiers.
 Duplicate or built-in-reserved titles receive a deterministic disambiguator.
 
 The sync writes atomically and records a manifest beside the generated files.
@@ -657,6 +657,10 @@ without contacting the gateway.
 Start a new task after syncing. Claude Code detects updates to an existing
 `agents/` directory within a few seconds, but if the sync created that directory
 for the first time while Claude was already running, restart Claude once.
+In Claude Desktop, selecting an agent on the first turn binds that Code session
+to the agent. Follow-up `@` menus continue to offer that same bound agent and
+insert a normal mention chip. Other agents remain available from a new task,
+because the running Claude session cannot safely switch agents mid-session.
 
 Invocation uses the clients' native agent behavior:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "impel-cli",
-  "version": "0.14.0",
+  "version": "0.14.2",
   "description": "Configure Claude Code and Codex CLI to talk to Impel's gateway, authenticated by an Impel Personal Access Token",
   "type": "module",
   "bin": {

package/src/agents.js

@@ -398,10 +398,16 @@ export function renderManagedAgents(client, tenantId, agents, invocation = impel
   return agents.map((agent, index) => {
     const name = names[index];
     const extension = client === "claude" ? ".md" : ".toml";
+    // Claude Desktop decides whether an SDK-reported agent is user-defined by
+    // comparing its reported name with the Markdown filename (without .md).
+    // A stable tenant/agent filename causes Desktop to classify the agent as a
+    // builtin and remove it from the @ mention picker. Claude names are already
+    // collision-safe, filesystem-safe identifiers, so use them as filenames.
+    const fileStem = client === "claude" ? name : fileStems[index];
     const contents = client === "claude"
       ? renderClaudeAgent({ tenantId: normalizedTenant, agent, name, invocation })
       : renderCodexAgent({ tenantId: normalizedTenant, agent, name, invocation });
-    return { agentId: agent.agentId, name, fileName: `${fileStems[index]}${extension}`, contents };
+    return { agentId: agent.agentId, name, fileName: `${fileStem}${extension}`, contents };
   });
 }
 
@@ -432,7 +438,7 @@ function readManifest(manifestPath) {
 
 function profileIsFresh(profile, tenantId, now, ttlMs) {
   const manifest = readManifest(path.join(profile.root, "agents", MANAGED_AGENT_DIRECTORY, MANAGED_AGENT_MANIFEST));
-  if (!manifest || manifest.version !== 2 || manifest.tenantId !== tenantId) return false;
+  if (!manifest || manifest.version !== 3 || manifest.tenantId !== tenantId) return false;
   const syncedAt = Date.parse(manifest.syncedAt || "");
   if (!Number.isFinite(syncedAt) || now - syncedAt >= ttlMs) return false;
   return manifest.files.every((fileName) =>
@@ -455,7 +461,7 @@ export function syncAgentProfile({ client, root, label, tenantId, agents, now =
   const prior = readManifest(manifestPath);
   const rendered = renderManagedAgents(client, tenantId, agents);
   const priorFiles = new Set(prior?.files || []);
-  const priorUsesDiscoveryRoot = prior?.version === 2;
+  const priorUsesDiscoveryRoot = prior?.version === 2 || prior?.version === 3;
 
   // Native clients discover standalone definitions directly under `agents/`.
   // Preflight every destination before writing so an unmanaged file with the
@@ -497,7 +503,7 @@ export function syncAgentProfile({ client, root, label, tenantId, agents, now =
     }
   }
   atomicPrivateWrite(manifestPath, `${JSON.stringify({
-    version: 2,
+    version: 3,
     tenantId,
     client,
     syncedAt: new Date(now).toISOString(),

package/src/apps.js

@@ -83,6 +83,12 @@ const CLAUDE_PLAN_USAGE_GUARD = "if(!Ze().hasOrgPolicyBackend())return;if((r=fr(
 const IMPEL_CLAUDE_PLAN_USAGE_GUARD = `${"if(!wze())return;".padEnd("if(!Ze().hasOrgPolicyBackend())return;".length, " ")}if((r=fr())`;
 const CLAUDE_PLAN_USAGE_REQUEST = 'P.net.fetch(`${ht()}/api/organizations/${o}/usage`,{signal:AbortSignal.timeout(mcr)})';
 const IMPEL_CLAUDE_PLAN_USAGE_REQUEST = 'P.net.fetch("https://gateway.useimpel.com/ui/a/u",{headers:{key:dR(Oe()).apiKey}})'.padEnd(CLAUDE_PLAN_USAGE_REQUEST.length, " ");
+const CLAUDE_DRAFT_AGENT_MENTIONS = 'agentMentionsEnabled:"draft"===Be&&s';
+const IMPEL_CLAUDE_BOUND_AGENT_MENTIONS = 'agentMentionsEnabled:"draft"===Be?s:Le?.agent??!1';
+const CLAUDE_AGENT_MENTION_FILTER = 'return e.filter(e=>"user"===e.source&&(!s||e.name.toLowerCase().includes(s))).slice(0,10)';
+const IMPEL_CLAUDE_BOUND_AGENT_MENTION_FILTER = 'return e.filter(e=>"user"===e.source&&(!s||e.name.toLowerCase().includes(s))&&("string"!=typeof y||e.name===y)).slice(0,10)';
+const CLAUDE_AGENT_MENTION_SELECT = 'onSelect:t=>(e(String(t)),Promise.resolve(null))';
+const IMPEL_CLAUDE_BOUND_AGENT_MENTION_SELECT = 'onSelect:t=>"string"==typeof y?Promise.resolve({chipText:String(t)}):(e(String(t)),Promise.resolve(null))';
 const LEGACY_CLAUDE_SAFE_STORAGE_NAME = "Claude";
 const CLAUDE_SAFE_STORAGE_METADATA = "safe-storage.json";
 
@@ -154,7 +160,7 @@ export const CURRENT_CONFIG_VERSION = 10;
 // — which is what made every `impel update` re-trigger macOS permission
 // prompts. Bump this ONLY when a code change alters the bytes of a built
 // bundle; leave it alone for changes that don't touch bundle contents.
-export const BUNDLE_BUILD_FINGERPRINT = "bundle-2026-07-16.3";
+export const BUNDLE_BUILD_FINGERPRINT = "bundle-2026-07-17.1";
 
 /** Parse the tenant's install manifest, or null when absent/corrupt. */
 export function readTenantManifest(homeDir = os.homedir(), tenantId = null) {
@@ -205,6 +211,12 @@ export function bundleIsCurrent(status, { signingMode = desiredSigningMode() } =
       && compatibility.browserData === status.browserData
     ))
     && managedBundleMatchesCompatibility(status, compatibility)
+    && (status.target !== "claude" || (
+      compatibility.patches?.followupBoundAgentEnable === 1
+      && compatibility.patches?.followupBoundAgentFilter === 1
+      && compatibility.patches?.followupBoundAgentChip === 1
+      && managedClaudeRendererMatchesCompatibility(status, compatibility)
+    ))
     && (status.target !== "chatgpt" || (
       compatibility.vendorSignaturesPreserved === true
       && compatibility.resourceRoot === CHATGPT_RESOURCE_ROOT
@@ -214,6 +226,19 @@ export function bundleIsCurrent(status, { signingMode = desiredSigningMode() } =
   );
 }
 
+function managedClaudeRendererMatchesCompatibility(status, compatibility) {
+  try {
+    const renderer = compatibility.rendererAssets?.agentMentions;
+    if (!renderer?.path || !renderer.sha256 || path.isAbsolute(renderer.path)) return false;
+    const root = path.resolve(status.launcher, "Contents", "Resources", "ion-dist");
+    const target = path.resolve(root, renderer.path);
+    if (!target.startsWith(`${root}${path.sep}`)) return false;
+    return sha256(fs.readFileSync(target)) === renderer.sha256;
+  } catch {
+    return false;
+  }
+}
+
 function managedBundleMatchesCompatibility(status, compatibility) {
   try {
     const managedVendorBundle = status.target === "claude"
@@ -1153,6 +1178,7 @@ function writeVendoredClaudeBundle(paths, vendorPath, gatewayUrl, safeStorageNam
       );
     }
 
+    const agentMentionPatches = patchClaudeFollowupAgentMentions(staging);
     const oldAsarHash = asarHeaderHash(asarPath);
     let planUsageGuardPatchCount = 0;
     let planUsageRequestPatchCount = 0;
@@ -1193,7 +1219,7 @@ function writeVendoredClaudeBundle(paths, vendorPath, gatewayUrl, safeStorageNam
 
     const macos = path.join(staging, "Contents", "MacOS");
     writeAtomic(path.join(staging, "Contents", "Resources", "impel-compatibility.json"), JSON.stringify({
-      schemaVersion: 3,
+      schemaVersion: 4,
       cliVersion: CLI_VERSION,
       bundleFingerprint: BUNDLE_BUILD_FINGERPRINT,
       signingMode: desiredSigningMode(),
@@ -1209,6 +1235,15 @@ function writeVendoredClaudeBundle(paths, vendorPath, gatewayUrl, safeStorageNam
         aggregatePlanUsageGuard: planUsageGuardPatchCount,
         aggregatePlanUsageRequest: planUsageRequestPatchCount,
         safeStorageName: safeStorageNamePatchCount,
+        followupBoundAgentEnable: agentMentionPatches.enable,
+        followupBoundAgentFilter: agentMentionPatches.filter,
+        followupBoundAgentChip: agentMentionPatches.chip,
+      },
+      rendererAssets: {
+        agentMentions: {
+          path: agentMentionPatches.path,
+          sha256: agentMentionPatches.sha256,
+        },
       },
       asarHeaderSha256: newAsarHash,
     }, null, 2) + "\n", 0o644);
@@ -1330,6 +1365,85 @@ function linkVendoredChatGPTResources(bundle, vendorBundleName) {
   }
 }
 
+function patchClaudeFollowupAgentMentions(bundle) {
+  const version = bundleVersion(bundle);
+  if (version !== PINNED_VENDOR_APPS.claude.version) {
+    throw new Error(
+      `Claude follow-up agent mentions support is pinned to ${PINNED_VENDOR_APPS.claude.version}; found ${version || "unknown"}`,
+    );
+  }
+
+  const ionDist = path.join(bundle, "Contents", "Resources", "ion-dist");
+  const contracts = [
+    CLAUDE_DRAFT_AGENT_MENTIONS,
+    CLAUDE_AGENT_MENTION_FILTER,
+    CLAUDE_AGENT_MENTION_SELECT,
+  ];
+  const candidates = listJavaScriptFiles(ionDist).filter((candidate) => {
+    const source = fs.readFileSync(candidate, "utf8");
+    return contracts.every((contract) => source.includes(contract));
+  });
+  if (candidates.length !== 1) {
+    throw new Error(
+      `Claude follow-up agent-mention renderer expected 1 matching asset, found ${candidates.length}; update impel-cli before installing this desktop app version`,
+    );
+  }
+
+  const renderer = candidates[0];
+  let source = fs.readFileSync(renderer, "utf8");
+  const enable = exactTextMatchCount(source, CLAUDE_DRAFT_AGENT_MENTIONS);
+  const filter = exactTextMatchCount(source, CLAUDE_AGENT_MENTION_FILTER);
+  const chip = exactTextMatchCount(source, CLAUDE_AGENT_MENTION_SELECT);
+  for (const [label, count] of [["enable", enable], ["filter", filter], ["chip", chip]]) {
+    if (count !== 1) {
+      throw new Error(`Claude follow-up agent-mention ${label} patch expected 1 match, found ${count}`);
+    }
+  }
+
+  source = source
+    .replace(CLAUDE_DRAFT_AGENT_MENTIONS, IMPEL_CLAUDE_BOUND_AGENT_MENTIONS)
+    .replace(CLAUDE_AGENT_MENTION_FILTER, IMPEL_CLAUDE_BOUND_AGENT_MENTION_FILTER)
+    .replace(CLAUDE_AGENT_MENTION_SELECT, IMPEL_CLAUDE_BOUND_AGENT_MENTION_SELECT);
+  writeAtomic(renderer, source, fs.statSync(renderer).mode & 0o777);
+  return {
+    enable,
+    filter,
+    chip,
+    path: path.relative(ionDist, renderer).split(path.sep).join("/"),
+    sha256: sha256(Buffer.from(source, "utf8")),
+  };
+}
+
+function listJavaScriptFiles(root) {
+  const files = [];
+  const visit = (directory) => {
+    let entries;
+    try {
+      entries = fs.readdirSync(directory, { withFileTypes: true });
+    } catch (error) {
+      if (error?.code === "ENOENT") return;
+      throw error;
+    }
+    for (const entry of entries) {
+      const target = path.join(directory, entry.name);
+      if (entry.isDirectory()) visit(target);
+      else if (entry.isFile() && entry.name.endsWith(".js")) files.push(target);
+    }
+  };
+  visit(root);
+  return files.sort();
+}
+
+function exactTextMatchCount(source, expected) {
+  let count = 0;
+  for (
+    let offset = source.indexOf(expected);
+    offset !== -1;
+    offset = source.indexOf(expected, offset + expected.length)
+  ) count += 1;
+  return count;
+}
+
 function patchFixedWidthAsarString(asarPath, original, replacement, label) {
   if (Buffer.byteLength(original, "latin1") !== Buffer.byteLength(replacement, "latin1")) {
     throw new Error(`${label} replacement must preserve the vendor ASAR byte length`);