@lenne.tech/cli 1.33.0 → 1.35.0

package/build/commands/dev/up.js

@@ -14,6 +14,7 @@ const path_1 = require("path");
 const caddy_1 = require("../../lib/caddy");
 const dev_env_1 = require("../../lib/dev-env");
 const dev_env_bridge_1 = require("../../lib/dev-env-bridge");
+const dev_package_manager_1 = require("../../lib/dev-package-manager");
 const dev_patches_1 = require("../../lib/dev-patches");
 const dev_process_1 = require("../../lib/dev-process");
 const dev_project_1 = require("../../lib/dev-project");
@@ -85,7 +86,8 @@ const UpCommand = {
         const { filesystem, parameters, print: { colors, error, info, success, warning }, } = toolbox;
         const layout = (0, dev_project_1.resolveLayout)(filesystem.cwd(), filesystem);
         if (!layout.apiDir && !layout.appDir) {
-            error('No API or App project detected at this path. Run `lt dev init` first.');
+            error('No lt project detected at this path. Expected an API (src/config.env.ts or nest-cli.json), ' +
+                'an App (nuxt.config.ts), or a monorepo with projects/api and/or projects/app.');
             if (!parameters.options.fromGluegunMenu)
                 process.exit(1);
             return 'dev up: not a project';
@@ -270,7 +272,9 @@ const UpCommand = {
                 apiUpstreamPort: existingEntry === null || existingEntry === void 0 ? void 0 : existingEntry.internalPorts.api,
                 appHostname: (_d = identity.subdomains.app) === null || _d === void 0 ? void 0 : _d.hostname,
                 appUpstreamPort: existingEntry === null || existingEntry === void 0 ? void 0 : existingEntry.internalPorts.app,
-                dbName: (_e = existingEntry === null || existingEntry === void 0 ? void 0 : existingEntry.dbName) !== null && _e !== void 0 ? _e : dbName,
+                // App-only projects have no database — don't advertise a Mongo URL
+                // that nothing listens on.
+                dbName: identity.subdomains.api ? ((_e = existingEntry === null || existingEntry === void 0 ? void 0 : existingEntry.dbName) !== null && _e !== void 0 ? _e : dbName) : undefined,
             });
             info('Run `lt dev down` first to force a full restart.');
             if (!parameters.options.fromGluegunMenu)
@@ -316,7 +320,8 @@ const UpCommand = {
             info(`  app: https://${identity.subdomains.app.hostname}  →  127.0.0.1:${appPort}`);
         if (identity.subdomains.api)
             info(`  api: https://${identity.subdomains.api.hostname}  →  127.0.0.1:${apiPort}`);
-        info(`  db:  mongodb://127.0.0.1/${dbName}`);
+        if (identity.subdomains.api)
+            info(`  db:  mongodb://127.0.0.1/${dbName}`);
         info('');
         // Build env per process.
         const devEnv = (0, dev_env_1.buildDevEnv)({
@@ -326,7 +331,6 @@ const UpCommand = {
             dbName,
             identity,
         });
-        const pnpmBin = process.env.LT_PNPM_BIN || 'pnpm';
         const pids = {};
         const rotationNotes = [];
         const started = [];
@@ -350,7 +354,12 @@ const UpCommand = {
             }
             else {
                 yield reclaimPort(existingSession === null || existingSession === void 0 ? void 0 : existingSession.pids.api, apiPort, apiHealth !== null && apiHealth !== void 0 ? apiHealth : 'dead');
-                const apiResult = (0, dev_process_1.spawnDetached)(pnpmBin, ['start'], {
+                // Per-component PM detection: a monorepo may have an npm api and a
+                // pnpm app, and the legacy hard-coded `pnpm start` would silently
+                // regenerate a foreign lockfile + crash on un-approved build
+                // scripts when run against an npm-only project.
+                const apiPm = (0, dev_package_manager_1.pickPackageManager)(layout.apiDir);
+                const apiResult = (0, dev_process_1.spawnDetached)(apiPm.bin, apiPm.runScript('start'), {
                     cwd: layout.apiDir,
                     env: devEnv.api.env,
                     logFile: (0, path_1.join)(layout.root, '.lt-dev', 'api.log'),
@@ -371,7 +380,8 @@ const UpCommand = {
             }
             else {
                 yield reclaimPort(existingSession === null || existingSession === void 0 ? void 0 : existingSession.pids.app, appPort, appHealth !== null && appHealth !== void 0 ? appHealth : 'dead');
-                const appResult = (0, dev_process_1.spawnDetached)(pnpmBin, ['dev'], {
+                const appPm = (0, dev_package_manager_1.pickPackageManager)(layout.appDir);
+                const appResult = (0, dev_process_1.spawnDetached)(appPm.bin, appPm.runScript('dev'), {
                     cwd: layout.appDir,
                     env: devEnv.app.env,
                     logFile: (0, path_1.join)(layout.root, '.lt-dev', 'app.log'),
@@ -408,7 +418,7 @@ const UpCommand = {
             apiUpstreamPort: apiPort,
             appHostname: (_l = identity.subdomains.app) === null || _l === void 0 ? void 0 : _l.hostname,
             appUpstreamPort: appPort,
-            dbName,
+            dbName: identity.subdomains.api ? dbName : undefined,
         });
         info(colors.dim('Logs: <root>/.lt-dev/api.log, <root>/.lt-dev/app.log'));
         for (const note of rotationNotes)

package/build/commands/frontend/nuxt.js

@@ -15,7 +15,7 @@ const workspace_integration_1 = require("../../lib/workspace-integration");
  * Create a new nuxt workspace
  *
  * Standalone counterpart to `lt fullstack init` / `lt fullstack add-app`
- * for Nuxt: clones nuxt-base-starter (or invokes create-nuxt-base) into
+ * for Nuxt: clones nuxt-base-starter into
  * a brand-new directory. Mirrors the same surface area as add-app where
  * applicable so behaviour is consistent across the four flows.
  */
@@ -170,7 +170,7 @@ const NewCommand = {
             info('Dry-run plan:');
             info(`  name:                       ${projName}`);
             info(`  projectDir:                 ${projectDir}`);
-            info(`  branch:                     ${branch || '(default — uses create-nuxt-base)'}`);
+            info(`  branch:                     ${branch || '(default branch)'}`);
             info(`  copy:                       ${copyPath || '(none)'}`);
             info(`  link:                       ${linkPath || '(none)'}`);
             info(`  frontendFrameworkMode:      ${frontendFrameworkMode}`);
@@ -183,11 +183,8 @@ const NewCommand = {
             else if (copyPath) {
                 info(`  1. copy ${copyPath} → ./${projectDir}`);
             }
-            else if (branch) {
-                info(`  1. clone nuxt-base-starter (branch: ${branch}) → ./${projectDir}`);
-            }
             else {
-                info(`  1. exec create-nuxt-base@latest → ./${projectDir}`);
+                info(`  1. clone nuxt-base-starter (${branch ? `branch: ${branch}` : 'default branch'}) → ./${projectDir}`);
             }
             if (frontendFrameworkMode === 'vendor') {
                 info(`  2. clone @lenne.tech/nuxt-extensions → /tmp`);
@@ -206,7 +203,7 @@ const NewCommand = {
             branch,
             copyPath,
             linkPath,
-            skipInstall: true, // Nuxt standalone doesn't need npm install (create-nuxt-base handles it)
+            skipInstall: false, // clone-based setup installs here (create-nuxt-base no longer used)
         });
         if (!result.success) {
             baseSpinner.fail(`Failed to setup nuxt workspace: ${result.path}`);

package/build/commands/fullstack/add-api.js

@@ -10,7 +10,6 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.help = void 0;
-const hoist_workspace_pnpm_config_1 = require("../../lib/hoist-workspace-pnpm-config");
 const workspace_integration_1 = require("../../lib/workspace-integration");
 /**
  * Add an API (`projects/api/`) to a fullstack workspace that currently
@@ -320,14 +319,12 @@ const NewCommand = {
         if (apiResult.method !== 'link') {
             (0, workspace_integration_1.writeApiConfig)({ apiDir: apiDest, apiMode, filesystem, frameworkMode });
         }
-        // Hoist pnpm config — `setupServerForFullstack` may have produced a
-        // sub-project package.json with `pnpm.overrides` etc. that pnpm
-        // ignores at non-root level.
-        (0, hoist_workspace_pnpm_config_1.hoistWorkspacePnpmConfig)({
-            filesystem,
-            projectDir: workspaceDir,
-            subProjects: ['projects/api', 'projects/app'],
-        });
+        // Normalize the workspace root: hoist pnpm config out of the sub-projects
+        // (`setupServerForFullstack` may have left `pnpm.overrides` etc. at a
+        // non-root level pnpm ignores), drop nested lockfiles the root supersedes,
+        // and guarantee a root `.dockerignore` (Docker never reads a sub-project's
+        // own when building from the root context).
+        (0, workspace_integration_1.finalizeWorkspaceRoot)({ filesystem, projectDir: workspaceDir });
         // Run install + format unless explicitly skipped (CI/agents may
         // want to chain multiple add-* calls before installing once).
         if (!skipInstall && !experimental) {

package/build/commands/fullstack/add-app.js

@@ -10,7 +10,6 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.help = void 0;
-const hoist_workspace_pnpm_config_1 = require("../../lib/hoist-workspace-pnpm-config");
 const workspace_integration_1 = require("../../lib/workspace-integration");
 /**
  * Add a frontend app (`projects/app/`) to a fullstack workspace that
@@ -249,12 +248,11 @@ const NewCommand = {
                 warning('Continuing with npm mode for frontend.');
             }
         }
-        // Hoist pnpm config (frontend templates may carry pnpm.overrides too).
-        (0, hoist_workspace_pnpm_config_1.hoistWorkspacePnpmConfig)({
-            filesystem,
-            projectDir: workspaceDir,
-            subProjects: ['projects/api', 'projects/app'],
-        });
+        // Normalize the workspace root: hoist pnpm config (frontend templates may
+        // carry pnpm.overrides too), drop the frontend starter's standalone
+        // pnpm-lock.yaml the root supersedes, and guarantee a root `.dockerignore`
+        // (else the image inherits the host's node_modules, .output and .env).
+        (0, workspace_integration_1.finalizeWorkspaceRoot)({ filesystem, projectDir: workspaceDir });
         if (!skipInstall) {
             const installSpinner = spin('Install workspace packages');
             try {

package/build/commands/fullstack/init.js

@@ -16,7 +16,6 @@ const gluegun_1 = require("gluegun");
 const caddy_1 = require("../../lib/caddy");
 const dev_migrate_helper_1 = require("../../lib/dev-migrate-helper");
 const dev_project_1 = require("../../lib/dev-project");
-const hoist_workspace_pnpm_config_1 = require("../../lib/hoist-workspace-pnpm-config");
 const package_name_1 = require("../../lib/package-name");
 const vendor_claude_md_1 = require("../../lib/vendor-claude-md");
 const workspace_integration_1 = require("../../lib/workspace-integration");
@@ -33,7 +32,7 @@ const NewCommand = {
     run: (toolbox) => __awaiter(void 0, void 0, void 0, function* () {
         var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t, _u, _v, _w, _x, _y, _z, _0;
         // Retrieve the tools we need
-        const { config, filesystem, frontendHelper, git, helper, parameters, patching, print: { colors, error, info, spin, success }, prompt: { ask, confirm }, server, strings: { kebabCase }, system, template, } = toolbox;
+        const { config, filesystem, frontendHelper, git, helper, parameters, patching, print: { colors, error, info, spin, success, warning }, prompt: { ask, confirm }, server, strings: { kebabCase }, system, template, } = toolbox;
         // Start timer
         const timer = system.startTimer();
         // Info
@@ -209,7 +208,9 @@ const NewCommand = {
         //            `/lt-dev:backend:update-nest-server-core`; local patches are logged
         //            in src/core/VENDOR.md.
         //
-        // Default is still 'npm' until the vendoring pilot is fully evaluated.
+        // Default is 'vendor' — the lt CLI integrates the framework core directly
+        // (clones the starter, vendors core into src/core/), which is the
+        // recommended setup; pass `--framework-mode npm` for the classic npm dep.
         let frameworkMode;
         if (experimental) {
             frameworkMode = 'npm';
@@ -226,16 +227,16 @@ const NewCommand = {
             info(`Using framework mode from lt.config: ${frameworkMode}`);
         }
         else if (noConfirm) {
-            frameworkMode = 'npm';
-            info('Using default framework mode: npm (noConfirm mode)');
+            frameworkMode = 'vendor';
+            info('Using default framework mode: vendor (noConfirm mode)');
         }
         else {
             const frameworkModeChoice = yield ask({
                 choices: [
                     'npm    - @lenne.tech/nest-server as npm dependency (classic, stable)',
-                    'vendor - framework core vendored into projects/api/src/core/ (pilot, allows local patches)',
+                    'vendor - framework core vendored into projects/api/src/core/ (default, allows local patches)',
                 ],
-                initial: 0,
+                initial: 1,
                 message: 'Framework consumption mode?',
                 name: 'frameworkMode',
                 type: 'select',
@@ -257,11 +258,11 @@ const NewCommand = {
             info(`Using frontend framework mode from lt.config: ${frontendFrameworkMode}`);
         }
         else if (noConfirm) {
-            frontendFrameworkMode = 'npm';
+            frontendFrameworkMode = 'vendor';
         }
         else {
-            // Default to npm without asking (unless user sets it explicitly)
-            frontendFrameworkMode = 'npm';
+            // Default to vendor without asking (unless user sets it explicitly)
+            frontendFrameworkMode = 'vendor';
         }
         // Determine remote push settings with priority: CLI > config > interactive
         // Git is always initialized; the question is whether to push to a remote
@@ -273,8 +274,13 @@ const NewCommand = {
             if (pushToRemote) {
                 gitLink = cliGitLink || configGitLink;
                 if (!gitLink) {
-                    error('--git-link is required when --git is true (or configure gitLink in lt.config)');
-                    return;
+                    // Git is always initialized locally (dev branch + initial commit);
+                    // --git only controls the remote push. Without a link there is
+                    // nothing to push to, so degrade to a local-only init instead of
+                    // aborting the whole scaffold — consistent with the config and
+                    // interactive branches below.
+                    warning('--git true without --git-link: initializing local git only (no remote push).');
+                    pushToRemote = false;
                 }
             }
         }
@@ -314,6 +320,14 @@ const NewCommand = {
                 }
             }
         }
+        else if (noConfirm) {
+            // Default in noConfirm mode: git true. Git is always initialized locally
+            // (dev branch + initial commit); push to a remote only when a gitLink is
+            // configured, otherwise stay local-only (nothing to push to).
+            gitLink = configGitLink;
+            pushToRemote = !!gitLink;
+            info(`Using default git: true${gitLink ? ' (push to configured gitLink)' : ' (local only — no --git-link configured)'}`);
+        }
         // Determine branches and copy/link paths with priority: CLI > config
         const apiBranch = cliApiBranch || configApiBranch;
         // Under `--next`, default the nuxt-base-starter ref to the `next`
@@ -606,14 +620,14 @@ const NewCommand = {
             else {
                 serverSpinner.warn('Nest Server Starter not integrated');
             }
-            // Hoist workspace-scoped pnpm config out of sub-projects. pnpm only
-            // honors `pnpm.overrides`, `pnpm.onlyBuiltDependencies`, and
-            // `pnpm.ignoredOptionalDependencies` at the workspace root; leaving
-            // them in projects/api/package.json or projects/app/package.json
-            // causes `WARN The field … was found in … This will not take
-            // effect. You should configure … at the root of the workspace
-            // instead.` and silently disables CVE overrides.
-            (0, hoist_workspace_pnpm_config_1.hoistWorkspacePnpmConfig)({ filesystem, projectDir, subProjects: ['projects/api', 'projects/app'] });
+            // Normalize the workspace root. Hoists workspace-scoped pnpm config out
+            // of the sub-projects (pnpm only honors `pnpm.overrides` etc. at the
+            // root — otherwise CVE overrides are silently disabled), drops the
+            // starters' standalone pnpm-lock.yaml files the root supersedes, and
+            // guarantees a root `.dockerignore` (else the images carry the host's
+            // node_modules, a stale .output, and a local .env that `nuxt build`
+            // bakes in).
+            (0, workspace_integration_1.finalizeWorkspaceRoot)({ filesystem, projectDir });
             // Install all packages
             if (!experimental) {
                 const installSpinner = spin('Install all packages');

package/build/commands/status.js

@@ -251,6 +251,8 @@ const StatusCommand = {
                     for (const [sub, host] of Object.entries(entry.subdomains)) {
                         info(`  ${sub.padEnd(6)} https://${host}`);
                     }
+                    // `loadRegistry` strips dbName from API-less entries, so its
+                    // presence alone means there is a database to advertise.
                     if (entry.dbName)
                         info(`  db     mongodb://127.0.0.1/${entry.dbName}`);
                     const session = (0, dev_state_1.loadSession)(registryRoot);

package/build/commands/ticket/list.js

@@ -61,6 +61,8 @@ const ListCommand = {
             if (entry) {
                 for (const [sub, host] of Object.entries(entry.subdomains))
                     info(`       ${sub.padEnd(4)} https://${host}`);
+                // `loadRegistry` strips dbName from API-less entries, so its presence
+                // alone means there is a database to show.
                 if (entry.dbName)
                     info(`       db   mongodb://127.0.0.1/${entry.dbName}`);
             }

package/build/commands/ticket/start.js

@@ -97,14 +97,16 @@ const StartCommand = {
         }
         // 2. tag the worktree with its ticket id (makes lt dev * ticket-aware).
         (0, dev_ticket_1.writeTicketMarker)(worktreePath, id);
-        // 3. install deps (pnpm hard-links from the shared store → fast).
+        // 3. install deps. Auto-detects pnpm/yarn/npm from the worktree's
+        // lockfile so an npm-only repo doesn't get a foreign pnpm-lock
+        // injected on every ticket start.
         if (parameters.options.install !== false) {
-            info(colors.dim('Installing dependencies (pnpm) …'));
+            info(colors.dim('Installing dependencies …'));
             try {
-                (0, dev_ticket_1.pnpmInstall)(worktreePath);
+                (0, dev_ticket_1.installWorktreeDeps)(worktreePath);
             }
             catch (e) {
-                warning(`pnpm install failed (${e.message}) — continuing; run it manually in the worktree.`);
+                warning(`install failed (${e.message}) — continuing; run it manually in the worktree.`);
             }
         }
         // 4. bring the isolated stack up (re-invokes THIS lt build so the marker is

package/build/extensions/frontend-helper.js

@@ -141,49 +141,54 @@ class FrontendHelper {
         return __awaiter(this, arguments, void 0, function* (dest, options = {}) {
             const { system, templateHelper } = this.toolbox;
             const { branch, copyPath, linkPath, skipInstall } = options;
-            // Use template extension for link/copy/branch operations
-            if (linkPath || copyPath || branch) {
-                const result = yield templateHelper.setup(dest, {
-                    branch,
-                    copyPath,
-                    isNuxt: true,
-                    linkPath,
-                    repoUrl: branch ? 'https://github.com/lenneTech/nuxt-base-starter.git' : undefined,
-                });
-                if (!result.success) {
-                    return { method: result.method, path: result.path, success: false };
-                }
-                // After a clone, flatten the wrapper layout so `projects/app/`
-                // IS the Nuxt app (the cloned root is the `create-nuxt-base`
-                // scaffolder, not the app — see flattenNuxtBaseTemplate).
-                // Skip on link mode: a symlink points at the user's local
-                // checkout and must not have its template subdir torn out.
-                if (result.method === 'clone') {
-                    yield this.flattenNuxtBaseTemplate(dest);
-                }
-                // Run install if not skipped and not a symlink
-                if (!skipInstall && result.method !== 'link') {
-                    try {
-                        const { pm } = this.toolbox;
-                        yield system.run(`cd "${dest}" && ${pm.install(pm.detect(dest))}`);
-                    }
-                    catch (err) {
-                        return { method: result.method, path: dest, success: false };
-                    }
-                }
-                return { method: result.method, path: result.path, success: true };
+            // Source the Nuxt app from the nuxt-base-starter repo (priority:
+            // link > copy > clone), mirroring the backend (nest-server-starter)
+            // flow. The default — no link/copy/branch — clones the repo's default
+            // branch, so a fresh init always integrates the CURRENT GitHub template.
+            //
+            // This deliberately replaces the former `create-nuxt-base@latest` npx
+            // path: that package is marked `private` and is no longer published
+            // (nuxt-base-starter commit b534af2 — "distribute via lt CLI fullstack
+            // template"), so `@latest` was frozen at an outdated copy and the
+            // bundled template drifted from GitHub. Cloning the repo directly is
+            // the same mechanism the backend already uses and removes that drift.
+            const result = yield templateHelper.setup(dest, {
+                branch,
+                copyPath,
+                isNuxt: true,
+                linkPath,
+                repoUrl: linkPath || copyPath ? undefined : 'https://github.com/lenneTech/nuxt-base-starter.git',
+            });
+            if (!result.success) {
+                return { method: result.method, path: result.path, success: false };
             }
-            // Default: use create-nuxt-base
-            try {
-                const { pm } = this.toolbox;
-                yield system.run(pm.exec(`create-nuxt-base@latest "${dest}"`));
-                // Fix package name - create-nuxt-base uses path as name which is invalid for lerna
+            // After a clone, flatten the wrapper layout so `dest` IS the Nuxt app
+            // (the cloned root carries the scaffolder; the app lives in
+            // `nuxt-base-template/` — see flattenNuxtBaseTemplate). Skip on link
+            // mode: a symlink points at the user's checkout and must not be torn out.
+            if (result.method === 'clone') {
+                yield this.flattenNuxtBaseTemplate(dest);
+            }
+            // Normalize the sub-project package name to a stable, valid workspace
+            // name ("app"). The template ships as `nuxt-base-template`; the former
+            // create-nuxt-base path renamed it the same way. Skip link mode — a
+            // symlinked checkout is shared with upstream and must not be mutated.
+            if (result.method !== 'link') {
                 yield this.fixPackageName(dest);
-                return { method: 'npx', path: dest, success: true };
             }
-            catch (err) {
-                return { method: 'npx', path: dest, success: false };
+            // Run install unless skipped or a symlink. Fullstack init installs at
+            // the monorepo level (skipInstall: true); standalone `lt frontend nuxt`
+            // installs here (create-nuxt-base used to handle that).
+            if (!skipInstall && result.method !== 'link') {
+                try {
+                    const { pm } = this.toolbox;
+                    yield system.run(`cd "${dest}" && ${pm.install(pm.detect(dest))}`);
+                }
+                catch (_a) {
+                    return { method: result.method, path: dest, success: false };
+                }
             }
+            return { method: result.method, path: result.path, success: true };
         });
     }
     /**