pi-soly 2.0.0 → 2.1.0

package/config.ts

@@ -49,13 +49,6 @@ export interface SolyConfig {
 		 *    "off"   — no gate.
 		 *  Booleans accepted for back-compat: true = "scope", false = "off". */
 		confirmBeforeCode: boolean | "off" | "ask" | "scope";
-		/** 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. */
@@ -148,7 +141,6 @@ export const DEFAULT_CONFIG: SolyConfig = {
 		nudgeNotify: false,
 		toolHints: true,
 		confirmBeforeCode: "scope",
-		preActionRuleReminder: true,
 	},
 	display: {
 		defaultRecommendedFirst: true,
@@ -248,8 +240,6 @@ function deepMerge(base: SolyConfig, over: RawConfig): SolyConfig {
 			over.agent.confirmBeforeCode === "scope"
 		)
 			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/core.ts

@@ -571,68 +571,6 @@ ${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,9 +32,7 @@ import {
 	CONTEXT_WINDOW_TOKENS,
 	DEFAULT_PROGRESS,
 	extractFilePathsFromPrompt,
-	formatRuleReminder,
 	formatTok,
-	getApplicableRulesForFile,
 	loadAllRules,
 	loadPhaseRules,
 	loadProjectState,
@@ -901,29 +899,6 @@ 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": "2.0.0",
+  "version": "2.1.0",
   "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 the workflow inline via the soly_workflow tool — no external subagent plugin.",
   "type": "module",
   "main": "index.ts",