slicejs-cli 3.5.1 → 3.6.1

package/commands/init/init.js

@@ -5,10 +5,107 @@ import ora from 'ora';
 import Print from '../Print.js';
 import { getProjectRoot, getApiPath, getSrcPath, getPath } from '../utils/PathHelper.js';
 import { execSync } from 'child_process';
+import {
+    resolvePackageManager,
+    getPackageManagerVersion,
+    installCommand
+} from '../utils/PackageManager.js';
+import { SLICE_SCRIPTS } from '../utils/sliceScripts.js';
 
 // Import ComponentRegistry class from getComponent
 import { ComponentRegistry } from '../getComponent/getComponent.js';
 
+// Fetch the latest published version straight from the npm registry. This is
+// informational only (we never pin installs to it): it avoids depending on
+// `npm view` (absent on pnpm-only machines) and plays nice with pnpm's
+// minimumReleaseAge quarantine, which may legitimately resolve an older version.
+async function fetchLatestVersion(packageName) {
+    try {
+        const response = await fetch(`https://registry.npmjs.org/${packageName}/latest`, {
+            headers: { 'Accept': 'application/json' }
+        });
+        if (!response.ok) return null;
+        const data = await response.json();
+        return data.version || null;
+    } catch {
+        return null;
+    }
+}
+
+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.
+// Exported for tests (init-project-isolation.test.js).
+export async function ensureProjectManifest(projectRoot, packageManager) {
+    const pkgPath = path.join(projectRoot, 'package.json');
+    if (await fs.pathExists(pkgPath)) return pkgPath;
+
+    const pkg = {
+        name: path.basename(projectRoot),
+        version: '1.0.0',
+        description: 'Slice.js project',
+        main: 'api/index.js',
+        type: 'module',
+        engines: { node: '>=20.0.0' },
+        scripts: {}
+    };
+
+    // Persist the chosen package manager (corepack convention) so every later
+    // command — slice update, slice doctor — detects it deterministically.
+    const pmVersion = getPackageManagerVersion(packageManager);
+    if (pmVersion) {
+        pkg.packageManager = `${packageManager}@${pmVersion}`;
+    }
+
+    await fs.writeFile(pkgPath, JSON.stringify(pkg, null, 2), 'utf8');
+    return pkgPath;
+}
+
 // Visual components used by the App Shell + MultiRoute starter project.
 // We install only these on init; newcomers add more on demand with `slice get <Name>`.
 const STARTER_VISUAL_COMPONENTS = [
@@ -30,32 +127,56 @@ const STARTER_SERVICE_COMPONENTS = [
    'LocalStorageManager'
 ];
 
-export default async function initializeProject(projectType) {
+export default async function initializeProject(options = {}) {
     try {
         const projectRoot = getProjectRoot(import.meta.url);
         const destinationApi = getApiPath(import.meta.url);
         const destinationSrc = getSrcPath(import.meta.url);
 
+        // Resolve the package manager chosen in `slice init` (or detect it when
+        // initializeProject is invoked directly, e.g. inside an existing folder).
+        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);
+
         const fwSpinner = ora('Ensuring latest Slice framework...').start();
         let latestVersion = null;
+        let installedVersion = null;
         let sliceBaseDir;
         try {
-            const latest = execSync('npm view slicejs-web-framework version', { cwd: projectRoot }).toString().trim();
-            latestVersion = latest;
+            latestVersion = await fetchLatestVersion('slicejs-web-framework');
             const installedPkgPath = getPath(import.meta.url, 'node_modules', 'slicejs-web-framework', 'package.json');
             let installed = null;
             if (await fs.pathExists(installedPkgPath)) {
                 const pkg = await fs.readJson(installedPkgPath);
                 installed = pkg.version;
             }
-            if (installed !== latest) {
-                execSync(`npm install slicejs-web-framework@${latest} --save`, { cwd: projectRoot, stdio: 'inherit' });
+            if (!installed || (latestVersion && installed !== latestVersion)) {
+                // Install WITHOUT pinning an exact version: the package manager
+                // resolves it under its own policies (e.g. pnpm minimumReleaseAge
+                // quarantines versions younger than the configured age — pinning
+                // the registry's freshest version would make resolution fail).
+                execSync(installCommand(packageManager, 'slicejs-web-framework'), { cwd: projectRoot, stdio: 'inherit' });
+            }
+            if (await fs.pathExists(installedPkgPath)) {
+                const pkg = await fs.readJson(installedPkgPath);
+                installedVersion = pkg.version;
             }
             sliceBaseDir = getPath(import.meta.url, 'node_modules', 'slicejs-web-framework');
-            fwSpinner.succeed(`slicejs-web-framework@${latest} ready`);
+            fwSpinner.succeed(`slicejs-web-framework@${installedVersion || 'unknown'} ready`);
+            if (latestVersion && installedVersion && installedVersion !== latestVersion) {
+                Print.info(`Latest published is ${latestVersion}; your package manager resolved ${installedVersion} (release-age policy or cached registry).`);
+            }
         } catch (err) {
             // Fallback uses __dirname-style path because it looks for a local development copy,
-            // not a project-relative path — npm install failed, so we fall back to monorepo sibling.
+            // not a project-relative path — the install failed, so we fall back to monorepo sibling.
             const fallback = path.join(path.dirname(fileURLToPath(import.meta.url)), '../../../slicejs-web-framework');
             if (await fs.pathExists(fallback)) {
                 sliceBaseDir = fallback;
@@ -67,6 +188,21 @@ export default async function initializeProject(projectType) {
             }
         }
 
+        // 0b. INSTALL THE CLI LOCALLY (devDependency) so the generated scripts
+        // (`npm run dev` → `slice dev`) resolve via local delegation to a version
+        // pinned per project, as the docs recommend.
+        const cliSpinner = ora('Installing slicejs-cli as devDependency...').start();
+        try {
+            const cliPkgPath = getPath(import.meta.url, 'node_modules', 'slicejs-cli', 'package.json');
+            if (!(await fs.pathExists(cliPkgPath))) {
+                execSync(installCommand(packageManager, 'slicejs-cli', { dev: true }), { cwd: projectRoot, stdio: 'inherit' });
+            }
+            cliSpinner.succeed('slicejs-cli installed locally');
+        } catch (err) {
+            cliSpinner.warn('Could not install slicejs-cli locally — scripts will use the global CLI');
+            Print.info(`You can add it later with: ${installCommand(packageManager, 'slicejs-cli', { dev: true })}`);
+        }
+
         // These derive from sliceBaseDir (which comes from npm install or fallback),
         // so they're already dynamic — no PathHelper needed.
         const apiDir = path.join(sliceBaseDir, 'api');
@@ -169,20 +305,20 @@ export default async function initializeProject(projectType) {
                     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
@@ -208,7 +344,7 @@ export default async function initializeProject(projectType) {
                     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');
                 }
@@ -218,7 +354,7 @@ export default async function initializeProject(projectType) {
         } 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
@@ -243,53 +379,43 @@ export default async function initializeProject(projectType) {
             pkg.scripts = pkg.scripts || {};
             pkg.dependencies = pkg.dependencies || {};
 
-            // Comandos principales
-            pkg.scripts['dev'] = 'slice dev';
-            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';
-
-            // Atajos de repositorio
-            pkg.scripts['get'] = 'slice get';
-            pkg.scripts['browse'] = 'slice browse';
-            pkg.scripts['sync'] = 'slice sync';
-
-            // Utilidades
-            pkg.scripts['slice:version'] = 'slice version';
-            pkg.scripts['slice:update'] = 'slice update';
-            pkg.scripts['slice:types'] = 'slice types generate';
-
-            // Legacy (compatibility)
-            pkg.scripts['slice:init'] = 'slice init';
-            pkg.scripts['slice:start'] = 'slice start';
-            pkg.scripts['slice:dev'] = 'slice dev';
-            pkg.scripts['slice:create'] = 'slice component create';
-            pkg.scripts['slice:list'] = 'slice component list';
-            pkg.scripts['slice:delete'] = 'slice component delete';
-            pkg.scripts['slice:get'] = 'slice get';
-            pkg.scripts['slice:browse'] = 'slice browse';
-            pkg.scripts['slice:sync'] = 'slice sync';
-            pkg.scripts['run'] = 'slice dev';
+            pkg.scripts['component:create'] = SLICE_SCRIPTS['slice:create'];
+            pkg.scripts['component:list'] = SLICE_SCRIPTS['slice:list'];
+            pkg.scripts['component:delete'] = SLICE_SCRIPTS['slice:delete'];
+
+            // 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_SCRIPTS['slice:dev'];
 
             // Module configuration
             pkg.type = 'module';
             pkg.engines = pkg.engines || { node: '>=20.0.0' };
 
-            // Ensure framework dependency is present
+            // Ensure framework dependency is present (the install above normally
+            // already wrote it; this is a fallback for the monorepo-sibling path).
             if (!pkg.dependencies['slicejs-web-framework']) {
-                pkg.dependencies['slicejs-web-framework'] = latestVersion ? latestVersion : 'latest';
+                pkg.dependencies['slicejs-web-framework'] = installedVersion ? `^${installedVersion}` : 'latest';
             }
 
             await fs.writeFile(pkgPath, JSON.stringify(pkg, null, 2), 'utf8');
-            pkgSpinner.succeed('npm scripts configured successfully');
+            pkgSpinner.succeed('Package scripts configured successfully');
 
             Print.title('New recommended commands:');
-            console.log('  npm run dev            - Start development server');
-            console.log('  npm run get            - Install components');
-            console.log('  npm run browse         - Browse components');
+            console.log(`  ${packageManager} run dev            - Start development server`);
+            console.log(`  ${packageManager} run get            - Install components`);
+            console.log(`  ${packageManager} run browse         - Browse components`);
         } catch (error) {
         pkgSpinner.fail('Failed to configure npm scripts');
         Print.error(error.message);
@@ -300,9 +426,10 @@ export default async function initializeProject(projectType) {
         Print.newLine();
         Print.title('Next steps:');
         console.log(`  cd ${projectName}`);
-        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 dev     - Start development server`);
+        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

@@ -0,0 +1,148 @@
+// commands/utils/PackageManager.js
+//
+// 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 pnpm (non-interactive).
+
+import fs from 'fs-extra'
+import path from 'path'
+import { spawnSync } from 'node:child_process'
+
+export const SUPPORTED_PACKAGE_MANAGERS = ['pnpm', 'npm']
+
+const LOCKFILES = {
+  'pnpm-lock.yaml': 'pnpm',
+  'package-lock.json': 'npm'
+}
+
+export function parseUserAgent(userAgent = process.env.npm_config_user_agent) {
+  if (!userAgent) return null
+  const match = userAgent.match(/^(npm|pnpm)\/(\S+)/)
+  if (!match) return null
+  return { name: match[1], version: match[2], source: 'user-agent' }
+}
+
+export function fromPackageManagerField(projectRoot) {
+  try {
+    const pkgPath = path.join(projectRoot, 'package.json')
+    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)@(\S+)/)
+    if (!match) return null
+    return { name: match[1], version: match[2], source: 'package-manager-field' }
+  } catch {
+    return null
+  }
+}
+
+export function fromLockfile(projectRoot) {
+  for (const [lockfile, name] of Object.entries(LOCKFILES)) {
+    if (fs.pathExistsSync(path.join(projectRoot, lockfile))) {
+      return { name, version: null, source: `lockfile (${lockfile})` }
+    }
+  }
+  return null
+}
+
+function runPmBinary(name, args) {
+  // On Windows the PM entry points are .cmd shims, which require shell resolution.
+  const isWindows = process.platform === 'win32'
+  return spawnSync(name, args, {
+    stdio: ['ignore', 'pipe', 'ignore'],
+    shell: isWindows,
+    encoding: 'utf-8'
+  })
+}
+
+export function isPackageManagerAvailable(name) {
+  try {
+    const result = runPmBinary(name, ['--version'])
+    return result.status === 0
+  } catch {
+    return false
+  }
+}
+
+export function getPackageManagerVersion(name) {
+  try {
+    const result = runPmBinary(name, ['--version'])
+    if (result.status !== 0) return null
+    return (result.stdout || '').toString().trim() || null
+  } catch {
+    return null
+  }
+}
+
+export function getAvailablePackageManagers() {
+  return SUPPORTED_PACKAGE_MANAGERS.filter(isPackageManagerAvailable)
+}
+
+/**
+ * Detect the package manager for a project.
+ * Returns { name, version, source } or null when genuinely ambiguous.
+ */
+export function detectPackageManager(projectRoot, { userAgent = process.env.npm_config_user_agent } = {}) {
+  if (projectRoot) {
+    const fromField = fromPackageManagerField(projectRoot)
+    if (fromField) return fromField
+
+    const fromLock = fromLockfile(projectRoot)
+    if (fromLock) return fromLock
+  }
+
+  const fromUa = parseUserAgent(userAgent)
+  if (fromUa) return fromUa
+
+  const available = getAvailablePackageManagers()
+  if (available.length === 1) {
+    return { name: available[0], version: null, source: 'only available binary' }
+  }
+
+  return null
+}
+
+/**
+ * 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).
+ */
+export function resolvePackageManager(projectRoot, options = {}) {
+  const detected = detectPackageManager(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: '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') {
+    return `${pmName} add${flagSuffix} ${pkgs}`
+  }
+  return `npm install${flagSuffix} ${pkgs}`
+}
+
+export function uninstallCommand(pmName, packages, { global: isGlobal = false } = {}) {
+  const pkgs = Array.isArray(packages) ? packages.join(' ') : packages
+  const flagSuffix = isGlobal ? ' -g' : ''
+  if (pmName === 'pnpm') {
+    return `${pmName} remove${flagSuffix} ${pkgs}`
+  }
+  return `npm uninstall${flagSuffix} ${pkgs}`
+}
+
+export function runScriptCommand(pmName, script) {
+  return `${pmName} run ${script}`
+}

package/commands/utils/VersionChecker.js

@@ -5,6 +5,7 @@ import path from "path";
 import { fileURLToPath } from "url";
 import Print from "../Print.js";
 import { getProjectRoot, getPath } from "../utils/PathHelper.js";
+import { resolvePackageManager } from "../utils/PackageManager.js";
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
@@ -110,17 +111,18 @@ class VersionChecker {
       const frameworkStatus = this.compareVersions(current.framework, latest.framework);
 
       if (!silent && (cliStatus === 'outdated' || frameworkStatus === 'outdated')) {
+        const pm = resolvePackageManager(getProjectRoot(import.meta.url)).name;
         console.log(''); // Line break
         Print.warning('📦 Available Updates:');
-        
+
         if (cliStatus === 'outdated') {
           console.log(`   🔧 CLI: ${current.cli} → ${latest.cli}`);
-          console.log(`       npm update slicejs-cli`);
+          console.log(`       ${pm} update slicejs-cli`);
         }
-        
+
         if (frameworkStatus === 'outdated') {
           console.log(`   ⚡ Framework: ${current.framework} → ${latest.framework}`);
-          console.log(`       npm update slicejs-web-framework`);
+          console.log(`       ${pm} update slicejs-web-framework`);
         }
         
         console.log('   📚 Changelog: https://github.com/VKneider/slice.js/releases');

package/commands/utils/sliceScripts.js

@@ -0,0 +1,23 @@
+// commands/utils/sliceScripts.js
+//
+// Single source of truth for the slice:* package scripts configured by the CLI.
+// Used by post.js (the postinstall hook), the `slice postinstall` command in
+// 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': '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',
+};