@mrclrchtr/supi-claude-md 1.1.2 → 1.2.0

package/README.md

@@ -1,73 +1,42 @@
 # @mrclrchtr/supi-claude-md
 
-Automatic `CLAUDE.md` / `AGENTS.md` context management for the [pi coding agent](https://github.com/earendil-works/pi).
+Subdirectory context for PI — your project's conventions follow the agent wherever it goes.
 
-## Install
-
-```bash
-pi install npm:@mrclrchtr/supi-claude-md
-```
+Pi loads your root `CLAUDE.md` by default. Claude-MD extends that downward: when the agent reaches into `src/auth/`, it picks up `src/auth/CLAUDE.md` too. Conventions are where the code is, not just at the project root.
 
-## What it adds
+Then it helps you keep those files in shape — audit quality, flag stale sections, and capture session learnings with your approval.
 
-This extension adds **subdirectory context discovery**: it injects `CLAUDE.md` / `AGENTS.md` from subdirectories when the agent touches files there.
-
-Pi loads root and ancestor instruction files natively into the system prompt on every turn. This package only handles subdirectories below `cwd`. To pick up root instruction file edits mid-session, use pi's `/reload` command or restart the session.
-
-If your install surface includes the shared SuPi settings command (for example via `@mrclrchtr/supi`), this package contributes a Claude-MD section there:
-
-```text
-/supi-settings
-```
+## What you get
 
-Inside `/supi-settings`, Claude-MD contributes:
+### Context that travels
 
-- `Subdirectory Discovery`: on/off toggle
-- `Subdirectory Re-read Interval`: text input; enter a number of turns or `0` to disable subdirectory re-reads
-- `Context Threshold`: common percentage values from `0` to `100`
-- `Context File Names`: comma-separated text input; empty input restores the default filenames
+Reads, writes, edits, LSP operations — any time the agent touches a file, it picks up the nearest `CLAUDE.md` or `AGENTS.md` in that directory. Each subdirectory's context is injected once (on first discovery) and available for the rest of the session.
 
-This package bundles two skills:
+### CLAUDE.md maintenance
 
-- `claude-md-improver`: audit CLAUDE.md files, evaluate quality, and propose targeted updates. SuPi-aware: compares CLAUDE.md sections against a synthesized context baseline from `supi-code-intelligence` and `supi-claude-md`, then flags redundant content and suggests compression when only part of a section should stay
-- `claude-md-revision`: capture session learnings into CLAUDE.md with user approval
+Two bundled skills:
 
-## Configuration
+- **claude-md-improver** — audit every CLAUDE.md in your repo. Flags redundancy, stale sections, and content already covered by SuPi's auto-injected context. Suggests targeted updates.
+- **claude-md-revision** — capture what you learned this session into CLAUDE.md. Ask the agent to remember a pattern, convention, or gotcha — it proposes the edit, you approve.
 
-Configuration uses the shared SuPi config system.
+## Install
 
-Config file locations:
+```bash
+pi install npm:@mrclrchtr/supi-claude-md
+```
 
-- global: `~/.pi/agent/supi/config.json`
-- project: `.pi/supi/config.json`
+## Settings
 
-Use the `claude-md` section:
+Configure via `/supi-settings` or directly in config:
 
 ```json
 {
   "claude-md": {
-    "rereadInterval": 3,
-    "contextThreshold": 80,
     "subdirs": true,
     "fileNames": ["CLAUDE.md", "AGENTS.md"]
   }
 }
 ```
 
-Options:
-
-- `rereadInterval`: turns between re-reading previously injected subdirectory context; `0` disables subdirectory re-reads (first-time discovery is unaffected)
-- `contextThreshold`: skip subdirectory re-injection when context usage is at or above this percent; `100` disables context gating; first-time discovery is always allowed
-- `subdirs`: enable or disable subdirectory discovery
-- `fileNames`: ordered list of context filenames to search for
-
-## Requirements
-
-- `@earendil-works/pi-coding-agent`
-- `@earendil-works/pi-tui`
-- `@mrclrchtr/supi-core`
-
-## Source
-
-- Entrypoint: `src/claude-md.ts`
-- Skills: `skills/claude-md-improver/`, `skills/claude-md-revision/`
+- `subdirs` — toggle subdirectory discovery on/off
+- `fileNames` — which filenames to look for (comma-separated)

package/node_modules/@mrclrchtr/supi-core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-core",
-  "version": "1.1.2",
+  "version": "1.2.0",
   "description": "SuPi core — shared infrastructure for SuPi extensions (XML context tags, config system)",
   "license": "MIT",
   "repository": {
@@ -22,5 +22,13 @@
     "@earendil-works/pi-coding-agent": "*",
     "@earendil-works/pi-tui": "*"
   },
+  "peerDependenciesMeta": {
+    "@earendil-works/pi-coding-agent": {
+      "optional": true
+    },
+    "@earendil-works/pi-tui": {
+      "optional": true
+    }
+  },
   "main": "src/index.ts"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-claude-md",
-  "version": "1.1.2",
+  "version": "1.2.0",
   "description": "SuPi claude-md extension — automatic subdirectory context injection for pi",
   "license": "MIT",
   "repository": {
@@ -21,7 +21,7 @@
     "!__tests__"
   ],
   "dependencies": {
-    "@mrclrchtr/supi-core": "1.1.2"
+    "@mrclrchtr/supi-core": "1.2.0"
   },
   "bundledDependencies": [
     "@mrclrchtr/supi-core"
@@ -30,6 +30,14 @@
     "@earendil-works/pi-coding-agent": "*",
     "@earendil-works/pi-tui": "*"
   },
+  "peerDependenciesMeta": {
+    "@earendil-works/pi-coding-agent": {
+      "optional": true
+    },
+    "@earendil-works/pi-tui": {
+      "optional": true
+    }
+  },
   "pi": {
     "extensions": [
       "./src/claude-md.ts"

package/src/claude-md.ts

@@ -13,17 +13,17 @@ import type {
   ExtensionContext,
   SessionCompactEvent,
   SessionStartEvent,
-  TurnEndEvent,
 } from "@earendil-works/pi-coding-agent";
 import { loadClaudeMdConfig } from "./config.ts";
+import type { DiscoveredContextFile } from "./discovery.ts";
 import {
   extractPathFromToolEvent,
   filterAlreadyLoaded,
   findSubdirContextFiles,
 } from "./discovery.ts";
 import { registerClaudeMdSettings } from "./settings-registration.ts";
-import { type ClaudeMdState, createInitialState, reconstructState } from "./state.ts";
-import type { ContextUsage, InjectionCheckOptions } from "./subdirectory.ts";
+import type { ClaudeMdState } from "./state.ts";
+import { createInitialState, reconstructState } from "./state.ts";
 import { formatSubdirContext, shouldInjectSubdir } from "./subdirectory.ts";
 
 const baseDir = dirname(dirname(fileURLToPath(import.meta.url)));
@@ -39,10 +39,8 @@ export default function claudeMdExtension(pi: ExtensionAPI) {
 
     try {
       const branch = ctx.sessionManager.getBranch();
-
       if (branch.length > 0) {
         const reconstructed = reconstructState(branch);
-        state.completedTurns = reconstructed.completedTurns;
         state.injectedDirs = reconstructed.injectedDirs;
       }
     } catch {
@@ -50,15 +48,6 @@ export default function claudeMdExtension(pi: ExtensionAPI) {
     }
   });
 
-  // ── Turn tracking ──────────────────────────────────────────
-
-  pi.on("turn_end", async (event: TurnEndEvent, _ctx: ExtensionContext) => {
-    const msg = event.message as { stopReason?: string };
-    if (msg?.stopReason === "stop") {
-      state.completedTurns++;
-    }
-  });
-
   // ── Compaction ─────────────────────────────────────────────
 
   pi.on("session_compact", async (_event: SessionCompactEvent, _ctx: ExtensionContext) => {
@@ -96,17 +85,11 @@ export default function claudeMdExtension(pi: ExtensionAPI) {
     );
     if (found.length === 0) return;
 
-    const dirsToInject = collectStaleDirs(found, {
-      injectedDirs: state.injectedDirs,
-      currentTurn: state.completedTurns,
-      rereadInterval: config.rereadInterval,
-      contextThreshold: config.contextThreshold,
-      contextUsage: _ctx.getContextUsage() as ContextUsage | undefined,
-    });
+    const dirsToInject = collectFreshDirs(found, state.injectedDirs);
     if (dirsToInject.size === 0) return;
 
     const filesToInject = Array.from(dirsToInject.values()).flat();
-    const contextText = formatSubdirContext(filesToInject, state.completedTurns);
+    const contextText = formatSubdirContext(filesToInject);
     if (!contextText) return;
 
     updateInjectedDirTracking(state, dirsToInject);
@@ -135,13 +118,13 @@ function captureNativePaths(
   }
 }
 
-function collectStaleDirs(
-  found: ReturnType<typeof findSubdirContextFiles>,
-  injectionOpts: InjectionCheckOptions,
-): Map<string, typeof found> {
-  const dirsToInject = new Map<string, typeof found>();
+function collectFreshDirs(
+  found: DiscoveredContextFile[],
+  injectedDirs: Set<string>,
+): Map<string, DiscoveredContextFile[]> {
+  const dirsToInject = new Map<string, DiscoveredContextFile[]>();
   for (const file of found) {
-    if (shouldInjectSubdir(file.dir, injectionOpts)) {
+    if (shouldInjectSubdir(file.dir, injectedDirs)) {
       const existing = dirsToInject.get(file.dir) ?? [];
       existing.push(file);
       dirsToInject.set(file.dir, existing);
@@ -152,12 +135,9 @@ function collectStaleDirs(
 
 function updateInjectedDirTracking(
   state: ClaudeMdState,
-  dirsToInject: Map<string, Array<{ dir: string; relativePath: string }>>,
+  dirsToInject: Map<string, DiscoveredContextFile[]>,
 ): void {
-  for (const [dir, files] of dirsToInject) {
-    const firstFile = files[0];
-    if (firstFile) {
-      state.injectedDirs.set(dir, { turn: state.completedTurns, file: firstFile.relativePath });
-    }
+  for (const dir of dirsToInject.keys()) {
+    state.injectedDirs.add(dir);
   }
 }

package/src/config.ts

@@ -2,8 +2,6 @@
 //
 // Config shape (in supi shared config, "claude-md" section):
 // {
-//   "rereadInterval": 3,      // turns between subdirectory re-reads (0 = off)
-//   "contextThreshold": 80,    // skip injection when context % >= threshold
 //   "subdirs": true,           // enable subdirectory context discovery
 //   "fileNames": ["CLAUDE.md", "AGENTS.md"]  // context file names to look for
 // }
@@ -11,10 +9,6 @@
 import { loadSupiConfig } from "@mrclrchtr/supi-core";
 
 export interface ClaudeMdConfig {
-  /** Turns between re-reading previously injected subdirectory context. 0 = disabled. Default: 3 */
-  rereadInterval: number;
-  /** Skip injection when context window usage % >= threshold. 0 = always skip, 100 = never skip. Default: 80 */
-  contextThreshold: number;
   /** Enable subdirectory context discovery. Default: true */
   subdirs: boolean;
   /** Context file names to look for (first match per directory). Default: ["CLAUDE.md", "AGENTS.md"] */
@@ -22,8 +16,6 @@ export interface ClaudeMdConfig {
 }
 
 export const CLAUDE_MD_DEFAULTS: ClaudeMdConfig = {
-  rereadInterval: 3,
-  contextThreshold: 80,
   subdirs: true,
   fileNames: ["CLAUDE.md", "AGENTS.md"],
 };

package/src/settings-registration.ts

@@ -8,30 +8,6 @@ import {
 } from "@mrclrchtr/supi-core";
 import { CLAUDE_MD_DEFAULTS, type ClaudeMdConfig } from "./config.ts";
 
-const THRESHOLD_VALUES = [
-  "0",
-  "5",
-  "10",
-  "15",
-  "20",
-  "25",
-  "30",
-  "35",
-  "40",
-  "45",
-  "50",
-  "55",
-  "60",
-  "65",
-  "70",
-  "75",
-  "80",
-  "85",
-  "90",
-  "95",
-  "100",
-];
-
 // ── Settings registration ────────────────────────────────────
 
 export function registerClaudeMdSettings(): void {
@@ -58,16 +34,6 @@ function handleSettingChange(
       helpers.set("subdirs", value === "on");
       break;
     }
-    case "rereadInterval": {
-      const num = Number.parseInt(value, 10);
-      helpers.set("rereadInterval", Number.isNaN(num) ? 0 : num);
-      break;
-    }
-    case "contextThreshold": {
-      const num = Number.parseInt(value, 10);
-      helpers.set("contextThreshold", Number.isNaN(num) ? 80 : num);
-      break;
-    }
     case "fileNames": {
       const names = value
         .split(",")
@@ -92,21 +58,6 @@ function buildClaudeMdSettingItems(settings: ClaudeMdConfig): SettingItem[] {
       currentValue: settings.subdirs ? "on" : "off",
       values: ["on", "off"],
     },
-    {
-      id: "rereadInterval",
-      label: "Subdirectory Re-read Interval",
-      description: "Turns between re-reading previously injected subdirectory context (0 = off)",
-      currentValue: String(settings.rereadInterval),
-      submenu: (currentValue, done) =>
-        createInputSubmenu(currentValue, "Interval (0 = off):", done),
-    },
-    {
-      id: "contextThreshold",
-      label: "Context Threshold",
-      description: "Skip injection when context window usage % ≥ threshold (100 = never skip)",
-      currentValue: String(settings.contextThreshold),
-      values: THRESHOLD_VALUES,
-    },
     {
       id: "fileNames",
       label: "Context File Names",

package/src/state.ts

@@ -5,18 +5,9 @@
 
 import type { SessionEntry } from "@earendil-works/pi-coding-agent";
 
-export interface InjectedDir {
-  /** Turn number when this directory's context was last injected */
-  turn: number;
-  /** Relative path of the context file that was injected */
-  file: string;
-}
-
 export interface ClaudeMdState {
-  /** Count of completed assistant turns (stopReason: "stop") */
-  completedTurns: number;
-  /** Map of directory path → injection info */
-  injectedDirs: Map<string, InjectedDir>;
+  /** Set of directory paths whose context has already been injected */
+  injectedDirs: Set<string>;
   /** Set of paths already loaded by pi natively (dedup) */
   nativeContextPaths: Set<string>;
   /** Whether this is the first before_agent_start (for native path capture) */
@@ -25,41 +16,27 @@ export interface ClaudeMdState {
 
 export function createInitialState(): ClaudeMdState {
   return {
-    completedTurns: 0,
-    injectedDirs: new Map(),
+    injectedDirs: new Set(),
     nativeContextPaths: new Set(),
     firstAgentStart: true,
   };
 }
 
-const CONTEXT_TAG_REGEX =
-  /<extension-context\s+source="supi-claude-md"\s+file="([^"]+)"\s+turn="(\d+)">/g;
+const CONTEXT_TAG_REGEX = /<extension-context\s+source="supi-claude-md"\s+file="([^"]+)"[^>]*>/g;
 
 export function reconstructState(branch: SessionEntry[]): {
-  completedTurns: number;
-  injectedDirs: Map<string, InjectedDir>;
+  injectedDirs: Set<string>;
 } {
-  let completedTurns = 0;
-  const injectedDirs = new Map<string, InjectedDir>();
+  const injectedDirs = new Set<string>();
 
   for (const entry of branch) {
-    if (isCompletedAssistantTurn(entry)) completedTurns++;
-
     const toolResultContent = getToolResultContent(entry);
     if (toolResultContent) {
       extractInjectedDirs(toolResultContent, injectedDirs);
     }
   }
 
-  return { completedTurns, injectedDirs };
-}
-
-function isCompletedAssistantTurn(entry: SessionEntry): boolean {
-  return (
-    entry.type === "message" &&
-    entry.message.role === "assistant" &&
-    entry.message.stopReason === "stop"
-  );
+  return { injectedDirs };
 }
 
 function getToolResultContent(entry: SessionEntry): unknown {
@@ -69,7 +46,7 @@ function getToolResultContent(entry: SessionEntry): unknown {
   return entry.message.content;
 }
 
-function extractInjectedDirs(content: unknown, injectedDirs: Map<string, InjectedDir>): void {
+function extractInjectedDirs(content: unknown, injectedDirs: Set<string>): void {
   const parts = content as Array<{ type?: string; text?: string }> | undefined;
   if (!parts) return;
 
@@ -80,15 +57,14 @@ function extractInjectedDirs(content: unknown, injectedDirs: Map<string, Injecte
   }
 }
 
-function parseContextTags(text: string, injectedDirs: Map<string, InjectedDir>): void {
+function parseContextTags(text: string, injectedDirs: Set<string>): void {
   const matches = text.matchAll(CONTEXT_TAG_REGEX);
   for (const match of matches) {
     const file = match[1];
-    const turn = Number.parseInt(match[2] ?? "0", 10);
     if (file) {
       const lastSlash = Math.max(file.lastIndexOf("/"), file.lastIndexOf("\\"));
       const dir = lastSlash >= 0 ? file.substring(0, lastSlash) : ".";
-      injectedDirs.set(dir, { turn, file });
+      injectedDirs.add(dir);
     }
   }
 }

package/src/subdirectory.ts

@@ -1,27 +1,17 @@
 // Subdirectory context injection logic.
 //
 // Handles formatting discovered context files into <extension-context> blocks
-// and determining whether injection should occur based on staleness.
+// and determining whether injection should occur.
 
 import * as fs from "node:fs";
 import { wrapExtensionContext } from "@mrclrchtr/supi-core";
 import type { DiscoveredContextFile } from "./discovery.ts";
-import type { InjectedDir } from "./state.ts";
-
-/**
- * Context usage info from pi's ctx.getContextUsage().
- */
-export interface ContextUsage {
-  tokens: number | null;
-  contextWindow: number;
-  percent: number | null;
-}
 
 /**
  * Format discovered context files into <extension-context> blocks.
  * Each file is read and wrapped individually.
  */
-export function formatSubdirContext(files: DiscoveredContextFile[], turn: number): string {
+export function formatSubdirContext(files: DiscoveredContextFile[]): string {
   const parts: string[] = [];
 
   for (const file of files) {
@@ -31,7 +21,6 @@ export function formatSubdirContext(files: DiscoveredContextFile[], turn: number
         parts.push(
           wrapExtensionContext("supi-claude-md", content, {
             file: file.relativePath,
-            turn,
           }),
         );
       }
@@ -43,42 +32,10 @@ export function formatSubdirContext(files: DiscoveredContextFile[], turn: number
   return parts.join("\n\n");
 }
 
-export interface InjectionCheckOptions {
-  injectedDirs: Map<string, InjectedDir>;
-  currentTurn: number;
-  rereadInterval: number;
-  contextThreshold: number;
-  contextUsage?: ContextUsage;
-}
-
 /**
  * Determine if subdirectory context should be injected.
- * Returns true if:
- * - The directory has not been injected yet (always, even under context pressure)
- * - The directory was injected but is stale (turn delta >= rereadInterval)
- *   AND context usage is below the threshold
- * - rereadInterval is 0 (disabled — always false for re-injections)
+ * Returns true only if the directory has not been injected yet.
  */
-export function shouldInjectSubdir(dir: string, options: InjectionCheckOptions): boolean {
-  const { injectedDirs, currentTurn, rereadInterval, contextThreshold, contextUsage } = options;
-
-  // Never-injected directory: always inject (even when rereadInterval is 0)
-  // First-time discovery is always allowed regardless of context pressure
-  const injected = injectedDirs.get(dir);
-  if (!injected) return true;
-
-  // Already-injected directory: skip if reread is disabled
-  if (rereadInterval === 0) return false;
-
-  // Re-injection: skip when context usage is at or above threshold
-  if (
-    contextThreshold < 100 &&
-    contextUsage &&
-    contextUsage.percent != null &&
-    contextUsage.percent >= contextThreshold
-  ) {
-    return false;
-  }
-
-  return currentTurn - injected.turn >= rereadInterval;
+export function shouldInjectSubdir(dir: string, injectedDirs: Set<string>): boolean {
+  return !injectedDirs.has(dir);
 }