@semiont/react-ui 0.2.33 → 0.2.34-build.89

package/src/components/resource/panels/__tests__/TaggingPanel.test.tsx

@@ -6,7 +6,7 @@ import userEvent from '@testing-library/user-event';
 import '@testing-library/jest-dom';
 import { TaggingPanel } from '../TaggingPanel';
 import { EventBusProvider, resetEventBusForTesting, useEventBus } from '../../../../contexts/EventBusContext';
-import type { components } from '@semiont/api-client';
+import type { components } from '@semiont/core';
 
 type Annotation = components['schemas']['Annotation'];
 
@@ -29,16 +29,16 @@ function createEventTracker() {
         events.push({ event: eventName, payload });
       };
 
-      const panelEvents = ['annotation:create', 'detection:start'];
+      const panelEvents = ['annotation:create', 'detection:start'] as const;
 
       panelEvents.forEach(eventName => {
         const handler = trackEvent(eventName);
-        eventBus.on(eventName, handler);
-        handlers.push(() => eventBus.off(eventName, handler));
+        const subscription = eventBus.get(eventName).subscribe(handler);
+        handlers.push(subscription);
       });
 
       return () => {
-        handlers.forEach(cleanup => cleanup());
+        handlers.forEach(sub => sub.unsubscribe());
       };
     }, [eventBus]);
 

package/src/components/settings/SettingsPanel.tsx

@@ -38,7 +38,7 @@ export function SettingsPanel({
   const handleLocaleChange = (newLocale: string) => {
     const localeName = LOCALES.find(l => l.code === newLocale)?.nativeName || newLocale;
     announceLanguageChanging(localeName);
-    eventBus.emit('settings:locale-changed', { locale: newLocale });
+    eventBus.get('settings:locale-changed').next({ locale: newLocale });
   };
 
   // Announce when language has successfully changed
@@ -67,7 +67,7 @@ export function SettingsPanel({
               type="button"
               role="switch"
               aria-checked={showLineNumbers}
-              onClick={() => eventBus.emit('settings:line-numbers-toggled', undefined)}
+              onClick={() => eventBus.get('settings:line-numbers-toggled').next(undefined)}
               className={`semiont-toggle ${
                 showLineNumbers ? 'semiont-toggle--active' : ''
               }`}
@@ -91,7 +91,7 @@ export function SettingsPanel({
           </label>
           <div className="semiont-settings-panel__button-group">
             <button
-              onClick={() => eventBus.emit('settings:theme-changed', { theme: 'light' })}
+              onClick={() => eventBus.get('settings:theme-changed').next({ theme: 'light' })}
               className={`semiont-panel-button ${
                 theme === 'light' ? 'semiont-panel-button-active' : ''
               }`}
@@ -100,7 +100,7 @@ export function SettingsPanel({
               ☀️ {t('themeLight')}
             </button>
             <button
-              onClick={() => eventBus.emit('settings:theme-changed', { theme: 'dark' })}
+              onClick={() => eventBus.get('settings:theme-changed').next({ theme: 'dark' })}
               className={`semiont-panel-button ${
                 theme === 'dark' ? 'semiont-panel-button-active' : ''
               }`}
@@ -109,7 +109,7 @@ export function SettingsPanel({
               🌙 {t('themeDark')}
             </button>
             <button
-              onClick={() => eventBus.emit('settings:theme-changed', { theme: 'system' })}
+              onClick={() => eventBus.get('settings:theme-changed').next({ theme: 'system' })}
               className={`semiont-panel-button ${
                 theme === 'system' ? 'semiont-panel-button-active' : ''
               }`}

package/src/components/viewers/ImageViewer.tsx

@@ -1,4 +1,4 @@
-import type { ResourceUri } from '@semiont/api-client';
+import type { ResourceUri } from '@semiont/core';
 
 interface ImageViewerProps {
   resourceUri: ResourceUri;

package/src/features/resource-compose/components/ResourceComposePage.tsx

@@ -7,7 +7,7 @@
  */
 
 import React, { useState, useEffect } from 'react';
-import type { components } from '@semiont/api-client';
+import type { components } from '@semiont/core';
 import { isImageMimeType, isPdfMimeType, LOCALES } from '@semiont/api-client';
 import { buttonStyles } from '../../../lib/button-styles';
 import { CodeMirrorRenderer } from '../../../components/CodeMirrorRenderer';

package/src/features/resource-discovery/components/ResourceCard.tsx

@@ -6,7 +6,7 @@
  */
 
 import React from 'react';
-import type { components } from '@semiont/api-client';
+import type { components } from '@semiont/core';
 
 type ResourceDescriptor = components['schemas']['ResourceDescriptor'];
 

package/src/features/resource-discovery/components/ResourceDiscoveryPage.tsx

@@ -6,7 +6,7 @@
  */
 
 import React, { useState, useCallback, useRef } from 'react';
-import type { components } from '@semiont/api-client';
+import type { components } from '@semiont/core';
 import { getResourceId } from '@semiont/api-client';
 import { useRovingTabIndex } from '../../../hooks/useRovingTabIndex';
 import { Toolbar } from '../../../components/Toolbar';

package/src/features/resource-viewer/__tests__/AnnotationCreationPending.test.tsx

@@ -25,10 +25,10 @@ import { EventBusProvider, useEventBus, resetEventBusForTesting } from '../../..
 import { ApiClientProvider } from '../../../contexts/ApiClientContext';
 import { AuthTokenProvider } from '../../../contexts/AuthTokenContext';
 import { SemiontApiClient } from '@semiont/api-client';
-import { resourceUri } from '@semiont/api-client';
+import { resourceUri } from '@semiont/core';
 import type { Emitter } from 'mitt';
-import type { EventMap } from '../../../contexts/EventBusContext';
-import type { Motivation, Selector } from '@semiont/api-client';
+import type { EventMap } from '@semiont/core';
+import type { Motivation, Selector } from '@semiont/core';
 
 const TEST_URI = resourceUri('http://localhost:4000/resources/test-resource');
 
@@ -85,7 +85,7 @@ function renderDetectionFlow(testUri: string) {
   return {
     getEventBus: () => eventBusInstance,
     emit: <K extends keyof EventMap>(event: K, payload: EventMap[K]) => {
-      eventBusInstance.emit(event, payload);
+      eventBusInstance.get(event).next(payload);
     },
   };
 }
@@ -239,7 +239,7 @@ describe('Annotation creation clears pendingAnnotation', () => {
     const createdListener = vi.fn();
     // Set listener after first render so eventBus is captured
     await waitFor(() => expect(getEventBus()).toBeDefined());
-    getEventBus().on('annotation:created', createdListener);
+    const subscription = getEventBus().get('annotation:created').subscribe(createdListener);
 
     act(() => {
       emit('annotation:requested', { selector: TEXT_SELECTOR, motivation: 'linking' });
@@ -257,6 +257,8 @@ describe('Annotation creation clears pendingAnnotation', () => {
       expect(createdListener).toHaveBeenCalledTimes(1);
       expect(createdListener).toHaveBeenCalledWith({ annotation: MOCK_ANNOTATION });
     });
+
+    subscription.unsubscribe();
   });
 
   it('does NOT clear pendingAnnotation if API call fails', async () => {

package/src/features/resource-viewer/__tests__/AnnotationDeletionIntegration.test.tsx

@@ -30,7 +30,8 @@ import { useDetectionFlow } from '../../../hooks/useDetectionFlow';
 import { EventBusProvider, useEventBus, resetEventBusForTesting } from '../../../contexts/EventBusContext';
 import { ApiClientProvider } from '../../../contexts/ApiClientContext';
 import { AuthTokenProvider } from '../../../contexts/AuthTokenContext';
-import { SemiontApiClient, resourceUri, accessToken } from '@semiont/api-client';
+import { SemiontApiClient } from '@semiont/api-client';
+import { resourceUri, accessToken } from '@semiont/core';
 
 describe('Annotation Deletion - Feature Integration', () => {
   let deleteAnnotationSpy: ReturnType<typeof vi.fn>;
@@ -78,7 +79,7 @@ describe('Annotation Deletion - Feature Integration', () => {
     return {
       emitDelete: (annotationId: string) => {
         act(() => {
-          eventBusInstance!.emit('annotation:delete', { annotationId });
+          eventBusInstance!.get('annotation:delete').next({ annotationId });
         });
       },
       eventBus: eventBusInstance!,
@@ -127,7 +128,7 @@ describe('Annotation Deletion - Feature Integration', () => {
     const deletedListener = vi.fn();
 
     // Subscribe to success event
-    eventBus.on('annotation:deleted', deletedListener);
+    eventBus.get('annotation:deleted').subscribe(deletedListener);
 
     emitDelete('annotation-789');
 
@@ -152,7 +153,7 @@ describe('Annotation Deletion - Feature Integration', () => {
     const failedListener = vi.fn();
 
     // Subscribe to failure event
-    eventBus.on('annotation:delete-failed', failedListener);
+    eventBus.get('annotation:delete-failed').subscribe(failedListener);
 
     emitDelete('annotation-error');
 

package/src/features/resource-viewer/__tests__/DetectionFlowBug.test.tsx

@@ -76,7 +76,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     // Emit detection:start event (exactly like production)
     act(() => {
-      eventBusInstance.emit('detection:start', {
+      eventBusInstance.get('detection:start').next({
         motivation: 'linking',
         options: { entityTypes: ['Location'] }
       });
@@ -124,7 +124,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     // Emit detection:progress event (exactly like production)
     act(() => {
-      eventBusInstance.emit('detection:progress', {
+      eventBusInstance.get('detection:progress').next({
         status: 'started',
         resourceId: 'test',
         totalEntityTypes: 1,
@@ -183,7 +183,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
     // Exactly like production logs
     act(() => {
       console.log('[EventBus] emit: detection:start {motivation: "linking", options: {...}}');
-      eventBusInstance.emit('detection:start', {
+      eventBusInstance.get('detection:start').next({
         motivation: 'linking',
         options: { entityTypes: ['Location'] }
       });
@@ -193,7 +193,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     act(() => {
       console.log('[EventBus] emit: detection:progress {status: "started", ...}');
-      eventBusInstance.emit('detection:progress', {
+      eventBusInstance.get('detection:progress').next({
         status: 'started',
         resourceId: 'f45fd44f9cb0b0fe1b7980d3d034bc61',
         totalEntityTypes: 1,
@@ -206,7 +206,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     act(() => {
       console.log('[EventBus] emit: detection:progress {status: "scanning", ...}');
-      eventBusInstance.emit('detection:progress', {
+      eventBusInstance.get('detection:progress').next({
         status: 'scanning',
         resourceId: 'f45fd44f9cb0b0fe1b7980d3d034bc61',
         currentEntityType: 'Location',

package/src/features/resource-viewer/__tests__/DetectionFlowIntegration.test.tsx

@@ -30,32 +30,16 @@ import { EventBusProvider, useEventBus, resetEventBusForTesting } from '../../..
 import { ApiClientProvider } from '../../../contexts/ApiClientContext';
 import { AuthTokenProvider } from '../../../contexts/AuthTokenContext';
 import { SSEClient } from '@semiont/api-client';
-import type { Motivation } from '@semiont/api-client';
-import { resourceUri } from '@semiont/api-client';
+import type { Motivation } from '@semiont/core';
+import { resourceUri } from '@semiont/core';
 import type { Emitter } from 'mitt';
-import type { EventMap } from '../../../contexts/EventBusContext';
+import type { EventMap } from '@semiont/core';
 
-// Mock SSE stream that we can control in tests
+// Mock SSE stream - SSE now emits directly to EventBus, no callbacks
 const createMockSSEStream = () => {
-  const stream = {
-    onProgressCallback: null as ((chunk: any) => void) | null,
-    onCompleteCallback: null as (() => void) | null,
-    onErrorCallback: null as ((error: Error) => void) | null,
-    onProgress: vi.fn((callback: (chunk: any) => void) => {
-      stream.onProgressCallback = callback;
-      return stream;
-    }),
-    onComplete: vi.fn((callback: () => void) => {
-      stream.onCompleteCallback = callback;
-      return stream;
-    }),
-    onError: vi.fn((callback: (error: Error) => void) => {
-      stream.onErrorCallback = callback;
-      return stream;
-    }),
+  return {
     close: vi.fn(),
   };
-  return stream;
 };
 
 describe('Detection Flow - Feature Integration', () => {
@@ -102,14 +86,14 @@ describe('Detection Flow - Feature Integration', () => {
       expect(detectReferencesSpy).toHaveBeenCalledTimes(1);
     });
 
-    // Verify correct parameters
+    // Verify correct parameters (eventBus is passed but we don't need to verify its exact value)
     expect(detectReferencesSpy).toHaveBeenCalledWith(
       testUri,
       {
         entityTypes: ['Person', 'Organization'],
         includeDescriptiveReferences: false,
       },
-      { auth: undefined }
+      expect.objectContaining({ auth: undefined })
     );
   });
 
@@ -117,7 +101,7 @@ describe('Detection Flow - Feature Integration', () => {
     const testUri = resourceUri('http://localhost:4000/resources/test-resource');
 
     // Render with state observer
-    const { emitDetectionStart } = renderDetectionFlow(testUri);
+    const { emitDetectionStart, getEventBus } = renderDetectionFlow(testUri);
 
     // Start detection
     act(() => {
@@ -131,9 +115,9 @@ describe('Detection Flow - Feature Integration', () => {
       expect(detectReferencesSpy).toHaveBeenCalled();
     });
 
-    // Simulate SSE progress callback being invoked
+    // Simulate SSE progress event being emitted to EventBus (how SSE actually works now)
     act(() => {
-      mockStream.onProgressCallback!({
+      getEventBus().get('detection:progress').next({
         status: 'scanning',
         message: 'Scanning for Person...',
         currentEntityType: 'Person',
@@ -152,7 +136,7 @@ describe('Detection Flow - Feature Integration', () => {
 
   it('should handle multiple progress updates correctly', async () => {
     const testUri = resourceUri('http://localhost:4000/resources/test-resource');
-    const { emitDetectionStart } = renderDetectionFlow(testUri);
+    const { emitDetectionStart, getEventBus } = renderDetectionFlow(testUri);
 
     // Start detection
     act(() => {
@@ -165,9 +149,9 @@ describe('Detection Flow - Feature Integration', () => {
       expect(detectHighlightsSpy).toHaveBeenCalledTimes(1);
     });
 
-    // First progress update
+    // First progress update via EventBus
     act(() => {
-      mockStream.onProgressCallback!({
+      getEventBus().get('detection:progress').next({
         status: 'started',
         message: 'Starting analysis...',
         percentage: 0,
@@ -178,9 +162,9 @@ describe('Detection Flow - Feature Integration', () => {
       expect(screen.getByTestId('progress')).toHaveTextContent('Starting analysis...');
     });
 
-    // Second progress update
+    // Second progress update via EventBus
     act(() => {
-      mockStream.onProgressCallback!({
+      getEventBus().get('detection:progress').next({
         status: 'analyzing',
         message: 'Analyzing text...',
         percentage: 50,
@@ -191,9 +175,9 @@ describe('Detection Flow - Feature Integration', () => {
       expect(screen.getByTestId('progress')).toHaveTextContent('Analyzing text...');
     });
 
-    // Final progress update
+    // Final progress update via EventBus
     act(() => {
-      mockStream.onProgressCallback!({
+      getEventBus().get('detection:progress').next({
         status: 'complete',
         message: 'Created 14 highlights',
         percentage: 100,
@@ -218,9 +202,9 @@ describe('Detection Flow - Feature Integration', () => {
       expect(screen.getByTestId('detecting')).toHaveTextContent('highlighting');
     });
 
-    // Send final progress
+    // Send final progress via EventBus
     act(() => {
-      mockStream.onProgressCallback!({
+      getEventBus().get('detection:progress').next({
         status: 'complete',
         message: 'Created 14 highlights',
       });
@@ -232,7 +216,7 @@ describe('Detection Flow - Feature Integration', () => {
 
     // Emit completion event
     act(() => {
-      getEventBus().emit('detection:complete', { motivation: 'highlighting' });
+      getEventBus().get('detection:complete').next({ motivation: 'highlighting' });
     });
 
     // Verify: detecting flag cleared BUT progress still visible
@@ -251,9 +235,9 @@ describe('Detection Flow - Feature Integration', () => {
       emitDetectionStart('linking', { entityTypes: ['Person'] });
     });
 
-    // Add some progress
+    // Add some progress via EventBus
     act(() => {
-      mockStream.onProgressCallback!({
+      getEventBus().get('detection:progress').next({
         status: 'scanning',
         message: 'Scanning...',
       });
@@ -265,7 +249,7 @@ describe('Detection Flow - Feature Integration', () => {
 
     // Emit failure
     act(() => {
-      getEventBus().emit('detection:failed', { type: 'job.failed', resourceId: 'test-resource' as any, payload: { jobId: 'job-1' as any, jobType: 'detection', error: 'Network error' } });
+      getEventBus().get('detection:failed').next({ type: 'job.failed', resourceId: 'test-resource' as any, payload: { jobId: 'job-1' as any, jobType: 'detection', error: 'Network error' } });
     });
 
     // Verify: both detecting and progress cleared
@@ -288,7 +272,7 @@ describe('Detection Flow - Feature Integration', () => {
       expect(detectHighlightsSpy).toHaveBeenCalledTimes(1);
       expect(detectHighlightsSpy).toHaveBeenCalledWith(testUri, {
         instructions: 'Find important text',
-      }, { auth: undefined });
+      }, expect.objectContaining({ auth: undefined }));
     });
 
     // Reset for next test
@@ -309,7 +293,7 @@ describe('Detection Flow - Feature Integration', () => {
       expect(detectCommentsSpy).toHaveBeenCalledWith(testUri, {
         instructions: 'Add helpful comments',
         tone: 'educational',
-      }, { auth: undefined });
+      }, expect.objectContaining({ auth: undefined }));
     });
   });
 
@@ -322,7 +306,7 @@ describe('Detection Flow - Feature Integration', () => {
 
     // Add an additional event listener (simulating multiple subscribers)
     const additionalListener = vi.fn();
-    getEventBus().on('detection:start', additionalListener);
+    const subscription = getEventBus().get('detection:start').subscribe(additionalListener);
 
     // Trigger detection
     act(() => {
@@ -339,6 +323,8 @@ describe('Detection Flow - Feature Integration', () => {
 
     // VERIFY: Our additional listener was called (events work)
     expect(additionalListener).toHaveBeenCalledTimes(1);
+
+    subscription.unsubscribe();
   });
 });
 
@@ -381,7 +367,7 @@ function renderDetectionFlow(testUri: string) {
 
   return {
     emitDetectionStart: (motivation: Motivation, options: any) => {
-      eventBusInstance.emit('detection:start', { motivation, options });
+      eventBusInstance.get('detection:start').next({ motivation, options });
     },
     getEventBus: () => eventBusInstance,
   };

package/src/features/resource-viewer/__tests__/DetectionProgressDismissal.test.tsx

@@ -25,7 +25,8 @@ import { useDetectionFlow } from '../../../hooks/useDetectionFlow';
 import { EventBusProvider, resetEventBusForTesting, useEventBus } from '../../../contexts/EventBusContext';
 import { ApiClientProvider } from '../../../contexts/ApiClientContext';
 import { AuthTokenProvider } from '../../../contexts/AuthTokenContext';
-import { SSEClient, resourceUri } from '@semiont/api-client';
+import { SSEClient } from '@semiont/api-client';
+import { resourceUri } from '@semiont/core';
 
 describe('Detection Progress Dismissal Bug', () => {
   let mockStream: any;
@@ -36,9 +37,6 @@ describe('Detection Progress Dismissal Bug', () => {
     vi.clearAllMocks();
 
     mockStream = {
-      onProgress: vi.fn().mockReturnThis(),
-      onComplete: vi.fn().mockReturnThis(),
-      onError: vi.fn().mockReturnThis(),
       close: vi.fn(),
     };
 
@@ -85,7 +83,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // User clicks detect button (emits detection:start)
     act(() => {
-      eventBusInstance.emit('detection:start', {
+      eventBusInstance.get('detection:start').next({
         motivation: 'linking',
         options: { entityTypes: ['Location'] }
       });
@@ -98,7 +96,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // SSE sends progress update
     act(() => {
-      eventBusInstance.emit('detection:progress', {
+      eventBusInstance.get('detection:progress').next({
         status: 'scanning',
         message: 'Processing: Location',
         currentEntityType: 'Location',
@@ -112,7 +110,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // Detection completes (SSE finishes, backend emits detection:complete)
     act(() => {
-      eventBusInstance.emit('detection:complete', { motivation: 'linking' });
+      eventBusInstance.get('detection:complete').next({ motivation: 'linking' });
     });
 
     // detectingMotivation cleared immediately
@@ -151,15 +149,15 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // First detection with stuck progress
     act(() => {
-      eventBusInstance.emit('detection:start', { motivation: 'linking', options: {} });
+      eventBusInstance.get('detection:start').next({ motivation: 'linking', options: {} });
     });
 
     act(() => {
-      eventBusInstance.emit('detection:progress', { message: 'Old progress stuck here' });
+      eventBusInstance.get('detection:progress').next({ message: 'Old progress stuck here' });
     });
 
     act(() => {
-      eventBusInstance.emit('detection:complete', { motivation: 'linking' });
+      eventBusInstance.get('detection:complete').next({ motivation: 'linking' });
     });
 
     await waitFor(() => {
@@ -168,7 +166,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // WORKAROUND: Start new detection clears old progress
     act(() => {
-      eventBusInstance.emit('detection:start', { motivation: 'highlighting', options: {} });
+      eventBusInstance.get('detection:start').next({ motivation: 'highlighting', options: {} });
     });
 
     await waitFor(() => {
@@ -204,18 +202,18 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // Show progress
     act(() => {
-      eventBusInstance.emit('detection:start', { motivation: 'linking', options: {} });
+      eventBusInstance.get('detection:start').next({ motivation: 'linking', options: {} });
     });
 
     act(() => {
-      eventBusInstance.emit('detection:progress', {
+      eventBusInstance.get('detection:progress').next({
         status: 'complete',
         message: 'Complete! Created 5 annotations'
       });
     });
 
     act(() => {
-      eventBusInstance.emit('detection:complete', { motivation: 'linking' });
+      eventBusInstance.get('detection:complete').next({ motivation: 'linking' });
     });
 
     // Progress visible initially
@@ -231,12 +229,10 @@ describe('Detection Progress Dismissal Bug', () => {
     });
   });
 
-  it('FIXED: useResolutionFlow now forwards final completion chunk data', async () => {
+  it('FIXED: SSE emits final completion chunk data as detection:progress', async () => {
     /**
-     * This test verifies the fix for the useResolutionFlow bug.
-     *
-     * FIX: useResolutionFlow.ts stream.onComplete(finalChunk) now emits detection:progress
-     * with the final chunk data BEFORE emitting detection:complete.
+     * This test verifies that SSE emits the final chunk as detection:progress
+     * BEFORE emitting detection:complete.
      *
      * This ensures the UI can display the final completion message with status:'complete'.
      */
@@ -255,20 +251,6 @@ describe('Detection Progress Dismissal Bug', () => {
       );
     }
 
-    // Mock SSE stream to simulate backend behavior
-    let onProgressCallback: any;
-    let onCompleteCallback: any;
-
-    mockStream.onProgress.mockImplementation((cb: any) => {
-      onProgressCallback = cb;
-      return mockStream;
-    });
-
-    mockStream.onComplete.mockImplementation((cb: any) => {
-      onCompleteCallback = cb;
-      return mockStream;
-    });
-
     render(
       <EventBusProvider>
         <AuthTokenProvider token={null}>
@@ -281,15 +263,15 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // Start detection (triggers SSE stream creation)
     act(() => {
-      eventBusInstance.emit('detection:start', {
+      eventBusInstance.get('detection:start').next({
         motivation: 'linking',
         options: { entityTypes: ['Location'] }
       });
     });
 
-    // Simulate SSE scanning chunk via stream.onProgress()
+    // Simulate SSE scanning chunk
     act(() => {
-      onProgressCallback?.({
+      eventBusInstance.get('detection:progress').next({
         status: 'scanning',
         message: 'Processing: Location',
       });
@@ -299,10 +281,9 @@ describe('Detection Progress Dismissal Bug', () => {
       expect(screen.getByTestId('progress-status')).toHaveTextContent('scanning');
     });
 
-    // Simulate backend sending final chunk to stream.onComplete(finalChunk)
-    // useResolutionFlow should forward this as detection:progress
+    // Simulate SSE emitting final chunk as detection:progress
     act(() => {
-      onCompleteCallback?.({
+      eventBusInstance.get('detection:progress').next({
         status: 'complete',
         message: 'Complete! Found 5 entities',
         foundCount: 5,