akm-cli 0.7.4 → 0.8.0-rc.10

package/dist/core/config-schema.js

@@ -0,0 +1,534 @@
+// This Source Code Form is subject to the terms of the Mozilla Public
+// License, v. 2.0. If a copy of the MPL was not distributed with this
+// file, You can obtain one at https://mozilla.org/MPL/2.0/.
+/**
+ * Zod schema for AkmConfig — the single source of truth for the on-disk shape.
+ *
+ * Two responsibilities:
+ * 1. **Validate + transform** the raw JSON-parsed config object into the runtime
+ *    `AkmConfig` shape consumed by the rest of the codebase. Replaces the
+ *    ~1.4k LOC of legacy per-shape parsers (parseLlmConfig, parseEmbeddingConfig,
+ *    parseIndexConfig, etc.) — see `loadConfig` in `./config.ts`.
+ * 2. **Reject hard-errored values** (openviking source type, legacy
+ *    `stashes[]` key) at load time via `superRefine`.
+ *
+ * Design rules:
+ * - Top-level uses `.passthrough()` so unknown future keys round-trip intact on
+ *   read; `sanitizeConfigForWrite` decides what to persist.
+ * - Most nested sub-objects use `.catch(undefined)` so malformed entries are
+ *   silently dropped (matches the legacy parser's warn-and-ignore semantics for
+ *   field-level shape errors — keeps cold-start working when a user has a
+ *   typo in their config).
+ * - Two exceptions (hard-rejected): openviking source type and legacy
+ *   `stashes[]` key. Both have explicit migration paths; silently dropping
+ *   would mask user data loss.
+ * - `.strict()` walls still gate `registries[]`, `sources[]`, `profiles.*`
+ *   sub-shapes so typos in those structured records are caught (#462).
+ * - `defaultWriteTarget` resolution and similar cross-field invariants are
+ *   enforced at save time via `superRefine` on the top-level schema.
+ */
+import { z } from "zod";
+// ── Reusable atomic schemas ─────────────────────────────────────────────────
+/** Positive integer (used for tokens, timeouts, batch sizes). */
+const positiveInt = z.number().int().positive();
+/** Non-negative finite number (used for scores, weights, days). */
+const nonNegativeNumber = z.number().finite().min(0);
+/** Non-empty string (rejects "" and whitespace-only). */
+const nonEmptyString = z
+    .string()
+    .min(1)
+    .refine((v) => v.trim().length > 0, { message: "expected a non-empty string" });
+/** HTTP(S) URL string. */
+const httpUrl = z.string().refine((v) => v.startsWith("http://") || v.startsWith("https://"), {
+    message: "endpoint must start with http:// or https://",
+});
+// ── Feedback failure modes ──────────────────────────────────────────────────
+export const FEEDBACK_FAILURE_MODES = ["incorrect", "outdated", "dangerous", "incomplete", "redundant"];
+// ── Connection configs (LLM / embedding) ────────────────────────────────────
+const LlmCapabilitiesSchema = z
+    .object({
+    structuredOutput: z.boolean().optional(),
+})
+    .strict();
+/**
+ * Connection config used for both top-level `llm` (after migration) and
+ * `profiles.llm[*]`. `model` is required at schema level — partial entries
+ * created by `akm config set llm.endpoint <url>` (where model is left absent)
+ * are normalized to `model: ""` *before* Zod sees them by the load-time
+ * pre-Zod migrator hook, so this strict shape gates CLI writes without
+ * breaking legacy load-time partial configs.
+ */
+export const LlmConnectionConfigSchema = z
+    .object({
+    provider: z.string().optional(),
+    endpoint: z.string(),
+    model: z.string(),
+    apiKey: z.string().optional(),
+    temperature: z.number().finite().optional(),
+    maxTokens: positiveInt.optional(),
+    timeoutMs: positiveInt.optional(),
+    concurrency: positiveInt.optional(),
+    capabilities: LlmCapabilitiesSchema.optional(),
+    extraParams: z.record(z.unknown()).optional(),
+    contextLength: positiveInt.optional(),
+    judgeModel: z.string().min(1).optional(),
+    enableThinking: z.boolean().optional(),
+})
+    .strict();
+export const LlmProfileConfigSchema = LlmConnectionConfigSchema.extend({
+    supportsJsonSchema: z.boolean().optional(),
+}).strict();
+const EmbeddingOllamaOptionsSchema = z
+    .object({
+    num_ctx: positiveInt.optional(),
+})
+    .strict();
+/**
+ * Embedding connection config. Both `endpoint` and `model` are optional:
+ *   - Remote: provide `endpoint` (http/https URL) + `model`.
+ *   - Local-only: omit `endpoint`/`model`; set `localModel` (or fall back to
+ *     {@link DEFAULT_LOCAL_MODEL}).
+ *
+ * Consumers route via `hasRemoteEndpoint()` which checks for an http(s)
+ * endpoint — absent fields take the local path naturally, no sentinels needed.
+ */
+export const EmbeddingConnectionConfigSchema = z
+    .object({
+    provider: z.string().optional(),
+    endpoint: z.string().optional(),
+    model: z.string().optional(),
+    apiKey: z.string().optional(),
+    dimension: positiveInt.optional(),
+    localModel: z.string().min(1).optional(),
+    maxTokens: positiveInt.optional(),
+    batchSize: positiveInt.optional(),
+    chunkSize: positiveInt.optional(),
+    contextLength: positiveInt.optional(),
+    ollamaOptions: EmbeddingOllamaOptionsSchema.optional(),
+})
+    .strict();
+// ── Agent profiles ──────────────────────────────────────────────────────────
+const AgentPlatformSchema = z.enum(["opencode", "claude", "opencode-sdk"]);
+export const AgentProfileConfigSchema = z
+    .object({
+    platform: AgentPlatformSchema,
+    bin: z.string().min(1).optional(),
+    args: z.array(z.string()).optional(),
+    workspace: z.string().min(1).optional(),
+    model: z.string().min(1).optional(),
+})
+    .strict();
+// ── Improve profile / process ──────────────────────────────────────────────
+export const ImproveProcessConfigSchema = z
+    .object({
+    enabled: z.boolean().optional(),
+    mode: z.enum(["llm", "agent", "sdk"]).optional(),
+    profile: z.string().min(1).optional(),
+    timeoutMs: z.union([positiveInt, z.null()]).optional(),
+    allowedTypes: z.array(z.string().min(1)).optional(),
+    qualityGate: z.object({ enabled: z.boolean().optional() }).strict().optional(),
+    contradictionDetection: z.object({ enabled: z.boolean().optional() }).strict().optional(),
+    // Extract process config (only meaningful for extract process)
+    defaultSince: z.string().min(1).optional(),
+    maxTotalChars: positiveInt.optional(),
+    maxChunkSize: z.number().int().min(1).max(50).optional(),
+})
+    .strict();
+const ImproveProfileProcessesSchema = z
+    .object({
+    reflect: ImproveProcessConfigSchema.optional(),
+    distill: ImproveProcessConfigSchema.optional(),
+    consolidate: ImproveProcessConfigSchema.optional(),
+    memoryInference: ImproveProcessConfigSchema.optional(),
+    graphExtraction: ImproveProcessConfigSchema.optional(),
+    validation: ImproveProcessConfigSchema.optional(),
+})
+    .passthrough()
+    .superRefine((val, ctx) => {
+    // 0.8.0 removed the duplicated `feedbackDistillation` process key — it was
+    // a thin wrapper around `processes.distill.enabled`. Single source of truth.
+    const raw = val;
+    if ("feedbackDistillation" in raw) {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: "feedbackDistillation was removed in 0.8.0 — use processes.distill.enabled instead. " +
+                "It now controls both the orchestration gate and the LLM-call gate.",
+        });
+        return;
+    }
+    const allowed = new Set([
+        "reflect",
+        "distill",
+        "consolidate",
+        "memoryInference",
+        "graphExtraction",
+        "validation",
+        "extract",
+    ]);
+    for (const k of Object.keys(raw)) {
+        if (!allowed.has(k)) {
+            ctx.addIssue({
+                code: z.ZodIssueCode.unrecognized_keys,
+                keys: [k],
+                message: `Unrecognized improve process key: "${k}".`,
+            });
+        }
+    }
+});
+export const ImproveProfileConfigSchema = z
+    .object({
+    description: z.string().min(1).optional(),
+    processes: ImproveProfileProcessesSchema.optional(),
+    autoAccept: nonNegativeNumber.optional(),
+    limit: positiveInt.optional(),
+})
+    .strict();
+// ── Profiles / defaults ────────────────────────────────────────────────────
+export const ProfilesSchema = z
+    .object({
+    llm: z.record(z.string(), LlmProfileConfigSchema).optional(),
+    agent: z.record(z.string(), AgentProfileConfigSchema).optional(),
+    improve: z.record(z.string(), ImproveProfileConfigSchema).optional(),
+})
+    .strict();
+export const DefaultsSchema = z
+    .object({
+    llm: z.string().min(1).optional(),
+    agent: z.string().min(1).optional(),
+    improve: z.string().min(1).optional(),
+})
+    .strict();
+// ── Sources / registries / installed ────────────────────────────────────────
+const SourceConfigEntryOptionsSchema = z
+    .object({
+    pushOnCommit: z.boolean().optional(),
+})
+    .passthrough();
+export const SourceConfigEntrySchema = z
+    .object({
+    type: nonEmptyString,
+    path: z.string().min(1).optional(),
+    url: z.string().min(1).optional(),
+    name: z.string().min(1).optional(),
+    enabled: z.boolean().optional(),
+    writable: z.boolean().optional(),
+    primary: z.boolean().optional(),
+    options: SourceConfigEntryOptionsSchema.optional(),
+    wikiName: z.string().min(1).optional(),
+})
+    .strict()
+    .superRefine((entry, ctx) => {
+    if (entry.writable === true && (entry.type === "website" || entry.type === "npm")) {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: `writable: true is only supported on filesystem and git sources (got "${entry.type}"` +
+                (entry.name ? ` on source "${entry.name}"` : "") +
+                ").",
+        });
+    }
+});
+export const RegistryConfigEntrySchema = z
+    .object({
+    url: httpUrl,
+    name: z.string().min(1).optional(),
+    enabled: z.boolean().optional(),
+    provider: z.string().min(1).optional(),
+    options: z.record(z.unknown()).optional(),
+})
+    .strict();
+const KitSourceSchema = z.enum(["filesystem", "git", "npm", "github", "website", "local"]);
+export const InstalledStashEntrySchema = z
+    .object({
+    id: nonEmptyString,
+    source: KitSourceSchema,
+    ref: nonEmptyString,
+    artifactUrl: nonEmptyString,
+    stashRoot: nonEmptyString,
+    cacheDir: nonEmptyString,
+    installedAt: nonEmptyString,
+    writable: z.boolean().optional(),
+    resolvedVersion: z.string().min(1).optional(),
+    resolvedRevision: z.string().min(1).optional(),
+    wikiName: z.string().min(1).optional(),
+})
+    .strict()
+    .superRefine((entry, ctx) => {
+    if (entry.writable === true && entry.source !== "git" && entry.source !== "filesystem") {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: `writable: true is only supported on filesystem and git sources (got "${entry.source}" on installed entry "${entry.id}").`,
+        });
+    }
+});
+// ── Output ──────────────────────────────────────────────────────────────────
+export const OutputConfigSchema = z
+    .object({
+    format: z.enum(["json", "yaml", "text"]).optional(),
+    detail: z.enum(["brief", "normal", "full"]).optional(),
+})
+    .strict();
+// ── Search ──────────────────────────────────────────────────────────────────
+const SearchGraphBoostSchema = z
+    .object({
+    directBoostPerEntity: nonNegativeNumber.optional(),
+    directBoostCap: nonNegativeNumber.optional(),
+    hopBoostPerEntity: nonNegativeNumber.optional(),
+    hopBoostCap: nonNegativeNumber.optional(),
+    /** Hard-capped at 3; values > 3 hard-error so users see the typo. */
+    maxHops: positiveInt.max(3).optional(),
+    confidenceMode: z.enum(["off", "blend", "multiply"]).default("blend").optional(),
+    /** Range [0, 1]; values > 1 hard-error (no silent clamp). */
+    confidenceWeight: z.number().finite().min(0).max(1).default(0.2).optional(),
+})
+    .strict();
+export const SearchConfigSchema = z
+    .object({
+    minScore: nonNegativeNumber.optional(),
+    curateRerank: z.object({ enabled: z.boolean().optional() }).strict().optional(),
+    graphBoost: SearchGraphBoostSchema.optional(),
+})
+    .strict();
+// ── Feedback ────────────────────────────────────────────────────────────────
+export const FeedbackConfigSchema = z
+    .object({
+    requireReason: z.boolean().optional(),
+    allowedFailureModes: z.array(nonEmptyString).optional(),
+})
+    .strict();
+// ── Improve top-level (utility decay, event retention) ─────────────────────
+const ImproveUtilityDecaySchema = z
+    .object({
+    halfLifeDays: z.number().finite().min(0.1).optional(),
+    feedbackStabilityBoost: z.number().finite().min(1).optional(),
+})
+    .strict();
+export const ImproveConfigSchema = z
+    .object({
+    utilityDecay: ImproveUtilityDecaySchema.optional(),
+    eventRetentionDays: nonNegativeNumber.optional(),
+})
+    .strict();
+// ── Index / per-pass ────────────────────────────────────────────────────────
+const GRAPH_EXTRACTION_INCLUDE_TYPES_ALLOWED = [
+    "memory",
+    "knowledge",
+    "skill",
+    "command",
+    "agent",
+    "workflow",
+    "lesson",
+    "task",
+    "wiki",
+];
+const INDEX_PASS_PROVIDER_KEYS = new Set([
+    "endpoint",
+    "model",
+    "provider",
+    "apiKey",
+    "baseUrl",
+    "temperature",
+    "maxTokens",
+    "capabilities",
+]);
+const INDEX_PASS_KNOWN_KEYS = new Set([
+    "llm",
+    "graphExtractionBatchSize",
+    "graphExtractionIncludeTypes",
+    "memoryInferenceBatchSize",
+]);
+/**
+ * Per-pass `index.<pass>` entry. Uses preprocess + manual validation so we can
+ * emit the legacy parser's targeted error messages ("Duplicate LLM provider
+ * configuration", "Unknown key `index.<pass>.<key>`", "expected a boolean")
+ * instead of Zod's generic `Unrecognized key` / `Expected boolean, received
+ * string` strings — keeps `akm` startup errors actionable.
+ */
+export const IndexPassConfigSchema = z.preprocess((raw, ctx) => {
+    if (typeof raw !== "object" || raw === null || Array.isArray(raw)) {
+        return raw; // let z.object below produce the type error
+    }
+    const obj = raw;
+    for (const key of Object.keys(obj)) {
+        if (INDEX_PASS_PROVIDER_KEYS.has(key)) {
+            ctx.addIssue({
+                code: z.ZodIssueCode.custom,
+                message: `Duplicate LLM provider configuration: \`${[...(ctx.path ?? []), key].join(".")}\` is not allowed. ` +
+                    "Configure provider/model/endpoint under `profiles.llm` only; per-pass entries support `{ llm: false }` opt-out.",
+            });
+            return raw;
+        }
+        if (!INDEX_PASS_KNOWN_KEYS.has(key)) {
+            ctx.addIssue({
+                code: z.ZodIssueCode.custom,
+                message: `Unknown key \`${[...(ctx.path ?? []), key].join(".")}\`. Per-pass entries support \`llm\` ` +
+                    "(boolean opt-out), `graphExtractionBatchSize`, `graphExtractionIncludeTypes`, and " +
+                    "`memoryInferenceBatchSize`.",
+            });
+            return raw;
+        }
+    }
+    if ("llm" in obj && typeof obj.llm !== "boolean") {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: `Invalid \`${[...(ctx.path ?? []), "llm"].join(".")}\`: expected a boolean (true to use the default LLM profile, false to opt out). Got ${typeof obj.llm}.`,
+        });
+        return raw;
+    }
+    return raw;
+}, z
+    .object({
+    llm: z.boolean().optional(),
+    graphExtractionBatchSize: positiveInt.optional(),
+    graphExtractionIncludeTypes: z.array(z.enum(GRAPH_EXTRACTION_INCLUDE_TYPES_ALLOWED)).nonempty().optional(),
+    memoryInferenceBatchSize: positiveInt.optional(),
+})
+    .passthrough());
+const MetadataEnhanceSchema = z.object({ enabled: z.boolean().optional() }).strict();
+const StalenessDetectionSchema = z
+    .object({
+    enabled: z.boolean().optional(),
+    thresholdDays: positiveInt.optional(),
+})
+    .strict();
+/**
+ * Index config is a union of reserved feature sections and per-pass entries.
+ * Passthrough so per-pass entries (keyed by arbitrary pass names like `graph`,
+ * `enrichment`) can live next to the reserved keys.
+ *
+ * The outer preprocess emits the legacy parser's actionable error messages
+ * for the two most common type-shape mistakes:
+ *   - An array at the `index` block.
+ *   - A non-object at `index.<passName>`.
+ * Inner field validation (graphExtractionIncludeTypes enum, llm boolean,
+ * provider-key rejection) is delegated to {@link IndexPassConfigSchema}.
+ */
+export const IndexConfigSchema = z.preprocess((raw, ctx) => {
+    if (raw === undefined || raw === null)
+        return raw;
+    if (Array.isArray(raw)) {
+        ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: 'Invalid `index` config: expected an object keyed by pass name (e.g. `{ "enrichment": { "llm": false } }`).',
+        });
+        return raw;
+    }
+    if (typeof raw !== "object")
+        return raw;
+    for (const [passName, value] of Object.entries(raw)) {
+        if (typeof value !== "object" || value === null || Array.isArray(value)) {
+            ctx.addIssue({
+                code: z.ZodIssueCode.custom,
+                message: `Invalid \`index.${passName}\` config: expected an object like \`{ "llm": false }\`.`,
+            });
+            return raw;
+        }
+        if (passName !== "metadataEnhance" &&
+            passName !== "stalenessDetection" &&
+            Array.isArray(value.graphExtractionIncludeTypes)) {
+            const arr = value.graphExtractionIncludeTypes;
+            const invalid = [];
+            for (const t of arr) {
+                if (typeof t === "string" &&
+                    !GRAPH_EXTRACTION_INCLUDE_TYPES_ALLOWED.includes(t.toLowerCase())) {
+                    invalid.push(t);
+                }
+            }
+            if (invalid.length > 0) {
+                ctx.addIssue({
+                    code: z.ZodIssueCode.custom,
+                    message: `Invalid \`index.${passName}.graphExtractionIncludeTypes\`: unsupported type(s): ${invalid.join(", ")}.`,
+                });
+                return raw;
+            }
+        }
+    }
+    return raw;
+}, z
+    .object({
+    metadataEnhance: MetadataEnhanceSchema.optional(),
+    stalenessDetection: StalenessDetectionSchema.optional(),
+})
+    .catchall(IndexPassConfigSchema));
+// ── Top-level AkmConfig ────────────────────────────────────────────────────
+/**
+ * Base object schema used both as the top-level shape and as the source of
+ * truth for {@link listTopLevelConfigKeys}. {@link AkmConfigSchema} wraps this
+ * with cross-field refinements (`.superRefine()`).
+ *
+ * All fields validate loudly — typos and shape errors throw at load time. The
+ * legacy parser's warn-and-drop tolerance was a frequent source of silent
+ * configuration loss; the migration module ({@link migrateConfigShape}) handles
+ * one-time 0.7→0.8 input transforms before the schema sees the value.
+ */
+export const AkmConfigShape = {
+    configVersion: z.union([z.string().min(1), z.number()]).optional(),
+    profiles: ProfilesSchema.optional(),
+    defaults: DefaultsSchema.optional(),
+    stashDir: nonEmptyString.optional(),
+    semanticSearchMode: z.enum(["off", "auto"]).default("auto"),
+    embedding: EmbeddingConnectionConfigSchema.optional(),
+    index: IndexConfigSchema.optional(),
+    installed: z.array(InstalledStashEntrySchema).optional(),
+    registries: z.array(RegistryConfigEntrySchema).optional(),
+    sources: z.array(SourceConfigEntrySchema).optional(),
+    output: OutputConfigSchema.optional(),
+    writable: z.boolean().optional(),
+    defaultWriteTarget: nonEmptyString.optional(),
+    search: SearchConfigSchema.optional(),
+    feedback: FeedbackConfigSchema.optional(),
+    archiveRetentionDays: nonNegativeNumber.optional(),
+    improve: ImproveConfigSchema.optional(),
+};
+export const AkmConfigBaseSchema = z.object(AkmConfigShape).strict();
+export const AkmConfigSchema = AkmConfigBaseSchema.superRefine((config, ctx) => {
+    // #464.a: defaultWriteTarget must name a configured source when sources
+    // are present. With no sources configured, error out instead of silently
+    // accepting (no implicit "first writable" fallback — see locked decision 3).
+    if (config.defaultWriteTarget !== undefined) {
+        const knownNames = (config.sources ?? [])
+            .map((s) => s.name)
+            .filter((n) => typeof n === "string" && n.length > 0);
+        if (knownNames.length === 0) {
+            ctx.addIssue({
+                code: z.ZodIssueCode.custom,
+                path: ["defaultWriteTarget"],
+                message: `defaultWriteTarget "${config.defaultWriteTarget}" cannot be resolved: no sources configured. ` +
+                    "Add at least one entry to `sources` with a matching `name` first.",
+            });
+        }
+        else if (!knownNames.includes(config.defaultWriteTarget)) {
+            ctx.addIssue({
+                code: z.ZodIssueCode.custom,
+                path: ["defaultWriteTarget"],
+                message: `defaultWriteTarget "${config.defaultWriteTarget}" does not match any configured source name: ${knownNames.map((n) => `"${n}"`).join(", ")}.`,
+            });
+        }
+    }
+});
+/**
+ * Validate a raw object against {@link AkmConfigSchema}. Returns a structured
+ * result so callers can render errors as a list (instead of throwing on the
+ * first issue).
+ */
+export function validateConfigShape(raw) {
+    const result = AkmConfigSchema.safeParse(raw);
+    if (result.success) {
+        return { ok: true, value: result.data, errors: [] };
+    }
+    return {
+        ok: false,
+        errors: result.error.issues.map((issue) => ({
+            path: issue.path.join("."),
+            message: issue.message,
+        })),
+    };
+}
+// ── Top-level key listing (for hint messages) ───────────────────────────────
+/**
+ * Return the sorted list of top-level config keys recognized by the schema.
+ * Used by error hints so the list stays in sync with the schema automatically
+ * (#460).
+ */
+export function listTopLevelConfigKeys() {
+    return Object.keys(AkmConfigShape).sort();
+}

package/dist/core/config-sources.js

@@ -0,0 +1,108 @@
+// This Source Code Form is subject to the terms of the Mozilla Public
+// License, v. 2.0. If a copy of the MPL was not distributed with this
+// file, You can obtain one at https://mozilla.org/MPL/2.0/.
+/**
+ * Runtime helpers that derive {@link ConfiguredSource} values from the
+ * persisted {@link SourceConfigEntry} / {@link InstalledStashEntry} shapes
+ * in an {@link AkmConfig}.
+ */
+import { createHash } from "node:crypto";
+/**
+ * Synthesize a stable identifier when a {@link SourceConfigEntry} omits its
+ * `name`. Uses a short hash of the discriminating fields so two equivalent
+ * entries collapse to the same generated name.
+ */
+function deriveStashEntryName(entry) {
+    if (entry.name)
+        return entry.name;
+    const seed = JSON.stringify({
+        type: entry.type,
+        path: entry.path ?? null,
+        url: entry.url ?? null,
+    });
+    const hash = createHash("sha256").update(seed).digest("hex").slice(0, 8);
+    return `${entry.type}-${hash}`;
+}
+/**
+ * Convert a persisted {@link SourceConfigEntry} into the runtime
+ * {@link SourceSpec} discriminated union. Returns `undefined` when the entry
+ * is missing the fields its provider type requires (e.g. a `filesystem`
+ * entry with no `path`); callers should drop or warn for those.
+ */
+export function parseSourceSpec(entry) {
+    switch (entry.type) {
+        case "filesystem":
+            return entry.path ? { type: "filesystem", path: entry.path } : undefined;
+        case "git":
+            return entry.url ? { type: "git", url: entry.url } : undefined;
+        case "website":
+            return entry.url
+                ? {
+                    type: "website",
+                    url: entry.url,
+                    ...(typeof entry.options?.maxPages === "number" ? { maxPages: entry.options.maxPages } : {}),
+                }
+                : undefined;
+        case "npm":
+            return entry.path ? { type: "npm", package: entry.path } : undefined;
+        default:
+            // Unknown provider — best-effort fallback so callers still get something.
+            return entry.path ? { type: "filesystem", path: entry.path } : undefined;
+    }
+}
+/**
+ * Build the full ordered list of runtime {@link ConfiguredSource} values from
+ * a loaded {@link AkmConfig}:
+ *   1. The entry marked `primary: true` (or a synthetic entry from `stashDir`).
+ *   2. Remaining `sources[]` entries in declared order.
+ *   3. Legacy `installed[]` entries, mapped into runtime entries.
+ *
+ * Entries with `enabled: false` are still emitted — callers decide whether to
+ * honour the flag. Entries that fail {@link parseSourceSpec} drop silently.
+ */
+export function resolveConfiguredSources(config) {
+    const entries = [];
+    const sources = config.sources ?? [];
+    let primary = sources.find((entry) => entry.primary === true);
+    if (!primary && config.stashDir) {
+        primary = { type: "filesystem", path: config.stashDir, primary: true };
+    }
+    if (primary) {
+        const runtime = toConfiguredSource(primary, true);
+        if (runtime)
+            entries.push(runtime);
+    }
+    for (const entry of sources) {
+        if (entry === primary)
+            continue;
+        const runtime = toConfiguredSource(entry, false);
+        if (runtime)
+            entries.push(runtime);
+    }
+    for (const installed of config.installed ?? []) {
+        entries.push({
+            name: installed.id,
+            type: "filesystem",
+            source: { type: "filesystem", path: installed.stashRoot },
+            enabled: true,
+            writable: installed.writable,
+            ...(installed.wikiName ? { wikiName: installed.wikiName } : {}),
+        });
+    }
+    return entries;
+}
+function toConfiguredSource(persisted, isPrimary) {
+    const source = parseSourceSpec(persisted);
+    if (!source)
+        return undefined;
+    return {
+        name: deriveStashEntryName(persisted),
+        type: persisted.type,
+        source,
+        ...(persisted.enabled !== undefined ? { enabled: persisted.enabled } : {}),
+        ...(persisted.writable !== undefined ? { writable: persisted.writable } : {}),
+        ...(isPrimary || persisted.primary ? { primary: true } : {}),
+        ...(persisted.options ? { options: persisted.options } : {}),
+        ...(persisted.wikiName ? { wikiName: persisted.wikiName } : {}),
+    };
+}

package/dist/core/config-types.js

@@ -0,0 +1,4 @@
+// This Source Code Form is subject to the terms of the Mozilla Public
+// License, v. 2.0. If a copy of the MPL was not distributed with this
+// file, You can obtain one at https://mozilla.org/MPL/2.0/.
+export {};