akm-cli 0.6.0-rc1 → 0.6.0-rc2

package/dist/{metadata.js → indexer/metadata.js}

@@ -1,10 +1,10 @@
 import fs from "node:fs";
 import path from "node:path";
-import { deriveCanonicalAssetName, deriveCanonicalAssetNameFromStashRoot, isRelevantAssetFile } from "./asset-spec";
-import { isAssetType } from "./common";
+import { deriveCanonicalAssetName, deriveCanonicalAssetNameFromStashRoot, isRelevantAssetFile, } from "../core/asset-spec";
+import { isAssetType } from "../core/common";
+import { parseFrontmatter, toStringOrUndefined } from "../core/frontmatter";
+import { warn } from "../core/warn";
 import { buildFileContext, buildRenderContext, getRenderer, runMatchers } from "./file-context";
-import { parseFrontmatter, toStringOrUndefined } from "./frontmatter";
-import { warn } from "./warn";
 // ── Load / Write ────────────────────────────────────────────────────────────
 const STASH_FILENAME = ".stash.json";
 export function stashFilePath(dirPath) {
@@ -568,7 +568,11 @@ export async function generateMetadataFlat(stashRoot, files) {
 }
 function buildMetadataSkipWarning(filePath, assetType, error) {
     const detail = error instanceof Error ? error.message : String(error);
-    const warning = `Skipped malformed ${assetType} asset at ${filePath}: ${detail}`;
+    // Workflow errors are already multi-line `path:line — message` blocks; print
+    // them as-is so the author sees a flat list without a redundant prefix.
+    const warning = assetType === "workflow"
+        ? `Skipped workflow ${filePath}:\n${detail}`
+        : `Skipped malformed ${assetType} asset at ${filePath}: ${detail}`;
     warn(warning);
     return warning;
 }

package/dist/{search-source.js → indexer/search-source.js}

@@ -1,10 +1,14 @@
 import fs from "node:fs";
 import path from "node:path";
-import { resolveStashDir } from "./common";
-import { loadConfig } from "./config";
-import { ensureGitMirror, getCachePaths, parseGitRepoUrl } from "./stash-providers/git";
-import { ensureWebsiteMirror, getCachePaths as getWebsiteCachePaths } from "./stash-providers/website";
-import { warn } from "./warn";
+import { resolveStashDir } from "../core/common";
+import { loadConfig } from "../core/config";
+import { resolveSourceProviderFactory } from "../sources/provider-factory";
+// Eager side-effect imports so all built-in source providers self-register
+// before resolveEntryContentDir() runs.
+import "../sources/providers/index";
+import { warn } from "../core/warn";
+import { ensureGitMirror, getCachePaths, parseGitRepoUrl } from "../sources/providers/git";
+import { ensureWebsiteMirror } from "../sources/providers/website";
 // Legacy "context-hub" / "github" type aliases are normalized to "git" at
 // config-load time (see src/config.ts), so this set only contains the canonical
 // type.
@@ -17,7 +21,7 @@ const GIT_STASH_TYPES = new Set(["git"]);
  *   1. The primary stash directory (the entry marked `primary: true`, or the
  *      legacy top-level `stashDir`). Always emitted, even when the directory
  *      does not yet exist on disk, so callers can use it as the clone target.
- *   2. Each entry in `config.stashes[]` (in declared order), excluding the
+ *   2. Each entry in `config.sources ?? config.stashes[]` (in declared order), excluding the
  *      one already emitted as the primary.
  *   3. Each entry in `config.installed[]` (registry-managed stashes).
  *
@@ -25,7 +29,7 @@ const GIT_STASH_TYPES = new Set(["git"]);
  * for each provider kind. Disabled entries (`enabled: false`) and entries
  * whose disk path doesn't exist are filtered after deduplication.
  */
-export function resolveStashSources(overrideStashDir, existingConfig) {
+export function resolveSourceEntries(overrideStashDir, existingConfig) {
     const stashDir = overrideStashDir ?? resolveStashDir();
     const config = existingConfig ?? loadConfig();
     const sources = [{ path: stashDir }];
@@ -49,7 +53,7 @@ export function resolveStashSources(overrideStashDir, existingConfig) {
     // (1) + (2) Single pass over declared stashes — primary first if present,
     // then the rest in declared order. The primary's directory is already
     // injected as `sources[0]` above, so we only need to dedupe the source set.
-    const stashes = config.stashes ?? [];
+    const stashes = config.sources ?? config.stashes ?? [];
     const primaryIdx = stashes.findIndex((entry) => entry.primary === true);
     const ordered = [];
     if (primaryIdx >= 0) {
@@ -78,43 +82,54 @@ export function resolveStashSources(overrideStashDir, existingConfig) {
 }
 /**
  * Resolve the content directory the indexer should walk for a given config
- * entry. Returns `undefined` if the entry has no walkable content (e.g. an
- * `openviking` remote stash) so the caller can skip it.
+ * entry. Returns `undefined` if the entry has no walkable content
+ * so the caller can skip it.
+ *
+ * Single source of truth: each provider owns its own path. We instantiate the
+ * registered {@link import("../sources/provider").SourceProvider} for the entry
+ * and call `provider.path()`. This replaces the old per-kind switch ladder
+ * (filesystem path / git cache / website cache) that lived here in 0.6.0 —
+ * see spec §10 step 4 and §7 "Removed from 0.6.0".
+ *
+ * The git case still does one extra step: the provider returns the cloned
+ * repo dir, but the indexer walks the `content/` subdirectory inside it.
+ * That convention is part of the akm content layout, not a provider concern,
+ * so it stays here.
  */
 function resolveEntryContentDir(entry) {
-    if (entry.type === "filesystem" && entry.path) {
-        return entry.path;
+    const factory = resolveSourceProviderFactory(entry.type);
+    if (!factory)
+        return undefined;
+    let provider;
+    try {
+        provider = factory(entry);
     }
-    if (GIT_STASH_TYPES.has(entry.type) && entry.url) {
-        try {
-            const repo = parseGitRepoUrl(entry.url);
-            const cachePaths = getCachePaths(repo.canonicalUrl);
-            // The content/ subdirectory inside the extracted repo is the actual
-            // stash root containing DOC.md / SKILL.md files that the walker indexes.
-            return path.join(cachePaths.repoDir, "content");
-        }
-        catch (err) {
-            warn(`Warning: failed to resolve git stash cache for "${entry.url}": ${err instanceof Error ? err.message : String(err)}`);
-            return undefined;
-        }
+    catch (err) {
+        warn(`Warning: failed to construct ${entry.type} source provider for "${entry.name ?? entry.url ?? entry.path}": ${err instanceof Error ? err.message : String(err)}`);
+        return undefined;
     }
-    if (entry.type === "website" && entry.url) {
-        try {
-            return getWebsiteCachePaths(entry.url).stashDir;
-        }
-        catch (err) {
-            warn(`Warning: failed to resolve website stash cache for "${entry.url}": ${err instanceof Error ? err.message : String(err)}`);
-            return undefined;
-        }
+    let dir;
+    try {
+        dir = provider.path();
     }
-    // Remote-only providers (openviking) have no walkable directory.
-    return undefined;
+    catch (err) {
+        warn(`Warning: failed to resolve ${entry.type} source path for "${entry.name ?? entry.url ?? entry.path}": ${err instanceof Error ? err.message : String(err)}`);
+        return undefined;
+    }
+    // Git providers expose the cloned repo root as their path. The akm content
+    // layout puts indexable files under `<repo>/content/`, so the walker needs
+    // that subdirectory. This is a content-layout convention, not a provider
+    // capability — keep it here.
+    if (GIT_STASH_TYPES.has(entry.type)) {
+        return path.join(dir, "content");
+    }
+    return dir;
 }
 /**
  * Convenience: returns just the directory paths, preserving priority order.
  */
 export function resolveAllStashDirs(overrideStashDir) {
-    return resolveStashSources(overrideStashDir).map((s) => s.path);
+    return resolveSourceEntries(overrideStashDir).map((s) => s.path);
 }
 /**
  * Find which source a file path belongs to.
@@ -204,10 +219,10 @@ function isValidDirectory(dir) {
 /**
  * Ensure all cache-backed stash providers are refreshed so their cache
  * directories exist on disk. Must be called (async) before
- * `resolveStashSources()` so the content directories pass the
+ * `resolveSourceEntries()` so the content directories pass the
  * `isValidDirectory()` check.
  */
-export async function ensureStashCaches(config) {
+export async function ensureSourceCaches(config) {
     const cfg = config ?? loadConfig();
     for (const entry of cfg.stashes ?? []) {
         if (!GIT_STASH_TYPES.has(entry.type) || !entry.url || entry.enabled === false)
@@ -232,7 +247,3 @@ export async function ensureStashCaches(config) {
         }
     }
 }
-/** @deprecated Use ensureStashCaches instead. */
-export const ensureGitCaches = ensureStashCaches;
-/** @deprecated Use ensureStashCaches instead. */
-export const ensureContextHubCaches = ensureStashCaches;

package/dist/{semantic-status.js → indexer/semantic-status.js}

@@ -1,7 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
-import { DEFAULT_LOCAL_MODEL } from "./embedder";
-import { getCacheDir, getSemanticStatusPath } from "./paths";
+import { getCacheDir, getSemanticStatusPath } from "../core/paths";
+import { DEFAULT_LOCAL_MODEL } from "../llm/embedder";
 export function deriveSemanticProviderFingerprint(embedding) {
     if (embedding?.endpoint) {
         return `remote:${embedding.endpoint}|${embedding.model}|${embedding.dimension ?? "default"}`;

package/dist/{walker.js → indexer/walker.js}

@@ -7,7 +7,7 @@
  */
 import fs from "node:fs";
 import path from "node:path";
-import { isRelevantAssetFile } from "./asset-spec";
+import { isRelevantAssetFile } from "../core/asset-spec";
 import { buildFileContext } from "./file-context";
 const SKIP_DIRS = new Set([".git", "node_modules", "bin", ".cache"]);
 /**

package/dist/{lockfile.js → integrations/lockfile.js}

@@ -1,6 +1,6 @@
 import fs from "node:fs";
 import path from "node:path";
-import { getConfigDir } from "./config";
+import { getConfigDir } from "../core/config";
 // ── Paths ───────────────────────────────────────────────────────────────────
 const LOCKFILE_NAME = "akm.lock";
 const LEGACY_LOCKFILE_NAME = "stash.lock";

package/dist/{llm-client.js → llm/client.js}

@@ -7,7 +7,7 @@
  *
  * `llm.ts` re-exports everything from this module for backward compatibility.
  */
-import { fetchWithTimeout } from "./common";
+import { fetchWithTimeout } from "../core/common";
 export async function chatCompletion(config, messages, options) {
     const headers = { "Content-Type": "application/json" };
     if (config.apiKey) {

package/dist/{embedders → llm/embedders}/local.js

@@ -7,8 +7,8 @@
  * shared instance for the production code path.
  */
 import path from "node:path";
-import { getCacheDir } from "../paths";
-import { warn } from "../warn";
+import { getCacheDir } from "../../core/paths";
+import { warn } from "../../core/warn";
 /**
  * Default local transformer model for embeddings.
  * `bge-small-en-v1.5` scores higher on MTEB benchmarks than the previous

package/dist/{embedders → llm/embedders}/remote.js

@@ -4,7 +4,7 @@
  * Calls the configured `/embeddings` endpoint and L2-normalizes the returned
  * vectors so the scoring pipeline's L2-to-cosine conversion is correct.
  */
-import { fetchWithTimeout, isHttpUrl } from "../common";
+import { fetchWithTimeout, isHttpUrl } from "../../core/common";
 const REMOTE_BATCH_SIZE = 100;
 export class RemoteEmbedder {
     config;

package/dist/{embedders → llm/embedders}/types.js

@@ -36,4 +36,4 @@ export function cosineSimilarity(a, b) {
 }
 // Imported lazily to keep this types module dependency-free where possible;
 // `warn` is a thin printf wrapper so the cost is negligible.
-import { warn } from "../warn";
+import { warn } from "../../core/warn";

package/dist/{metadata-enhance.js → llm/metadata-enhance.js}

@@ -3,9 +3,9 @@
  *
  * Split out of `llm.ts` so the higher-level workflow (prompting the LLM to
  * improve descriptions/tags/searchHints) lives separately from the low-level
- * transport client in `llm-client.ts`.
+ * transport client in `client.ts`.
  */
-import { chatCompletion, parseJsonResponse } from "./llm-client";
+import { chatCompletion, parseJsonResponse } from "./client";
 const SYSTEM_PROMPT = `You are a metadata generator for a developer asset registry. Given a script/skill/command/agent entry, generate improved metadata. Respond with ONLY valid JSON, no markdown fencing.`;
 /**
  * Use an LLM to enhance a stash entry's metadata: improve description,

package/dist/{cli-hints.js → output/cli-hints.js}

@@ -121,6 +121,7 @@ akm remember --name release-retro < notes.md   # Save multiline memory from stdi
 akm import ./docs/auth-flow.md                 # Import a file as knowledge
 akm import - --name scratch-notes < notes.md   # Import stdin as a knowledge doc
 akm workflow create ship-release               # Create a workflow asset in the stash
+akm workflow validate workflows/foo.md         # Validate a workflow file or ref; lists every error
 akm workflow next workflow:ship-release        # Start or resume the next workflow step
 akm feedback skill:code-review --positive      # Record that an asset helped
 akm feedback agent:reviewer --negative         # Record that an asset missed the mark

package/dist/{output-context.js → output/context.js}

@@ -8,7 +8,7 @@
  *
  * Initialized from `cli.ts` before `runMain`.
  */
-import { UsageError } from "./errors";
+import { UsageError } from "../core/errors";
 export const OUTPUT_FORMATS = ["json", "yaml", "text", "jsonl"];
 export const DETAIL_LEVELS = ["brief", "normal", "full", "summary", "agent"];
 export function parseOutputFormat(value) {
@@ -16,14 +16,14 @@ export function parseOutputFormat(value) {
         return undefined;
     if (OUTPUT_FORMATS.includes(value))
         return value;
-    throw new UsageError(`Invalid value for --format: ${value}. Expected one of: ${OUTPUT_FORMATS.join("|")}`);
+    throw new UsageError(`Invalid value for --format: ${value}. Expected one of: ${OUTPUT_FORMATS.join("|")}`, "INVALID_FORMAT_VALUE");
 }
 export function parseDetailLevel(value) {
     if (!value)
         return undefined;
     if (DETAIL_LEVELS.includes(value))
         return value;
-    throw new UsageError(`Invalid value for --detail: ${value}. Expected one of: ${DETAIL_LEVELS.join("|")}`);
+    throw new UsageError(`Invalid value for --detail: ${value}. Expected one of: ${DETAIL_LEVELS.join("|")}`, "INVALID_DETAIL_VALUE");
 }
 export function parseFlagValue(argv, flag) {
     for (let i = 0; i < argv.length; i++) {
@@ -38,6 +38,24 @@ export function parseFlagValue(argv, flag) {
 export function hasBooleanFlag(argv, flag) {
     return argv.some((arg) => arg === flag || arg === `${flag}=true`);
 }
+/**
+ * Read a hyphenated arg out of citty's parsed `args` object.
+ *
+ * citty does not auto-camelise hyphenated arg keys (see `--max-pages`,
+ * `--with-sources` for the existing convention), so command handlers end up
+ * casting `args` to a string-indexed record at every read site. This helper
+ * encapsulates the cast.
+ */
+export function getHyphenatedArg(args, key) {
+    if (typeof args !== "object" || args === null)
+        return undefined;
+    const value = args[key];
+    return value === undefined ? undefined : value;
+}
+/** Boolean variant of {@link getHyphenatedArg} for `--<flag>` switches. */
+export function getHyphenatedBoolean(args, key) {
+    return Boolean(getHyphenatedArg(args, key));
+}
 /**
  * Resolve output mode from a synthetic argv array and config defaults.
  * Pure function — no IO. Suitable for unit tests.

package/dist/{renderers.js → output/renderers.js}

@@ -8,15 +8,13 @@
  */
 import fs from "node:fs";
 import path from "node:path";
-import { hasErrnoCode } from "./common";
-import { UsageError } from "./errors";
-import { registerRenderer } from "./file-context";
-import { parseFrontmatter, toStringOrUndefined } from "./frontmatter";
-import { extractFrontmatterOnly, extractLineRange, extractSection, formatToc, parseMarkdownToc } from "./markdown";
-import { extractDescriptionFromComments, loadStashFile } from "./metadata";
-import { makeAssetRef } from "./stash-ref";
-import { listKeys as listVaultKeys } from "./vault";
-import { parseWorkflowMarkdown, WorkflowValidationError } from "./workflow-markdown";
+import { listKeys as listVaultKeys } from "../commands/vault";
+import { hasErrnoCode } from "../core/common";
+import { parseFrontmatter, toStringOrUndefined } from "../core/frontmatter";
+import { extractFrontmatterOnly, extractLineRange, extractSection, formatToc, parseMarkdownToc, } from "../core/markdown";
+import { registerRenderer } from "../indexer/file-context";
+import { extractDescriptionFromComments, loadStashFile } from "../indexer/metadata";
+import { buildWorkflowAction, workflowMdRenderer } from "../workflows/renderer";
 // ── Interpreter auto-detection map ───────────────────────────────────────────
 const INTERPRETER_MAP = {
     ".sh": "bash",
@@ -153,12 +151,7 @@ function deriveName(ctx) {
     const ext = path.extname(ctx.relPath);
     return ext ? ctx.relPath.slice(0, -ext.length) : ctx.relPath;
 }
-function shellQuote(value) {
-    return `'${value.replace(/'/g, `'\\''`)}'`;
-}
-export function buildWorkflowAction(ref) {
-    return `Resume the active run or start a new run with \`akm workflow next ${shellQuote(ref)}\`.`;
-}
+export { buildWorkflowAction };
 /**
  * Load the matching StashEntry for a file path from the directory's .stash.json.
  */
@@ -465,56 +458,7 @@ const memoryMdRenderer = {
     },
 };
 // ── 6. workflow-md ───────────────────────────────────────────────────────────
-const workflowMdRenderer = {
-    name: "workflow-md",
-    buildShowResponse(ctx) {
-        const name = deriveName(ctx);
-        const workflow = parseWorkflowForRendering(ctx.content());
-        const ref = makeAssetRef("workflow", name, ctx.origin);
-        return {
-            type: "workflow",
-            name,
-            path: ctx.absPath,
-            action: buildWorkflowAction(ref),
-            description: workflow.description,
-            workflowTitle: workflow.title,
-            parameters: workflow.parameters?.map((parameter) => parameter.name),
-            workflowParameters: workflow.parameters,
-            steps: workflow.steps,
-        };
-    },
-    extractMetadata(entry, ctx) {
-        const workflow = parseWorkflowForRendering(ctx.content());
-        const hints = new Set(entry.searchHints ?? []);
-        hints.add(workflow.title);
-        for (const step of workflow.steps) {
-            hints.add(step.title);
-            hints.add(step.id);
-            hints.add(step.instructions);
-            for (const criterion of step.completionCriteria ?? []) {
-                hints.add(criterion);
-            }
-        }
-        entry.searchHints = Array.from(hints).filter(Boolean);
-        if (workflow.parameters?.length) {
-            entry.parameters = workflow.parameters.map((parameter) => ({
-                name: parameter.name,
-                ...(parameter.description ? { description: parameter.description } : {}),
-            }));
-        }
-    },
-};
-function parseWorkflowForRendering(content) {
-    try {
-        return parseWorkflowMarkdown(content);
-    }
-    catch (error) {
-        if (error instanceof WorkflowValidationError) {
-            throw new UsageError(error.message);
-        }
-        throw error;
-    }
-}
+// Defined in src/workflows/renderer.ts and imported above.
 // ── 7. script-source ─────────────────────────────────────────────────────────
 const scriptSourceRenderer = {
     name: "script-source",

package/dist/{output-shapes.js → output/shapes.js}

@@ -81,10 +81,20 @@ export function shapeAssetHit(hit, detail) {
 }
 export function shapeSearchHit(hit, detail) {
     if (hit.type === "registry") {
-        if (detail === "brief")
-            return pickFields(hit, ["name", "action"]);
+        if (detail === "brief") {
+            // RegistrySearchHit uses `title` (not `name`); always project installRef
+            // and score so callers can use the result without --detail full (QA #28).
+            const out = pickFields(hit, ["title", "name", "installRef", "score"]);
+            // Normalise: if only title exists, expose it as `name` for consistency
+            if (out.title && !out.name)
+                out.name = out.title;
+            return out;
+        }
         if (detail === "normal") {
-            return capDescription(pickFields(hit, ["name", "description", "action", "curated"]), NORMAL_DESCRIPTION_LIMIT);
+            const out = capDescription(pickFields(hit, ["title", "name", "description", "action", "installRef", "score", "curated"]), NORMAL_DESCRIPTION_LIMIT);
+            if (out.title && !out.name)
+                out.name = out.title;
+            return out;
         }
         return hit;
     }
@@ -176,6 +186,10 @@ export function shapeShowOutput(result, detail, forAgent = false) {
         "cwd",
         "keys",
         "comments",
+        // path and editable are always projected so JSON consumers can locate and
+        // edit the asset without needing --detail full (QA #7).
+        "path",
+        "editable",
     ]);
     if (detail !== "full") {
         return base;
@@ -183,7 +197,7 @@ export function shapeShowOutput(result, detail, forAgent = false) {
     return {
         schemaVersion: 1,
         ...base,
-        ...pickFields(result, ["path", "editable", "editHint"]),
+        ...pickFields(result, ["editHint"]),
     };
 }
 export function pickFields(source, fields) {

package/dist/{output-text.js → output/text.js}

@@ -5,7 +5,7 @@
  *
  * Pure functions — no IO.
  */
-import { formatInstallAuditSummary } from "./install-audit";
+import { formatInstallAuditSummary } from "../commands/install-audit";
 export function outputJsonl(command, shaped) {
     if (command === "search" || command === "registry-search") {
         const r = shaped;

package/dist/{registry-build-index.js → registry/build-index.js}

@@ -1,14 +1,23 @@
+/**
+ * Build the v2 JSON registry index consumed by the `static-index` registry
+ * provider. This module emits artifacts that conform to the v2 schema; the
+ * schema itself is the input contract owned by `src/registry/providers/static-index.ts`
+ * (see v1 architecture spec §3.3 — "the v2 JSON index schema belongs to
+ * static-index"). When the schema changes, both the parser in `static-index.ts`
+ * and the JSON Schema in `docs/technical/registry-index.schema.json` must be
+ * updated together with this builder.
+ */
 import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
-import { fetchWithRetry, jsonWithByteCap } from "./common";
-import { asRecord, asString, GITHUB_API_BASE, githubHeaders } from "./github";
-import { generateMetadataFlat, loadStashFile } from "./metadata";
+import { fetchWithRetry, jsonWithByteCap } from "../core/common";
+import { generateMetadataFlat, loadStashFile } from "../indexer/metadata";
+import { walkStashFlat } from "../indexer/walker";
+import { asRecord, asString, GITHUB_API_BASE, githubHeaders } from "../integrations/github";
+import { copyIncludedPaths, findNearestIncludeConfig } from "../sources/include";
+import { detectStashRoot } from "../sources/providers/provider-utils";
+import { extractTarGzSecure } from "../sources/providers/tar-utils";
 import { parseRegistryIndex } from "./providers/static-index";
-import { copyIncludedPaths, findNearestIncludeConfig } from "./stash-include";
-import { detectStashRoot } from "./stash-providers/provider-utils";
-import { extractTarGzSecure } from "./stash-providers/tar-utils";
-import { walkStashFlat } from "./walker";
 const DEFAULT_NPM_REGISTRY_BASE = "https://registry.npmjs.org";
 const DEFAULT_MANUAL_ENTRIES_PATH = path.resolve("manual-entries.json");
 const DEFAULT_OUTPUT_PATH = path.resolve("index.json");

package/dist/{create-provider-registry.js → registry/create-provider-registry.js}

@@ -2,8 +2,8 @@
  * Generic factory-map utility.
  *
  * Creates a lightweight registry that maps string keys to factory functions.
- * Both registry-factory.ts (stash discovery) and stash-provider-factory.ts
- * (stash source providers) are built on this utility.
+ * Both registry/factory.ts (registry discovery) and sources/provider-factory.ts
+ * (source providers) are built on this utility.
  */
 export function createProviderRegistry() {
     const map = new Map();
@@ -14,5 +14,9 @@ export function createProviderRegistry() {
         resolve(type) {
             return map.get(type) ?? null;
         },
+        /** Snapshot of all registered keys. Iteration order matches insertion order. */
+        list() {
+            return [...map.keys()];
+        },
     };
 }

package/dist/registry/factory.js

@@ -0,0 +1,33 @@
+/**
+ * Registry provider factory map.
+ *
+ * Maps registry provider type identifiers (e.g. "static-index", "skills-sh")
+ * to factory functions that create RegistryProvider instances.
+ *
+ * "Registry" here refers to the kit discovery registries (static index files,
+ * skills.sh API) — not to be confused with the source provider factory map in
+ * `sources/provider-factory.ts` or the installed-source operations in
+ * `installed-stashes.ts`.
+ *
+ * Phase 6 (v1 architecture refactor): factories are now the
+ * `RegistryProviderFactory` type owned by `src/registry/providers/types.ts`.
+ * The legacy alias in `src/registry-provider.ts` is kept as a thin re-export
+ * for transitional callers and will be removed after the dust settles.
+ */
+import { createProviderRegistry } from "./create-provider-registry";
+// ── Factory map ─────────────────────────────────────────────────────────────
+const registry = createProviderRegistry();
+export function registerProvider(type, factory) {
+    registry.register(type, factory);
+}
+export function resolveProviderFactory(type) {
+    return registry.resolve(type);
+}
+/**
+ * Iterate over all registered registry providers. Used by the orchestrator
+ * (`src/commands/registry-search.ts`) to fan out queries through the same
+ * `RegistryProvider` interface regardless of provider kind.
+ */
+export function listProviderTypes() {
+    return registry.list();
+}

package/dist/{origin-resolve.js → registry/origin-resolve.js}

@@ -1,5 +1,5 @@
 import path from "node:path";
-import { parseRegistryRef } from "./registry-resolve";
+import { parseRegistryRef } from "./resolve";
 /**
  * Given an origin string (from an AssetRef) and the full list of stash
  * sources, return the subset of sources to search.

package/dist/{providers → registry/providers}/index.js

@@ -5,7 +5,7 @@
  * providers with the provider registry. This replaces the individual
  * side-effect imports that were duplicated in registry-search.ts.
  *
- * Mirrors the pattern used by `stash-providers/index.ts`.
+ * Mirrors the pattern used by `sources/providers/index.ts`.
  */
 import "./static-index";
 import "./skills-sh";