@payloadcms/figma 0.0.1-alpha.63 → 0.0.1-alpha.64

package/dist/utils/payload-package-check.js

@@ -40,24 +40,33 @@ import * as log from './log.js';
  * Install a package as a production dependency
  * @param version - Version to install (without caret). Defaults to 'latest'.
  */ export async function installPackage(projectPath, packageName, packageManager, version = 'latest') {
+    return installPackages(projectPath, [
+        {
+            name: packageName,
+            version
+        }
+    ], packageManager);
+}
+/**
+ * Install multiple packages as production dependencies in a single package-manager call.
+ * No-op when packages is empty. Throws on failure with exit code and truncated stderr.
+ */ export async function installPackages(projectPath, packages, packageManager) {
+    if (packages.length === 0) {
+        return;
+    }
     return new Promise((resolve, reject)=>{
         const command = packageManager;
-        const packageSpec = packageName.endsWith('.tgz') ? packageName : `${packageName}@${version}`;
+        const packageSpecs = packages.map(({ name, version })=>name.endsWith('.tgz') ? name : `${name}@${version ?? 'latest'}`);
         const args = [
-            'add',
-            packageSpec
+            packageManager === 'npm' ? 'install' : 'add',
+            ...packageSpecs
         ];
-        // npm uses 'install' instead of 'add'
-        if (packageManager === 'npm') {
-            args[0] = 'install';
-        }
         log.debug(`Running: ${command} ${args.join(' ')} in ${projectPath}`);
         let stderr = '';
         const child = spawn(command, args, {
             cwd: projectPath,
             stdio: 'pipe'
         });
-        // Capture stderr for debug logging
         if (child.stderr) {
             child.stderr.on('data', (data)=>{
                 stderr += data.toString();
@@ -69,15 +78,17 @@ import * as log from './log.js';
         });
         child.on('close', (code)=>{
             if (code === 0) {
-                log.debug(`Successfully installed ${packageName}`);
+                log.debug(`Successfully installed ${packageSpecs.join(', ')}`);
                 resolve();
-            } else {
-                log.debug(`Failed to install ${packageName} (exit code: ${code})`);
-                if (stderr) {
-                    log.debug(`stderr: ${stderr}`);
-                }
-                reject(new Error(`Failed to install ${packageName}`));
+                return;
+            }
+            log.debug(`Batched install failed (exit code: ${code})`);
+            if (stderr) {
+                log.debug(`stderr: ${stderr}`);
             }
+            const truncatedStderr = stderr.trim().slice(-500);
+            const detail = truncatedStderr ? `\n${truncatedStderr}` : '';
+            reject(new Error(`Failed to install: ${packageSpecs.join(', ')} (exit code ${code})${detail}`));
         });
     });
 }
@@ -85,29 +96,58 @@ import * as log from './log.js';
  * Uninstall a package
  * Does not throw on failure - orphaned package is better than broken config
  */ export async function uninstallPackage(projectPath, packageName, packageManager) {
+    return uninstallPackages(projectPath, [
+        packageName
+    ], packageManager);
+}
+/**
+ * Uninstall multiple packages in a single package-manager call.
+ * No-op when packageNames is empty. Does not throw — orphaned packages are
+ * preferable to a crash mid-config-rewrite. Returns a success flag so callers
+ * can avoid claiming "Uninstalled X" when the package manager actually failed.
+ */ export async function uninstallPackages(projectPath, packageNames, packageManager) {
+    if (packageNames.length === 0) {
+        return {
+            failedPackages: [],
+            success: true
+        };
+    }
     return new Promise((resolve)=>{
         const command = packageManager;
         const args = [
-            'remove',
-            packageName
+            packageManager === 'npm' ? 'uninstall' : 'remove',
+            ...packageNames
         ];
-        // npm uses 'uninstall' instead of 'remove'
-        if (packageManager === 'npm') {
-            args[0] = 'uninstall';
-        }
         log.debug(`Running: ${command} ${args.join(' ')} in ${projectPath}`);
         const child = spawn(command, args, {
             cwd: projectPath,
             stdio: 'pipe'
         });
+        child.on('error', (error)=>{
+            log.debug(`Spawn error for ${command}: ${error.message}`);
+            resolve({
+                failedPackages: [
+                    ...packageNames
+                ],
+                success: false
+            });
+        });
         child.on('close', (code)=>{
             if (code === 0) {
-                log.debug(`Successfully uninstalled ${packageName}`);
-            } else {
-                log.debug(`Failed to uninstall ${packageName} (exit code: ${code}), continuing anyway`);
+                log.debug(`Successfully uninstalled ${packageNames.join(', ')}`);
+                resolve({
+                    failedPackages: [],
+                    success: true
+                });
+                return;
             }
-            // Always resolve, even on error
-            resolve();
+            log.debug(`Batched uninstall failed (exit code: ${code}), continuing anyway`);
+            resolve({
+                failedPackages: [
+                    ...packageNames
+                ],
+                success: false
+            });
         });
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@payloadcms/figma",
-  "version": "0.0.1-alpha.63",
+  "version": "0.0.1-alpha.64",
   "license": "SEE LICENSE IN LICENSE.md",
   "type": "module",
   "exports": {