ferst-core 0.5.1 → 0.5.2

package/bin/check-thin.mjs

@@ -6,10 +6,25 @@
 // only content DATA, pages, the content config re-export, and a `custom/`
 // override sandbox. Shipped from core (its own single source) and run in each
 // client's CI/build: `ferst-check-thin [srcDir]` (defaults to ./src).
-import { readdirSync, statSync, existsSync } from 'node:fs';
+import { readdirSync, statSync, existsSync, realpathSync } from 'node:fs';
 import { join, relative } from 'node:path';
 import { pathToFileURL } from 'node:url';
 
+/**
+ * True when run as a CLI (not imported). Handles npm's `node_modules/.bin` symlink,
+ * where `process.argv[1]` is the symlink path but `import.meta.url` is the realpath —
+ * a plain `===` check silently skips `main()`, making this guard a false pass.
+ */
+export function isDirectInvocation(argv1) {
+  if (!argv1) return false;
+  if (import.meta.url === pathToFileURL(argv1).href) return true;
+  try {
+    return import.meta.url === pathToFileURL(realpathSync(argv1)).href;
+  } catch {
+    return false;
+  }
+}
+
 // Directories directly under src/ that belong to the core, not the client.
 const CORE_OWNED_DIRS = ['components', 'layouts', 'lib', 'utils', 'styles'];
 // Files under src/content/ that are the core's contract, not client data.
@@ -69,6 +84,6 @@ function main() {
 }
 
 // Run only when invoked directly as a bin (not when imported by a test).
-if (process.argv[1] && import.meta.url === pathToFileURL(process.argv[1]).href) {
+if (isDirectInvocation(process.argv[1])) {
   main();
 }

package/bin/cms-config.mjs

@@ -14,11 +14,27 @@
 //
 // Output path: dist/admin/config.yml under the CWD (the client repo root at build
 // time). Override with a first CLI arg.
-import { readFileSync, writeFileSync, mkdirSync } from 'node:fs';
+import { readFileSync, writeFileSync, mkdirSync, realpathSync } from 'node:fs';
 import { execSync } from 'node:child_process';
 import { dirname, resolve } from 'node:path';
 import { fileURLToPath, pathToFileURL } from 'node:url';
 
+/**
+ * True when this file is being run as a CLI (not imported). Handles npm's
+ * `node_modules/.bin` **symlink**: there `process.argv[1]` is the symlink path but
+ * `import.meta.url` is the realpath, so a plain `===` check silently skips `main()`
+ * and the config is never generated — the bug this guards against.
+ */
+export function isDirectInvocation(argv1) {
+  if (!argv1) return false;
+  if (import.meta.url === pathToFileURL(argv1).href) return true;
+  try {
+    return import.meta.url === pathToFileURL(realpathSync(argv1)).href;
+  } catch {
+    return false;
+  }
+}
+
 const HERE = dirname(fileURLToPath(import.meta.url));
 export const TEMPLATE_PATH = resolve(HERE, '../cms/config.template.yml');
 
@@ -132,7 +148,7 @@ function main() {
   console.log(`ferst-cms-config: wrote ${outPath} (repo=${repo}, branch="${branch}", base=${base})`);
 }
 
-// Run only when invoked directly as a bin (not when imported by a test).
-if (process.argv[1] && import.meta.url === pathToFileURL(process.argv[1]).href) {
+// Run when invoked directly as a bin — including via npm's node_modules/.bin symlink.
+if (isDirectInvocation(process.argv[1])) {
   main();
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ferst-core",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "type": "module",
   "description": "Ferst Core — the shared, brand-agnostic client-site system: Astro components, layouts, content schemas, the layered config resolver, and a neutral styling architecture that every client overrides.",
   "license": "BUSL-1.1",