akm-cli 0.7.0-rc1 → 0.7.1

package/dist/tests/commands/search.test.js

@@ -1,537 +0,0 @@
-import { afterAll, afterEach, beforeEach, describe, expect, test } from "bun:test";
-import fs from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { akmSearch } from "../../src/commands/search";
-import { saveConfig } from "../../src/core/config";
-import { akmIndex } from "../../src/indexer/indexer";
-import { createWiki, stashRaw } from "../../src/wiki/wiki";
-// ── Temp directory tracking ─────────────────────────────────────────────────
-const createdTmpDirs = [];
-function expectDefined(value) {
-    expect(value).toBeDefined();
-    if (value === undefined || value === null) {
-        throw new Error("Expected value to be defined");
-    }
-    return value;
-}
-function createTmpDir(prefix = "akm-search-") {
-    const dir = fs.mkdtempSync(path.join(os.tmpdir(), prefix));
-    createdTmpDirs.push(dir);
-    return dir;
-}
-afterAll(() => {
-    for (const dir of createdTmpDirs) {
-        fs.rmSync(dir, { recursive: true, force: true });
-    }
-});
-// ── Helpers ─────────────────────────────────────────────────────────────────
-function writeFile(filePath, content = "") {
-    fs.mkdirSync(path.dirname(filePath), { recursive: true });
-    fs.writeFileSync(filePath, content);
-}
-/**
- * Create a stash directory with all required subdirectories.
- */
-function tmpStash() {
-    const dir = createTmpDir("akm-search-stash-");
-    for (const sub of ["skills", "commands", "agents", "knowledge", "scripts"]) {
-        fs.mkdirSync(path.join(dir, sub), { recursive: true });
-    }
-    return dir;
-}
-/**
- * Build an index for a stash directory from a set of files and their content.
- * Also writes a config with semanticSearchMode disabled so embedding is not attempted.
- */
-async function buildTestIndex(stashDir, files) {
-    for (const [relPath, content] of Object.entries(files)) {
-        const fullPath = path.join(stashDir, relPath);
-        fs.mkdirSync(path.dirname(fullPath), { recursive: true });
-        fs.writeFileSync(fullPath, content);
-    }
-    process.env.AKM_STASH_DIR = stashDir;
-    saveConfig({ semanticSearchMode: "off" });
-    await akmIndex({ stashDir, full: true });
-}
-// ── Environment isolation ───────────────────────────────────────────────────
-const originalXdgCacheHome = process.env.XDG_CACHE_HOME;
-const originalXdgConfigHome = process.env.XDG_CONFIG_HOME;
-const originalAkmStashDir = process.env.AKM_STASH_DIR;
-let testCacheDir = "";
-let testConfigDir = "";
-beforeEach(() => {
-    testCacheDir = createTmpDir("akm-search-cache-");
-    testConfigDir = createTmpDir("akm-search-config-");
-    process.env.XDG_CACHE_HOME = testCacheDir;
-    process.env.XDG_CONFIG_HOME = testConfigDir;
-});
-afterEach(() => {
-    if (originalXdgCacheHome === undefined) {
-        delete process.env.XDG_CACHE_HOME;
-    }
-    else {
-        process.env.XDG_CACHE_HOME = originalXdgCacheHome;
-    }
-    if (originalXdgConfigHome === undefined) {
-        delete process.env.XDG_CONFIG_HOME;
-    }
-    else {
-        process.env.XDG_CONFIG_HOME = originalXdgConfigHome;
-    }
-    if (originalAkmStashDir === undefined) {
-        delete process.env.AKM_STASH_DIR;
-    }
-    else {
-        process.env.AKM_STASH_DIR = originalAkmStashDir;
-    }
-    if (testCacheDir) {
-        fs.rmSync(testCacheDir, { recursive: true, force: true });
-        testCacheDir = "";
-    }
-    if (testConfigDir) {
-        fs.rmSync(testConfigDir, { recursive: true, force: true });
-        testConfigDir = "";
-    }
-});
-// ── 2.1 Database search path (FTS scoring) ──────────────────────────────────
-describe("Database search path (FTS scoring)", () => {
-    test("registered external wiki hits use canonical wiki refs and actions", async () => {
-        const stashDir = tmpStash();
-        const externalWiki = createTmpDir("akm-search-ext-wiki-");
-        writeFile(path.join(externalWiki, "tools", "documentation", "how-to", "001-get-started-with-ics-documentation.md"), "---\ndescription: Documentation getting started guide\n---\n# Start\n");
-        process.env.AKM_STASH_DIR = stashDir;
-        saveConfig({
-            semanticSearchMode: "off",
-            sources: [{ type: "filesystem", path: externalWiki, name: "ics-docs", wikiName: "ics-docs" }],
-        });
-        await akmIndex({ stashDir, full: true });
-        const result = await akmSearch({ query: "documentation", type: "wiki", source: "stash" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        const wikiHit = expectDefined(localHits.find((hit) => hit.ref === "wiki:ics-docs/tools/documentation/how-to/001-get-started-with-ics-documentation"));
-        expect(wikiHit.origin).toBeNull();
-        expect(wikiHit.action).toBe("akm show wiki:ics-docs/tools/documentation/how-to/001-get-started-with-ics-documentation -> read the wiki page");
-    });
-    test("stash-owned raw wiki pages are discoverable in global wiki search", async () => {
-        const stashDir = tmpStash();
-        process.env.AKM_STASH_DIR = stashDir;
-        saveConfig({ semanticSearchMode: "off" });
-        createWiki(stashDir, "notes");
-        stashRaw({
-            stashDir,
-            wikiName: "notes",
-            content: "# Hello World\n\nThis raw page should be searchable.\n",
-            preferredName: "hello-world",
-        });
-        await akmIndex({ stashDir, full: true });
-        const result = await akmSearch({ query: "hello world", type: "wiki", source: "stash" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits.some((hit) => hit.ref === "wiki:notes/raw/hello-world")).toBe(true);
-    });
-    test("FTS search returns scored results for matching query", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "deploy", "deploy.sh"), "#!/bin/bash\necho deploy\n");
-        writeFile(path.join(stashDir, "scripts", "deploy", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "deploy",
-                    type: "script",
-                    description: "Deploy application to production servers",
-                    filename: "deploy.sh",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "deploy", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits.length).toBeGreaterThanOrEqual(1);
-        const deployHit = localHits.find((h) => h.name === "deploy");
-        const resolvedDeployHit = expectDefined(deployHit);
-        expect(resolvedDeployHit.ref).toContain("script:deploy");
-        expect(resolvedDeployHit.action).toContain("akm show");
-        expect(resolvedDeployHit.size).toBeDefined();
-        expect(resolvedDeployHit.score).toBeDefined();
-        expect(resolvedDeployHit.score).toBeGreaterThan(0);
-    });
-    test("FTS search filters by asset type", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "lint", "lint.sh"), "#!/bin/bash\necho lint\n");
-        writeFile(path.join(stashDir, "scripts", "lint", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "lint",
-                    type: "script",
-                    description: "Lint source code for errors",
-                    filename: "lint.sh",
-                },
-            ],
-        }));
-        writeFile(path.join(stashDir, "skills", "code-review", "SKILL.md"), "# Code Review\nReview code for quality.\n");
-        writeFile(path.join(stashDir, "skills", "code-review", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "code-review",
-                    type: "skill",
-                    description: "Review code for quality issues",
-                    filename: "SKILL.md",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "code", type: "script", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        for (const hit of localHits) {
-            expect(hit.type).toBe("script");
-        }
-    });
-    test("empty query returns all entries", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "alpha", "alpha.sh"), "#!/bin/bash\necho alpha\n");
-        writeFile(path.join(stashDir, "scripts", "alpha", ".stash.json"), JSON.stringify({
-            entries: [{ name: "alpha", type: "script", description: "Alpha tool", filename: "alpha.sh" }],
-        }));
-        writeFile(path.join(stashDir, "scripts", "beta", "beta.sh"), "#!/bin/bash\necho beta\n");
-        writeFile(path.join(stashDir, "scripts", "beta", ".stash.json"), JSON.stringify({
-            entries: [{ name: "beta", type: "script", description: "Beta tool", filename: "beta.sh" }],
-        }));
-        writeFile(path.join(stashDir, "scripts", "gamma", "gamma.sh"), "#!/bin/bash\necho gamma\n");
-        writeFile(path.join(stashDir, "scripts", "gamma", ".stash.json"), JSON.stringify({
-            entries: [{ name: "gamma", type: "script", description: "Gamma tool", filename: "gamma.sh" }],
-        }));
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits.length).toBe(3);
-    });
-    test('query "." enumerates indexed agent assets when the type filter is agent', async () => {
-        const stashDir = tmpStash();
-        await buildTestIndex(stashDir, {
-            "skills/plugins/dotnet-msbuild/agents/msbuild-code-review.agent.md": "---\ndescription: MSBuild code review agent\nmodel: gpt-5\n---\nReview MSBuild files\n",
-            "skills/plugins/dotnet-msbuild/agents/msbuild.agent.md": "---\ndescription: MSBuild agent\nmodel: gpt-5\n---\nHelp with MSBuild\n",
-        });
-        const result = await akmSearch({ query: ".", type: "agent", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits).toHaveLength(2);
-        expect(localHits.map((hit) => hit.name).sort()).toEqual([
-            "skills/plugins/dotnet-msbuild/agents/msbuild-code-review.agent",
-            "skills/plugins/dotnet-msbuild/agents/msbuild.agent",
-        ]);
-        expect(localHits.every((hit) => hit.type === "agent")).toBe(true);
-        expect(localHits.every((hit) => hit.ref.startsWith("agent:"))).toBe(true);
-        expect(localHits.map((hit) => hit.ref.slice("agent:".length)).sort()).toEqual(localHits.map((hit) => hit.name).sort());
-        expect(localHits.every((hit) => hit.path.endsWith(".agent.md"))).toBe(true);
-    });
-    test("limit parameter caps results", async () => {
-        const stashDir = tmpStash();
-        const names = ["aaa", "bbb", "ccc", "ddd", "eee"];
-        for (const name of names) {
-            writeFile(path.join(stashDir, "scripts", name, `${name}.sh`), `#!/bin/bash\necho ${name}\n`);
-            writeFile(path.join(stashDir, "scripts", name, ".stash.json"), JSON.stringify({
-                entries: [{ name, type: "script", description: `${name} tool for testing`, filename: `${name}.sh` }],
-            }));
-        }
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "", limit: 3, source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits.length).toBe(3);
-    });
-    test("scores use multiplicative boosts without clamping", async () => {
-        const stashDir = tmpStash();
-        // Create an entry with tags, searchHints, and name all matching the query
-        writeFile(path.join(stashDir, "scripts", "clamp-deploy", "clamp-deploy.sh"), "#!/bin/bash\necho deploy\n");
-        writeFile(path.join(stashDir, "scripts", "clamp-deploy", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "clamp-deploy",
-                    type: "script",
-                    description: "Deploy deploy deploy application",
-                    tags: ["deploy", "deployment"],
-                    searchHints: ["deploy services", "deploy to production"],
-                    filename: "clamp-deploy.sh",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "deploy", source: "local" });
-        expect(result.hits.length).toBeGreaterThan(0);
-        const deployHit = result.hits.find((h) => h.name === "clamp-deploy");
-        const resolvedDeployHit = expectDefined(deployHit);
-        expect(resolvedDeployHit.score).toBeDefined();
-        expect(resolvedDeployHit.score).toBeGreaterThan(0);
-    });
-});
-// ── 2.2 Score boosts ────────────────────────────────────────────────────────
-describe("Score boosts", () => {
-    test("tag match boosts score", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "deploy", "deploy.sh"), "#!/bin/bash\necho deploy\n");
-        writeFile(path.join(stashDir, "scripts", "deploy", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "deploy",
-                    type: "script",
-                    description: "Deploy application",
-                    tags: ["deploy", "production"],
-                    filename: "deploy.sh",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "deploy", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        const deployHit = localHits.find((h) => h.name === "deploy");
-        const resolvedDeployHit = expectDefined(deployHit);
-        expect(resolvedDeployHit.whyMatched).toBeDefined();
-        expect(resolvedDeployHit.whyMatched).toContain("matched tags");
-    });
-    test("name match boosts score", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "formatter", "formatter.sh"), "#!/bin/bash\necho format\n");
-        writeFile(path.join(stashDir, "scripts", "formatter", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "formatter",
-                    type: "script",
-                    description: "Format source files",
-                    filename: "formatter.sh",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "formatter", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        const hit = localHits.find((h) => h.name === "formatter");
-        const resolvedHit = expectDefined(hit);
-        expect(resolvedHit.whyMatched).toBeDefined();
-        // Exact name match is now reported as "exact name match" or "near-exact name match"
-        const hasNameMatch = resolvedHit.whyMatched?.some((r) => r.includes("name match") || r.includes("name tokens"));
-        expect(hasNameMatch).toBe(true);
-    });
-    test("curated metadata gets quality boost", async () => {
-        const stashDir = tmpStash();
-        // Curated entry (quality absent or "curated")
-        writeFile(path.join(stashDir, "scripts", "curated", "curated.sh"), "#!/bin/bash\necho curated\n");
-        writeFile(path.join(stashDir, "scripts", "curated", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "curated",
-                    type: "script",
-                    description: "A testing utility",
-                    quality: "curated",
-                    filename: "curated.sh",
-                },
-            ],
-        }));
-        // Generated entry — identical description so FTS score is the same;
-        // only the `quality` field differs, isolating the curated boost.
-        writeFile(path.join(stashDir, "scripts", "generated", "generated.sh"), "#!/bin/bash\necho generated\n");
-        writeFile(path.join(stashDir, "scripts", "generated", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "generated",
-                    type: "script",
-                    description: "A testing utility",
-                    quality: "generated",
-                    filename: "generated.sh",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "testing utility", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        const curatedHit = localHits.find((h) => h.name === "curated");
-        const generatedHit = localHits.find((h) => h.name === "generated");
-        const resolvedCuratedHit = expectDefined(curatedHit);
-        const resolvedGeneratedHit = expectDefined(generatedHit);
-        expect(resolvedCuratedHit.score).toBeDefined();
-        expect(resolvedGeneratedHit.score).toBeDefined();
-        // Scores are rounded to 2 decimal places, so small boosts may tie.
-        // Verify curated ranks at least as high (sort order preserves pre-rounding order).
-        expect(resolvedCuratedHit.score).toBeGreaterThanOrEqual(resolvedGeneratedHit.score);
-        expect(resolvedCuratedHit.whyMatched).toBeDefined();
-        expect(resolvedCuratedHit.whyMatched).toContain("curated metadata boost");
-    });
-});
-// ── 2.3 Substring fallback ──────────────────────────────────────────────────
-describe("Substring fallback", () => {
-    test("falls back to substring search when no index exists", async () => {
-        const stashDir = tmpStash();
-        // Do NOT call akmIndex — just create files on disk
-        writeFile(path.join(stashDir, "scripts", "deploy", "deploy.sh"), "#!/bin/bash\necho deploy\n");
-        process.env.AKM_STASH_DIR = stashDir;
-        saveConfig({ semanticSearchMode: "off" });
-        const result = await akmSearch({ query: "deploy", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits.length).toBeGreaterThanOrEqual(1);
-        const deployHit = localHits.find((h) => h.name.includes("deploy"));
-        expect(deployHit).toBeDefined();
-        // Substring fallback computes a relevance score but has no whyMatched
-        expect(deployHit?.score).toBeGreaterThan(0);
-        expect(deployHit?.score).toBeLessThanOrEqual(1);
-        expect(deployHit?.whyMatched).toBeUndefined();
-    });
-    test("substring search is case-insensitive", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "Deploy", "Deploy.sh"), "#!/bin/bash\necho deploy\n");
-        process.env.AKM_STASH_DIR = stashDir;
-        saveConfig({ semanticSearchMode: "off" });
-        // Do NOT call akmIndex
-        const result = await akmSearch({ query: "deploy", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits.length).toBeGreaterThanOrEqual(1);
-        const hit = localHits.find((h) => h.name.toLowerCase().includes("deploy"));
-        expect(hit).toBeDefined();
-    });
-    test("substring fallback searches descriptions and returns them", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "agents", "agentic-systems-architect.md"), "---\ndescription: Designs agent coordination patterns and context assembly\n---\nYou are an architect.\n");
-        process.env.AKM_STASH_DIR = stashDir;
-        saveConfig({ semanticSearchMode: "off" });
-        const result = await akmSearch({ query: "coordination", type: "agent", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits).toHaveLength(1);
-        expect(localHits[0]?.name).toBe("agentic-systems-architect");
-        expect(localHits[0]?.description).toContain("agent coordination patterns");
-    });
-    test("nested markdown files under agents/ are indexed as agent assets", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "agent-stash", "agents", "blog", "topic-discovery.md"), [
-            "---",
-            "type: agent",
-            "mode: subagent",
-            "description: Discovers blog topics from source material",
-            "---",
-            "You are a blog topic discovery agent.",
-        ].join("\n"));
-        process.env.AKM_STASH_DIR = stashDir;
-        saveConfig({ semanticSearchMode: "off" });
-        const result = await akmSearch({ query: "blog topics", type: "agent", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits).toHaveLength(1);
-        expect(localHits[0]?.type).toBe("agent");
-        expect(localHits[0]?.name).toBe("agent-stash/agents/blog/topic-discovery");
-        expect(localHits[0]?.description).toContain("blog topics");
-        expect(localHits[0]?.ref).toBe("agent:agent-stash/agents/blog/topic-discovery");
-    });
-    test("substring fallback honors curated .stash.json metadata", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "doctor", "doctor.sh"), "#!/bin/bash\necho doctor\n");
-        writeFile(path.join(stashDir, "scripts", "doctor", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "doctor",
-                    type: "script",
-                    description: "Diagnose workspace health issues",
-                    tags: ["health", "diagnostics"],
-                    filename: "doctor.sh",
-                },
-            ],
-        }));
-        process.env.AKM_STASH_DIR = stashDir;
-        saveConfig({ semanticSearchMode: "off" });
-        const result = await akmSearch({ query: "diagnostics", source: "local" });
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        const doctorHit = localHits.find((h) => h.name === "doctor");
-        expect(doctorHit).toBeDefined();
-        expect(doctorHit?.description).toBe("Diagnose workspace health issues");
-        expect(doctorHit?.tags).toContain("diagnostics");
-    });
-});
-// ── 2.4 Source filtering ────────────────────────────────────────────────────
-describe("Source filtering", () => {
-    test("source: local skips registry search", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "local-tool", "local-tool.sh"), "#!/bin/bash\necho local\n");
-        writeFile(path.join(stashDir, "scripts", "local-tool", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "local-tool",
-                    type: "script",
-                    description: "A local tool",
-                    filename: "local-tool.sh",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        const result = await akmSearch({ query: "local", source: "local" });
-        // All hits should be local, no registry hits
-        for (const hit of result.hits) {
-            expect(hit.type).not.toBe("registry");
-            if (hit.type !== "registry") {
-                expect(hit.origin).toBeNull();
-                expect(hit.action).toContain("akm show");
-            }
-        }
-        // No warnings from registry search failures
-        expect(result.warnings).toBeUndefined();
-    });
-    test("source: registry skips local search", async () => {
-        const stashDir = createTmpDir();
-        // Create a local tool so we know local hits would exist if local were searched
-        writeFile(path.join(stashDir, "scripts", "deploy.sh"), "#!/bin/bash\necho deploy\n");
-        process.env.AKM_STASH_DIR = stashDir;
-        saveConfig({ semanticSearchMode: "off", registries: [] });
-        const result = await akmSearch({ query: "deploy", source: "registry" });
-        // Registry source puts results in registryHits, hits is empty
-        expect(result.source).toBe("registry");
-        expect(result.hits.length).toBe(0);
-        if (result.registryHits) {
-            for (const hit of result.registryHits) {
-                expect(hit.type).toBe("registry");
-            }
-        }
-    });
-    test("source: both includes local hits without crashing", async () => {
-        const stashDir = createTmpDir();
-        writeFile(path.join(stashDir, "scripts", "merge-test.sh"), "#!/bin/bash\necho merge\n");
-        await buildTestIndex(stashDir, {});
-        saveConfig({ semanticSearchMode: "off", registries: [] });
-        const result = await akmSearch({ query: "merge", source: "both" });
-        expect(result.source).toBe("both");
-        // Should have at least the local hit
-        const localHits = result.hits.filter((h) => h.type !== "registry");
-        expect(localHits.length).toBeGreaterThan(0);
-    });
-});
-// ── 2.5 Edge cases ──────────────────────────────────────────────────────────
-describe("Edge cases", () => {
-    test("search with special characters does not crash", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "safe", "safe.sh"), "#!/bin/bash\necho safe\n");
-        writeFile(path.join(stashDir, "scripts", "safe", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "safe",
-                    type: "script",
-                    description: "A safe tool",
-                    filename: "safe.sh",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        // Should not throw
-        const result = await akmSearch({ query: "<script>", source: "local" });
-        expect(result).toBeDefined();
-        expect(result.hits).toBeDefined();
-    });
-    test("search with very long query", async () => {
-        const stashDir = tmpStash();
-        writeFile(path.join(stashDir, "scripts", "simple", "simple.sh"), "#!/bin/bash\necho simple\n");
-        writeFile(path.join(stashDir, "scripts", "simple", ".stash.json"), JSON.stringify({
-            entries: [
-                {
-                    name: "simple",
-                    type: "script",
-                    description: "A simple tool",
-                    filename: "simple.sh",
-                },
-            ],
-        }));
-        await buildTestIndex(stashDir, {});
-        const longQuery = "a".repeat(10_000);
-        // Should not throw
-        const result = await akmSearch({ query: longQuery, source: "local" });
-        expect(result).toBeDefined();
-        expect(result.hits).toBeDefined();
-    });
-});

package/dist/tests/commands/show-indexer-parity.test.js

@@ -1,117 +0,0 @@
-/**
- * Parity regression for Phase 4 (spec §10 step 4 / §6.2).
- *
- * `akm show` now consults `indexer.lookup(ref)` first, then reads the file
- * from disk. The risk called out in the v1 implementation plan is that
- * origin-prefixed refs (e.g. `local//skill:foo`) silently regress when the
- * indexer is consulted instead of the directory walker.
- *
- * This test pins both forms — bare ref and origin-prefixed ref — and asserts
- * that `indexer.lookup` returns the same on-disk path that `akmShowUnified`
- * resolves to. If a future refactor changes how the indexer keys assets, this
- * test fails fast instead of silently breaking show for installed sources.
- */
-import { afterAll, afterEach, beforeEach, describe, expect, test } from "bun:test";
-import fs from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { akmShowUnified } from "../../src/commands/show";
-import { parseAssetRef } from "../../src/core/asset-ref";
-import { resetConfigCache, saveConfig } from "../../src/core/config";
-import { akmIndex, lookup } from "../../src/indexer/indexer";
-import "../../src/sources/providers/index";
-const createdTmpDirs = [];
-function createTmpDir(prefix = "akm-parity-") {
-    const dir = fs.mkdtempSync(path.join(os.tmpdir(), prefix));
-    createdTmpDirs.push(dir);
-    return dir;
-}
-function writeFile(filePath, content) {
-    fs.mkdirSync(path.dirname(filePath), { recursive: true });
-    fs.writeFileSync(filePath, content, "utf8");
-}
-const originalXdgCacheHome = process.env.XDG_CACHE_HOME;
-const originalXdgConfigHome = process.env.XDG_CONFIG_HOME;
-const originalStashDir = process.env.AKM_STASH_DIR;
-let stashDir = "";
-beforeEach(() => {
-    process.env.XDG_CACHE_HOME = createTmpDir("akm-parity-cache-");
-    process.env.XDG_CONFIG_HOME = createTmpDir("akm-parity-config-");
-    stashDir = createTmpDir("akm-parity-stash-");
-    for (const sub of ["scripts", "skills", "commands", "agents", "knowledge"]) {
-        fs.mkdirSync(path.join(stashDir, sub), { recursive: true });
-    }
-    process.env.AKM_STASH_DIR = stashDir;
-    resetConfigCache();
-    saveConfig({ semanticSearchMode: "off" });
-    resetConfigCache();
-});
-afterEach(() => {
-    if (originalXdgCacheHome === undefined)
-        delete process.env.XDG_CACHE_HOME;
-    else
-        process.env.XDG_CACHE_HOME = originalXdgCacheHome;
-    if (originalXdgConfigHome === undefined)
-        delete process.env.XDG_CONFIG_HOME;
-    else
-        process.env.XDG_CONFIG_HOME = originalXdgConfigHome;
-    if (originalStashDir === undefined)
-        delete process.env.AKM_STASH_DIR;
-    else
-        process.env.AKM_STASH_DIR = originalStashDir;
-});
-afterAll(() => {
-    for (const dir of createdTmpDirs) {
-        fs.rmSync(dir, { recursive: true, force: true });
-    }
-});
-describe("Phase 4 parity: indexer.lookup ↔ akmShowUnified", () => {
-    test("indexed asset: lookup() returns the same file akmShow renders", async () => {
-        const skillBody = [
-            "---",
-            "name: parity-skill",
-            "description: A skill used to verify Phase 4 parity",
-            "---",
-            "# Parity skill",
-            "",
-            "Body content used by the parity test.",
-        ].join("\n");
-        // Skills live at skills/<name>/SKILL.md (see asset-spec.ts)
-        writeFile(path.join(stashDir, "skills", "parity-skill", "SKILL.md"), skillBody);
-        await akmIndex({ stashDir, full: true });
-        const ref = "skill:parity-skill";
-        const parsed = parseAssetRef(ref);
-        const indexed = await lookup(parsed);
-        expect(indexed).not.toBeNull();
-        if (!indexed)
-            return;
-        expect(indexed.type).toBe("skill");
-        expect(indexed.name).toBe("parity-skill");
-        // Reading the indexer-resolved path should yield the on-disk content.
-        const fileBody = fs.readFileSync(indexed.filePath, "utf8");
-        expect(fileBody).toBe(skillBody);
-        // akmShow returns the same path in its rendered response.
-        const shown = await akmShowUnified({ ref });
-        expect(shown.path).toBe(indexed.filePath);
-    });
-    test("origin-prefixed ref: local//skill:foo resolves to primary stash path", async () => {
-        const body = ["---", "name: origin-skill", "description: Test", "---", "# origin"].join("\n");
-        writeFile(path.join(stashDir, "skills", "origin-skill", "SKILL.md"), body);
-        await akmIndex({ stashDir, full: true });
-        const bare = await lookup(parseAssetRef("skill:origin-skill"));
-        const local = await lookup(parseAssetRef("local//skill:origin-skill"));
-        expect(bare).not.toBeNull();
-        expect(local).not.toBeNull();
-        expect(local?.filePath).toBe(bare?.filePath);
-        // Show parity for both ref forms.
-        const shownBare = await akmShowUnified({ ref: "skill:origin-skill" });
-        const shownLocal = await akmShowUnified({ ref: "local//skill:origin-skill" });
-        expect(shownBare.path).toBe(shownLocal.path);
-        expect(shownBare.path).toBe(bare?.filePath);
-    });
-    test("missing asset: lookup returns null", async () => {
-        await akmIndex({ stashDir, full: true });
-        const result = await lookup(parseAssetRef("skill:does-not-exist"));
-        expect(result).toBeNull();
-    });
-});