@botdocs/cli 0.10.2 → 0.11.0

package/dist/lib/manifest.js

@@ -4,6 +4,18 @@ export class ManifestError extends Error {
         this.name = 'ManifestError';
     }
 }
+/**
+ * The placeholder description `botdocs init` writes into a fresh manifest.
+ * Duplicated here (kept in sync with the constant in `commands/init.ts`) so
+ * the validator can reject the exact string without commands/init.ts and
+ * lib/manifest.ts having a circular dependency (init imports validate
+ * indirectly through the publish flow; manifest is upstream of both).
+ *
+ * If you change this, change `INIT_PLACEHOLDER_DESCRIPTION` in
+ * `commands/init.ts` to match — a string-literal diff in either file
+ * silently breaks the "you left the placeholder" detection.
+ */
+export const INIT_PLACEHOLDER_DESCRIPTION = 'TODO: describe your skill in one sentence.';
 const SEMVER = /^\d+\.\d+\.\d+(-[\w.-]+)?(\+[\w.-]+)?$/;
 function parseSkillRef(raw) {
     const cleaned = raw.startsWith('@') ? raw.slice(1) : raw;
@@ -38,8 +50,23 @@ export function parseManifest(input) {
     if (type !== 'SPEC' && !SEMVER.test(version)) {
         throw new ManifestError(`Version must be semver (e.g. 1.0.0), got: "${version}"`);
     }
-    if (typeof data.title !== 'string' || typeof data.description !== 'string' || !data.title || !data.description) {
-        throw new ManifestError('title and description are required');
+    // Split title and description validation into separate errors so the
+    // author sees exactly which field is missing. The old combined check
+    // ("title and description are required") was generic — a friend who'd
+    // filled in the title still saw the same line and had to guess.
+    if (typeof data.title !== 'string' || !data.title.trim()) {
+        throw new ManifestError('title is required in botdocs.json');
+    }
+    if (typeof data.description !== 'string' || !data.description.trim()) {
+        throw new ManifestError('description is required in botdocs.json');
+    }
+    // Reject the unchanged placeholder that `botdocs init` writes. Without
+    // this, an author who scaffolded but never edited the manifest would
+    // publish a skill called "TODO: describe your skill in one sentence." —
+    // a recurring source of registry noise. The message names the file so
+    // the fix is obvious.
+    if (data.description.trim() === INIT_PLACEHOLDER_DESCRIPTION) {
+        throw new ManifestError(`Description still says "${INIT_PLACEHOLDER_DESCRIPTION}" — edit botdocs.json before publishing.`);
     }
     const title = data.title;
     const description = data.description;

package/dist/lib/node-preflight.d.ts

@@ -0,0 +1,20 @@
+/**
+ * Pure Node-version check. Lives in its own module so the inline preflight at
+ * the top of `src/index.ts` can be unit-tested without having to spawn a
+ * subprocess with a doctored `process.versions.node`.
+ *
+ * Returns `{ ok: true }` when the version satisfies the floor; otherwise
+ * `{ ok: false, message }` where `message` is the exact stderr line we want
+ * the CLI to print. Keeping the wording in one place means tests pin the
+ * message and we can't accidentally diverge between runtime and docs.
+ *
+ * Safe to import before any Node-20-only globals are touched — this file
+ * uses only `Number.parseInt` and string ops that exist on every supported
+ * Node version, including the legacy runtimes we're trying to reject.
+ */
+export interface NodePreflightResult {
+    ok: boolean;
+    message?: string;
+}
+export declare const MIN_NODE_MAJOR = 20;
+export declare function checkNodeVersion(nodeVersion: string): NodePreflightResult;

package/dist/lib/node-preflight.js

@@ -0,0 +1,11 @@
+export const MIN_NODE_MAJOR = 20;
+export function checkNodeVersion(nodeVersion) {
+    const major = Number.parseInt(nodeVersion.split('.')[0] ?? '0', 10);
+    if (!Number.isFinite(major) || major < MIN_NODE_MAJOR) {
+        return {
+            ok: false,
+            message: `BotDocs CLI requires Node.js ${MIN_NODE_MAJOR} or newer. You're on Node ${nodeVersion}. Install from https://nodejs.org`,
+        };
+    }
+    return { ok: true };
+}

package/dist/lib/skill-caps.d.ts

@@ -0,0 +1,17 @@
+/**
+ * Client-side mirror of the server's per-skill size and count caps. The server
+ * enforces these on every ingest + publish-from-source request — duplicating
+ * the constants here lets `validate` flag violations locally so authors don't
+ * blow through the publish flow only to get a 413 back from the API.
+ *
+ * MUST stay in sync with apps/web/src/lib/skill-caps.ts. A shared package
+ * would eliminate the duplication long-term; for now the constants are tiny
+ * enough that mirroring is the simplest path. If you change one, change the
+ * other.
+ */
+export declare const PER_FILE_BYTE_CAP: number;
+export declare const PER_SKILL_BYTE_CAP: number;
+export declare const PER_SKILL_FILE_CAP = 25;
+/** Pretty-print a byte count as KB with one decimal. Used in validate error
+ * messages so authors see "71.3 KB" instead of "73012 bytes". */
+export declare function formatKB(bytes: number): string;

package/dist/lib/skill-caps.js

@@ -0,0 +1,19 @@
+/**
+ * Client-side mirror of the server's per-skill size and count caps. The server
+ * enforces these on every ingest + publish-from-source request — duplicating
+ * the constants here lets `validate` flag violations locally so authors don't
+ * blow through the publish flow only to get a 413 back from the API.
+ *
+ * MUST stay in sync with apps/web/src/lib/skill-caps.ts. A shared package
+ * would eliminate the duplication long-term; for now the constants are tiny
+ * enough that mirroring is the simplest path. If you change one, change the
+ * other.
+ */
+export const PER_FILE_BYTE_CAP = 64 * 1024;
+export const PER_SKILL_BYTE_CAP = 512 * 1024;
+export const PER_SKILL_FILE_CAP = 25;
+/** Pretty-print a byte count as KB with one decimal. Used in validate error
+ * messages so authors see "71.3 KB" instead of "73012 bytes". */
+export function formatKB(bytes) {
+    return `${(bytes / 1024).toFixed(1)} KB`;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@botdocs/cli",
-  "version": "0.10.2",
+  "version": "0.11.0",
   "description": "CLI for BotDocs — author, publish, install, and sync agent skills across Claude, Claude Code, Cursor, Codex, ChatGPT, Windsurf, Copilot, Gemini, Antigravity, and OpenCode.",
   "keywords": [
     "botdocs",
@@ -33,7 +33,7 @@
   "license": "MIT",
   "author": "BotDocs contributors",
   "bin": {
-    "botdocs": "./dist/index.js"
+    "botdocs": "./bin/botdocs.cjs"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
@@ -49,6 +49,7 @@
     "vitest": "^3.1.0"
   },
   "files": [
+    "bin",
     "dist",
     "templates",
     "README.md"