From 6f7d7981894ad34553d8f0ba04c1b099ab1451ff Mon Sep 17 00:00:00 2001 From: Vedant Mahajan Date: Sat, 10 Jan 2026 22:19:15 +0530 Subject: [PATCH] remove unused sessionHookTriggers and exports (#16324) --- packages/core/src/core/sessionHookTriggers.ts | 108 ------------------ packages/core/src/hooks/index.ts | 7 -- 2 files changed, 115 deletions(-) delete mode 100644 packages/core/src/core/sessionHookTriggers.ts diff --git a/packages/core/src/core/sessionHookTriggers.ts b/packages/core/src/core/sessionHookTriggers.ts deleted file mode 100644 index 149a84edbd..0000000000 --- a/packages/core/src/core/sessionHookTriggers.ts +++ /dev/null @@ -1,108 +0,0 @@ -/** - * @license - * Copyright 2025 Google LLC - * SPDX-License-Identifier: Apache-2.0 - */ - -import type { MessageBus } from '../confirmation-bus/message-bus.js'; -import { - MessageBusType, - type HookExecutionRequest, - type HookExecutionResponse, -} from '../confirmation-bus/types.js'; -import { - type SessionStartSource, - type SessionEndReason, - type PreCompressTrigger, - createHookOutput, - type DefaultHookOutput, -} from '../hooks/types.js'; -import { debugLogger } from '../utils/debugLogger.js'; - -/** - * Fires the SessionStart hook. - * - * @param messageBus The message bus to use for hook communication - * @param source The source/trigger of the session start - * @returns The output from the SessionStart hook, or undefined if failed/no output - */ -export async function fireSessionStartHook( - messageBus: MessageBus, - source: SessionStartSource, -): Promise { - try { - const response = await messageBus.request< - HookExecutionRequest, - HookExecutionResponse - >( - { - type: MessageBusType.HOOK_EXECUTION_REQUEST, - eventName: 'SessionStart', - input: { - source, - }, - }, - MessageBusType.HOOK_EXECUTION_RESPONSE, - ); - - if (response.output) { - return createHookOutput('SessionStart', response.output); - } - return undefined; - } catch (error) { - debugLogger.debug(`SessionStart hook failed:`, error); - return undefined; - } -} - -/** - * Fires the SessionEnd hook. - * - * @param messageBus The message bus to use for hook communication - * @param reason The reason for the session end - */ -export async function fireSessionEndHook( - messageBus: MessageBus, - reason: SessionEndReason, -): Promise { - try { - await messageBus.request( - { - type: MessageBusType.HOOK_EXECUTION_REQUEST, - eventName: 'SessionEnd', - input: { - reason, - }, - }, - MessageBusType.HOOK_EXECUTION_RESPONSE, - ); - } catch (error) { - debugLogger.debug(`SessionEnd hook failed:`, error); - } -} - -/** - * Fires the PreCompress hook. - * - * @param messageBus The message bus to use for hook communication - * @param trigger The trigger type (manual or auto) - */ -export async function firePreCompressHook( - messageBus: MessageBus, - trigger: PreCompressTrigger, -): Promise { - try { - await messageBus.request( - { - type: MessageBusType.HOOK_EXECUTION_REQUEST, - eventName: 'PreCompress', - input: { - trigger, - }, - }, - MessageBusType.HOOK_EXECUTION_RESPONSE, - ); - } catch (error) { - debugLogger.debug(`PreCompress hook failed:`, error); - } -} diff --git a/packages/core/src/hooks/index.ts b/packages/core/src/hooks/index.ts index 223ac25e42..b8e54fdc2f 100644 --- a/packages/core/src/hooks/index.ts +++ b/packages/core/src/hooks/index.ts @@ -20,10 +20,3 @@ export type { HookRegistryEntry } from './hookRegistry.js'; export { ConfigSource } from './types.js'; export type { AggregatedHookResult } from './hookAggregator.js'; export type { HookEventContext } from './hookPlanner.js'; - -// Export hook trigger functions -export { - fireSessionStartHook, - fireSessionEndHook, - firePreCompressHook, -} from '../core/sessionHookTriggers.js';