@oh-my-pi/pi-coding-agent 8.1.0 → 8.2.0

package/src/modes/components/plugin-settings.ts

@@ -7,10 +7,6 @@
  *     - Feature toggles
  *     - Config value editor
  */
-
-import { PluginManager } from "@oh-my-pi/pi-coding-agent/extensibility/plugins/manager";
-import type { InstalledPlugin, PluginSettingSchema } from "@oh-my-pi/pi-coding-agent/extensibility/plugins/types";
-import { getSelectListTheme, getSettingsListTheme, theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
 import {
 	Container,
 	Input,
@@ -21,6 +17,9 @@ import {
 	Spacer,
 	Text,
 } from "@oh-my-pi/pi-tui";
+import { PluginManager } from "../../extensibility/plugins/manager";
+import type { InstalledPlugin, PluginSettingSchema } from "../../extensibility/plugins/types";
+import { getSelectListTheme, getSettingsListTheme, theme } from "../../modes/theme/theme";
 import { DynamicBorder } from "./dynamic-border";
 
 // =============================================================================
@@ -62,7 +61,7 @@ export class PluginListComponent extends Container {
 			return;
 		}
 
-		const items: SelectItem[] = plugins.map((p) => {
+		const items: SelectItem[] = plugins.map(p => {
 			const status = p.enabled
 				? theme.fg("success", theme.status.enabled)
 				: theme.fg("muted", theme.status.disabled);
@@ -83,8 +82,8 @@ export class PluginListComponent extends Container {
 
 		this.selectList = new SelectList(items, Math.min(items.length, 8), getSelectListTheme());
 
-		this.selectList.onSelect = (item) => {
-			const plugin = this.plugins.find((p) => p.name === item.value);
+		this.selectList.onSelect = item => {
+			const plugin = this.plugins.find(p => p.name === item.value);
 			if (plugin) {
 				callbacks.onPluginSelect(plugin);
 			}
@@ -132,10 +131,10 @@ export class PluginDetailComponent extends Container {
 		this.manager = manager;
 		this.callbacks = callbacks;
 
-		this.rebuild();
+		void this.rebuild();
 	}
 
-	private rebuild(): void {
+	private async rebuild(): Promise<void> {
 		this.clear();
 
 		const plugin = this.plugin;
@@ -184,7 +183,7 @@ export class PluginDetailComponent extends Container {
 
 		// Config settings
 		if (manifest.settings && Object.keys(manifest.settings).length > 0) {
-			const settings = this.manager.getPluginSettings(plugin.name);
+			const settings = await this.manager.getPluginSettings(plugin.name);
 
 			for (const [key, schema] of Object.entries(manifest.settings)) {
 				const currentValue = settings[key] ?? schema.default;
@@ -210,7 +209,7 @@ export class PluginDetailComponent extends Container {
 								schema.description || `Select value for ${key}`,
 								schema.values,
 								cv,
-								(value) => {
+								value => {
 									this.callbacks.onConfigChange(key, value);
 									done(value);
 								},
@@ -229,7 +228,7 @@ export class PluginDetailComponent extends Container {
 								key,
 								schema,
 								cv === "(not set)" ? "" : cv,
-								(value) => {
+								value => {
 									const parsed = schema.type === "number" ? Number(value) : value;
 									this.callbacks.onConfigChange(key, parsed);
 									done(String(value));
@@ -278,6 +277,7 @@ export class PluginDetailComponent extends Container {
 	}
 
 	handleInput(data: string): void {
+		if (!this.settingsList) return;
 		this.settingsList.handleInput(data);
 	}
 }
@@ -309,7 +309,7 @@ class ConfigEnumSubmenu extends Container {
 		}
 		this.addChild(new Spacer(1));
 
-		const items: SelectItem[] = values.map((v) => ({ value: v, label: v }));
+		const items: SelectItem[] = values.map(v => ({ value: v, label: v }));
 		this.selectList = new SelectList(items, Math.min(items.length, 8), getSelectListTheme());
 
 		const currentIndex = values.indexOf(currentValue);
@@ -317,7 +317,7 @@ class ConfigEnumSubmenu extends Container {
 			this.selectList.setSelectedIndex(currentIndex);
 		}
 
-		this.selectList.onSelect = (item) => onSelect(item.value);
+		this.selectList.onSelect = item => onSelect(item.value);
 		this.selectList.onCancel = onCancel;
 
 		this.addChild(this.selectList);
@@ -374,7 +374,7 @@ class ConfigInputSubmenu extends Container {
 			this.input.setValue(currentValue);
 		}
 
-		this.input.onSubmit = (value) => {
+		this.input.onSubmit = value => {
 			if (value.trim()) {
 				this.onSubmit(value);
 			} else {
@@ -438,7 +438,7 @@ export class PluginSettingsComponent extends Container {
 		const plugins = await this.manager.list();
 
 		this.viewComponent = new PluginListComponent(plugins, {
-			onPluginSelect: (plugin) => this.showPluginDetail(plugin),
+			onPluginSelect: plugin => this.showPluginDetail(plugin),
 			onCancel: () => this.callbacks.onClose(),
 		});
 
@@ -451,12 +451,12 @@ export class PluginSettingsComponent extends Container {
 		this.clear();
 
 		this.viewComponent = new PluginDetailComponent(plugin, this.manager, {
-			onEnabledChange: async (enabled) => {
+			onEnabledChange: async enabled => {
 				await this.manager.setEnabled(plugin.name, enabled);
 				this.callbacks.onPluginChanged();
 			},
 			onFeatureChange: async (feature, enabled) => {
-				const current = new Set(this.manager.getEnabledFeatures(plugin.name) ?? []);
+				const current = new Set((await this.manager.getEnabledFeatures(plugin.name)) ?? []);
 				if (enabled) {
 					current.add(feature);
 				} else {
@@ -465,8 +465,8 @@ export class PluginSettingsComponent extends Container {
 				await this.manager.setEnabledFeatures(plugin.name, [...current]);
 				this.callbacks.onPluginChanged();
 			},
-			onConfigChange: (key, value) => {
-				this.manager.setPluginSetting(plugin.name, key, value);
+			onConfigChange: async (key, value) => {
+				await this.manager.setPluginSetting(plugin.name, key, value);
 				this.callbacks.onPluginChanged();
 			},
 			onBack: () => this.showPluginList(),

package/src/modes/components/python-execution.ts

@@ -2,12 +2,11 @@
  * Component for displaying user-initiated Python execution with streaming output.
  * Shares the same kernel session as the agent's Python tool.
  */
-
-import { getSymbolTheme, highlightCode, theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import type { TruncationMeta } from "@oh-my-pi/pi-coding-agent/tools/output-meta";
-import { formatSize } from "@oh-my-pi/pi-coding-agent/tools/truncate";
 import { Container, Loader, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
 import stripAnsi from "strip-ansi";
+import { getSymbolTheme, highlightCode, theme } from "../../modes/theme/theme";
+import type { TruncationMeta } from "../../tools/output-meta";
+import { formatSize } from "../../tools/truncate";
 import { DynamicBorder } from "./dynamic-border";
 import { truncateToVisualLines } from "./visual-truncate";
 
@@ -51,8 +50,8 @@ export class PythonExecutionComponent extends Container {
 
 		this.loader = new Loader(
 			ui,
-			(spinner) => theme.fg(colorKey, spinner),
-			(text) => theme.fg("muted", text),
+			spinner => theme.fg(colorKey, spinner),
+			text => theme.fg("muted", text),
 			`Running${theme.format.ellipsis} (esc to cancel)`,
 			getSymbolTheme().spinnerFrames,
 		);
@@ -117,10 +116,10 @@ export class PythonExecutionComponent extends Container {
 
 		if (availableLines.length > 0) {
 			if (this.expanded) {
-				const displayText = availableLines.map((line) => theme.fg("muted", line)).join("\n");
+				const displayText = availableLines.map(line => theme.fg("muted", line)).join("\n");
 				this.contentContainer.addChild(new Text(`\n${displayText}`, 1, 0));
 			} else {
-				const styledOutput = previewLogicalLines.map((line) => theme.fg("muted", line)).join("\n");
+				const styledOutput = previewLogicalLines.map(line => theme.fg("muted", line)).join("\n");
 				const previewText = `\n${styledOutput}`;
 				this.contentContainer.addChild({
 					render: (width: number) => {

package/src/modes/components/queue-mode-selector.ts

@@ -1,5 +1,5 @@
-import { getSelectListTheme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
 import { Container, type SelectItem, SelectList } from "@oh-my-pi/pi-tui";
+import { getSelectListTheme } from "../../modes/theme/theme";
 import { DynamicBorder } from "./dynamic-border";
 
 /**
@@ -31,12 +31,12 @@ export class QueueModeSelectorComponent extends Container {
 		this.selectList = new SelectList(queueModes, 2, getSelectListTheme());
 
 		// Preselect current mode
-		const currentIndex = queueModes.findIndex((item) => item.value === currentMode);
+		const currentIndex = queueModes.findIndex(item => item.value === currentMode);
 		if (currentIndex !== -1) {
 			this.selectList.setSelectedIndex(currentIndex);
 		}
 
-		this.selectList.onSelect = (item) => {
+		this.selectList.onSelect = item => {
 			onSelect(item.value as "all" | "one-at-a-time");
 		};
 

package/src/modes/components/read-tool-group.ts

@@ -1,7 +1,7 @@
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import { shortenPath } from "@oh-my-pi/pi-coding-agent/tools/render-utils";
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Container, Text } from "@oh-my-pi/pi-tui";
+import { theme } from "../../modes/theme/theme";
+import { shortenPath } from "../../tools/render-utils";
 import type { ToolExecutionHandle } from "./tool-execution";
 
 type ReadRenderArgs = {

package/src/modes/components/session-selector.ts

@@ -1,6 +1,3 @@
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import type { SessionInfo } from "@oh-my-pi/pi-coding-agent/session/session-manager";
-import { fuzzyFilter } from "@oh-my-pi/pi-coding-agent/utils/fuzzy";
 import {
 	type Component,
 	Container,
@@ -11,6 +8,9 @@ import {
 	truncateToWidth,
 	visibleWidth,
 } from "@oh-my-pi/pi-tui";
+import { theme } from "../../modes/theme/theme";
+import type { SessionInfo } from "../../session/session-manager";
+import { fuzzyFilter } from "../../utils/fuzzy";
 import { DynamicBorder } from "./dynamic-border";
 
 /**
@@ -45,7 +45,7 @@ class SessionList implements Component {
 	}
 
 	private filterSessions(query: string): void {
-		this.filteredSessions = fuzzyFilter(this.allSessions, query, (session) => {
+		this.filteredSessions = fuzzyFilter(this.allSessions, query, session => {
 			const parts = [
 				session.id,
 				session.title ?? "",