context-vault 3.1.6 → 3.1.8

package/src/tools/{context-status.js → context-status.ts}

@@ -1,42 +1,39 @@
-import { existsSync, readFileSync } from "node:fs";
-import { join } from "node:path";
-import { gatherVaultStatus, computeGrowthWarnings } from "../status.js";
-import { errorLogPath, errorLogCount } from "../error-log.js";
-import { ok, err } from "../helpers.js";
-
-function relativeTime(ts) {
+import { existsSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
+import { gatherVaultStatus, computeGrowthWarnings } from '../status.js';
+import { errorLogPath, errorLogCount } from '../error-log.js';
+import { ok, err } from '../helpers.js';
+import type { LocalCtx, ToolResult } from '../types.js';
+
+function relativeTime(ts: number): string {
   const secs = Math.floor((Date.now() - ts) / 1000);
   if (secs < 60) return `${secs}s ago`;
   const mins = Math.floor(secs / 60);
-  if (mins < 60) return `${mins} minute${mins === 1 ? "" : "s"} ago`;
+  if (mins < 60) return `${mins} minute${mins === 1 ? '' : 's'} ago`;
   const hrs = Math.floor(mins / 60);
-  return `${hrs} hour${hrs === 1 ? "" : "s"} ago`;
+  return `${hrs} hour${hrs === 1 ? '' : 's'} ago`;
 }
 
-export const name = "context_status";
+export const name = 'context_status';
 
 export const description =
-  "Show vault health: resolved config, file counts per kind, database size, and any issues. Use to verify setup or troubleshoot. Call this when a user asks about their vault or to debug search issues.";
+  'Show vault health: resolved config, file counts per kind, database size, and any issues. Use to verify setup or troubleshoot. Call this when a user asks about their vault or to debug search issues.';
 
 export const inputSchema = {};
 
-/**
- * @param {object} _args
- * @param {import('../types.js').BaseCtx & Partial<import('../types.js').HostedCtxExtensions>} ctx
- */
-export function handler(_args, ctx) {
+export function handler(_args: Record<string, any>, ctx: LocalCtx): ToolResult {
   try {
     const { config } = ctx;
 
     const status = gatherVaultStatus(ctx);
 
-    const hasIssues = status.stalePaths || status.embeddingStatus?.missing > 0;
-    const healthIcon = hasIssues ? "⚠" : "✓";
+    const hasIssues = status.stalePaths || (status.embeddingStatus?.missing ?? 0) > 0;
+    const healthIcon = hasIssues ? '⚠' : '✓';
 
     const lines = [
       `## ${healthIcon} Vault Status (connected)`,
       ``,
-      `Vault:     ${config.vaultDir} (${config.vaultDirExists ? status.fileCount + " files" : "missing"})`,
+      `Vault:     ${config.vaultDir} (${config.vaultDirExists ? status.fileCount + ' files' : 'missing'})`,
       `Database:  ${config.dbPath} (${status.dbSize})`,
       `Dev dir:   ${config.devDir}`,
       `Data dir:  ${config.dataDir}`,
@@ -51,26 +48,21 @@ export function handler(_args, ctx) {
       lines.push(`Embeddings: ${indexed}/${total} (${pct}%)`);
     }
     if (status.embedModelAvailable === false) {
-      lines.push(
-        `Embed model: unavailable (semantic search disabled, FTS still works)`,
-      );
+      lines.push(`Embed model: unavailable (semantic search disabled, FTS still works)`);
     } else if (status.embedModelAvailable === true) {
       lines.push(`Embed model: loaded`);
     }
-    lines.push(
-      `Decay:     ${config.eventDecayDays} days (event recency window)`,
-    );
+    lines.push(`Decay:     ${config.eventDecayDays} days (event recency window)`);
     if (status.expiredCount > 0) {
       lines.push(
-        `Expired:   ${status.expiredCount} entries pending prune (run \`context-vault prune\` to remove now)`,
+        `Expired:   ${status.expiredCount} entries pending prune (run \`context-vault prune\` to remove now)`
       );
     }
 
     lines.push(``, `### Indexed`);
 
     if (status.kindCounts.length) {
-      for (const { kind, c } of status.kindCounts)
-        lines.push(`- ${c} ${kind}s`);
+      for (const { kind, c } of status.kindCounts) lines.push(`- ${c} ${kind}s`);
     } else {
       lines.push(`- (empty)`);
     }
@@ -78,23 +70,19 @@ export function handler(_args, ctx) {
     if (status.categoryCounts.length) {
       lines.push(``);
       lines.push(`### Categories`);
-      for (const { category, c } of status.categoryCounts)
-        lines.push(`- ${category}: ${c}`);
+      for (const { category, c } of status.categoryCounts) lines.push(`- ${category}: ${c}`);
     }
 
     if (status.subdirs.length) {
       lines.push(``);
       lines.push(`### Disk Directories`);
-      for (const { name, count } of status.subdirs)
-        lines.push(`- ${name}/: ${count} files`);
+      for (const { name, count } of status.subdirs) lines.push(`- ${name}/: ${count} files`);
     }
 
     if (status.stalePaths) {
       lines.push(``);
       lines.push(`### ⚠ Stale Paths`);
-      lines.push(
-        `DB contains ${status.staleCount} paths not matching current vault dir.`,
-      );
+      lines.push(`DB contains ${status.staleCount} paths not matching current vault dir.`);
       lines.push(`Auto-reindex will fix this on next search or save.`);
     }
 
@@ -102,19 +90,13 @@ export function handler(_args, ctx) {
       lines.push(``);
       lines.push(`### ⚠ Potentially Stale Knowledge`);
       lines.push(
-        `Not updated within kind staleness window (pattern: 180d, decision: 365d, reference: 90d):`,
+        `Not updated within kind staleness window (pattern: 180d, decision: 365d, reference: 90d):`
       );
       for (const entry of status.staleKnowledge) {
-        const lastUpdated = entry.last_updated
-          ? entry.last_updated.split("T")[0]
-          : "unknown";
-        lines.push(
-          `- "${entry.title}" (${entry.kind}) — last updated ${lastUpdated}`,
-        );
+        const lastUpdated = entry.last_updated ? entry.last_updated.split('T')[0] : 'unknown';
+        lines.push(`- "${entry.title}" (${entry.kind}) — last updated ${lastUpdated}`);
       }
-      lines.push(
-        `Use save_context to refresh or add expires_at to retire stale entries.`,
-      );
+      lines.push(`Use save_context to refresh or add expires_at to retire stale entries.`);
     }
 
     // Error log
@@ -127,10 +109,10 @@ export function handler(_args, ctx) {
     }
 
     // Last startup error
-    const lastErrorPath = join(config.dataDir, ".last-error");
+    const lastErrorPath = join(config.dataDir, '.last-error');
     if (existsSync(lastErrorPath)) {
       try {
-        const lastError = readFileSync(lastErrorPath, "utf-8").trim();
+        const lastError = readFileSync(lastErrorPath, 'utf-8').trim();
         lines.push(``, `### Last Startup Error`);
         lines.push(`\`\`\``);
         lines.push(lastError);
@@ -145,13 +127,11 @@ export function handler(_args, ctx) {
       lines.push(`- Tool calls (session): ${ts.ok} ok, ${ts.errors} errors`);
       if (ts.lastError) {
         const { tool, code, timestamp } = ts.lastError;
-        lines.push(
-          `- Last error: ${tool ?? "unknown"} — ${code} (${relativeTime(timestamp)})`,
-        );
+        lines.push(`- Last error: ${tool ?? 'unknown'} — ${code} (${relativeTime(timestamp)})`);
       }
       if (status.autoCapturedFeedbackCount > 0) {
         lines.push(
-          `- Auto-captured feedback entries: ${status.autoCapturedFeedbackCount} (run get_context with kind:feedback tags:auto-captured)`,
+          `- Auto-captured feedback entries: ${status.autoCapturedFeedbackCount} (run get_context with kind:feedback tags:auto-captured)`
         );
       }
     }
@@ -159,19 +139,19 @@ export function handler(_args, ctx) {
     // Growth warnings
     const growth = computeGrowthWarnings(status, config.thresholds);
     if (growth.hasWarnings) {
-      lines.push("", "### ⚠ Vault Growth Warning");
+      lines.push('', '### ⚠ Vault Growth Warning');
       for (const w of growth.warnings) {
         lines.push(`  ${w.message}`);
       }
       if (growth.kindBreakdown.length) {
-        lines.push("");
-        lines.push("  Breakdown by kind:");
+        lines.push('');
+        lines.push('  Breakdown by kind:');
         for (const { kind, count, pct } of growth.kindBreakdown) {
           lines.push(`    ${kind}: ${count.toLocaleString()} (${pct}%)`);
         }
       }
       if (growth.actions.length) {
-        lines.push("", "Suggested growth actions:");
+        lines.push('', 'Suggested growth actions:');
         for (const a of growth.actions) {
           lines.push(`  • ${a}`);
         }
@@ -180,23 +160,20 @@ export function handler(_args, ctx) {
 
     // Suggested actions
     const actions = [];
-    if (status.stalePaths)
-      actions.push("- Run `context-vault reindex` to fix stale paths");
-    if (status.embeddingStatus?.missing > 0)
-      actions.push(
-        "- Run `context-vault reindex` to generate missing embeddings",
-      );
+    if (status.stalePaths) actions.push('- Run `context-vault reindex` to fix stale paths');
+    if ((status.embeddingStatus?.missing ?? 0) > 0)
+      actions.push('- Run `context-vault reindex` to generate missing embeddings');
     if (!config.vaultDirExists)
-      actions.push("- Run `context-vault setup` to create the vault directory");
+      actions.push('- Run `context-vault setup` to create the vault directory');
     if (status.kindCounts.length === 0 && config.vaultDirExists)
-      actions.push("- Use `save_context` to add your first entry");
+      actions.push('- Use `save_context` to add your first entry');
 
     if (actions.length) {
-      lines.push("", "### Suggested Actions", ...actions);
+      lines.push('', '### Suggested Actions', ...actions);
     }
 
-    return ok(lines.join("\n"));
+    return ok(lines.join('\n'));
   } catch (e) {
-    return err(e.message, "STATUS_FAILED");
+    return err(e instanceof Error ? e.message : String(e), 'STATUS_FAILED');
   }
 }

package/src/tools/{create-snapshot.js → create-snapshot.ts}

@@ -1,106 +1,106 @@
-import { z } from "zod";
-import { hybridSearch } from "@context-vault/core/search";
-import { captureAndIndex } from "@context-vault/core/capture";
-import { normalizeKind } from "@context-vault/core/files";
-import { ok, err, ensureVaultExists } from "../helpers.js";
-
-const NOISE_KINDS = new Set(["prompt-history", "task-notification"]);
+import { z } from 'zod';
+import { hybridSearch } from '@context-vault/core/search';
+import { captureAndIndex } from '@context-vault/core/capture';
+import { normalizeKind } from '@context-vault/core/files';
+import { ok, err, ensureVaultExists } from '../helpers.js';
+import type { LocalCtx, SharedCtx, ToolResult } from '../types.js';
+
+const NOISE_KINDS = new Set(['prompt-history', 'task-notification']);
 const MAX_ENTRIES_FOR_GATHER = 40;
 const MAX_BODY_PER_ENTRY = 600;
 
-export const name = "create_snapshot";
+export const name = 'create_snapshot';
 
 export const description =
   "Pull all relevant vault entries matching a topic, deduplicate, and save them as a structured context brief (kind: 'brief'). Entries are formatted as markdown — no external API or LLM call required. The calling agent can synthesize the gathered content directly. Retrieve with: get_context(kind: 'brief', identity_key: '<key>').";
 
 export const inputSchema = {
-  topic: z.string().describe("The topic or project name to snapshot"),
+  topic: z.string().describe('The topic or project name to snapshot'),
   tags: z
     .array(z.string())
     .optional()
-    .describe("Optional tag filters — entries must match at least one"),
+    .describe('Optional tag filters — entries must match at least one'),
   buckets: z
     .array(z.string())
     .optional()
     .describe(
-      "Filter by project-scoped buckets. Each name expands to a 'bucket:<name>' tag. Composes with 'tags' via OR (entries matching any tag or any bucket are included).",
+      "Filter by project-scoped buckets. Each name expands to a 'bucket:<name>' tag. Composes with 'tags' via OR (entries matching any tag or any bucket are included)."
     ),
   kinds: z
     .array(z.string())
     .optional()
-    .describe("Optional kind filters to restrict which entry types are pulled"),
+    .describe('Optional kind filters to restrict which entry types are pulled'),
   identity_key: z
     .string()
     .optional()
     .describe(
-      "Deterministic key for the saved brief (defaults to slugified topic). Use the same key to overwrite a previous snapshot.",
+      'Deterministic key for the saved brief (defaults to slugified topic). Use the same key to overwrite a previous snapshot.'
     ),
 };
 
-function formatGatheredEntries(topic, entries) {
+function formatGatheredEntries(topic: string, entries: any[]): string {
   const header = [
     `# ${topic} — Context Brief`,
-    "",
-    `*Gathered from ${entries.length} vault ${entries.length === 1 ? "entry" : "entries"}. Synthesize the content below to extract key decisions, patterns, and constraints.*`,
-    "",
-    "---",
-    "",
-  ].join("\n");
+    '',
+    `*Gathered from ${entries.length} vault ${entries.length === 1 ? 'entry' : 'entries'}. Synthesize the content below to extract key decisions, patterns, and constraints.*`,
+    '',
+    '---',
+    '',
+  ].join('\n');
 
   const body = entries
-    .map((e, i) => {
+    .map((e: any, i: number) => {
       const tags = e.tags ? JSON.parse(e.tags) : [];
-      const tagStr = tags.length ? tags.join(", ") : "none";
-      const updated = e.updated_at || e.created_at || "unknown";
+      const tagStr = tags.length ? tags.join(', ') : 'none';
+      const updated = e.updated_at || e.created_at || 'unknown';
       const bodyText = e.body
-        ? e.body.slice(0, MAX_BODY_PER_ENTRY) +
-          (e.body.length > MAX_BODY_PER_ENTRY ? "…" : "")
-        : "(no body)";
+        ? e.body.slice(0, MAX_BODY_PER_ENTRY) + (e.body.length > MAX_BODY_PER_ENTRY ? '…' : '')
+        : '(no body)';
       const title = e.title || `Entry ${i + 1}`;
       return [
         `## ${i + 1}. [${e.kind}] ${title}`,
-        "",
+        '',
         `**Tags:** ${tagStr}`,
         `**Updated:** ${updated}`,
         `**ID:** \`${e.id}\``,
-        "",
+        '',
         bodyText,
-        "",
-        "---",
-        "",
-      ].join("\n");
+        '',
+        '---',
+        '',
+      ].join('\n');
     })
-    .join("");
+    .join('');
 
   return header + body;
 }
 
-function slugifyTopic(topic) {
+function slugifyTopic(topic: string): string {
   return topic
     .toLowerCase()
-    .replace(/[^a-z0-9]+/g, "-")
-    .replace(/^-+|-+$/g, "")
+    .replace(/[^a-z0-9]+/g, '-')
+    .replace(/^-+|-+$/g, '')
     .slice(0, 120);
 }
 
 export async function handler(
-  { topic, tags, buckets, kinds, identity_key },
-  ctx,
-  { ensureIndexed },
-) {
+  { topic, tags, buckets, kinds, identity_key }: Record<string, any>,
+  ctx: LocalCtx,
+  { ensureIndexed }: SharedCtx
+): Promise<ToolResult> {
   const { config } = ctx;
 
   const vaultErr = ensureVaultExists(config);
   if (vaultErr) return vaultErr;
 
   if (!topic?.trim()) {
-    return err("Required: topic (non-empty string)", "INVALID_INPUT");
+    return err('Required: topic (non-empty string)', 'INVALID_INPUT');
   }
 
   await ensureIndexed();
 
   const normalizedKinds = kinds?.map(normalizeKind) ?? [];
-  const bucketTags = buckets?.length ? buckets.map((b) => `bucket:${b}`) : [];
+  const bucketTags = buckets?.length ? buckets.map((b: string) => `bucket:${b}`) : [];
   const effectiveTags = [...(tags ?? []), ...bucketTags];
 
   let candidates = [];
@@ -130,7 +130,7 @@ export async function handler(
       });
     }
   } catch (e) {
-    return err(e.message, "SEARCH_FAILED");
+    return err(e instanceof Error ? e.message : String(e), 'SEARCH_FAILED');
   }
 
   if (effectiveTags.length) {
@@ -140,40 +140,33 @@ export async function handler(
     });
   }
 
-  const noiseIds = candidates
-    .filter((r) => NOISE_KINDS.has(r.kind))
-    .map((r) => r.id);
+  const noiseIds = candidates.filter((r) => NOISE_KINDS.has(r.kind)).map((r) => r.id);
 
   const gatherEntries = candidates.filter((r) => !NOISE_KINDS.has(r.kind));
 
   if (gatherEntries.length === 0) {
     return err(
       `No entries found for topic "${topic}". Try a broader topic or different tags.`,
-      "NO_ENTRIES",
+      'NO_ENTRIES'
     );
   }
 
   const briefBody = formatGatheredEntries(topic, gatherEntries);
 
-  const effectiveIdentityKey =
-    identity_key ?? `snapshot-${slugifyTopic(topic)}`;
+  const effectiveIdentityKey = identity_key ?? `snapshot-${slugifyTopic(topic)}`;
 
-  const briefTags = [
-    "snapshot",
-    ...(tags ?? []),
-    ...(normalizedKinds.length > 0 ? [] : []),
-  ];
+  const briefTags = ['snapshot', ...(tags ?? []), ...(normalizedKinds.length > 0 ? [] : [])];
 
   const supersedes = noiseIds.length > 0 ? noiseIds : undefined;
 
   let entry;
   try {
     entry = await captureAndIndex(ctx, {
-      kind: "brief",
+      kind: 'brief',
       title: `${topic} — Context Brief`,
       body: briefBody,
       tags: briefTags,
-      source: "create_snapshot",
+      source: 'create_snapshot',
       identity_key: effectiveIdentityKey,
       supersedes,
 
@@ -185,7 +178,7 @@ export async function handler(
       },
     });
   } catch (e) {
-    return err(e.message, "SAVE_FAILED");
+    return err(e instanceof Error ? e.message : String(e), 'SAVE_FAILED');
   }
 
   const parts = [
@@ -193,16 +186,12 @@ export async function handler(
     `  title: ${entry.title}`,
     `  identity_key: ${effectiveIdentityKey}`,
     `  synthesized from: ${gatherEntries.length} entries`,
-    noiseIds.length > 0
-      ? `  noise superseded: ${noiseIds.length} entries`
-      : null,
-    "",
-    "_Retrieve with: get_context(kind: 'brief', identity_key: '" +
-      effectiveIdentityKey +
-      "')_",
+    noiseIds.length > 0 ? `  noise superseded: ${noiseIds.length} entries` : null,
+    '',
+    "_Retrieve with: get_context(kind: 'brief', identity_key: '" + effectiveIdentityKey + "')_",
   ]
     .filter((l) => l !== null)
-    .join("\n");
+    .join('\n');
 
   return ok(parts);
 }

package/src/tools/delete-context.ts

@@ -0,0 +1,53 @@
+import { z } from 'zod';
+import { unlinkSync } from 'node:fs';
+import { ok, err } from '../helpers.js';
+import type { LocalCtx, SharedCtx, ToolResult } from '../types.js';
+
+export const name = 'delete_context';
+
+export const description =
+  'Delete an entry from your vault by its ULID id. Removes the file from disk and cleans up the search index.';
+
+export const inputSchema = {
+  id: z.string().describe('The entry ULID to delete'),
+};
+
+export async function handler(
+  { id }: Record<string, any>,
+  ctx: LocalCtx,
+  { ensureIndexed }: SharedCtx
+): Promise<ToolResult> {
+  if (!id?.trim()) return err('Required: id (non-empty string)', 'INVALID_INPUT');
+  await ensureIndexed();
+
+  const entry = ctx.stmts.getEntryById.get(id);
+  if (!entry) return err(`Entry not found: ${id}`, 'NOT_FOUND');
+
+  try {
+    // Delete DB record first — if this fails, the file stays and no orphan is created
+    const rowidResult = ctx.stmts.getRowid.get(id);
+    if (rowidResult?.rowid) {
+      try {
+        ctx.deleteVec(Number(rowidResult.rowid));
+      } catch {}
+    }
+    ctx.stmts.deleteEntry.run(id);
+
+    // Delete file from disk after successful DB delete
+    let fileWarning = null;
+    if (entry.file_path) {
+      try {
+        unlinkSync(entry.file_path as string);
+      } catch (e: any) {
+        if (e.code !== 'ENOENT') {
+          fileWarning = `file could not be removed from disk (${e.code}): ${entry.file_path}`;
+        }
+      }
+    }
+
+    const msg = `Deleted ${entry.kind}: ${entry.title || '(untitled)'} [${id}]`;
+    return ok(fileWarning ? `${msg}\nWarning: ${fileWarning}` : msg);
+  } catch (e) {
+    return err(e instanceof Error ? e.message : String(e), 'DELETE_FAILED');
+  }
+}