fix: gate dual-mode triage skip on emit success

Made-with: Cursor

Author: preethamr

packages/utils/src/helpers/alerts.ts

@@ -91,6 +91,7 @@ export async function sendAlerts(
   const methodContext = createMethodContext(sendAlerts.name);
   const mode = getPipelineMode();
   const { emitterConfig, secretEmpty } = getEmitterConfig(config);
+  let eventEmissionSucceeded = false;
 
   if (secretEmpty && emitterConfig) {
     logger.warn('Event emitter webhook secret is empty — HMAC signatures will provide no authentication', requestContext, methodContext);
@@ -100,6 +101,7 @@ export async function sendAlerts(
   if (mode !== 'legacy' && emitterConfig) {
     try {
       await emitEvent(report, emitterConfig, logger, requestContext);
+      eventEmissionSucceeded = true;
     } catch (emitErr) {
       logger.error('Event emission failed; continuing with legacy path', requestContext, methodContext, {
         type: 'EventEmissionError',
@@ -123,7 +125,6 @@ export async function sendAlerts(
   // In dual mode, when event emission succeeded the everclear-agents pipeline
   // handles triage. Skip the monorepo triage interceptor to avoid double LLM
   // calls and potentially contradictory verdicts.
-  const eventEmissionSucceeded = mode === 'dual' && emitterConfig != null;
   const skipTriage = mode === 'dual' && eventEmissionSucceeded;
 
   const triageOutput = skipTriage