pi-byterover 0.2.4 → 0.2.5

package/README.md

@@ -7,7 +7,7 @@
 <p>
   <a href="https://www.npmjs.com/package/pi-byterover"><img src="https://img.shields.io/npm/v/pi-byterover?style=for-the-badge&color=5B8DEF" alt="npm version"></a>
   <a href="LICENSE"><img src="https://img.shields.io/npm/l/pi-byterover?style=for-the-badge&color=34D399" alt="license"></a>
-  <a href="https://github.com/ian-pascoe/pi-byterover"><img src="https://img.shields.io/badge/github-pi--byterover-111827?style=for-the-badge&logo=github" alt="GitHub repository"></a>
+  <a href="https://github.com/ian-pascoe/pi-extensions/tree/main/packages/pi-byterover"><img src="https://img.shields.io/badge/github-pi--extensions-111827?style=for-the-badge&logo=github" alt="GitHub repository"></a>
 </p>
 
 </div>
@@ -16,9 +16,12 @@
 
 It automatically recalls relevant ByteRover memory before an agent response and injects that context into the session. After useful completed turns, it persists durable facts, decisions, preferences, and technical details back to ByteRover.
 
+Recalled memory is framed as **untrusted reference material**. The extension tells the agent not to treat recalled text as system, developer, user, or tool instructions.
+
 ## Prerequisites
 
 - Pi installed.
+- Node `>=22.19.0` and Pi `>=0.84.1`.
 - ByteRover CLI installed as `brv`, or a custom executable path configured with `brvPath`.
 - A project where ByteRover can bootstrap and use `.brv` state.
 
@@ -34,8 +37,8 @@ For local development:
 
 ```bash
 pnpm install
-pnpm build
-pi -e ./dist/index.js
+pnpm --filter pi-byterover test
+pi -e ./src/index.ts
 ```
 
 ## Configuration
@@ -103,7 +106,6 @@ With `autoRecall` and `autoPersist` enabled, routine memory behavior is automati
 pnpm install
 pnpm format:check
 pnpm lint
-pnpm test
-pnpm typecheck
-pnpm build
+pnpm --filter pi-byterover test
+pnpm --filter pi-byterover typecheck
 ```

package/package.json

@@ -1,37 +1,32 @@
 {
   "name": "pi-byterover",
-  "version": "0.2.4",
-  "description": "Pi ByteRover extension",
+  "version": "0.2.5",
+  "private": false,
+  "description": "Pi extension that recalls and persists ByteRover memory.",
   "keywords": [
     "byterover",
     "memory",
     "pi",
+    "pi-extension",
     "pi-package"
   ],
-  "homepage": "https://github.com/ian-pascoe/pi-byterover#readme",
+  "homepage": "https://github.com/ian-pascoe/pi-extensions#readme",
   "bugs": {
-    "url": "https://github.com/ian-pascoe/pi-byterover/issues"
+    "url": "https://github.com/ian-pascoe/pi-extensions/issues"
   },
   "license": "MIT",
-  "author": "Ian Pascoe",
+  "author": "Ian Pascoe <ian.g.pascoe@gmail.com>",
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/ian-pascoe/pi-byterover.git"
+    "url": "git+https://github.com/ian-pascoe/pi-extensions.git",
+    "directory": "packages/pi-byterover"
   },
   "files": [
-    "dist",
-    "README.md"
+    "src",
+    "README.md",
+    "LICENSE"
   ],
   "type": "module",
-  "main": "dist/index.js",
-  "types": "dist/index.d.ts",
-  "exports": {
-    ".": {
-      "types": "./dist/index.d.ts",
-      "import": "./dist/index.js",
-      "default": "./dist/index.js"
-    }
-  },
   "publishConfig": {
     "access": "public",
     "provenance": true
@@ -41,41 +36,23 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@changesets/changelog-github": "^0.7.0",
-    "@changesets/cli": "^2.31.0",
-    "@types/node": "^25.9.1",
-    "@typescript/native-preview": "7.0.0-dev.20260527.1",
-    "husky": "^9.1.7",
-    "oxfmt": "^0.52.0",
-    "oxlint": "^1.67.0",
-    "rolldown": "1.0.2",
-    "typescript": "^6.0.3",
-    "vitest": "^4.1.7"
+    "byterover-cli": "3.16.1"
   },
   "peerDependencies": {
     "@earendil-works/pi-ai": "*",
     "@earendil-works/pi-coding-agent": "*",
     "typebox": "*"
   },
+  "engines": {
+    "node": ">=22.19.0"
+  },
   "pi": {
     "extensions": [
-      "./dist/index.js"
+      "./src/index.ts"
     ]
   },
   "scripts": {
-    "build": "pnpm run clean && pnpm run build:js && pnpm run build:types",
-    "build:js": "rolldown -c",
-    "build:types": "tsgo --project tsconfig.build.json --emitDeclarationOnly",
-    "build:watch": "rolldown -c -w",
-    "clean": "rm -rf dist",
-    "changeset": "changeset",
-    "format": "oxfmt",
-    "format:check": "oxfmt --check",
-    "lint": "oxlint",
-    "lint:fix": "oxlint --fix",
-    "release": "pnpm run build && changeset publish",
-    "test": "vitest run",
-    "typecheck": "tsgo --noEmit",
-    "version-packages": "changeset version"
+    "test": "vitest run --config ../../vitest.config.ts --root .",
+    "typecheck": "tsc --noEmit -p tsconfig.json"
   }
 }

package/src/config-loader.ts

@@ -0,0 +1,58 @@
+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";
+
+export type ByteroverConfig = z.infer<typeof ConfigSchema>;
+
+export type LoadConfigOptions = {
+  cwd: string;
+  homeDir?: string;
+};
+
+export type LoadConfigResult =
+  | { success: true; config: ByteroverConfig; source?: string }
+  | { success: false; source: string; error: Error };
+
+const hasCode = (error: unknown, code: string) => {
+  return typeof error === "object" && error !== null && "code" in error && error.code === code;
+};
+
+const errorMessage = (error: unknown) => {
+  return error instanceof Error ? error.message : String(error);
+};
+
+const invalidConfig = (source: string, error: unknown): LoadConfigResult => ({
+  success: false,
+  source,
+  error: new Error(`Invalid Byterover configuration in ${source}: ${errorMessage(error)}`),
+});
+
+export const loadConfig = async ({
+  cwd,
+  homeDir = homedir(),
+}: LoadConfigOptions): Promise<LoadConfigResult> => {
+  const candidates = [
+    join(cwd, ".pi", "byterover.json"),
+    join(homeDir, ".pi", "agent", "byterover.json"),
+  ];
+
+  for (const source of candidates) {
+    let raw: string;
+    try {
+      raw = await readFile(source, "utf8");
+    } catch (error) {
+      if (hasCode(error, "ENOENT")) continue;
+      return invalidConfig(source, error);
+    }
+
+    try {
+      return { success: true, source, config: ConfigSchema.parse(JSON.parse(raw)) };
+    } catch (error) {
+      return invalidConfig(source, error);
+    }
+  }
+
+  return { success: true, config: ConfigSchema.parse(undefined) };
+};

package/src/config.ts

@@ -0,0 +1,76 @@
+import * as z from "zod/v4";
+
+export const brvGitignoreBeginMarker = "# BEGIN pi-byterover";
+export const brvGitignoreEndMarker = "# END pi-byterover";
+
+export const brvGitignoreRules = `# Dream state and logs
+dream-log/
+dream-state.json
+dream.lock
+
+# Review backups
+review-backups/
+
+# Generated files
+config.json
+_queue_status.json
+.snapshot.json
+_manifest.json
+_index.md
+*.abstract.md
+*.overview.md
+`;
+
+export const brvGitignore = `${brvGitignoreBeginMarker}\n${brvGitignoreRules}${brvGitignoreEndMarker}\n`;
+
+export const configDefaults = {
+  enabled: true,
+  brvPath: "brv",
+  searchTimeoutMs: 30_000,
+  recallTimeoutMs: 30_000,
+  persistTimeoutMs: 60_000,
+  quiet: false,
+  autoRecall: true,
+  autoPersist: true,
+  manualTools: true,
+  contextTagName: "byterover-context",
+  recallPrompt:
+    `Recall any relevant context that would help answer the latest user message.\n` +
+    `Use the recent conversation only to resolve references and intent.\n` +
+    `Do not restate the query in your findings.`,
+  persistPrompt:
+    `The following is a conversation between a user and an AI assistant.\n` +
+    `Curate only information with lasting value: facts, decisions, technical details, preferences, or notable outcomes.\n` +
+    `Skip trivial messages such as greetings, acknowledgments ("ok", "thanks", "sure", "got it"), one-word replies, anything with no substantive content.`,
+  maxRecallTurns: 3,
+  maxRecallChars: 4096,
+};
+
+export const maxCuratedTurnCacheSize = 500;
+
+const positiveInteger = () => z.number().int().positive();
+const nonEmptyString = () => z.string().trim().min(1);
+
+export const ConfigSchema = z
+  .object({
+    enabled: z.boolean().default(configDefaults.enabled),
+    // BrvBridge options
+    brvPath: nonEmptyString().optional().default(configDefaults.brvPath),
+    searchTimeoutMs: positiveInteger().default(configDefaults.searchTimeoutMs),
+    recallTimeoutMs: positiveInteger().default(configDefaults.recallTimeoutMs),
+    persistTimeoutMs: positiveInteger().default(configDefaults.persistTimeoutMs),
+    // 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);

package/src/gitignore.ts

@@ -0,0 +1,84 @@
+import { access, mkdir, readFile, writeFile } from "node:fs/promises";
+import { join } from "node:path";
+import {
+  brvGitignore,
+  brvGitignoreBeginMarker,
+  brvGitignoreEndMarker,
+  brvGitignoreRules,
+} from "./config.js";
+
+const hasCode = (error: unknown, code: string) => {
+  return typeof error === "object" && error !== null && "code" in error && error.code === code;
+};
+
+const escapeRegExp = (value: string) => {
+  return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+};
+
+const managedGitignoreRules = new Set(
+  brvGitignoreRules.split("\n").filter((line) => line.length > 0 && !line.startsWith("#")),
+);
+
+const managedGitignoreBlock = new RegExp(
+  `(?:^|\\r?\\n)${escapeRegExp(brvGitignoreBeginMarker)}[\\s\\S]*?${escapeRegExp(
+    brvGitignoreEndMarker,
+  )}\\r?\\n?`,
+  "gu",
+);
+
+export const normalizeBrvGitignore = (existing: string) => {
+  const output: Array<string> = [];
+  let insertedManagedBlock = false;
+  let skippingManagedBlock = false;
+
+  const insertManagedBlock = () => {
+    if (insertedManagedBlock) return;
+    if (output.length > 0 && output[output.length - 1] !== "") output.push("");
+    output.push(...brvGitignore.trimEnd().split("\n"));
+    insertedManagedBlock = true;
+  };
+
+  for (const line of existing
+    .replace(managedGitignoreBlock, (match) => {
+      const separator = match.startsWith("\r\n") ? "\r\n" : match.startsWith("\n") ? "\n" : "";
+      return `${separator}${brvGitignore}`;
+    })
+    .split(/\r?\n/)) {
+    if (line === brvGitignoreBeginMarker) {
+      insertManagedBlock();
+      skippingManagedBlock = true;
+      continue;
+    }
+    if (skippingManagedBlock) {
+      if (line === brvGitignoreEndMarker) skippingManagedBlock = false;
+      continue;
+    }
+    if (line === "# ByteRover generated files" || managedGitignoreRules.has(line)) {
+      insertManagedBlock();
+      continue;
+    }
+    output.push(line);
+  }
+
+  while (output.length > 0 && output[output.length - 1] === "") output.pop();
+  if (!insertedManagedBlock) insertManagedBlock();
+
+  return `${output.join("\n")}\n`;
+};
+
+export const ensureBrvGitignore = async (cwd: string) => {
+  await access(cwd);
+  await mkdir(join(cwd, ".brv"), { recursive: true });
+
+  const gitignorePath = join(cwd, ".brv", ".gitignore");
+
+  try {
+    const existing = await readFile(gitignorePath, "utf8");
+    const normalized = normalizeBrvGitignore(existing);
+    if (existing === normalized) return;
+    await writeFile(gitignorePath, normalized, "utf8");
+  } catch (error) {
+    if (!hasCode(error, "ENOENT")) throw error;
+    await writeFile(gitignorePath, brvGitignore, "utf8");
+  }
+};