@goodfoot/wiki 0.5.63 → 0.5.66

package/bin/wiki.exe

@@ -0,0 +1,6 @@
+@goodfoot/wiki placeholder. scripts/postinstall.js replaces this file
+with the native binary for your platform. The name ends in .exe and the
+first line intentionally has NO "#!" shebang so npm/yarn/pnpm cmd-shim
+generates a direct-exec wrapper that runs on Windows; Unix ignores the
+.exe suffix and execs by mode bit. If you are seeing this text, the
+postinstall step did not run.

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@goodfoot/wiki",
-  "version": "0.5.63",
-  "bin": "bin/wiki",
+  "version": "0.5.66",
+  "bin": "bin/wiki.exe",
   "scripts": {
     "postinstall": "node scripts/postinstall.js",
     "build": "env PATH=\"$HOME/.cargo/bin:$PATH\" CARGO_BUILD_JOBS=1 CARGO_TARGET_DIR=target/build cargo build --release",
@@ -12,15 +12,15 @@
     "typecheck": "env PATH=\"$HOME/.cargo/bin:$PATH\" CARGO_BUILD_JOBS=1 CARGO_TARGET_DIR=target/typecheck cargo check --quiet"
   },
   "files": [
-    "bin/",
+    "bin/wiki.exe",
     "scripts/postinstall.js"
   ],
   "optionalDependencies": {
-    "@goodfoot/wiki-darwin-arm64": "0.5.63",
-    "@goodfoot/wiki-darwin-x64": "0.5.63",
-    "@goodfoot/wiki-linux-arm64": "0.5.63",
-    "@goodfoot/wiki-linux-x64": "0.5.63",
-    "@goodfoot/wiki-win32-x64": "0.5.63"
+    "@goodfoot/wiki-darwin-arm64": "0.5.66",
+    "@goodfoot/wiki-darwin-x64": "0.5.66",
+    "@goodfoot/wiki-linux-arm64": "0.5.66",
+    "@goodfoot/wiki-linux-x64": "0.5.66",
+    "@goodfoot/wiki-win32-x64": "0.5.66"
   },
   "repository": {
     "type": "git",

package/scripts/postinstall.js

@@ -2,6 +2,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const { spawnSync } = require('child_process');
 
 const PLATFORM_MAP = {
   linux: 'linux',
@@ -14,15 +15,74 @@ const ARCH_MAP = {
   arm64: 'arm64'
 };
 
+function fail(message, error) {
+  console.error(message);
+  if (error) {
+    console.error(error.message);
+  }
+  process.exit(1);
+}
+
+function buildFromSource(destBinary, binaryName) {
+  const cargoToml = path.join(__dirname, '..', 'Cargo.toml');
+  if (!fs.existsSync(cargoToml)) {
+    fail(`@goodfoot/wiki: Binary not found at ${destBinary} and no Cargo.toml available to build from source.`);
+  }
+
+  console.log(`@goodfoot/wiki: Prebuilt binary missing; building from source via cargo...`);
+
+  const targetDir = path.join(__dirname, '..', 'target-cache', 'build');
+  const result = spawnSync('cargo', ['build', '--release', '--manifest-path', cargoToml], {
+    stdio: 'inherit',
+    env: { ...process.env, CARGO_BUILD_JOBS: '1', CARGO_TARGET_DIR: targetDir }
+  });
+
+  if (result.error || result.status !== 0) {
+    fail(
+      `@goodfoot/wiki: Failed to build binary from source. Install Rust/cargo or publish the platform package.`,
+      result.error
+    );
+  }
+
+  const builtBinary = path.join(targetDir, 'release', binaryName);
+  if (!fs.existsSync(builtBinary)) {
+    fail(`@goodfoot/wiki: cargo build succeeded but binary not found at ${builtBinary}.`);
+  }
+
+  fs.mkdirSync(path.dirname(destBinary), { recursive: true });
+  fs.copyFileSync(builtBinary, destBinary);
+  fs.chmodSync(destBinary, 0o755);
+}
+
+function isInsideSourceTree() {
+  // Walk up from this script looking for a `.git` entry. If found, this is
+  // the @goodfoot/wiki source repo (developer doing `yarn install`), not a
+  // consumer install under node_modules. Mutating the tracked `bin/wiki`
+  // shim in that case dirties the working tree and — if committed — ships a
+  // dangling symlink (or a leaked absolute build path) in the published
+  // tarball.
+  let dir = path.resolve(__dirname, '..');
+  for (let i = 0; i < 16; i++) {
+    if (fs.existsSync(path.join(dir, '.git'))) return true;
+    const parent = path.dirname(dir);
+    if (parent === dir) return false;
+    if (path.basename(parent) === 'node_modules') return false;
+    dir = parent;
+  }
+  return false;
+}
+
 function main() {
+  if (isInsideSourceTree()) {
+    console.log('@goodfoot/wiki: postinstall skipped (running inside source tree).');
+    return;
+  }
+
   const platform = PLATFORM_MAP[process.platform];
   const arch = ARCH_MAP[process.arch];
 
   if (!platform || !arch) {
-    console.log(
-      `@goodfoot/wiki: No prebuilt binary available for ${process.platform}-${process.arch}. Build from source with "cargo build --release".`
-    );
-    process.exit(0);
+    fail(`@goodfoot/wiki: No prebuilt binary available for ${process.platform}-${process.arch}.`);
   }
 
   const packageName = `@goodfoot/wiki-${platform}-${arch}`;
@@ -31,31 +91,51 @@ function main() {
   let packageDir;
   try {
     packageDir = path.dirname(require.resolve(`${packageName}/package.json`));
-  } catch {
-    // Platform package not installed -- user may have built from source
-    console.log(`@goodfoot/wiki: Optional package ${packageName} not found. Skipping binary setup.`);
-    process.exit(0);
+  } catch (error) {
+    fail(`@goodfoot/wiki: Required platform package ${packageName} not found.`, error);
   }
 
   const sourceBinary = path.join(packageDir, 'bin', sourceBinaryName);
   if (!fs.existsSync(sourceBinary)) {
-    console.log(`@goodfoot/wiki: Binary not found in ${packageName}. The package may not have been published yet.`);
-    process.exit(0);
+    buildFromSource(sourceBinary, sourceBinaryName);
   }
 
-  const binWiki = path.join(__dirname, '..', 'bin', 'wiki');
+  // Always write to bin/wiki.exe — the package.json `bin` field points
+  // here on every platform. The `.exe` extension plus a no-shebang stub makes
+  // npm's cmd-shim (generated at install time, before this postinstall) emit
+  // a direct exec on Windows; Unix executes by mode bit + ELF/Mach-O header
+  // and ignores the `.exe` suffix entirely. No resident Node process — the
+  // `wiki` command execs the native binary directly. Same pattern as
+  // Bun's and Claude Code's npm packages.
+  const binWiki = path.join(__dirname, '..', 'bin', 'wiki.exe');
 
+  // Hardlink first (instant, zero extra disk for the binary; src and dest are
+  // both under node_modules so same-filesystem is the common case), then fall
+  // back to copy across devices / on link-permission errors.
   try {
-    fs.unlinkSync(binWiki);
-  } catch {
-    // Ignore if it doesn't exist
+    fs.linkSync(sourceBinary, binWiki);
+  } catch (linkError) {
+    if (linkError.code === 'EEXIST') {
+      try {
+        fs.unlinkSync(binWiki);
+        fs.linkSync(sourceBinary, binWiki);
+      } catch {
+        fs.copyFileSync(sourceBinary, binWiki);
+      }
+    } else if (linkError.code === 'EXDEV' || linkError.code === 'EPERM') {
+      fs.copyFileSync(sourceBinary, binWiki);
+    } else {
+      try {
+        fs.copyFileSync(sourceBinary, binWiki);
+      } catch (copyError) {
+        fail(
+          `@goodfoot/wiki: Could not install binary from ${sourceBinary} to ${binWiki}.`,
+          new Error(`link failed: ${linkError.message}\ncopy failed: ${copyError.message}`)
+        );
+      }
+    }
   }
-
-  // Try symlink first, fall back to copy
-  try {
-    fs.symlinkSync(sourceBinary, binWiki);
-  } catch {
-    fs.copyFileSync(sourceBinary, binWiki);
+  if (process.platform !== 'win32') {
     fs.chmodSync(binWiki, 0o755);
   }