@nickderobertis/allowlister-remote-plugin 0.5.0 → 0.6.1

package/install.mjs

@@ -1,4 +1,4 @@
-// Post-install step: link the native binary directly onto the command path.
+// Post-install step: link the native binaries directly onto the command path.
 //
 // npm has already installed the one platform package that matches this host
 // (the others are skipped by their `os`/`cpu` fields). On macOS and Linux we
@@ -8,6 +8,10 @@
 // spawned per invocation, which matters because allowlister can call the plugin
 // hundreds of times in a single agent session.
 //
+// We also drop the `allowlister-remote-daemon` binary next to it in the same
+// `bin/` directory so the plugin's sibling lookup (`resolve_daemon_bin`) finds
+// the daemon it auto-starts without it having to be separately on PATH.
+//
 // On Windows npm generates `.cmd`/`.ps1` shims that invoke the launcher through
 // Node, so replacing the target in place would break them; we keep the JS
 // launcher there instead. Either way, if anything goes wrong we leave the
@@ -17,7 +21,7 @@ import { chmodSync, copyFileSync, existsSync, readFileSync, realpathSync } from
 import { createRequire } from "node:module";
 import { basename, dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
-import { binarySpecifier } from "./lib/platform.mjs";
+import { binarySpecifier, daemonSpecifier } from "./lib/platform.mjs";
 
 const require = createRequire(import.meta.url);
 // Resolve symlinks so a workspace link does not make us look like an install.
@@ -65,8 +69,13 @@ if (inWorkspaceCheckout(here)) {
       const onPath = join(here, "bin", "allowlister-remote-plugin");
       copyFileSync(native, onPath);
       chmodSync(onPath, 0o755);
+      // Place the daemon next to the plugin so its sibling lookup succeeds.
+      const daemon = require.resolve(daemonSpecifier());
+      const daemonOnPath = join(here, "bin", "allowlister-remote-daemon");
+      copyFileSync(daemon, daemonOnPath);
+      chmodSync(daemonOnPath, 0o755);
       console.log(
-        "allowlister-remote-plugin: linked the native binary directly onto the command path",
+        "allowlister-remote-plugin: linked the native plugin and daemon binaries directly onto the command path",
       );
     }
   } catch (error) {

package/lib/platform.mjs

@@ -1,40 +1,42 @@
 // Shared resolution for the per-platform native binary packages.
 //
-// Each supported platform ships the native `allowlister-remote-plugin` binary in
-// its own npm package (declared as an optional dependency of this package). npm
-// installs only the package whose `os`/`cpu` match the host, so resolving the
-// binary is a matter of mapping the running platform/arch onto that package and
-// asking Node where it landed in `node_modules`.
+// Each supported platform ships two native binaries in its own npm package
+// (declared as an optional dependency of this package): the
+// `allowlister-remote-plugin` binary and the `allowlister-remote-daemon` it
+// auto-starts. npm installs only the package whose `os`/`cpu` match the host, so
+// resolving a binary is a matter of mapping the running platform/arch onto that
+// package and asking Node where it landed in `node_modules`. The `.exe`
+// extension on Windows is appended per-binary via `suffix`.
 
 const PLATFORM_PACKAGES = new Map([
-  [
-    "darwin-arm64",
-    { pkg: "allowlister-remote-plugin-darwin-arm64", file: "allowlister-remote-plugin" },
-  ],
-  ["linux-x64", { pkg: "allowlister-remote-plugin-linux-x64", file: "allowlister-remote-plugin" }],
-  [
-    "win32-x64",
-    { pkg: "allowlister-remote-plugin-win32-x64", file: "allowlister-remote-plugin.exe" },
-  ],
+  ["darwin-arm64", { pkg: "allowlister-remote-plugin-darwin-arm64", suffix: "" }],
+  ["linux-x64", { pkg: "allowlister-remote-plugin-linux-x64", suffix: "" }],
+  ["win32-x64", { pkg: "allowlister-remote-plugin-win32-x64", suffix: ".exe" }],
 ]);
 
 const SCOPE = "@nickderobertis";
 
 /**
  * Describe the platform package for a given platform/arch, throwing for any
- * combination we do not publish a binary for.
+ * combination we do not publish a binary for. `file` is the plugin binary's
+ * name (with the platform's `.exe` suffix); `daemonFile` is the daemon's.
  */
 export function platformPackage(platform = process.platform, arch = process.arch) {
   const entry = PLATFORM_PACKAGES.get(`${platform}-${arch}`);
   if (!entry) {
     throw new Error(`Unsupported platform: ${platform}-${arch}`);
   }
-  return { name: `${SCOPE}/${entry.pkg}`, file: entry.file };
+  return {
+    name: `${SCOPE}/${entry.pkg}`,
+    file: `allowlister-remote-plugin${entry.suffix}`,
+    daemonFile: `allowlister-remote-daemon${entry.suffix}`,
+  };
 }
 
 /**
- * The bare specifier Node uses to locate the native binary inside the installed
- * platform package, e.g. `@nickderobertis/allowlister-remote-plugin-linux-x64/bin/allowlister-remote-plugin`.
+ * The bare specifier Node uses to locate the native plugin binary inside the
+ * installed platform package, e.g.
+ * `@nickderobertis/allowlister-remote-plugin-linux-x64/bin/allowlister-remote-plugin`.
  */
 export function binarySpecifier(platform = process.platform, arch = process.arch) {
   const { name, file } = platformPackage(platform, arch);
@@ -42,9 +44,19 @@ export function binarySpecifier(platform = process.platform, arch = process.arch
 }
 
 /**
- * Resolve the absolute path to the native binary using the caller's `require`
- * (created with `createRequire(import.meta.url)`). Throws if the matching
- * platform package is not installed.
+ * The bare specifier Node uses to locate the native daemon binary inside the
+ * installed platform package, e.g.
+ * `@nickderobertis/allowlister-remote-plugin-linux-x64/bin/allowlister-remote-daemon`.
+ */
+export function daemonSpecifier(platform = process.platform, arch = process.arch) {
+  const { name, daemonFile } = platformPackage(platform, arch);
+  return `${name}/bin/${daemonFile}`;
+}
+
+/**
+ * Resolve the absolute path to the native plugin binary using the caller's
+ * `require` (created with `createRequire(import.meta.url)`). Throws if the
+ * matching platform package is not installed.
  */
 export function resolveNativeBinary(require, platform = process.platform, arch = process.arch) {
   return require.resolve(binarySpecifier(platform, arch));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nickderobertis/allowlister-remote-plugin",
-  "version": "0.5.0",
+  "version": "0.6.1",
   "description": "npm installer for the allowlister remote dynamic approval plugin",
   "type": "module",
   "license": "UNLICENSED",
@@ -28,8 +28,8 @@
     "node": ">=20"
   },
   "optionalDependencies": {
-    "@nickderobertis/allowlister-remote-plugin-darwin-arm64": "0.5.0",
-    "@nickderobertis/allowlister-remote-plugin-linux-x64": "0.5.0",
-    "@nickderobertis/allowlister-remote-plugin-win32-x64": "0.5.0"
+    "@nickderobertis/allowlister-remote-plugin-darwin-arm64": "0.6.1",
+    "@nickderobertis/allowlister-remote-plugin-linux-x64": "0.6.1",
+    "@nickderobertis/allowlister-remote-plugin-win32-x64": "0.6.1"
   }
 }