codebuff-cli 1.0.12 → 1.0.15

package/NOTICE

@@ -0,0 +1,4 @@
+Codebuff
+Copyright 2025 Codebuff
+
+This product includes software developed for the Codebuff project. 

package/cli/scripts/download-binary.cjs

@@ -7,9 +7,6 @@ const os = require('os');
 
 const REPO = 'Marcus-Mok-GH/codebuff-cli';
 const BINARY_NAME = 'codebuff';
-const MAX_RETRIES = 3;
-const REQUEST_TIMEOUT_MS = 120000;
-const MAX_REDIRECTS = 10;
 
 function getVersion() {
   const candidates = [
@@ -26,28 +23,13 @@ function getVersion() {
 }
 
 function getBinaryPath() {
-  const binDir = path.join(os.homedir(), '.codebuff', 'bin');
+  const binDir = path.join(os.homedir(), '.codebuff');
   const name = process.platform === 'win32' ? `${BINARY_NAME}.exe` : BINARY_NAME;
   return path.join(binDir, name);
 }
 
-function cleanup(dest) {
-  try {
-    if (fs.existsSync(dest)) {
-      fs.unlinkSync(dest);
-    }
-  } catch {
-    // ignore cleanup errors
-  }
-}
-
-function download(url, dest, redirectCount = 0) {
+function download(url, dest) {
   return new Promise((resolve, reject) => {
-    if (redirectCount > MAX_REDIRECTS) {
-      reject(new Error(`Too many redirects (max ${MAX_REDIRECTS}) while downloading from ${url}`));
-      return;
-    }
-
     const client = url.startsWith('https:') ? https : http;
     const file = fs.createWriteStream(dest);
 
@@ -55,10 +37,10 @@ function download(url, dest, redirectCount = 0) {
       url,
       { headers: { 'User-Agent': 'codebuff-cli-installer' } },
       (res) => {
-        if (res.statusCode === 301 || res.statusCode === 302 || res.statusCode === 307 || res.statusCode === 308) {
+        if (res.statusCode === 301 || res.statusCode === 302) {
           file.close();
-          cleanup(dest);
-          download(new URL(res.headers.location, url).href, dest, redirectCount + 1)
+          if (fs.existsSync(dest)) fs.unlinkSync(dest);
+          download(new URL(res.headers.location, url).href, dest)
             .then(resolve)
             .catch(reject);
           return;
@@ -66,8 +48,8 @@ function download(url, dest, redirectCount = 0) {
 
         if (res.statusCode !== 200) {
           file.close();
-          cleanup(dest);
-          reject(new Error(`Failed to download from ${url}: HTTP ${res.statusCode}`));
+          if (fs.existsSync(dest)) fs.unlinkSync(dest);
+          reject(new Error(`HTTP ${res.statusCode}`));
           return;
         }
 
@@ -81,36 +63,26 @@ function download(url, dest, redirectCount = 0) {
 
     req.on('error', (err) => {
       file.close();
-      cleanup(dest);
-      reject(new Error(`Request failed for ${url}: ${err.message}`));
+      if (fs.existsSync(dest)) fs.unlinkSync(dest);
+      reject(err);
     });
 
-    req.setTimeout(REQUEST_TIMEOUT_MS, () => {
+    req.setTimeout(30000, () => {
       req.destroy();
       file.close();
-      cleanup(dest);
-      reject(new Error(`Request timeout after ${REQUEST_TIMEOUT_MS}ms for ${url}`));
+      if (fs.existsSync(dest)) fs.unlinkSync(dest);
+      reject(new Error('Request timeout'));
     });
   });
 }
 
-async function downloadWithRetry(url, dest, retries = MAX_RETRIES) {
-  let lastError;
-  for (let attempt = 0; attempt <= retries; attempt++) {
-    try {
-      await download(url, dest);
-      return true;
-    } catch (err) {
-      lastError = err;
-      if (attempt < retries) {
-        const delay = Math.pow(2, attempt) * 1000;
-        console.warn(`Download attempt ${attempt + 1} failed for ${url}, retrying in ${delay}ms...`);
-        await new Promise((resolve) => setTimeout(resolve, delay));
-      }
-    }
+async function tryDownload(url, dest) {
+  try {
+    await download(url, dest);
+    return true;
+  } catch {
+    return false;
   }
-  console.error(`Download failed after ${retries + 1} attempts for ${url}: ${lastError.message}`);
-  return false;
 }
 
 async function main() {
@@ -133,9 +105,9 @@ async function main() {
 
   fs.mkdirSync(path.dirname(binaryPath), { recursive: true });
 
-  if (await downloadWithRetry(platformUrl, binaryPath)) {
+  if (await tryDownload(platformUrl, binaryPath)) {
     console.log(`Downloaded platform-specific binary: ${platformName}`);
-  } else if (await downloadWithRetry(genericUrl, binaryPath)) {
+  } else if (await tryDownload(genericUrl, binaryPath)) {
     console.log(`Downloaded generic binary: ${BINARY_NAME}`);
   } else {
     console.error(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codebuff-cli",
-  "version": "1.0.12",
+  "version": "1.0.15",
   "license": "Apache-2.0",
   "bin": {
     "codebuff": "cli/bin/codebuff.cjs",
@@ -10,7 +10,8 @@
     "cli/bin/codebuff.cjs",
     "cli/scripts/download-binary.cjs",
     "README.md",
-    "LICENSE"
+    "LICENSE",
+    "NOTICE"
   ],
   "publishConfig": {
     "access": "public"
@@ -42,7 +43,8 @@
     "test": "bun --filter='{@codebuff/common,@codebuff/agents,@codebuff/agent-runtime,@codebuff/sdk,@codebuff/cli,@codebuff/evals,@codebuff/scripts}' run test",
     "init-worktree": "bun scripts/init-worktree.ts",
     "cleanup-worktree": "bun scripts/cleanup-worktree.ts",
-    "generate-tool-definitions": "bun scripts/generate-tool-definitions.ts"
+    "generate-tool-definitions": "bun scripts/generate-tool-definitions.ts",
+    "postinstall": "node cli/scripts/download-binary.cjs"
   },
   "dependencies": {
     "@t3-oss/env-nextjs": "^0.7.3",

package/cli/bin/codebuff.cjs

@@ -1,165 +0,0 @@
-#!/usr/bin/env node
-const { spawn } = require('child_process');
-const fs = require('fs');
-const path = require('path');
-const os = require('os');
-const https = require('https');
-const http = require('http');
-
-const BINARY_NAME = 'codebuff';
-const REPO = 'Marcus-Mok-GH/codebuff-cli';
-
-const moduleBinary = path.join(__dirname, process.platform === 'win32' ? `${BINARY_NAME}.exe` : BINARY_NAME);
-const localDir = path.join(os.homedir(), '.codebuff', 'bin');
-const localBinary = path.join(localDir, process.platform === 'win32' ? `${BINARY_NAME}.exe` : BINARY_NAME);
-
-function resolveBinaryPath() {
-  if (fs.existsSync(localBinary)) return localBinary;
-  if (fs.existsSync(moduleBinary)) return moduleBinary;
-  return null;
-}
-
-function getVersion() {
-  const candidates = [
-    path.join(__dirname, '..', '..', 'package.json'),
-    path.join(__dirname, '..', 'package.json'),
-  ];
-  for (const p of candidates) {
-    try {
-      const pkg = JSON.parse(fs.readFileSync(p, 'utf8'));
-      if (pkg.version) return pkg.version;
-    } catch {}
-  }
-  return null;
-}
-
-function download(url, dest) {
-  return new Promise((resolve, reject) => {
-    const client = url.startsWith('https:') ? https : http;
-    const file = fs.createWriteStream(dest);
-
-    const req = client.get(
-      url,
-      { headers: { 'User-Agent': 'codebuff-cli-installer' } },
-      (res) => {
-        if (res.statusCode === 301 || res.statusCode === 302) {
-          file.close();
-          if (fs.existsSync(dest)) fs.unlinkSync(dest);
-          download(new URL(res.headers.location, url).href, dest)
-            .then(resolve)
-            .catch(reject);
-          return;
-        }
-
-        if (res.statusCode !== 200) {
-          file.close();
-          if (fs.existsSync(dest)) fs.unlinkSync(dest);
-          reject(new Error(`HTTP ${res.statusCode}`));
-          return;
-        }
-
-        res.pipe(file);
-        file.on('finish', () => {
-          file.close();
-          resolve();
-        });
-      }
-    );
-
-    req.on('error', (err) => {
-      file.close();
-      if (fs.existsSync(dest)) fs.unlinkSync(dest);
-      reject(err);
-    });
-
-    req.setTimeout(30000, () => {
-      req.destroy();
-      file.close();
-      if (fs.existsSync(dest)) fs.unlinkSync(dest);
-      reject(new Error('Request timeout'));
-    });
-  });
-}
-
-async function tryDownload(url, dest) {
-  try {
-    await download(url, dest);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-async function downloadBinary(destPath) {
-  const version = getVersion();
-  if (!version) {
-    throw new Error('Could not determine version from package.json');
-  }
-
-  const baseUrl = `https://github.com/${REPO}/releases/download/v${version}`;
-  const platformName = `${BINARY_NAME}-${process.platform}-${process.arch}${process.platform === 'win32' ? '.exe' : ''}`;
-  const platformUrl = `${baseUrl}/${platformName}`;
-  const genericUrl = `${baseUrl}/${BINARY_NAME}`;
-
-  fs.mkdirSync(path.dirname(destPath), { recursive: true });
-
-  if (await tryDownload(platformUrl, destPath)) {
-    console.log(`Downloaded platform-specific binary: ${platformName}`);
-  } else if (await tryDownload(genericUrl, destPath)) {
-    console.log(`Downloaded generic binary: ${BINARY_NAME}`);
-  } else {
-    throw new Error(
-      `Failed to download binary from:\n  ${platformUrl}\n  ${genericUrl}`
-    );
-  }
-
-  if (process.platform !== 'win32') {
-    fs.chmodSync(destPath, 0o755);
-  }
-}
-
-function runBinary(binaryPath) {
-  const child = spawn(binaryPath, process.argv.slice(2), {
-    stdio: 'inherit',
-  });
-
-  child.on('exit', (code, signal) => {
-    process.exit(signal ? 1 : code || 0);
-  });
-
-  child.on('error', (err) => {
-    console.error('Failed to start codebuff:', err.message);
-    process.exit(1);
-  });
-}
-
-async function main() {
-  let binaryPath = resolveBinaryPath();
-
-  if (binaryPath) {
-    runBinary(binaryPath);
-    return;
-  }
-
-  console.log('Codebuff binary not found. Downloading...');
-
-  try {
-    await downloadBinary(localBinary);
-    binaryPath = localBinary;
-  } catch (err) {
-    console.error('Failed to download codebuff:', err.message);
-    process.exit(1);
-  }
-
-  if (!fs.existsSync(binaryPath)) {
-    console.error('Binary still not found after download.');
-    process.exit(1);
-  }
-
-  runBinary(binaryPath);
-}
-
-main().catch((err) => {
-  console.error('Error:', err.message);
-  process.exit(1);
-});