@pellux/goodvibes-agent 1.4.4 → 1.5.1

package/src/runtime/bootstrap.ts

@@ -69,10 +69,10 @@ const GOODVIBES_AGENT_OPERATOR_POLICY = [
   '- Act as one user-facing autonomous assistant. Prefer the lowest-friction safe path that completes the user outcome; do not expose internal package or host ownership unless it is needed for diagnosis, setup, or safety.',
   '- Work serially in the main conversation by default for ordinary chat, research, planning, setup, local context, and short tool work. Use visible schedules, work plans, operator actions, or delegated/remote routes for durable or long-running autonomy.',
   '- Connected-host lifecycle is not ambient. If the host is unavailable, explain the shortest user action to make the assistant reachable; do not pretend a missing host route worked.',
-  '- Read tools: `route action:"plan|status"`, `schedule action:"list"`, `setup action:"status|item|repair|checkpoint"`, `settings action:"list|get"`, `vibe action:"status|show"`, `context action:"status|files|file|prompt|receipts|receipt"`, `memory action:"status|provider|curator|candidate|list|search|get"`, `channels action:"status|channel|setup|triage|deliveries"`, `models action:"status|route|local|providers|provider"`, `personal_ops action:"briefing|status|queue|intake|lane"`, `autonomy action:"intake|queue|item|status"`, `delegation action:"status|routes|route"`, `execution action:"status|route|history|record|processes|process_capabilities|process|recovery"`, `security action:"status|finding|explain"`, `support action:"status|bundle"`, `sessions action:"list|get"`, `audit action:"readiness|item|evidence|artifact"`, `computer action:"status|plan|control|browser|setup|mcp"`, `research action:"plan|search|runner|runs|run|sources|source|bundle|reports|report_artifact"`, `device action:"status|capability|browser|control|voice|provider"`, `workspace action:"status|actions|action|surfaces|surface|panels|panel|shortcuts|keybindings|keybinding|commands|command|cli_commands|cli_command"`, `host action:"status|capabilities|capability|services|service|methods|method"`, `import_goodvibes_settings action:"preview"`, and `agent_operator_briefing` for connected work/approvals/automation/schedules, `agent_knowledge` for isolated Agent Knowledge, and `agent_harness` for harness catalogs/status/capability discovery. Use `agent_artifacts` for saved artifact list/preview/export/package/archive; write modes are confirmation-gated.',
-  '- Harness access: use `settings action:"set|reset|import"` and `workspace action:"run|run_command|open|open_panel|run_keybinding|set_keybinding|reset_keybinding"` to use the same surfaces the user can use; lower-level `agent_harness` modes remain for detail inspection and compatibility.',
+  '- Read tools: `route action:"plan|status"`, `schedule action:"list"`, `setup action:"status|item|repair|checkpoint"`, `settings action:"list|get"`, `vibe action:"status|show"`, `context action:"status|files|file|prompt|receipts|receipt"`, `memory action:"status|provider|curator|candidate|list|search|get"`, `channels action:"status|channel|setup|triage|deliveries"`, `models action:"status|route|local|providers|provider"`, `personal_ops action:"briefing|status|queue|intake|lane"`, `autonomy action:"intake|queue|item|status"`, `delegation action:"status|routes|route"`, `execution action:"status|route|history|record|processes|process_capabilities|process|recovery"`, `security action:"status|finding|explain"`, `support action:"status|bundle"`, `sessions action:"list|get"`, `audit action:"readiness|item|evidence|artifact"`, `computer action:"status|plan|control|browser|setup|mcp"`, `research action:"plan|search|runner|runs|run|sources|source|bundle|reports|report_artifact"`, `device action:"status|capability|browser|control|voice|provider"`, `workspace action:"status|actions|action|surfaces|surface|shortcuts|keybindings|keybinding|commands|command|cli_commands|cli_command"`, `host action:"status|capabilities|capability|services|service|methods|method"`, `import_goodvibes_settings action:"preview"`, and `agent_operator_briefing` for connected work/approvals/automation/schedules, `agent_knowledge` for isolated Agent Knowledge, and `agent_harness` for harness catalogs/status/capability discovery. Use `agent_artifacts` for saved artifact list/preview/export/package/archive; write modes are confirmation-gated.',
+  '- Harness access: use `settings action:"set|reset|import"` and `workspace action:"run|run_command|open|run_keybinding|set_keybinding|reset_keybinding"` to use the same surfaces the user can use; lower-level `agent_harness` modes remain for detail inspection and compatibility.',
   '- State tools: `agent_work_plan` for visible local work items; `agent_local_registry` for Agent-local notes, memory, personas, skills, bundles, and routines; `agent_learning_consolidation` for confirmed duplicate cleanup phases; `agent_documents` for versioned Agent document drafts. Keep records non-secret, sourced, and reviewable.',
-  '- Confirmed tools: use `schedule`, `setup action:"save_checkpoint|clear_checkpoint|token|smoke|finish|import_settings"`, `vibe action:"init|import_persona"`, `models action:"smoke"` for local model server checks, `personal_ops action:"read"` for one live read-only inbox/calendar connector operation, `research action:"create_run|start_run|checkpoint|pause|resume|cancel|complete|fail|delete_run|add_source|review_source|reject_source|use_source|delete_source|report"`, `computer action:"open_browser"`, `device action:"open_browser|open_tts_provider|open_tts_voice"`, `workspace action:"run|run_command|open|open_panel|run_keybinding|set_keybinding|reset_keybinding"`, `import_goodvibes_settings`, `agent_operator_action`, `agent_artifacts` export/package/archive, `agent_documents`, `agent_review_packet_presets` save/refresh, `agent_review_packet_share`, `agent_knowledge_ingest`, `agent_learning_consolidation`, `agent_media_generate`, `agent_model_compare`, `agent_research_runs`, `agent_research_sources`, `agent_research_report`, `agent_notify`, `agent_channel_send`, `agent_autonomy_schedule`, `agent_reminder_schedule`, and `agent_schedule_edit` only for explicit user requests with confirm:true and explicitUserRequest.',
+  '- Confirmed tools: use `schedule`, `setup action:"save_checkpoint|clear_checkpoint|token|smoke|finish|import_settings"`, `vibe action:"init|import_persona"`, `models action:"smoke"` for local model server checks, `personal_ops action:"read"` for one live read-only inbox/calendar connector operation, `research action:"create_run|start_run|checkpoint|pause|resume|cancel|complete|fail|delete_run|add_source|review_source|reject_source|use_source|delete_source|report"`, `computer action:"open_browser"`, `device action:"open_browser|open_tts_provider|open_tts_voice"`, `workspace action:"run|run_command|open|run_keybinding|set_keybinding|reset_keybinding"`, `import_goodvibes_settings`, `agent_operator_action`, `agent_artifacts` export/package/archive, `agent_documents`, `agent_review_packet_presets` save/refresh, `agent_review_packet_share`, `agent_knowledge_ingest`, `agent_learning_consolidation`, `agent_media_generate`, `agent_model_compare`, `agent_research_runs`, `agent_research_sources`, `agent_research_report`, `agent_notify`, `agent_channel_send`, `agent_autonomy_schedule`, `agent_reminder_schedule`, and `agent_schedule_edit` only for explicit user requests with confirm:true and explicitUserRequest.',
   '- Agent Knowledge must use only `/api/goodvibes-agent/knowledge/*` and fail closed. Do not use default knowledge or non-Agent knowledge spaces.',
   '- External delivery, media generation, reminders, settings writes, slash-command mirrors, workspace action mirrors, and destructive local changes require explicit user intent and the owning tool/command confirmation.',
   '- Autonomous work must be visible, reviewable, and cancellable. Never create silent hidden jobs; when work should continue later, create or use an explicit schedule, reminder, work-plan item, operator action, or delegated/remote task route.',
@@ -237,7 +237,6 @@ export async function bootstrapRuntime(
   const {
     automationManager,
     hookDispatcher,
-    panelManager,
     pluginManager,
   } = services;
 

package/src/runtime/context.ts

@@ -19,7 +19,7 @@ import type { FeatureFlagManager } from '@/runtime/index.ts';
 import type { MutableRuntimeState } from '@/runtime/index.ts';
 import type { SessionSnapshot } from '@/runtime/index.ts';
 import type { RuntimeServices } from './services.ts';
-import type { ComponentHealthMonitor } from './perf/panel-health-monitor.ts';
+import type { ComponentHealthMonitor } from './perf/component-health.ts';
 import type { WorktreeRegistry } from '@/runtime/index.ts';
 import type { SandboxSessionRegistry } from '@/runtime/index.ts';
 

package/src/runtime/diagnostics/panels/index.ts

@@ -19,6 +19,5 @@ export { TransportPanel } from '@/runtime/index.ts';
 export type { TransportPanelSnapshot } from '@/runtime/index.ts';
 export { OpsPanel } from './ops.ts';
 export type { OpsAuditEntry } from './ops.ts';
-export { PanelResourcesPanel } from './panel-resources.ts';
 export { SecurityPanel } from '@/runtime/index.ts';
 export type { SecurityPanelSnapshot } from '@/runtime/index.ts';

package/src/runtime/diagnostics/panels/policy.ts

@@ -155,7 +155,7 @@ export class PolicyDiagnosticsPanel {
    */
   public subscribe(callback: () => void): () => void {
     this._subscribers.add(callback);
-    return () => this._subscribers.delete(callback);
+    return () => { this._subscribers.delete(callback); };
   }
 
   /**

package/src/runtime/execution-ledger.ts

@@ -98,6 +98,21 @@ function stringArg(args: Record<string, unknown>, keys: readonly string[]): stri
   return undefined;
 }
 
+/**
+ * Classify a tool call into a route kind for display and filtering.
+ *
+ * Keyword precedence (first match wins):
+ *   1. browser   — browser/desktop/screenshot keywords
+ *   2. delegation — delegate/agent keywords
+ *   3. shell     — exec/shell/bash keywords OR args.command is a string
+ *   4. write     — write/edit/patch/delete keywords
+ *   5. network   — fetch/web/http/url keywords
+ *   6. read      — read/find/inspect/grep/search keywords
+ *   7. other     — fallback for unrecognized tool names
+ *
+ * N2: precedence is intentional — shell beats write/read so that bash-exec tools
+ * that also match 'exec' are not misclassified as filesystem writes.
+ */
 function routeKindForTool(tool: string, args: Record<string, unknown>): AgentExecutionRouteKind {
   const name = tool.toLowerCase();
   if (name.includes('browser') || name.includes('desktop') || name.includes('screenshot')) return 'browser';
@@ -162,7 +177,7 @@ export class AgentExecutionLedger {
 
   public subscribe(callback: () => void): () => void {
     this.subscribers.add(callback);
-    return () => this.subscribers.delete(callback);
+    return () => { this.subscribers.delete(callback); };
   }
 
   public dispose(): void {

package/src/runtime/index.ts

@@ -325,6 +325,11 @@ export const getDistributedNodeHostContract = operations.getDistributedNodeHostC
 export const CURRENT_PROTOCOL_VERSION = operations.CURRENT_PROTOCOL_VERSION;
 export const VersionMismatchError = operations.VersionMismatchError;
 export const negotiateProtocolVersion = operations.negotiateProtocolVersion;
+// Protocol version types — re-exported for transport compatibility tests.
+// ProtocolVersion, VersionNegotiationResult, NegotiatedProtocol are available
+// via operations namespace (operations.ProtocolVersion etc.) but cannot be
+// re-exported here without a registered subpath. Tests that need these types
+// should import from the operations namespace types directly.
 export const createTaskManager = operations.createTaskManager;
 export const PhasedToolExecutor = operations.PhasedToolExecutor;
 export const budgetPhase = operations.budgetPhase;
@@ -614,7 +619,7 @@ export type BundleProvenance = Security.BundleProvenance;
 export type DecisionReason = Security.DecisionReason;
 export type DivergenceReport = Security.DivergenceReport;
 export type EnforceGateResult = Security.EnforceGateResult;
-export type SignedPolicyBundle = Security.SignedPolicyBundle;
+export type SignedPolicyBundle<T = unknown> = Security.SignedPolicyBundle<T>;
 
 export interface InspectableDomain {
   readonly name: string;

package/src/runtime/onboarding/index.ts

@@ -5,3 +5,4 @@ export * from './apply.ts';
 export * from './verify.ts';
 export * from './markers.ts';
 export * from './state.ts';
+export * from './onboarding-state.ts';

package/src/runtime/onboarding/onboarding-state.ts

@@ -0,0 +1,200 @@
+import type { SetupPlanItem, SetupPlanStatus } from '../../tools/agent-harness-setup-posture-types.ts';
+import type { OnboardingCheckMarkerState } from './types.ts';
+
+/**
+ * Maps each setup plan item id to the Agent Workspace category that resolves it.
+ * This is the single sequencing table — every onboarding surface reads from here.
+ */
+const PLAN_ITEM_CATEGORY_MAP: Readonly<Record<string, string>> = {
+  'connected-host-readiness': 'setup',
+  'connected-host-auth': 'setup',
+  'goodvibes-settings-import': 'setup',
+  'provider-access': 'account-model',
+  'install-smoke': 'setup',
+  'local-model-readiness': 'account-model',
+  'agent-knowledge': 'setup',
+  'vibe-personality': 'assistant-behavior',
+  'local-behavior': 'onboarding-context',
+  'communication-channels': 'onboarding-channels',
+  'automation-review': 'tools-permissions',
+  'browser-desktop-control': 'tools-permissions',
+  'sudo-execution-posture': 'tools-permissions',
+  'build-delegation': 'tools-permissions',
+  'finish-onboarding': 'setup',
+};
+
+const DEFAULT_CATEGORY_ID = 'setup';
+
+function planItemCategoryId(planItemId: string): string {
+  return PLAN_ITEM_CATEGORY_MAP[planItemId] ?? DEFAULT_CATEGORY_ID;
+}
+
+export interface OnboardingStep {
+  readonly id: string;
+  readonly label: string;
+  readonly status: SetupPlanStatus;
+  readonly blocksAutonomy: boolean;
+  /** Human-readable hint shown in the UI for the next action. */
+  readonly nextLabel: string;
+  /** Agent Workspace category id that resolves this step. */
+  readonly categoryId: string;
+}
+
+export interface OnboardingRecap {
+  /** Single headline sentence describing overall readiness. */
+  readonly headline: string;
+  /** One line per ready capability plus optional trailing lines. */
+  readonly lines: readonly string[];
+}
+
+export type OnboardingPhase = 'fresh' | 'in-progress' | 'complete';
+
+export interface OnboardingState {
+  /**
+   * Phase derived from markers:
+   * - 'complete'     completion marker present
+   * - 'in-progress'  check marker present but no completion marker
+   * - 'fresh'        neither marker present
+   */
+  readonly phase: OnboardingPhase;
+  /** All setup plan items converted to lightweight steps. */
+  readonly steps: readonly OnboardingStep[];
+  /** First non-ready step that blocks autonomy, or first non-ready step if none, or null. */
+  readonly currentStepId: string | null;
+  /** Human-readable progress, e.g. "2 of 5 ready". */
+  readonly progressLabel: string;
+  /** Steps that are not ready and block autonomy. */
+  readonly blockers: readonly OnboardingStep[];
+  /**
+   * True when a usable model route exists:
+   * the provider-access plan item is 'ready', or local-model-readiness is 'ready'.
+   */
+  readonly readyToChat: boolean;
+  /** Summary shown at the end of onboarding or as a re-entry summary. */
+  readonly recap: OnboardingRecap;
+}
+
+export interface DeriveOnboardingStateContext {
+  /** Fully-built setup plan (from buildSetupPlan). */
+  readonly plan: readonly SetupPlanItem[];
+  /** User-scope check marker (from readOnboardingCheckMarker). */
+  readonly checkMarker: OnboardingCheckMarkerState;
+  /** User-scope completion marker (from readOnboardingCompletionMarker). */
+  readonly completionMarker: OnboardingCheckMarkerState;
+}
+
+function derivePhase(
+  checkMarker: OnboardingCheckMarkerState,
+  completionMarker: OnboardingCheckMarkerState,
+): OnboardingPhase {
+  if (completionMarker.exists && completionMarker.payload !== null) return 'complete';
+  if (checkMarker.exists) return 'in-progress';
+  return 'fresh';
+}
+
+function planItemToStep(item: SetupPlanItem): OnboardingStep {
+  return {
+    id: item.id,
+    label: item.label,
+    status: item.status,
+    blocksAutonomy: item.blocksAutonomy,
+    nextLabel: item.nextAction,
+    categoryId: planItemCategoryId(item.id),
+  };
+}
+
+function deriveCurrentStepId(steps: readonly OnboardingStep[]): string | null {
+  // Priority: first non-ready blocker, then first non-ready step overall.
+  const firstBlocker = steps.find((s) => s.status !== 'ready' && s.blocksAutonomy);
+  if (firstBlocker) return firstBlocker.id;
+  const firstNonReady = steps.find((s) => s.status !== 'ready');
+  return firstNonReady?.id ?? null;
+}
+
+function deriveProgressLabel(plan: readonly SetupPlanItem[]): string {
+  const ready = plan.filter((item) => item.status === 'ready').length;
+  const total = plan.length;
+  return `${ready} of ${total} ready`;
+}
+
+function deriveReadyToChat(plan: readonly SetupPlanItem[]): boolean {
+  const providerItem = plan.find((item) => item.id === 'provider-access');
+  if (providerItem?.status === 'ready') return true;
+  const localModelItem = plan.find((item) => item.id === 'local-model-readiness');
+  return localModelItem?.status === 'ready';
+}
+
+function deriveRecap(
+  phase: OnboardingPhase,
+  steps: readonly OnboardingStep[],
+  plan: readonly SetupPlanItem[],
+): OnboardingRecap {
+  const readySteps = steps.filter((s) => s.status === 'ready');
+  const optionalCount = plan.filter(
+    (item) => item.status !== 'ready' && !item.blocksAutonomy,
+  ).length;
+
+  const headline =
+    phase === 'complete'
+      ? "You're set up. Here's what you can do now."
+      : phase === 'in-progress'
+        ? "Setup is in progress. Here's what's ready so far."
+        : 'Getting started — a few things to set up before the full experience.';
+
+  const lines: string[] = readySteps.map((s) => s.label);
+
+  // Capability examples: show what the user can actually do right now.
+  // Always-safe examples (research/draft/summarize) are shown whenever a model route is ready.
+  // Channel-dependent examples (reminders/messaging) only appear when that lane is configured.
+  const modelReady = deriveReadyToChat(plan);
+  if (modelReady && phase !== 'fresh') {
+    const channelsReady = plan.some(
+      (item) => item.id === 'communication-channels' && item.status === 'ready',
+    );
+    const capabilityLines: string[] = [
+      'Ask me to research a topic, draft an email, summarize a file, or compare models.',
+    ];
+    if (channelsReady) {
+      capabilityLines.push('You can also set reminders and send messages through your connected channels.');
+    }
+    lines.push(...capabilityLines);
+  }
+
+  if (optionalCount > 0 && phase !== 'fresh') {
+    lines.push(`Still optional: ${optionalCount} item${optionalCount === 1 ? '' : 's'} available`);
+  }
+
+  if (phase !== 'fresh') {
+    lines.push('Try: "What can you help me with today?"');
+  }
+
+  return { headline, lines };
+}
+
+/**
+ * Pure selector: derives the full OnboardingState from an already-built plan and
+ * the current marker files. No side effects, no Date.now() calls.
+ */
+export function deriveOnboardingState(
+  context: DeriveOnboardingStateContext,
+): OnboardingState {
+  const { plan, checkMarker, completionMarker } = context;
+
+  const phase = derivePhase(checkMarker, completionMarker);
+  const steps = plan.map(planItemToStep);
+  const currentStepId = deriveCurrentStepId(steps);
+  const progressLabel = deriveProgressLabel(plan);
+  const blockers = steps.filter((s) => s.status !== 'ready' && s.blocksAutonomy);
+  const readyToChat = deriveReadyToChat(plan);
+  const recap = deriveRecap(phase, steps, plan);
+
+  return {
+    phase,
+    steps,
+    currentStepId,
+    progressLabel,
+    blockers,
+    readyToChat,
+    recap,
+  };
+}

package/src/{panels → runtime}/provider-account-snapshot.ts

@@ -1,7 +1,7 @@
 import { listBuiltinSubscriptionProviders } from '@pellux/goodvibes-sdk/platform/config';
 import type {
   ProviderAccountInspectionQuery,
-} from '../runtime/ui-service-queries.ts';
+} from './ui-service-queries.ts';
 
 export type ProviderAuthRoute = 'api-key' | 'subscription' | 'service-oauth' | 'unconfigured';
 export type ProviderAuthFreshness = 'healthy' | 'expiring' | 'expired' | 'pending' | 'unconfigured';
@@ -261,7 +261,10 @@ export async function buildProviderAccountSnapshot(
             : 'No usable auth route is configured for this provider.',
       authFreshness: freshness,
       fallbackRoute: activeRoute !== preferredRoute ? activeRoute : undefined,
-      fallbackRisk: hasSubscription && hasApiKey
+      // D7 fix: fallbackRisk is only meaningful when a fallbackRoute exists (i.e. activeRoute
+      // differs from preferredRoute). Setting it on a healthy provider where activeRoute===
+      // preferredRoute would show a spurious risk advisory with no fallback target.
+      fallbackRisk: (activeRoute !== preferredRoute) && hasSubscription && hasApiKey
         ? (isExpired(subscription?.expiresAt)
           ? 'preferred subscription path is expired; active route falls back to API key.'
           : 'Both subscription and API key are present; check route priority.')

package/src/runtime/services.ts

@@ -46,7 +46,6 @@ import { IntegrationHelperService } from '@/runtime/index.ts';
 import { VoiceProviderRegistry, VoiceService, ensureBuiltinVoiceProviders } from '@pellux/goodvibes-sdk/platform/voice';
 import { WebSearchProviderRegistry, WebSearchService } from '@pellux/goodvibes-sdk/platform/web-search';
 import { MemoryEmbeddingProviderRegistry } from '@pellux/goodvibes-sdk/platform/state';
-import { PanelManager } from '../panels/panel-manager.ts';
 import { HookActivityTracker } from '@pellux/goodvibes-sdk/platform/hooks';
 import { HookDispatcher, createHookWorkbench, type HookWorkbench } from '@pellux/goodvibes-sdk/platform/hooks';
 import { PluginManager } from '@pellux/goodvibes-sdk/platform/plugins';
@@ -388,7 +387,6 @@ export interface RuntimeServices extends SdkRuntimeServices {
   readonly runtimeBus: RuntimeEventBus;
   readonly runtimeStore: RuntimeStore;
   readonly runtimeDispatch: DomainDispatch;
-  readonly panelManager: PanelManager;
   readonly keybindingsManager: KeybindingsManager;
   readonly routeBindings: RouteBindingManager;
   readonly surfaceRegistry: SurfaceRegistry;
@@ -488,7 +486,6 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
   const featureFlags = options.featureFlags ?? createFeatureFlagManager();
   const runtimeDispatch = createDomainDispatch(options.runtimeStore);
   const gatewayMethods = new GatewayMethodCatalog();
-  const panelManager = new PanelManager();
   const keybindingsManager = new KeybindingsManager({
     configPath: shellPaths.resolveUserPath(GOODVIBES_AGENT_SURFACE_ROOT, 'keybindings.json'),
   });
@@ -738,6 +735,7 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
   const tokenAuditor = new ApiTokenAuditor({ managed: false });
   const componentHealthMonitor = new ComponentHealthMonitor();
   const worktreeRegistry = createDisabledAgentWorktreeRegistry(workingDirectory);
+  // Configured and attached to the runtime bus during bootstrap when webhook URLs are present.
   const webhookNotifier = new WebhookNotifier();
   const replayEngine = new DeterministicReplayEngine(workingDirectory);
   const providerOptimizer = new ProviderOptimizer(providerRegistry, providerCapabilityRegistry, false);
@@ -761,9 +759,24 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
   const modeManager = new ModeManager();
   const fileUndoManager = new FileUndoManager();
   const executionLedger = new AgentExecutionLedger(options.runtimeBus);
+  // The SDK's foundation/integration contracts still expect a panel manager;
+  // the Agent shell has no panel UI (the Activity sidebar replaced it), so we
+  // satisfy those contracts with a no-op implementation.
+  const NOOP_PANEL_MANAGER = (() => {
+    const emptyPane = { panels: [], activeIndex: 0 } as const;
+    return {
+      getTopPane: () => emptyPane,
+      getBottomPane: () => emptyPane,
+      getRegisteredTypes: () => [],
+      open: () => undefined,
+      show: () => {},
+    };
+  })();
+  const panelManager = NOOP_PANEL_MANAGER;
   const integrationHelpers = new IntegrationHelperService({
     workingDirectory,
     homeDirectory,
+    panelManager,
     runtimeStore: options.runtimeStore,
     runtimeBus: options.runtimeBus,
     configManager,
@@ -774,7 +787,6 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     distributedRuntime,
     remoteRunnerRegistry,
     remoteSupervisor,
-    panelManager,
     localUserAuthManager,
     providerRegistry,
     serviceRegistry,

package/src/runtime/terminal-output-guard.ts

@@ -89,6 +89,13 @@ export function allowTerminalWrite<T>(fn: () => T): T {
 }
 
 export function installTerminalOutputGuard(options: TerminalOutputGuardOptions): TerminalOutputGuard {
+  // D6 fix: if a guard already exists and hasn't been disposed, dispose it first
+  // so the monkeypatch chain is cleanly unwound BEFORE we snapshot the originals.
+  // This ensures stdout.write is the real underlying write when we capture it below.
+  if (currentGuard !== null) {
+    currentGuard.dispose();
+  }
+
   const stdout = options.stdout;
   const stderr = options.stderr ?? process.stderr;
   const originalStdoutWriteMethod = stdout.write;

package/src/runtime/tool-permission-safety.ts

@@ -71,7 +71,7 @@ const READ_ONLY_MEMORY_ACTIONS = new Set(['', 'status', 'summary', 'posture', 'm
 const READ_ONLY_COMPUTER_ACTIONS = new Set(['', 'status', 'summary', 'overview', 'computer', 'computer_use', 'plan', 'route', 'control_plan', 'browser_plan', 'desktop_plan', 'control', 'browser_control', 'desktop', 'desktop_control', 'screenshot', 'screen', 'screen_recording', 'observe', 'browser', 'pwa', 'cockpit', 'browser_cockpit', 'web', 'setup', 'configure', 'browser_desktop_control', 'mcp', 'tools', 'servers', 'mcp_servers']);
 const READ_ONLY_DEVICE_ACTIONS = new Set(['', 'status', 'map', 'capabilities', 'device', 'devices', 'mobile', 'phone', 'pairing', 'capability', 'route', 'pairing_route', 'show', 'inspect', 'browser', 'pwa', 'cockpit', 'browser_cockpit', 'web', 'control', 'browser_control', 'desktop', 'desktop_control', 'computer_use', 'voice', 'media', 'voice_media', 'workflows', 'provider', 'media_provider', 'voice_provider']);
 const READ_ONLY_MODELS_ACTIONS = new Set(['', 'status', 'routing', 'routes', 'models', 'model', 'readiness', 'route_readiness', 'route', 'model_route', 'inspect', 'show', 'candidate', 'endpoint', 'local', 'cookbook', 'local_cookbook', 'recipes', 'recipe', 'ollama', 'llama_cpp', 'llamacpp', 'vllm', 'local_servers', 'providers', 'provider_accounts', 'accounts', 'subscriptions', 'auth', 'logins', 'provider', 'provider_account', 'account', 'subscription', 'auth_status']);
-const READ_ONLY_WORKSPACE_ACTIONS = new Set(['', 'status', 'summary', 'home', 'workspace', 'categories', 'workspace_categories', 'actions', 'list_actions', 'workspace_actions', 'tasks', 'action', 'show', 'inspect', 'workspace_action', 'surfaces', 'ui_surfaces', 'screens', 'views', 'surface', 'ui_surface', 'screen', 'view', 'panels', 'panes', 'panel', 'pane', 'shortcuts', 'shortcut_help', 'help', 'keybindings', 'bindings', 'keys', 'keybinding', 'binding', 'key', 'commands', 'slash_commands', 'command_catalog', 'command', 'slash_command', 'inspect_command', 'cli_commands', 'cli_catalog', 'cli_command', 'inspect_cli_command']);
+const READ_ONLY_WORKSPACE_ACTIONS = new Set(['', 'status', 'summary', 'home', 'workspace', 'categories', 'workspace_categories', 'actions', 'list_actions', 'workspace_actions', 'tasks', 'action', 'show', 'inspect', 'workspace_action', 'surfaces', 'ui_surfaces', 'screens', 'views', 'surface', 'ui_surface', 'screen', 'view', 'shortcuts', 'shortcut_help', 'help', 'keybindings', 'bindings', 'keys', 'keybinding', 'binding', 'key', 'commands', 'slash_commands', 'command_catalog', 'command', 'slash_command', 'inspect_command', 'cli_commands', 'cli_catalog', 'cli_command', 'inspect_cli_command']);
 const READ_ONLY_AUTONOMY_ACTIONS = new Set(['', 'intake', 'request', 'route', 'plan', 'triage', 'autonomy_intake', 'queue', 'list', 'work', 'ongoing', 'autonomy_queue', 'item', 'card', 'show', 'inspect', 'autonomy_queue_item', 'status', 'summary', 'overview']);
 const READ_ONLY_DELEGATION_ACTIONS = new Set(['', 'status', 'summary', 'overview', 'policy', 'decision', 'decisions', 'delegation_posture', 'routes', 'list', 'catalog', 'posture', 'route', 'item', 'card', 'show', 'inspect', 'delegation_route']);
 const READ_ONLY_EXECUTION_ACTIONS = new Set(['', 'status', 'summary', 'overview', 'routes', 'posture', 'execution_posture', 'route', 'show_route', 'inspect_route', 'execution_route', 'history', 'activity', 'records', 'execution_history', 'record', 'item', 'show', 'inspect', 'execution_history_item', 'processes', 'background', 'backgrounds', 'background_processes', 'capabilities', 'process_capabilities', 'process', 'background_process', 'recovery', 'file_recovery', 'undo_redo']);

package/src/runtime/ui/model-picker/data-provider.ts

@@ -84,7 +84,7 @@ export class ModelPickerDataProvider {
    */
   public subscribe(callback: () => void): () => void {
     this._subscribers.add(callback);
-    return () => this._subscribers.delete(callback);
+    return () => { this._subscribers.delete(callback); };
   }
 
   /**

package/src/runtime/ui/model-picker/health-enrichment.ts

@@ -18,7 +18,7 @@ import type {
   ProviderHealthContext,
   CapabilityFlags,
   ProviderLatencyStats,
-} from '@/runtime/index.ts';
+} from './types.ts';
 
 /** Status sort priority (lower = shown first). */
 const STATUS_ORDER: Record<string, number> = {
@@ -46,7 +46,7 @@ function buildHealthContext(record: ProviderHealthRecord | undefined): ProviderH
   if (record.stats.totalCalls > 0) {
     latency = {
       avgMs: record.stats.avgLatencyMs,
-      p95Ms: record.stats.maxLatencyMs,
+      maxMs: record.stats.maxLatencyMs,
       minMs: record.stats.minLatencyMs,
     };
   }

package/src/runtime/ui/model-picker/types.ts

@@ -32,8 +32,8 @@ export interface CapabilityFlags {
 export interface ProviderLatencyStats {
   /** Moving average latency in ms (last N calls). */
   readonly avgMs: number;
-  /** Approximate 95th-percentile latency in ms (max of recent observations). */
-  readonly p95Ms: number;
+  /** Maximum observed latency in ms (max of recent observations). */
+  readonly maxMs: number;
   /** Minimum observed latency in ms. */
   readonly minMs: number;
 }

package/src/runtime/ui/provider-health/data-provider.ts

@@ -15,7 +15,7 @@ import type {
   ProviderHealthEntry,
   HealthTimeline,
   HealthTimelinePoint,
-} from '@/runtime/index.ts';
+} from './types.ts';
 
 /**
  * Number of timeline points retained per provider.
@@ -87,7 +87,7 @@ export class ProviderHealthDataProvider {
    */
   public subscribe(callback: () => void): () => void {
     this._subscribers.add(callback);
-    return () => this._subscribers.delete(callback);
+    return () => { this._subscribers.delete(callback); };
   }
 
   /**
@@ -196,7 +196,7 @@ export class ProviderHealthDataProvider {
         isConfigured: record.isConfigured,
         successRate,
         errorRate,
-        p95LatencyMs: record.stats.maxLatencyMs,
+        maxLatencyMs: record.stats.maxLatencyMs,
         avgLatencyMs: record.stats.avgLatencyMs,
         totalCalls: total,
         // cacheHitRate is populated only when cache-capability is wired to the provider record.

package/src/runtime/ui/provider-health/types.ts

@@ -59,8 +59,8 @@ export interface ProviderHealthEntry {
   readonly successRate: number;
   /** Error rate (0–1), derived from totalCalls and errorCalls. */
   readonly errorRate: number;
-  /** Approximate p95 latency in ms (max of recent observations). */
-  readonly p95LatencyMs: number;
+  /** Maximum observed latency in ms (max of recent observations). */
+  readonly maxLatencyMs: number;
   /** Moving average latency in ms. */
   readonly avgLatencyMs: number;
   /** Total API calls recorded. */

package/src/runtime/ui-services.ts

@@ -19,7 +19,6 @@ export interface UiEnvironmentServices {
 
 export interface UiShellServices {
   readonly keybindingsManager: RuntimeServices['keybindingsManager'];
-  readonly panelManager: RuntimeServices['panelManager'];
   readonly processManager: RuntimeServices['processManager'];
   readonly profileManager: RuntimeServices['profileManager'];
   readonly bookmarkManager: RuntimeServices['bookmarkManager'];
@@ -135,7 +134,6 @@ export function createUiRuntimeServices(
     },
     shell: {
       keybindingsManager: runtimeServices.keybindingsManager,
-      panelManager: runtimeServices.panelManager,
       processManager: runtimeServices.processManager,
       profileManager: runtimeServices.profileManager,
       bookmarkManager: runtimeServices.bookmarkManager,

package/src/shell/agent-workspace-fullscreen.ts

@@ -28,7 +28,6 @@ export function createFullscreenCompositeFromLines(
     viewport: normalizeFullscreenViewport(lines, width, height),
     footer: [],
     forceFullRedraw: true,
-    panelWidth: 0,
   };
 }