ancoder-skill-cli 0.2.4 → 0.4.0

package/README.md

@@ -12,7 +12,15 @@ npm install -g ancoder-skill-cli
 npx ancoder-skill-cli --help
 ```
 
-After install, the binary is either downloaded from GitHub Releases (if you set `repository` in `package.json` and publish releases) or you can build from source once and the wrapper will use it.
+The npm package is self-contained and includes prebuilt binaries for:
+
+- macOS arm64
+- macOS x64
+- Linux arm64
+- Linux x64
+- Windows x64
+
+After install, the wrapper selects the correct bundled binary for the current platform automatically.
 
 ## Build from source (Go)
 
@@ -35,18 +43,24 @@ go build -o bin/skill-cli .
 ## Publish to npm
 
 1. Set `repository.url` in `package.json` to your GitHub repo (e.g. `git+https://github.com/your-org/skill-cli.git`).
-2. For install-from-GitHub flow: build binaries per platform and attach to a GitHub Release with names:
+2. Build binaries per platform before publishing the npm package:
+
+   ```bash
+   bash scripts/build-all.sh
+   ```
+
+3. Optionally attach the same binaries to a GitHub Release with names:
    - `skill-cli-darwin-arm64`, `skill-cli-darwin-x64`
    - `skill-cli-linux-x64`, `skill-cli-linux-arm64`
    - `skill-cli-win32-x64.exe`
-3. Publish the package:
+4. Publish the package:
 
    ```bash
    npm login --registry=https://registry.npmjs.org/
    npm publish --access public --registry=https://registry.npmjs.org/ --userconfig ~/.npmrc
    ```
 
-Users who `npm install -g ancoder-skill-cli` will get the wrapper; `postinstall` will try to download the binary for their platform from the GitHub release. If no release exists, they can still build from source and put the binary in `bin/skill-cli` (or `bin/skill-cli.exe` on Windows).
+Users who `npm install -g ancoder-skill-cli` get a fully bundled package. No extra binary download is required during install.
 
 ## License
 

package/bin/skill-cli.js

@@ -6,14 +6,38 @@ const { spawn } = require('child_process');
 const fs = require('fs');
 
 const isWindows = process.platform === 'win32';
-const binName = isWindows ? 'skill-cli.exe' : 'skill-cli';
-const binPath = path.join(__dirname, binName);
+const key = `${process.platform}-${process.arch}`;
+const targetMap = {
+  'darwin-arm64': 'skill-cli-darwin-arm64',
+  'darwin-x64': 'skill-cli-darwin-x64',
+  'linux-arm64': 'skill-cli-linux-arm64',
+  'linux-x64': 'skill-cli-linux-x64',
+  'win32-x64': 'skill-cli-win32-x64.exe',
+};
+
+const bundledName = targetMap[key];
+const bundledPath = bundledName ? path.join(__dirname, 'targets', bundledName) : null;
+const fallbackName = isWindows ? 'skill-cli.exe' : 'skill-cli';
+const fallbackPath = path.join(__dirname, fallbackName);
+
+const binPath = bundledPath && fs.existsSync(bundledPath) ? bundledPath : fallbackPath;
 
 if (!fs.existsSync(binPath)) {
-  console.error('skill-cli: binary not found. Run "npm install" or build from source: go build -o ' + binName + ' .');
+  console.error(
+    'skill-cli: binary not found for ' + key + '. ' +
+    'Expected bundled binary: ' + (bundledName || 'unsupported-platform') + '.'
+  );
   process.exit(1);
 }
 
+if (!isWindows) {
+  try {
+    fs.chmodSync(binPath, 0o755);
+  } catch (_) {
+    // ignore chmod failures on readonly installs
+  }
+}
+
 const child = spawn(binPath, process.argv.slice(2), {
   stdio: 'inherit',
   shell: false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ancoder-skill-cli",
-  "version": "0.2.4",
+  "version": "0.4.0",
   "description": "CLI for managing everything-claude-code (ECC) components — agents, skills, commands, rules, hooks, MCP configs. Single binary, all assets embedded.",
   "bin": {
     "skill-cli": "bin/skill-cli.js"
@@ -37,6 +37,7 @@
   },
   "files": [
     "bin/skill-cli.js",
+    "bin/targets",
     "scripts/postinstall.js",
     "scripts/check-bin.js",
     "README.md",

package/scripts/check-bin.js

@@ -5,10 +5,22 @@ const path = require('path');
 const fs = require('fs');
 
 const binDir = path.join(__dirname, '..', 'bin');
-const hasUnix = fs.existsSync(path.join(binDir, 'skill-cli'));
-const hasWin = fs.existsSync(path.join(binDir, 'skill-cli.exe'));
+const targetsDir = path.join(binDir, 'targets');
+const expected = [
+  'skill-cli-darwin-arm64',
+  'skill-cli-darwin-x64',
+  'skill-cli-linux-arm64',
+  'skill-cli-linux-x64',
+  'skill-cli-win32-x64.exe',
+];
 
-if (!hasUnix && !hasWin) {
-  console.warn('skill-cli: no binary in bin/. Users will get the binary from GitHub Releases on install.');
-  console.warn('To ship a binary in the package, run: go build -o bin/skill-cli .');
+const missing = expected.filter((name) => !fs.existsSync(path.join(targetsDir, name)));
+
+if (missing.length > 0) {
+  console.error('skill-cli: missing bundled binaries for npm publish:');
+  for (const name of missing) {
+    console.error('  - ' + name);
+  }
+  console.error('Run: bash scripts/build-all.sh');
+  process.exit(1);
 }

package/scripts/postinstall.js

@@ -3,12 +3,6 @@
 
 const fs = require('fs');
 const path = require('path');
-const https = require('https');
-const http = require('http');
-
-const pkg = require(path.join(__dirname, '..', 'package.json'));
-const version = pkg.version;
-const repo = 'AncoderAI/Ancoder_SkillsCreator';
 
 const platform = process.platform;
 const arch = process.arch;
@@ -23,73 +17,20 @@ const assetNames = {
 };
 
 const key = `${platform}-${arch}`;
-const assetName = assetNames[key];
-const binDir = path.join(__dirname, '..', 'bin');
-const binName = isWindows ? 'skill-cli.exe' : 'skill-cli';
-const binPath = path.join(binDir, binName);
+const bundledName = assetNames[key];
+const bundledPath = bundledName ? path.join(__dirname, '..', 'bin', 'targets', bundledName) : null;
 
-// Already installed
-if (fs.existsSync(binPath)) {
-  try {
-    fs.accessSync(binPath, fs.constants.X_OK);
-    process.exit(0);
-  } catch (_) {
-    fs.chmodSync(binPath, 0o755);
-    process.exit(0);
-  }
-}
-
-if (!assetName) {
-  console.warn(`skill-cli: no prebuilt binary for ${key}`);
-  console.warn(`Build from source: go build -o bin/${binName} .`);
+if (!bundledName || !bundledPath || !fs.existsSync(bundledPath)) {
+  console.warn(`skill-cli: bundled binary missing for ${key}`);
   process.exit(0);
 }
 
-const url = `https://github.com/${repo}/releases/download/v${version}/${assetName}`;
-
-function fetch(url, dest, redirects) {
-  if (redirects > 5) {
-    console.error('skill-cli: too many redirects');
-    process.exit(1);
+if (!isWindows) {
+  try {
+    fs.chmodSync(bundledPath, 0o755);
+  } catch (_) {
+    // ignore chmod failures
   }
-
-  const proto = url.startsWith('https') ? https : http;
-
-  proto.get(url, {
-    headers: { 'User-Agent': 'skill-cli-npm-postinstall' }
-  }, (res) => {
-    // Follow redirects (GitHub releases always redirect)
-    if (res.statusCode === 301 || res.statusCode === 302 || res.statusCode === 307) {
-      const location = res.headers.location;
-      if (location) {
-        res.resume(); // consume response to free socket
-        fetch(location, dest, redirects + 1);
-        return;
-      }
-    }
-
-    if (res.statusCode !== 200) {
-      console.warn(`skill-cli: HTTP ${res.statusCode} downloading ${url}`);
-      console.warn(`Build from source: go build -o bin/${binName} .`);
-      process.exit(0);
-    }
-
-    const file = fs.createWriteStream(dest, { mode: 0o755 });
-    res.pipe(file);
-    file.on('finish', () => {
-      file.close();
-      console.log(`skill-cli: installed binary for ${key} (v${version})`);
-    });
-    file.on('error', (err) => {
-      fs.unlink(dest, () => {});
-      console.warn('skill-cli: write error:', err.message);
-    });
-  }).on('error', (err) => {
-    console.warn('skill-cli: download error:', err.message);
-    console.warn(`Build from source: go build -o bin/${binName} .`);
-  });
 }
 
-fs.mkdirSync(binDir, { recursive: true });
-console.log(`skill-cli: downloading binary for ${key}...`);
-fetch(url, binPath, 0);
+console.log(`skill-cli: bundled binary ready for ${key}`);