@mercury-fw/core 0.31.0 → 0.32.0

package/CHANGELOG.md

@@ -1,5 +1,32 @@
 # @mercury-fw/core
 
+## 0.32.0
+
+### Minor Changes
+
+- 1fa3a97: - Every tool plugin declared in `mercury.config.ts` loads: `MERCURY_CLIS` is no longer read. An app that used it to keep a declared plugin off now gets that plugin on; remove the plugin from the config instead.
+  - A new app's env example has no `MERCURY_CLIS`, and its config's comment says every declared plugin and channel is active.
+  - A plugin caught in a dependency cycle is always reported at startup.
+  - The plugins' READMEs no longer ask to list them in `MERCURY_CLIS`.
+
+### Patch Changes
+
+- @mercury-fw/plugin-types@0.32.0
+- @mercury-fw/channel-types@0.32.0
+- @mercury-fw/cli-engine@0.32.0
+- @mercury-fw/confirm-engine@0.32.0
+
+## 0.31.1
+
+### Patch Changes
+
+- 7926aa2: - Wiki grep ignores case, for the agent, the nightly review, `mfw vault grep` and the HTTP `/wiki/grep` route.
+  - The agent's `write_file` and the nightly review's `write_curated` take a path starting with `curated/`, as listing, reading and grepping give it, instead of writing under `curated/curated/`.
+  - @mercury-fw/plugin-types@0.31.1
+  - @mercury-fw/channel-types@0.31.1
+  - @mercury-fw/cli-engine@0.31.1
+  - @mercury-fw/confirm-engine@0.31.1
+
 ## 0.31.0
 
 ### Patch Changes

package/README.md

@@ -27,7 +27,6 @@ A scaffolded app's `src/index.ts` (the service) and `src/repl.ts` (the REPL) are
 | `OLLAMA_THINK` | `false` for a model that doesn't support thinking. |
 | `QDRANT_URL` | Qdrant, for the episodic memory (default `http://qdrant:6333`). |
 | `WIKI_VAULT_PATH` | Where the wiki lives. Required. |
-| `MERCURY_CLIS` | The tool plugins this deployment turns on, by name, comma-separated. |
 
 Qdrant being unreachable degrades memory, it doesn't stop the agent.
 

package/dist/src/config/define-config.d.ts

@@ -15,8 +15,8 @@ import type { Plugin } from "@mercury-fw/plugin-types";
 import type { ChannelPlugin } from "@mercury-fw/channel-types";
 import type { Persona } from "../session/system-prompt.ts";
 /** The shape of a Mercury instance's composition config: the tool plugins
- * (gated by MERCURY_CLIS) and the channel plugins (enabled by being declared
- * here — declared = active), each loaded by its own loader, plus the
+ * and the channel plugins (each enabled by being declared here: declared =
+ * active), each loaded by its own loader, plus the
  * assistant's persona (its identity and tone; the defaults when left out). */
 export type MercuryConfig = {
     plugins: Plugin[];

package/dist/src/plugins/plugin-loader.d.ts

@@ -6,9 +6,8 @@
  * nothing about any specific plugin — or about CLIs — the composition root
  * names them, this loop processes them identically.
  *
- * Two properties it guarantees, both required by the plan's fail-soft step:
- *  - a plugin contributes only when it is enabled on this instance (its name is
- *    in MERCURY_CLIS);
+ * Two properties it guarantees:
+ *  - every plugin the app declares is loaded: declaring it is what enables it;
  *  - a plugin that fails — its `build()` throws — degrades as a single unit:
  *    none of its contributions land, the failure is
  *    logged with detail, and every other plugin and the process itself carry
@@ -50,10 +49,9 @@ export interface LoadedPlugins {
      * config map to infer activation from anymore. */
     activated: string[];
 }
-/** Everything the loader needs from the composition root: which plugins are
- * enabled on this instance, and the runtime context every `build()` gets. */
+/** Everything the loader needs from the composition root: the runtime
+ * context every `build()` gets. */
 export interface PluginLoadContext {
-    enabledClis: string[];
     model: LanguageModel;
     env: Record<string, string | undefined>;
     log: (msg: string) => void;
@@ -79,7 +77,7 @@ export declare function orderByDependencies(plugins: Plugin[]): {
  * dependency between them. Never throws — a plugin that fails is logged and
  * skipped, its contributions staged and merged only once the whole plugin
  * succeeds so a later failure can't leave it half-wired. A plugin whose
- * declared `dependsOn` isn't fully activated (a dependency disabled, failed,
- * unknown, or itself skipped) is skipped fail-soft too, transitively.
+ * declared `dependsOn` isn't fully activated (a dependency failed, unknown,
+ * or itself skipped) is skipped fail-soft too, transitively.
  */
 export declare function loadPlugins(plugins: Plugin[], ctx: PluginLoadContext): Promise<LoadedPlugins>;

package/dist/src/wiki/wiki-note.d.ts

@@ -1,3 +1,6 @@
+/** `path` relative to curated/: a vault-relative one (`curated/x.md`, as
+ * listing, reading and grepping give it) loses its leading `curated/`. */
+export declare function relativeToCurated(path: string): string;
 /** Writes a curated doc at `curated/<relativePath>` (e.g. "standards/jira-fields.md"). */
 export declare function writeCuratedNote(vaultPath: string, relativePath: string, fields: {
     author?: string;

package/dist/src/wiki/wiki-read.d.ts

@@ -19,7 +19,8 @@ export type WikiGrepMatch = {
     line: number;
     text: string;
 };
-/** Searches every file under `roots` for `pattern` (a regular expression), line by line. */
+/** Searches every file under `roots` for `pattern` (a regular expression,
+ * case-insensitive: a note's wording isn't the question's), line by line. */
 export declare function grepWikiInRoots(vaultPath: string, roots: string[], pattern: string): Promise<WikiGrepMatch[]>;
 /** Searches every file visible to `userId` for `pattern` (a regular expression), line by line. */
 export declare function grepWiki(vaultPath: string, userId: string, pattern: string): Promise<WikiGrepMatch[]>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mercury-fw/core",
-  "version": "0.31.0",
+  "version": "0.32.0",
   "type": "module",
   "license": "MIT",
   "repository": {
@@ -31,10 +31,10 @@
     "test": "bun test"
   },
   "dependencies": {
-    "@mercury-fw/channel-types": "0.31.0",
-    "@mercury-fw/cli-engine": "0.31.0",
-    "@mercury-fw/confirm-engine": "0.31.0",
-    "@mercury-fw/plugin-types": "0.31.0",
+    "@mercury-fw/channel-types": "0.32.0",
+    "@mercury-fw/cli-engine": "0.32.0",
+    "@mercury-fw/confirm-engine": "0.32.0",
+    "@mercury-fw/plugin-types": "0.32.0",
     "@qdrant/js-client-rest": "^1.19.0",
     "ai": "^7.0.126",
     "ai-sdk-ollama": "^4.4.0",

package/src/admin/wiki-routes.ts

@@ -31,7 +31,8 @@ export async function readWikiVaultFile(vaultPath: string, relativePath: string)
 }
 
 export async function grepWikiVault(vaultPath: string, pattern: string): Promise<WikiGrepMatch[]> {
-  const regex = new RegExp(pattern);
+  // Case-insensitive, like every other wiki grep.
+  const regex = new RegExp(pattern, "i");
   const files = await listWikiVault(vaultPath);
   const matches: WikiGrepMatch[] = [];
   for (const file of files) {

package/src/compose.ts

@@ -123,11 +123,6 @@ function requireEnv(name: string): string {
  * app) reads its own `mercury.config.ts` and passes it in. See {@link ComposedApp}.
  */
 export async function composeMercury(config: MercuryConfig): Promise<ComposedApp> {
-  const enabledClis = (process.env.MERCURY_CLIS ?? "")
-    .split(",")
-    .map((s) => s.trim())
-    .filter(Boolean);
-
   // The model is constructed up front, before the plugins load and the system
   // prompt is built: a plugin's post-turn guard can be model-backed (Jira's
   // issue-list corrector is), and the system prompt is assembled from the
@@ -149,13 +144,11 @@ export async function composeMercury(config: MercuryConfig): Promise<ComposedApp
   // handed in — this file no longer names them. The loader processes an opaque
   // list (see plugins/plugin-loader.ts): each supplies its allowlist as data, a
   // system-prompt fragment, and a `build()` that turns the runtime context into
-  // post-processors and post-turn guards. A plugin contributes only when it's
-  // both listed in MERCURY_CLIS and its allowlist validates; one that fails
-  // degrades only itself.
+  // post-processors and post-turn guards. Every declared plugin loads; one
+  // that fails (an invalid allowlist, missing configuration) degrades only itself.
   const plugins = config.plugins;
 
   const loadedPlugins = await loadPlugins(plugins, {
-    enabledClis,
     model,
     env: process.env,
     log: (msg) => console.error(msg),

package/src/config/define-config.ts

@@ -16,8 +16,8 @@ import type { ChannelPlugin } from "@mercury-fw/channel-types";
 import type { Persona } from "../session/system-prompt.ts";
 
 /** The shape of a Mercury instance's composition config: the tool plugins
- * (gated by MERCURY_CLIS) and the channel plugins (enabled by being declared
- * here — declared = active), each loaded by its own loader, plus the
+ * and the channel plugins (each enabled by being declared here: declared =
+ * active), each loaded by its own loader, plus the
  * assistant's persona (its identity and tone; the defaults when left out). */
 export type MercuryConfig = {
   plugins: Plugin[];

package/src/plugins/plugin-loader.ts

@@ -6,9 +6,8 @@
  * nothing about any specific plugin — or about CLIs — the composition root
  * names them, this loop processes them identically.
  *
- * Two properties it guarantees, both required by the plan's fail-soft step:
- *  - a plugin contributes only when it is enabled on this instance (its name is
- *    in MERCURY_CLIS);
+ * Two properties it guarantees:
+ *  - every plugin the app declares is loaded: declaring it is what enables it;
  *  - a plugin that fails — its `build()` throws — degrades as a single unit:
  *    none of its contributions land, the failure is
  *    logged with detail, and every other plugin and the process itself carry
@@ -54,10 +53,9 @@ export interface LoadedPlugins {
   activated: string[];
 }
 
-/** Everything the loader needs from the composition root: which plugins are
- * enabled on this instance, and the runtime context every `build()` gets. */
+/** Everything the loader needs from the composition root: the runtime
+ * context every `build()` gets. */
 export interface PluginLoadContext {
-  enabledClis: string[];
   model: LanguageModel;
   env: Record<string, string | undefined>;
   log: (msg: string) => void;
@@ -116,8 +114,8 @@ export function orderByDependencies(plugins: Plugin[]): { ordered: Plugin[]; cyc
  * dependency between them. Never throws — a plugin that fails is logged and
  * skipped, its contributions staged and merged only once the whole plugin
  * succeeds so a later failure can't leave it half-wired. A plugin whose
- * declared `dependsOn` isn't fully activated (a dependency disabled, failed,
- * unknown, or itself skipped) is skipped fail-soft too, transitively.
+ * declared `dependsOn` isn't fully activated (a dependency failed, unknown,
+ * or itself skipped) is skipped fail-soft too, transitively.
  */
 export async function loadPlugins(plugins: Plugin[], ctx: PluginLoadContext): Promise<LoadedPlugins> {
   const promptFragments: string[] = [];
@@ -127,13 +125,9 @@ export async function loadPlugins(plugins: Plugin[], ctx: PluginLoadContext): Pr
   const postTurnGuards: PostTurnGuard[] = [];
 
   const { ordered, cyclic } = orderByDependencies(plugins);
-  // A plugin caught in a cycle can't be ordered, so it can't load. Report it
-  // only when it's enabled — a disabled plugin contributes nothing regardless,
-  // same silence as any other disabled plugin.
+  // A plugin caught in a cycle can't be ordered, so it can't load.
   for (const plugin of cyclic) {
-    if (ctx.enabledClis.includes(plugin.name)) {
-      ctx.log(`plugin "${plugin.name}" not activated: part of or depends on a dependency cycle`);
-    }
+    ctx.log(`plugin "${plugin.name}" not activated: part of or depends on a dependency cycle`);
   }
 
   // Names that fully activated — a dependency must be in here for a dependent
@@ -142,11 +136,6 @@ export async function loadPlugins(plugins: Plugin[], ctx: PluginLoadContext): Pr
   const activated = new Set<string>();
 
   for (const plugin of ordered) {
-    // Not enabled on this instance: contribute nothing, and don't even
-    // validate the config — same as a CLI left out of MERCURY_CLIS.
-    if (!ctx.enabledClis.includes(plugin.name)) {
-      continue;
-    }
     // Contract-version skew: core and plugin are versioned and installed
     // separately, so a plugin built against a different contract than this core
     // supports is possible. Refuse it fail-soft rather than run it against a
@@ -160,8 +149,8 @@ export async function loadPlugins(plugins: Plugin[], ctx: PluginLoadContext): Pr
     }
     // Every declared dependency must have fully activated first. Because we
     // process in dependency-first order, a dependency that was going to load
-    // already has; anything still missing is disabled, failed, unknown, or
-    // itself skipped — so this dependent degrades fail-soft too. Checked before
+    // already has; anything still missing failed, is unknown, or was itself
+    // skipped — so this dependent degrades fail-soft too. Checked before
     // touching this plugin's own build, so a doomed plugin does no work.
     const missingDeps = (plugin.dependsOn ?? []).filter((dep) => !activated.has(dep));
     if (missingDeps.length > 0) {

package/src/wiki/self-review-tools.ts

@@ -19,7 +19,7 @@ import type { ExecutableTool } from "@mercury-fw/plugin-types";
 import { tool } from "ai";
 import { z } from "zod";
 import { listWikiFilesInRoots, readWikiFileInRoots, grepWikiInRoots, selfReviewRoots, readIndexFile } from "./wiki-read.ts";
-import { writeCuratedNote, writeIndexFile, deleteRawEntry, deleteCuratedEntry } from "./wiki-note.ts";
+import { writeCuratedNote, writeIndexFile, deleteRawEntry, deleteCuratedEntry, relativeToCurated } from "./wiki-note.ts";
 import { normalizeIndexKey, upsertIndexEntry, removeIndexEntry } from "./index-entry.ts";
 
 export type SelfReviewToolsDeps = { vaultPath: string };
@@ -76,11 +76,12 @@ export function createSelfReviewTools(
   });
 
   const write_curated = tool({
-    description: 'Create or overwrite a curated doc. "path" is relative to curated/, e.g. "standards/jira-fields.md".',
+    description:
+      'Create or overwrite a curated doc. "path" is relative to curated/, e.g. "standards/jira-fields.md"; the path list_files and grep give ("curated/standards/jira-fields.md") works too.',
     inputSchema: z.object({ path: z.string().min(1), content: z.string() }),
     execute: async ({ path, content }) => {
       try {
-        await writeCuratedNote(vaultPath, path, {}, content);
+        await writeCuratedNote(vaultPath, relativeToCurated(path), {}, content);
         return { ok: true as const };
       } catch (err) {
         return { ok: false as const, error: String(err) };

package/src/wiki/vault-cli.ts

@@ -127,7 +127,7 @@ async function main(): Promise<void> {
     case "grep": {
       const pattern = args[0];
       if (!pattern) usage();
-      const regex = new RegExp(pattern);
+      const regex = new RegExp(pattern, "i");
       const glob = new Bun.Glob("**/*.md");
       for await (const file of glob.scan({ cwd: vaultPath })) {
         const content = await Bun.file(`${vaultPath}/${file}`).text();

package/src/wiki/wiki-note.ts

@@ -201,6 +201,12 @@ async function deleteVaultFile(vaultPath: string, fullPath: string, commitMessag
   });
 }
 
+/** `path` relative to curated/: a vault-relative one (`curated/x.md`, as
+ * listing, reading and grepping give it) loses its leading `curated/`. */
+export function relativeToCurated(path: string): string {
+  return path.replace(/^curated\//, "");
+}
+
 /** Writes a curated doc at `curated/<relativePath>` (e.g. "standards/jira-fields.md"). */
 export async function writeCuratedNote(
   vaultPath: string,

package/src/wiki/wiki-read.ts

@@ -88,9 +88,10 @@ export async function readWikiFile(vaultPath: string, userId: string, relativePa
 
 export type WikiGrepMatch = { path: string; line: number; text: string };
 
-/** Searches every file under `roots` for `pattern` (a regular expression), line by line. */
+/** Searches every file under `roots` for `pattern` (a regular expression,
+ * case-insensitive: a note's wording isn't the question's), line by line. */
 export async function grepWikiInRoots(vaultPath: string, roots: string[], pattern: string): Promise<WikiGrepMatch[]> {
-  const regex = new RegExp(pattern);
+  const regex = new RegExp(pattern, "i");
   const files = await listWikiFilesInRoots(vaultPath, roots);
   const matches: WikiGrepMatch[] = [];
 

package/src/wiki/wiki-tools.ts

@@ -15,7 +15,7 @@ import type { ExecutableTool } from "@mercury-fw/plugin-types";
 import { tool } from "ai";
 import { z } from "zod";
 import { listWikiFiles, readWikiFile, grepWiki, readWikiFileInRoots } from "./wiki-read.ts";
-import { writeCuratedNote } from "./wiki-note.ts";
+import { writeCuratedNote, relativeToCurated } from "./wiki-note.ts";
 
 export type WikiToolsDeps = { vaultPath: string; userId: string };
 
@@ -55,12 +55,13 @@ export function createWikiTools(
   const write_file = tool({
     description:
       'Write or update a curated wiki document (team knowledge — conventions, standards, decisions). "path" ' +
-      'is relative to curated/, e.g. "standards/jira-fields.md". This can only write under curated/ — your ' +
+      'is relative to curated/, e.g. "standards/jira-fields.md"; the path grep and read_file give ' +
+      '("curated/standards/jira-fields.md") works too. This can only write under curated/ — your ' +
       "own semantic notes are managed automatically by the memory consolidation process, not through this tool.",
     inputSchema: z.object({ path: z.string().min(1), content: z.string() }),
     execute: async ({ path, content }) => {
       try {
-        await writeCuratedNote(vaultPath, path, { last_updated: new Date().toISOString().slice(0, 10) }, content);
+        await writeCuratedNote(vaultPath, relativeToCurated(path), { last_updated: new Date().toISOString().slice(0, 10) }, content);
         return { ok: true as const };
       } catch (err) {
         return { ok: false as const, error: String(err) };
@@ -70,7 +71,7 @@ export function createWikiTools(
 
   const grep = tool({
     description:
-      "Search wiki documents (curated/ plus your own inferred/ notes) for a regular expression pattern. " +
+      "Search wiki documents (curated/ plus your own inferred/ notes) for a regular expression pattern, ignoring case. " +
       "Returns matching lines with their file path and line number.",
     inputSchema: z.object({ pattern: z.string().min(1) }),
     execute: async ({ pattern }) => {