decorated-pi 0.7.0 → 0.7.2

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "decorated-pi",
-    "version": "0.7.0",
+    "version": "0.7.2",
     "description": "decorated-pi is a practical enhancement pack for pi coding agent — token-efficient workflow, cache-friendly design, and smarter tools.",
     "keywords": [
         "pi",
@@ -12,7 +12,9 @@
         "lsp",
         "lsp-client",
         "secret-redaction",
-        "codegraph"
+        "codegraph",
+        "fff",
+        "fuzzy-search"
     ],
     "license": "MIT",
     "repository": {
@@ -22,6 +24,7 @@
     "homepage": "https://github.com/lcwecker/decorated-pi#readme",
     "bugs": "https://github.com/lcwecker/decorated-pi/issues",
     "dependencies": {
+        "@ff-labs/fff-node": "^0.9.4",
         "@modelcontextprotocol/sdk": "^1.29.0",
         "file-type": "^21.3.4",
         "openai": "^6.37.0"
@@ -39,13 +42,15 @@
     },
     "devDependencies": {
         "@types/node": "^22.15.3",
+        "@vitest/coverage-v8": "4.1.8",
         "depcheck": "^1.4.7",
         "publint": "^0.3.21",
-        "vitest": "^4.1.6"
+        "vitest": "4.1.8"
     },
     "scripts": {
         "prepack": "depcheck --fail-on-missing",
-        "test": "vitest run",
+        "test": "vitest run --coverage",
+        "bench": "vitest bench --run",
         "typecheck": "tsc --noEmit",
         "prepublishOnly": "npm install --no-audit --no-fund && npm test && npm run typecheck && depcheck --fail-on-missing && publint"
     }

package/settings.ts

@@ -82,9 +82,11 @@ export function loadConfig(): DecoratedPiConfig {
   try {
     if (fs.existsSync(CONFIG_FILE)) {
       const config = JSON.parse(fs.readFileSync(CONFIG_FILE, "utf-8")) as DecoratedPiConfig;
-      if (migrateModuleSettings(config)) {
-        saveConfig(config);
-      }
+      // Mutate in place. Do NOT call saveConfig here — saveConfig calls
+      // loadConfig first, which would re-trigger the migration (and the
+      // file on disk hasn't been written yet), causing deep recursion.
+      // The migrated shape is persisted on the next explicit saveConfig.
+      migrateModuleSettings(config);
       return config;
     }
   } catch {}

package/skills/pi-docs/SKILL.md

@@ -0,0 +1,13 @@
+---
+name: pi-docs
+description: pi docs resources
+---
+
+Pi documentation (read only when the user asks about pi itself, its SDK, extensions, themes, skills, or TUI):
+- Main documentation: /home/liuchang/.local/share/fnm/node-versions/v24.14.0/installation/lib/node_modules/@earendil-works/pi-coding-agent/README.md
+- Additional docs: /home/liuchang/.local/share/fnm/node-versions/v24.14.0/installation/lib/node_modules/@earendil-works/pi-coding-agent/docs
+- Examples: /home/liuchang/.local/share/fnm/node-versions/v24.14.0/installation/lib/node_modules/@earendil-works/pi-coding-agent/examples (extensions, custom tools, SDK)
+- When reading pi docs or examples, resolve docs/... under Additional docs and examples/... under Examples, not the current working directory
+- When asked about: extensions (docs/extensions.md, examples/extensions/), themes (docs/themes.md), skills (docs/skills.md), prompt templates (docs/prompt-templates.md), TUI components (docs/tui.md), keybindings (docs/keybindings.md), SDK integrations (docs/sdk.md), custom providers (docs/custom-provider.md), adding models (docs/models.md), pi packages (docs/packages.md)
+- When working on pi topics, read the docs and examples, and follow .md cross-references before implementing
+- Always read pi .md files completely and follow links to related docs (e.g., tui.md for TUI API details)

package/tools/lsp/tools.ts

@@ -69,7 +69,6 @@ function severityLabel(s: number): string {
 // ─── Register tools ───────────────────────────────────────────────────────
 
 export function registerLspTools(pi: ExtensionAPI, manager: LspServerManager) {
-
   // ── lsp_diagnostics ────────────────────────────────────────────────────
   pi.registerTool({
     name: "lsp_diagnostics",

package/tools/mcp/builtin/codegraph.ts

@@ -14,7 +14,7 @@ export const CODEGRAPH_BUILTIN: Omit<McpServerConfig, "source"> = {
     name: "codegraph",
     command: "codegraph",
     args: ["serve", "--mcp"],
-    enabled: false,
+    enabled: true,
     description:
         "Local code knowledge graph (colbymchenry/codegraph). Enable via /mcp.",
     canUseInProject: (cwd: string) => fs.existsSync(path.join(cwd, ".codegraph")),

package/tools/mcp/client.ts

@@ -16,7 +16,7 @@ export class McpConnection {
   client: Client;
   transport: StreamableHTTPClientTransport | SSEClientTransport | StdioClientTransport | undefined;
   tools: McpToolSpec[] = [];
-  private connected = false;
+  connected = false;
 
   source: string = "unknown";
 

package/tools/patch/core.ts

@@ -1076,12 +1076,18 @@ interface LineRange {
   endLine: number;
 }
 
-/** Build line offset table: offsets[i] = character offset of line i+1 (1-based) */
+/** Build line offset table: offsets[i] = character offset of line i+1 (1-based).
+ *  If the content does not end with a newline, the final line has no
+ *  trailing marker; push an extra offset at content.length so callers
+ *  like lineAtOffset / extractLineRange handle the last line correctly. */
 function buildLineOffsets(content: string): number[] {
   const offsets = [0];
   for (let i = 0; i < content.length; i++) {
     if (content[i] === "\n") offsets.push(i + 1);
   }
+  if (content.length > 0 && content[content.length - 1] !== "\n") {
+    offsets.push(content.length);
+  }
   return offsets;
 }
 
@@ -1515,3 +1521,13 @@ function truncate(s: string, maxLen = 60): string {
   if (firstLine.length <= maxLen) return firstLine;
   return firstLine.slice(0, maxLen - 3) + "...";
 }
+
+// Test exports
+export const __patchCoreTest = {
+  charOffsetToLine,
+  detectTabWidth,
+  normalizeIndentForFuzzy,
+  truncate,
+  collapseSequentialReplacements,
+  generateReplacementDiff,
+};