doc-detective 4.37.0 → 4.37.1

package/bin/runner-entrypoint.js

@@ -35,9 +35,11 @@
  */
 
 import { spawn } from 'node:child_process';
+import { realpathSync } from 'node:fs';
 import { mkdir, rm, writeFile } from 'node:fs/promises';
 import path from 'node:path';
 import process from 'node:process';
+import { pathToFileURL } from 'node:url';
 
 const WORKSPACE_DIR = '/workspace';
 const SPECS_SUBDIR = 'specs';
@@ -767,10 +769,29 @@ async function main() {
 }
 
 // Module-import guard so the test file can `import` from this module
-// without triggering main(). Node's `import.meta.url === ...` idiom
-// for "am I the entrypoint?" — works under both `node` and `tsx` and
-// doesn't require require.main.
-const isEntry = import.meta.url === `file://${process.argv[1]}`;
+// without triggering main().
+//
+// Both halves of this comparison matter. npm installs `bin` entries as
+// symlinks, so when the container runs `doc-detective-runner`,
+// process.argv[1] is /usr/local/bin/doc-detective-runner while
+// import.meta.url is the realpath under node_modules — hence realpathSync
+// before comparing. And a hand-built `file://${...}` template mangles
+// Windows paths and anything containing a space or `#`, so the conversion
+// goes through pathToFileURL. Getting either wrong makes this silently
+// false: main() never runs, nothing is logged, and the process exits 0
+// looking like a clean success. See ADR 01087.
+const isEntry = (() => {
+	const argv1 = process.argv[1];
+	if (!argv1) return false;
+	try {
+		return import.meta.url === pathToFileURL(realpathSync(argv1)).href;
+	} catch {
+		// argv[1] can be unresolvable (deleted file, permission error). Not
+		// being the entrypoint is the safe answer — worst case the process
+		// no-ops, exactly as it would have before.
+		return false;
+	}
+})();
 if (isEntry) {
 	main()
 		.then((code) => process.exit(code))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "doc-detective",
-  "version": "4.37.0",
+  "version": "4.37.1",
   "description": "Treat doc content as testable assertions to validate doc accuracy and product UX.",
   "bin": {
     "doc-detective": "bin/doc-detective.js",