@botdocs/cli 0.3.2 → 0.4.0

package/dist/commands/install.test.js

@@ -1,253 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-import { install } from './install.js';
-import { captureConsole, mockFetch, withTempDir } from '../test-utils.js';
-import { loadLockfile } from '../lib/lockfile.js';
-vi.mock('../lib/library-sync.js', () => ({
-    syncLibrary: vi.fn(),
-}));
-import * as librarySync from '../lib/library-sync.js';
-describe('install', () => {
-    let tmp;
-    let captured;
-    let restoreFetch = () => { };
-    const origHome = os.homedir;
-    let homeTmp;
-    beforeEach(() => {
-        tmp = withTempDir();
-        homeTmp = fs.mkdtempSync(path.join(os.tmpdir(), 'install-home-'));
-        os.homedir = () => homeTmp;
-        process.env.BOTDOCS_API_URL = 'http://test.local';
-        captured = captureConsole();
-    });
-    afterEach(() => {
-        restoreFetch();
-        captured.restore();
-        tmp.cleanup();
-        fs.rmSync(homeTmp, { recursive: true, force: true });
-        os.homedir = origHome;
-        vi.restoreAllMocks();
-    });
-    it('installs a SKILL bundle to the right destinations', 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: 'claude/SKILL.md', rawUrl: 'http://test.local/raw/skill' },
-                            { filename: 'cursor/rules/code-review.mdc', rawUrl: 'http://test.local/raw/cursor' },
-                        ],
-                    },
-                },
-            },
-            { url: '/raw/skill', response: { body: '# Claude SKILL\n\nbody', contentType: 'text/plain' } },
-            { url: '/raw/cursor', response: { body: '# Cursor rule', contentType: 'text/plain' } },
-        ]);
-        restoreFetch = fm.restore;
-        await install('@alice/code-review', { project: tmp.dir });
-        const claudePath = path.join(homeTmp, '.claude', 'skills', 'alice', 'code-review', 'SKILL.md');
-        const cursorPath = path.join(tmp.dir, '.cursor', 'rules', 'code-review.mdc');
-        expect(fs.existsSync(claudePath)).toBe(true);
-        expect(fs.existsSync(cursorPath)).toBe(true);
-        const lf = loadLockfile();
-        expect(lf.installs).toHaveLength(1);
-        expect(lf.installs[0].ref).toBe('@alice/code-review');
-        expect(lf.installs[0].files).toHaveLength(2);
-    });
-    it('installs a BUNDLE by recursively installing each skill', async () => {
-        const fm = mockFetch([
-            {
-                url: '/api/skills/teamco/eng-skills/manifest',
-                response: {
-                    body: {
-                        ref: { username: 'teamco', slug: 'eng-skills' },
-                        type: 'BUNDLE',
-                        version: '1.0.0',
-                        skills: [
-                            {
-                                ref: { username: 'teamco', slug: 'code-review' },
-                                version: '1.0.0',
-                                sourceEcosystem: 'claude-code',
-                                files: [{ filename: 'claude/SKILL.md', rawUrl: 'http://test.local/raw/cr' }],
-                            },
-                            {
-                                ref: { username: 'teamco', slug: 'pr-summary' },
-                                version: '1.0.0',
-                                sourceEcosystem: 'claude-code',
-                                files: [{ filename: 'claude/SKILL.md', rawUrl: 'http://test.local/raw/ps' }],
-                            },
-                        ],
-                    },
-                },
-            },
-            { url: '/raw/cr', response: { body: '# CR', contentType: 'text/plain' } },
-            { url: '/raw/ps', response: { body: '# PS', contentType: 'text/plain' } },
-        ]);
-        restoreFetch = fm.restore;
-        await install('@teamco/eng-skills', { project: tmp.dir });
-        expect(fs.existsSync(path.join(homeTmp, '.claude', 'skills', 'teamco', 'code-review', 'SKILL.md'))).toBe(true);
-        expect(fs.existsSync(path.join(homeTmp, '.claude', 'skills', 'teamco', 'pr-summary', 'SKILL.md'))).toBe(true);
-        const lf = loadLockfile();
-        expect(lf.installs.map((i) => i.ref).sort()).toEqual([
-            '@teamco/code-review',
-            '@teamco/eng-skills',
-            '@teamco/pr-summary',
-        ]);
-    });
-    it('emits JSON on --json', async () => {
-        const fm = mockFetch([
-            {
-                url: '/api/skills/alice/x/manifest',
-                response: {
-                    body: {
-                        ref: { username: 'alice', slug: 'x' },
-                        type: 'SKILL',
-                        version: '1.0.0',
-                        sourceEcosystem: null,
-                        files: [],
-                    },
-                },
-            },
-        ]);
-        restoreFetch = fm.restore;
-        await install('@alice/x', { project: tmp.dir, json: true });
-        const out = JSON.parse(captured.stdout[0]);
-        expect(out.ref).toBe('@alice/x');
-        expect(out.installed).toBeDefined();
-    });
-    it('exits 1 on bad ref', async () => {
-        await expect(install('not-a-ref', {})).rejects.toThrow();
-    });
-    it('--clean removes files from the prior install before re-installing', async () => {
-        const dest = path.join(homeTmp, '.claude', 'skills', 'alice', 'x', 'OLD.md');
-        fs.mkdirSync(path.dirname(dest), { recursive: true });
-        fs.writeFileSync(dest, 'old content');
-        const { saveLockfile } = await import('../lib/lockfile.js');
-        saveLockfile({
-            version: 1,
-            installs: [
-                {
-                    ref: '@alice/x',
-                    type: 'SKILL',
-                    version: '1.0.0',
-                    installedAt: 't',
-                    files: [{ src: 'claude/OLD.md', dest, fingerprint: 'old-fp' }],
-                },
-            ],
-        });
-        const fm = mockFetch([
-            {
-                url: '/api/skills/alice/x/manifest',
-                response: {
-                    body: {
-                        ref: { username: 'alice', slug: 'x' },
-                        type: 'SKILL',
-                        version: '2.0.0',
-                        sourceEcosystem: null,
-                        files: [{ filename: 'claude/SKILL.md', rawUrl: 'http://test.local/raw/new' }],
-                    },
-                },
-            },
-            { url: '/raw/new', response: { body: 'new content', contentType: 'text/plain' } },
-        ]);
-        restoreFetch = fm.restore;
-        await install('@alice/x', { project: tmp.dir, clean: true });
-        // Old file should be gone, new one should be present
-        expect(fs.existsSync(dest)).toBe(false);
-        expect(fs.existsSync(path.join(homeTmp, '.claude', 'skills', 'alice', 'x', 'SKILL.md'))).toBe(true);
-    });
-    it('--clean cascades file removal across a bundle\'s child skills', async () => {
-        // Pre-populate a bundle and child skill in the lockfile + on disk
-        const childDest = path.join(homeTmp, '.claude', 'skills', 'teamco', 'old-skill', 'SKILL.md');
-        fs.mkdirSync(path.dirname(childDest), { recursive: true });
-        fs.writeFileSync(childDest, 'old child content', 'utf-8');
-        const { saveLockfile } = await import('../lib/lockfile.js');
-        saveLockfile({
-            version: 1,
-            installs: [
-                { ref: '@teamco/eng', type: 'BUNDLE', version: '1.0.0', installedAt: 't', files: [], skills: ['@teamco/old-skill'] },
-                { ref: '@teamco/old-skill', type: 'SKILL', version: '1.0.0', installedAt: 't', files: [{ src: 'claude/SKILL.md', dest: childDest, fingerprint: 'old' }] },
-            ],
-        });
-        // New manifest replaces old-skill with new-skill
-        const fm = mockFetch([
-            {
-                url: '/api/skills/teamco/eng/manifest',
-                response: {
-                    body: {
-                        ref: { username: 'teamco', slug: 'eng' },
-                        type: 'BUNDLE',
-                        version: '1.1.0',
-                        skills: [
-                            {
-                                ref: { username: 'teamco', slug: 'new-skill' },
-                                version: '1.0.0',
-                                sourceEcosystem: 'claude',
-                                files: [{ filename: 'claude/SKILL.md', rawUrl: 'http://test.local/raw/new' }],
-                            },
-                        ],
-                    },
-                },
-            },
-            { url: '/raw/new', response: { body: 'new content', contentType: 'text/plain' } },
-        ]);
-        restoreFetch = fm.restore;
-        await install('@teamco/eng', { project: tmp.dir, clean: true });
-        // Old child file should be deleted by the cascade
-        expect(fs.existsSync(childDest)).toBe(false);
-        // New skill installed
-        expect(fs.existsSync(path.join(homeTmp, '.claude', 'skills', 'teamco', 'new-skill', 'SKILL.md'))).toBe(true);
-    });
-    it('--json suppresses the manual-paste console.log for chatgpt files', async () => {
-        const fm = mockFetch([
-            {
-                url: '/api/skills/alice/x/manifest',
-                response: {
-                    body: {
-                        ref: { username: 'alice', slug: 'x' },
-                        type: 'SKILL',
-                        version: '1.0.0',
-                        sourceEcosystem: null,
-                        files: [{ filename: 'chatgpt/instructions.md', rawUrl: 'http://test.local/raw/cg' }],
-                    },
-                },
-            },
-            // Note: under --json, the manual-paste branch should NOT call fetchRawContent at all
-            // (it's wrapped in `if (!options.json)`), so no /raw/cg mock is needed.
-        ]);
-        restoreFetch = fm.restore;
-        await install('@alice/x', { project: tmp.dir, json: true });
-        // stdout has exactly one entry — the JSON output
-        expect(captured.stdout).toHaveLength(1);
-        const out = JSON.parse(captured.stdout[0]);
-        expect(out.ref).toBe('@alice/x');
-        // Verify no manual-paste content leaked into stdout
-        expect(captured.stdout.join('\n')).not.toMatch(/Manual paste required/);
-    });
-    it('invokes syncLibrary after a successful install', async () => {
-        const fm = mockFetch([
-            {
-                url: '/api/skills/alice/x/manifest',
-                response: {
-                    body: {
-                        ref: { username: 'alice', slug: 'x' },
-                        type: 'SKILL',
-                        version: '1.0.0',
-                        sourceEcosystem: null,
-                        files: [],
-                    },
-                },
-            },
-        ]);
-        restoreFetch = fm.restore;
-        await install('@alice/x', { project: tmp.dir });
-        expect(librarySync.syncLibrary).toHaveBeenCalledTimes(1);
-    });
-});

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

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

package/dist/commands/list.test.js

@@ -1,51 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-import { list } from './list.js';
-import { captureConsole } from '../test-utils.js';
-import { saveLockfile } from '../lib/lockfile.js';
-describe('list', () => {
-    let captured;
-    const origHome = os.homedir;
-    let homeTmp;
-    beforeEach(() => {
-        homeTmp = fs.mkdtempSync(path.join(os.tmpdir(), 'list-home-'));
-        os.homedir = () => homeTmp;
-        captured = captureConsole();
-    });
-    afterEach(() => {
-        captured.restore();
-        fs.rmSync(homeTmp, { recursive: true, force: true });
-        os.homedir = origHome;
-    });
-    it('says "nothing installed" when lockfile is empty', async () => {
-        await list({});
-        expect(captured.stdout.join('\n')).toMatch(/nothing installed/i);
-    });
-    it('groups skills by their parent bundle', async () => {
-        saveLockfile({
-            version: 1,
-            installs: [
-                { ref: '@a/eng', type: 'BUNDLE', version: '1.0.0', installedAt: 't', files: [], skills: ['@a/cr', '@a/ps'] },
-                { ref: '@a/cr', type: 'SKILL', version: '1.0.0', installedAt: 't', files: [] },
-                { ref: '@a/ps', type: 'SKILL', version: '1.0.0', installedAt: 't', files: [] },
-                { ref: '@a/standalone', type: 'SKILL', version: '1.0.0', installedAt: 't', files: [] },
-            ],
-        });
-        await list({});
-        const out = captured.stdout.join('\n');
-        expect(out).toContain('@a/eng');
-        expect(out).toContain('@a/cr');
-        expect(out).toContain('@a/standalone');
-    });
-    it('emits JSON on --json', async () => {
-        saveLockfile({
-            version: 1,
-            installs: [{ ref: '@a/x', type: 'SKILL', version: '1.0.0', installedAt: 't', files: [] }],
-        });
-        await list({ json: true });
-        const out = JSON.parse(captured.stdout[0]);
-        expect(out.installs).toHaveLength(1);
-    });
-});

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

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

package/dist/commands/publish.test.js

@@ -1,138 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-import { withTempDir, captureConsole, mockFetch } from '../test-utils.js';
-import { saveAuth } from '../lib/config.js';
-vi.mock('./compile.js', () => ({
-    compile: vi.fn(),
-}));
-import * as compileMod from './compile.js';
-import { publish } from './publish.js';
-describe('publish auto-compile gate', () => {
-    let tmp;
-    let captured;
-    let restoreFetch = () => { };
-    const origHome = os.homedir;
-    let homeTmp;
-    beforeEach(() => {
-        tmp = withTempDir();
-        homeTmp = fs.mkdtempSync(path.join(os.tmpdir(), 'publish-'));
-        os.homedir = () => homeTmp;
-        captured = captureConsole();
-        process.env.BOTDOCS_API_URL = 'http://test.local';
-        saveAuth({ githubToken: 't', username: 'u', displayName: 'U' });
-    });
-    afterEach(() => {
-        restoreFetch();
-        captured.restore();
-        fs.rmSync(homeTmp, { recursive: true, force: true });
-        os.homedir = origHome;
-        tmp.cleanup();
-        vi.restoreAllMocks();
-    });
-    function setupCanonicalSkill() {
-        const root = path.join(tmp.dir, 'my-skill');
-        fs.mkdirSync(path.join(root, 'claude-code', 'commands'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'index.md'), '# Skill\n\n' + 'a'.repeat(150), 'utf-8');
-        fs.writeFileSync(path.join(root, 'botdocs.json'), JSON.stringify({
-            type: 'skill',
-            version: '1.0.0',
-            title: 'My Skill',
-            description: 'Reviews PRs',
-            sourceEcosystem: 'claude-code',
-            ecosystems: ['claude-code', 'claude'],
-        }));
-        fs.writeFileSync(path.join(root, 'claude-code', 'commands', 'my-skill.md'), '# CC content');
-        // Generated claude/SKILL.md is missing → stale
-        return root;
-    }
-    it('--no-compile does NOT call compile even when generated files are stale', async () => {
-        const root = setupCanonicalSkill();
-        const fm = mockFetch([
-            {
-                method: 'POST',
-                url: '/api/botdocs',
-                response: { body: { id: 'b1', slug: 'my-skill', url: 'http://x/y' } },
-            },
-        ]);
-        restoreFetch = fm.restore;
-        await publish(root, { description: 'desc', noCompile: true });
-        expect(compileMod.compile).not.toHaveBeenCalled();
-    });
-    it('auto-calls compile when generated files are stale', async () => {
-        const root = setupCanonicalSkill();
-        const fm = mockFetch([
-            {
-                method: 'POST',
-                url: '/api/botdocs',
-                response: { body: { id: 'b1', slug: 'my-skill', url: 'http://x/y' } },
-            },
-        ]);
-        restoreFetch = fm.restore;
-        await publish(root, { description: 'desc' });
-        expect(compileMod.compile).toHaveBeenCalledTimes(1);
-    });
-    it('uploads nested files with directory-prefixed filenames', async () => {
-        // Build a tree with files in subdirectories — we need to know what
-        // ends up in the POST body so install can route by the `claude/` /
-        // `claude-code/` prefix.
-        const root = path.join(tmp.dir, 'nested-skill');
-        fs.mkdirSync(path.join(root, 'claude'), { recursive: true });
-        fs.mkdirSync(path.join(root, 'claude-code', 'commands'), { recursive: true });
-        fs.writeFileSync(path.join(root, 'index.md'), '# Top\n\n' + 'a'.repeat(150));
-        fs.writeFileSync(path.join(root, 'claude', 'SKILL.md'), '# Claude content');
-        fs.writeFileSync(path.join(root, 'claude-code', 'commands', 'cli.md'), '# CC content');
-        const fm = mockFetch([
-            {
-                method: 'POST',
-                url: '/api/botdocs',
-                response: { body: { id: 'b1', slug: 'nested-skill', url: 'http://x/y' } },
-            },
-        ]);
-        restoreFetch = fm.restore;
-        await publish(root, { description: 'desc', noCompile: true });
-        const post = fm.calls.find((c) => c.url.includes('/api/botdocs') && c.method === 'POST');
-        const body = post?.body;
-        const filenames = (body?.files ?? []).map((f) => f.filename).sort();
-        expect(filenames).toEqual([
-            'claude-code/commands/cli.md',
-            'claude/SKILL.md',
-            'index.md',
-        ]);
-    });
-    it('forwards botdocType + sourceEcosystem from botdocs.json', async () => {
-        const root = setupCanonicalSkill();
-        const fm = mockFetch([
-            {
-                method: 'POST',
-                url: '/api/botdocs',
-                response: { body: { id: 'b1', slug: 'my-skill', url: 'http://x/y' } },
-            },
-        ]);
-        restoreFetch = fm.restore;
-        await publish(root, { description: 'desc', noCompile: true });
-        const post = fm.calls.find((c) => c.url.includes('/api/botdocs') && c.method === 'POST');
-        const body = post?.body;
-        expect(body.botdocType).toBe('SKILL');
-        expect(body.sourceEcosystem).toBe('claude-code');
-    });
-    it('defaults to SPEC when no botdocs.json is present', async () => {
-        const root = path.join(tmp.dir, 'plain-spec');
-        fs.mkdirSync(root, { recursive: true });
-        fs.writeFileSync(path.join(root, 'index.md'), '# Spec\n\n' + 'a'.repeat(150));
-        const fm = mockFetch([
-            {
-                method: 'POST',
-                url: '/api/botdocs',
-                response: { body: { id: 'b1', slug: 'plain-spec', url: 'http://x/y' } },
-            },
-        ]);
-        restoreFetch = fm.restore;
-        await publish(root, { description: 'desc' });
-        const post = fm.calls.find((c) => c.url.includes('/api/botdocs') && c.method === 'POST');
-        const body = post?.body;
-        expect(body.botdocType).toBe('SPEC');
-        expect(body.sourceEcosystem).toBeNull();
-    });
-});

package/dist/commands/pull.d.ts

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

package/dist/commands/pull.js

@@ -1,78 +0,0 @@
-import fs from 'node:fs';
-import path from 'node:path';
-import { apiFetch, fetchRawContent } from '../lib/api.js';
-export async function pull(ref, options = {}) {
-    const parsed = parseRef(ref);
-    if (!parsed) {
-        console.error('Invalid reference. Use format: username/slug');
-        process.exit(1);
-    }
-    const { username, slug } = parsed;
-    const outDir = path.resolve(slug);
-    // Check for existing clone metadata
-    const metaPath = path.join(outDir, '.botdocs.json');
-    if (!fs.existsSync(metaPath)) {
-        console.error(`No clone found at ./${slug}/`);
-        console.error('Use `botdocs clone` first.');
-        process.exit(1);
-    }
-    console.log(`Updating ${username}/${slug}...`);
-    // Fetch latest manifest
-    let manifest;
-    try {
-        manifest = await apiFetch(`/@${username}/${slug}/manifest`);
-    }
-    catch {
-        console.error(`BotDoc not found: ${username}/${slug}`);
-        process.exit(1);
-    }
-    // Download all files (overwrite)
-    let updated = 0;
-    for (const file of manifest.files) {
-        const filePath = path.join(outDir, file.filename);
-        const content = await fetchRawContent(file.rawUrl);
-        const existing = fs.existsSync(filePath) ? fs.readFileSync(filePath, 'utf-8') : null;
-        if (existing !== content) {
-            fs.writeFileSync(filePath, content, 'utf-8');
-            console.log(`  Updated: ${file.filename}`);
-            updated++;
-        }
-    }
-    // Remove files that no longer exist in the BotDoc
-    const metadata = JSON.parse(fs.readFileSync(metaPath, 'utf-8'));
-    const remoteFiles = new Set(manifest.files.map((f) => f.filename));
-    for (const oldFile of metadata.files) {
-        if (!remoteFiles.has(oldFile)) {
-            const oldPath = path.join(outDir, oldFile);
-            if (fs.existsSync(oldPath)) {
-                fs.unlinkSync(oldPath);
-                console.log(`  Removed: ${oldFile}`);
-                updated++;
-            }
-        }
-    }
-    // Update metadata
-    const newMetadata = {
-        username,
-        slug,
-        clonedAt: metadata.clonedAt,
-        files: manifest.files.map((f) => f.filename),
-    };
-    fs.writeFileSync(metaPath, JSON.stringify(newMetadata, null, 2), 'utf-8');
-    if (options.json) {
-        console.log(JSON.stringify({ success: true, updated, files: manifest.files.map(f => f.filename) }));
-    }
-    else if (updated === 0) {
-        console.log('Already up to date.');
-    }
-    else {
-        console.log(`\nUpdated ${updated} file(s).`);
-    }
-}
-function parseRef(ref) {
-    const cleaned = ref.startsWith('@') ? ref.slice(1) : ref;
-    const parts = cleaned.split('/');
-    if (parts.length !== 2 || !parts[0] || !parts[1])
-        return null;
-    return { username: parts[0], slug: parts[1] };
-}

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

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