pi-soly 1.12.2 → 1.13.1

package/config.ts

@@ -45,6 +45,13 @@ export interface SolyConfig {
 		 *  user (ask "ready to implement, or discuss first?") before writing code.
 		 *  Default on. */
 		confirmBeforeCode: boolean;
+		/** Inject a compact list of rules applicable to the file being
+		 *  edited/written, appended to the tool's result. Forces the LLM to
+		 *  confirm in the next message which rules were applied — closes the
+		 *  "rules read at start of turn, forgotten by edit time" gap.
+		 *  Default on. Disable for verbose rule sets that would otherwise
+		 *  flood the per-edit context. */
+		preActionRuleReminder: boolean;
 	};
 	display: {
 		/** Always show the recommended (⭐) option as the first row. */
@@ -125,6 +132,7 @@ export const DEFAULT_CONFIG: SolyConfig = {
 		nudgeNotify: false,
 		toolHints: true,
 		confirmBeforeCode: true,
+		preActionRuleReminder: true,
 	},
 	display: {
 		defaultRecommendedFirst: true,
@@ -216,6 +224,8 @@ function deepMerge(base: SolyConfig, over: RawConfig): SolyConfig {
 			merged.agent.toolHints = over.agent.toolHints;
 		if (typeof over.agent.confirmBeforeCode === "boolean")
 			merged.agent.confirmBeforeCode = over.agent.confirmBeforeCode;
+		if (typeof over.agent.preActionRuleReminder === "boolean")
+			merged.agent.preActionRuleReminder = over.agent.preActionRuleReminder;
 	}
 	if (over.display) {
 		if (typeof over.display.defaultRecommendedFirst === "boolean")

package/context-manager.ts

@@ -0,0 +1,54 @@
+// =============================================================================
+// context-manager.ts — single owner of pi's `context` channel
+// =============================================================================
+//
+// pi fires a `context` event before each LLM call; a handler may return
+// `{ messages }` to replace what the model sees for that one call. Only ONE
+// owner should do this, or rewriters fight. soly takes ownership here and
+// exposes a single pluggable rewriter slot — workflows (e.g. `/verify`'s
+// fresh-context mode) install/remove a rewriter through this manager instead
+// of each registering their own `context` handler.
+//
+// The rewriter must return a subset/reordering of the SAME message objects it
+// is given (never fabricate messages — their full shape is owned by pi). The
+// handler is fail-safe: a throwing rewriter leaves the turn untouched.
+// =============================================================================
+
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+
+/** Minimal message shape the manager needs (pi's AgentMessage is a superset). */
+export type AgentMessageLike = { role: string };
+
+/** Transforms the message list for the next LLM call. Returns real messages. */
+export type ContextRewriter = (messages: AgentMessageLike[]) => AgentMessageLike[];
+
+export type ContextManager = {
+	/** Install a rewriter, or pass null to restore pass-through. */
+	setRewriter(rewriter: ContextRewriter | null): void;
+	/** Whether a rewriter is currently active. */
+	hasRewriter(): boolean;
+};
+
+/** Create the manager and register the single `context` handler. */
+export function createContextManager(pi: ExtensionAPI): ContextManager {
+	let rewriter: ContextRewriter | null = null;
+
+	pi.on("context", (event) => {
+		if (!rewriter) return;
+		try {
+			const messages = rewriter(event.messages) as typeof event.messages;
+			return { messages };
+		} catch {
+			return; // never break a turn over a rewriter bug
+		}
+	});
+
+	return {
+		setRewriter(next): void {
+			rewriter = next;
+		},
+		hasRewriter(): boolean {
+			return rewriter !== null;
+		},
+	};
+}

package/core.ts

@@ -571,6 +571,68 @@ ${blocks.join("\n\n---\n\n")}${skippedNote}
   return { section, loaded: applicable.map(ruleKey), interactive };
 }
 
+// ============================================================================
+// Per-file rule lookup (used by pre-action reminder in tool_result hook)
+// ============================================================================
+//
+// Returns the subset of `rules` that apply to a single file path. Mirrors the
+// glob logic in `buildRulesSection` (always/no-globs = universal; otherwise
+// match `filePath` against `meta.globs`). Excludes disabled rules. Use this
+// to feed the pre-action reminder — LLM should see applicable rules AT THE
+// MOMENT of edit, not just at the start of the turn (where they decay).
+export function getApplicableRulesForFile(
+  filePath: string,
+  rules: RuleFile[],
+): RuleFile[] {
+  return rules.filter((r) => {
+    if (!r.enabled) return false;
+    if (r.meta.always === true) return true;
+    const globs = r.meta.globs;
+    if (!globs || globs.length === 0) return true;
+    return globs.some((g) => matchesGlob(filePath, g));
+  });
+}
+
+// Formats a compact reminder block for the LLM to see right after editing
+// `filePath`. Sorted by priority (high → medium → low → undefined), capped
+// at `cap` rules (default 3) so the LLM isn't flooded. Returns "" when no
+// rules apply — caller should treat that as "don't inject anything".
+export function formatRuleReminder(
+  rules: RuleFile[],
+  filePath: string,
+  cap = 3,
+): string {
+  if (rules.length === 0) return "";
+
+  const priorityOrder: Record<string, number> = { high: 0, medium: 1, low: 2 };
+  const sorted = [...rules].sort((a, b) => {
+    const pa = priorityOrder[a.meta.priority ?? ""] ?? 1;
+    const pb = priorityOrder[b.meta.priority ?? ""] ?? 1;
+    if (pa !== pb) return pa - pb;
+    return a.relPath.localeCompare(b.relPath);
+  });
+
+  const shown = sorted.slice(0, cap);
+  const remaining = rules.length - shown.length;
+
+  const lines: string[] = [];
+  lines.push(`\n\n---\n\n📋 **Applicable rules for \`${filePath}\`:**\n`);
+  shown.forEach((r, i) => {
+    const desc = r.meta.description ? ` — ${r.meta.description}` : "";
+    lines.push(`${i + 1}. [\`${r.relPath}\`]${desc}`);
+  });
+  if (remaining > 0) {
+    lines.push(
+      `\n_…and ${remaining} more — use \`/why ${filePath}\` to see all._`,
+    );
+  }
+  lines.push(
+    `\nIn your next message, briefly confirm which were applied (or note why not).`,
+  );
+
+  return lines.join("\n");
+}
+
 // ============================================================================
 // Phase-scoped rules loader
 // ============================================================================

package/index.ts

@@ -32,7 +32,9 @@ import {
 	CONTEXT_WINDOW_TOKENS,
 	DEFAULT_PROGRESS,
 	extractFilePathsFromPrompt,
+	formatRuleReminder,
 	formatTok,
+	getApplicableRulesForFile,
 	loadAllRules,
 	loadPhaseRules,
 	loadProjectState,
@@ -834,6 +836,29 @@ export default function solyExtension(pi: ExtensionAPI) {
 		}
 	});
 
+	// ============================================================================
+	// tool_result: inject a compact reminder of rules applicable to the file
+	// just edited/written, so the LLM sees them AT THE MOMENT of action (not
+	// 30 turns later when they've decayed in attention). The LLM is asked to
+	// confirm in its next message which rules were applied — closing the gap
+	// where rules loaded at the start of a turn are forgotten by the time the
+	// agent actually edits something.
+	// ============================================================================
+	pi.on("tool_result", async (event, _ctx) => {
+		if (!getActiveConfig().agent.preActionRuleReminder) return;
+		if (event.toolName !== "edit" && event.toolName !== "write") return;
+		if (event.isError) return; // tool failed — don't pile on
+		const input = event.input as { path?: string };
+		const filePath = input?.path;
+		if (!filePath) return;
+		const applicable = getApplicableRulesForFile(filePath, combinedRules());
+		const reminder = formatRuleReminder(applicable, filePath);
+		if (!reminder) return;
+		return {
+			content: [...event.content, { type: "text" as const, text: reminder }],
+		};
+	});
+
 	// ============================================================================
 	// Chrome: working-indicator telemetry lifecycle + reactive data refresh
 	// ============================================================================

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-soly",
-  "version": "1.12.2",
+  "version": "1.13.1",
   "description": "Project management + workflow framework for pi-coding-agent. Plans, state, MANDATORY rules, self-review, multi-question picker, native footer + welcome chrome — one npm install, zero config. The LLM drives execution and delegates to a worker subagent when available.",
   "type": "module",
   "main": "index.ts",
@@ -30,6 +30,7 @@
     "README.md",
     "commands.ts",
     "config.ts",
+    "context-manager.ts",
     "core.ts",
     "docs.ts",
     "env.ts",