@lessonkit/cli 1.2.0 → 1.3.1

package/README.md

@@ -32,7 +32,7 @@ lessonkit package --target scorm12   # LMS artifact
 
 Every project includes a root `lessonkit.json` manifest (`schemaVersion: 1`).
 
-Subprocess timeout defaults to **30 minutes** (`LESSONKIT_CMD_TIMEOUT_MS`; set `0` to disable).
+Subprocess timeout defaults to **30 minutes** (`LESSONKIT_CMD_TIMEOUT_MS`). To disable the timeout, pass `timeoutMs: 0` in the exec API (the env var falls back to 30 minutes when set to `0` or omitted).
 
 ## Docs
 

package/dist/bin.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 
 // src/index.ts
-import { createRequire } from "module";
+import { createRequire as createRequire2 } from "module";
 import { Command } from "commander";
 
 // src/commands/init.ts
@@ -62,7 +62,7 @@ function resolveCommandTimeoutMs(explicit) {
   const raw = process.env.LESSONKIT_CMD_TIMEOUT_MS;
   if (raw === void 0 || raw === "") return DEFAULT_CMD_TIMEOUT_MS;
   const parsed = Number(raw);
-  if (!Number.isFinite(parsed) || parsed <= 0) return void 0;
+  if (!Number.isFinite(parsed) || parsed <= 0) return DEFAULT_CMD_TIMEOUT_MS;
   return parsed;
 }
 async function runCommand(command, args, opts) {
@@ -250,6 +250,7 @@ async function runInit(opts, logger) {
 // src/lib/project.ts
 import { readFileSync, existsSync as existsSync2 } from "fs";
 import { readFile as readFile2 } from "fs/promises";
+import { createRequire } from "module";
 import { dirname as dirname2, join as join2, parse, resolve as resolve2 } from "path";
 import { parseLessonkitManifest } from "@lessonkit/lxpack";
 var LESSONKIT_JSON = "lessonkit.json";
@@ -372,14 +373,22 @@ function assertViteProject(pkg, projectRoot) {
     );
   }
 }
+function projectDeclaresVite(pkg) {
+  return !!(pkg.devDependencies?.vite ?? pkg.dependencies?.vite);
+}
 function resolveViteJs(projectRoot) {
-  let dir = resolve2(projectRoot);
-  const fsRoot = parse(dir).root;
-  while (true) {
-    const viteJs = join2(dir, "node_modules", "vite", "bin", "vite.js");
-    if (existsSync2(viteJs)) return viteJs;
-    if (dir === fsRoot) break;
-    dir = dirname2(dir);
+  const root = resolve2(projectRoot);
+  const localViteJs = join2(root, "node_modules", "vite", "bin", "vite.js");
+  if (existsSync2(localViteJs)) return localViteJs;
+  try {
+    const pkg = JSON.parse(readFileSync(join2(root, PACKAGE_JSON), "utf8"));
+    if (projectDeclaresVite(pkg)) {
+      const req = createRequire(join2(root, "package.json"));
+      const vitePkg = req.resolve("vite/package.json");
+      const hoistedViteJs = join2(dirname2(vitePkg), "bin", "vite.js");
+      if (existsSync2(hoistedViteJs)) return hoistedViteJs;
+    }
+  } catch {
   }
   throw new CliError(
     `Vite not found near ${projectRoot}. Run npm install in the project first.`,
@@ -457,7 +466,10 @@ async function runDev(opts) {
   const pkg = await readPackageJson(project.root);
   assertViteProject(pkg, project.root);
   const viteJs = resolveViteJs(project.root);
-  await runCommand(process.execPath, [viteJs, ...opts.viteArgs ?? []], { cwd: project.root });
+  await runCommand(process.execPath, [viteJs, ...opts.viteArgs ?? []], {
+    cwd: project.root,
+    timeoutMs: 0
+  });
   return { ok: true, command: "dev", projectRoot: project.root };
 }
 async function runBuild(opts) {
@@ -541,6 +553,17 @@ async function runPackage(opts) {
       issues: result.issues
     });
   }
+  const warnings = result.validation?.issues?.filter((issue) => {
+    const severity = issue.severity?.toLowerCase();
+    return severity !== "error" && severity !== "fatal";
+  });
+  if (warnings?.length && !opts.json) {
+    for (const issue of warnings) {
+      const location = issue.path ?? "course";
+      process.stderr.write(`[lessonkit] packaging warning: ${location}: ${issue.message}
+`);
+    }
+  }
   return {
     ok: true,
     command: "package",
@@ -548,7 +571,8 @@ async function runPackage(opts) {
     projectRoot: project.root,
     outputPath: result.outputPath,
     outputDir: result.outputDir,
-    fileCount: result.fileCount
+    fileCount: result.fileCount,
+    ...warnings?.length ? { warnings } : {}
   };
 }
 
@@ -566,7 +590,7 @@ function createLogger(opts) {
 }
 
 // src/index.ts
-var require2 = createRequire(import.meta.url);
+var require2 = createRequire2(import.meta.url);
 var { version } = require2("../package.json");
 async function handleCommand(fn, logger, json) {
   try {

package/dist/index.js

@@ -1,5 +1,5 @@
 // src/index.ts
-import { createRequire } from "module";
+import { createRequire as createRequire2 } from "module";
 import { Command } from "commander";
 
 // src/commands/init.ts
@@ -60,7 +60,7 @@ function resolveCommandTimeoutMs(explicit) {
   const raw = process.env.LESSONKIT_CMD_TIMEOUT_MS;
   if (raw === void 0 || raw === "") return DEFAULT_CMD_TIMEOUT_MS;
   const parsed = Number(raw);
-  if (!Number.isFinite(parsed) || parsed <= 0) return void 0;
+  if (!Number.isFinite(parsed) || parsed <= 0) return DEFAULT_CMD_TIMEOUT_MS;
   return parsed;
 }
 async function runCommand(command, args, opts) {
@@ -248,6 +248,7 @@ async function runInit(opts, logger) {
 // src/lib/project.ts
 import { readFileSync, existsSync as existsSync2 } from "fs";
 import { readFile as readFile2 } from "fs/promises";
+import { createRequire } from "module";
 import { dirname as dirname2, join as join2, parse, resolve as resolve2 } from "path";
 import { parseLessonkitManifest } from "@lessonkit/lxpack";
 var LESSONKIT_JSON = "lessonkit.json";
@@ -370,14 +371,22 @@ function assertViteProject(pkg, projectRoot) {
     );
   }
 }
+function projectDeclaresVite(pkg) {
+  return !!(pkg.devDependencies?.vite ?? pkg.dependencies?.vite);
+}
 function resolveViteJs(projectRoot) {
-  let dir = resolve2(projectRoot);
-  const fsRoot = parse(dir).root;
-  while (true) {
-    const viteJs = join2(dir, "node_modules", "vite", "bin", "vite.js");
-    if (existsSync2(viteJs)) return viteJs;
-    if (dir === fsRoot) break;
-    dir = dirname2(dir);
+  const root = resolve2(projectRoot);
+  const localViteJs = join2(root, "node_modules", "vite", "bin", "vite.js");
+  if (existsSync2(localViteJs)) return localViteJs;
+  try {
+    const pkg = JSON.parse(readFileSync(join2(root, PACKAGE_JSON), "utf8"));
+    if (projectDeclaresVite(pkg)) {
+      const req = createRequire(join2(root, "package.json"));
+      const vitePkg = req.resolve("vite/package.json");
+      const hoistedViteJs = join2(dirname2(vitePkg), "bin", "vite.js");
+      if (existsSync2(hoistedViteJs)) return hoistedViteJs;
+    }
+  } catch {
   }
   throw new CliError(
     `Vite not found near ${projectRoot}. Run npm install in the project first.`,
@@ -455,7 +464,10 @@ async function runDev(opts) {
   const pkg = await readPackageJson(project.root);
   assertViteProject(pkg, project.root);
   const viteJs = resolveViteJs(project.root);
-  await runCommand(process.execPath, [viteJs, ...opts.viteArgs ?? []], { cwd: project.root });
+  await runCommand(process.execPath, [viteJs, ...opts.viteArgs ?? []], {
+    cwd: project.root,
+    timeoutMs: 0
+  });
   return { ok: true, command: "dev", projectRoot: project.root };
 }
 async function runBuild(opts) {
@@ -539,6 +551,17 @@ async function runPackage(opts) {
       issues: result.issues
     });
   }
+  const warnings = result.validation?.issues?.filter((issue) => {
+    const severity = issue.severity?.toLowerCase();
+    return severity !== "error" && severity !== "fatal";
+  });
+  if (warnings?.length && !opts.json) {
+    for (const issue of warnings) {
+      const location = issue.path ?? "course";
+      process.stderr.write(`[lessonkit] packaging warning: ${location}: ${issue.message}
+`);
+    }
+  }
   return {
     ok: true,
     command: "package",
@@ -546,7 +569,8 @@ async function runPackage(opts) {
     projectRoot: project.root,
     outputPath: result.outputPath,
     outputDir: result.outputDir,
-    fileCount: result.fileCount
+    fileCount: result.fileCount,
+    ...warnings?.length ? { warnings } : {}
   };
 }
 
@@ -564,7 +588,7 @@ function createLogger(opts) {
 }
 
 // src/index.ts
-var require2 = createRequire(import.meta.url);
+var require2 = createRequire2(import.meta.url);
 var { version } = require2("../package.json");
 async function handleCommand(fn, logger, json) {
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lessonkit/cli",
-  "version": "1.2.0",
+  "version": "1.3.1",
   "private": false,
   "description": "LessonKit CLI — init, dev, build, and package learning experiences.",
   "license": "Apache-2.0",
@@ -42,8 +42,8 @@
     "lint": "echo \"(no lint configured yet)\""
   },
   "dependencies": {
-    "@lessonkit/core": "1.2.0",
-    "@lessonkit/lxpack": "1.2.0",
+    "@lessonkit/core": "1.3.1",
+    "@lessonkit/lxpack": "1.3.1",
     "commander": "^14.0.1"
   },
   "engines": {

package/template/vite-react/package.json

@@ -13,16 +13,16 @@
     "test:coverage": "vitest run --coverage --passWithNoTests=false"
   },
   "dependencies": {
-    "@lessonkit/core": "^1.2.0",
-    "@lessonkit/react": "^1.2.0",
-    "@lessonkit/themes": "^1.2.0",
-    "@lessonkit/xapi": "^1.2.0",
+    "@lessonkit/core": "^1.3.1",
+    "@lessonkit/react": "^1.3.1",
+    "@lessonkit/themes": "^1.3.1",
+    "@lessonkit/xapi": "^1.3.1",
     "react": "^18.3.1",
     "react-dom": "^18.3.1"
   },
   "devDependencies": {
-    "@lessonkit/cli": "^1.2.0",
-    "@lessonkit/lxpack": "^1.2.0",
+    "@lessonkit/cli": "^1.3.1",
+    "@lessonkit/lxpack": "^1.3.1",
     "@testing-library/react": "^16.3.0",
     "@types/react": "^18.3.23",
     "@types/react-dom": "^18.3.7",
@@ -30,6 +30,6 @@
     "jsdom": "^26.1.0",
     "typescript": "^5.8.3",
     "vite": "^7.1.3",
-    "vitest": "^3.2.4"
+    "vitest": "^4.1.8"
   }
 }