kibi-mcp 0.14.1 → 0.14.3

package/dist/server/docs.js

@@ -33,6 +33,7 @@ function renderToolsDoc() {
         const required = Array.isArray(tool.inputSchema?.required)
             ? tool.inputSchema.required.join(", ")
             : "none";
+        lines.push(`| \`${tool.name}\` | ${tool.description} | ${required || "none"} |`);
     }
     lines.push("");
     lines.push("Modeling note: Kibi has eight core entity types grouped into common authoring (req, scenario, test, fact) and supporting/system (adr, flag, event, symbol).");
@@ -62,7 +63,7 @@ export const PROMPTS = [
             "Call `kb_autopilot_generate` with the gathered context to synthesize candidate entities.",
             "",
             "This tool is **read-only**. It returns additive `structuredContent` with:",
-            "- `promptBlock`: review text that can be surfaced in optional human-facing briefs",
+            "- `promptBlock`: review text that can be surfaced for optional human review",
             "- `recommendedActions`: agent-facing next steps, including any REQ/SCEN/TEST authoring routed for manual handling",
             "- `declaredContext`: the user-provided bootstrap context",
             "- `confidence`: confidence summary for the generated output",
@@ -72,7 +73,7 @@ export const PROMPTS = [
             "",
             "## Step 3: Optional Human Review",
             "",
-            "Surface the `promptBlock` and a summary of `candidates` when optional human review is useful. Human review is post-hoc/optional via VS Code briefs and must not block writes.",
+            "Surface the `promptBlock` and a summary of `candidates` when optional human review is useful. Human review is post-hoc/optional and must not block writes.",
             "",
             "## Step 4: Apply Candidates",
             "",
@@ -82,37 +83,11 @@ export const PROMPTS = [
             "3. Run `kb_check` after the batch to verify KB integrity.",
             "",
             "## Rules",
-            "- Human review is optional and post-hoc via VS Code briefs; do not gate writes on synchronous sign-off.",
+            "- Human review is optional and post-hoc; do not gate writes on synchronous sign-off.",
             "- `kb_autopilot_generate` is strictly read-only; synthesis is the backend, not the actor.",
             "- Guidance must stay MCP-only; do not suggest `kibi` CLI commands.",
         ].join("\n"),
     },
-    {
-        name: "brief-kibi",
-        description: "Start-task workflow for generating a citation-backed Kibi briefing before risky work.",
-        text: [
-            "# Kibi Briefing Workflow",
-            "",
-            "Use this workflow at the start of a task when you need a deterministic, citation-backed Kibi briefing.",
-            "",
-            "## Step 1: Generate the briefing",
-            "",
-            "Call `kb_briefing_generate` with any relevant `taskText`, `sourceFiles`, and `seedIds`.",
-            "",
-            "This tool is read-only. It returns `briefingState`, `activationState`, `activationReason`, `freshness`, `confidence`, `tldr`, `promptBlock`, `entities`, `constraints`, `regressionRisks`, `missingEvidence`, and `citations`.",
-            "",
-            "## Step 2: Inspect readiness",
-            "",
-            "Inspect `briefingState` before acting.",
-            "- If `briefingState` is `ready`, continue using only cited output from the briefing.",
-            "- If `briefingState` is `no_briefing`, stop and proceed without inventing briefing claims.",
-            "",
-            "## Step 3: Use the cited output",
-            "",
-            "Use `constraints`, `regressionRisks`, `missingEvidence`, and `promptBlock` only when their claims are backed by the returned `citations` and cited `entities`.",
-            "Do not add uncited assertions, and do not treat omitted topics as verified.",
-        ].join("\n"),
-    },
     {
         name: "kibi_overview",
         description: "High-level model for using kibi-mcp safely and effectively.",

package/dist/server/tools.js

@@ -2,18 +2,18 @@ import { z } from "zod";
 import { DIAGNOSTIC_MODE_ENABLED, appendUsageLogLine, deriveDiagnosticFields, extractToolCallPayload, } from "../diagnostics.js";
 import { isMcpDebugEnabled } from "../env.js";
 import { TOOLS } from "../tools-config.js";
+import { handleKbAutopilotGenerate, } from "../tools/autopilot-generate.js";
 import { handleKbCheck } from "../tools/check.js";
 import { handleKbCoverage } from "../tools/coverage.js";
 import { handleKbDelete } from "../tools/delete.js";
 import { handleKbFindGaps } from "../tools/find-gaps.js";
 import { handleKbGraph } from "../tools/graph.js";
+import { handleKbModelRequirement, } from "../tools/model-requirement.js";
 import { handleKbQuery } from "../tools/query.js";
 import { handleKbSearch } from "../tools/search.js";
+import { handleKbSkillsList, handleKbSkillsLoad, handleKbSkillsRead, } from "../tools/skills.js";
 import { handleKbStatus } from "../tools/status.js";
 import { handleKbUpsert } from "../tools/upsert.js";
-import { handleKbModelRequirement, } from "../tools/model-requirement.js";
-import { handleKbAutopilotGenerate, } from "../tools/autopilot-generate.js";
-import { handleKbBriefingGenerate, } from "../tools/briefing-generate.js";
 const defaultToolsServerDeps = {
     getSessionModule: () => import("./session.js"),
 };
@@ -59,10 +59,12 @@ const DEFAULT_TOOLS_RUNTIME = {
     handleKbQuery,
     handleKbSearch,
     handleKbStatus,
+    handleKbSkillsList,
+    handleKbSkillsLoad,
+    handleKbSkillsRead,
     handleKbUpsert,
     handleKbModelRequirement,
     handleKbAutopilotGenerate,
-    handleKbBriefingGenerate,
 };
 // implements REQ-008
 function debugLog(...args) {
@@ -299,6 +301,9 @@ runtime = DEFAULT_TOOLS_RUNTIME) {
         const prolog = await runtime.ensureProlog();
         return runtime.handleKbStatus(prolog, args);
     }, runtime);
+    addTool(server, "kb_skills_list", toolDef("kb_skills_list").description, toolDef("kb_skills_list").inputSchema, async (args) => runtime.handleKbSkillsList(args), runtime);
+    addTool(server, "kb_skills_load", toolDef("kb_skills_load").description, toolDef("kb_skills_load").inputSchema, async (args) => runtime.handleKbSkillsLoad(args), runtime);
+    addTool(server, "kb_skills_read", toolDef("kb_skills_read").description, toolDef("kb_skills_read").inputSchema, async (args) => runtime.handleKbSkillsRead(args), runtime);
     addTool(server, "kb_find_gaps", toolDef("kb_find_gaps").description, toolDef("kb_find_gaps").inputSchema, async (args) => {
         const prolog = await runtime.ensureProlog();
         return runtime.handleKbFindGaps(prolog, args);
@@ -331,8 +336,4 @@ runtime = DEFAULT_TOOLS_RUNTIME) {
         const prolog = await runtime.ensureProlog();
         return runtime.handleKbAutopilotGenerate(prolog, args);
     }, runtime);
-    addTool(server, "kb_briefing_generate", toolDef("kb_briefing_generate").description, toolDef("kb_briefing_generate").inputSchema, async (args) => {
-        const prolog = await runtime.ensureProlog();
-        return runtime.handleKbBriefingGenerate(prolog, args);
-    }, runtime);
 }

package/dist/tools/autopilot-candidates.js

@@ -3,8 +3,8 @@
 import { extractFromManifest } from "kibi-cli/extractors/manifest";
 import { extractFromMarkdown } from "kibi-cli/extractors/markdown";
 import { buildStrictWriteSet, modelRequirementClaims, } from "kibi-cli/public/check-types";
-import path from "node:path";
 import fs from "node:fs";
+import path from "node:path";
 import { createRepoIgnorePolicy } from "kibi-cli/ignore-policy";
 import { estimateNormativeSignalConfidence, extractRequirementClaim, strictWriteSetToApplyPlan, writeSetPrimaryEntityId, } from "./model-requirement.js";
 function slugify(value, maxLength = 80) {
@@ -223,7 +223,8 @@ export function buildGenericMarkdownCandidates(discoveryResult, existingEntities
                 let type = null;
                 let confidence = 0;
                 // ADR heuristic: headings that mention ADR or Architectural Decision
-                if (/\badr\b/i.test(heading) || /architectur.*decision/i.test(heading)) {
+                if (/\badr\b/i.test(heading) ||
+                    /architectur.*decision/i.test(heading)) {
                     type = "adr";
                     confidence = 0.9;
                 }
@@ -336,7 +337,8 @@ export function collectSourceOnlyAuthoringSignals(discoveryResult, existingEntit
                         evidence: [`generic_heading:${textRef}`],
                     }, seen);
                 }
-                if (/\b(tests?|verification)\b/i.test(heading) && 0.82 >= minConfidence) {
+                if (/\b(tests?|verification)\b/i.test(heading) &&
+                    0.82 >= minConfidence) {
                     pushSignal(signals, {
                         kind: "test",
                         title: `Author tests from ${heading}`,
@@ -354,7 +356,8 @@ export function collectSourceOnlyAuthoringSignals(discoveryResult, existingEntit
     for (const item of discoveryResult.evidence ?? []) {
         const confidence = typeof item.data.confidence === "number" ? item.data.confidence : 0;
         if (item.kind === "test_topology" && confidence >= minConfidence) {
-            const sourcePath = item.absolutePath ?? path.resolve(workspaceRoot, item.relativePath ?? item.label);
+            const sourcePath = item.absolutePath ??
+                path.resolve(workspaceRoot, item.relativePath ?? item.label);
             const relativePath = item.relativePath ?? item.label;
             pushSignal(signals, {
                 kind: "test",
@@ -504,7 +507,9 @@ export function buildProviderEvidenceCandidates(discoveryResult, existingEntitie
         const generatedId = `FACT-GEN-${slugify(slugSource, 64) || "evidence"}`.toUpperCase();
         if (existingEntities.ids.has(generatedId))
             continue;
-        const textRef = relativePath.includes("#") ? relativePath : `${relativePath}`;
+        const textRef = relativePath.includes("#")
+            ? relativePath
+            : `${relativePath}`;
         const evidence = Array.isArray(item.data.evidence)
             ? item.data.evidence.filter((value) => typeof value === "string")
             : [];

package/dist/tools/autopilot-discovery.js

@@ -7,11 +7,12 @@
 import fs from "node:fs";
 import path from "node:path";
 import fg from "fast-glob";
-import { createRepoIgnorePolicy } from "kibi-cli/ignore-policy";
 import * as cliSymbolCoordinator from "kibi-cli/extractors/symbols-coordinator";
+import { createRepoIgnorePolicy } from "kibi-cli/ignore-policy";
 import { runJsonModuleQuery } from "./core-module.js";
 // implements REQ-001
 export const AUTOPILOT_PROVIDER_ORDER = [
+    // implements REQ-001
     "typed_kibi_docs",
     "generic_repo_docs",
     "repo_metadata",
@@ -323,7 +324,8 @@ function detectLanguagesFromPackageJson(packageJson) {
     const bin = packageJson.bin;
     if (typeof scripts === "object" && scripts) {
         for (const value of Object.values(scripts)) {
-            if (typeof value === "string" && /\.(cts|mts|ts|tsx)\b|\b(tsx|ts-node)\b/i.test(value)) {
+            if (typeof value === "string" &&
+                /\.(cts|mts|ts|tsx)\b|\b(tsx|ts-node)\b/i.test(value)) {
                 detected.add("typescript");
             }
             if (typeof value === "string" && /\.(cjs|mjs|js|jsx)\b/i.test(value)) {
@@ -416,11 +418,21 @@ function runRepoMetadataProvider(workspaceRoot) {
     };
 }
 function runRepoLayoutProvider(workspaceRoot, vendoredRoots) {
-    const layoutRoots = ["src", "app", "apps", "packages", "tests", "test", "docs", "scripts"];
+    const layoutRoots = [
+        "src",
+        "app",
+        "apps",
+        "packages",
+        "tests",
+        "test",
+        "docs",
+        "scripts",
+    ];
     const evidence = [];
     for (const relativePath of layoutRoots) {
         const absolutePath = path.join(workspaceRoot, relativePath);
-        if (!fs.existsSync(absolutePath) || !fs.statSync(absolutePath).isDirectory()) {
+        if (!fs.existsSync(absolutePath) ||
+            !fs.statSync(absolutePath).isDirectory()) {
             continue;
         }
         evidence.push({
@@ -547,7 +559,8 @@ function runSourceSymbolsProvider(workspaceRoot, vendoredRoots) {
     const scanWarnings = [];
     for (const absolutePath of sortUnique(sourceFiles)) {
         const relativePath = toRelativePosixPath(workspaceRoot, absolutePath);
-        const language = SOURCE_LANGUAGE_EXTENSIONS[path.extname(absolutePath).toLowerCase()] ?? "unknown";
+        const language = SOURCE_LANGUAGE_EXTENSIONS[path.extname(absolutePath).toLowerCase()] ??
+            "unknown";
         detectedLanguages.add(language);
         try {
             const content = fs.readFileSync(absolutePath, "utf8");
@@ -558,7 +571,8 @@ function runSourceSymbolsProvider(workspaceRoot, vendoredRoots) {
                     language,
                     providerId: null,
                     module: {
-                        title: path.basename(relativePath, path.extname(relativePath)) || relativePath,
+                        title: path.basename(relativePath, path.extname(relativePath)) ||
+                            relativePath,
                         analysisMode: "fallback",
                         fallbackReason: "provider_unavailable",
                     },

package/dist/tools/autopilot-generate.js

@@ -1,11 +1,11 @@
 import path from "node:path";
 import fg from "fast-glob";
 import { createRepoIgnorePolicy } from "kibi-cli/ignore-policy";
-import { buildNormativeRequirementCandidates, collectSourceOnlyAuthoringSignals, buildGenericMarkdownCandidates, buildProviderEvidenceCandidates, buildTypedMarkdownCandidates, buildSymbolManifestCandidates, } from "./autopilot-candidates.js";
-import { getWorkspaceMigrationWarning } from "./model-requirement.js";
+import { resolveWorkspaceRoot } from "../workspace.js";
+import { buildGenericMarkdownCandidates, buildNormativeRequirementCandidates, buildProviderEvidenceCandidates, buildSymbolManifestCandidates, buildTypedMarkdownCandidates, collectSourceOnlyAuthoringSignals, } from "./autopilot-candidates.js";
 import { discoverProviderEvidence, resolveActivationPolicy, } from "./autopilot-discovery.js";
 import { loadEntities } from "./entity-query.js";
-import { resolveWorkspaceRoot } from "../workspace.js";
+import { getWorkspaceMigrationWarning } from "./model-requirement.js";
 function clamp(value, min, max) {
     return Math.max(min, Math.min(max, value));
 }
@@ -115,8 +115,9 @@ function buildPromptBlock(workspaceRoot, activationState, activationMode, activa
     else if (declaredContext.verificationAnchors.length > 0) {
         bullets.push(`- Verify after kb_check with ${listSummary(declaredContext.verificationAnchors, 2)}.`);
     }
-    if (activationMode === "attached_thin_handoff" || activationMode === "attached_seeded_handoff") {
-        bullets.push("- Handoff: use kb_search, kb_briefing_generate, or kb_find_gaps to work with existing KB.");
+    if (activationMode === "attached_thin_handoff" ||
+        activationMode === "attached_seeded_handoff") {
+        bullets.push("- Handoff: use kb_search, kb_query, or gap/coverage tools to work with existing KB.");
     }
     if (scanWarnings.length > 0) {
         bullets.push(`- Scan diagnostics: ${scanWarnings.length} warning(s) during evidence collection.`);
@@ -159,7 +160,8 @@ function buildRecommendedActions(workspaceRoot, activationMode, activationReason
     const candidateIds = candidateRecords
         .map((candidate) => String(candidate.candidateId ?? ""))
         .filter(Boolean);
-    const isActiveRepo = activationMode === "attached_thin_handoff" || activationMode === "attached_seeded_handoff";
+    const isActiveRepo = activationMode === "attached_thin_handoff" ||
+        activationMode === "attached_seeded_handoff";
     actions.push({
         order: order++,
         kind: "query",
@@ -176,7 +178,7 @@ function buildRecommendedActions(workspaceRoot, activationMode, activationReason
         actions.push({
             order: order++,
             kind: "handoff",
-            description: "Use kb_briefing_generate with task-relevant seed IDs for a citation-backed briefing.",
+            description: "Use kb_query or kb_graph with task-relevant IDs to inspect cited KB context.",
         });
         actions.push({
             order: order++,
@@ -190,7 +192,8 @@ function buildRecommendedActions(workspaceRoot, activationMode, activationReason
         actions.push({
             order: order++,
             kind: "handoff",
-            description: handoffMessage ?? blockedActivationMessage(activationMode, activationReason),
+            description: handoffMessage ??
+                blockedActivationMessage(activationMode, activationReason),
         });
     }
     else if (candidateIds.length > 0) {
@@ -276,7 +279,11 @@ function buildConfidence(activationMode, applyBlocked, declaredContext, candidat
     }
     const rounded = roundScore(score);
     const level = rounded > 0.7 ? "high" : rounded >= 0.4 ? "medium" : "low";
-    const policy = level === "high" ? "full_actions" : level === "medium" ? "review_required" : "handoff_only";
+    const policy = level === "high"
+        ? "full_actions"
+        : level === "medium"
+            ? "review_required"
+            : "handoff_only";
     if (policy === "review_required") {
         reasons.push("Medium confidence: review recommended before applying.");
     }
@@ -295,7 +302,8 @@ function buildTldr(activationMode, applyBlocked, candidateRecords, sourceOnlySig
         if (candidateRecords.length > 0 || sourceOnlySignals.length > 0) {
             return `Bootstrap guidance is ready in ${activationMode}: ${candidateRecords.length} safe candidate(s), ${sourceOnlySignals.length} source-only authoring follow-up(s), and apply remains blocked.`;
         }
-        return handoffMessage ?? blockedActivationMessage(activationMode, activationReason);
+        return (handoffMessage ??
+            blockedActivationMessage(activationMode, activationReason));
     }
     if (candidateRecords.length > 0 || sourceOnlySignals.length > 0) {
         return `Bootstrap output is ready with ${candidateRecords.length} safe candidate(s) and ${sourceOnlySignals.length} source-only authoring follow-up(s).`;
@@ -352,7 +360,8 @@ function splitDiscoveredSources(workspaceRoot, candidates) {
     }
     return { markdownFiles, manifestFiles };
 }
-export async function handleKbAutopilotGenerate(// implements REQ-mcp-init-kibi-autopilot-v1
+export async function handleKbAutopilotGenerate(
+// implements REQ-mcp-init-kibi-autopilot-v1
 _prolog, args) {
     const { includeGenericMarkdown = true, minConfidence = 0.8, maxCandidates = 50, entityTypes, bootstrapContext, } = args;
     const normalizedMinConfidence = clamp(minConfidence, 0.6, 0.95);
@@ -473,7 +482,9 @@ _prolog, args) {
             const sourcePath = String(candidate.sourcePath || "");
             const textRef = extractTextRefFromApplyPlan(candidate.applyPlan);
             const titleKey = normalizeTitle(entityType, title);
-            const upsert = Array.isArray(candidate.applyPlan) ? candidate.applyPlan[0] : null;
+            const upsert = Array.isArray(candidate.applyPlan)
+                ? candidate.applyPlan[0]
+                : null;
             let upsertId = "";
             if (upsert && typeof upsert === "object") {
                 const upsertRecord = upsert;
@@ -545,7 +556,8 @@ _prolog, args) {
             const explain = repoIgnore.explain(rel);
             if (explain.ignored) {
                 // avoid duplicating existing suppressed entries for the same source
-                if (!suppressed.some((s) => String(s.sourcePath ?? "") === rel && s.reason === "ignored_source")) {
+                if (!suppressed.some((s) => String(s.sourcePath ?? "") === rel &&
+                    s.reason === "ignored_source")) {
                     suppressed.push({
                         candidateId: String("") /* no candidate id for ignored source */,
                         reason: "ignored_source",
@@ -567,7 +579,9 @@ _prolog, args) {
     const recommendedActions = buildRecommendedActions(workspaceRoot, activation.activationMode, activation.reason, activation.handoffMessage, activation.applyBlocked, declaredContext, candidateRecords, sourceOnlySignals);
     const tldr = buildTldr(activation.activationMode, activation.applyBlocked, candidateRecords, sourceOnlySignals, activation.reason, activation.handoffMessage);
     // Apply confidence policy: medium and low confidence force applyBlocked
-    const effectiveApplyBlocked = activation.applyBlocked || confidence.level === "medium" || confidence.level === "low";
+    const effectiveApplyBlocked = activation.applyBlocked ||
+        confidence.level === "medium" ||
+        confidence.level === "low";
     const effectiveTldr = confidence.level === "low" && !activation.applyBlocked
         ? `Low-confidence bootstrap (${confidence.score}): review diagnostics before proceeding. ${tldr}`
         : tldr;

package/dist/tools/delete.js

@@ -1,10 +1,10 @@
-import { escapeAtom, parseEntityFromList, parseListOfLists } from "kibi-cli/prolog/codec";
-import { writeBriefPendingMarker } from "../utils/brief-marker.js";
+import { escapeAtom, parseEntityFromList, parseListOfLists, } from "kibi-cli/prolog/codec";
 /**
  * Handle kb.delete tool calls
  * Prevents deletion of entities with dependents (referential integrity)
  */
-export async function handleKbDelete(// implements REQ-002, REQ-011
+export async function handleKbDelete(
+// implements REQ-002, REQ-011
 prolog, args) {
     const { ids } = args;
     if (!ids || ids.length === 0) {
@@ -13,8 +13,6 @@ prolog, args) {
     let deleted = 0;
     let skipped = 0;
     const errors = [];
-    const pendingEntityIds = [];
-    const pendingRelationships = [];
     try {
         for (const id of ids) {
             const safeId = escapeAtom(id);
@@ -45,7 +43,6 @@ prolog, args) {
             }
             // No dependents, safe to delete
             const entityMetadata = await loadEntityMetadataForDelete(prolog, id, safeId);
-            const relationships = await loadOutgoingRelationshipsForDelete(prolog, safeId);
             const deleteGoal = buildDeleteGoal(safeId, entityMetadata);
             const deleteResult = await prolog.query(deleteGoal);
             if (!deleteResult.success) {
@@ -54,8 +51,6 @@ prolog, args) {
             }
             else {
                 deleted++;
-                pendingEntityIds.push(id);
-                pendingRelationships.push(...relationships);
             }
         }
         // Save KB to disk
@@ -63,14 +58,6 @@ prolog, args) {
         if (!saveResult.success) {
             throw new Error(`Failed to save KB after delete: ${saveResult.error || "Unknown error"}`);
         }
-        if (pendingEntityIds.length > 0 || pendingRelationships.length > 0) {
-            writeBriefPendingMarker({
-                ...(args._requestId ? { sessionId: args._requestId } : {}),
-                operation: "delete",
-                entityIds: pendingEntityIds,
-                relationships: pendingRelationships,
-            });
-        }
         prolog.invalidateCache();
         return {
             content: [
@@ -96,7 +83,9 @@ async function loadEntityMetadataForDelete(prolog, id, safeId) {
     if (!result.success) {
         throw new Error(`Failed to load metadata for entity ${id}: ${result.error || "Unknown error"}`);
     }
-    const rows = result.bindings.Results ? parseListOfLists(result.bindings.Results) : [];
+    const rows = result.bindings.Results
+        ? parseListOfLists(result.bindings.Results)
+        : [];
     if (rows.length === 0) {
         throw new Error(`Failed to load metadata for entity ${id}: Entity not found`);
     }
@@ -105,40 +94,11 @@ async function loadEntityMetadataForDelete(prolog, id, safeId) {
     const { id: _entityId, type: _entityType, ...props } = entity;
     return { type, props };
 }
-async function loadOutgoingRelationshipsForDelete(prolog, safeId) {
-    const result = await prolog.query(`findall([Type,'${safeId}',To], (member(Type, [depends_on, verified_by, validates, specified_by, relates_to, guards, publishes, consumes, implements, covered_by, executable_for, constrains, requires_property, supersedes, constrained_by]), kb_relationship(Type, '${safeId}', To)), Relationships)`);
-    if (!result.success) {
-        throw new Error(`Failed to load outgoing relationships for entity ${safeId}: ${result.error || "Unknown error"}`);
-    }
-    const rows = result.bindings.Relationships
-        ? parseListOfLists(result.bindings.Relationships)
-        : [];
-    return rows.flatMap((row) => {
-        const type = row[0];
-        const from = row[1];
-        const to = row[2];
-        if (type === undefined || from === undefined || to === undefined) {
-            return [];
-        }
-        return [
-            {
-                type: normalizeDeleteRelationshipValue(type),
-                from: normalizeDeleteRelationshipValue(from),
-                to: normalizeDeleteRelationshipValue(to),
-            },
-        ];
-    });
-}
-function normalizeDeleteRelationshipValue(value) {
-    const normalized = String(value);
-    if ((normalized.startsWith("'") && normalized.endsWith("'")) ||
-        (normalized.startsWith('"') && normalized.endsWith('"'))) {
-        return normalized.slice(1, -1);
-    }
-    return normalized;
-}
 function buildDeleteGoal(safeId, metadata) {
-    const auditProps = [`id='${safeId}'`, ...serializeDeleteProps(metadata.props)];
+    const auditProps = [
+        `id='${safeId}'`,
+        ...serializeDeleteProps(metadata.props),
+    ];
     return `kb_retract_entity('${safeId}', ${metadata.type}, [${auditProps.join(", ")}])`;
 }
 function serializeDeleteProps(props) {

package/dist/tools/model-requirement.js

@@ -29,11 +29,15 @@ function normalizeSourceFiles(sourceFiles) {
     return normalized;
 }
 function clampConfidence(confidence) {
-    const numeric = typeof confidence === "number" && Number.isFinite(confidence) ? confidence : 0.8;
+    const numeric = typeof confidence === "number" && Number.isFinite(confidence)
+        ? confidence
+        : 0.8;
     return Math.round(Math.min(1, Math.max(0, numeric)) * 100) / 100;
 }
 function normalizeClaimValue(value) {
-    if (typeof value === "string" || typeof value === "number" || typeof value === "boolean") {
+    if (typeof value === "string" ||
+        typeof value === "number" ||
+        typeof value === "boolean") {
         if (typeof value === "number" && !Number.isFinite(value)) {
             throw new Error("Requirement modeling failed: value must be a finite number");
         }
@@ -58,7 +62,10 @@ function cleanPredicate(value) {
     return trimSentenceTail(stripListPrefix(value)) || "statement";
 }
 function fallbackSubjectFromSource(source) {
-    const basename = path.basename(source, path.extname(source)).replace(/[-_]+/g, " ").trim();
+    const basename = path
+        .basename(source, path.extname(source))
+        .replace(/[-_]+/g, " ")
+        .trim();
     return basename || "Requirement";
 }
 function hasExplicitClaimFields(args) {

package/dist/tools/skills.js

@@ -0,0 +1,75 @@
+import { createHash } from "node:crypto";
+import { listBundledSkills, loadBundledSkill, readBundledSkillResource, } from "kibi-cli/skills";
+// implements REQ-001
+export async function handleKbSkillsList(_args) {
+    try {
+        const skills = listBundledSkills();
+        const ids = skills.map((skill) => skill.id).join(", ") || "none";
+        return {
+            content: [
+                { type: "text", text: `Found ${skills.length} bundled skills: ${ids}` },
+            ],
+            structuredContent: { skills },
+        };
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        throw new Error(`Skills list failed: ${message}`);
+    }
+}
+// implements REQ-001
+export async function handleKbSkillsLoad(args) {
+    try {
+        assertNonEmptyString(args.id, "id");
+        const bundle = loadBundledSkill(args.id);
+        const resources = bundle.manifest.resources ?? [];
+        const payload = {
+            metadata: bundle.manifest,
+            body: bundle.body,
+            resources,
+            contentHash: createHash("sha256")
+                .update(bundle.body, "utf8")
+                .digest("hex"),
+            sourceType: "bundled",
+        };
+        return {
+            content: [
+                {
+                    type: "text",
+                    text: `Loaded bundled skill ${bundle.manifest.id} with ${resources.length} resources`,
+                },
+            ],
+            structuredContent: payload,
+        };
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        throw new Error(`Skills load failed: ${message}`);
+    }
+}
+// implements REQ-001
+export async function handleKbSkillsRead(args) {
+    try {
+        assertNonEmptyString(args.id, "id");
+        assertNonEmptyString(args.resource, "resource");
+        const resourceContent = readBundledSkillResource(args.id, args.resource);
+        return {
+            content: [
+                {
+                    type: "text",
+                    text: `Read bundled skill resource ${args.id}/${args.resource}`,
+                },
+            ],
+            structuredContent: { content: resourceContent },
+        };
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        throw new Error(`Skills read failed: ${message}`);
+    }
+}
+function assertNonEmptyString(value, field) {
+    if (typeof value !== "string" || value.trim() === "") {
+        throw new Error(`${field} must be a non-empty string`);
+    }
+}

package/dist/tools/symbols.js

@@ -270,7 +270,7 @@ function normalizeCoordinateRecord(value) {
     if (!isRecord(value)) {
         return null;
     }
-    const { sourceColumn, sourceEndColumn, sourceEndLine, sourceFile, sourceLine } = value;
+    const { sourceColumn, sourceEndColumn, sourceEndLine, sourceFile, sourceLine, } = value;
     if (typeof sourceFile !== "string" ||
         typeof sourceLine !== "number" ||
         typeof sourceColumn !== "number" ||

package/dist/tools/upsert.js

@@ -20,7 +20,6 @@ import { escapeAtom, toPrologAtom, toPrologString, } from "kibi-cli/prolog/codec
 import entitySchema from "kibi-cli/schemas/entity";
 import relationshipSchema from "kibi-cli/schemas/relationship";
 import { isMcpDebugEnabled } from "../env.js";
-import { writeBriefPendingMarker } from "../utils/brief-marker.js";
 import { refreshCoordinatesForSymbolId } from "./symbols.js";
 let refreshCoordinatesForSymbolIdImpl = refreshCoordinatesForSymbolId;
 const ajv = new Ajv({ strict: false });
@@ -156,16 +155,6 @@ export async function handleKbUpsert(prolog, args) {
         if (!saveResult.success) {
             throw new Error(`Failed to save KB after upsert: ${saveResult.error || "Unknown error"}`);
         }
-        writeBriefPendingMarker({
-            ...(args._requestId ? { sessionId: args._requestId } : {}),
-            operation: "upsert",
-            entityIds: [id],
-            relationships: relationships.map((rel) => ({
-                from: String(rel.from),
-                to: String(rel.to),
-                type: String(rel.type),
-            })),
-        });
         if (type === "symbol") {
             try {
                 await refreshCoordinatesForSymbolIdImpl(id);