@adhdev/daemon-core 1.0.18 → 1.0.20

package/dist/providers/auto-approve-modes.d.ts

@@ -10,5 +10,5 @@ export declare function deriveAutoApproveModeRisk(mode: Pick<AutoApproveMode, 'r
  * Resolve new mode settings before the legacy boolean. A stale/unknown explicit
  * mode id fails closed instead of falling through to an enabled legacy setting.
  */
-export declare function resolveProviderAutoApproveMode(provider: ProviderModule, settings: Record<string, unknown> | undefined): ResolvedAutoApproveMode;
+export declare function resolveProviderAutoApproveMode(provider: ProviderModule | null | undefined, settings: Record<string, unknown> | undefined): ResolvedAutoApproveMode;
 export declare function findProviderAutoApproveMode(provider: ProviderModule | undefined, modeId: string): AutoApproveMode | undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adhdev/daemon-core",
-  "version": "1.0.18",
+  "version": "1.0.20",
   "description": "ADHDev daemon core — CDP, IDE detection, providers, command execution",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/cli-adapters/provider-cli-shared.ts

@@ -286,7 +286,14 @@ export function isPurePtyTranscriptProvider(provider: {
     transcriptAuthority?: 'provider' | 'daemon';
     nativeHistory?: unknown;
     tui?: Record<string, unknown>;
-}): boolean {
+} | null | undefined): boolean {
+    // No provider (e.g. an instance whose module isn't set when the stall watchdog
+    // ticks) ⇒ not the pure-PTY class. Guarding here protects every caller — the
+    // stall-reconcile and turn-start paths pass this.provider unguarded, and
+    // this.provider is genuinely nullable at runtime (see this.provider?.nativeHistory
+    // in cli-provider-instance.ts). Matches resolveLocalSessionPurePty's own
+    // no-provider ⇒ not-pure-PTY contract.
+    if (!provider) return false;
     if (provider.transcriptAuthority === 'provider') return false;
     // nativeHistory is a top-level provider field not surfaced on
     // CliProviderModule; read it via the same structural shape the adapter uses.

package/src/providers/auto-approve-modes.ts

@@ -61,9 +61,15 @@ function resolveConfiguredMode(
  * mode id fails closed instead of falling through to an enabled legacy setting.
  */
 export function resolveProviderAutoApproveMode(
-  provider: ProviderModule,
+  provider: ProviderModule | null | undefined,
   settings: Record<string, unknown> | undefined,
 ): ResolvedAutoApproveMode {
+  // No provider (e.g. an instance whose module isn't set when a modal-park /
+  // auto-approve classification runs) ⇒ auto-approve is inactive. this.provider is
+  // genuinely nullable at runtime (the instance reads this.provider?.nativeHistory
+  // elsewhere), and the sibling helper findProviderAutoApproveMode already guards
+  // with provider?.autoApproveModes. Guard the entry so every caller is safe.
+  if (!provider) return inactiveMode();
   const config = provider.autoApproveModes;
   const explicitModeId = settings?.autoApproveMode;
   if (typeof explicitModeId === 'string') {