@unisphere/nx 3.9.0 → 3.11.0

package/dist/migrations/3-1-0/fix-workspaces-pattern.d.ts

@@ -19,8 +19,8 @@ import { Tree } from '@nx/devkit';
  * This automatically excludes unisphere/applications/documentation/** without
  * needing a negation pattern.
  *
- * Returns a callback to run npm install after changes are written to disk,
- * ensuring package-lock.json stays in sync with the new workspaces pattern.
+ * After updating workspaces, runs npm install to sync package-lock.json.
+ * Note: Nx migrations don't execute callbacks, so we flush changes and run npm install directly.
  */
-export default function update(tree: Tree): Promise<void | (() => void)>;
+export default function update(tree: Tree): Promise<string[]>;
 //# sourceMappingURL=fix-workspaces-pattern.d.ts.map

package/dist/migrations/3-1-0/fix-workspaces-pattern.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fix-workspaces-pattern.d.ts","sourceRoot":"","sources":["../../../src/migrations/3-1-0/fix-workspaces-pattern.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAA+B,MAAM,YAAY,CAAC;AAG/D;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,wBAA8B,MAAM,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,GAAG,CAAC,MAAM,IAAI,CAAC,CAAC,CA2E7E"}
+{"version":3,"file":"fix-workspaces-pattern.d.ts","sourceRoot":"","sources":["../../../src/migrations/3-1-0/fix-workspaces-pattern.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAA+B,MAAM,YAAY,CAAC;AAI/D;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,wBAA8B,MAAM,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CA+ElE"}

package/dist/migrations/3-1-0/fix-workspaces-pattern.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = update;
 const devkit_1 = require("@nx/devkit");
+const tree_1 = require("nx/src/generators/tree");
 const child_process_1 = require("child_process");
 /**
  * Migration: Fix npm workspaces pattern to use explicit includes
@@ -23,20 +24,20 @@ const child_process_1 = require("child_process");
  * This automatically excludes unisphere/applications/documentation/** without
  * needing a negation pattern.
  *
- * Returns a callback to run npm install after changes are written to disk,
- * ensuring package-lock.json stays in sync with the new workspaces pattern.
+ * After updating workspaces, runs npm install to sync package-lock.json.
+ * Note: Nx migrations don't execute callbacks, so we flush changes and run npm install directly.
  */
 async function update(tree) {
     devkit_1.logger.info('🔄 Fixing npm workspaces pattern to use explicit includes');
     const packageJsonPath = 'package.json';
     if (!tree.exists(packageJsonPath)) {
         devkit_1.logger.warn('No package.json found, skipping migration');
-        return;
+        return [];
     }
     const packageJson = (0, devkit_1.readJson)(tree, packageJsonPath);
     if (!packageJson.workspaces || !Array.isArray(packageJson.workspaces)) {
         devkit_1.logger.warn('No workspaces array found in package.json, skipping migration');
-        return;
+        return [];
     }
     const oldWorkspaces = packageJson.workspaces;
     // Check if using the old catch-all pattern
@@ -54,7 +55,7 @@ async function update(tree) {
         const hasAllExpectedPatterns = expectedPatterns.every(pattern => oldWorkspaces.includes(pattern));
         if (hasAllExpectedPatterns) {
             devkit_1.logger.info('Workspaces already using explicit patterns, skipping migration');
-            return;
+            return [];
         }
     }
     // Replace with explicit patterns
@@ -73,17 +74,19 @@ async function update(tree) {
     devkit_1.logger.info(`   ${JSON.stringify(newWorkspaces)}`);
     devkit_1.logger.info('');
     devkit_1.logger.info('This allows documentation apps (like Docusaurus) to have their own node_modules.');
-    // Return callback to run npm install after tree changes are written to disk
-    return () => {
-        devkit_1.logger.info('');
-        devkit_1.logger.info('Syncing package-lock.json after workspaces update...');
-        try {
-            (0, child_process_1.execSync)('npm install', { stdio: 'inherit' });
-            devkit_1.logger.info('✅ package-lock.json synced successfully');
-        }
-        catch (error) {
-            devkit_1.logger.error(`Failed to sync package-lock.json: ${error}`);
-            throw error;
-        }
-    };
+    // Flush tree changes to disk before running npm install
+    // Note: Nx doesn't execute callbacks from migrations, so we must flush and run directly
+    const changes = tree.listChanges();
+    (0, tree_1.flushChanges)(tree.root, changes);
+    devkit_1.logger.info('');
+    devkit_1.logger.info('Syncing package-lock.json after workspaces update...');
+    try {
+        (0, child_process_1.execSync)('npm install', { stdio: 'inherit', cwd: tree.root });
+        devkit_1.logger.info('✅ package-lock.json synced successfully');
+    }
+    catch (error) {
+        devkit_1.logger.error(`Failed to sync package-lock.json: ${error}`);
+        devkit_1.logger.warn('⚠️  Please run "npm install" manually to sync package-lock.json');
+    }
+    return [];
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unisphere/nx",
-  "version": "3.9.0",
+  "version": "3.11.0",
   "private": false,
   "main": "./dist/index.js",
   "module": "./dist/index.js",