engineering-memory 0.2.2 → 0.2.3

package/install/installer.mjs

@@ -107,8 +107,29 @@ export async function installEngineeringMemory(options = {}) {
       dispatcherSection: dispatcherSections.claude,
     },
   };
+  const mcpPlans = await planMcpRegistrations({
+    selectedClients,
+    bridgeEntry,
+    nodePath,
+    apiUrl,
+    clientVersion,
+    state: existingState,
+    commandRunner,
+    clientsWereRequested: options.selectedClients !== undefined,
+  });
+  const absentClients = mcpPlans
+    .filter((plan) => plan.action === 'absent')
+    .map((plan) => plan.clientName);
+  const installedClients = selectedClients.filter(
+    (clientName) => !absentClients.includes(clientName),
+  );
+  if (installedClients.length === 0) {
+    throw new Error(
+      `None of the supported agent CLIs are on PATH: ${selectedClients.join(', ')}. Install one, then rerun the installer.`,
+    );
+  }
   const clientPlans = await Promise.all(
-    selectedClients.map(async (clientName) => {
+    installedClients.map(async (clientName) => {
       const target = clientTargets[clientName];
       return {
         clientName,
@@ -120,15 +141,6 @@ export async function installEngineeringMemory(options = {}) {
       };
     }),
   );
-  const mcpPlans = await planMcpRegistrations({
-    selectedClients,
-    bridgeEntry,
-    nodePath,
-    apiUrl,
-    clientVersion,
-    state: existingState,
-    commandRunner,
-  });
   const hookPlan = await planGitHook({
     repoRoot: options.repoRoot ? resolve(options.repoRoot) : null,
     mode: options.hookMode ?? 'cancel',
@@ -189,7 +201,8 @@ export async function installEngineeringMemory(options = {}) {
       statePath,
       skillPaths: clientPlans.map((plan) => plan.skillPath),
       dispatcherPaths: clientPlans.map((plan) => plan.dispatcherPath),
-      selectedClients,
+      selectedClients: installedClients,
+      absentClients,
       apiUrl,
       clientVersion,
       runtimePath,

package/install/mcp-registration.mjs

@@ -48,6 +48,7 @@ export async function planMcpRegistrations({
   clientVersion,
   state,
   commandRunner,
+  clientsWereRequested = true,
 }) {
   const plans = [];
   for (const clientName of selectedClients) {
@@ -55,6 +56,10 @@ export async function planMcpRegistrations({
     if (!client) throw new Error(`Unsupported client: ${clientName}`);
     const version = await commandRunner(client.executable, ['--version']);
     if (version.code !== 0) {
+      if (!clientsWereRequested && !state?.mcp?.[clientName]) {
+        plans.push({ clientName, client, action: 'absent' });
+        continue;
+      }
       throw new Error(
         `${clientName} CLI is unavailable. Install its native CLI, ensure it is on PATH, then rerun the installer. ${commandFailure(client.executable, version)}`,
       );
@@ -121,6 +126,7 @@ export async function planMcpRegistrations({
 export async function applyMcpRegistrations(plans, commandRunner, transaction) {
   const result = {};
   for (const plan of plans) {
+    if (plan.action === 'absent') continue;
     if (plan.action === 'add') {
       await addRegistration(plan, plan.registration, commandRunner);
       transaction.add(() => removeRegistration(plan, commandRunner));
@@ -211,6 +217,19 @@ function registrationFingerprint(clientName, registration) {
     .digest('hex');
 }
 
+function fingerprintBeforeClientVersion(clientName, registration) {
+  return createHash('sha256')
+    .update(
+      JSON.stringify({
+        clientName,
+        nodePath: registration.nodePath,
+        bridgeEntry: registration.bridgeEntry,
+        apiUrl: registration.apiUrl ?? null,
+      }),
+    )
+    .digest('hex');
+}
+
 function sameRegistration(left, right) {
   return (
     left?.nodePath === right.nodePath &&
@@ -462,9 +481,13 @@ function isOwnedRegistration(clientName, managed) {
       typeof managed.bridgeEntry === 'string'
     );
   }
+  if (typeof managed.apiUrl !== 'string') return false;
+  if (managed.fingerprint === registrationFingerprint(clientName, managed)) {
+    return true;
+  }
   return (
-    typeof managed.apiUrl === 'string' &&
-    managed.fingerprint === registrationFingerprint(clientName, managed)
+    managed.clientVersion === undefined &&
+    managed.fingerprint === fingerprintBeforeClientVersion(clientName, managed)
   );
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "engineering-memory",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Installs the Engineering Memory skill and its local MCP bridge. Sign in after installing; your organization and project are resolved from your account.",
   "license": "UNLICENSED",
   "type": "module",