pi-byterover 0.2.6 → 0.2.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-byterover",
-  "version": "0.2.6",
+  "version": "0.2.7",
   "private": false,
   "description": "Pi extension that recalls and persists ByteRover memory.",
   "keywords": [
@@ -32,8 +32,7 @@
     "provenance": true
   },
   "dependencies": {
-    "@byterover/brv-bridge": "^1.2.0",
-    "zod": "^4.4.3"
+    "@byterover/brv-bridge": "^1.2.0"
   },
   "devDependencies": {
     "byterover-cli": "3.16.1"

package/src/byterover-lifecycle.ts

@@ -343,22 +343,7 @@ export const createByteRoverExtension = (
 
     if (config.manualTools) {
       registerManualTools({
-        pi: {
-          registerTool: (tool) => {
-            switch (tool.name) {
-              case "brv_recall":
-                pi.registerTool(tool);
-                return;
-              case "brv_search":
-                pi.registerTool(tool);
-                return;
-              case "brv_persist":
-                pi.registerTool(tool);
-                return;
-            }
-          },
-        },
-        config,
+        pi,
         bridge,
         createBridge,
       });

package/src/config-loader.ts

@@ -1,10 +1,9 @@
 import { readFile } from "node:fs/promises";
 import { homedir } from "node:os";
 import { join } from "node:path";
-import type * as z from "zod/v4";
-import { ConfigSchema } from "./config.js";
+import { parseConfigDocument, type ByteroverConfig } from "./config.js";
 
-export type ByteroverConfig = z.infer<typeof ConfigSchema>;
+export type { ByteroverConfig };
 
 export type LoadConfigOptions = {
   cwd: string;
@@ -21,7 +20,7 @@ const invalidConfig = (source: string, error: Error): LoadConfigResult => ({
   error: new Error(`Invalid Byterover configuration in ${source}: ${error.message}`),
 });
 
-/** Loads the highest-precedence ByteRover JSON configuration through its Zod boundary. */
+/** Loads the highest-precedence ByteRover JSON configuration through its TypeBox boundary. */
 export const loadConfig = async ({
   cwd,
   homeDir = homedir(),
@@ -41,11 +40,11 @@ export const loadConfig = async ({
     }
 
     try {
-      return { success: true, source, config: ConfigSchema.parse(JSON.parse(raw)) };
+      return { success: true, source, config: parseConfigDocument(JSON.parse(raw)) };
     } catch (cause) {
       return invalidConfig(source, cause instanceof Error ? cause : new Error(String(cause)));
     }
   }
 
-  return { success: true, config: ConfigSchema.parse(undefined) };
+  return { success: true, config: parseConfigDocument(undefined) };
 };

package/src/config.ts

@@ -1,4 +1,5 @@
-import * as z from "zod/v4";
+import { Decode, type StaticDecode, Type } from "typebox";
+import { Value } from "typebox/value";
 
 export const brvGitignoreBeginMarker = "# BEGIN pi-byterover";
 export const brvGitignoreEndMarker = "# END pi-byterover";
@@ -46,29 +47,44 @@ export const configDefaults = {
   maxRecallChars: 4096,
 };
 
-const positiveInteger = () => z.number().int().positive();
-const nonEmptyString = () => z.string().trim().min(1);
+/** Raw, partially-specified Byterover configuration document. */
+export type ByteroverConfigDocument = StaticDecode<typeof ConfigSchema>;
 
-export const ConfigSchema = z
-  .object({
-    enabled: z.boolean().default(configDefaults.enabled),
+/** Fully defaulted Byterover configuration. */
+export type ByteroverConfig = ByteroverConfigDocument & typeof configDefaults;
+
+/** Parses one Byterover configuration document, rejecting invalid values, then applies defaults. */
+// oxlint-disable-next-line anti-slop/no-unknown-parameters -- This function IS the untrusted-document parser boundary.
+export const parseConfigDocument = (value: unknown): ByteroverConfig => ({
+  ...configDefaults,
+  ...Value.Decode(ConfigSchema, value === undefined ? {} : value),
+});
+
+const trimmedNonEmptyString = () =>
+  Decode(Type.String({ minLength: 1, pattern: "\\S" }), (value) => value.trim());
+
+export const ConfigSchema = Type.Object(
+  {
+    enabled: Type.Optional(Type.Boolean()),
     // BrvBridge options
-    brvPath: nonEmptyString().optional().default(configDefaults.brvPath),
-    searchTimeoutMs: positiveInteger().default(configDefaults.searchTimeoutMs),
-    recallTimeoutMs: positiveInteger().default(configDefaults.recallTimeoutMs),
-    persistTimeoutMs: positiveInteger().default(configDefaults.persistTimeoutMs),
+    brvPath: Type.Optional(trimmedNonEmptyString()),
+    searchTimeoutMs: Type.Optional(Type.Integer({ minimum: 1 })),
+    recallTimeoutMs: Type.Optional(Type.Integer({ minimum: 1 })),
+    persistTimeoutMs: Type.Optional(Type.Integer({ minimum: 1 })),
     // Plugin options
-    quiet: z.boolean().default(configDefaults.quiet),
-    autoRecall: z.boolean().default(configDefaults.autoRecall),
-    autoPersist: z.boolean().default(configDefaults.autoPersist),
-    manualTools: z.boolean().default(configDefaults.manualTools),
-    contextTagName: nonEmptyString()
-      .regex(/^[A-Za-z][A-Za-z0-9._-]*$/u)
-      .default(configDefaults.contextTagName),
-    recallPrompt: nonEmptyString().default(configDefaults.recallPrompt),
-    persistPrompt: nonEmptyString().default(configDefaults.persistPrompt),
-    maxRecallTurns: positiveInteger().default(configDefaults.maxRecallTurns),
-    maxRecallChars: positiveInteger().default(configDefaults.maxRecallChars),
-  })
-  .optional()
-  .default(configDefaults);
+    quiet: Type.Optional(Type.Boolean()),
+    autoRecall: Type.Optional(Type.Boolean()),
+    autoPersist: Type.Optional(Type.Boolean()),
+    manualTools: Type.Optional(Type.Boolean()),
+    contextTagName: Type.Optional(
+      Decode(Type.String({ minLength: 1, pattern: "^\\s*[A-Za-z][A-Za-z0-9._-]*\\s*$" }), (value) =>
+        value.trim(),
+      ),
+    ),
+    recallPrompt: Type.Optional(trimmedNonEmptyString()),
+    persistPrompt: Type.Optional(trimmedNonEmptyString()),
+    maxRecallTurns: Type.Optional(Type.Integer({ minimum: 1 })),
+    maxRecallChars: Type.Optional(Type.Integer({ minimum: 1 })),
+  },
+  { additionalProperties: false },
+);

package/src/gitignore.ts

@@ -7,7 +7,8 @@ import {
   brvGitignoreRules,
 } from "./config.js";
 
-const escapeRegExp = (value: string) => {
+/** Escape one string for verbatim interpolation into a regular expression. */
+export const escapeRegExp = (value: string) => {
   return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 };
 

package/src/recall.ts

@@ -1,6 +1,4 @@
-const escapeRegExp = (value: string) => {
-  return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-};
+import { escapeRegExp } from "./gitignore.js";
 
 export const stripEchoedRecallQuery = (content: string, query: string) => {
   const trimmedContent = content.trim();

package/src/tools.ts

@@ -6,15 +6,11 @@ import type {
 } from "@earendil-works/pi-coding-agent";
 import { type Static, type TSchema, Type } from "typebox";
 import type { ByteRoverBridge, ByteRoverBridgeFactory } from "./byterover-bridge.js";
-import type { ConfigSchema } from "./config.js";
 import { stripEchoedRecallQuery } from "./recall.js";
 
-type Config = ReturnType<typeof ConfigSchema.parse>;
-
 export type RegisterManualToolsInput = {
   pi: ByteRoverManualToolHost;
   bridge: ByteRoverBridge;
-  config: Config;
   createBridge: ByteRoverBridgeFactory;
 };
 
@@ -114,9 +110,21 @@ export type ByteRoverManualToolDefinition =
   | ManualToolDefinition<"brv_search", typeof SearchParameters>
   | ManualToolDefinition<"brv_persist", typeof PersistParameters>;
 
-/** Registers typed ByteRover manual tools with Pi or a faithful recording host. */
+/** Lists the three registered manual-memory tool names. */
+export const BYTE_ROVER_MANUAL_TOOL_NAMES = ["brv_recall", "brv_search", "brv_persist"] as const;
+
+/** Narrows any tool definition to one of the three ByteRover manual tools by name. */
+export const isByteRoverManualToolDefinition = (tool: {
+  readonly name: string;
+}): tool is ByteRoverManualToolDefinition =>
+  // SAFETY: widening the const tuple to readonly string[] only relaxes literal checking for includes().
+  (BYTE_ROVER_MANUAL_TOOL_NAMES as readonly string[]).includes(tool.name);
+
+/** Registers ByteRover manual tools with Pi or a faithful recording host. */
 export interface ByteRoverManualToolHost {
-  registerTool(tool: ByteRoverManualToolDefinition): void;
+  registerTool<TParams extends TSchema = TSchema, TDetails = unknown, TState = unknown>(
+    tool: ToolDefinition<TParams, TDetails, TState>,
+  ): void;
 }
 
 const textResult = (text: string): AgentToolResult<undefined> => ({
@@ -124,8 +132,6 @@ const textResult = (text: string): AgentToolResult<undefined> => ({
   details: undefined,
 });
 
-const errorMessage = (error: Error) => error.message;
-
 export const formatSearchResults = (
   results: readonly SearchResultItem[],
   totalFound: number,
@@ -158,11 +164,9 @@ export const formatSearchResults = (
 export const registerManualTools = ({
   pi,
   bridge,
-  config,
   createBridge,
-}: RegisterManualToolsInput) => {
-  if (!config.manualTools) return;
-
+}: Omit<RegisterManualToolsInput, "config">) => {
+  // ponytail: the session-start caller gates on config.manualTools before reaching this registration.
   pi.registerTool({
     name: "brv_recall",
     label: "ByteRover Recall",
@@ -185,7 +189,7 @@ export const registerManualTools = ({
         return textResult(content || "No relevant ByteRover context found.");
       } catch (cause) {
         const error = cause instanceof Error ? cause : new Error(String(cause));
-        return textResult(`ByteRover recall failed: ${errorMessage(error)}`);
+        return textResult(`ByteRover recall failed: ${error.message}`);
       }
     },
   });
@@ -214,7 +218,7 @@ export const registerManualTools = ({
         );
       } catch (cause) {
         const error = cause instanceof Error ? cause : new Error(String(cause));
-        return textResult(`ByteRover search failed: ${errorMessage(error)}`);
+        return textResult(`ByteRover search failed: ${error.message}`);
       }
     },
   });
@@ -243,7 +247,7 @@ export const registerManualTools = ({
         return textResult(`ByteRover persist ${brvResult.status}${suffix}`);
       } catch (cause) {
         const error = cause instanceof Error ? cause : new Error(String(cause));
-        return textResult(`ByteRover persist failed: ${errorMessage(error)}`);
+        return textResult(`ByteRover persist failed: ${error.message}`);
       }
     },
   });