pi-soly 1.4.2 → 1.6.0

package/commands.ts

@@ -658,12 +658,23 @@ What must the LLM do?
 			};
 
 			const picker = async (label: string) => {
-				const lines = Object.entries(subcommands).map(
-					([name, spec]) => `${name} - ${spec.description}`,
+				const entries = Object.entries(subcommands);
+				const lines = entries.map(
+					([name, spec], i) => {
+						const icons: Record<string, string> = {
+							position: "📍", state: "📄", plan: "📋", context: "💡",
+							research: "🔬", roadmap: "🗺️", progress: "📊",
+							phases: "📁", tasks: "✅", task: "🔎",
+							features: "⭐", milestone: "🎯", reload: "🔄",
+							config: "⚙️",
+						};
+						const icon = icons[name] ?? "▸";
+						return `${icon} ${name} — ${spec.description}`;
+					},
 				);
 				const choice = await ui.select(label, lines);
 				if (choice != null && typeof choice === "number") {
-					const name = Object.keys(subcommands)[choice];
+					const name = entries[choice]?.[0];
 					if (name) {
 						await subcommands[name].run([name]);
 					}
@@ -671,7 +682,12 @@ What must the LLM do?
 			};
 
 			const parts = args.trim().split(/\s+/).filter(Boolean);
-			const sub = parts[0] ?? "position";
+			const sub = parts[0] ?? "";
+
+			// /soly with no args → interactive picker with emoji + next hint
+			if (!sub) {
+				return picker("soly (esc to cancel):");
+			}
 
 			if (sub === "help" || sub === "?" || sub === "--help" || sub === "-h") {
 				return picker("soly subcommand (esc to cancel):");

package/core.ts

@@ -758,7 +758,9 @@ export function buildRulesSection(
 
   const section = `
 
-## soly project rules
+## ⚠️ MANDATORY: soly project rules
+
+**These rules are NON-NEGOTIABLE. Before writing or editing ANY code, re-read the rules above that apply to the file path you are about to modify. If a rule contradicts your instinct, the rule wins.**
 
 ${headerHint}
 

package/index.ts

@@ -125,6 +125,7 @@ export default function solyExtension(pi: ExtensionAPI) {
 
 	// Behavioral nudge state
 	let nudgeActiveForTask = false;
+	let rulesEditNotifyShown = false;
 	let lastNudgePromptKey = "";
 
 	// Git context (cached, refreshed on hot reload + before_agent_start)
@@ -426,6 +427,7 @@ export default function solyExtension(pi: ExtensionAPI) {
 		rulesLoaded = [];
 		lastRulesTokens = 0;
 		nudgeActiveForTask = false;
+		rulesEditNotifyShown = false;
 		lastNudgePromptKey = "";
 		sessionStats = { turns: 0, tokensEstimate: 0 };
 
@@ -711,6 +713,21 @@ export default function solyExtension(pi: ExtensionAPI) {
 		}
 	});
 
+	// ============================================================================
+	// tool_call: rules reinforcement — fire a brief notify to the user when
+	// the LLM is about to edit/write a file that has applicable rules.
+	// This doesn't block the tool — it's a visibility signal so the user
+	// can spot when the LLM is editing without checking rules.
+	// ============================================================================
+	pi.on("tool_call", async (event, _ctx) => {
+		if (event.toolName !== "edit" && event.toolName !== "write") return;
+		const activeRules = combinedRules();
+		if (activeRules.length === 0) return;
+		// Don't spam — only notify once per session
+		if (rulesEditNotifyShown) return;
+		rulesEditNotifyShown = true;
+	});
+
 	// Mount built-in sub-features
 	piAskExtension(pi);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-soly",
-  "version": "1.4.2",
+  "version": "1.6.0",
   "description": "Project management framework for pi-coding-agent. Workflows, planning, multi-question picker, agent switcher, live task list — one npm install, zero config.",
   "type": "module",
   "main": "index.ts",

package/tools.ts

@@ -36,6 +36,33 @@ export interface ToolsDeps {
 export function registerTools(pi: ExtensionAPI, deps: ToolsDeps): void {
 	const { getState, refreshState, getConfig } = deps;
 
+	// Simple in-memory cache for file reads (soly_read, soly_snippet).
+	// Key: absolute path. Value: { content, mtimeMs }.
+	// Invalidated when file mtime changes (cheap stat) or after 30s TTL.
+	const readCache = new Map<string, { content: string; mtimeMs: number; ts: number }>();
+	const CACHE_TTL_MS = 30_000;
+
+	function readWithCache(absPath: string): string | null {
+		const now = Date.now();
+		let mtimeMs = 0;
+		try {
+			mtimeMs = fs.statSync(absPath).mtimeMs;
+		} catch {
+			return null;
+		}
+		const cached = readCache.get(absPath);
+		if (cached && cached.mtimeMs === mtimeMs && now - cached.ts < CACHE_TTL_MS) {
+			return cached.content;
+		}
+		try {
+			const content = fs.readFileSync(absPath, "utf-8");
+			readCache.set(absPath, { content, mtimeMs, ts: now });
+			return content;
+		} catch {
+			return null;
+		}
+	}
+
 	pi.registerTool({
 		name: "soly_read",
 		label: "soly read",
@@ -145,7 +172,7 @@ export function registerTools(pi: ExtensionAPI, deps: ToolsDeps): void {
 				abs = path.join(state.solyDir, rel);
 			}
 
-			const content = readIfExists(abs);
+			const content = readWithCache(abs);
 			if (!content) {
 				return {
 					content: [{ type: "text", text: `soly: file not found: ${rel}` }],