pi-soly 1.4.1 → 1.5.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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-soly",
-  "version": "1.4.1",
+  "version": "1.5.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}` }],