@oh-my-pi/pi-coding-agent 11.0.3 → 11.1.0

package/CHANGELOG.md

@@ -2,6 +2,38 @@
 
 ## [Unreleased]
 
+## [11.1.0] - 2026-02-05
+
+### Added
+
+- Added `sortDiagnostics()` utility function to sort diagnostics by severity, location, and message for consistent output ordering
+- Added `task.isolation.enabled` setting to control whether subagents run in isolated git worktrees
+- Added dynamic task schema that conditionally includes `isolated` parameter based on isolation setting
+- Added `openInEditor()` utility function to centralize external editor handling with support for custom file extensions and stdio configuration
+- Added `getEditorCommand()` utility function to retrieve the user's preferred editor from $VISUAL or $EDITOR environment variables
+
+### Changed
+
+- Changed diagnostic output to sort results by severity (errors first), then by file location and message for improved readability
+- Changed task tool to validate isolation setting and reject `isolated` parameter when isolation is disabled
+- Changed task API to use `assignment` field instead of `args` for per-task instructions, with shared `context` prepended to every task
+- Changed task template rendering to use structured context/assignment separation with `<swarm_context>` wrapper instead of placeholder-based substitution
+- Changed task item schema to require `assignment` string (complete per-task instructions) instead of optional `args` object
+- Changed `TaskItem` to remove `args` field and add `assignment` field for clearer per-task instruction semantics
+- Changed agent frontmatter to use `thinking-level` field name instead of `thinkingLevel` for consistency
+- Refactored task rendering to display full task text instead of args in progress and result views
+- Changed `SubmenuSettingDef.getOptions()` method to `options` getter property for cleaner API access
+- Converted static option providers from functions to direct array definitions for improved performance
+- Added `createSubmenuSettingDef()` helper function to support both static and dynamic option providers
+- Modified `setThinkingLevel()` API to accept optional `persist` parameter (defaults to false) for controlling whether thinking level changes are saved to settings
+- Refactored hook editor and input controller to use shared external editor utilities, reducing code duplication
+
+### Removed
+
+- Removed `context` parameter from `ExecutorOptions` — context now prepended at template level before task execution
+- Removed `args` field from `AgentProgress` and `SingleResult` interfaces
+- Removed placeholder-based template rendering in favor of structured context/assignment model
+
 ## [11.0.3] - 2026-02-05
 ### Added
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "11.0.3",
+	"version": "11.1.0",
 	"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.3",
-		"@oh-my-pi/pi-agent-core": "11.0.3",
-		"@oh-my-pi/pi-ai": "11.0.3",
-		"@oh-my-pi/pi-natives": "11.0.3",
-		"@oh-my-pi/pi-tui": "11.0.3",
-		"@oh-my-pi/pi-utils": "11.0.3",
+		"@oh-my-pi/omp-stats": "11.1.0",
+		"@oh-my-pi/pi-agent-core": "11.1.0",
+		"@oh-my-pi/pi-ai": "11.1.0",
+		"@oh-my-pi/pi-natives": "11.1.0",
+		"@oh-my-pi/pi-tui": "11.1.0",
+		"@oh-my-pi/pi-utils": "11.1.0",
 		"@openai/agents": "^0.4.5",
 		"@sinclair/typebox": "^0.34.48",
 		"ajv": "^8.17.1",

package/src/cli/config-cli.ts

@@ -48,7 +48,7 @@ function getSettingValues(def: SettingDef): readonly string[] | undefined {
 		return def.values;
 	}
 	if (def.type === "submenu") {
-		const options = def.getOptions();
+		const options = def.options;
 		if (options.length > 0) {
 			return options.map(o => o.value);
 		}

package/src/commit/agentic/tools/analyze-file.ts

@@ -62,7 +62,6 @@ export function createAnalyzeFileTool(options: {
 		async execute(toolCallId, params, onUpdate, ctx, signal) {
 			const toolSession = buildToolSession(ctx, options);
 			const taskTool = await TaskTool.create(toolSession);
-			const context = "{{prompt}}";
 			const numstat = options.state.overview?.numstat ?? [];
 			const tasks = params.files.map((file, index) => {
 				const relatedFiles = formatRelatedFiles(params.files, file, numstat);
@@ -74,12 +73,11 @@ export function createAnalyzeFileTool(options: {
 				return {
 					id: `AnalyzeFile${index + 1}`,
 					description: `Analyze ${file}`,
-					args: { prompt },
+					assignment: prompt,
 				};
 			});
 			const taskParams: TaskParams = {
 				agent: "quick_task",
-				context,
 				schema: analyzeFileOutputSchema,
 				tasks,
 			};

package/src/commit/git/errors.ts

@@ -1,11 +1,9 @@
 export class GitError extends Error {
-	readonly command: string;
-	readonly stderr: string;
-
-	constructor(command: string, stderr: string) {
+	constructor(
+		readonly command: string,
+		readonly stderr: string,
+	) {
 		super(`${command} failed: ${stderr || "unknown error"}`);
-		this.command = command;
-		this.stderr = stderr;
 		this.name = "GitError";
 	}
 }

package/src/config/keybindings.ts

@@ -151,11 +151,9 @@ export function formatKeyHints(keys: KeyId | KeyId[]): string {
  * Manages all keybindings (app + editor).
  */
 export class KeybindingsManager {
-	private config: KeybindingsConfig;
 	private appActionToKeys: Map<AppAction, KeyId[]>;
 
-	private constructor(config: KeybindingsConfig) {
-		this.config = config;
+	private constructor(private readonly config: KeybindingsConfig) {
 		this.appActionToKeys = new Map();
 		this.buildMaps();
 	}

package/src/config/settings-schema.ts

@@ -399,6 +399,16 @@ export const SETTINGS_SCHEMA = {
 	// ─────────────────────────────────────────────────────────────────────────
 	// Task tool settings
 	// ─────────────────────────────────────────────────────────────────────────
+	"task.isolation.enabled": {
+		type: "boolean",
+		default: false,
+		ui: {
+			tab: "tools",
+			label: "Task isolation",
+			description: "Run subagents in isolated git worktrees",
+			submenu: true,
+		},
+	},
 	"task.maxConcurrency": {
 		type: "number",
 		default: 32,

package/src/exa/mcp-client.ts

@@ -249,17 +249,14 @@ export async function fetchMCPToolSchema(
 export class MCPWrappedTool implements CustomTool<TSchema, ExaRenderDetails> {
 	public readonly name: string;
 	public readonly label: string;
-	public readonly description: string;
-	public readonly parameters: TSchema;
 
-	private readonly config: MCPToolWrapperConfig;
-
-	constructor(config: MCPToolWrapperConfig, schema: TSchema, description: string) {
-		this.config = config;
+	constructor(
+		private readonly config: MCPToolWrapperConfig,
+		public readonly parameters: TSchema,
+		public readonly description: string,
+	) {
 		this.name = config.name;
 		this.label = config.label;
-		this.description = description;
-		this.parameters = schema;
 	}
 
 	async execute(

package/src/extensibility/extensions/loader.ts

@@ -103,17 +103,14 @@ class ConcreteExtensionAPI implements ExtensionAPI, IExtensionRuntime {
 	readonly logger = logger;
 	readonly typebox = TypeBox;
 	readonly pi = piCodingAgent;
-	readonly events: EventBus;
 	readonly flagValues = new Map<string, boolean | string>();
 
 	constructor(
-		private extension: Extension,
-		private runtime: IExtensionRuntime,
-		private cwd: string,
-		eventBus: EventBus,
-	) {
-		this.events = eventBus;
-	}
+		private readonly extension: Extension,
+		private readonly runtime: IExtensionRuntime,
+		private readonly cwd: string,
+		public readonly events: EventBus,
+	) {}
 
 	on<F extends HandlerFn>(event: string, handler: F): void {
 		const list = this.extension.handlers.get(event) ?? [];
@@ -214,8 +211,8 @@ class ConcreteExtensionAPI implements ExtensionAPI, IExtensionRuntime {
 		return this.runtime.getThinkingLevel();
 	}
 
-	setThinkingLevel(level: ThinkingLevel): void {
-		this.runtime.setThinkingLevel(level);
+	setThinkingLevel(level: ThinkingLevel, persist?: boolean): void {
+		this.runtime.setThinkingLevel(level, persist);
 	}
 }
 

package/src/extensibility/extensions/runner.ts

@@ -105,12 +105,7 @@ const noOpUIContext: ExtensionUIContext = {
 };
 
 export class ExtensionRunner {
-	private extensions: Extension[];
-	private runtime: ExtensionRuntime;
 	private uiContext: ExtensionUIContext;
-	private cwd: string;
-	private sessionManager: SessionManager;
-	private modelRegistry: ModelRegistry;
 	private errorListeners: Set<ExtensionErrorListener> = new Set();
 	private getModel: () => Model | undefined = () => undefined;
 	private isIdleFn: () => boolean = () => true;
@@ -125,18 +120,13 @@ export class ExtensionRunner {
 	private shutdownHandler: ShutdownHandler = () => {};
 
 	constructor(
-		extensions: Extension[],
-		runtime: ExtensionRuntime,
-		cwd: string,
-		sessionManager: SessionManager,
-		modelRegistry: ModelRegistry,
+		private readonly extensions: Extension[],
+		private readonly runtime: ExtensionRuntime,
+		private readonly cwd: string,
+		private readonly sessionManager: SessionManager,
+		private readonly modelRegistry: ModelRegistry,
 	) {
-		this.extensions = extensions;
-		this.runtime = runtime;
 		this.uiContext = noOpUIContext;
-		this.cwd = cwd;
-		this.sessionManager = sessionManager;
-		this.modelRegistry = modelRegistry;
 	}
 
 	initialize(

package/src/extensibility/extensions/types.ts

@@ -839,7 +839,7 @@ export type SetModelHandler = (model: Model) => Promise<boolean>;
 
 export type GetThinkingLevelHandler = () => ThinkingLevel;
 
-export type SetThinkingLevelHandler = (level: ThinkingLevel) => void;
+export type SetThinkingLevelHandler = (level: ThinkingLevel, persist?: boolean) => void;
 
 /** Shared state created by loader, used during registration and runtime. */
 export interface ExtensionRuntimeState {

package/src/extensibility/hooks/runner.ts

@@ -62,12 +62,8 @@ const noOpUIContext: HookUIContext = {
  * HookRunner executes hooks and manages event emission.
  */
 export class HookRunner {
-	private hooks: LoadedHook[];
 	private uiContext: HookUIContext;
 	private hasUI: boolean;
-	private cwd: string;
-	private sessionManager: SessionManager;
-	private modelRegistry: ModelRegistry;
 	private errorListeners: Set<HookErrorListener> = new Set();
 	private getModel: () => Model | undefined = () => undefined;
 	private isIdleFn: () => boolean = () => true;
@@ -78,13 +74,14 @@ export class HookRunner {
 	private branchHandler: BranchHandler = async () => ({ cancelled: false });
 	private navigateTreeHandler: NavigateTreeHandler = async () => ({ cancelled: false });
 
-	constructor(hooks: LoadedHook[], cwd: string, sessionManager: SessionManager, modelRegistry: ModelRegistry) {
-		this.hooks = hooks;
+	constructor(
+		private readonly hooks: LoadedHook[],
+		private readonly cwd: string,
+		private readonly sessionManager: SessionManager,
+		private readonly modelRegistry: ModelRegistry,
+	) {
 		this.uiContext = noOpUIContext;
 		this.hasUI = false;
-		this.cwd = cwd;
-		this.sessionManager = sessionManager;
-		this.modelRegistry = modelRegistry;
 	}
 
 	/**

package/src/ipy/kernel.ts

@@ -16,11 +16,11 @@ const PRELUDE_INTROSPECTION_SNIPPET = "import json\nprint(json.dumps(__omp_prelu
 const debugStartup = $env.PI_DEBUG_STARTUP ? (stage: string) => process.stderr.write(`[startup] ${stage}\n`) : () => {};
 
 class SharedGatewayCreateError extends Error {
-	readonly status: number;
-
-	constructor(status: number, message: string) {
+	constructor(
+		readonly status: number,
+		message: string,
+	) {
 		super(message);
-		this.status = status;
 	}
 }
 
@@ -265,12 +265,6 @@ export function serializeWebSocketMessage(msg: JupyterMessage): ArrayBuffer {
 }
 
 export class PythonKernel {
-	readonly id: string;
-	readonly kernelId: string;
-	readonly gatewayUrl: string;
-	readonly sessionId: string;
-	readonly username: string;
-	readonly isSharedGateway: boolean;
 	readonly #authToken?: string;
 
 	#ws: WebSocket | null = null;
@@ -281,20 +275,14 @@ export class PythonKernel {
 	#pendingExecutions = new Map<string, (reason: string) => void>();
 
 	private constructor(
-		id: string,
-		kernelId: string,
-		gatewayUrl: string,
-		sessionId: string,
-		username: string,
-		isSharedGateway: boolean,
+		readonly id: string,
+		readonly kernelId: string,
+		readonly gatewayUrl: string,
+		readonly sessionId: string,
+		readonly username: string,
+		readonly isSharedGateway: boolean,
 		authToken?: string,
 	) {
-		this.id = id;
-		this.kernelId = kernelId;
-		this.gatewayUrl = gatewayUrl;
-		this.sessionId = sessionId;
-		this.username = username;
-		this.isSharedGateway = isSharedGateway;
 		this.#authToken = authToken;
 	}
 

package/src/lsp/clients/biome-client.ts

@@ -107,18 +107,15 @@ async function runBiome(
  * Parses Biome's --reporter=json output into LSP Diagnostic format.
  */
 export class BiomeClient implements LinterClient {
-	private config: ServerConfig;
-	private cwd: string;
-
 	/** Factory method for creating BiomeClient instances */
 	static create(config: ServerConfig, cwd: string): LinterClient {
 		return new BiomeClient(config, cwd);
 	}
 
-	constructor(config: ServerConfig, cwd: string) {
-		this.config = config;
-		this.cwd = cwd;
-	}
+	constructor(
+		private readonly config: ServerConfig,
+		private readonly cwd: string,
+	) {}
 
 	async format(filePath: string, content: string): Promise<string> {
 		// Write content to file first

package/src/lsp/clients/lsp-linter-client.ts

@@ -21,8 +21,6 @@ const DEFAULT_FORMAT_OPTIONS = {
  * Wraps the existing LSP client infrastructure.
  */
 export class LspLinterClient implements LinterClient {
-	private config: ServerConfig;
-	private cwd: string;
 	private client: LspClient | null = null;
 
 	/** Factory method for creating LspLinterClient instances */
@@ -30,10 +28,10 @@ export class LspLinterClient implements LinterClient {
 		return new LspLinterClient(config, cwd);
 	}
 
-	constructor(config: ServerConfig, cwd: string) {
-		this.config = config;
-		this.cwd = cwd;
-	}
+	constructor(
+		private readonly config: ServerConfig,
+		private readonly cwd: string,
+	) {}
 
 	private async getClient(): Promise<LspClient> {
 		if (!this.client) {

package/src/lsp/index.ts

@@ -50,6 +50,7 @@ import {
 	formatLocation,
 	formatSymbolInformation,
 	formatWorkspaceEdit,
+	sortDiagnostics,
 	symbolKindToIcon,
 } from "./utils";
 
@@ -458,6 +459,7 @@ async function getDiagnosticsForFile(
 		}
 	}
 
+	sortDiagnostics(uniqueDiagnostics);
 	const formatted = uniqueDiagnostics.map(d => formatDiagnostic(d, relPath));
 	const limited = limitDiagnosticMessages(formatted);
 	const summary = formatDiagnosticsSummary(uniqueDiagnostics);
@@ -858,10 +860,7 @@ export class LspTool implements AgentTool<typeof lspSchema, LspToolDetails, Them
 	public readonly mergeCallAndResult = true;
 	public readonly inline = true;
 
-	private readonly session: ToolSession;
-
-	constructor(session: ToolSession) {
-		this.session = session;
+	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(lspDescription);
 	}
 
@@ -966,6 +965,8 @@ export class LspTool implements AgentTool<typeof lspSchema, LspToolDetails, Them
 					}
 				}
 
+				sortDiagnostics(uniqueDiagnostics);
+
 				if (!detailed && targets.length === 1) {
 					if (uniqueDiagnostics.length === 0) {
 						return {

package/src/lsp/utils.ts

@@ -205,6 +205,24 @@ export function severityToString(severity?: DiagnosticSeverity): string {
 	return SEVERITY_NAMES[severity ?? 1] ?? "unknown";
 }
 
+/**
+ * Sort diagnostics by severity, then by location and message.
+ */
+export function sortDiagnostics(diagnostics: Diagnostic[]): Diagnostic[] {
+	return diagnostics.sort((a, b) => {
+		const aSeverity = a.severity ?? 1;
+		const bSeverity = b.severity ?? 1;
+		if (aSeverity !== bSeverity) return aSeverity - bSeverity;
+		const aLine = a.range.start.line;
+		const bLine = b.range.start.line;
+		if (aLine !== bLine) return aLine - bLine;
+		const aCol = a.range.start.character;
+		const bCol = b.range.start.character;
+		if (aCol !== bCol) return aCol - bCol;
+		return a.message.localeCompare(b.message);
+	});
+}
+
 /**
  * Get icon for diagnostic severity.
  */

package/src/modes/components/armin.ts

@@ -57,7 +57,6 @@ function buildFinalGrid(): string[][] {
 }
 
 export class ArminComponent implements Component {
-	private ui: TUI;
 	private interval: ReturnType<typeof setInterval> | null = null;
 	private effect: Effect;
 	private finalGrid: string[][];
@@ -68,8 +67,7 @@ export class ArminComponent implements Component {
 	private gridVersion = 0;
 	private cachedVersion = -1;
 
-	constructor(ui: TUI) {
-		this.ui = ui;
+	constructor(private readonly ui: TUI) {
 		this.effect = EFFECTS[Math.floor(Math.random() * EFFECTS.length)];
 		this.finalGrid = buildFinalGrid();
 		this.currentGrid = this.createEmptyGrid();

package/src/modes/components/assistant-message.ts

@@ -8,15 +8,15 @@ import { getMarkdownTheme, theme } from "../../modes/theme/theme";
  */
 export class AssistantMessageComponent extends Container {
 	private contentContainer: Container;
-	private hideThinkingBlock: boolean;
 	private lastMessage?: AssistantMessage;
 	private prerenderInFlight = false;
 
-	constructor(message?: AssistantMessage, hideThinkingBlock = false) {
+	constructor(
+		message?: AssistantMessage,
+		private hideThinkingBlock = false,
+	) {
 		super();
 
-		this.hideThinkingBlock = hideThinkingBlock;
-
 		// Container for text/thinking content
 		this.contentContainer = new Container();
 		this.addChild(this.contentContainer);

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

@@ -12,7 +12,6 @@ import { truncateToVisualLines } from "./visual-truncate";
 const PREVIEW_LINES = 20;
 
 export class BashExecutionComponent extends Container {
-	private command: string;
 	private outputLines: string[] = [];
 	private status: "running" | "complete" | "cancelled" | "error" = "running";
 	private exitCode: number | undefined = undefined;
@@ -21,9 +20,12 @@ export class BashExecutionComponent extends Container {
 	private expanded = false;
 	private contentContainer: Container;
 
-	constructor(command: string, ui: TUI, excludeFromContext = false) {
+	constructor(
+		private readonly command: string,
+		ui: TUI,
+		excludeFromContext = false,
+	) {
 		super();
-		this.command = command;
 
 		// Use dim border for excluded-from-context commands (!! prefix)
 		const colorKey = excludeFromContext ? "dim" : "bashMode";

package/src/modes/components/branch-summary-message.ts

@@ -8,11 +8,9 @@ import type { BranchSummaryMessage } from "../../session/messages";
  */
 export class BranchSummaryMessageComponent extends Box {
 	private expanded = false;
-	private message: BranchSummaryMessage;
 
-	constructor(message: BranchSummaryMessage) {
+	constructor(private readonly message: BranchSummaryMessage) {
 		super(1, 1, t => theme.bg("customMessageBg", t));
-		this.message = message;
 		this.updateDisplay();
 	}
 

package/src/modes/components/compaction-summary-message.ts

@@ -8,11 +8,9 @@ import type { CompactionSummaryMessage } from "../../session/messages";
  */
 export class CompactionSummaryMessageComponent extends Box {
 	private expanded = false;
-	private message: CompactionSummaryMessage;
 
-	constructor(message: CompactionSummaryMessage) {
+	constructor(private readonly message: CompactionSummaryMessage) {
 		super(1, 1, t => theme.bg("customMessageBg", t));
-		this.message = message;
 		this.updateDisplay();
 	}
 

package/src/modes/components/custom-message.ts

@@ -10,16 +10,15 @@ import type { CustomMessage } from "../../session/messages";
  * Uses distinct styling to differentiate from user messages.
  */
 export class CustomMessageComponent extends Container {
-	private message: CustomMessage<unknown>;
-	private customRenderer?: MessageRenderer;
 	private box: Box;
 	private customComponent?: Component;
 	private _expanded = false;
 
-	constructor(message: CustomMessage<unknown>, customRenderer?: MessageRenderer) {
+	constructor(
+		private readonly message: CustomMessage<unknown>,
+		private readonly customRenderer?: MessageRenderer,
+	) {
 		super();
-		this.message = message;
-		this.customRenderer = customRenderer;
 
 		this.addChild(new Spacer(1));
 

package/src/modes/components/extensions/extension-dashboard.ts

@@ -33,17 +33,15 @@ export class ExtensionDashboard extends Container {
 	private state!: DashboardState;
 	private mainList!: ExtensionList;
 	private inspector!: InspectorPanel;
-	private settingsInstance: Settings | null;
-	private cwd: string;
-	private terminalHeight: number;
 
 	public onClose?: () => void;
 
-	private constructor(cwd: string, settingsInstance: Settings | null, terminalHeight: number) {
+	private constructor(
+		private readonly cwd: string,
+		private readonly settingsInstance: Settings | null,
+		private readonly terminalHeight: number,
+	) {
 		super();
-		this.cwd = cwd;
-		this.settingsInstance = settingsInstance;
-		this.terminalHeight = terminalHeight;
 	}
 
 	static async create(
@@ -292,15 +290,11 @@ export class ExtensionDashboard extends Container {
  * Two-column body component for side-by-side rendering.
  */
 class TwoColumnBody implements Component {
-	private leftPane: ExtensionList;
-	private rightPane: InspectorPanel;
-	private maxHeight: number;
-
-	constructor(left: ExtensionList, right: InspectorPanel, maxHeight: number) {
-		this.leftPane = left;
-		this.rightPane = right;
-		this.maxHeight = maxHeight;
-	}
+	constructor(
+		private readonly leftPane: ExtensionList,
+		private readonly rightPane: InspectorPanel,
+		private readonly maxHeight: number,
+	) {}
 
 	render(width: number): string[] {
 		const leftWidth = Math.floor(width * 0.5);

package/src/modes/components/extensions/extension-list.ts

@@ -31,19 +31,19 @@ type ListItem =
 	| { type: "extension"; item: Extension };
 
 export class ExtensionList implements Component {
-	private extensions: Extension[] = [];
 	private listItems: ListItem[] = [];
 	private selectedIndex = 0;
 	private scrollOffset = 0;
 	private searchQuery = "";
 	private focused = false;
-	private callbacks: ExtensionListCallbacks;
 	private masterSwitchProvider: string | null = null;
 	private maxVisible: number;
 
-	constructor(extensions: Extension[], callbacks: ExtensionListCallbacks = {}, maxVisible?: number) {
-		this.extensions = extensions;
-		this.callbacks = callbacks;
+	constructor(
+		private extensions: Extension[],
+		private readonly callbacks: ExtensionListCallbacks = {},
+		maxVisible?: number,
+	) {
 		this.masterSwitchProvider = callbacks.masterSwitchProvider ?? null;
 		this.maxVisible = maxVisible ?? DEFAULT_MAX_VISIBLE;
 		this.rebuildList();

package/src/modes/components/footer.ts

@@ -42,16 +42,13 @@ async function findGitHeadPath(): Promise<{ path: string; content: string } | nu
  * Footer component that shows pwd, token stats, and context usage
  */
 export class FooterComponent implements Component {
-	private session: AgentSession;
 	private cachedBranch: string | null | undefined = undefined; // undefined = not checked yet, null = not in git repo, string = branch name
 	private gitWatcher: fs.FSWatcher | null = null;
 	private onBranchChange: (() => void) | null = null;
 	private autoCompactEnabled: boolean = true;
 	private extensionStatuses: Map<string, string> = new Map();
 
-	constructor(session: AgentSession) {
-		this.session = session;
-	}
+	constructor(private readonly session: AgentSession) {}
 
 	setAutoCompactEnabled(enabled: boolean): void {
 		this.autoCompactEnabled = enabled;