@pks-cli/cli 7.0.0 → 7.0.2

package/bin/pks.js

@@ -42,19 +42,21 @@ function getBinaryPath() {
   // Binary may be at package root or in bin/ subdirectory
   const scopeDir = join(__dirname, '..', '..');           // node_modules/@pks-cli/
   const parentScopeDir = join(__dirname, '..', '..', '..', '@pks-cli'); // ../node_modules/@pks-cli/
-
-  const paths = [
-    // Scoped: node_modules/@pks-cli/cli-linux-x64/bin/pks
-    join(scopeDir, packageDir, 'bin', 'pks'),
-    join(scopeDir, packageDir, 'bin', 'pks.exe'),
-    join(scopeDir, packageDir, 'pks'),
-    join(scopeDir, packageDir, 'pks.exe'),
-    // Parent node_modules (hoisted)
-    join(parentScopeDir, packageDir, 'bin', 'pks'),
-    join(parentScopeDir, packageDir, 'bin', 'pks.exe'),
-    join(parentScopeDir, packageDir, 'pks'),
-    join(parentScopeDir, packageDir, 'pks.exe'),
-  ];
+  // Not hoisted: npm nests the optional dependency inside this package when the
+  // install has nowhere to hoist it to -- which is exactly what `npm i -g` does,
+  // so a global install found no binary at all until this root was added.
+  const ownScopeDir = join(__dirname, '..', 'node_modules', '@pks-cli');
+
+  const roots = [scopeDir, parentScopeDir, ownScopeDir];
+  const paths = [];
+  for (const root of roots) {
+    paths.push(
+      join(root, packageDir, 'bin', 'pks'),
+      join(root, packageDir, 'bin', 'pks.exe'),
+      join(root, packageDir, 'pks'),
+      join(root, packageDir, 'pks.exe'),
+    );
+  }
 
   for (const path of paths) {
     if (existsSync(path)) {
@@ -62,6 +64,25 @@ function getBinaryPath() {
     }
   }
 
+  // Last resort: let Node walk the node_modules chain itself, which covers layouts
+  // (pnpm, yarn PnP-less workspaces) that no hardcoded root above anticipates.
+  try {
+    const manifest = require.resolve(`${packageName}/package.json`, { paths: [__dirname] });
+    const packageRoot = join(manifest, '..');
+    for (const candidate of [
+      join(packageRoot, 'bin', 'pks'),
+      join(packageRoot, 'bin', 'pks.exe'),
+      join(packageRoot, 'pks'),
+      join(packageRoot, 'pks.exe'),
+    ]) {
+      if (existsSync(candidate)) {
+        return candidate;
+      }
+    }
+  } catch {
+    // Fall through to the error below -- resolution failing just means "not installed".
+  }
+
   console.error(`PKS CLI binary not found for ${platform}-${arch}`);
   console.error('Please ensure the platform-specific package is installed.');
   console.error(`Expected package: ${packageName}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pks-cli/cli",
-  "version": "7.0.0",
+  "version": "7.0.2",
   "description": "The next agentic CLI for .NET developers",
   "bin": {
     "pks": "./bin/pks.js"
@@ -9,12 +9,12 @@
     "postinstall": "node postinstall.js"
   },
   "optionalDependencies": {
-    "@pks-cli/cli-linux-x64": "7.0.0",
-    "@pks-cli/cli-linux-arm64": "7.0.0",
-    "@pks-cli/cli-osx-x64": "7.0.0",
-    "@pks-cli/cli-osx-arm64": "7.0.0",
-    "@pks-cli/cli-win-x64": "7.0.0",
-    "@pks-cli/cli-win-arm64": "7.0.0"
+    "@pks-cli/cli-linux-x64": "7.0.2",
+    "@pks-cli/cli-linux-arm64": "7.0.2",
+    "@pks-cli/cli-osx-x64": "7.0.2",
+    "@pks-cli/cli-osx-arm64": "7.0.2",
+    "@pks-cli/cli-win-x64": "7.0.2",
+    "@pks-cli/cli-win-arm64": "7.0.2"
   },
   "engines": {
     "node": ">=18.0.0"

package/postinstall.js

@@ -39,6 +39,8 @@ function checkPlatformPackage() {
     join(__dirname, '..', packageDir),
     // Parent node_modules (hoisted)
     join(__dirname, '..', '..', '@pks-cli', packageDir),
+    // Nested, i.e. not hoisted anywhere -- what `npm i -g` produces
+    join(__dirname, 'node_modules', '@pks-cli', packageDir),
   ];
 
   for (const path of paths) {
@@ -48,7 +50,15 @@ function checkPlatformPackage() {
     }
   }
 
-  return false;
+  // Same fallback as bin/pks.js: Node's own resolution covers layouts the
+  // hardcoded paths above do not.
+  try {
+    require.resolve(`${packageName}/package.json`, { paths: [__dirname] });
+    console.log(`✅ PKS CLI installed successfully for ${platform}-${arch}`);
+    return true;
+  } catch {
+    return false;
+  }
 }
 
 /**