slicejs-cli 3.6.0 → 3.6.1

package/README.md

@@ -70,6 +70,38 @@ so each project keeps its pinned CLI version.
 
 ## Main commands
 
+Inside initialized projects, prefer package scripts (`pnpm run ...`, `npm run ...`)
+over direct binary calls.
+
+Common script workflow:
+
+```bash
+pnpm run dev
+pnpm run build
+pnpm run start
+pnpm run browse
+pnpm run get -- Button
+pnpm run sync
+```
+
+Alternative with local devDependency resolution:
+
+```bash
+pnpm exec slice dev
+```
+
+If `slicejs-cli` is installed globally, `slice` can be executed directly from PATH.
+
+For pnpm v10+, if build scripts are restricted, configure `allowBuilds` in
+`pnpm-workspace.yaml`:
+
+```yaml
+allowBuilds:
+  slicejs-cli: true
+```
+
+`slice init --pm pnpm` now writes this automatically.
+
 | Command | Description |
 |---------|-------------|
 | `slice init` | Initialize a Slice.js project |
@@ -89,17 +121,21 @@ When you install `slicejs-cli`, the `postinstall` script automatically configure
 ```json
 {
   "scripts": {
-    "slice:dev": "slice dev",
-    "slice:start": "slice start",
-    "slice:create": "slice component create",
-    "slice:list": "slice component list",
-    "slice:delete": "slice component delete",
-    "slice:init": "slice init",
-    "slice:get": "slice get",
-    "slice:browse": "slice browse",
-    "slice:sync": "slice sync",
-    "slice:version": "slice version",
-    "slice:update": "slice update"
+    "slice:init": "node ./node_modules/slicejs-cli/client.js init",
+    "slice:dev": "node ./node_modules/slicejs-cli/client.js dev",
+    "slice:build": "node ./node_modules/slicejs-cli/client.js build",
+    "slice:start": "node ./node_modules/slicejs-cli/client.js start",
+    "slice:create": "node ./node_modules/slicejs-cli/client.js component create",
+    "slice:list": "node ./node_modules/slicejs-cli/client.js component list",
+    "slice:delete": "node ./node_modules/slicejs-cli/client.js component delete",
+    "slice:get": "node ./node_modules/slicejs-cli/client.js get",
+    "slice:browse": "node ./node_modules/slicejs-cli/client.js browse",
+    "slice:sync": "node ./node_modules/slicejs-cli/client.js sync",
+    "slice:doctor": "node ./node_modules/slicejs-cli/client.js doctor",
+    "slice:version": "node ./node_modules/slicejs-cli/client.js version",
+    "slice:help": "node ./node_modules/slicejs-cli/client.js --help",
+    "slice:update": "node ./node_modules/slicejs-cli/client.js update",
+    "slice:types": "node ./node_modules/slicejs-cli/client.js types generate"
   }
 }
 ```

package/client.js

@@ -31,6 +31,8 @@ import {
   detectPackageManager,
   getAvailablePackageManagers,
   isPackageManagerAvailable,
+  resolvePackageManager,
+  runScriptCommand,
   SUPPORTED_PACKAGE_MANAGERS
 } from './commands/utils/PackageManager.js';
 import { SLICE_SCRIPTS } from './commands/utils/sliceScripts.js';
@@ -103,7 +105,7 @@ sliceClient
   .command("init")
   .description("Initialize a new Slice.js project")
   .option("-y, --yes [name]", "Skip prompts and initialize with project name")
-  .option("--pm <packageManager>", "Package manager to use (npm, pnpm or yarn). Auto-detected when omitted")
+  .option("--pm <packageManager>", "Package manager to use (pnpm or npm). Auto-detected when omitted")
   .action(async (options) => {
     let projectName = 'my-slice-app';
     if (options.yes) {
@@ -150,7 +152,7 @@ sliceClient
       } else if (!options.yes) {
         const available = getAvailablePackageManagers();
         if (available.length === 0) {
-          Print.error('No package manager found (npm, pnpm or yarn). Install one and retry.');
+          Print.error('No package manager found (pnpm or npm). Install one and retry.');
           return;
         }
         const { pm } = await inquirer.prompt([
@@ -165,9 +167,9 @@ sliceClient
         packageManager = pm;
       } else {
         const available = getAvailablePackageManagers();
-        packageManager = available.includes('npm') ? 'npm' : available[0];
+        packageManager = available.includes('pnpm') ? 'pnpm' : available[0];
         if (!packageManager) {
-          Print.error('No package manager found (npm, pnpm or yarn). Install one and retry.');
+          Print.error('No package manager found (pnpm or npm). Install one and retry.');
           return;
         }
       }
@@ -638,6 +640,7 @@ sliceClient
 
     const projectRoot = getProjectRoot(import.meta.url);
     const pkgPath = path.join(projectRoot, 'package.json');
+    const packageManager = resolvePackageManager(projectRoot).name;
 
     // Shared with post.js and slice init — see commands/utils/sliceScripts.js
     const sliceScripts = SLICE_SCRIPTS;
@@ -665,12 +668,12 @@ sliceClient
       }
 
       fs.writeFileSync(pkgPath, JSON.stringify(pkg, null, 2), 'utf-8');
-      console.log(`✅  slicejs-cli installed successfully. Added ${addedCount} npm scripts to package.json.`);
-      console.log('   Run: npm run slice:dev');
+      console.log(`✅  slicejs-cli installed successfully. Added ${addedCount} package scripts to package.json.`);
+      console.log(`   Run: ${runScriptCommand(packageManager, 'slice:dev')}`);
     } catch (err) {
       console.log('✅  slicejs-cli installed successfully.');
       console.log('   Could not auto-configure scripts:', err.message);
-      console.log('   Run: npx slice dev');
+      console.log(`   Configure scripts manually and run: ${runScriptCommand(packageManager, 'slice:dev')}`);
     }
   });
 

package/commands/buildProduction/buildProduction.js

@@ -6,7 +6,8 @@ import { minify as terserMinify } from 'terser';
 import { minify } from 'html-minifier-terser';
 import CleanCSS from 'clean-css';
 import Print from '../Print.js';
-import { getSrcPath, getDistPath, getConfigPath } from '../utils/PathHelper.js';
+import { getSrcPath, getDistPath, getConfigPath, getProjectRoot } from '../utils/PathHelper.js';
+import { resolvePackageManager, runScriptCommand } from '../utils/PackageManager.js';
 
 /**
  * Loads configuration from sliceConfig.json
@@ -441,6 +442,7 @@ async function analyzeBuild() {
  */
 export default async function buildProduction(options = {}) {
   const startTime = Date.now();
+  const packageManager = resolvePackageManager(getProjectRoot(import.meta.url)).name;
   
   try {
     Print.title('🔨 Building Slice.js project for production...');
@@ -483,7 +485,7 @@ export default async function buildProduction(options = {}) {
     Print.info('Your optimized project is ready in the /dist directory');
     Print.newLine();
     Print.info('Next steps:');
-    console.log('  • Use "npm run slice:start" to test the production build');
+    console.log(`  • Use "${runScriptCommand(packageManager, 'start')}" to test the production build`);
     console.log('  • All Slice.js components and architecture preserved');
     
     return true;
@@ -499,6 +501,7 @@ export default async function buildProduction(options = {}) {
  */
 export async function serveProductionBuild(port) {
   try {
+    const packageManager = resolvePackageManager(getProjectRoot(import.meta.url)).name;
     const config = loadConfig();
     const defaultPort = config?.server?.port || 3001;
     const finalPort = port || defaultPort;
@@ -533,7 +536,7 @@ export async function serveProductionBuild(port) {
       Print.success(`Production preview server running at http://localhost:${finalPort}`);
       Print.info('Press Ctrl+C to stop the server');
       Print.info('This server previews your production build from /dist');
-      Print.warning('This is a preview server - use "npm run slice:start" for the full production server');
+      Print.warning(`This is a preview server - use "${runScriptCommand(packageManager, 'start')}" for the full production server`);
     });
     
   } catch (error) {

package/commands/doctor/doctor.js

@@ -154,8 +154,7 @@ export async function checkPackageManagerSetup(projectRoot = getProjectRoot(impo
 
     const LOCKFILE_PM = {
         'package-lock.json': 'npm',
-        'pnpm-lock.yaml': 'pnpm',
-        'yarn.lock': 'yarn'
+        'pnpm-lock.yaml': 'pnpm'
     };
     const lockfiles = [];
     for (const lockfile of Object.keys(LOCKFILE_PM)) {

package/commands/init/init.js

@@ -32,6 +32,51 @@ async function fetchLatestVersion(packageName) {
     }
 }
 
+async function ensurePnpmAllowBuilds(projectRoot) {
+    const workspacePath = path.join(projectRoot, 'pnpm-workspace.yaml');
+    const allowBuildLine = '  slicejs-cli: true';
+
+    if (!(await fs.pathExists(workspacePath))) {
+        await fs.writeFile(workspacePath, `allowBuilds:\n${allowBuildLine}\n`, 'utf8');
+        return;
+    }
+
+    const raw = await fs.readFile(workspacePath, 'utf8');
+    const lines = raw.split(/\r?\n/);
+    const allowIdx = lines.findIndex((line) => /^allowBuilds:\s*$/.test(line));
+
+    if (allowIdx === -1) {
+        const suffix = raw.endsWith('\n') ? '' : '\n';
+        await fs.writeFile(workspacePath, `${raw}${suffix}allowBuilds:\n${allowBuildLine}\n`, 'utf8');
+        return;
+    }
+
+    let blockEnd = lines.length;
+    for (let i = allowIdx + 1; i < lines.length; i++) {
+        const line = lines[i];
+        if (!line.trim()) continue;
+        if (!/^\s/.test(line)) {
+            blockEnd = i;
+            break;
+        }
+    }
+
+    let found = false;
+    for (let i = allowIdx + 1; i < blockEnd; i++) {
+        if (/^\s+slicejs-cli\s*:/.test(lines[i])) {
+            lines[i] = allowBuildLine;
+            found = true;
+            break;
+        }
+    }
+
+    if (!found) {
+        lines.splice(blockEnd, 0, allowBuildLine);
+    }
+
+    await fs.writeFile(workspacePath, `${lines.join('\n').replace(/\n*$/, '\n')}`, 'utf8');
+}
+
 // Create the project manifest BEFORE any install runs. Without a package.json in
 // the project folder, npm/pnpm walk up the directory tree looking for the nearest
 // manifest and anchor node_modules (and the dependency entry) OUTSIDE the project.
@@ -93,6 +138,10 @@ export default async function initializeProject(options = {}) {
         const packageManager = options.packageManager
             || resolvePackageManager(projectRoot).name;
 
+        if (packageManager === 'pnpm') {
+            await ensurePnpmAllowBuilds(projectRoot);
+        }
+
         // 0. CREATE PROJECT MANIFEST FIRST — must exist before any install so the
         // package manager anchors node_modules inside the project folder.
         await ensureProjectManifest(projectRoot, packageManager);
@@ -256,20 +305,20 @@ export default async function initializeProject(options = {}) {
                     componentsSpinner.succeed(`All ${successful} Visual components installed successfully`);
                 } else if (successful > 0) {
                 componentsSpinner.warn(`${successful} components installed, ${failed} failed`);
-                Print.info('You can install failed components later using "slice get <component-name>"');
+                Print.info(`You can install failed components later using "${packageManager} run get -- <component-name>"`);
             } else {
                 componentsSpinner.fail('Failed to install Visual components');
             }
         } else {
             componentsSpinner.warn('No Visual components found in registry');
-            Print.info('You can add components later using "slice get <component-name>"');
+            Print.info(`You can add components later using "${packageManager} run get -- <component-name>"`);
         }
 
         } catch (error) {
             componentsSpinner.fail('Could not download Visual components from official repository');
             Print.error(`Repository error: ${error.message}`);
             Print.info('Project initialized without Visual components');
-            Print.info('You can add them later using "slice get <component-name>"');
+            Print.info(`You can add them later using "${packageManager} run get -- <component-name>"`);
         }
 
         // 3b. DOWNLOAD STARTER SERVICE COMPONENTS FROM OFFICIAL REPOSITORY
@@ -295,7 +344,7 @@ export default async function initializeProject(options = {}) {
                     serviceSpinner.succeed(`All ${successful} Service components installed successfully`);
                 } else if (successful > 0) {
                     serviceSpinner.warn(`${successful} Service components installed, ${failed} failed`);
-                    Print.info('You can install failed components later using "slice get <component-name>"');
+                    Print.info(`You can install failed components later using "${packageManager} run get -- <component-name>"`);
                 } else {
                     serviceSpinner.fail('Failed to install Service components');
                 }
@@ -305,7 +354,7 @@ export default async function initializeProject(options = {}) {
         } catch (error) {
             serviceSpinner.fail('Could not download Service components from official repository');
             Print.error(`Repository error: ${error.message}`);
-            Print.info('You can add them later using "slice get <component-name>"');
+            Print.info(`You can add them later using "${packageManager} run get -- <component-name>"`);
         }
 
         // 4. CONFIGURE SCRIPTS IN PROJECT package.json
@@ -330,25 +379,25 @@ export default async function initializeProject(options = {}) {
             pkg.scripts = pkg.scripts || {};
             pkg.dependencies = pkg.dependencies || {};
 
-            // Comandos principales
-            pkg.scripts['dev'] = 'slice dev';
-            pkg.scripts['build'] = 'slice build';
-            pkg.scripts['start'] = 'slice start';
+            // Main scripts (local CLI path, no global launcher dependency)
+            pkg.scripts['dev'] = SLICE_SCRIPTS['slice:dev'];
+            pkg.scripts['build'] = SLICE_SCRIPTS['slice:build'];
+            pkg.scripts['start'] = SLICE_SCRIPTS['slice:start'];
 
             // Component management
-            pkg.scripts['component:create'] = 'slice component create';
-            pkg.scripts['component:list'] = 'slice component list';
-            pkg.scripts['component:delete'] = 'slice component delete';
+            pkg.scripts['component:create'] = SLICE_SCRIPTS['slice:create'];
+            pkg.scripts['component:list'] = SLICE_SCRIPTS['slice:list'];
+            pkg.scripts['component:delete'] = SLICE_SCRIPTS['slice:delete'];
 
-            // Atajos de repositorio
-            pkg.scripts['get'] = 'slice get';
-            pkg.scripts['browse'] = 'slice browse';
-            pkg.scripts['sync'] = 'slice sync';
+            // Registry shortcuts
+            pkg.scripts['get'] = SLICE_SCRIPTS['slice:get'];
+            pkg.scripts['browse'] = SLICE_SCRIPTS['slice:browse'];
+            pkg.scripts['sync'] = SLICE_SCRIPTS['slice:sync'];
 
             // slice:* namespaced set — shared with post.js and `slice postinstall`
             // (commands/utils/sliceScripts.js) so the three never drift apart.
             Object.assign(pkg.scripts, SLICE_SCRIPTS);
-            pkg.scripts['run'] = 'slice dev';
+            pkg.scripts['run'] = SLICE_SCRIPTS['slice:dev'];
 
             // Module configuration
             pkg.type = 'module';
@@ -378,9 +427,9 @@ export default async function initializeProject(options = {}) {
         Print.title('Next steps:');
         console.log(`  cd ${projectName}`);
         console.log(`  ${packageManager} run dev     - Start development server`);
-        console.log('  slice browse          - View available components');
-        console.log('  slice get Button      - Install specific components');
-        console.log('  slice sync            - Update all components to latest versions');
+        console.log(`  ${packageManager} run browse  - View available components`);
+        console.log(`  ${packageManager} run get -- Button - Install specific components`);
+        console.log(`  ${packageManager} run sync    - Update all components to latest versions`);
 
     } catch (error) {
         Print.error('Unexpected error initializing project:', error.message);

package/commands/utils/PackageManager.js

@@ -1,29 +1,28 @@
 // commands/utils/PackageManager.js
 //
-// Package manager detection and command building (npm / pnpm / yarn).
+// Package manager detection and command building (pnpm / npm).
 // Resolution priority for an existing project:
 //   1. "packageManager" field in the project package.json (corepack convention)
 //   2. Lockfile present at the project root
 //   3. npm_config_user_agent (set when the CLI runs via `npx` / `pnpm dlx` / a PM script)
 //   4. The only PM binary available on PATH (if exactly one)
 // When everything is ambiguous, detectPackageManager() returns null so callers
-// can prompt the user (interactive init) or fall back to npm (non-interactive).
+// can prompt the user (interactive init) or fall back to pnpm (non-interactive).
 
 import fs from 'fs-extra'
 import path from 'path'
 import { spawnSync } from 'node:child_process'
 
-export const SUPPORTED_PACKAGE_MANAGERS = ['npm', 'pnpm', 'yarn']
+export const SUPPORTED_PACKAGE_MANAGERS = ['pnpm', 'npm']
 
 const LOCKFILES = {
   'pnpm-lock.yaml': 'pnpm',
-  'package-lock.json': 'npm',
-  'yarn.lock': 'yarn'
+  'package-lock.json': 'npm'
 }
 
 export function parseUserAgent(userAgent = process.env.npm_config_user_agent) {
   if (!userAgent) return null
-  const match = userAgent.match(/^(npm|pnpm|yarn)\/(\S+)/)
+  const match = userAgent.match(/^(npm|pnpm)\/(\S+)/)
   if (!match) return null
   return { name: match[1], version: match[2], source: 'user-agent' }
 }
@@ -34,7 +33,7 @@ export function fromPackageManagerField(projectRoot) {
     if (!fs.pathExistsSync(pkgPath)) return null
     const pkg = fs.readJsonSync(pkgPath)
     if (typeof pkg.packageManager !== 'string') return null
-    const match = pkg.packageManager.match(/^(npm|pnpm|yarn)@(\S+)/)
+    const match = pkg.packageManager.match(/^(npm|pnpm)@(\S+)/)
     if (!match) return null
     return { name: match[1], version: match[2], source: 'package-manager-field' }
   } catch {
@@ -109,7 +108,7 @@ export function detectPackageManager(projectRoot, { userAgent = process.env.npm_
 }
 
 /**
- * Non-interactive resolution: detection first, then npm if available,
+ * Non-interactive resolution: detection first, then pnpm if available,
  * then whatever binary exists. Never returns null so update/doctor flows
  * always have a usable PM name (commands fail later with a clear error
  * if no PM is actually installed).
@@ -119,16 +118,17 @@ export function resolvePackageManager(projectRoot, options = {}) {
   if (detected) return detected
 
   const available = getAvailablePackageManagers()
+  if (available.includes('pnpm')) return { name: 'pnpm', version: null, source: 'fallback' }
   if (available.includes('npm')) return { name: 'npm', version: null, source: 'fallback' }
   if (available.length > 0) return { name: available[0], version: null, source: 'fallback' }
-  return { name: 'npm', version: null, source: 'fallback (none detected)' }
+  return { name: 'pnpm', version: null, source: 'fallback (none detected)' }
 }
 
 export function installCommand(pmName, packages, { dev = false, global: isGlobal = false } = {}) {
   const pkgs = Array.isArray(packages) ? packages.join(' ') : packages
   const flags = [isGlobal ? '-g' : '', dev ? '-D' : ''].filter(Boolean).join(' ')
   const flagSuffix = flags ? ` ${flags}` : ''
-  if (pmName === 'pnpm' || pmName === 'yarn') {
+  if (pmName === 'pnpm') {
     return `${pmName} add${flagSuffix} ${pkgs}`
   }
   return `npm install${flagSuffix} ${pkgs}`
@@ -137,7 +137,7 @@ export function installCommand(pmName, packages, { dev = false, global: isGlobal
 export function uninstallCommand(pmName, packages, { global: isGlobal = false } = {}) {
   const pkgs = Array.isArray(packages) ? packages.join(' ') : packages
   const flagSuffix = isGlobal ? ' -g' : ''
-  if (pmName === 'pnpm' || pmName === 'yarn') {
+  if (pmName === 'pnpm') {
     return `${pmName} remove${flagSuffix} ${pkgs}`
   }
   return `npm uninstall${flagSuffix} ${pkgs}`

package/commands/utils/sliceScripts.js

@@ -5,17 +5,19 @@
 // client.js, and `slice init` — so the three can never drift apart (they did:
 // client.js was missing slice:types, and none of them had slice:build).
 export const SLICE_SCRIPTS = {
-  'slice:init': 'slice init',
-  'slice:dev': 'slice dev',
-  'slice:build': 'slice build',
-  'slice:start': 'slice start',
-  'slice:create': 'slice component create',
-  'slice:list': 'slice component list',
-  'slice:delete': 'slice component delete',
-  'slice:get': 'slice get',
-  'slice:browse': 'slice browse',
-  'slice:sync': 'slice sync',
-  'slice:version': 'slice version',
-  'slice:update': 'slice update',
-  'slice:types': 'slice types generate',
+  'slice:init': 'node ./node_modules/slicejs-cli/client.js init',
+  'slice:dev': 'node ./node_modules/slicejs-cli/client.js dev',
+  'slice:build': 'node ./node_modules/slicejs-cli/client.js build',
+  'slice:start': 'node ./node_modules/slicejs-cli/client.js start',
+  'slice:create': 'node ./node_modules/slicejs-cli/client.js component create',
+  'slice:list': 'node ./node_modules/slicejs-cli/client.js component list',
+  'slice:delete': 'node ./node_modules/slicejs-cli/client.js component delete',
+  'slice:get': 'node ./node_modules/slicejs-cli/client.js get',
+  'slice:browse': 'node ./node_modules/slicejs-cli/client.js browse',
+  'slice:sync': 'node ./node_modules/slicejs-cli/client.js sync',
+  'slice:doctor': 'node ./node_modules/slicejs-cli/client.js doctor',
+  'slice:version': 'node ./node_modules/slicejs-cli/client.js version',
+  'slice:help': 'node ./node_modules/slicejs-cli/client.js --help',
+  'slice:update': 'node ./node_modules/slicejs-cli/client.js update',
+  'slice:types': 'node ./node_modules/slicejs-cli/client.js types generate',
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slicejs-cli",
-  "version": "3.6.0",
+  "version": "3.6.1",
   "description": "Command client for developing web applications with Slice.js framework",
   "main": "client.js",
   "bin": {

package/post.js

@@ -3,6 +3,7 @@ import path from 'path';
 import { fileURLToPath } from 'url';
 import { getProjectRoot, getPath } from './commands/utils/PathHelper.js';
 import { SLICE_SCRIPTS } from './commands/utils/sliceScripts.js';
+import { resolvePackageManager, runScriptCommand } from './commands/utils/PackageManager.js';
 
 const __filename = fileURLToPath(import.meta.url);
 
@@ -21,6 +22,7 @@ if (isGlobal) {
 
 const projectRoot = getProjectRoot(import.meta.url);
 const pkgPath = getPath(import.meta.url, 'package.json');
+const packageManager = resolvePackageManager(projectRoot).name;
 
 const sliceScripts = SLICE_SCRIPTS;
 
@@ -47,12 +49,12 @@ try {
     }
 
     fs.writeFileSync(pkgPath, JSON.stringify(pkg, null, 2), 'utf-8');
-    console.log(`✅  slicejs-cli installed successfully. Added ${addedCount} npm scripts to package.json.`);
-    console.log('   Run: npm run slice:dev');
+    console.log(`✅  slicejs-cli installed successfully. Added ${addedCount} package scripts to package.json.`);
+    console.log(`   Run: ${runScriptCommand(packageManager, 'slice:dev')}`);
 } catch (err) {
     console.log('✅  slicejs-cli installed successfully.');
     console.log('   Could not auto-configure scripts:', err.message);
-    console.log('   Run: npx slice dev');
+    console.log(`   Configure scripts manually and run: ${runScriptCommand(packageManager, 'slice:dev')}`);
 }
 
 process.exit(0);