@oh-my-pi/pi-coding-agent 11.0.0 → 11.0.2

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [11.0.2] - 2026-02-05
+
+### Fixed
+
+- Fixed role model cycling to expand role aliases (e.g., roles pointing at `pi/plan`) so slow/default/smol cycles resolve correctly
+
 ## [11.0.0] - 2026-02-05
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "11.0.0",
+	"version": "11.0.2",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -80,12 +80,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "11.0.0",
-		"@oh-my-pi/pi-agent-core": "11.0.0",
-		"@oh-my-pi/pi-ai": "11.0.0",
-		"@oh-my-pi/pi-natives": "11.0.0",
-		"@oh-my-pi/pi-tui": "11.0.0",
-		"@oh-my-pi/pi-utils": "11.0.0",
+		"@oh-my-pi/omp-stats": "11.0.2",
+		"@oh-my-pi/pi-agent-core": "11.0.2",
+		"@oh-my-pi/pi-ai": "11.0.2",
+		"@oh-my-pi/pi-natives": "11.0.2",
+		"@oh-my-pi/pi-tui": "11.0.2",
+		"@oh-my-pi/pi-utils": "11.0.2",
 		"@openai/agents": "^0.4.5",
 		"@sinclair/typebox": "^0.34.48",
 		"ajv": "^8.17.1",

package/src/session/agent-session.ts

@@ -33,7 +33,7 @@ import { YAML } from "bun";
 import type { Rule } from "../capability/rule";
 import { getAgentDbPath } from "../config";
 import { MODEL_ROLE_IDS, type ModelRegistry, type ModelRole } from "../config/model-registry";
-import { parseModelString } from "../config/model-resolver";
+import { expandRoleAlias, parseModelString } from "../config/model-resolver";
 import {
 	expandPromptTemplate,
 	type PromptTemplate,
@@ -1871,13 +1871,14 @@ export class AgentSession {
 					: this.settings.getModelRole(role);
 			if (!roleModelStr) continue;
 
-			const parsed = parseModelString(roleModelStr);
+			const expandedRoleModelStr = expandRoleAlias(roleModelStr, this.settings);
+			const parsed = parseModelString(expandedRoleModelStr);
 			let match: Model | undefined;
 			if (parsed) {
 				match = availableModels.find(m => m.provider === parsed.provider && m.id === parsed.id);
 			}
 			if (!match) {
-				match = availableModels.find(m => m.id.toLowerCase() === roleModelStr.toLowerCase());
+				match = availableModels.find(m => m.id.toLowerCase() === expandedRoleModelStr.toLowerCase());
 			}
 			if (!match) continue;
 

package/src/session/session-manager.ts

@@ -579,23 +579,22 @@ function extractFirstUserPrompt(entries: Array<Record<string, unknown>>): string
 async function getSortedSessions(sessionDir: string, storage: SessionStorage): Promise<RecentSessionInfo[]> {
 	try {
 		const files: string[] = storage.listFilesSync(sessionDir, "*.jsonl");
-		const results = await Promise.all(
+		const sessions: RecentSessionInfo[] = [];
+		await Promise.all(
 			files.map(async (path: string) => {
 				try {
 					const content = await storage.readTextPrefix(path, 4096);
 					const entries = parseJsonlEntries<Record<string, unknown>>(content);
-					if (entries.length === 0) return null;
+					if (entries.length === 0) return;
 					const header = entries[0] as Record<string, unknown>;
-					if (header.type !== "session" || typeof header.id !== "string") return null;
+					if (header.type !== "session" || typeof header.id !== "string") return;
 					const mtime = storage.statSync(path).mtimeMs;
 					const firstPrompt = header.title ? undefined : extractFirstUserPrompt(entries);
-					return new RecentSessionInfo(path, mtime, header, firstPrompt);
-				} catch {
-					return null;
-				}
+					sessions.push(new RecentSessionInfo(path, mtime, header, firstPrompt));
+				} catch {}
 			}),
 		);
-		return results.filter((item): item is RecentSessionInfo => item !== null).sort((a, b) => b.mtime - a.mtime);
+		return sessions.sort((a, b) => b.mtime - a.mtime);
 	} catch {
 		return [];
 	}
@@ -908,62 +907,65 @@ function extractTextFromContent(content: Message["content"]): string {
 async function collectSessionsFromFiles(files: string[], storage: SessionStorage): Promise<SessionInfo[]> {
 	const sessions: SessionInfo[] = [];
 
-	for (const file of files) {
-		try {
-			const content = await storage.readText(file);
-			const entries = parseJsonlEntries<Record<string, unknown>>(content);
-			if (entries.length === 0) continue;
-
-			// Check first entry for valid session header
-			type SessionHeaderShape = { type: string; id: string; cwd?: string; title?: string; timestamp: string };
-			const header = entries[0] as SessionHeaderShape;
-			if (header.type !== "session" || !header.id) continue;
-
-			const stats = storage.statSync(file);
-			let messageCount = 0;
-			let firstMessage = "";
-			const allMessages: string[] = [];
-			let shortSummary: string | undefined;
-
-			for (let i = 1; i < entries.length; i++) {
-				const entry = entries[i] as { type?: string; message?: Message; shortSummary?: string };
-
-				if (entry.type === "compaction" && typeof entry.shortSummary === "string") {
-					shortSummary = entry.shortSummary;
-				}
+	// Collect session info for all files in parallel
+	await Promise.all(
+		files.map(async file => {
+			try {
+				const content = await storage.readText(file);
+				const entries = parseJsonlEntries<Record<string, unknown>>(content);
+				if (entries.length === 0) return;
+
+				// Check first entry for valid session header
+				type SessionHeaderShape = { type: string; id: string; cwd?: string; title?: string; timestamp: string };
+				const header = entries[0] as SessionHeaderShape;
+				if (header.type !== "session" || !header.id) return;
+
+				let messageCount = 0;
+				let firstMessage = "";
+				const allMessages: string[] = [];
+				let shortSummary: string | undefined;
+
+				for (let i = 1; i < entries.length; i++) {
+					const entry = entries[i] as { type?: string; message?: Message; shortSummary?: string };
+
+					if (entry.type === "compaction" && typeof entry.shortSummary === "string") {
+						shortSummary = entry.shortSummary;
+					}
 
-				if (entry.type === "message" && entry.message) {
-					messageCount++;
+					if (entry.type === "message" && entry.message) {
+						messageCount++;
 
-					if (entry.message.role === "user" || entry.message.role === "assistant") {
-						const textContent = extractTextFromContent(entry.message.content);
+						if (entry.message.role === "user" || entry.message.role === "assistant") {
+							const textContent = extractTextFromContent(entry.message.content);
 
-						if (textContent) {
-							allMessages.push(textContent);
+							if (textContent) {
+								allMessages.push(textContent);
 
-							if (!firstMessage && entry.message.role === "user") {
-								firstMessage = textContent;
+								if (!firstMessage && entry.message.role === "user") {
+									firstMessage = textContent;
+								}
 							}
 						}
 					}
 				}
-			}
 
-			sessions.push({
-				path: file,
-				id: header.id,
-				cwd: typeof header.cwd === "string" ? header.cwd : "",
-				title: header.title ?? shortSummary,
-				created: new Date(header.timestamp),
-				modified: stats.mtime,
-				messageCount,
-				firstMessage: firstMessage || "(no messages)",
-				allMessagesText: allMessages.join(" "),
-			});
-		} catch {
-			// Skip files that can't be read
-		}
-	}
+				if (messageCount) {
+					const stats = storage.statSync(file);
+					sessions.push({
+						path: file,
+						id: header.id,
+						cwd: typeof header.cwd === "string" ? header.cwd : "",
+						title: header.title ?? shortSummary,
+						created: new Date(header.timestamp),
+						modified: stats.mtime,
+						messageCount,
+						firstMessage: firstMessage || "(no messages)",
+						allMessagesText: allMessages.join(" "),
+					});
+				}
+			} catch {}
+		}),
+	);
 
 	sessions.sort((a, b) => b.modified.getTime() - a.modified.getTime());
 	return sessions;