@glossarist/concept-browser 0.7.45 → 0.7.47

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@glossarist/concept-browser",
-  "version": "0.7.45",
+  "version": "0.7.47",
   "description": "Vue SPA for browsing Glossarist terminology datasets with cross-reference resolution, graph visualization, and multi-language support",
   "type": "module",
   "bin": {
@@ -11,6 +11,9 @@
     "build": "vue-tsc --noEmit && vite build",
     "postbuild": "node scripts/generate-404.js",
     "preview": "vite preview",
+    "check:scripts": "node scripts/check-syntax.mjs",
+    "pretest": "node scripts/check-syntax.mjs",
+    "prebuild": "node scripts/check-syntax.mjs",
     "fetch-datasets": "node scripts/fetch-datasets.mjs",
     "generate-data": "node scripts/generate-data.mjs",
     "generate-ontology": "node scripts/generate-ontology-data.mjs && node scripts/generate-ontology-schema.mjs",

package/scripts/check-syntax.mjs

@@ -0,0 +1,78 @@
+#!/usr/bin/env node
+import { spawnSync } from "node:child_process";
+import { readdirSync, statSync } from "node:fs";
+import path from "node:path";
+import process from "node:process";
+
+const ROOT = path.resolve(path.dirname(new URL(import.meta.url).pathname), "..");
+
+const TARGET_DIRS = ["scripts", "cli"];
+const EXTENSIONS = new Set([".mjs", ".js", ".cjs"]);
+const SKIP_DIRS = new Set(["__tests__", "node_modules"]);
+
+function walk(dir) {
+  const out = [];
+  for (const entry of readdirSync(dir)) {
+    if (SKIP_DIRS.has(entry)) continue;
+    const full = path.join(dir, entry);
+    const st = statSync(full);
+    if (st.isDirectory()) {
+      out.push(...walk(full));
+    } else if (EXTENSIONS.has(path.extname(entry))) {
+      out.push(full);
+    }
+  }
+  return out;
+}
+
+export function checkAllScripts({ root = ROOT } = {}) {
+  const files = [];
+  for (const sub of TARGET_DIRS) {
+    const dir = path.join(root, sub);
+    try {
+      statSync(dir);
+    } catch {
+      continue;
+    }
+    files.push(...walk(dir));
+  }
+  files.sort();
+
+  const failures = [];
+  for (const file of files) {
+    const result = spawnSync(process.execPath, ["--check", file], {
+      encoding: "utf8",
+    });
+    if (result.status !== 0) {
+      failures.push({
+        file: path.relative(root, file),
+        stderr: result.stderr || "",
+      });
+    }
+  }
+
+  return { files, failures };
+}
+
+function main() {
+  const { files, failures } = checkAllScripts();
+  if (failures.length === 0) {
+    process.stdout.write(`syntax OK: ${files.length} file(s) checked\n`);
+    return;
+  }
+  process.stderr.write(
+    `syntax check failed: ${failures.length} of ${files.length} file(s)\n\n`,
+  );
+  for (const { file, stderr } of failures) {
+    process.stderr.write(`--- ${file} ---\n${stderr}\n`);
+  }
+  process.exit(1);
+}
+
+const isDirectEntry = process.argv[1] &&
+  path.resolve(process.argv[1]) ===
+    path.resolve(new URL(import.meta.url).pathname);
+
+if (isDirectEntry) {
+  main();
+}

package/scripts/generate-data.mjs

@@ -783,7 +783,7 @@ ${bodyEntries}
 `;
 }
 
-function processDataset(dir, register, opts) {
+async function processDataset(dir, register, opts) {
   const files = fs.readdirSync(dir).filter(f => f.endsWith('.yaml')).sort((a, b) => naturalSort(a.replace('.yaml', ''), b.replace('.yaml', '')));
 
   console.log(`Processing ${register}: ${files.length} files`);
@@ -1095,7 +1095,7 @@ for (let i = 0; i < config.datasets.length; i++) {
   // Resolve title: site-config override, then ref from register
   const resolvedTitle = ds.title || reg?.ref || ds.id;
 
-  counts[ds.id] = processDataset(dir, ds.id, {
+  counts[ds.id] = await processDataset(dir, ds.id, {
     title: resolvedTitle,
     description: resolvedDescription,
     owner: ds.owner || reg?.owner,

package/src/__tests__/scripts-syntax.test.ts

@@ -0,0 +1,28 @@
+import { describe, it, expect } from 'vitest';
+import { spawnSync } from 'node:child_process';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const repoRoot = path.resolve(
+  path.dirname(fileURLToPath(import.meta.url)),
+  '../..',
+);
+
+describe('scripts syntax gate', () => {
+  it('`npm run check:scripts` exits 0 (every .mjs/.js/.cjs in scripts/ and cli/ parses)', () => {
+    const result = spawnSync(
+      process.execPath,
+      [path.join(repoRoot, 'scripts', 'check-syntax.mjs')],
+      { encoding: 'utf8' },
+    );
+
+    if (result.status !== 0) {
+      throw new Error(
+        `check-syntax.mjs failed (exit ${result.status}):\n${result.stdout || ''}${result.stderr || ''}`,
+      );
+    }
+
+    expect(result.status).toBe(0);
+    expect(result.stdout).toMatch(/syntax OK:/);
+  });
+});