@lamalibre/install-portlama-e2e-mcp 0.1.1 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lamalibre/install-portlama-e2e-mcp",
-  "version": "0.1.1",
+  "version": "0.1.4",
   "description": "MCP server for Portlama E2E test infrastructure — VM lifecycle, snapshots, test execution",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE.md",

package/src/lib/deps.js

@@ -79,7 +79,6 @@ export const SINGLE_VM_DEPS = {
   11: [3], // input-validation
   12: [3], // user-invitations
   13: [3], // site-lifecycle
-  14: [3], // shell-lifecycle
   15: [3], // plugin-lifecycle
   16: [3], // enrollment-tokens
 };
@@ -99,7 +98,6 @@ export const THREE_VM_DEPS = {
   7: [1], // site-visitor-journey
   8: [1], // invitation-journey
   9: [1], // agent-site-deploy
-  10: [1], // shell-lifecycle
   11: [1], // plugin-lifecycle
   12: [1], // enrollment-lifecycle
 };

package/src/lib/multipass.js

@@ -42,15 +42,9 @@ export async function launch(name, { cpus, memory, disk }) {
   ]);
 }
 
-/** Delete a VM and purge all deleted instances. */
+/** Delete a VM and purge only that VM (not other users' deleted VMs). */
 export async function deleteVm(name) {
-  await run(['delete', name], { allowFailure: true });
-  await run(['purge'], { allowFailure: true });
-}
-
-/** Delete a VM without purging (caller handles purge). */
-export async function deleteVmNoPurge(name) {
-  await run(['delete', name], { allowFailure: true });
+  await run(['delete', '--purge', name], { allowFailure: true });
 }
 
 /** Get info for a VM as JSON. Returns null if VM doesn't exist. */

package/src/tools/provision.js

@@ -69,12 +69,21 @@ export const provisionHostTool = {
     const tarball = await packPackage('create-portlama');
     steps.push(`Packed installer: ${tarball}`);
 
-    // 2. Install npm on host
-    await mp.exec(VM_HOST, 'apt-get install -y npm', {
-      sudo: true,
-      timeout: 120_000,
-    });
-    steps.push('npm installed on host');
+    // 2. Ensure npm is available (NodeSource images ship with Node+npm)
+    const npmCheck = await mp.exec(VM_HOST, 'npm --version', { allowFailure: true });
+    if (npmCheck.exitCode !== 0) {
+      await mp.exec(VM_HOST, 'apt-get update', {
+        sudo: true,
+        timeout: 120_000,
+      });
+      await mp.exec(VM_HOST, 'apt-get install -y npm', {
+        sudo: true,
+        timeout: 120_000,
+      });
+      steps.push('npm installed on host');
+    } else {
+      steps.push(`npm already available (v${npmCheck.stdout.trim()})`);
+    }
 
     // 3. Transfer and install
     await mp.transfer(tarball, `${VM_HOST}:/tmp/create-portlama.tgz`);

package/src/tools/vm.js

@@ -30,21 +30,16 @@ export const vmCreateTool = {
 
     const results = [];
 
-    // Delete existing VMs in parallel, single purge at the end
-    let needsPurge = false;
+    // Delete existing VMs in parallel (per-VM purge, won't affect other VMs)
     await Promise.all(
       targets.map(async (name) => {
         const existing = await mp.info(name);
         if (existing) {
-          await mp.deleteVmNoPurge(name);
-          needsPurge = true;
+          await mp.deleteVm(name);
           results.push(`Deleted existing ${name}`);
         }
       }),
     );
-    if (needsPurge) {
-      await mp.run(['purge'], { allowFailure: true });
-    }
 
     // Create VMs in parallel
     await Promise.all(
@@ -105,14 +100,13 @@ export const vmDeleteTool = {
   async handler({ vms } = {}) {
     const targets = vms ? vms.map((v) => VM_NAME_MAP[v]) : ALL_VMS;
 
-    // Delete VMs in parallel (purge runs once at the end)
+    // Delete VMs in parallel (per-VM purge, won't affect other VMs)
     await Promise.all(
       targets.map(async (name) => {
-        await mp.run(['delete', name], { allowFailure: true });
+        await mp.deleteVm(name);
         removeVmState(name);
       }),
     );
-    await mp.run(['purge'], { allowFailure: true });
 
     return {
       content: [