pi-soly 2.2.6 → 2.3.1

package/README.md

@@ -1,3 +1,5 @@
+![banner](./packages/pi-soly/.assets/banner.png)
+
 <div align="center">
 
 # ⚡ pi-soly
@@ -11,8 +13,6 @@
 
 </div>
 
-![banner](./packages/pi-soly/.assets/banner.png)
-
 ## What it is
 
 pi-soly turns a plain pi-coding-agent session into a structured project:

package/commands/_helpers.ts

@@ -40,6 +40,12 @@ export interface CommandsDeps {
 	getConfig: () => SolyConfig;
 	reloadConfig: () => void;
 	getIntentDocs: () => IntentDoc[];
+	/** Record a non-error soly event. Rendered as a sub-line under the
+	 *  Working indicator (└─ prefixed, dim/yellow by level). Use this for
+	 *  informational events (e.g. "reloaded 47 rules") — they shouldn't be
+	 *  popups. Errors must still go through `ui.notify(text, "error")` for
+	 *  the popup surface. */
+	recordEvent: (text: string, level?: "info" | "warning") => void;
 }
 
 /** Open a focused list modal (overlay) for the given grouped items. */

package/commands/artifacts.ts

@@ -6,16 +6,16 @@ import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-c
 import { getArtifactServer, ensureArtifactServer, artifactDir } from "../artifact/session.ts";
 import { openListPanel, openExternally, type CommandsDeps } from "./_helpers.ts";
 
-type ArtifactsDeps = Pick<CommandsDeps, "getConfig">;
+type ArtifactsDeps = Pick<CommandsDeps, "getConfig" | "recordEvent">;
 
 export function registerArtifactsCommand(pi: ExtensionAPI, deps: ArtifactsDeps): void {
-	const { getConfig } = deps;
+	const { getConfig, recordEvent } = deps;
 
 	pi.registerCommand("artifacts", {
 		description: "browse this project's html_artifact gallery (list, open, clear)",
 		handler: async (args, ctx) => {
 			if (!getConfig().artifacts.server) {
-				ctx.ui.notify("soly: artifact server is disabled (artifacts.server=false)", "info");
+				recordEvent("artifact server is disabled (artifacts.server=false)");
 				return;
 			}
 			// Always re-resolve: reuse another window's server or start ours, and
@@ -27,7 +27,7 @@ export function registerArtifactsCommand(pi: ExtensionAPI, deps: ArtifactsDeps):
 				// ignore — handled by the count check below
 			}
 			if (!server || server.count === 0) {
-				ctx.ui.notify("soly: no artifacts for this project yet (use the html_artifact tool)", "info");
+				recordEvent("no artifacts for this project yet (use the html_artifact tool)");
 				return;
 			}
 			const gallery = server.galleryUrl();
@@ -35,15 +35,15 @@ export function registerArtifactsCommand(pi: ExtensionAPI, deps: ArtifactsDeps):
 
 			if (sub === "clear") {
 				const n = server.clear();
-				ctx.ui.notify(`soly: cleared ${n} artifact(s)`, "info");
+				recordEvent(`cleared ${n} artifact(s)`);
 				return;
 			}
 			if (sub === "open" || sub === "gallery") {
 				try {
 					await openExternally(pi, gallery);
-					ctx.ui.notify(`soly: opened ${gallery}`, "info");
+					recordEvent(`opened ${gallery}`);
 				} catch {
-					ctx.ui.notify(`soly artifacts gallery: ${gallery}`, "info");
+					recordEvent(`soly artifacts gallery: ${gallery}`);
 				}
 				return;
 			}
@@ -80,7 +80,7 @@ export function registerArtifactsCommand(pi: ExtensionAPI, deps: ArtifactsDeps):
 
 			// Non-TUI: print the list + gallery URL.
 			const lines = server.list().map((a) => `🖼 ${a.title} — ${a.url}`);
-			ctx.ui.notify([`soly artifacts gallery: ${gallery}`, "", ...lines].join("\n"), "info");
+			recordEvent([`soly artifacts gallery: ${gallery}`, "", ...lines].join("\n"));
 		},
 	});
 }

package/commands/docs.ts

@@ -7,10 +7,10 @@ import { formatTok, solyDirFor } from "../core.ts";
 import { buildIntentStats, formatIntentStats, loadInlineIntentBodies, type IntentInlineDoc } from "../intent.ts";
 import { openListPanel, type CommandUI, type CommandsDeps } from "./_helpers.ts";
 
-type DocsDeps = Pick<CommandsDeps, "getIntentDocs">;
+type DocsDeps = Pick<CommandsDeps, "getIntentDocs" | "recordEvent">;
 
 export function registerDocsCommand(pi: ExtensionAPI, deps: DocsDeps): void {
-	const { getIntentDocs } = deps;
+	const { getIntentDocs, recordEvent } = deps;
 
 	pi.registerCommand("docs", {
 		description: "manage soly intent docs (stats — show context breakdown)",
@@ -30,7 +30,7 @@ export function registerDocsCommand(pi: ExtensionAPI, deps: DocsDeps): void {
 			if (sub === "list") {
 				const docs = getIntentDocs();
 				if (docs.length === 0) {
-					ui.notify("no intent docs found in .agents/docs/ — drop your vision/domain docs there", "info");
+					recordEvent("no intent docs found in .agents/docs/ — drop your vision/domain docs there");
 					return;
 				}
 				if (ctx.mode === "tui") {
@@ -55,7 +55,7 @@ export function registerDocsCommand(pi: ExtensionAPI, deps: DocsDeps): void {
 					});
 					return;
 				}
-				ui.notify(docs.map((d) => `○ ${d.title || d.relPath} (${formatTok(d.tokens)} tok)`).join("\n"), "info");
+				recordEvent(docs.map((d) => `○ ${d.title || d.relPath} (${formatTok(d.tokens)} tok)`).join("\n"));
 				return;
 			}
 
@@ -63,7 +63,7 @@ export function registerDocsCommand(pi: ExtensionAPI, deps: DocsDeps): void {
 				const docs = getIntentDocs();
 				const inlineBodies: IntentInlineDoc[] = loadInlineIntentBodies(docs);
 				const stats = buildIntentStats(docs, inlineBodies);
-				ui.notify(formatIntentStats(stats), "info");
+				recordEvent(formatIntentStats(stats));
 				return;
 			}
 

package/commands/rules.ts

@@ -30,10 +30,10 @@ function ruleItem(r: RuleFile): ListItem {
 	};
 }
 
-type RulesDeps = Pick<CommandsDeps, "getRules" | "getOverridden" | "refreshRules" | "updateStatus">;
+type RulesDeps = Pick<CommandsDeps, "getRules" | "getOverridden" | "refreshRules" | "updateStatus" | "recordEvent">;
 
 export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
-	const { getRules, getOverridden, refreshRules, updateStatus } = deps;
+	const { getRules, getOverridden, refreshRules, updateStatus, recordEvent } = deps;
 
 	pi.registerCommand("rules", {
 		description: "manage soly rules (list, show, stats, analytics, reload, enable, disable)",
@@ -55,7 +55,7 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 				const rules = getRules();
 				const overridden = getOverridden();
 				if (rules.length === 0 && overridden.length === 0) {
-					ui.notify("soly: no rules loaded — check `.agents/rules/` or `~/.agents/rules/`", "warning");
+					recordEvent("no rules loaded — check `.agents/rules/` or `~/.agents/rules/`", "warning");
 					return;
 				}
 				// Rich modal in the TUI; plain select elsewhere (RPC/print).
@@ -85,7 +85,7 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 				for (const p of overridden) {
 					lines.push(`⊘ [overridden] ${p}`);
 				}
-				ui.notify(lines.join("\n"), "info");
+				recordEvent(lines.join("\n"));
 				return;
 			}
 
@@ -100,20 +100,20 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 					return;
 				}
 				const text = `---\n${r.meta.description ? `description: ${r.meta.description}\n` : ""}source: ${r.sourceLabel}\nenabled: ${r.enabled}\n---\n\n${r.body}`;
-				ui.notify(text, "info");
+				recordEvent(text);
 				return;
 			}
 
 			if (sub === "stats") {
 				const analytics = analyzeRules(getRules(), CONTEXT_WINDOW_TOKENS);
-				ui.notify(formatAnalyticsFull(analytics), "info");
+				recordEvent(formatAnalyticsFull(analytics));
 				return;
 			}
 
 			if (sub === "analytics") {
 				const rules = getRules();
 				const stats = buildRulesContextStats(rules, CONTEXT_WINDOW_TOKENS);
-				ui.notify(formatRulesContextStats(stats), "info");
+				recordEvent(formatRulesContextStats(stats));
 				return;
 			}
 
@@ -129,14 +129,14 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 				}
 				r.enabled = sub === "enable";
 				updateStatus(ui);
-				ui.notify(`Rule ${target}: ${sub}d`, "info");
+				recordEvent(`Rule ${target}: ${sub}d`);
 				return;
 			}
 
 			if (sub === "reload") {
 				refreshRules();
 				updateStatus(ui);
-				ui.notify(`Reloaded ${getRules().length} rules`, "info");
+				recordEvent(`Reloaded ${getRules().length} rules`);
 				return;
 			}
 

package/commands/rulewizard.ts

@@ -3,8 +3,10 @@
 // =============================================================================
 
 import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
+import type { CommandsDeps } from "./_helpers.ts";
 
-export function registerRulewizardCommand(pi: ExtensionAPI): void {
+export function registerRulewizardCommand(pi: ExtensionAPI, deps: CommandsDeps): void {
+	void deps; // rulewizard doesn't currently emit non-error events
 	pi.registerCommand("rulewizard", {
 		description:
 			"interactive guide: decide whether a constraint should be a soly rule, an .editorconfig entry, or a linter config (eslint/biome/prettier). Use this BEFORE writing a new rule to avoid duplicating what linters already enforce.",

package/commands/soly.ts

@@ -25,10 +25,10 @@ import { initSolyProject } from "../init.js";
 import { parseSolyCommand, type SolyCommand, type WorkflowVerb } from "../workflows/parser.ts";
 import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 
-type SolyDeps = Pick<CommandsDeps, "getState" | "getConfig" | "reloadConfig" | "updateStatus" | "refreshState">;
+type SolyDeps = Pick<CommandsDeps, "getState" | "getConfig" | "reloadConfig" | "updateStatus" | "refreshState" | "recordEvent">;
 
 export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
-	const { getState, getConfig, reloadConfig, updateStatus, refreshState } = deps;
+	const { getState, getConfig, reloadConfig, updateStatus, refreshState, recordEvent } = deps;
 
 	const solyBody = async (args: string, ctx: ExtensionCommandContext): Promise<void> => {
 			const ui: CommandUI = {
@@ -105,22 +105,22 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 				switch (verb) {
 					case "new": {
 						const r = buildNewTransform(cmd, state, ui, ctx.cwd, getConfig().plan.defaultBranchPrefix);
-						if (r.handled && r.transformedText) ui.notify(r.transformedText, "info");
+						if (r.handled && r.transformedText) recordEvent(r.transformedText);
 						return;
 					}
 					case "done": {
 						const r = buildDoneTransform(cmd, state, ui, ctx.cwd, { defaultBranchPrefix: getConfig().plan.defaultBranchPrefix });
-						if (r.handled && r.transformedText) ui.notify(r.transformedText, "info");
+						if (r.handled && r.transformedText) recordEvent(r.transformedText);
 						return;
 					}
 					case "migrate": {
 						const r = buildMigrateTransform(state, ui, ctx.cwd);
-						if (r.handled && r.transformedText) ui.notify(r.transformedText, "info");
+						if (r.handled && r.transformedText) recordEvent(r.transformedText);
 						return;
 					}
 					case "plan": {
 						const r = buildPlanTransform(cmd, state);
-						if (r.handled && r.transformedText) ui.notify(r.transformedText, "info");
+						if (r.handled && r.transformedText) recordEvent(r.transformedText);
 						return;
 					}
 					case "discuss": {
@@ -129,13 +129,13 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 						// plain-text discussion format; the LLM-driven path still
 						// gets the richer ask_pro flow.
 						const r = buildDiscussTransform(cmd, state, { hasAskPro: false });
-						if (r.handled && r.transformedText) ui.notify(r.transformedText, "info");
+						if (r.handled && r.transformedText) recordEvent(r.transformedText);
 						return;
 					}
 					case "execute": {
 						// Slash-command path doesn't have live interactive rules.
 						const r = buildExecuteTransform(cmd, state, []);
-						if (r.handled && r.transformedText) ui.notify(r.transformedText, "info");
+						if (r.handled && r.transformedText) recordEvent(r.transformedText);
 						return;
 					}
 					default:
@@ -181,7 +181,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 								"info",
 							);
 						} else {
-							ui.notify(`${s.milestone} — no position set`, "info");
+							recordEvent(`${s.milestone} — no position set`);
 						}
 					},
 				},
@@ -244,7 +244,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 					run: () => {
 						const s = getState();
 						const line = `${s.progress.percent}% · ${s.progress.completedPhases}/${s.progress.totalPhases} phases · ${s.progress.completedPlans}/${s.progress.totalPlans} plans`;
-						ui.notify(line, "info");
+						recordEvent(line);
 					},
 				},
 				phases: {
@@ -252,7 +252,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 					run: () => {
 						const phases = getState().phases;
 						if (phases.length === 0) {
-							ui.notify("no phases", "info");
+							recordEvent("no phases");
 							return;
 						}
 						const current = getState().currentPhase?.number;
@@ -261,7 +261,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 							const cr = (p.contextExists ? "C" : "·") + (p.researchExists ? "R" : "·");
 							return `${marker} ${String(p.number).padStart(2, "0")}. ${p.name}  [${cr}]  plans=${p.planCount}`;
 						});
-						ui.notify(lines.join("\n"), "info");
+						recordEvent(lines.join("\n"));
 					},
 				},
 				tasks: {
@@ -269,7 +269,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 					run: () => {
 						const s = getState();
 						if (s.tasks.length === 0) {
-							ui.notify("no tasks", "info");
+							recordEvent("no tasks");
 							return;
 						}
 						const byFeature = new Map<string, typeof s.tasks>();
@@ -288,7 +288,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 							}
 							out.push("");
 						}
-						ui.notify(out.join("\n"), "info");
+						recordEvent(out.join("\n"));
 					},
 				},
 				task: {
@@ -319,7 +319,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 						if (summaryBody) {
 							showFile("SUMMARY.md", summaryBody);
 						} else {
-							ui.notify("no SUMMARY.md yet", "info");
+							recordEvent("no SUMMARY.md yet");
 						}
 					},
 				},
@@ -328,14 +328,14 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 					run: () => {
 						const features = getState().features;
 						if (features.length === 0) {
-							ui.notify("no features", "info");
+							recordEvent("no features");
 							return;
 						}
 						const lines = features.map((f) => {
 							const rm = f.readmeExists ? "R" : "·";
 							return `  ${f.name.padEnd(28)} tasks=${f.taskCount}  [${rm}]`;
 						});
-						ui.notify(lines.join("\n"), "info");
+						recordEvent(lines.join("\n"));
 					},
 				},
 				milestone: {
@@ -343,7 +343,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 					run: () => {
 						const s = getState();
 						if (!s.milestone || s.milestone === "—") {
-							ui.notify("soly: no milestone set", "info");
+							recordEvent("no milestone set");
 							return;
 						}
 						const candidates = [
@@ -369,7 +369,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 						refreshState();
 						updateStatus(ui);
 						const s = getState();
-						ui.notify(`re-read state: ${s.phases.length} phases · ${s.tasks.length} tasks`, "info");
+						recordEvent(`re-read state: ${s.phases.length} phases · ${s.tasks.length} tasks`);
 					},
 				},
 				// ------------------------------------------------------------------

package/commands/why.ts

@@ -5,10 +5,10 @@
 import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import type { CommandsDeps } from "./_helpers.ts";
 
-type WhyDeps = Pick<CommandsDeps, "getState" | "getRules">;
+type WhyDeps = Pick<CommandsDeps, "getState" | "getRules" | "recordEvent">;
 
 export function registerWhyCommand(pi: ExtensionAPI, deps: WhyDeps): void {
-	const { getState, getRules } = deps;
+	const { getState, getRules, recordEvent } = deps;
 
 	pi.registerCommand("why", {
 		description:

package/commands.ts

@@ -39,6 +39,6 @@ export function registerCommands(pi: ExtensionAPI, deps: CommandsDeps): void {
 	registerDocsCommand(pi, deps);
 	registerSolyCommand(pi, deps); // also registers /sly and /s
 	registerArtifactsCommand(pi, deps);
-	registerRulewizardCommand(pi);
+	registerRulewizardCommand(pi, deps);
 	registerWhyCommand(pi, deps);
 }

package/index.ts

@@ -384,6 +384,10 @@ export default function solyExtension(pi: ExtensionAPI) {
 			}
 		},
 		getIntentDocs: () => intentDocs,
+		// Non-error soly events land in the Working sub-line (└─ prefixed).
+		// The route goes through the chrome so the sub-line auto-clears on
+		// the next agent_start.
+		recordEvent: (text, level) => chrome.recordEvent(text, level),
 	});
 
 	registerTools(pi, {
@@ -393,6 +397,7 @@ export default function solyExtension(pi: ExtensionAPI) {
 	});
 
 	registerWorkflows(pi, {
+		recordEvent: (text, level) => chrome.recordEvent(text, level),
 		getState: () => state,
 		getInteractiveRules: () =>
 			combinedRules()
@@ -531,9 +536,11 @@ export default function solyExtension(pi: ExtensionAPI) {
 			},
 		});
 		// Editors save in bursts (write to .tmp, rename, touch). Coalesce
-		// those rapid reload events into a single user-visible notify.
+		// those rapid reload events into a single sub-line event under the
+		// Working indicator (└─ reloaded 47 rules). Errors here are real
+		// (disk I/O failed) and stay as popups.
 		hotReload.setNotifyHandler((reason) => {
-			ctx.ui.notify(`soly: rules reloaded (${reason})`, "info");
+			chrome.recordEvent(`reloaded rules (${reason})`);
 		});
 
 		// Notifications (one-shot at startup)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-soly",
-  "version": "2.2.6",
+  "version": "2.3.1",
   "description": "Workflow + project management for pi-coding-agent. Plans, state, MANDATORY rules, self-review, multi-question picker. One npm install, zero config. LLM drives the workflow inline via the soly_workflow tool — no external subagent plugin.",
   "type": "module",
   "main": "index.ts",

package/visual/chrome.ts

@@ -61,6 +61,11 @@ export type Chrome = {
 	updateWorking(ui: ExtensionUIContext, outputTokens: number): void;
 	/** Stop the telemetry line and restore the default message (agent_end). */
 	stopWorking(ui: ExtensionUIContext): void;
+	/** Record a non-error soly event. Rendered as a sub-line under the
+	 *  Working indicator (└─ prefixed, level-marker glyph). Auto-cleared on
+	 *  the next `startWorking`. Errors are still surfaced via
+	 *  `ui.notify(text, "error")` — call that directly for the popup. */
+	recordEvent(text: string, level?: "info" | "warning"): void;
 	/** Restore pi's native footer/widgets/indicator (session_shutdown / disable). */
 	dispose(ui?: ExtensionUIContext): void;
 };
@@ -98,9 +103,15 @@ export function createChrome(getConfig: () => ChromeConfig): Chrome {
 	};
 
 	return {
+		recordEvent(text: string, level: "info" | "warning" = "info"): void {
+			data.recentEvent = text;
+			data.recentEventLevel = level;
+			try { tui?.requestRender(); } catch { /* not mounted yet */ }
+		},
+
 		data,
 
-		install(ui): void {
+		install(ui: ExtensionUIContext): void {
 			if (!getConfig().enabled) return;
 			ui.setWorkingIndicator({ frames: getConfig().spinnerFrames, intervalMs: getConfig().spinnerIntervalMs });
 			ui.setFooter((t, theme, footerData) => {
@@ -133,6 +144,10 @@ export function createChrome(getConfig: () => ChromeConfig): Chrome {
 		startWorking(ui): void {
 			if (!getConfig().enabled || !getConfig().telemetry) return;
 			clearWorking();
+			// Each new turn starts with a clean sub-line — old events were
+			// from the previous turn, not relevant to the current one.
+			data.recentEvent = null;
+			data.recentEventLevel = null;
 			working = { ui, startMs: Date.now(), inputTokens: data.ctxTokens ?? 0, outputTokens: 0, timer: null };
 			renderWorking();
 			working.timer = setInterval(renderWorking, 1000);

package/visual/data.ts

@@ -39,6 +39,12 @@ export type ChromeData = {
 	verbLabel: string | null;
 	/** html_artifacts created this session (0 = no segment). */
 	artifactCount: number;
+	/** Most recent non-error soly event (e.g. "reloaded 47 rules"). Rendered
+	 *  as a sub-line under the Working indicator; auto-cleared by the next
+	 *  agent_start. null = no recent event. */
+	recentEvent: string | null;
+	/** Level of the recent event (used for glyph/color). */
+	recentEventLevel: "info" | "warning" | "error" | null;
 };
 
 /** A fresh ChromeData with everything empty/idle. */
@@ -58,5 +64,7 @@ export function emptyChromeData(): ChromeData {
 		phaseLabel: null,
 		verbLabel: null,
 		artifactCount: 0,
+		recentEvent: null,
+		recentEventLevel: null,
 	};
 }

package/visual/footer.ts

@@ -124,6 +124,15 @@ export class SolyFooter implements Component {
 	}
 
 	render(width: number): string[] {
-		return [buildFooterLine(this.data, this.fd, width, { ascii: this.getAscii(), styler: themeStyler(this.theme) })];
+		const out: string[] = [buildFooterLine(this.data, this.fd, width, { ascii: this.getAscii(), styler: themeStyler(this.theme) })];
+		const event = this.data.recentEvent;
+		if (event) {
+			const glyph = this.data.recentEventLevel === "warning" ? "└─ ⚠" : "└─";
+			const styled = this.data.recentEventLevel === "warning"
+				? themeStyler(this.theme).fg("warning", `${glyph} ${event}`)
+				: themeStyler(this.theme).dim(`${glyph} ${event}`);
+			out.push("    " + styled);
+		}
+		return out;
 	}
 }

package/workflows/done.ts

@@ -95,7 +95,7 @@ export function buildDoneTransform(
 	if ("error" in parsed) {
 		return reply(`soly done: ${parsed.error}\n\nUsage: soly done <slug> OR soly done <prefix>/<slug>`);
 	}
-	const { name, prefix } = parsed;
+	const { name, prefix, autoSlugified, originalInput } = parsed;
 	const effectivePrefix = prefix ?? opts.defaultBranchPrefix ?? "";
 	const branchName = effectivePrefix ? `${effectivePrefix}/${name}` : name;
 

package/workflows/index.ts

@@ -32,6 +32,7 @@ import type { SolyState } from "../core.js";
 import type { SolyConfig } from "../config.js";
 
 export interface WorkflowsDeps {
+	recordEvent: (text: string, level?: "info" | "warning") => void;
 	getState: () => SolyState;
 	/** List of rule relPaths marked `interactive: true` — inlined into the
 	 *  execute instruction so the model knows which rules are explicitly out of
@@ -57,7 +58,7 @@ export interface WorkflowsDeps {
 const MODE_VERBS: readonly string[] = ["execute", "plan", "discuss", "resume"];
 
 export function registerWorkflows(pi: ExtensionAPI, deps: WorkflowsDeps): void {
-	const { getState, getInteractiveRules, getActiveTools, getConfig, onWorkflowUsed } = deps;
+	const { getState, getInteractiveRules, getActiveTools, getConfig, onWorkflowUsed, recordEvent } = deps;
 
 	// Self-review loop ("soly verify"). Owns its own agent_end + input hooks;
 	// fresh-context mode rewrites the next LLM call through the context manager.
@@ -276,7 +277,7 @@ State inspection lives on the slash form — \`/soly <sub>\`:
 				"and .agents/.continue-here.md. Preserve milestone/phase/plan position and key " +
 				"decisions in the summary. Drop implementation-detail noise.",
 			onComplete: () => {
-				ctx.ui.notify("soly: session compacted. Use `soly resume` to pick up.", "info");
+				recordEvent("session compacted — use 'soly resume' to pick up");
 			},
 			onError: (err) => {
 				ctx.ui.notify(`soly: compact failed — ${err.message}`, "error");

package/workflows/new.ts

@@ -89,7 +89,7 @@ export function buildNewTransform(
 	const parsed = parsePlanName(cmd.args.join(" "));
 	if ("error" in parsed) return reply(`soly new: ${parsed.error}`);
 
-	const { name, prefix } = parsed;
+	const { name, prefix, autoSlugified, originalInput } = parsed;
 	// Branch can be one of three shapes:
 	//   - "feature/statistic-preparation" (user typed <prefix>/<slug>)
 	//   - "feature/statistic-preparation" (config has defaultBranchPrefix "feature", user typed "statistic-preparation")
@@ -123,18 +123,45 @@ export function buildNewTransform(
 
 	const currentBranch = git(["branch", "--show-current"], { cwd: projectRoot }) || "HEAD (detached)";
 	// A plan branch is a kebab-case slug (no `<type>/` prefix after 1.15.x).
-	// The branch can also be the long-lived integration branches `master`
-	// or `main`. Anything else (release tags, weird suffixes) → user must
-	// checkout first.
+	// The base can also be the long-lived integration branches `master`/`main`.
+	// Anything else (release tags, weird suffixes, feature branches from
+	// previous work) — we auto-checkout the base branch instead of
+	// blocking the user. The LLM driving soly shouldn't have to manage
+	// git state.
 	if (
 		currentBranch !== "master" &&
 		currentBranch !== "main" &&
 		!/^[a-z0-9][a-z0-9-]*[a-z0-9]$/.test(currentBranch)
 	) {
-		return reply(
-			`soly new: currently on "${currentBranch}" (not master/main, not a soly plan branch). ` +
-				`Switch back to master first with \`git checkout master\`.`,
-		);
+		// Detect which base branch the repo uses: try `main` first, fall
+		// back to `master`. If neither exists, give up — the repo is in
+		// a state the user needs to resolve by hand.
+		const baseBranch = ["main", "master"].find((b) => {
+			try {
+				git(["rev-parse", "--verify", b], { cwd: projectRoot });
+				return true;
+			} catch {
+				return false;
+			}
+		});
+		if (!baseBranch) {
+			return reply(
+				`soly new: cannot find a base branch to base off — repo has neither "main" nor "master". ` +
+					`Run \`git checkout <base>\` first, or create a default branch.`,
+			);
+		}
+		try {
+			git(["checkout", baseBranch], { cwd: projectRoot });
+			ui.notify(
+				`auto-checkout ${baseBranch} (was on ${currentBranch})`,
+				"info",
+			);
+		} catch (err) {
+			const msg = err instanceof Error ? err.message : String(err);
+			return reply(
+				`soly new: failed to checkout ${baseBranch} (was on ${currentBranch}): ${msg}`,
+			);
+		}
 	}
 
 	let branchExisted = false;

package/workflows/parser.ts

@@ -37,12 +37,36 @@ export type WorkflowVerb =
  *   soly new statistic-preparation
  *   soly new login-redirect-bug
  *   soly new stats-rollup
+ *
+ * If the input doesn't match the strict kebab-case shape, we attempt
+ * to auto-slugify it (lowercase, replace non-slug chars with `-`,
+ * collapse, trim). For free-form prose like "this is a feature" or
+ * even Cyrillic text, the auto-slugified result becomes the plan name
+ * and the original input is preserved as the plan's description so
+ * the LLM has the human-authored context.
  */
 export function parsePlanName(
 	raw: string,
-): { name: string; prefix: string | null } | { error: string } {
+): { name: string; prefix: string | null; autoSlugified: boolean; originalInput: string } | { error: string; tried: string } {
 	const trimmed = raw.trim();
-	if (!trimmed) return { error: "missing plan name" };
+	if (!trimmed) return { error: "missing plan name", tried: "" };
+
+	// Strict regex: ASCII alphanumerics + hyphen, must start and end with
+	// an alnum. The kebab-case shape we want to encourage.
+	// Unicode regex used for the lenient path below (e.g. Cyrillic, accents).
+	const SLUG_STRICT = /^[a-z0-9][a-z0-9-]*[a-z0-9]$/u;
+	const SLUG_LENIENT = /^[\p{L}\p{N}][\p{L}\p{N}_-]*[\p{L}\p{N}]$/u;
+	const MAX_LEN = 64;
+	const MIN_LEN = 2;
+
+	/** Lowercase, replace non-slug chars with `-`, collapse, trim. */
+	const autoSlugify = (s: string): string =>
+		s
+			.toLowerCase()
+			.normalize("NFKD")
+			.replace(/[^\p{L}\p{N}_-]/gu, "-")
+			.replace(/-+/g, "-")
+			.replace(/^-+|-+$/g, "");
 
 	// Optional `<prefix>/<slug>` form: prefix is one kebab-case word
 	// (no internal slashes), slug is the same kebab-case shape. Examples:
@@ -54,49 +78,67 @@ export function parsePlanName(
 	if (slashIdx > 0) {
 		const prefix = trimmed.slice(0, slashIdx);
 		const slug = trimmed.slice(slashIdx + 1);
-		const prefixOk = /^[a-z][a-z0-9-]*[a-z0-9]$/.test(prefix);
-		const slugOk = /^[a-z][a-z0-9-]*[a-z0-9]$/.test(slug);
+		const prefixOk = SLUG_STRICT.test(prefix);
+		const slugOk = SLUG_STRICT.test(slug);
 		if (!prefixOk || !slugOk) {
 			return {
 				error:
 					`bad plan name "${trimmed}".\n` +
-					`\nExpected: <slug> OR <prefix>/<slug>\n` +
+					`\nExpected: <prefix>/<slug>  (e.g. "feature/login-redirect-bug")\n` +
 					`  prefix: one kebab-case word (e.g. "feature", "fix", "chore")\n` +
 					`  slug:   kebab-case (lowercase letters, digits, hyphens)\n` +
-					`\nExample: soly new feature/statistic-preparation`,
+					`\nTip: pass free-form prose instead and soly will auto-slugify:\n` +
+					`  soly new "add a button to the settings page"  →  add-a-button-to-the-settings-page`,
+				tried: trimmed,
 			};
 		}
-		return { name: slug, prefix };
+		return { name: slug, prefix, autoSlugified: false, originalInput: trimmed };
 	}
 
-	// Plain slug form. Must contain at least one letter — pure-digit strings
-	// are reserved for phase numbers (`soly plan 11` → phase 11), not slugs.
-	if (!/[a-z]/.test(trimmed)) {
-		return {
-			error:
-				`bad plan name "${trimmed}".\n` +
-				`\nExpected: <slug> OR <prefix>/<slug>\n` +
-				`  chars:  lowercase letters, digits, and hyphens\n` +
-				`         must start and end with a letter or digit\n` +
-				`\nExample: soly new statistic-preparation`,
-		};
-	}
-	const m = trimmed.match(/^[a-z0-9][a-z0-9-]*[a-z0-9]$/);
-	if (!m) {
-		return {
-			error:
-				`bad plan name "${trimmed}".\n` +
-				`\nExpected: <slug> OR <prefix>/<slug>\n` +
-				`  length: 2-64 chars\n` +
-				`  chars:  lowercase letters, digits, and hyphens\n` +
-				`         must start and end with a letter or digit\n` +
-				`\nExample: soly new statistic-preparation`,
-		};
+	// Plain slug form. If it's already kebab-case, accept as-is.
+	// Exception: pure-digit strings are reserved for phase numbers
+	// (`soly plan 11` → phase 11), not slugs.
+	if (
+		SLUG_STRICT.test(trimmed) &&
+		/[a-z]/.test(trimmed) &&
+		trimmed.length >= MIN_LEN &&
+		trimmed.length <= MAX_LEN
+	) {
+		return { name: trimmed, prefix: null, autoSlugified: false, originalInput: trimmed };
 	}
-	if (trimmed.length > 64) {
-		return { error: `name "${trimmed}" is too long (max 64 chars)` };
+
+	// Otherwise, try to auto-slugify (handles free-form prose, Cyrillic,
+	// accented chars, mixed punctuation). The result must be a valid slug
+	// under the lenient Unicode regex.
+	//
+	// Exception: pure-digit strings (e.g. "11") are reserved for phase
+	// numbers (`soly plan 11` → phase 11), not slugs. Don't auto-slugify them.
+	const candidate = autoSlugify(trimmed);
+	const isPureDigits = /^\d+$/.test(candidate);
+	if (
+		!isPureDigits &&
+		candidate.length >= MIN_LEN &&
+		candidate.length <= MAX_LEN &&
+		SLUG_LENIENT.test(candidate)
+	) {
+		return { name: candidate, prefix: null, autoSlugified: true, originalInput: trimmed };
 	}
-	return { name: trimmed, prefix: null };
+
+	// Couldn't make anything valid out of the input. Surface a clear error
+	// that explains what we tried and what works.
+	return {
+		error:
+			`bad plan name "${trimmed}".\n` +
+			`\nExpected: <slug> OR <prefix>/<slug>\n` +
+			`  length: ${MIN_LEN}-${MAX_LEN} chars\n` +
+			`  chars:  Unicode letters/digits/hyphens/underscores (auto-slugified to kebab-case)\n` +
+			`\nTip: pass the plan description as a free-form string and soly will\n` +
+			`auto-slugify it for you:\n` +
+			`  soly new "add a button to the settings page"  →  add-a-button-to-the-settings-page\n` +
+			`\nOr pick a short kebab-case name yourself:\n` +
+			`  soly new statistic-preparation`,
+		tried: candidate,
+	};
 }
 
 export interface SolyCommand {
@@ -242,7 +284,7 @@ function parseNewTaskFlag(
  */
 export type ExecuteTarget =
 	| { kind: "phase"; phase: number; plan: number | null; raw: string }
-	| { kind: "plan"; name: string; prefix: string | null; raw: string }
+	| { kind: "plan"; name: string; prefix: string | null; autoSlugified: boolean; originalInput: string; raw: string }
 	| { kind: "task"; taskId: string; raw: string }
 	| { kind: "all"; raw: string }
 	| { kind: "feature"; feature: string; raw: string };
@@ -286,16 +328,19 @@ export function describeExecuteTarget(args: string[]): ExecuteTarget | null {
 		return { kind: "task", taskId, raw };
 	}
 
-	const plan = parsePlanName(target);
-	if (!("error" in plan)) {
-		return { kind: "plan", name: plan.name, prefix: plan.prefix, raw };
-	}
-
+	// Check phase shape FIRST — pure digits and NN.MM patterns are phase
+	// numbers, not plan slugs. Must come before parsePlanName because
+	// auto-slugify would turn "11.02" into "11-02" (a valid plan slug).
 	const phase = parsePhaseShape(target);
 	if (phase) {
 		return { kind: "phase", phase: phase.phase, plan: phase.plan, raw };
 	}
 
+	const plan = parsePlanName(target);
+	if (!("error" in plan)) {
+		return { kind: "plan", name: plan.name, prefix: plan.prefix, autoSlugified: plan.autoSlugified, originalInput: plan.originalInput, raw };
+	}
+
 	return null;
 }
 
@@ -314,7 +359,7 @@ export function describeExecuteTarget(args: string[]): ExecuteTarget | null {
  */
 export type PlanTarget =
 	| { kind: "phase"; phase: number; raw: string }
-	| { kind: "plan"; name: string; prefix: string | null; raw: string }
+	| { kind: "plan"; name: string; prefix: string | null; autoSlugified: boolean; originalInput: string; raw: string }
 	| { kind: "task"; taskId: string; raw: string }
 	| { kind: "new-task"; slug: string; feature: string; raw: string }
 	| { kind: "feature"; feature: string; raw: string };
@@ -360,17 +405,18 @@ export function describePlanTarget(args: string[]): PlanTarget | null {
 		return { kind: "task", taskId, raw };
 	}
 
-	const plan = parsePlanName(target);
-	if (!("error" in plan)) {
-		return { kind: "plan", name: plan.name, prefix: plan.prefix, raw };
-	}
-
-	// Plan target only matches plain N (no .MM — plan is per-phase, executed
-	// at the phase level by `soly execute <N.MM>`).
+	// Check phase shape FIRST — pure digits are phase numbers, not slugs.
+	// Must come before parsePlanName because auto-slugify would turn digits
+	// into a valid plan slug.
 	const phase = parsePhaseOnlyShape(target);
 	if (phase) {
 		return { kind: "phase", phase: phase.phase, raw };
 	}
 
+	const plan = parsePlanName(target);
+	if (!("error" in plan)) {
+		return { kind: "plan", name: plan.name, prefix: plan.prefix, autoSlugified: plan.autoSlugified, originalInput: plan.originalInput, raw };
+	}
+
 	return null;
 }