@techninja/clearstack 0.4.34 → 0.4.36

package/lib/check.js

@@ -6,13 +6,13 @@
 
 import { existsSync } from 'node:fs';
 import { resolve } from 'node:path';
-import { runCmd } from './spec-utils.js';
+import { runCmd, elapsed, normalise, printResult } from './spec-utils.js';
 import { countFiles, checkFileLines, checkImports } from './spec-scan.js';
 import { checkI18n } from './spec-i18n.js';
 import { loadConfig, buildCmds, detectRunner } from './spec-config.js';
 import { findTypeConfigs } from './spec-types.js';
 
-export { runCmd, runExtCmd, elapsed } from './spec-utils.js';
+export { runCmd, runExtCmd, elapsed, normalise, printResult } from './spec-utils.js';
 export { findFiles, countFiles, checkFileLines, checkImports } from './spec-scan.js';
 export { checkI18n } from './spec-i18n.js';
 export { loadConfig, buildCmds, detectRunner } from './spec-config.js';
@@ -20,7 +20,7 @@ export { loadConfig, buildCmds, detectRunner } from './spec-config.js';
 /** @typedef {{ key: string, name: string, parent?: string, watchExts?: string[], fix?: () => void, run: (opts?: object) => (boolean | import('./spec-utils.js').CheckResult | Promise<import('./spec-utils.js').CheckResult>) }} Check */
 
 /**
- *
+ * Load project extensions from clearstack.spec.js.
  */
 async function loadExtensions(projectDir) {
   const extPath = resolve(projectDir, 'clearstack.spec.js');
@@ -93,14 +93,28 @@ export async function check(projectDir, scope, opts) {
       console.log(`Available: ${[...tops, ...parentKeys(checks)].join(', ')}`);
       process.exit(1);
     }
-    const results = await Promise.all(matched.map((c) => c.run(opts)));
-    if (!results.every((r) => typeof r === 'boolean' ? r : r.pass)) process.exit(1);
+    const results = [];
+    for (const c of matched) {
+      const start = performance.now();
+      const raw = await c.run({ ...opts, quiet: true });
+      const r = normalise(raw, c.name, elapsed(start));
+      printResult(r);
+      results.push(r);
+    }
+    if (!results.every((r) => r.pass)) process.exit(1);
     return;
   }
   console.log('🔍 Clearstack compliance checking now... 💙\n');
   const runnable = checks.filter((c) => !c.keyBinding);
-  const results = await Promise.all(runnable.map((c) => c.run(opts)));
-  const passed = results.filter((r) => typeof r === 'boolean' ? r : r.pass).length;
+  const results = [];
+  for (const c of runnable) {
+    const start = performance.now();
+    const raw = await c.run({ ...opts, quiet: true });
+    const r = normalise(raw, c.name, elapsed(start));
+    printResult(r);
+    results.push(r);
+  }
+  const passed = results.filter((r) => r.pass).length;
   console.log(`\n${'='.repeat(40)}`);
   console.log(`${passed}/${results.length} checks passed.`);
   if (passed < results.length) process.exit(1);

package/lib/spec-utils-ext.js

@@ -0,0 +1,76 @@
+/**
+ * runExtCmd — shell command runner for clearstack.spec.js extensions.
+ * @module lib/spec-utils-ext
+ */
+
+import { execSync, spawn } from 'node:child_process';
+import { elapsed } from './spec-utils.js';
+
+/**
+ * Run an arbitrary shell command for a custom extension check.
+ * Handles timing, output filtering, and consistent ✅/❌ printing.
+ * Ignores lines matching node_modules or any paths in `ignorePaths`.
+ * Use this in clearstack.spec.js extensions instead of execSync directly.
+ *
+ * @param {string} label - Display name for the check
+ * @param {string} cmd - Shell command to run
+ * @param {{ quiet?: boolean, cwd?: string, ignorePaths?: string[], onProgress?: (line: string) => void }} [opts]
+ * @returns {import('./spec-utils.js').CheckResult | Promise<import('./spec-utils.js').CheckResult>}
+ */
+export function runExtCmd(label, cmd, opts = {}) {
+  const start = performance.now();
+  const cwd = opts.cwd ?? process.cwd();
+  const suffix = () => ` (${elapsed(start)})`;
+  const ignoreRe = [
+    /[/\\]node_modules[/\\]/,
+    ...(opts.ignorePaths ?? []).map((p) => new RegExp(p.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'))),
+  ];
+  const filter = (l) => l.trim() && !ignoreRe.some((re) => re.test(l)) && (!opts.lineFilter || opts.lineFilter(l));
+
+  if (opts.onProgress) {
+    return new Promise((resolve) => {
+      const [bin, ...args] = cmd.split(/\s+(?=(?:[^"]*"[^"]*")*[^"]*$)/);
+      const proc = spawn(bin, args, { cwd, shell: true });
+      const lines = [];
+      const onLine = (chunk) => {
+        for (const l of chunk.toString().split('\n')) {
+          if (!filter(l)) continue;
+          lines.push(l);
+          opts.onProgress(l);
+        }
+      };
+      proc.stdout.on('data', onLine);
+      proc.stderr.on('data', onLine);
+      proc.on('close', (code) => {
+        if (code === 0 || lines.length === 0) {
+          if (!opts.quiet) console.log(`  ✅ ${label}${suffix()}`);
+          resolve({ pass: true, label, time: elapsed(start) });
+        } else {
+          if (!opts.quiet) {
+            console.log(`  ❌ ${label}${suffix()}`);
+            for (const l of lines) console.log(`     ${l}`);
+          }
+          resolve({ pass: false, label, time: elapsed(start), errors: lines });
+        }
+      });
+    });
+  }
+
+  try {
+    execSync(cmd, { cwd, stdio: 'pipe', encoding: 'utf-8' });
+    if (!opts.quiet) console.log(`  ✅ ${label}${suffix()}`);
+    return { pass: true, label, time: elapsed(start) };
+  } catch (err) {
+    const out = ((err.stdout || '') + (err.stderr || '')).trim();
+    const errors = out.split('\n').filter(filter);
+    if (errors.length === 0) {
+      if (!opts.quiet) console.log(`  ✅ ${label}${suffix()}`);
+      return { pass: true, label, time: elapsed(start) };
+    }
+    if (!opts.quiet) {
+      console.log(`  ❌ ${label}${suffix()}`);
+      for (const line of errors) console.log(`     ${line}`);
+    }
+    return { pass: false, label, time: elapsed(start), errors };
+  }
+}

package/lib/spec-utils.js

@@ -1,10 +1,13 @@
 /**
- * Shared spec utilities — timing and command running.
+ * Shared spec utilities — timing, result normalisation, and command running.
+ * runExtCmd (for clearstack.spec.js extensions) lives in spec-utils-ext.js.
  * File scanning lives in spec-scan.js.
  * @module lib/spec-utils
  */
 
-import { execSync, spawn } from 'node:child_process';
+import { execSync } from 'node:child_process';
+
+export { runExtCmd } from './spec-utils-ext.js';
 
 /** @param {number} start */
 export function elapsed(start) {
@@ -14,6 +17,40 @@ export function elapsed(start) {
 
 /** @typedef {{ pass: boolean, label: string, time: string, detail?: string, files?: number, errors?: string[], violations?: object[] }} CheckResult */
 
+/**
+ * Normalise any check return value to a CheckResult.
+ * @param {boolean | object} r
+ * @param {string} name
+ * @param {string} time
+ * @returns {CheckResult}
+ */
+export function normalise(r, name, time) {
+  if (typeof r === 'boolean') return { pass: r, label: name, time };
+  return { label: name, time, ...r };
+}
+
+/**
+ * Print a single CheckResult line (✅/❌ + errors/violations).
+ * @param {CheckResult} r
+ */
+export function printResult(r) {
+  const detail = r.detail ? ` — ${r.detail}` : '';
+  const files = r.files ? ` (${r.files} files, ${r.time})` : ` (${r.time})`;
+  if (r.pass) {
+    console.log(`  ✅ ${r.label}${r.detail ? detail + ` (${r.time})` : files}`);
+  } else {
+    const suffix = r.violations?.length
+      ? ` — ${r.violations.length} violation(s):`
+      : r.errors?.length
+        ? ` — ${r.errors.length} error(s):`
+        : '';
+    console.log(`  ❌ ${r.label}${suffix} (${r.time})`);
+    for (const v of r.violations ?? []) console.log(`     ${v.file ?? v}: ${v.lines} lines (max ${v.max})`);
+    for (const e of r.errors ?? []) console.log(`     ${e}`);
+    if (!r.violations?.length && !r.errors?.length) console.log(`     (no details)`);
+  }
+}
+
 /**
  * Run a shell command, report pass/fail with timing.
  * @param {string} label
@@ -49,73 +86,3 @@ export function runCmd(label, cmd, cwd, stats, opts) {
     return { pass: false, label, time: elapsed(start), errors: display };
   }
 }
-
-/**
- * Run an arbitrary shell command for a custom extension check.
- * Handles timing, output filtering, and consistent ✅/❌ printing.
- * Ignores lines matching node_modules or any paths in `ignorePaths`.
- * Use this in clearstack.spec.js extensions instead of execSync directly.
- *
- * @param {string} label - Display name for the check
- * @param {string} cmd - Shell command to run
- * @param {{ quiet?: boolean, cwd?: string, ignorePaths?: string[] }} [opts]
- * @returns {CheckResult}
- */
-export function runExtCmd(label, cmd, opts = {}) {
-  const start = performance.now();
-  const cwd = opts.cwd ?? process.cwd();
-  const suffix = () => ` (${elapsed(start)})`;
-  const ignoreRe = [
-    /[/\\]node_modules[/\\]/,
-    ...(opts.ignorePaths ?? []).map((p) => new RegExp(p.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'))),
-  ];
-  const filter = (l) => l.trim() && !ignoreRe.some((re) => re.test(l)) && (!opts.lineFilter || opts.lineFilter(l));
-
-  // Streaming mode — used when onProgress is provided
-  if (opts.onProgress) {
-    return new Promise((resolve) => {
-      const [bin, ...args] = cmd.split(/\s+(?=(?:[^"]*"[^"]*")*[^"]*$)/);
-      const proc = spawn(bin, args, { cwd, shell: true });
-      const lines = [];
-      const onLine = (chunk) => {
-        for (const l of chunk.toString().split('\n')) {
-          if (!filter(l)) continue;
-          lines.push(l);
-          opts.onProgress(l);
-        }
-      };
-      proc.stdout.on('data', onLine);
-      proc.stderr.on('data', onLine);
-      proc.on('close', (code) => {
-        if (code === 0 || lines.length === 0) {
-          if (!opts.quiet) console.log(`  ✅ ${label}${suffix()}`);
-          resolve({ pass: true, label, time: elapsed(start) });
-        } else {
-          if (!opts.quiet) {
-            console.log(`  ❌ ${label}${suffix()}`);
-            for (const l of lines) console.log(`     ${l}`);
-          }
-          resolve({ pass: false, label, time: elapsed(start), errors: lines });
-        }
-      });
-    });
-  }
-
-  try {
-    execSync(cmd, { cwd, stdio: 'pipe', encoding: 'utf-8' });
-    if (!opts.quiet) console.log(`  ✅ ${label}${suffix()}`);
-    return { pass: true, label, time: elapsed(start) };
-  } catch (err) {
-    const out = ((err.stdout || '') + (err.stderr || '')).trim();
-    const errors = out.split('\n').filter(filter);
-    if (errors.length === 0) {
-      if (!opts.quiet) console.log(`  ✅ ${label}${suffix()}`);
-      return { pass: true, label, time: elapsed(start) };
-    }
-    if (!opts.quiet) {
-      console.log(`  ❌ ${label}${suffix()}`);
-      for (const line of errors) console.log(`     ${line}`);
-    }
-    return { pass: false, label, time: elapsed(start), errors };
-  }
-}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@techninja/clearstack",
-  "version": "0.4.34",
+  "version": "0.4.36",
   "type": "module",
   "description": "A no-build web component framework specification — scaffold, validate, and evolve spec-compliant projects",
   "bin": {
@@ -54,6 +54,9 @@
     "hybrids"
   ],
   "license": "MIT",
+  "overrides": {
+    "smol-toml": "^1.8.0"
+  },
   "dependencies": {
     "@inquirer/prompts": "^8.3.2",
     "blessed": "^0.1.81",
@@ -62,7 +65,7 @@
   "devDependencies": {
     "@open-wc/testing": "^4.0.0",
     "@types/ws": "^8.18.1",
-    "@web/test-runner": "^0.20.0",
+    "@web/test-runner": "^1.0.0",
     "@web/test-runner-playwright": "^0.11.0",
     "eslint": "^10.1.0",
     "eslint-config-prettier": "^10.1.8",