@botdocs/cli 0.3.2 → 0.5.0

package/dist/lib/lockfile.test.js

@@ -1,99 +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 { loadLockfile, saveLockfile, upsertInstall, removeInstall, fingerprintFile, fingerprintContent, } from './lockfile.js';
-describe('lockfile', () => {
-    const origHome = os.homedir;
-    let tmp;
-    beforeEach(() => {
-        tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'lockfile-test-'));
-        os.homedir = () => tmp;
-    });
-    afterEach(() => {
-        os.homedir = origHome;
-        fs.rmSync(tmp, { recursive: true, force: true });
-    });
-    it('loadLockfile returns empty when file does not exist', () => {
-        const lf = loadLockfile();
-        expect(lf).toEqual({ version: 1, installs: [] });
-    });
-    it('saveLockfile writes a valid JSON file', () => {
-        const lf = { version: 1, installs: [{ ref: '@a/b', type: 'SKILL', version: '1.0.0', installedAt: '2026-04-29T00:00:00Z', files: [] }] };
-        saveLockfile(lf);
-        const read = loadLockfile();
-        expect(read.installs).toHaveLength(1);
-        expect(read.installs[0].ref).toBe('@a/b');
-    });
-    it('upsertInstall replaces an existing entry with the same ref', () => {
-        saveLockfile({
-            version: 1,
-            installs: [{ ref: '@a/b', type: 'SKILL', version: '1.0.0', installedAt: 't', files: [] }],
-        });
-        upsertInstall({
-            ref: '@a/b',
-            type: 'SKILL',
-            version: '2.0.0',
-            installedAt: 't2',
-            files: [],
-        });
-        const lf = loadLockfile();
-        expect(lf.installs).toHaveLength(1);
-        expect(lf.installs[0].version).toBe('2.0.0');
-    });
-    it('removeInstall removes the entry by ref', () => {
-        saveLockfile({
-            version: 1,
-            installs: [
-                { ref: '@a/b', type: 'SKILL', version: '1.0.0', installedAt: 't', files: [] },
-                { ref: '@a/c', type: 'SKILL', version: '1.0.0', installedAt: 't', files: [] },
-            ],
-        });
-        removeInstall('@a/b');
-        const lf = loadLockfile();
-        expect(lf.installs).toHaveLength(1);
-        expect(lf.installs[0].ref).toBe('@a/c');
-    });
-    it('fingerprintFile normalizes CRLF to LF before hashing', () => {
-        const lf = path.join(tmp, 'lf.txt');
-        const crlf = path.join(tmp, 'crlf.txt');
-        fs.writeFileSync(lf, 'one\ntwo\nthree\n');
-        fs.writeFileSync(crlf, 'one\r\ntwo\r\nthree\r\n');
-        expect(fingerprintFile(lf)).toBe(fingerprintFile(crlf));
-    });
-    it('fingerprintFile hashes binary files (NUL bytes) as raw bytes', () => {
-        const binPath = path.join(tmp, 'bin.dat');
-        // A buffer with a NUL byte — looks binary
-        fs.writeFileSync(binPath, Buffer.from([0x00, 0x01, 0xff, 0xfe, 0x80]));
-        // Same buffer twice should give the same fingerprint
-        expect(fingerprintFile(binPath)).toBe(fingerprintFile(binPath));
-    });
-    it('fingerprintFile treats files with U+FFFD as text (no false binary classification)', () => {
-        const txtPath = path.join(tmp, 'with-replacement.md');
-        // Text content that legitimately contains U+FFFD — should still be text-hashed
-        fs.writeFileSync(txtPath, '# Header\n\nReplacement: �\n');
-        const fp1 = fingerprintFile(txtPath);
-        // Same content with CRLF — should fingerprint identically (text path normalized)
-        const txtCrlfPath = path.join(tmp, 'with-replacement-crlf.md');
-        fs.writeFileSync(txtCrlfPath, '# Header\r\n\r\nReplacement: �\r\n');
-        expect(fingerprintFile(txtCrlfPath)).toBe(fp1);
-    });
-    it('fingerprintContent agrees with fingerprintFile for the same text content', () => {
-        const filePath = path.join(tmp, 'agree.txt');
-        const content = 'hello\nworld\n';
-        fs.writeFileSync(filePath, content);
-        expect(fingerprintContent(content)).toBe(fingerprintFile(filePath));
-    });
-    it('fingerprintContent normalizes CRLF the same way as fingerprintFile', () => {
-        const filePath = path.join(tmp, 'crlf.txt');
-        fs.writeFileSync(filePath, 'a\r\nb\r\n');
-        expect(fingerprintContent('a\nb\n')).toBe(fingerprintFile(filePath));
-    });
-    it('loadLockfile returns empty when JSON is corrupt (does not throw)', () => {
-        const lf = path.join(tmp, '.botdocs', 'installed.json');
-        fs.mkdirSync(path.dirname(lf), { recursive: true });
-        fs.writeFileSync(lf, '{ this is not valid JSON');
-        const result = loadLockfile();
-        expect(result).toEqual({ version: 1, installs: [] });
-    });
-});

package/dist/lib/manifest.test.d.ts

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

package/dist/lib/manifest.test.js

@@ -1,72 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { parseManifest, ManifestError } from './manifest.js';
-describe('parseManifest', () => {
-    it('parses a SKILL manifest', () => {
-        const m = parseManifest({
-            type: 'skill',
-            version: '1.0.0',
-            title: 'Code Review',
-            description: 'Reviews PRs',
-            sourceEcosystem: 'claude-code',
-        });
-        expect(m.type).toBe('SKILL');
-        expect(m.version).toBe('1.0.0');
-        expect(m.sourceEcosystem).toBe('claude-code');
-    });
-    it('parses a BUNDLE manifest with refs', () => {
-        const m = parseManifest({
-            type: 'bundle',
-            version: '1.0.0',
-            title: 'Eng Skills',
-            description: 'All engineering skills',
-            skills: ['@alice/code-review', '@alice/pr-summary'],
-        });
-        expect(m.type).toBe('BUNDLE');
-        expect(m.skills).toEqual([
-            { username: 'alice', slug: 'code-review' },
-            { username: 'alice', slug: 'pr-summary' },
-        ]);
-    });
-    it('defaults type to SPEC when missing', () => {
-        const m = parseManifest({ title: 't', description: 'd' });
-        expect(m.type).toBe('SPEC');
-    });
-    it('throws on non-semver version', () => {
-        expect(() => parseManifest({ type: 'skill', version: 'latest', title: 't', description: 'd' })).toThrow(ManifestError);
-    });
-    it('throws when bundle skills entry is malformed', () => {
-        expect(() => parseManifest({ type: 'bundle', version: '1.0.0', title: 't', description: 'd', skills: ['not-a-ref'] })).toThrow(ManifestError);
-    });
-    it('accepts skills with leading @', () => {
-        const m = parseManifest({
-            type: 'bundle',
-            version: '1.0.0',
-            title: 't',
-            description: 'd',
-            skills: ['@alice/x', 'bob/y'],
-        });
-        expect(m.skills).toHaveLength(2);
-    });
-    it('throws on non-object input', () => {
-        expect(() => parseManifest(null)).toThrow(ManifestError);
-        expect(() => parseManifest('string')).toThrow(ManifestError);
-        expect(() => parseManifest(42)).toThrow(ManifestError);
-    });
-    it('throws on non-string type field', () => {
-        expect(() => parseManifest({ type: 42, title: 't', description: 'd' })).toThrow(ManifestError);
-    });
-    it('throws when bundle is missing skills array', () => {
-        expect(() => parseManifest({ type: 'bundle', version: '1.0.0', title: 't', description: 'd' })).toThrow(ManifestError);
-    });
-    it('parses ecosystems[] field on a SKILL manifest', () => {
-        const m = parseManifest({
-            type: 'skill',
-            version: '1.0.0',
-            title: 'X',
-            description: 'Y',
-            sourceEcosystem: 'claude-code',
-            ecosystems: ['claude', 'claude-code', 'cursor'],
-        });
-        expect(m.ecosystems).toEqual(['claude', 'claude-code', 'cursor']);
-    });
-});

package/dist/lib/shell-hook.test.d.ts

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

package/dist/lib/shell-hook.test.js

@@ -1,68 +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 { detectShell, installShellHook, uninstallShellHook, BEGIN, END } from './shell-hook.js';
-describe('shell-hook', () => {
-    const origHome = os.homedir;
-    let homeTmp;
-    beforeEach(() => {
-        homeTmp = fs.mkdtempSync(path.join(os.tmpdir(), 'sh-'));
-        os.homedir = () => homeTmp;
-    });
-    afterEach(() => {
-        fs.rmSync(homeTmp, { recursive: true, force: true });
-        os.homedir = origHome;
-    });
-    it('detectShell returns zsh for ZSH_VERSION env', () => {
-        expect(detectShell({ ZSH_VERSION: '5.9' })).toBe('zsh');
-    });
-    it('detectShell returns bash from BASH env', () => {
-        expect(detectShell({ BASH: '/bin/bash' })).toBe('bash');
-    });
-    it('detectShell falls back to SHELL env', () => {
-        expect(detectShell({ SHELL: '/usr/bin/fish' })).toBe('fish');
-        expect(detectShell({ SHELL: '/bin/zsh' })).toBe('zsh');
-    });
-    it('detectShell returns null for unknown shells', () => {
-        expect(detectShell({ SHELL: '/bin/csh' })).toBeNull();
-    });
-    it('detectShell prefers $SHELL over BASH env var (handles bash -c from zsh)', () => {
-        // User's login shell is zsh; they ran the command via `bash -c botdocs ...`
-        // which exports BASH=/bin/bash. We want zsh, not bash.
-        expect(detectShell({ SHELL: '/bin/zsh', BASH: '/bin/bash' })).toBe('zsh');
-    });
-    it('installShellHook creates a delimited block in .zshrc', () => {
-        const result = installShellHook('zsh');
-        const rc = fs.readFileSync(path.join(homeTmp, '.zshrc'), 'utf-8');
-        expect(rc).toContain(BEGIN);
-        expect(rc).toContain(END);
-        expect(rc).toContain('botdocs check-updates --quiet');
-        expect(result.action).toBe('created');
-    });
-    it('installShellHook is idempotent on re-run', () => {
-        installShellHook('zsh');
-        const r2 = installShellHook('zsh');
-        const rc = fs.readFileSync(path.join(homeTmp, '.zshrc'), 'utf-8');
-        expect(rc.match(new RegExp(BEGIN.replace(/[/\\$.*+?()[\]{}|]/g, '\\$&'), 'g'))).toHaveLength(1);
-        expect(r2.action).toBe('updated');
-    });
-    it('installShellHook preserves user content outside the markers', () => {
-        fs.writeFileSync(path.join(homeTmp, '.bashrc'), 'export EDITOR=vim\n', 'utf-8');
-        installShellHook('bash');
-        const rc = fs.readFileSync(path.join(homeTmp, '.bashrc'), 'utf-8');
-        expect(rc).toContain('export EDITOR=vim');
-        expect(rc).toContain(BEGIN);
-    });
-    it('uninstallShellHook removes the block', () => {
-        installShellHook('zsh');
-        uninstallShellHook('zsh');
-        const rc = fs.readFileSync(path.join(homeTmp, '.zshrc'), 'utf-8');
-        expect(rc).not.toContain(BEGIN);
-    });
-    it('writes the fish-syntax variant for fish', () => {
-        installShellHook('fish');
-        const rc = fs.readFileSync(path.join(homeTmp, '.config', 'fish', 'config.fish'), 'utf-8');
-        expect(rc).toContain('botdocs check-updates --quiet');
-    });
-});

package/dist/test-utils.d.ts

@@ -1,43 +0,0 @@
-/** Thrown by the mocked process.exit so tests can assert on the exit code
- * without actually killing the test runner. */
-export declare class ProcessExitError extends Error {
-    readonly code: number;
-    constructor(code: number);
-}
-export interface CapturedConsole {
-    stdout: string[];
-    stderr: string[];
-    restore: () => void;
-}
-/** Patches console.log/console.error and process.exit so commands can be
- * driven from tests without writing to the real terminal or exiting. */
-export declare function captureConsole(): CapturedConsole;
-export interface MockFetchCall {
-    url: string;
-    method: string;
-    headers: Record<string, string>;
-    body: unknown;
-}
-export interface MockFetchResponse {
-    status?: number;
-    body?: unknown;
-    contentType?: string;
-}
-/** Installs a fetch stub that matches requests by (method, urlSubstring) and
- * returns the configured response. Each handler can fire only once unless
- * `repeat: true` is set. */
-export declare function mockFetch(handlers: Array<{
-    method?: string;
-    url: string | RegExp;
-    response: MockFetchResponse | ((call: MockFetchCall) => MockFetchResponse);
-    repeat?: boolean;
-}>): {
-    calls: MockFetchCall[];
-    restore: () => void;
-};
-/** Creates a unique temporary directory and changes into it. The returned
- * cleanup function restores the prior cwd and removes the dir. */
-export declare function withTempDir(): {
-    dir: string;
-    cleanup: () => void;
-};

package/dist/test-utils.js

@@ -1,101 +0,0 @@
-import { vi } from 'vitest';
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-/** Thrown by the mocked process.exit so tests can assert on the exit code
- * without actually killing the test runner. */
-export class ProcessExitError extends Error {
-    code;
-    constructor(code) {
-        super(`process.exit(${code})`);
-        this.name = 'ProcessExitError';
-        this.code = code;
-    }
-}
-/** Patches console.log/console.error and process.exit so commands can be
- * driven from tests without writing to the real terminal or exiting. */
-export function captureConsole() {
-    const stdout = [];
-    const stderr = [];
-    const logSpy = vi.spyOn(console, 'log').mockImplementation((...args) => {
-        stdout.push(args.map(String).join(' '));
-    });
-    const errSpy = vi.spyOn(console, 'error').mockImplementation((...args) => {
-        stderr.push(args.map(String).join(' '));
-    });
-    const exitSpy = vi.spyOn(process, 'exit').mockImplementation(((code) => {
-        throw new ProcessExitError(code ?? 0);
-    }));
-    return {
-        stdout,
-        stderr,
-        restore: () => {
-            logSpy.mockRestore();
-            errSpy.mockRestore();
-            exitSpy.mockRestore();
-        },
-    };
-}
-/** Installs a fetch stub that matches requests by (method, urlSubstring) and
- * returns the configured response. Each handler can fire only once unless
- * `repeat: true` is set. */
-export function mockFetch(handlers) {
-    const calls = [];
-    const used = new Set();
-    const original = global.fetch;
-    global.fetch = vi.fn(async (input, init) => {
-        const url = typeof input === 'string' ? input : input.url;
-        const method = (init?.method || 'GET').toUpperCase();
-        const headers = (init?.headers || {});
-        let body = undefined;
-        if (init?.body) {
-            try {
-                body = JSON.parse(init.body);
-            }
-            catch {
-                body = init.body;
-            }
-        }
-        const call = { url, method, headers, body };
-        calls.push(call);
-        for (let i = 0; i < handlers.length; i++) {
-            const h = handlers[i];
-            if (!h.repeat && used.has(i))
-                continue;
-            const matchUrl = typeof h.url === 'string' ? url.includes(h.url) : h.url.test(url);
-            const matchMethod = !h.method || h.method.toUpperCase() === method;
-            if (matchUrl && matchMethod) {
-                used.add(i);
-                const r = typeof h.response === 'function' ? h.response(call) : h.response;
-                const status = r.status ?? 200;
-                const isJson = r.contentType ? r.contentType.includes('json') : typeof r.body !== 'string';
-                const payload = isJson ? JSON.stringify(r.body ?? {}) : r.body;
-                return new Response(payload, {
-                    status,
-                    headers: { 'content-type': r.contentType ?? (isJson ? 'application/json' : 'text/plain') },
-                });
-            }
-        }
-        throw new Error(`Unhandled fetch ${method} ${url}`);
-    });
-    return {
-        calls,
-        restore: () => {
-            global.fetch = original;
-        },
-    };
-}
-/** Creates a unique temporary directory and changes into it. The returned
- * cleanup function restores the prior cwd and removes the dir. */
-export function withTempDir() {
-    const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'botdocs-cli-test-'));
-    const prevCwd = process.cwd();
-    process.chdir(dir);
-    return {
-        dir,
-        cleanup: () => {
-            process.chdir(prevCwd);
-            fs.rmSync(dir, { recursive: true, force: true });
-        },
-    };
-}