@zosmaai/pi-llm-wiki 0.7.4 → 0.8.0

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 ## [Unreleased]
 
+### Fixed
+- **Personal wiki created at doubled path `~/.llm-wiki/.llm-wiki/…`**: `getPersonalWikiRoot()` returned the dot-dir itself (`~/.llm-wiki`) while `getVaultPaths()` then appended another `.llm-wiki/` segment, so the personal vault was written to `~/.llm-wiki/.llm-wiki/wiki/…`. Fixed by aligning `getPersonalWikiRoot()` with the same "root = parent of `.llm-wiki/`" contract used by project vaults. `WIKI_HOME` continues to override the parent.
+
+### Added
+- **`migrateDoubledPersonalVault()`** helper (`extensions/llm-wiki/lib/utils.ts`): Idempotent, in-place flatten of any vault that was already written to the broken doubled layout. Moves entries from `<root>/.llm-wiki/.llm-wiki/*` up to `<root>/.llm-wiki/*`, preserves outer entries on collision, removes the inner dir only when fully drained. Returns `null` when the layout is already correct, so it is safe to call on every session start.
+- **Auto-migration on `session_start`**: The extension now runs `migrateDoubledPersonalVault()` on the personal wiki at every session start. Existing broken vaults are flattened the next time the user opens or reloads pi — no manual step required. A one-line status message is shown when a flatten actually happens; otherwise the check is silent.
+- **`scripts/migrate-llm-wiki.js --fix-doubled`** flag: Manual recovery for arbitrary roots (`--fix-doubled ~/`, `--fix-doubled /some/project`). Supports `--dry-run` and `--force`.
+- **9 regression tests** (`test/personal-wiki-paths.test.ts`): pin `getPersonalWikiRoot()` to the parent-of-dotdir contract, exercise `WIKI_HOME`, and verify the migration helper across no-op, idempotent, and collision paths.
+
 ## [0.7.0] - 2026-05-13
 
 ### Added

package/extensions/llm-wiki/index.ts

@@ -25,6 +25,7 @@ import {
   ensureVaultStructure,
   fmtDate,
   getVaultPaths,
+  migrateDoubledPersonalVault,
   resolveVaultPaths,
   writeJson,
 } from "./lib/utils.js";
@@ -70,6 +71,22 @@ export default function (pi: ExtensionAPI) {
   let needsTopicInference = false;
 
   pi.on("session_start", async (_event, ctx) => {
+    // One-shot recovery for vaults created with the broken personal-root
+    // (~/.llm-wiki/.llm-wiki/… doubled layout). Runs on every session start
+    // because it is a cheap existence-check no-op when the layout is correct.
+    try {
+      const migration = migrateDoubledPersonalVault();
+      if (migration && migration.moved.length > 0) {
+        ctx.ui.setStatus(
+          "llm-wiki",
+          `🧠 Personal wiki layout fixed: flattened ${migration.moved.length} entries out of ${migration.from} (see CHANGELOG)`,
+        );
+      }
+    } catch (err) {
+      // Never let migration crash session start.
+      console.warn(`[llm-wiki] doubled-dotdir migration skipped: ${(err as Error).message}`);
+    }
+
     const paths = resolveVaultPaths(process.cwd());
     if (!existsSync(join(paths.dotWiki, "config.json"))) {
       // Silently create the wiki vault — no UI prompts

package/extensions/llm-wiki/lib/utils.ts

@@ -1,4 +1,13 @@
-import { existsSync, mkdirSync, readFileSync, readdirSync, statSync, writeFileSync } from "node:fs";
+import {
+  existsSync,
+  mkdirSync,
+  readFileSync,
+  readdirSync,
+  renameSync,
+  rmdirSync,
+  statSync,
+  writeFileSync,
+} from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join, resolve } from "node:path";
 import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
@@ -33,11 +42,24 @@ export function detectVaultFormat(dir: string): VaultFormat {
   return "none";
 }
 
-/** Get the personal wiki root directory (~/.llm-wiki/). */
+/**
+ * Get the personal wiki root directory.
+ *
+ * The "root" follows the same contract as project wikis: it is the directory
+ * that *contains* the `.llm-wiki/` dot-dir, NOT the dot-dir itself.
+ * So the personal vault lives at `<root>/.llm-wiki/`.
+ *
+ * Default root: `homedir()` → personal vault at `~/.llm-wiki/`.
+ * Override:     `WIKI_HOME` env var → personal vault at `$WIKI_HOME/.llm-wiki/`.
+ *
+ * NOTE: Previously this returned `~/.llm-wiki` (the dot-dir itself), which
+ * caused `getVaultPaths()` to compose paths like `~/.llm-wiki/.llm-wiki/raw`.
+ * See `migrateDoubledPersonalVault()` for the one-shot recovery.
+ */
 export function getPersonalWikiRoot(): string {
   const envWiki = process.env.WIKI_HOME;
   if (envWiki) return envWiki;
-  return join(homedir(), ".llm-wiki");
+  return homedir();
 }
 
 /** Get VaultPaths for the personal wiki. */
@@ -45,6 +67,54 @@ export function getPersonalWikiPaths(): VaultPaths {
   return getVaultPaths(getPersonalWikiRoot());
 }
 
+/**
+ * One-shot, idempotent migration for vaults that were created with the broken
+ * `getPersonalWikiRoot()` (returned the dot-dir itself, so `getVaultPaths()`
+ * composed `<root>/.llm-wiki/.llm-wiki/...`).
+ *
+ * Detects a doubled layout at `<root>/.llm-wiki/.llm-wiki/config.json` and
+ * flattens it up by one level. Safe to call on every session start: if the
+ * doubled sentinel is absent, this is a no-op.
+ *
+ * Returns a description of the action taken (or `null` if no migration was
+ * needed) so callers can surface a one-line status message.
+ */
+export function migrateDoubledPersonalVault(
+  parentRoot: string = getPersonalWikiRoot(),
+): { moved: string[]; from: string; to: string; skipped: string[] } | null {
+  const outerDotWiki = join(parentRoot, ".llm-wiki");
+  const innerDotWiki = join(outerDotWiki, ".llm-wiki");
+  const innerSentinel = join(innerDotWiki, "config.json");
+
+  if (!existsSync(innerSentinel)) return null;
+
+  const moved: string[] = [];
+  const skipped: string[] = [];
+
+  for (const entry of readdirSync(innerDotWiki)) {
+    const src = join(innerDotWiki, entry);
+    const dest = join(outerDotWiki, entry);
+    if (existsSync(dest)) {
+      // Collision — leave the inner copy in place rather than clobber.
+      skipped.push(entry);
+      continue;
+    }
+    renameSync(src, dest);
+    moved.push(entry);
+  }
+
+  // Only remove the inner dir if it is fully drained.
+  if (skipped.length === 0) {
+    try {
+      rmdirSync(innerDotWiki);
+    } catch {
+      // Leave behind if something raced us; harmless.
+    }
+  }
+
+  return { moved, from: innerDotWiki, to: outerDotWiki, skipped };
+}
+
 /**
  * Check if a vault is the personal wiki location.
  * Used in layered recall to avoid double-counting.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zosmaai/pi-llm-wiki",
-  "version": "0.7.4",
+  "version": "0.8.0",
   "description": "Self-maintaining LLM Wiki for Pi — Karpathy-pattern knowledge base with immutable source capture, automated ingestion, search, linting, and Obsidian-compatible vault. auto-updating personal & company wiki.",
   "keywords": [
     "pi",