pi-smart-voice-notify 0.5.0 → 0.5.2

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## [0.5.2] - 2026-06-01
+
+### Changed
+- Deferred notification service initialization until first use to reduce startup work.
+- Replaced shared agent-directory lookup with a local `PI_CODING_AGENT_DIR`-aware resolver for config and permission-forwarding paths.
+- Widened peer dependency ranges to `^0.74.0 || ^0.75.0 || ^0.77.0 || ^0.78.0`.
+
+### Fixed
+- Avoid rewriting the config file when normalized content is unchanged.
+
+## [0.5.1] - 2026-05-26
+
+### Changed
+- Suppressed error notifications when the agent has pending continuation messages.
+- Widened peer dependency ranges to `^0.74.0 || ^0.75.0`.
+- Aligned `@types/node` dev dependency to `25.9.1`.
+
 ## [0.5.0] - 2026-05-22
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-smart-voice-notify",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Windows-optimized smart voice, sound, and desktop notifications for Pi coding agent.",
   "type": "module",
   "main": "./index.ts",
@@ -61,14 +61,14 @@
     ]
   },
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.75.4",
-    "@earendil-works/pi-tui": "^0.75.4"
+    "@earendil-works/pi-coding-agent": "^0.74.0 || ^0.75.0 || ^0.77.0 || ^0.78.0",
+    "@earendil-works/pi-tui": "^0.74.0 || ^0.75.0 || ^0.77.0 || ^0.78.0"
   },
   "dependencies": {
     "node-notifier": "^10.0.1",
     "undici": "^8.3.0"
   },
   "devDependencies": {
-    "@types/node": "20.17.57"
+    "@types/node": "25.9.1"
   }
 }

package/src/agent-dir.ts

@@ -0,0 +1,32 @@
+import { homedir } from "node:os";
+import { join } from "node:path";
+
+const PI_AGENT_DIR_ENV_VAR = "PI_CODING_AGENT_DIR";
+
+interface AgentDirEnvironment {
+	[name: string]: string | undefined;
+}
+
+function expandHomeDirectory(configuredDir: string, homeDirectory: string): string {
+	if (configuredDir === "~") {
+		return homeDirectory;
+	}
+
+	if (configuredDir.startsWith("~/") || configuredDir.startsWith("~\\")) {
+		return join(homeDirectory, configuredDir.slice(2));
+	}
+
+	return configuredDir;
+}
+
+export function resolvePiAgentDir(
+	env: AgentDirEnvironment = process.env,
+	homeDirectory = homedir(),
+): string {
+	const configuredDir = env[PI_AGENT_DIR_ENV_VAR];
+	if (!configuredDir) {
+		return join(homeDirectory, ".pi", "agent");
+	}
+
+	return expandHomeDirectory(configuredDir, homeDirectory);
+}

package/src/config-store.ts

@@ -1,7 +1,7 @@
-import { getAgentDir } from "@earendil-works/pi-coding-agent";
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { isAbsolute, join } from "node:path";
 
+import { resolvePiAgentDir } from "./agent-dir.ts";
 import type {
 	ConcreteTTSEngine,
 	MessageSet,
@@ -14,7 +14,7 @@ import type {
 
 export const EXTENSION_ID = "pi-smart-voice-notify";
 export const STATUS_KEY = "smart-voice-notify";
-export const CONFIG_DIR = join(getAgentDir(), "extensions", EXTENSION_ID);
+export const CONFIG_DIR = join(resolvePiAgentDir(), "extensions", EXTENSION_ID);
 export const CONFIG_PATH = join(CONFIG_DIR, "config.json");
 export const DEBUG_DIR = join(CONFIG_DIR, "debug");
 export const DEBUG_LOG_PATH = join(DEBUG_DIR, `${EXTENSION_ID}.log`);
@@ -991,7 +991,10 @@ export function readConfigFromDisk(): VoiceNotifyConfig {
 		const parsed = JSON.parse(raw) as unknown;
 		const normalized = normalizeConfig(parsed);
 		const validation = validateConfig(normalized);
-		writeFileSync(CONFIG_PATH, `${JSON.stringify(validation.config, null, 2)}\n`, "utf-8");
+		const serialized = `${JSON.stringify(validation.config, null, 2)}\n`;
+		if (raw !== serialized) {
+			writeFileSync(CONFIG_PATH, serialized, "utf-8");
+		}
 		const runtimeConfig = applyEnvironmentOverrides(validation.config);
 		return validateConfig(runtimeConfig).config;
 	} catch {

package/src/index.ts

@@ -6,7 +6,7 @@ import type {
 import type { SettingItem } from "@earendil-works/pi-tui";
 import { basename } from "node:path";
 
-import { initializeAIMessageService, type AIMessageConfig } from "./ai-messages.ts";
+import type { AIMessageConfig, AIMessageService } from "./ai-messages.ts";
 import {
 	BOOLEAN_VALUES,
 	clampInt,
@@ -30,25 +30,23 @@ import {
 	boolValue,
 	ensureDebugDirectory,
 } from "./config-store.ts";
-import { sendDesktopNotification } from "./desktop-notify.ts";
-import { clearFocusDetectCache, isTerminalFocused } from "./focus-detect.ts";
-import { detectLinuxSession, getIdleTime, wakeMonitor as wakeLinuxMonitor } from "./linux.ts";
+import type { FocusDetectOptions } from "./focus-detect.ts";
 import { createExtensionLogger, getErrorMessage } from "./logging.ts";
-import { AudioNotificationService } from "./notify-audio.ts";
-import { PermissionForwardingWatcher } from "./permission-forwarding-watcher.ts";
-import { clearProjectSoundCache } from "./per-project-sound.ts";
+import type { AudioNotificationService } from "./notify-audio.ts";
+import type {
+	PermissionForwardingWatcherConfig,
+	PermissionForwardingWatcherOptions,
+} from "./permission-forwarding-watcher.ts";
 import { ReminderPlaybackController } from "./reminder-playback.ts";
-import { SoundThemeService, type SoundThemeConfig } from "./sound-theme.ts";
-import { initializeTTSService } from "./tts.ts";
+import type { SoundThemeConfig, SoundThemeService } from "./sound-theme.ts";
 import type {
 	NotificationType,
 	NotifyLevel,
 	ReminderState,
 	VoiceNotifyConfig,
 } from "./types.ts";
-import type { TTSConfig, TTSService } from "./types/tts.ts";
-import { createWebhookService, type WebhookConfig } from "./webhook.ts";
-import { ZellijModal, ZellijSettingsModal } from "./zellij-modal.ts";
+import type { TTSConfig, TTSService, TTSServiceOptions } from "./types/tts.ts";
+import type { WebhookConfig, WebhookService } from "./webhook.ts";
 
 type SessionStartReason = "startup" | "reload" | "new" | "resume" | "fork";
 
@@ -167,6 +165,14 @@ function formatAgentErrorNotification(reason: string | undefined): string {
 	return `❌ Agent ended with an error: ${normalizedReason.slice(0, 160)}`;
 }
 
+function hasPendingAgentMessages(ctx: ExtensionContext): boolean {
+	try {
+		return ctx.hasPendingMessages();
+	} catch {
+		return false;
+	}
+}
+
 function statusLine(config: VoiceNotifyConfig): string | undefined {
 	if (!config.enabled) {
 		return "voice:off";
@@ -272,15 +278,21 @@ const REMINDER_EVENT_TYPE: Record<NotificationType, string> = {
 };
 
 type ReminderKey = string;
-type PermissionForwardingWatcherController = Pick<PermissionForwardingWatcher, "start" | "updateConfig" | "stop">;
+type FocusDetector = (options?: FocusDetectOptions) => Promise<boolean>;
+
+type PermissionForwardingWatcherController = {
+	start(config: PermissionForwardingWatcherConfig): void;
+	updateConfig(config: PermissionForwardingWatcherConfig): void;
+	stop(): void;
+};
 
 export interface SmartVoiceNotifyDependencies {
 	readConfigFromDisk?: typeof readConfigFromDisk;
-	initializeTTSService?: (options?: Parameters<typeof initializeTTSService>[0]) => TTSService;
+	initializeTTSService?: (options?: TTSServiceOptions) => TTSService;
 	createPermissionForwardingWatcher?: (
-		options: ConstructorParameters<typeof PermissionForwardingWatcher>[0],
+		options: PermissionForwardingWatcherOptions,
 	) => PermissionForwardingWatcherController;
-	isTerminalFocused?: typeof isTerminalFocused;
+	isTerminalFocused?: FocusDetector;
 }
 
 function defaultReminderKey(type: NotificationType): ReminderKey {
@@ -367,12 +379,7 @@ export default function smartVoiceNotifyExtension(
 	dependencies: SmartVoiceNotifyDependencies = {},
 ): void {
 	const readConfig = dependencies.readConfigFromDisk ?? readConfigFromDisk;
-	const createTTSService = dependencies.initializeTTSService ?? initializeTTSService;
-	const createPermissionForwardingWatcher =
-		dependencies.createPermissionForwardingWatcher ??
-		((options: ConstructorParameters<typeof PermissionForwardingWatcher>[0]): PermissionForwardingWatcherController => {
-			return new PermissionForwardingWatcher(options);
-		});
+	const createInjectedTTSService = dependencies.initializeTTSService;
 	let config = readConfig();
 	let lastUserActivityAt = Date.now();
 	let hadErrorInTurn = false;
@@ -408,13 +415,16 @@ export default function smartVoiceNotifyExtension(
 		ensureDebugDirectory,
 	});
 
-	const audioService = new AudioNotificationService({
-		execRunner: pi,
-		getConfig: () => config,
-		debug: logger.debug,
-	});
+	let audioService: AudioNotificationService | null = null;
+	let ttsService: TTSService | null = null;
+	let aiMessageService: AIMessageService | null = null;
+	let webhookService: WebhookService | null = null;
+	let soundThemeService: SoundThemeService | null = null;
+	let permissionForwardingWatcher: PermissionForwardingWatcherController | null = null;
+	let focusModulePromise: Promise<typeof import("./focus-detect.ts")> | null = null;
+
 	const projectName = basename(process.cwd()) || "project";
-	const detectTerminalFocus = dependencies.isTerminalFocused ?? isTerminalFocused;
+	const injectedTerminalFocusDetector = dependencies.isTerminalFocused;
 	const focusDetectionEnabled = envBoolean(
 		process.platform === "linux" || process.platform === "win32",
 		"PI_SMART_NOTIFY_FOCUS_DETECTION",
@@ -425,11 +435,6 @@ export default function smartVoiceNotifyExtension(
 		0,
 		envInteger(10_000, "PI_SMART_NOTIFY_AGENT_ERROR_GRACE_MS"),
 	);
-	const soundThemeService = new SoundThemeService({
-		debugLog: (message) => {
-			logger.debug("sound_theme.debug", { message });
-		},
-	});
 	const buildTTSServiceConfig = (): TTSConfig => {
 		return {
 			enableTts: config.enableTts,
@@ -533,26 +538,96 @@ export default function smartVoiceNotifyExtension(
 			customSoundDirectories: configuredCustomSoundDirectories.length > 0 ? configuredCustomSoundDirectories : undefined,
 		};
 	};
-	let ttsService = createTTSService({
-		execRunner: pi,
-		config: buildTTSServiceConfig(),
-		debug: logger.debug,
-	});
-	const aiMessageService = initializeAIMessageService({
-		config: buildAIMessageConfig(),
-		debugLog: (message, details = {}) => {
-			logger.debug(`ai_messages.${message}`, details);
-		},
-	});
-	const webhookService = createWebhookService(buildWebhookConfig());
-	const linuxSession = detectLinuxSession();
-	logger.debug("linux.session.detected", {
-		sessionType: linuxSession.sessionType,
-		display: linuxSession.display,
-		waylandDisplay: linuxSession.waylandDisplay,
-	});
+	const getAudioService = async (): Promise<AudioNotificationService> => {
+		if (!audioService) {
+			const { AudioNotificationService: AudioNotificationServiceCtor } = await import("./notify-audio.ts");
+			audioService = new AudioNotificationServiceCtor({
+				execRunner: pi,
+				getConfig: () => config,
+				debug: logger.debug,
+			});
+		}
+		return audioService;
+	};
+
+	const getTTSService = async (): Promise<TTSService> => {
+		if (!ttsService) {
+			const createTTSService = createInjectedTTSService
+				?? (await import("./tts.ts")).initializeTTSService;
+			ttsService = createTTSService({
+				execRunner: pi,
+				config: buildTTSServiceConfig(),
+				debug: logger.debug,
+			});
+		}
+		return ttsService;
+	};
+
+	const getAIMessageService = async (): Promise<AIMessageService> => {
+		if (!aiMessageService) {
+			const { initializeAIMessageService } = await import("./ai-messages.ts");
+			aiMessageService = initializeAIMessageService({
+				config: buildAIMessageConfig(),
+				debugLog: (message, details = {}) => {
+					logger.debug(`ai_messages.${message}`, details);
+				},
+			});
+		}
+		return aiMessageService;
+	};
+
+	const getWebhookService = async (): Promise<WebhookService> => {
+		if (!webhookService) {
+			const { createWebhookService } = await import("./webhook.ts");
+			webhookService = createWebhookService(buildWebhookConfig());
+		}
+		return webhookService;
+	};
+
+	const getSoundThemeService = async (): Promise<SoundThemeService> => {
+		if (!soundThemeService) {
+			const { SoundThemeService: SoundThemeServiceCtor } = await import("./sound-theme.ts");
+			soundThemeService = new SoundThemeServiceCtor({
+				debugLog: (message) => {
+					logger.debug("sound_theme.debug", { message });
+				},
+			});
+		}
+		return soundThemeService;
+	};
+
+	const getTerminalFocusDetector = async (): Promise<FocusDetector> => {
+		if (injectedTerminalFocusDetector) {
+			return injectedTerminalFocusDetector;
+		}
+		focusModulePromise ??= import("./focus-detect.ts");
+		const module = await focusModulePromise;
+		return module.isTerminalFocused;
+	};
 
-	const permissionForwardingWatcher = createPermissionForwardingWatcher({
+	const clearFocusDetectCacheIfLoaded = (): void => {
+		if (!focusModulePromise) {
+			return;
+		}
+		void focusModulePromise
+			.then((module) => module.clearFocusDetectCache())
+			.catch((error) => {
+				logger.debug("focus.cache_clear_failed", { error: getErrorMessage(error) });
+			});
+	};
+
+	const clearProjectSoundCacheIfLoaded = (): void => {
+		if (!soundThemeService) {
+			return;
+		}
+		void import("./per-project-sound.ts")
+			.then((module) => module.clearProjectSoundCache())
+			.catch((error) => {
+				logger.debug("sound_theme.cache_clear_failed", { error: getErrorMessage(error) });
+			});
+	};
+
+	const permissionForwardingWatcherOptions: PermissionForwardingWatcherOptions = {
 		onRequest: (event) => {
 			if (!config.enabled || !config.enablePermissionNotification || !config.enableForwardedPermissionWatcher) {
 				return;
@@ -609,28 +684,52 @@ export default function smartVoiceNotifyExtension(
 		debugLog: (event, details = {}) => {
 			logger.debug(event, details);
 		},
-	});
+	};
 
-	const syncPermissionForwardingWatcher = (): void => {
+	if (dependencies.createPermissionForwardingWatcher) {
+		permissionForwardingWatcher = dependencies.createPermissionForwardingWatcher(permissionForwardingWatcherOptions);
+	}
+
+	const getPermissionForwardingWatcher = async (): Promise<PermissionForwardingWatcherController> => {
+		if (!permissionForwardingWatcher) {
+			const { PermissionForwardingWatcher } = await import("./permission-forwarding-watcher.ts");
+			permissionForwardingWatcher = new PermissionForwardingWatcher(permissionForwardingWatcherOptions);
+		}
+		return permissionForwardingWatcher;
+	};
+
+	const syncPermissionForwardingWatcher = async (): Promise<void> => {
 		if (!activeSessionContext) {
-			permissionForwardingWatcher.stop();
+			permissionForwardingWatcher?.stop();
 			return;
 		}
-		permissionForwardingWatcher.start({
+
+		const watcherConfig: PermissionForwardingWatcherConfig = {
 			enabled: config.enabled && config.enablePermissionNotification && config.enableForwardedPermissionWatcher,
 			watchLegacyPath: config.watchLegacyForwardedPermissionPath,
 			targetSessionId: getPermissionForwardingSessionId(activeSessionContext),
-		});
+		};
+
+		if (!watcherConfig.enabled && !permissionForwardingWatcher) {
+			return;
+		}
+
+		const watcher = watcherConfig.enabled
+			? await getPermissionForwardingWatcher()
+			: permissionForwardingWatcher;
+		watcher?.start(watcherConfig);
 	};
 
 	const refreshIntegratedServiceConfig = (): void => {
-		ttsService = createTTSService({
-			execRunner: pi,
-			config: buildTTSServiceConfig(),
-			debug: logger.debug,
-		});
-		aiMessageService.updateConfig(buildAIMessageConfig());
-		webhookService.updateConfig(buildWebhookConfig());
+		ttsService = createInjectedTTSService
+			? createInjectedTTSService({
+				execRunner: pi,
+				config: buildTTSServiceConfig(),
+				debug: logger.debug,
+			})
+			: null;
+		aiMessageService?.updateConfig(buildAIMessageConfig());
+		webhookService?.updateConfig(buildWebhookConfig());
 	};
 
 	const rememberScopedToolCallId = (toolCallId: string, seenToolCallIds: Set<string>): boolean => {
@@ -789,6 +888,7 @@ export default function smartVoiceNotifyExtension(
 		}
 
 		try {
+			const detectTerminalFocus = await getTerminalFocusDetector();
 			const focused = await detectTerminalFocus({
 				debug: config.debugLog,
 				cacheTtlMs: config.focusCacheTtlMs,
@@ -828,22 +928,25 @@ export default function smartVoiceNotifyExtension(
 		}
 
 		const eventType = options.isReminder ? REMINDER_EVENT_TYPE[type] : type;
-		try {
-			const generated = await aiMessageService.generateMessage(eventType, {
-				projectName,
-				time: new Date().toLocaleTimeString([], { hour: "2-digit", minute: "2-digit" }),
-				count: options.followUpCount,
-				reason: options.reason,
-			});
-			if (generated.trim().length > 0) {
-				return generated;
+		if (config.aiMessages.enabled) {
+			try {
+				const service = await getAIMessageService();
+				const generated = await service.generateMessage(eventType, {
+					projectName,
+					time: new Date().toLocaleTimeString([], { hour: "2-digit", minute: "2-digit" }),
+					count: options.followUpCount,
+					reason: options.reason,
+				});
+				if (generated.trim().length > 0) {
+					return generated;
+				}
+			} catch (error) {
+				logger.debug("message.generate.error", {
+					type,
+					eventType,
+					error: getErrorMessage(error),
+				});
 			}
-		} catch (error) {
-			logger.debug("message.generate.error", {
-				type,
-				eventType,
-				error: getErrorMessage(error),
-			});
 		}
 
 		return pickRandom(MESSAGE_LIBRARY[type][options.isReminder ? "reminder" : "initial"]);
@@ -855,6 +958,7 @@ export default function smartVoiceNotifyExtension(
 		}
 
 		if (process.platform === "linux") {
+			const { getIdleTime, wakeMonitor: wakeLinuxMonitor } = await import("./linux.ts");
 			const idleSeconds = await getIdleTime({
 				debugLog: (message) => {
 					logger.debug("linux.idle", { message });
@@ -876,7 +980,8 @@ export default function smartVoiceNotifyExtension(
 			return;
 		}
 
-		await audioService.wakeMonitor();
+		const service = await getAudioService();
+		await service.wakeMonitor();
 	};
 
 	const playNotificationSound = async (type: NotificationType): Promise<boolean> => {
@@ -886,7 +991,8 @@ export default function smartVoiceNotifyExtension(
 
 		if (process.platform === "linux") {
 			try {
-				const played = await soundThemeService.playEventSound(type, buildSoundThemeConfig(), SOUND_LOOPS[type]);
+				const service = await getSoundThemeService();
+				const played = await service.playEventSound(type, buildSoundThemeConfig(), SOUND_LOOPS[type]);
 				if (played) {
 					return true;
 				}
@@ -899,7 +1005,8 @@ export default function smartVoiceNotifyExtension(
 		}
 
 		try {
-			await audioService.playWindowsSound(type);
+			const service = await getAudioService();
+			await service.playWindowsSound(type);
 			return isWindows();
 		} catch (error) {
 			logger.debug("sound.play.legacy_failed", {
@@ -915,7 +1022,8 @@ export default function smartVoiceNotifyExtension(
 			return false;
 		}
 
-		const spoken = await ttsService.speak(message, config.ttsEngine, {
+		const service = await getTTSService();
+		const spoken = await service.speak(message, config.ttsEngine, {
 			signal,
 			sapiVoice: config.sapiVoice,
 			sapiRate: config.sapiRate,
@@ -926,7 +1034,8 @@ export default function smartVoiceNotifyExtension(
 
 		if (isWindows()) {
 			try {
-				await audioService.speakWithSapi(message, signal);
+				const audio = await getAudioService();
+				await audio.speakWithSapi(message, signal);
 				return !signal?.aborted;
 			} catch (error) {
 				logger.debug("tts.sapi_fallback_failed", {
@@ -944,6 +1053,7 @@ export default function smartVoiceNotifyExtension(
 		}
 
 		try {
+			const { sendDesktopNotification } = await import("./desktop-notify.ts");
 			const result = await sendDesktopNotification({
 				type,
 				message,
@@ -977,9 +1087,14 @@ export default function smartVoiceNotifyExtension(
 		}
 	};
 
-	const dispatchWebhook = (type: NotificationType, message: string): void => {
+	const dispatchWebhook = async (type: NotificationType, message: string): Promise<void> => {
+		if (!config.webhook.enabled) {
+			return;
+		}
+
 		try {
-			const dispatchResult = webhookService.dispatch({
+			const service = await getWebhookService();
+			const dispatchResult = service.dispatch({
 				type,
 				title: `Pi Notification - ${type}`,
 				message,
@@ -1286,7 +1401,7 @@ export default function smartVoiceNotifyExtension(
 					}
 
 					if (!shutdownRequested) {
-						dispatchWebhook(type, spokenMessage);
+						await dispatchWebhook(type, spokenMessage);
 					}
 				});
 				if (!shutdownRequested && options.scheduleReminder !== false) {
@@ -1312,6 +1427,14 @@ export default function smartVoiceNotifyExtension(
 		if (!config.enabled || !config.enableErrorNotification) {
 			return;
 		}
+		if (hasPendingAgentMessages(ctx)) {
+			logger.debug("agent.error_notification.skipped", {
+				reason: "pending_messages",
+				errorReason: outcome.reason,
+				stage: "schedule",
+			});
+			return;
+		}
 
 		const timeoutId = setTimeout(() => {
 			if (pendingAgentErrorNotification !== timeoutId) {
@@ -1326,6 +1449,15 @@ export default function smartVoiceNotifyExtension(
 				return;
 			}
 
+			if (hasPendingAgentMessages(ctx)) {
+				logger.debug("agent.error_notification.skipped", {
+					reason: "pending_messages",
+					errorReason: outcome.reason,
+					stage: "fire",
+				});
+				return;
+			}
+
 			logger.debug("agent.error_notification.fired", {
 				reason: outcome.reason,
 				graceMs: agentErrorNotificationGraceMs,
@@ -1594,6 +1726,7 @@ export default function smartVoiceNotifyExtension(
 		const overlayOptions = { anchor: "center" as const, width: 92, maxHeight: "85%" as const, margin: 1 };
 		const advancedConfigPath = CONFIG_PATH;
 		const description = `Recommended settings only.\nFor advanced settings, manually edit: ${advancedConfigPath}`;
+		const { ZellijModal, ZellijSettingsModal } = await import("./zellij-modal.ts");
 
 		await ctx.ui.custom<void>(
 			(tui, theme, _keybindings, done) => {
@@ -1607,7 +1740,7 @@ export default function smartVoiceNotifyExtension(
 							applySetting(draft, id, newValue);
 							config = normalizeConfig(draft);
 							refreshIntegratedServiceConfig();
-							syncPermissionForwardingWatcher();
+							void syncPermissionForwardingWatcher();
 							if (config.debugLog && !previousConfig.debugLog) {
 								logger.debug("debug.enabled", { debugLogPath: DEBUG_LOG_PATH });
 							}
@@ -1683,7 +1816,7 @@ export default function smartVoiceNotifyExtension(
 		if (subcommand === "reload") {
 			config = readConfig();
 			refreshIntegratedServiceConfig();
-			syncPermissionForwardingWatcher();
+			await syncPermissionForwardingWatcher();
 			refreshQuestionToolAvailability();
 			cancelReminderActivity("command_reload");
 			updateStatus(ctx);
@@ -1694,7 +1827,7 @@ export default function smartVoiceNotifyExtension(
 		if (subcommand === "on" || subcommand === "off") {
 			config.enabled = subcommand === "on";
 			refreshIntegratedServiceConfig();
-			syncPermissionForwardingWatcher();
+			await syncPermissionForwardingWatcher();
 			persistConfig(ctx);
 			if (!config.enabled) {
 				cancelReminderActivity("command_disabled");
@@ -1734,9 +1867,9 @@ export default function smartVoiceNotifyExtension(
 	pi.on("resources_discover", (event, _ctx) => {
 		if (event.reason === "reload") {
 			// Clear caches on reload
-			clearFocusDetectCache();
-			clearProjectSoundCache();
-			aiMessageService.clearCache();
+			clearFocusDetectCacheIfLoaded();
+			clearProjectSoundCacheIfLoaded();
+			aiMessageService?.clearCache();
 		}
 	});
 
@@ -1750,14 +1883,14 @@ export default function smartVoiceNotifyExtension(
 		config = readConfig();
 		refreshIntegratedServiceConfig();
 		activeSessionContext = ctx;
-		syncPermissionForwardingWatcher();
+		await syncPermissionForwardingWatcher();
 		refreshQuestionToolAvailability();
-		clearFocusDetectCache();
-		clearProjectSoundCache();
+		clearFocusDetectCacheIfLoaded();
+		clearProjectSoundCacheIfLoaded();
 
 		// Clear AI message cache on reload to pick up config changes
 		if (reason === "reload") {
-			aiMessageService.clearCache();
+			aiMessageService?.clearCache();
 		}
 		lastUserActivityAt = Date.now();
 		hadErrorInTurn = false;
@@ -1804,17 +1937,17 @@ export default function smartVoiceNotifyExtension(
 		shutdownPromise = (async () => {
 			logger.debug("session.shutdown", {});
 			activeSessionContext = null;
-			permissionForwardingWatcher.stop();
+			permissionForwardingWatcher?.stop();
 			pendingPermissionToolCallIds.clear();
 			processedToolResultToolCallIds.clear();
 			resetPermissionBatch("session_shutdown");
 			cancelReminderActivity("session_shutdown");
-			clearFocusDetectCache();
-			clearProjectSoundCache();
+			clearFocusDetectCacheIfLoaded();
+			clearProjectSoundCacheIfLoaded();
 			try {
 				// Forward abort signal to flush if available (added in pi-coding-agent 0.67.x)
 				const abortSignal = 'signal' in ctx ? (ctx as { signal?: AbortSignal }).signal : undefined;
-				await webhookService.flush(abortSignal);
+				await webhookService?.flush(abortSignal);
 			} catch (error) {
 				const message = `Failed to flush webhook queue during shutdown: ${getErrorMessage(error)}`;
 				logger.error(new Error(message));

package/src/permission-forwarding-watcher.ts

@@ -1,14 +1,14 @@
-import { getAgentDir } from "@earendil-works/pi-coding-agent";
 import { existsSync, readdirSync, readFileSync, watch, type FSWatcher } from "node:fs";
 import { basename, join } from "node:path";
 
+import { resolvePiAgentDir } from "./agent-dir.ts";
 import { toRecord } from "./config-store.ts";
 import { getErrorMessage } from "./logging.ts";
 
 export type PermissionForwardingSource = "primary" | "legacy";
 export type ForwardedPermissionResolutionReason = "request_removed" | "watch_disabled" | "watcher_stopped";
 
-const AGENT_DIR = getAgentDir();
+const AGENT_DIR = resolvePiAgentDir();
 const PERMISSION_FORWARDING_ROOT_DIR = join(AGENT_DIR, "sessions", "permission-forwarding");
 const SESSION_FORWARDING_ROOT_DIRECTORY_NAME = "sessions";
 const SESSION_FORWARDING_REQUESTS_DIRECTORY_NAME = "requests";
@@ -60,7 +60,7 @@ export interface PermissionForwardingWatcherConfig {
 	targetSessionId?: string | null;
 }
 
-interface PermissionForwardingWatcherOptions {
+export interface PermissionForwardingWatcherOptions {
 	onRequest: (event: ForwardedPermissionRequestEvent) => void;
 	onResolve: (event: ForwardedPermissionResolutionEvent) => void;
 	debugLog: (event: string, details?: Record<string, unknown>) => void;