mcpill 1.3.0 → 1.6.0

package/MANUAL-TEST-GUIDE-SPEC.md

@@ -1,270 +0,0 @@
----
-title: "Feature Specification"
-type: planning
-tags: [planning, specification, scoping]
----
-
-# Specification: mcpill compile-hooks — Manual Test Guide
-
-**Date:** 2026-05-26
-**Author:** ruco
-**Status:** Draft
-
----
-
-## Overview
-
-> `mcpill compile` auto-generates a `PreToolUse` hook in `.claude/settings.json` that warns Claude to use the pill's MCP tools instead of native file tools (`Read`, `Bash`, `Edit`, `Write`). The hook derives its matcher and message from the `## Tools` table in `AGENT.md`, making each pill self-enforcing without manual setup. Key behaviours: merge without overwrite, `--no-hooks` opt-out, and `mcpill validate` warning when the hook is absent.
-
----
-
-## Background & Motivation
-
-Without this feature, a pill author must manually wire a `PreToolUse` hook into `.claude/settings.json` every time they publish or update a pill. Agents forget to use the registered MCP tools and fall back to native tools, bypassing the pill entirely. The compile step already has all the information needed (pill name, tool names, which native tools each replaces) — generating the hook at compile time ties it to the source of truth.
-
-## Goals
-
-- `mcpill compile` writes or updates a `PreToolUse` hook in `<project>/.claude/settings.json` derived from the pill's `AGENT.md ## Tools` table.
-- Existing entries in `settings.json` (e.g. `mcpServers`) are preserved; only the matching `PreToolUse` entry is replaced.
-- `--no-hooks` opts out and writes `.mcpill/.no-hooks` as a sentinel so `validate` skips the check.
-- `mcpill validate` emits a non-fatal warning when `AGENT.md` declares replacements but no hook is present.
-
-## Non-Goals
-
-- Auto-generating `AGENT.md` — the pill author writes it manually.
-- Writing to the global `~/.claude/settings.json` — scope is always the project root.
-- Enforcing hook presence (fatal error) — the warning is advisory only.
-
-## Detailed Design
-
-### Data Model
-
-```
-AGENT.md ## Tools table — expected columns (order-independent, case-insensitive header):
-  | Tool         | Description       | Replaces      |
-  |--------------|-------------------|---------------|
-  | read-chunks  | Read file chunks  | Read          |
-  | run-query    | Execute SQL       | Bash,Read     |
-
-"Replaces" cell: comma-separated native tool names (Read, Bash, Edit, Write).
-Omitting the column or leaving a cell blank → no hook generated for that row.
-
-.claude/settings.json — hooks section written by compile:
-{
-  "hooks": {
-    "PreToolUse": [
-      {
-        "matcher": "Read|Bash",
-        "hooks": [{ "type": "command", "command": "echo \"[mcpill:<name>] Use pill tools instead: <tool>→<native>; ... See .mcpill/AGENT.md.\"" }]
-      }
-    ]
-  }
-}
-
-Merge key: matcher string (exact match). Replace entry if matcher already exists; append otherwise.
-
-.mcpill/.no-hooks — presence signals that hook generation was opted out; written by --no-hooks flag.
-```
-
-### API / Interface
-
-```
-# CLI
-mcpill compile [--dir <path>] [--no-hooks]
-
-# Programmatic
-runCompile(opts: { dir?: string; toMd?: boolean; strict?: boolean; noHooks?: boolean }): Promise<void>
-
-# AGENT.md lookup order
-1. <baseDir>/AGENT.md
-2. <baseDir>/.mcpill/AGENT.md
-(first found wins; if neither exists, hook generation is skipped silently)
-
-# Hook message format
-[mcpill:<pillName>] Use pill tools instead: <tool1>→<native1>,<native2>; <tool2>→<native3>. See .mcpill/AGENT.md.
-```
-
-### Behavior
-
-1. `mcpill compile` runs the standard compile pipeline (server.md → .mcpill/server/).
-2. If `--no-hooks` was passed: write `.mcpill/.no-hooks`, return early — no hook written.
-3. Locate `AGENT.md` at `baseDir/AGENT.md` or `.mcpill/AGENT.md`; if absent, skip hook generation.
-4. Parse `## Tools` table — find the `replaces` column (case-insensitive). Extract comma-separated native tool names per row. Skip rows with empty/missing replaces cell.
-5. If no rows have replacements, skip hook generation.
-6. Build `HookEntry`: matcher = all unique native tool names joined by `|`; command = static `echo` with pill name and full tool→native summary baked in.
-7. Read `.claude/settings.json` (or start from `{}`). Find existing `hooks.PreToolUse` array entry with same matcher; replace it, or append if not found. Write back.
-8. `mcpill validate` — after structural validation passes, check: if `.mcpill/.no-hooks` absent AND `AGENT.md` declares replacements AND `.claude/settings.json` has no `PreToolUse` entries → emit `⚠  No PreToolUse hook in .claude/settings.json — run mcpill compile to add it`.
-
-## Error Handling
-
-| Error Case | Behavior | User-Facing Message |
-|---|---|---|
-| `.claude/settings.json` exists but is invalid JSON | `JSON.parse` throws → compile crashes with a JS stack trace | *(raw error)* — fix or delete the malformed settings.json |
-| `AGENT.md` exists but has no `## Tools` section | `parseAgentMdTools` returns `[]` → hook generation silently skipped | *(none)* |
-| `AGENT.md` has `## Tools` but no `replaces` column | `parseAgentMdTools` returns `[]` → silently skipped | *(none)* |
-| Compile run with `--no-hooks`; later `validate` called | `.no-hooks` sentinel present → validate skips hook check entirely | *(none)* |
-| `baseDir/.claude/` does not exist | `mkdirSync(claudeDir, { recursive: true })` creates it | *(none — transparent)* |
-
-## Security Considerations
-
-- Hook command is a static `echo` — no shell variable expansion, no user-controlled input interpolated into the command string at runtime. `JSON.stringify(message)` in `buildHookEntry` ensures the baked-in string is safely quoted.
-- `settings.json` is scoped to `baseDir/.claude/` — no writes to global config.
-- `AGENT.md` content is read and parsed; no `eval`/`exec` of its contents.
-
-## Testing Plan
-
-Run all automated tests first: `npm test` — all 6 hook tests in `src/__tests__/compile.test.ts` must pass.
-
-Then follow these manual steps against a real pill directory.
-
-### Setup
-
-Create a scratch pill for testing:
-
-```sh
-mkdir /tmp/test-pill && cd /tmp/test-pill
-mcpill init
-```
-
-Add a minimal `AGENT.md` at the project root:
-
-```markdown
-# AGENT
-
-## Tools
-
-| Tool | Description | Replaces |
-|------|-------------|----------|
-| read-chunks | Read a file in chunks | Read |
-| run-query | Execute a SQL query | Bash,Read |
-```
-
----
-
-### Test 1 — Happy path: hook is written
-
-- [ ] Run `mcpill compile` in `/tmp/test-pill`.
-- [ ] Verify `.claude/settings.json` was created.
-- [ ] Open it and confirm:
-  - `hooks.PreToolUse` is an array with one entry.
-  - `matcher` is `"Read|Bash"` (or `"Bash|Read"` — order may vary).
-  - `hooks[0].command` contains `echo`.
-  - The echo string contains `[mcpill:` followed by the pill name (from `.mcpill/server.md` or `PILL.md`).
-  - The echo string contains `read-chunks→Read` and `run-query→Bash,Read`.
-  - The echo string ends with `See .mcpill/AGENT.md.`
-
----
-
-### Test 2 — Merge: existing settings.json is preserved
-
-- [ ] Add a fake `mcpServers` entry to `.claude/settings.json`:
-  ```json
-  { "mcpServers": { "some-server": { "command": "node", "args": [] } } }
-  ```
-- [ ] Run `mcpill compile` again.
-- [ ] Verify `.claude/settings.json` still contains `mcpServers.some-server`.
-- [ ] Verify `hooks.PreToolUse` is also present (not the only key).
-
----
-
-### Test 3 — Re-compile replaces the hook, not appends
-
-- [ ] Change the `Replaces` cell of `run-query` to just `Bash` (remove `Read`).
-- [ ] Run `mcpill compile`.
-- [ ] Verify `hooks.PreToolUse` still has exactly **one** entry (not two).
-- [ ] Verify the matcher reflects the updated set.
-
----
-
-### Test 4 — `--no-hooks` flag
-
-- [ ] Delete `.claude/settings.json` and `.mcpill/.no-hooks` if they exist.
-- [ ] Run `mcpill compile --no-hooks`.
-- [ ] Verify `.claude/settings.json` was **not** created.
-- [ ] Verify `.mcpill/.no-hooks` **was** created (empty file).
-
----
-
-### Test 5 — `mcpill validate` warns when hook is absent
-
-- [ ] Delete `.claude/settings.json` and `.mcpill/.no-hooks`.
-- [ ] Run `mcpill compile --no-hooks` (ensures `.no-hooks` is present, then delete it manually).
-- [ ] Delete `.mcpill/.no-hooks`.
-- [ ] Ensure `AGENT.md` still has a `Replaces` column with values.
-- [ ] Run `mcpill validate`.
-- [ ] Verify output contains `⚠  No PreToolUse hook in .claude/settings.json`.
-- [ ] Verify the exit code is **0** (non-fatal — validate still passes).
-
----
-
-### Test 6 — `mcpill validate` skips hook check when `.no-hooks` present
-
-- [ ] Run `mcpill compile --no-hooks`.
-- [ ] Run `mcpill validate`.
-- [ ] Verify **no** `⚠  No PreToolUse hook` warning appears in output.
-
----
-
-### Test 7 — No AGENT.md: hook silently skipped
-
-- [ ] Move `AGENT.md` out of the project root (e.g. `/tmp/AGENT.md.bak`).
-- [ ] Delete `.claude/settings.json` if present.
-- [ ] Run `mcpill compile`.
-- [ ] Verify `.claude/settings.json` was **not** created.
-- [ ] Restore `AGENT.md`.
-
----
-
-### Test 8 — AGENT.md in `.mcpill/` subdirectory
-
-- [ ] Move `AGENT.md` to `.mcpill/AGENT.md`.
-- [ ] Delete `.claude/settings.json` if present.
-- [ ] Run `mcpill compile`.
-- [ ] Verify `.claude/settings.json` **was** created with the hook.
-- [ ] Move `AGENT.md` back to project root.
-
----
-
-### Test 9 — AGENT.md with no `replaces` column: no hook
-
-- [ ] Replace `AGENT.md` content with a table that has no `Replaces` column:
-  ```markdown
-  ## Tools
-  | Tool | Description |
-  |------|-------------|
-  | read-chunks | Read a file |
-  ```
-- [ ] Delete `.claude/settings.json`.
-- [ ] Run `mcpill compile`.
-- [ ] Verify `.claude/settings.json` was **not** created.
-- [ ] Restore the original `AGENT.md`.
-
----
-
-### Test 10 — Real pill smoke test (xtage or mcpster)
-
-- [ ] Run `mcpill compile` in a pill repo that already has a real `AGENT.md` with tool replacements.
-- [ ] Open `.claude/settings.json` and read the generated hook command.
-- [ ] Start a Claude Code session in that repo and deliberately use `Read` on a file.
-- [ ] Verify the hook fires and the reminder message is shown in the terminal output.
-- [ ] Confirm the message names specific MCP tools and the pill name — not a generic warning.
-
----
-
-## Open Questions
-
-- [ ] Should the hook message include a direct invocation hint (`call read-chunks with path=...`) or keep it as a passive reminder? Currently a passive reminder. Defer until Test 10 reveals whether agents change behavior.
-- [ ] Should `mcpill validate` check that the *specific matcher* in `settings.json` covers *all* declared replacements, or just that any `PreToolUse` entry exists? Currently the latter (any entry = ok). Tighten in a follow-up if agents still miss tools.
-
-## Alternatives Considered
-
-| Alternative | Pros | Cons | Decision |
-|---|---|---|---|
-| Read AGENT.md at hook-trigger time (not baked in) | Always up-to-date without recompile | I/O on every `Read`/`Bash` call; hook latency spikes | Rejected — bake at compile time |
-| Write to `~/.claude/settings.json` (global) | Works without project setup | Bleeds hook into every project on the machine | Rejected — project-scoped only |
-| Fatal error in `validate` if hook missing | Strong enforcement | Breaks projects that manage hooks separately | Rejected — non-fatal warning only |
-| Generate `AGENT.md` at compile time | Less manual work for pill authors | Inverts ownership — `replaces` is semantic knowledge only the author can declare | Rejected — user writes it manually |
-
----
-
-*Made with [mdblu](https://github.com/ruco-ai/mdblu) · source: `templates/SPEC.md.template`*

package/src/__tests__/compile.test.ts

@@ -1,203 +0,0 @@
-import { describe, it, expect, vi, afterEach } from 'vitest';
-import fs from 'fs';
-import path from 'path';
-import os from 'os';
-import { runCompile } from '../commands/compile.js';
-
-function scaffoldPill(baseDir: string, name = 'test-pill') {
-  const mcpillDir = path.join(baseDir, '.mcpill');
-  fs.mkdirSync(mcpillDir, { recursive: true });
-  fs.writeFileSync(
-    path.join(mcpillDir, 'server.md'),
-    `## Config\nname: ${name}\ntransport: stdio\n`,
-  );
-}
-
-function makeAgentMd(tools: Array<{ name: string; replaces: string }>): string {
-  const rows = tools.map((t) => `| ${t.name} | A tool | ${t.replaces} |`).join('\n');
-  return `# AGENT\n\n## Tools\n\n| Tool | Description | Replaces |\n|------|-------------|----------|\n${rows}\n`;
-}
-
-describe('runCompile — hook generation', () => {
-  const dirs: string[] = [];
-
-  function mkTmp(): string {
-    const d = fs.mkdtempSync(path.join(os.tmpdir(), 'mcpill-compile-'));
-    dirs.push(d);
-    return d;
-  }
-
-  afterEach(() => {
-    vi.restoreAllMocks();
-    for (const d of dirs.splice(0)) {
-      fs.rmSync(d, { recursive: true, force: true });
-    }
-  });
-
-  it('writes PreToolUse hook when AGENT.md has a replaces column', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    fs.writeFileSync(
-      path.join(base, 'AGENT.md'),
-      makeAgentMd([{ name: 'read-chunks', replaces: 'Read' }]),
-    );
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base });
-
-    const settingsPath = path.join(base, '.claude', 'settings.json');
-    expect(fs.existsSync(settingsPath)).toBe(true);
-    const settings = JSON.parse(fs.readFileSync(settingsPath, 'utf-8'));
-    expect(settings.hooks?.PreToolUse).toBeDefined();
-    expect(settings.hooks.PreToolUse[0].matcher).toBe('Read');
-    expect(settings.hooks.PreToolUse[0].hooks[0].command).toContain('read-chunks');
-    expect(settings.hooks.PreToolUse[0].hooks[0].command).toContain('my-pill');
-  });
-
-  it('merges hook without overwriting existing settings.json entries', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    fs.writeFileSync(
-      path.join(base, 'AGENT.md'),
-      makeAgentMd([{ name: 'read-chunks', replaces: 'Read' }]),
-    );
-
-    const claudeDir = path.join(base, '.claude');
-    fs.mkdirSync(claudeDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(claudeDir, 'settings.json'),
-      JSON.stringify({ mcpServers: { 'existing-server': { command: 'node', args: [] } } }, null, 2),
-    );
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base });
-
-    const settings = JSON.parse(fs.readFileSync(path.join(claudeDir, 'settings.json'), 'utf-8'));
-    expect(settings.mcpServers?.['existing-server']).toBeDefined();
-    expect(settings.hooks?.PreToolUse).toBeDefined();
-  });
-
-  it('builds matcher from multiple tools with overlapping replaces', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    fs.writeFileSync(
-      path.join(base, 'AGENT.md'),
-      makeAgentMd([
-        { name: 'read-chunks', replaces: 'Read' },
-        { name: 'run-query', replaces: 'Bash,Read' },
-      ]),
-    );
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base });
-
-    const settings = JSON.parse(
-      fs.readFileSync(path.join(base, '.claude', 'settings.json'), 'utf-8'),
-    );
-    const matcher: string = settings.hooks.PreToolUse[0].matcher;
-    expect(matcher.split('|').sort()).toEqual(['Bash', 'Read'].sort());
-  });
-
-  it('skips hook when AGENT.md has no replaces column', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    fs.writeFileSync(
-      path.join(base, 'AGENT.md'),
-      '# AGENT\n\n## Tools\n\n| Tool | Description |\n|------|-------------|\n| read-chunks | A tool |\n',
-    );
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base });
-
-    expect(fs.existsSync(path.join(base, '.claude', 'settings.json'))).toBe(false);
-  });
-
-  it('skips hook and writes sentinel with --no-hooks', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    fs.writeFileSync(
-      path.join(base, 'AGENT.md'),
-      makeAgentMd([{ name: 'read-chunks', replaces: 'Read' }]),
-    );
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base, noHooks: true });
-
-    expect(fs.existsSync(path.join(base, '.claude', 'settings.json'))).toBe(false);
-    expect(fs.existsSync(path.join(base, '.mcpill', '.no-hooks'))).toBe(true);
-  });
-
-  it('skips hook when no AGENT.md is present', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base });
-
-    expect(fs.existsSync(path.join(base, '.claude', 'settings.json'))).toBe(false);
-  });
-
-  it('writes PreToolUse hook from PILL.md replaces field', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    fs.writeFileSync(
-      path.join(base, 'PILL.md'),
-      `# Pill: my-pill\n\n---\n\n## Tool: read-chunks\n\ndescription: Read by chunks.\nreplaces: Read\nbehavior: |\n  impl\n`,
-    );
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base });
-
-    const settings = JSON.parse(
-      fs.readFileSync(path.join(base, '.claude', 'settings.json'), 'utf-8'),
-    );
-    expect(settings.hooks?.PreToolUse[0].matcher).toBe('Read');
-    expect(settings.hooks.PreToolUse[0].hooks[0].command).toContain('read-chunks');
-  });
-
-  it('merges PILL.md and AGENT.md tools, AGENT.md wins on name conflict', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    fs.writeFileSync(
-      path.join(base, 'PILL.md'),
-      `# Pill: my-pill\n\n---\n\n## Tool: read-chunks\n\ndescription: Read.\nreplaces: Read\nbehavior: |\n  impl\n`,
-    );
-    // AGENT.md overrides read-chunks to replace Bash instead
-    fs.writeFileSync(
-      path.join(base, 'AGENT.md'),
-      makeAgentMd([{ name: 'read-chunks', replaces: 'Bash' }]),
-    );
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base });
-
-    const settings = JSON.parse(
-      fs.readFileSync(path.join(base, '.claude', 'settings.json'), 'utf-8'),
-    );
-    const matcher: string = settings.hooks.PreToolUse[0].matcher;
-    // AGENT.md wins: Bash not Read
-    expect(matcher).toBe('Bash');
-  });
-
-  it('combines tools from PILL.md and AGENT.md when names differ', async () => {
-    const base = mkTmp();
-    scaffoldPill(base, 'my-pill');
-    fs.writeFileSync(
-      path.join(base, 'PILL.md'),
-      `# Pill: my-pill\n\n---\n\n## Tool: read-chunks\n\ndescription: Read.\nreplaces: Read\nbehavior: |\n  impl\n`,
-    );
-    fs.writeFileSync(
-      path.join(base, 'AGENT.md'),
-      makeAgentMd([{ name: 'run-query', replaces: 'Bash' }]),
-    );
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await runCompile({ dir: base });
-
-    const settings = JSON.parse(
-      fs.readFileSync(path.join(base, '.claude', 'settings.json'), 'utf-8'),
-    );
-    const matcher: string = settings.hooks.PreToolUse[0].matcher;
-    expect(matcher.split('|').sort()).toEqual(['Bash', 'Read'].sort());
-  });
-});

package/src/__tests__/init.test.ts

@@ -1,75 +0,0 @@
-import { describe, it, expect, vi, afterEach } from "vitest";
-import fs from "fs";
-import path from "path";
-import os from "os";
-import { runInit } from "../commands/init.js";
-
-describe("runInit", () => {
-  const dirs: string[] = [];
-
-  function mkTmp(): string {
-    const d = fs.mkdtempSync(path.join(os.tmpdir(), "mcpill-init-"));
-    dirs.push(d);
-    return d;
-  }
-
-  afterEach(() => {
-    vi.restoreAllMocks();
-    for (const d of dirs.splice(0)) {
-      fs.rmSync(d, { recursive: true, force: true });
-    }
-  });
-
-  it("scaffolds the expected files and directories", async () => {
-    const base = mkTmp();
-    vi.spyOn(console, "log").mockImplementation(() => {});
-
-    await runInit({ dir: base });
-
-    const mcpillDir = path.join(base, ".mcpill");
-    const serverDir = path.join(mcpillDir, "server");
-
-    expect(fs.existsSync(mcpillDir)).toBe(true);
-    expect(fs.existsSync(path.join(mcpillDir, "server.md"))).toBe(true);
-    expect(fs.existsSync(path.join(mcpillDir, "pill-agent-guide.md"))).toBe(true);
-    expect(fs.existsSync(path.join(mcpillDir, "pill-user-guide.md"))).toBe(true);
-    expect(fs.existsSync(path.join(serverDir, "mcpill.config.json"))).toBe(true);
-    expect(fs.existsSync(path.join(serverDir, "tools", "echo.md"))).toBe(true);
-    expect(fs.existsSync(path.join(serverDir, "prompts", "greeting.md"))).toBe(true);
-    expect(fs.existsSync(path.join(base, "PILL.md"))).toBe(true);
-    expect(fs.existsSync(path.join(base, "README.md"))).toBe(false);
-    expect(fs.existsSync(path.join(base, "package.json"))).toBe(false);
-
-    const config = JSON.parse(
-      fs.readFileSync(path.join(serverDir, "mcpill.config.json"), "utf-8")
-    );
-    expect(config).toMatchObject({ name: path.basename(base), transport: "stdio", port: 3333 });
-
-    const serverMd = fs.readFileSync(path.join(mcpillDir, "server.md"), "utf-8");
-    expect(serverMd).toContain(`name: ${path.basename(base)}`);
-
-    const pillMd = fs.readFileSync(path.join(base, "PILL.md"), "utf-8");
-    expect(pillMd).toContain(".mcpill/pill-agent-guide.md");
-  });
-
-  it("exits with code 1 and correct message when .mcpill/ already exists", async () => {
-    const base = mkTmp();
-    vi.spyOn(console, "log").mockImplementation(() => {});
-    vi.spyOn(console, "error").mockImplementation(() => {});
-
-    await runInit({ dir: base });
-
-    const exitSpy = vi
-      .spyOn(process, "exit")
-      .mockImplementation((_code?: number | string | null) => {
-        throw new Error("process.exit called");
-      });
-    const errSpy = vi.spyOn(console, "error").mockImplementation(() => {});
-
-    await expect(runInit({ dir: base })).rejects.toThrow("process.exit called");
-    expect(exitSpy).toHaveBeenCalledWith(1);
-    expect(errSpy).toHaveBeenCalledWith(
-      ".mcpill/ already exists. Remove it manually to re-init."
-    );
-  });
-});

package/src/__tests__/loaders/config.test.ts

@@ -1,54 +0,0 @@
-import { describe, it, expect, afterEach } from "vitest";
-import fs from "fs";
-import path from "path";
-import os from "os";
-import { loadConfig } from "../../loaders/config.js";
-
-describe("loadConfig", () => {
-  const dirs: string[] = [];
-
-  function mkTmp(): string {
-    const d = fs.mkdtempSync(path.join(os.tmpdir(), "mcpill-config-"));
-    dirs.push(d);
-    return d;
-  }
-
-  afterEach(() => {
-    for (const d of dirs.splice(0)) {
-      fs.rmSync(d, { recursive: true, force: true });
-    }
-  });
-
-  it("returns defaults when no mcpill.config.json exists", async () => {
-    const dir = mkTmp();
-    const config = await loadConfig(dir);
-    expect(config).toEqual({ name: "mcpill-server", transport: "stdio", port: 3333 });
-  });
-
-  it("file values override defaults", async () => {
-    const dir = mkTmp();
-    fs.writeFileSync(
-      path.join(dir, "mcpill.config.json"),
-      JSON.stringify({ name: "my-server", port: 4000 })
-    );
-    const config = await loadConfig(dir);
-    expect(config).toEqual({ name: "my-server", transport: "stdio", port: 4000 });
-  });
-
-  it("CLI opts override file values", async () => {
-    const dir = mkTmp();
-    fs.writeFileSync(
-      path.join(dir, "mcpill.config.json"),
-      JSON.stringify({ transport: "http", port: 4444 })
-    );
-    const config = await loadConfig(dir);
-
-    // Simulate CLI opt override (same merge logic used in runServer)
-    const cliOpts = { transport: "stdio" as const, port: 9999 };
-    const finalTransport = cliOpts.transport ?? config.transport;
-    const finalPort = cliOpts.port ?? config.port;
-
-    expect(finalTransport).toBe("stdio");
-    expect(finalPort).toBe(9999);
-  });
-});