@oh-my-pi/pi-coding-agent 13.9.14 → 13.9.15

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [13.9.15] - 2026-03-10
+### Added
+
+- Added `ensureLoadingAnimation()` method to manage loading animation lifecycle and prevent duplicate spinners
+
+### Changed
+
+- Refactored loading animation initialization to use centralized `ensureLoadingAnimation()` method in event and input controllers
+- Updated `showError()` to properly clean up loading animation state when errors occur
+
 ## [13.9.12] - 2026-03-09
 ### Added
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.9.14",
+	"version": "13.9.15",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "13.9.14",
-		"@oh-my-pi/pi-agent-core": "13.9.14",
-		"@oh-my-pi/pi-ai": "13.9.14",
-		"@oh-my-pi/pi-natives": "13.9.14",
-		"@oh-my-pi/pi-tui": "13.9.14",
-		"@oh-my-pi/pi-utils": "13.9.14",
+		"@oh-my-pi/omp-stats": "13.9.15",
+		"@oh-my-pi/pi-agent-core": "13.9.15",
+		"@oh-my-pi/pi-ai": "13.9.15",
+		"@oh-my-pi/pi-natives": "13.9.15",
+		"@oh-my-pi/pi-tui": "13.9.15",
+		"@oh-my-pi/pi-utils": "13.9.15",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/modes/controllers/event-controller.ts

@@ -107,19 +107,7 @@ export class EventController {
 					this.ctx.retryLoader = undefined;
 					this.ctx.statusContainer.clear();
 				}
-				if (this.ctx.loadingAnimation) {
-					this.ctx.loadingAnimation.stop();
-				}
-				this.ctx.statusContainer.clear();
-				this.ctx.loadingAnimation = new Loader(
-					this.ctx.ui,
-					spinner => theme.fg("accent", spinner),
-					text => theme.fg("muted", text),
-					`Working… (esc to interrupt)`,
-					getSymbolTheme().spinnerFrames,
-				);
-				this.ctx.statusContainer.addChild(this.ctx.loadingAnimation);
-				this.ctx.applyPendingWorkingMessage();
+				this.ctx.ensureLoadingAnimation();
 				this.ctx.ui.requestRender();
 				break;
 

package/src/modes/controllers/input-controller.ts

@@ -339,6 +339,7 @@ export class InputController {
 				};
 				this.ctx.addMessageToChat(optimisticMessage);
 				this.ctx.editor.setText("");
+				this.ctx.ensureLoadingAnimation();
 				this.ctx.ui.requestRender();
 
 				this.ctx.onInputCallback({ text, images });

package/src/modes/interactive-mode.ts

@@ -5,8 +5,8 @@
 import * as path from "node:path";
 import { type Agent, type AgentMessage, ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import type { AssistantMessage, ImageContent, Message, Model, UsageReport } from "@oh-my-pi/pi-ai";
-import type { Component, Loader, SlashCommand } from "@oh-my-pi/pi-tui";
-import { Container, Markdown, ProcessTerminal, Spacer, Text, TUI } from "@oh-my-pi/pi-tui";
+import type { Component, SlashCommand } from "@oh-my-pi/pi-tui";
+import { Container, Loader, Markdown, ProcessTerminal, Spacer, Text, TUI } from "@oh-my-pi/pi-tui";
 import { APP_NAME, getProjectDir, hsvToRgb, isEnoent, logger, postmortem } from "@oh-my-pi/pi-utils";
 import chalk from "chalk";
 import { KeybindingsManager } from "../config/keybindings";
@@ -46,7 +46,14 @@ import { SSHCommandController } from "./controllers/ssh-command-controller";
 import { OAuthManualInputManager } from "./oauth-manual-input";
 import { setMermaidRenderCallback } from "./theme/mermaid-cache";
 import type { Theme } from "./theme/theme";
-import { getEditorTheme, getMarkdownTheme, onTerminalAppearanceChange, onThemeChange, theme } from "./theme/theme";
+import {
+	getEditorTheme,
+	getMarkdownTheme,
+	getSymbolTheme,
+	onTerminalAppearanceChange,
+	onThemeChange,
+	theme,
+} from "./theme/theme";
 import type { CompactionQueuedMessage, InteractiveModeContext, TodoItem, TodoPhase } from "./types";
 import { UiHelpers } from "./utils/ui-helpers";
 
@@ -849,6 +856,12 @@ export class InteractiveMode implements InteractiveModeContext {
 
 	showError(message: string): void {
 		this.optimisticUserMessageSignature = undefined;
+		this.#pendingWorkingMessage = undefined;
+		if (this.loadingAnimation) {
+			this.loadingAnimation.stop();
+			this.loadingAnimation = undefined;
+			this.statusContainer.clear();
+		}
 		this.#uiHelpers.showError(message);
 	}
 
@@ -856,6 +869,22 @@ export class InteractiveMode implements InteractiveModeContext {
 		this.#uiHelpers.showWarning(message);
 	}
 
+	ensureLoadingAnimation(): void {
+		if (!this.loadingAnimation) {
+			this.statusContainer.clear();
+			this.loadingAnimation = new Loader(
+				this.ui,
+				spinner => theme.fg("accent", spinner),
+				text => theme.fg("muted", text),
+				this.#defaultWorkingMessage,
+				getSymbolTheme().spinnerFrames,
+			);
+			this.statusContainer.addChild(this.loadingAnimation);
+		}
+
+		this.applyPendingWorkingMessage();
+	}
+
 	setWorkingMessage(message?: string): void {
 		if (message === undefined) {
 			this.#pendingWorkingMessage = undefined;

package/src/modes/types.ts

@@ -128,6 +128,7 @@ export interface InteractiveModeContext {
 	flushPendingModelSwitch(): Promise<void>;
 	setWorkingMessage(message?: string): void;
 	applyPendingWorkingMessage(): void;
+	ensureLoadingAnimation(): void;
 	isKnownSlashCommand(text: string): boolean;
 	addMessageToChat(message: AgentMessage, options?: { populateHistory?: boolean }): void;
 	renderSessionContext(