Merge pull request #30 from tpsdev-ai/fix-install-ux

Install UX cleanup: postinstall + --version + CJS wrapper

Author: heskew

Dockerfile.test

@@ -24,6 +24,7 @@ COPY package.json bun.lock* tsconfig*.json ./
 
 # Copy package manifests first for layer caching
 COPY packages/cli/package.json packages/cli/tsconfig.json ./packages/cli/
+COPY packages/cli/scripts/ ./packages/cli/scripts/
 COPY packages/agent/package.json packages/agent/tsconfig.json ./packages/agent/
 
 RUN bun install --frozen-lockfile 2>/dev/null || bun install

bun.lock

@@ -7,6 +7,14 @@ const platform = process.platform;
 const arch = process.arch;
 const pkg = `@tpsdev-ai/cli-${platform}-${arch}`;
 
+if (process.argv.includes('--version') || process.argv.includes('-v')) {
+  // Fast-path version output even when native binary package is missing.
+  // eslint-disable-next-line @typescript-eslint/no-var-requires
+  const version = require('../package.json').version;
+  console.log(version);
+  process.exit(0);
+}
+
 // Search for the platform binary relative to this package, not the cwd.
 // npm nests optionalDependencies inside the parent's node_modules.
 const searchPaths = [path.join(__dirname, '..'), path.join(__dirname, '..', '..')];
@@ -21,10 +29,19 @@ function runBinary() {
     // Fall through to error message
   }
 
+  // Fallback for source/dev installs where dist JS exists.
+  try {
+    const jsCli = path.join(__dirname, '..', 'dist', 'bin', 'tps.js');
+    execFileSync(process.execPath, [jsCli, ...process.argv.slice(2)], { stdio: 'inherit' });
+    return;
+  } catch (_) {
+    // continue to error output
+  }
+
   console.error(`Failed to load native binding`);
   console.error(`TPS: no binary package available for ${platform}-${arch}.`);
-  console.error(`Run npm install -g ${pkg} to install the platform binary package.`);
-  console.error('Or run from source inside the repository via `bun run tps` in packages/cli.');
+  console.error(`Try reinstalling main package: npm install -g @tpsdev-ai/cli@${require('../package.json').version}`);
+  console.error(`Or install platform binary directly: npm install -g ${pkg}@${require('../package.json').version}`);
   process.exitCode = 1;
 }
 

packages/cli/bin/tps.cjs

@@ -122,6 +122,16 @@ async function checkNono() {
 }
 
 async function main() {
+  if (process.argv.includes("--version") || process.argv.includes("-v")) {
+    const { readFileSync } = await import("node:fs");
+    const { dirname, join } = await import("node:path");
+    const { fileURLToPath } = await import("node:url");
+    const here = dirname(fileURLToPath(import.meta.url));
+    const pkg = JSON.parse(readFileSync(join(here, "..", "package.json"), "utf-8"));
+    console.log(pkg.version);
+    return;
+  }
+
   await checkNono();
   switch (command) {
     case "hire": {

packages/cli/bin/tps.ts

@@ -7,10 +7,10 @@
     "tps": "./bin/tps.cjs"
   },
   "optionalDependencies": {
-    "@tpsdev-ai/cli-darwin-arm64": "0.4.2",
-    "@tpsdev-ai/cli-darwin-x64": "0.4.2",
-    "@tpsdev-ai/cli-linux-arm64": "0.4.2",
-    "@tpsdev-ai/cli-linux-x64": "0.4.2"
+    "@tpsdev-ai/cli-darwin-arm64": "0.5.0",
+    "@tpsdev-ai/cli-darwin-x64": "0.5.0",
+    "@tpsdev-ai/cli-linux-arm64": "0.5.0",
+    "@tpsdev-ai/cli-linux-x64": "0.5.0"
   },
   "scripts": {
     "build": "tsc",
@@ -25,7 +25,8 @@
     "lint": "biome lint ./src ./bin",
     "lint:ci": "biome lint ./src ./bin --max-diagnostics=200",
     "pretest": "tsc",
-    "test": "bun test"
+    "test": "bun test",
+    "postinstall": "node scripts/postinstall.cjs"
   },
   "keywords": [
     "cli",

packages/cli/package.json

@@ -0,0 +1,11 @@
+if (process.env.CI) process.exit(0);
+const { execSync } = require("child_process");
+const { arch, platform } = process;
+const pkg = `@tpsdev-ai/cli-${platform}-${arch}`;
+const version = require("../package.json").version;
+
+try {
+  execSync(`npm install -g ${pkg}@${version}`, { stdio: "inherit" });
+} catch {
+  console.warn(`Optional: install ${pkg}@${version} for native binary performance.`);
+}

packages/cli/scripts/postinstall.cjs

@@ -0,0 +1,10 @@
+const { execSync } = require("child_process");
+const { arch, platform } = process;
+const pkg = `@tpsdev-ai/cli-${platform}-${arch}`;
+const version = require("../package.json").version;
+
+try {
+  execSync(`npm install -g ${pkg}@${version}`, { stdio: "inherit" });
+} catch {
+  console.warn(`Optional: install ${pkg}@${version} for native binary performance.`);
+}