slicejs-cli 3.2.0 → 3.4.0

package/commands/startServer/watchServer.js

@@ -1,79 +1,79 @@
-import chokidar from 'chokidar';
-import chalk from 'chalk';
-import Print from '../Print.js';
-
-/**
- * Configura el watcher para archivos del proyecto
- * @param {ChildProcess} serverProcess - Proceso del servidor
- * @returns {FSWatcher} - Watcher de chokidar
- */
-export default function setupWatcher(serverProcess, onRestart) {
-    Print.info('Watch mode enabled - monitoring file changes...');
-    Print.newLine();
-
-    const watcher = chokidar.watch(['src/**/*', 'api/**/*'], {
-        ignored: [
-            /(^|[\/\\])\../,  // archivos ocultos
-            '**/node_modules/**',
-            '**/dist/**',
-            '**/bundles/**',
-            '**/*.log'
-        ],
-        persistent: true,
-        ignoreInitial: true,
-        awaitWriteFinish: {
-            stabilityThreshold: 100,
-            pollInterval: 50
-        }
-    });
-
-    let reloadTimeout;
-
-    watcher
-        .on('change', (path) => {
-            // Debounce para evitar múltiples reloads
-            clearTimeout(reloadTimeout);
-            reloadTimeout = setTimeout(() => {
-                if(onRestart) {
-                    console.log(chalk.yellow('🔄 Changes detected, restarting server...'));
-                    onRestart(path);
-                } else {
-                    console.log(chalk.yellow('🔄 Changes detected, server will reload automatically... (No handler)'));
-                }
-            }, 500);
-        })
-        .on('add', (path) => {
-             // console.log(chalk.green(`➕ New file added: ${path}`));
-             clearTimeout(reloadTimeout);
-             reloadTimeout = setTimeout(() => {
-                 if (onRestart) onRestart(path);
-             }, 500);
-        })
-        .on('unlink', (path) => {
-             // console.log(chalk.red(`➖ File removed: ${path}`));
-             clearTimeout(reloadTimeout);
-             reloadTimeout = setTimeout(() => {
-                 if (onRestart) onRestart(path);
-             }, 500);
-        })
-        .on('error', (error) => {
-            Print.error(`Watcher error: ${error.message}`);
-        })
-        .on('ready', () => {
-            console.log(chalk.gray('👀 Watching for file changes...'));
-            Print.newLine();
-        });
-
-    return watcher;
-}
-
-/**
- * Detiene el watcher de forma segura
- * @param {FSWatcher} watcher - Watcher a detener
- */
-export function stopWatcher(watcher) {
-    if (watcher) {
-        watcher.close();
-        console.log(chalk.gray('Watch mode stopped'));
-    }
-}
+import chokidar from 'chokidar';
+import chalk from 'chalk';
+import Print from '../Print.js';
+
+/**
+ * Configura el watcher para archivos del proyecto
+ * @param {ChildProcess} serverProcess - Proceso del servidor
+ * @returns {FSWatcher} - Watcher de chokidar
+ */
+export default function setupWatcher(serverProcess, onRestart) {
+    Print.info('Watch mode enabled - monitoring file changes...');
+    Print.newLine();
+
+    const watcher = chokidar.watch(['src/**/*', 'api/**/*'], {
+        ignored: [
+            /(^|[\/\\])\../,  // archivos ocultos
+            '**/node_modules/**',
+            '**/dist/**',
+            '**/bundles/**',
+            '**/*.log'
+        ],
+        persistent: true,
+        ignoreInitial: true,
+        awaitWriteFinish: {
+            stabilityThreshold: 100,
+            pollInterval: 50
+        }
+    });
+
+    let reloadTimeout;
+
+    watcher
+        .on('change', (path) => {
+            // Debounce para evitar múltiples reloads
+            clearTimeout(reloadTimeout);
+            reloadTimeout = setTimeout(() => {
+                if(onRestart) {
+                    console.log(chalk.yellow('🔄 Changes detected, restarting server...'));
+                    onRestart(path);
+                } else {
+                    console.log(chalk.yellow('🔄 Changes detected, server will reload automatically... (No handler)'));
+                }
+            }, 500);
+        })
+        .on('add', (path) => {
+             // console.log(chalk.green(`➕ New file added: ${path}`));
+             clearTimeout(reloadTimeout);
+             reloadTimeout = setTimeout(() => {
+                 if (onRestart) onRestart(path);
+             }, 500);
+        })
+        .on('unlink', (path) => {
+             // console.log(chalk.red(`➖ File removed: ${path}`));
+             clearTimeout(reloadTimeout);
+             reloadTimeout = setTimeout(() => {
+                 if (onRestart) onRestart(path);
+             }, 500);
+        })
+        .on('error', (error) => {
+            Print.error(`Watcher error: ${error.message}`);
+        })
+        .on('ready', () => {
+            console.log(chalk.gray('👀 Watching for file changes...'));
+            Print.newLine();
+        });
+
+    return watcher;
+}
+
+/**
+ * Detiene el watcher de forma segura
+ * @param {FSWatcher} watcher - Watcher a detener
+ */
+export function stopWatcher(watcher) {
+    if (watcher) {
+        watcher.close();
+        console.log(chalk.gray('Watch mode stopped'));
+    }
+}

package/commands/types/types.js

@@ -427,13 +427,17 @@ const ensureEditorConfigForTypes = async ({ projectRoot, outputPath }) => {
       return `${dir}/**/*.d.ts`;
     })();
 
-    if (!(await fs.pathExists(jsconfigPath))) {
+    const writeDefaultJsconfig = async () => {
       const jsconfig = {
         compilerOptions: { ...DEFAULT_EDITOR_COMPILER_OPTIONS },
         include: [...DEFAULT_EDITOR_INCLUDE, declarationGlob],
         exclude: [...DEFAULT_EDITOR_EXCLUDE]
       };
       await fs.writeFile(jsconfigPath, `${JSON.stringify(jsconfig, null, 2)}\n`, 'utf8');
+    };
+
+    if (!(await fs.pathExists(jsconfigPath))) {
+      await writeDefaultJsconfig();
       return { mode: 'created_jsconfig', filePath: jsconfigPath, includeAdded: true };
     }
 
@@ -441,14 +445,19 @@ const ensureEditorConfigForTypes = async ({ projectRoot, outputPath }) => {
     try {
       jsconfigRaw = await fs.readFile(jsconfigPath, 'utf8');
     } catch {
-      return { mode: 'jsconfig_unreadable', filePath: jsconfigPath, includeAdded: false };
+      // Don't fail — fall back to writing the default options.
+      await writeDefaultJsconfig();
+      return { mode: 'reset_jsconfig', reason: 'unreadable', filePath: jsconfigPath, includeAdded: true };
     }
 
     let parsed;
     try {
       parsed = JSON.parse(jsconfigRaw);
     } catch {
-      return { mode: 'jsconfig_invalid_json', filePath: jsconfigPath, includeAdded: false };
+      // The jsconfig was edited into invalid JSON (a typo, comments, trailing commas...).
+      // Don't fail — just write the default options so editor IntelliSense keeps working.
+      await writeDefaultJsconfig();
+      return { mode: 'reset_jsconfig', reason: 'invalid_json', filePath: jsconfigPath, includeAdded: true };
     }
 
     const include = Array.isArray(parsed.include) ? parsed.include : [];
@@ -513,12 +522,10 @@ const runGenerateTypes = async ({ projectRoot, outputPath }) => {
     Print.info(`jsconfig.json already includes declaration glob. Editor IntelliSense should pick generated types.`);
   } else if (editorConfig.mode === 'tsconfig_exists') {
     Print.info(`tsconfig.json detected. Types declaration is generated; ensure include covers ${toPosixRelative(projectRoot, outputPath)}.`);
-  } else if (editorConfig.mode === 'jsconfig_invalid_json') {
-    Print.warning(`Could not update jsconfig.json because it contains invalid JSON.`);
-    Print.info(`Fix ${editorConfig.filePath} and run 'slice types generate' again.`);
-  } else if (editorConfig.mode === 'jsconfig_unreadable') {
-    Print.warning(`Could not read jsconfig.json to verify declaration include.`);
-    Print.info(`Check permissions for ${editorConfig.filePath} and run 'slice types generate' again.`);
+  } else if (editorConfig.mode === 'reset_jsconfig') {
+    const why = editorConfig.reason === 'invalid_json' ? 'contained invalid JSON' : 'could not be read';
+    Print.warning(`jsconfig.json ${why}; wrote the default options so editor IntelliSense keeps working.`);
+    Print.info(`Review ${editorConfig.filePath} if you had custom settings there.`);
   } else if (editorConfig.mode === 'editor_config_error') {
     Print.warning(`Unexpected editor config setup error: ${editorConfig.errorMessage}`);
   }

package/commands/utils/LocalCliDelegation.js

@@ -1,53 +1,53 @@
-import fs from 'node:fs';
-import path from 'node:path';
-
-function getParentDirectory(dir) {
-  const parent = path.dirname(dir);
-  return parent === dir ? null : parent;
-}
-
-export function isLocalDelegationDisabled(env = process.env) {
-  return env.SLICE_NO_LOCAL_DELEGATION === '1';
-}
-
-export function findNearestLocalCliEntry(startDirectory, resolveCandidate) {
-  if (!startDirectory || typeof resolveCandidate !== 'function') {
-    return null;
-  }
-
-  let current = path.resolve(startDirectory);
-  while (current) {
-    const candidate = resolveCandidate(current);
-    if (candidate) {
-      return candidate;
-    }
-    current = getParentDirectory(current);
-  }
-
-  return null;
-}
-
-export function resolveLocalCliCandidate(directory) {
-  const candidate = path.join(directory, 'node_modules', 'slicejs-cli', 'client.js');
-
-  try {
-    const stats = fs.statSync(candidate);
-    return stats.isFile() ? candidate : null;
-  } catch {
-    return null;
-  }
-}
-
-export function shouldDelegateToLocalCli(currentEntryPath, localEntryPath) {
-  if (!localEntryPath) {
-    return false;
-  }
-
-  try {
-    const currentReal = fs.realpathSync(currentEntryPath);
-    const localReal = fs.realpathSync(localEntryPath);
-    return currentReal !== localReal;
-  } catch {
-    return path.resolve(currentEntryPath) !== path.resolve(localEntryPath);
-  }
-}
+import fs from 'node:fs';
+import path from 'node:path';
+
+function getParentDirectory(dir) {
+  const parent = path.dirname(dir);
+  return parent === dir ? null : parent;
+}
+
+export function isLocalDelegationDisabled(env = process.env) {
+  return env.SLICE_NO_LOCAL_DELEGATION === '1';
+}
+
+export function findNearestLocalCliEntry(startDirectory, resolveCandidate) {
+  if (!startDirectory || typeof resolveCandidate !== 'function') {
+    return null;
+  }
+
+  let current = path.resolve(startDirectory);
+  while (current) {
+    const candidate = resolveCandidate(current);
+    if (candidate) {
+      return candidate;
+    }
+    current = getParentDirectory(current);
+  }
+
+  return null;
+}
+
+export function resolveLocalCliCandidate(directory) {
+  const candidate = path.join(directory, 'node_modules', 'slicejs-cli', 'client.js');
+
+  try {
+    const stats = fs.statSync(candidate);
+    return stats.isFile() ? candidate : null;
+  } catch {
+    return null;
+  }
+}
+
+export function shouldDelegateToLocalCli(currentEntryPath, localEntryPath) {
+  if (!localEntryPath) {
+    return false;
+  }
+
+  try {
+    const currentReal = fs.realpathSync(currentEntryPath);
+    const localReal = fs.realpathSync(localEntryPath);
+    return currentReal !== localReal;
+  } catch {
+    return path.resolve(currentEntryPath) !== path.resolve(localEntryPath);
+  }
+}

package/commands/utils/PathHelper.js

@@ -1,68 +1,68 @@
-import path from 'path'
-import fs from 'fs-extra'
-import { fileURLToPath } from 'url'
-
-const sanitize = (s) => (s || '').replace(/^[/\\]+/, '')
-const dirOf = (url) => path.dirname(fileURLToPath(url))
-
-function candidates(moduleUrl) {
-  const dir = dirOf(moduleUrl)
-  return [
-    path.join(dir, '../../'),
-    path.join(dir, '../../../../')
-  ]
-}
-
-function resolveProjectRoot(moduleUrl) {
-  const initCwd = process.env.INIT_CWD
-  if (initCwd && fs.pathExistsSync(initCwd)) {
-    return initCwd
-  }
-
-  const cwd = process.cwd()
-  if (cwd && fs.pathExistsSync(cwd)) {
-    return cwd
-  }
-
-  const dirs = candidates(moduleUrl)
-  for (const root of dirs) {
-    const hasSrc = fs.pathExistsSync(path.join(root, 'src'))
-    const hasApi = fs.pathExistsSync(path.join(root, 'api'))
-    if (hasSrc || hasApi) return root
-  }
-  return dirs[1]
-}
-
-function joinProject(moduleUrl, ...segments) {
-  const root = resolveProjectRoot(moduleUrl)
-  const clean = segments.map(sanitize)
-  return path.join(root, ...clean)
-}
-
-export function getProjectRoot(moduleUrl) {
-  return resolveProjectRoot(moduleUrl)
-}
-
-export function getPath(moduleUrl, folder, ...segments) {
-  return joinProject(moduleUrl, folder, ...segments)
-}
-
-export function getSrcPath(moduleUrl, ...segments) {
-  return joinProject(moduleUrl, 'src', ...segments)
-}
-
-export function getApiPath(moduleUrl, ...segments) {
-  return joinProject(moduleUrl, 'api', ...segments)
-}
-
-export function getDistPath(moduleUrl, ...segments) {
-  return joinProject(moduleUrl, 'dist', ...segments)
-}
-
-export function getConfigPath(moduleUrl) {
-  return joinProject(moduleUrl, 'src', 'sliceConfig.json')
-}
-
-export function getComponentsJsPath(moduleUrl) {
-  return joinProject(moduleUrl, 'src', 'Components', 'components.js')
-}
+import path from 'path'
+import fs from 'fs-extra'
+import { fileURLToPath } from 'url'
+
+const sanitize = (s) => (s || '').replace(/^[/\\]+/, '')
+const dirOf = (url) => path.dirname(fileURLToPath(url))
+
+function candidates(moduleUrl) {
+  const dir = dirOf(moduleUrl)
+  return [
+    path.join(dir, '../../'),
+    path.join(dir, '../../../../')
+  ]
+}
+
+function resolveProjectRoot(moduleUrl) {
+  const initCwd = process.env.INIT_CWD
+  if (initCwd && fs.pathExistsSync(initCwd)) {
+    return initCwd
+  }
+
+  const cwd = process.cwd()
+  if (cwd && fs.pathExistsSync(cwd)) {
+    return cwd
+  }
+
+  const dirs = candidates(moduleUrl)
+  for (const root of dirs) {
+    const hasSrc = fs.pathExistsSync(path.join(root, 'src'))
+    const hasApi = fs.pathExistsSync(path.join(root, 'api'))
+    if (hasSrc || hasApi) return root
+  }
+  return dirs[1]
+}
+
+function joinProject(moduleUrl, ...segments) {
+  const root = resolveProjectRoot(moduleUrl)
+  const clean = segments.map(sanitize)
+  return path.join(root, ...clean)
+}
+
+export function getProjectRoot(moduleUrl) {
+  return resolveProjectRoot(moduleUrl)
+}
+
+export function getPath(moduleUrl, folder, ...segments) {
+  return joinProject(moduleUrl, folder, ...segments)
+}
+
+export function getSrcPath(moduleUrl, ...segments) {
+  return joinProject(moduleUrl, 'src', ...segments)
+}
+
+export function getApiPath(moduleUrl, ...segments) {
+  return joinProject(moduleUrl, 'api', ...segments)
+}
+
+export function getDistPath(moduleUrl, ...segments) {
+  return joinProject(moduleUrl, 'dist', ...segments)
+}
+
+export function getConfigPath(moduleUrl) {
+  return joinProject(moduleUrl, 'src', 'sliceConfig.json')
+}
+
+export function getComponentsJsPath(moduleUrl) {
+  return joinProject(moduleUrl, 'src', 'Components', 'components.js')
+}