@botdocs/cli 0.3.2 → 0.4.0

package/dist/commands/compile.test.js

@@ -1,110 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import fs from 'node:fs';
-import path from 'node:path';
-import { compile } from './compile.js';
-import { captureConsole, withTempDir } from '../test-utils.js';
-vi.mock('../lib/llm.js', () => ({
-    complete: vi.fn(),
-    detectProvider: vi.fn(() => ({ provider: 'anthropic', keyEnv: 'BOTDOCS_ANTHROPIC_KEY' })),
-    LlmError: class extends Error {
-    },
-}));
-import * as llm from '../lib/llm.js';
-describe('compile', () => {
-    let tmp;
-    let captured;
-    beforeEach(() => {
-        tmp = withTempDir();
-        captured = captureConsole();
-        process.env.BOTDOCS_ANTHROPIC_KEY = 'test-key';
-    });
-    afterEach(() => {
-        captured.restore();
-        tmp.cleanup();
-        vi.restoreAllMocks();
-        delete process.env.BOTDOCS_ANTHROPIC_KEY;
-    });
-    function setupSkill() {
-        const root = path.join(tmp.dir, 'my-skill');
-        fs.mkdirSync(path.join(root, 'claude-code', 'commands'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'botdocs.json'), JSON.stringify({
-            type: 'skill',
-            version: '1.0.0',
-            title: 'Code Review',
-            description: 'Reviews PRs',
-            sourceEcosystem: 'claude-code',
-            ecosystems: ['claude', 'claude-code', 'cursor'],
-        }));
-        fs.writeFileSync(path.join(root, 'claude-code', 'commands', 'my-skill.md'), '# Code Review\n\nReview the PR.');
-        return root;
-    }
-    it('generates per-ecosystem files (excluding the source ecosystem)', async () => {
-        vi.mocked(llm.complete).mockResolvedValue({
-            text: 'GENERATED OUTPUT',
-            usage: { inputTokens: 100, outputTokens: 50 },
-            provider: 'anthropic',
-        });
-        const root = setupSkill();
-        await compile(root, {});
-        expect(fs.existsSync(path.join(root, 'claude', 'SKILL.md'))).toBe(true);
-        expect(fs.existsSync(path.join(root, 'cursor', 'rules', 'my-skill.mdc'))).toBe(true);
-        // source-ecosystem file is untouched
-        expect(fs.readFileSync(path.join(root, 'claude-code', 'commands', 'my-skill.md'), 'utf-8'))
-            .toBe('# Code Review\n\nReview the PR.');
-        expect(llm.complete).toHaveBeenCalledTimes(2); // claude + cursor
-    });
-    it('--ecosystems flag subsets the generated set', async () => {
-        vi.mocked(llm.complete).mockResolvedValue({
-            text: 'OUT',
-            usage: { inputTokens: 10, outputTokens: 5 },
-            provider: 'anthropic',
-        });
-        const root = setupSkill();
-        await compile(root, { ecosystems: 'cursor' });
-        expect(fs.existsSync(path.join(root, 'cursor', 'rules', 'my-skill.mdc'))).toBe(true);
-        expect(fs.existsSync(path.join(root, 'claude', 'SKILL.md'))).toBe(false);
-        expect(llm.complete).toHaveBeenCalledTimes(1);
-    });
-    it('errors when no API key is set', async () => {
-        delete process.env.BOTDOCS_ANTHROPIC_KEY;
-        vi.mocked(llm.detectProvider).mockImplementation(() => {
-            throw new llm.LlmError('No LLM key');
-        });
-        const root = setupSkill();
-        await expect(compile(root, {})).rejects.toThrow();
-    });
-    it('exits 1 when sourceEcosystem in botdocs.json is unsupported', async () => {
-        vi.mocked(llm.complete).mockResolvedValue({
-            text: 'X',
-            usage: { inputTokens: 1, outputTokens: 1 },
-            provider: 'anthropic',
-        });
-        const root = path.join(tmp.dir, 'bad-source');
-        fs.mkdirSync(root, { recursive: true });
-        fs.writeFileSync(path.join(root, 'botdocs.json'), JSON.stringify({
-            type: 'skill',
-            version: '1.0.0',
-            title: 'X',
-            description: 'Y',
-            sourceEcosystem: 'claude-codee', // typo
-            ecosystems: ['claude'],
-        }));
-        await expect(compile(root, {})).rejects.toThrow();
-        expect(captured.stderr.join('\n')).toMatch(/Unsupported ecosystem.*claude-codee/);
-    });
-    it('exits 1 when --regenerate target is unsupported', async () => {
-        const root = path.join(tmp.dir, 'good');
-        fs.mkdirSync(path.join(root, 'claude-code', 'commands'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'botdocs.json'), JSON.stringify({
-            type: 'skill',
-            version: '1.0.0',
-            title: 'X',
-            description: 'Y',
-            sourceEcosystem: 'claude-code',
-            ecosystems: ['claude', 'claude-code'],
-        }));
-        fs.writeFileSync(path.join(root, 'claude-code', 'commands', 'good.md'), '# Body');
-        await expect(compile(root, { regenerate: 'cursor-but-typo' })).rejects.toThrow();
-        expect(captured.stderr.join('\n')).toMatch(/Unsupported ecosystem.*cursor-but-typo/);
-    });
-});

package/dist/commands/diff.d.ts

@@ -1,3 +0,0 @@
-export declare function diff(ref: string, options: {
-    json?: boolean;
-}): Promise<void>;

package/dist/commands/diff.js

@@ -1,65 +0,0 @@
-import { readFileSync, existsSync } from 'fs';
-import { join } from 'path';
-import { apiFetch, fetchRawContent } from '../lib/api.js';
-export async function diff(ref, options) {
-    const parts = ref.replace(/^@/, '').split('/');
-    if (parts.length !== 2 || !parts[0] || !parts[1]) {
-        console.error('Usage: botdocs diff <username/slug>');
-        process.exit(1);
-    }
-    const [username, slug] = parts;
-    const localDir = join(process.cwd(), slug);
-    if (!existsSync(localDir)) {
-        console.error(`No local directory found: ${slug}/`);
-        console.error(`Run 'botdocs clone ${ref}' first.`);
-        process.exit(1);
-    }
-    const manifest = await apiFetch(`/@${username}/${slug}/manifest`);
-    const changes = [];
-    const localFiles = new Set();
-    const metaPath = join(localDir, '.botdocs.json');
-    if (existsSync(metaPath)) {
-        const meta = JSON.parse(readFileSync(metaPath, 'utf-8'));
-        if (meta.files) {
-            for (const f of meta.files) {
-                localFiles.add(f);
-            }
-        }
-    }
-    for (const remoteFile of manifest.files) {
-        const localPath = join(localDir, remoteFile.filename);
-        if (!existsSync(localPath)) {
-            changes.push({ filename: remoteFile.filename, status: 'added' });
-            continue;
-        }
-        const localContent = readFileSync(localPath, 'utf-8');
-        const remoteContent = await fetchRawContent(remoteFile.rawUrl);
-        if (localContent !== remoteContent) {
-            changes.push({ filename: remoteFile.filename, status: 'modified' });
-        }
-        else {
-            changes.push({ filename: remoteFile.filename, status: 'unchanged' });
-        }
-        localFiles.delete(remoteFile.filename);
-    }
-    for (const filename of localFiles) {
-        if (filename === '.botdocs.json')
-            continue;
-        changes.push({ filename, status: 'removed' });
-    }
-    if (options.json) {
-        console.log(JSON.stringify({ ref, changes }));
-        return;
-    }
-    const modified = changes.filter((c) => c.status !== 'unchanged');
-    if (modified.length === 0) {
-        console.log('\n  Up to date — no remote changes.\n');
-        return;
-    }
-    console.log('');
-    for (const change of modified) {
-        const icon = change.status === 'added' ? '+' : change.status === 'removed' ? '-' : '~';
-        console.log(`  ${icon} ${change.filename}`);
-    }
-    console.log(`\n  ${modified.length} file(s) changed. Run 'botdocs pull ${ref}' to update.\n`);
-}

package/dist/commands/edit.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/commands/edit.test.js

@@ -1,102 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { edit } from './edit.js';
-import { captureConsole, mockFetch } from '../test-utils.js';
-import { saveAuth } from '../lib/config.js';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-vi.mock('../lib/llm.js', () => ({
-    complete: vi.fn(),
-    detectProvider: vi.fn(() => ({ provider: 'anthropic', keyEnv: 'BOTDOCS_ANTHROPIC_KEY' })),
-    LlmError: class extends Error {
-    },
-}));
-vi.mock('@inquirer/prompts', () => ({
-    input: vi.fn(),
-    select: vi.fn(),
-}));
-import * as llm from '../lib/llm.js';
-import * as inquirer from '@inquirer/prompts';
-describe('edit', () => {
-    let captured;
-    let restoreFetch = () => { };
-    const origHome = os.homedir;
-    let homeTmp;
-    beforeEach(() => {
-        homeTmp = fs.mkdtempSync(path.join(os.tmpdir(), 'edit-'));
-        os.homedir = () => homeTmp;
-        captured = captureConsole();
-        process.env.BOTDOCS_API_URL = 'http://test.local';
-        process.env.BOTDOCS_ANTHROPIC_KEY = 'test';
-        saveAuth({ githubToken: 't', username: 'u', displayName: 'U' });
-    });
-    afterEach(() => {
-        captured.restore();
-        restoreFetch();
-        fs.rmSync(homeTmp, { recursive: true, force: true });
-        os.homedir = origHome;
-        vi.restoreAllMocks();
-        delete process.env.BOTDOCS_ANTHROPIC_KEY;
-    });
-    it('round-trips: pulls, prompts user, calls LLM, pushes draft on accept', async () => {
-        const fm = mockFetch([
-            {
-                url: '/api/skills/alice/code-review/manifest',
-                response: {
-                    body: {
-                        ref: { username: 'alice', slug: 'code-review' },
-                        type: 'SKILL',
-                        version: '1.0.0',
-                        sourceEcosystem: 'claude-code',
-                        files: [
-                            { filename: 'cursor/rules/code-review.mdc', rawUrl: 'http://test.local/raw/cursor' },
-                            { filename: 'claude-code/commands/code-review.md', rawUrl: 'http://test.local/raw/source' },
-                        ],
-                    },
-                },
-            },
-            { url: '/raw/cursor', response: { body: '# Original cursor rule\n', contentType: 'text/plain' } },
-            { method: 'POST', url: '/api/skills/alice/code-review/draft', response: { body: { ok: true } } },
-        ]);
-        restoreFetch = fm.restore;
-        vi.mocked(inquirer.input).mockResolvedValue('Add a section about tests');
-        vi.mocked(inquirer.select).mockResolvedValue('accept');
-        vi.mocked(llm.complete).mockResolvedValue({
-            text: '# Original cursor rule\n\n## Tests\n',
-            usage: { inputTokens: 100, outputTokens: 50 },
-            provider: 'anthropic',
-        });
-        await edit('@alice/code-review', { ecosystem: 'cursor' });
-        const draftCall = fm.calls.find((c) => c.url.endsWith('/draft'));
-        expect(draftCall).toBeDefined();
-        expect(draftCall.body.ecosystem).toBe('cursor');
-    });
-    it('cancels without pushing when user picks cancel', async () => {
-        const fm = mockFetch([
-            {
-                url: '/api/skills/alice/x/manifest',
-                response: {
-                    body: {
-                        ref: { username: 'alice', slug: 'x' },
-                        type: 'SKILL',
-                        version: '1.0.0',
-                        sourceEcosystem: 'claude',
-                        files: [{ filename: 'claude/SKILL.md', rawUrl: 'http://test.local/raw/c' }],
-                    },
-                },
-            },
-            { url: '/raw/c', response: { body: 'orig', contentType: 'text/plain' } },
-        ]);
-        restoreFetch = fm.restore;
-        vi.mocked(inquirer.input).mockResolvedValue('change something');
-        vi.mocked(inquirer.select).mockResolvedValue('cancel');
-        vi.mocked(llm.complete).mockResolvedValue({
-            text: 'new',
-            usage: { inputTokens: 10, outputTokens: 5 },
-            provider: 'anthropic',
-        });
-        await edit('@alice/x', { ecosystem: 'claude' });
-        const draftCall = fm.calls.find((c) => c.url.endsWith('/draft'));
-        expect(draftCall).toBeUndefined();
-    });
-});

package/dist/commands/endorse.d.ts

@@ -1,7 +0,0 @@
-interface EndorseOptions {
-    rating: string;
-    comment?: string;
-    json?: boolean;
-}
-export declare function endorse(ref: string, options: EndorseOptions): Promise<void>;
-export {};

package/dist/commands/endorse.js

@@ -1,70 +0,0 @@
-import { ApiError, apiFetch } from '../lib/api.js';
-const VALID_RATINGS = ['positive', 'neutral', 'negative'];
-function parseRef(ref) {
-    const parts = ref.replace(/^@/, '').split('/');
-    if (parts.length !== 2 || !parts[0] || !parts[1])
-        return null;
-    return { username: parts[0], slug: parts[1] };
-}
-/** Treat any 403 whose message mentions "clone" as the server's
- * clone-required guard. The web detail page surfaces the same hint inline;
- * here we point users at the CLI command that fixes it. */
-function isCloneRequired(err) {
-    return err.status === 403 && /clone/i.test(err.message);
-}
-export async function endorse(ref, options) {
-    const parsed = parseRef(ref);
-    if (!parsed) {
-        console.error('Usage: botdocs endorse <username/slug> --rating <positive|neutral|negative>');
-        process.exit(1);
-    }
-    const { username, slug } = parsed;
-    const rating = options.rating?.toLowerCase();
-    if (!rating || !VALID_RATINGS.includes(rating)) {
-        console.error('Rating must be: positive, neutral, or negative');
-        process.exit(1);
-    }
-    let result;
-    try {
-        result = await apiFetch(`/api/endorsements/${username}/${slug}`, {
-            method: 'POST',
-            body: {
-                rating: rating.toUpperCase(),
-                comment: options.comment,
-                source: 'CLI',
-            },
-            auth: true,
-        });
-    }
-    catch (err) {
-        if (err instanceof ApiError) {
-            if (isCloneRequired(err)) {
-                console.error(`\n  ✗ ${err.message}\n` +
-                    `\n    Endorsements are reserved for builders who actually used the spec. Run:\n` +
-                    `      botdocs clone @${username}/${slug}\n` +
-                    `    …build something on top, then come back and endorse it.\n`);
-                process.exit(1);
-            }
-            if (err.status === 401) {
-                console.error('\n  ✗ Not authenticated. Run `botdocs login` first.\n');
-                process.exit(1);
-            }
-            if (err.status === 404) {
-                console.error(`\n  ✗ BotDoc not found: @${username}/${slug}\n`);
-                process.exit(1);
-            }
-            console.error(`\n  ✗ ${err.message}\n`);
-            process.exit(1);
-        }
-        throw err;
-    }
-    if (options.json) {
-        console.log(JSON.stringify(result));
-        return;
-    }
-    console.log(`\n  ✓ Endorsed @${username}/${slug} as ${rating}.`);
-    if (options.comment) {
-        console.log(`    Comment: "${options.comment}"`);
-    }
-    console.log('');
-}

package/dist/commands/ingest.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/commands/ingest.test.js

@@ -1,109 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import fs from 'node:fs';
-import path from 'node:path';
-import { ingest } from './ingest.js';
-import { captureConsole, mockFetch, withTempDir } from '../test-utils.js';
-vi.mock('../lib/config.js', () => ({
-    loadAuth: () => ({ githubToken: 'test-token', username: 'alice', displayName: 'Alice' }),
-    saveAuth: vi.fn(),
-    clearAuth: vi.fn(),
-}));
-describe('ingest', () => {
-    let tmp;
-    let captured;
-    let restoreFetch = () => { };
-    beforeEach(() => {
-        tmp = withTempDir();
-        process.env.BOTDOCS_API_URL = 'http://test.local';
-        captured = captureConsole();
-    });
-    afterEach(() => {
-        restoreFetch();
-        captured.restore();
-        tmp.cleanup();
-        vi.restoreAllMocks();
-    });
-    it('detects skills from claude/cursor/claude-code subtrees and POSTs them', async () => {
-        const root = path.join(tmp.dir, 'skills');
-        fs.mkdirSync(path.join(root, 'claude', 'code-review'), { recursive: true });
-        fs.mkdirSync(path.join(root, 'cursor', 'rules'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'claude', 'code-review', 'SKILL.md'), '# Code Review\n\nA spec.');
-        fs.writeFileSync(path.join(root, 'cursor', 'rules', 'pr-summary.mdc'), '# PR Summary\n\nA rule.');
-        const fm = mockFetch([
-            {
-                method: 'POST',
-                url: '/api/cli/ingest',
-                response: { body: { draftId: 'd1', reviewUrl: '/ingest/d1' } },
-            },
-        ]);
-        restoreFetch = fm.restore;
-        await ingest(root, {});
-        expect(fm.calls).toHaveLength(1);
-        const body = fm.calls[0].body;
-        const slugs = body.skills.map((s) => s.slug).sort();
-        expect(slugs).toEqual(['code-review', 'pr-summary']);
-        expect(captured.stdout.join('\n')).toMatch(/d1/);
-    });
-    it('respects --bundle to wrap skills into a bundle draft', async () => {
-        const root = path.join(tmp.dir, 'skills');
-        fs.mkdirSync(path.join(root, 'claude', 'code-review'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'claude', 'code-review', 'SKILL.md'), '# CR');
-        const fm = mockFetch([
-            { method: 'POST', url: '/api/cli/ingest', response: { body: { draftId: 'd2', reviewUrl: '/ingest/d2' } } },
-        ]);
-        restoreFetch = fm.restore;
-        await ingest(root, { bundle: 'Eng Skills' });
-        const body = fm.calls[0].body;
-        expect(body.bundle?.name).toBe('Eng Skills');
-    });
-    it('--dry-run prints findings without uploading', async () => {
-        const root = path.join(tmp.dir, 'skills');
-        fs.mkdirSync(path.join(root, 'claude', 'code-review'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'claude', 'code-review', 'SKILL.md'), '# CR');
-        const fm = mockFetch([]);
-        restoreFetch = fm.restore;
-        await ingest(root, { dryRun: true });
-        expect(fm.calls).toHaveLength(0);
-        expect(captured.stdout.join('\n')).toMatch(/code-review/);
-    });
-    it('skips node_modules, .git, dist, and other well-known ignored directories', async () => {
-        const root = path.join(tmp.dir, 'project');
-        // A real skill we want detected
-        fs.mkdirSync(path.join(root, 'claude', 'real'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'claude', 'real', 'SKILL.md'), '# Real');
-        // node_modules + dist + .git with planted skill files that MUST be ignored
-        fs.mkdirSync(path.join(root, 'node_modules', 'claude', 'fake'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'node_modules', 'claude', 'fake', 'SKILL.md'), '# Fake');
-        fs.mkdirSync(path.join(root, '.git', 'claude', 'fake'), { recursive: true });
-        fs.writeFileSync(path.join(root, '.git', 'claude', 'fake', 'SKILL.md'), '# Fake');
-        fs.mkdirSync(path.join(root, 'dist', 'claude', 'fake'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'dist', 'claude', 'fake', 'SKILL.md'), '# Fake');
-        const fm = mockFetch([
-            { method: 'POST', url: '/api/cli/ingest', response: { body: { draftId: 'd', reviewUrl: '/ingest/d' } } },
-        ]);
-        restoreFetch = fm.restore;
-        await ingest(root, {});
-        const body = fm.calls[0].body;
-        expect(body.skills.map((s) => s.slug)).toEqual(['real']);
-    });
-    it('groups multi-ecosystem files under the same slug into one logical skill', async () => {
-        // A skill named "code-review" has both a Claude SKILL.md and a Cursor rule.
-        // Without grouping these, the test ingest would POST 2 skills with the same slug
-        // and one would silently win in the DB.
-        const root = path.join(tmp.dir, 'skills');
-        fs.mkdirSync(path.join(root, 'claude', 'code-review'), { recursive: true });
-        fs.mkdirSync(path.join(root, 'cursor', 'rules'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'claude', 'code-review', 'SKILL.md'), '# Code Review\n\nClaude version.');
-        fs.writeFileSync(path.join(root, 'cursor', 'rules', 'code-review.mdc'), '# Code Review\n\nCursor version.');
-        const fm = mockFetch([
-            { method: 'POST', url: '/api/cli/ingest', response: { body: { draftId: 'd', reviewUrl: '/ingest/d' } } },
-        ]);
-        restoreFetch = fm.restore;
-        await ingest(root, {});
-        const body = fm.calls[0].body;
-        // The two files should collapse into ONE skill named code-review with both files
-        const codeReview = body.skills.find((s) => s.slug === 'code-review');
-        expect(codeReview).toBeDefined();
-        expect(codeReview.files).toHaveLength(2);
-    });
-});

package/dist/commands/install.test.d.ts

@@ -1 +0,0 @@
-export {};