@unisphere/nx 3.3.0 → 3.4.1

package/dist/migrations/3-0-0/add-env-to-application-gitignore.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"add-env-to-application-gitignore.d.ts","sourceRoot":"","sources":["../../../src/migrations/3-0-0/add-env-to-application-gitignore.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAoB,MAAM,YAAY,CAAC;AAsDpD,wBAA8B,MAAM,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAkF9D"}
+{"version":3,"file":"add-env-to-application-gitignore.d.ts","sourceRoot":"","sources":["../../../src/migrations/3-0-0/add-env-to-application-gitignore.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAU,MAAM,YAAY,CAAC;AAE1C,wBAA8B,MAAM,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CA6B9D"}

package/dist/migrations/3-0-0/add-env-to-application-gitignore.js

@@ -2,116 +2,30 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = update;
 const devkit_1 = require("@nx/devkit");
-const path_1 = require("path");
-const ENV_GITIGNORE_ENTRIES = `.env`;
-/**
- * Add .env entries to .gitignore for a given element
- * Returns: 'updated' | 'created' | 'skipped'
- */
-function addEnvToGitignore(tree, elementName, elementPath) {
-    const gitignorePath = (0, path_1.join)(elementPath, '.gitignore');
-    if (tree.exists(gitignorePath)) {
-        // Read existing .gitignore
-        const existingContent = tree.read(gitignorePath, 'utf-8') || '';
-        // Check if .env is already in gitignore
-        if (existingContent.includes('.env')) {
-            devkit_1.logger.info(`  ✅ ${elementName}: .gitignore already contains .env`);
-            return 'skipped';
-        }
-        // Append .env entries to existing .gitignore
-        const newContent = existingContent.trimEnd() + '\n\n# Environment files\n' + ENV_GITIGNORE_ENTRIES + '\n';
-        tree.write(gitignorePath, newContent);
-        devkit_1.logger.info(`  ✅ ${elementName}: Added .env to existing .gitignore`);
-        return 'updated';
-    }
-    else {
-        // Create new .gitignore with .env entries
-        const newContent = '# Environment files\n' + ENV_GITIGNORE_ENTRIES + '\n';
-        tree.write(gitignorePath, newContent);
-        devkit_1.logger.info(`  ✅ ${elementName}: Created .gitignore with .env`);
-        return 'created';
-    }
-}
 async function update(tree) {
-    devkit_1.logger.info('🔄 Adding .env to .gitignore files for all elements');
+    devkit_1.logger.info('🔄 Updating .gitignore to add migration-related entries...');
+    const gitignorePath = '.gitignore';
+    if (!tree.exists(gitignorePath)) {
+        devkit_1.logger.warn('⚠️ .gitignore file not found, creating one');
+        tree.write(gitignorePath, '');
+    }
     try {
-        // Read .unisphere config
-        if (!tree.exists('.unisphere')) {
-            devkit_1.logger.warn('⚠️ No .unisphere file found, skipping');
-            return;
-        }
-        const unisphereConfig = (0, devkit_1.readJson)(tree, '.unisphere');
-        const elements = unisphereConfig.elements;
-        if (!elements) {
-            devkit_1.logger.info('ℹ️ No elements found, skipping');
-            return;
-        }
-        let totalUpdated = 0;
-        let totalCreated = 0;
-        // Process applications
-        const applications = elements.applications || {};
-        if (Object.keys(applications).length > 0) {
-            devkit_1.logger.info('\n📦 Processing applications...');
-            for (const [name, config] of Object.entries(applications)) {
-                const result = addEnvToGitignore(tree, name, config.sourceRoot);
-                if (result === 'updated')
-                    totalUpdated++;
-                if (result === 'created')
-                    totalCreated++;
-            }
-        }
-        // Process packages
-        const packages = elements.packages || {};
-        if (Object.keys(packages).length > 0) {
-            devkit_1.logger.info('\n📦 Processing packages...');
-            for (const [name, config] of Object.entries(packages)) {
-                const result = addEnvToGitignore(tree, name, config.sourceRoot);
-                if (result === 'updated')
-                    totalUpdated++;
-                if (result === 'created')
-                    totalCreated++;
-            }
-        }
-        // Process runtimes
-        const runtimes = elements.runtimes || {};
-        if (Object.keys(runtimes).length > 0) {
-            devkit_1.logger.info('\n📦 Processing runtimes...');
-            for (const [name, config] of Object.entries(runtimes)) {
-                const result = addEnvToGitignore(tree, name, config.sourceRoot);
-                if (result === 'updated')
-                    totalUpdated++;
-                if (result === 'created')
-                    totalCreated++;
-            }
-        }
-        // Process workspace elements
-        const workspace = elements.workspace || {};
-        if (Object.keys(workspace).length > 0) {
-            devkit_1.logger.info('\n📦 Processing workspace elements...');
-            for (const [name, config] of Object.entries(workspace)) {
-                const result = addEnvToGitignore(tree, name, config.sourceRoot);
-                if (result === 'updated')
-                    totalUpdated++;
-                if (result === 'created')
-                    totalCreated++;
-            }
-        }
-        // Process loader elements
-        const loader = elements.loader || {};
-        if (Object.keys(loader).length > 0) {
-            devkit_1.logger.info('\n📦 Processing loader elements...');
-            for (const [name, config] of Object.entries(loader)) {
-                const result = addEnvToGitignore(tree, name, config.sourceRoot);
-                if (result === 'updated')
-                    totalUpdated++;
-                if (result === 'created')
-                    totalCreated++;
-            }
+        const currentContent = tree.read(gitignorePath, 'utf-8') || '';
+        let updatedContent = currentContent;
+        // Check if migrations.json is already in .gitignore
+        const migrationsJsonRegex = /^\.env$/gm;
+        if (!migrationsJsonRegex.test(updatedContent)) {
+            updatedContent += '.env\n';
+            tree.write(gitignorePath, updatedContent);
+            devkit_1.logger.info(`✅ Successfully updated .gitignore (added .env entry)`);
+        }
+        else {
+            devkit_1.logger.info('ℹ️ .env entry already exists in .gitignore');
         }
-        devkit_1.logger.info(`\n✅ Done: ${totalUpdated} updated, ${totalCreated} created`);
     }
     catch (error) {
-        devkit_1.logger.error(`❌ Failed to update .gitignore files: ${error?.message || 'Unknown error'}`);
-        throw error;
+        const errorMessage = error instanceof Error ? error.message : 'Unknown error';
+        devkit_1.logger.error(`❌ Failed to update .gitignore: ${errorMessage}`);
+        throw new Error(`Failed to update .gitignore: ${errorMessage}`);
     }
 }

package/dist/migrations/3-0-0/reorganize-applications-by-distribution-channel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reorganize-applications-by-distribution-channel.d.ts","sourceRoot":"","sources":["../../../src/migrations/3-0-0/reorganize-applications-by-distribution-channel.ts"],"names":[],"mappings":"AAAA;;;;;;;;EAQE;AAEF,OAAO,EACL,IAAI,EAOL,MAAM,YAAY,CAAC;AA8oBpB;;GAEG;AACH,wBAA8B,MAAM,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAyE9D"}
+{"version":3,"file":"reorganize-applications-by-distribution-channel.d.ts","sourceRoot":"","sources":["../../../src/migrations/3-0-0/reorganize-applications-by-distribution-channel.ts"],"names":[],"mappings":"AAAA;;;;;;;;EAQE;AAEF,OAAO,EACL,IAAI,EAML,MAAM,YAAY,CAAC;AAsmBpB;;GAEG;AACH,wBAA8B,MAAM,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAyE9D"}

package/dist/migrations/3-0-0/reorganize-applications-by-distribution-channel.js

@@ -11,49 +11,10 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = update;
 const devkit_1 = require("@nx/devkit");
+const generators_1 = require("@nx/workspace/generators");
 const child_process_1 = require("child_process");
-/**
- * Manually move all files from source to destination directory.
- * This avoids using moveGenerator which relies on the Nx project graph cache.
- * The project graph is read from disk, but during migrations the Tree (virtual FS)
- * may have changes that aren't flushed to disk yet, causing stale data issues.
- * Using manual move with Tree APIs ensures we always work with the correct state.
- */
-function moveProjectFiles(tree, sourcePath, destPath) {
-    const filesToMove = [];
-    // Collect all files to move
-    (0, devkit_1.visitNotIgnoredFiles)(tree, sourcePath, (filePath) => {
-        const relativePath = filePath.substring(sourcePath.length);
-        const newPath = `${destPath}${relativePath}`;
-        filesToMove.push({ source: filePath, dest: newPath });
-    });
-    // Move files (create at dest, delete at source)
-    for (const { source, dest } of filesToMove) {
-        const content = tree.read(source);
-        if (content !== null) {
-            tree.write(dest, content);
-            tree.delete(source);
-        }
-    }
-    devkit_1.logger.info(`  Moved ${filesToMove.length} files from ${sourcePath} to ${destPath}`);
-}
-/**
- * Update project.json after moving to new location
- */
-function updateProjectJsonAfterMove(tree, newPath, newProjectName) {
-    const projectJsonPath = `${newPath}/project.json`;
-    if (!tree.exists(projectJsonPath)) {
-        return;
-    }
-    const projectJson = (0, devkit_1.readJson)(tree, projectJsonPath);
-    // Update name if different
-    if (projectJson.name !== newProjectName) {
-        projectJson.name = newProjectName;
-    }
-    // Update sourceRoot to match new location
-    projectJson.sourceRoot = `${newPath}/src`;
-    (0, devkit_1.writeJson)(tree, projectJsonPath, projectJson);
-}
+const fs_1 = require("fs");
+const path_1 = require("path");
 /**
  * Reset Nx cache to ensure fresh project graph
  * This is important before moving projects, as Nx caches project locations
@@ -454,22 +415,33 @@ async function migrateApplication(tree, target) {
         devkit_1.logger.warn(`  Source project.json not found at ${sourceProjectJson}, skipping`);
         return;
     }
-    // Step 1: Manually move files using Tree APIs
-    // We avoid moveGenerator because it relies on the Nx project graph cache,
-    // which reads from disk. During migrations, the packages migration may have
-    // already moved packages in the Tree but those changes aren't flushed to disk
-    // yet, causing moveGenerator to use stale project locations.
-    // Since applications don't need import path updates (they're entry points),
-    // manual move is safe and reliable.
+    // Step 0: Clean node_modules from destination on disk if present.
+    // npm install may create node_modules/ at the target path (e.g. for workspace
+    // dependency resolution). tree.delete() only records a virtual deletion and
+    // doesn't remove on-disk children, so moveGenerator's checkDestination
+    // (which uses tree.children() -> readdirSync) would see node_modules and
+    // throw "Path is not empty". We must remove it from the real filesystem.
+    const destNodeModules = (0, path_1.join)(newPath, 'node_modules');
+    if ((0, fs_1.existsSync)(destNodeModules)) {
+        devkit_1.logger.info(`  Removing leftover node_modules at destination: ${destNodeModules}`);
+        (0, fs_1.rmSync)(destNodeModules, { recursive: true, force: true });
+    }
+    // Step 1: Use Nx moveGenerator for the heavy lifting
     try {
-        moveProjectFiles(tree, app.currentPath, newPath);
-        updateProjectJsonAfterMove(tree, newPath, newNxProjectName);
-        devkit_1.logger.info(`  Project moved successfully`);
+        await (0, generators_1.moveGenerator)(tree, {
+            projectName: app.currentNxProjectName,
+            destination: newPath,
+            newProjectName: newNxProjectName,
+            updateImportPath: false, // Applications are entry points, not libraries
+            skipFormat: true,
+        });
+        devkit_1.logger.info(`  Nx moved project successfully`);
         // Reset Nx cache after move to ensure fresh project graph for next operations
+        // This is important because moveGenerator uses the project graph, and it can become stale
         resetNxCache();
     }
     catch (error) {
-        devkit_1.logger.error(`  Failed to move project: ${error}`);
+        devkit_1.logger.error(`  Failed to move project with Nx: ${error}`);
         devkit_1.logger.error(`  Source path: ${app.currentPath}`);
         devkit_1.logger.error(`  Destination path: ${newPath}`);
         devkit_1.logger.error(`  Project name: ${app.currentNxProjectName}`);

package/dist/migrations/3-4-0/update-gitignore.d.ts

@@ -0,0 +1,9 @@
+/**
+ * Migration: Update .gitignore to exclude Claude Code local settings
+ *
+ * Adds .claude/settings.local.json to .gitignore to prevent local Claude Code
+ * settings from being committed to the repository.
+ */
+import { Tree } from '@nx/devkit';
+export default function update(tree: Tree): Promise<void>;
+//# sourceMappingURL=update-gitignore.d.ts.map

package/dist/migrations/3-4-0/update-gitignore.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"update-gitignore.d.ts","sourceRoot":"","sources":["../../../src/migrations/3-4-0/update-gitignore.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,IAAI,EAAU,MAAM,YAAY,CAAC;AAE1C,wBAA8B,MAAM,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CA0C9D"}

package/dist/migrations/3-4-0/update-gitignore.js

@@ -0,0 +1,45 @@
+"use strict";
+/**
+ * Migration: Update .gitignore to exclude Claude Code local settings
+ *
+ * Adds .claude/settings.local.json to .gitignore to prevent local Claude Code
+ * settings from being committed to the repository.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = update;
+const devkit_1 = require("@nx/devkit");
+async function update(tree) {
+    devkit_1.logger.info('🔄 Updating .gitignore to exclude Claude Code local settings...');
+    const gitignorePath = '.gitignore';
+    if (!tree.exists(gitignorePath)) {
+        devkit_1.logger.warn('⚠️ .gitignore file not found, creating one');
+        tree.write(gitignorePath, '');
+    }
+    try {
+        const currentContent = tree.read(gitignorePath, 'utf-8') || '';
+        let updatedContent = currentContent;
+        // Check if .claude/settings.local.json is already in .gitignore
+        const claudeSettingsRegex = /^\.claude\/settings\.local\.json\s*$/gm;
+        if (!claudeSettingsRegex.test(updatedContent)) {
+            // Add Claude Code local settings section and entry
+            if (!updatedContent.includes('# Claude Code local settings')) {
+                updatedContent =
+                    updatedContent.trim() + '\n\n# Claude Code local settings\n';
+            }
+            if (!updatedContent.includes('.claude/settings.local.json')) {
+                updatedContent += '.claude/settings.local.json\n';
+                devkit_1.logger.info('📝 Added .claude/settings.local.json to .gitignore');
+            }
+            tree.write(gitignorePath, updatedContent);
+            devkit_1.logger.info('✅ Successfully updated .gitignore');
+        }
+        else {
+            devkit_1.logger.info('ℹ️ .claude/settings.local.json already exists in .gitignore');
+        }
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : 'Unknown error';
+        devkit_1.logger.error(`❌ Failed to update .gitignore: ${errorMessage}`);
+        throw new Error(`Failed to update .gitignore: ${errorMessage}`);
+    }
+}

package/migrations.json

@@ -282,6 +282,14 @@
       "cli": {
         "postUpdateMessage": "✅ npm workspaces pattern updated to use explicit includes. Documentation apps can now have their own node_modules."
       }
+    },
+    "3-4-0-update-gitignore": {
+      "version": "3.4.0",
+      "description": "Updates .gitignore to exclude Claude Code local settings",
+      "factory": "./dist/migrations/3-4-0/update-gitignore.js",
+      "cli": {
+        "postUpdateMessage": "✅ .gitignore updated to exclude .claude/settings.local.json"
+      }
     }
   },
   "packageJsonUpdates": {
@@ -439,4 +447,4 @@
       }
     }
   }
-}
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unisphere/nx",
-  "version": "3.3.0",
+  "version": "3.4.1",
   "private": false,
   "main": "./dist/index.js",
   "module": "./dist/index.js",
@@ -91,4 +91,4 @@
   "nx-migrations": {
     "migrations": "./migrations.json"
   }
-}
+}