@semiont/react-ui 0.2.33-build.80 → 0.2.33-build.81

package/dist/test-utils.d.mts

@@ -2,8 +2,8 @@ import { ReactElement } from 'react';
 import { RenderOptions, RenderResult } from '@testing-library/react';
 export * from '@testing-library/react';
 import { QueryClient } from '@tanstack/react-query';
-import { T as TranslationManager, S as SessionManager, O as OpenResourcesManager, E as EventBus } from './EventBusContext-7GvDyO0d.mjs';
-export { r as resetEventBusForTesting } from './EventBusContext-7GvDyO0d.mjs';
+import { T as TranslationManager, S as SessionManager, O as OpenResourcesManager, E as EventBus } from './EventBusContext-CJjL_cCf.mjs';
+export { r as resetEventBusForTesting } from './EventBusContext-CJjL_cCf.mjs';
 export { vi } from 'vitest';
 import 'react/jsx-runtime';
 import 'mitt';

package/dist/test-utils.mjs

@@ -8,7 +8,7 @@ import {
   TranslationProvider,
   resetEventBusForTesting,
   useEventBus
-} from "./chunk-ZR4ZV2LY.mjs";
+} from "./chunk-QB52Q7EQ.mjs";
 import "./chunk-D7NBW4RV.mjs";
 import {
   __commonJS,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@semiont/react-ui",
-  "version": "0.2.33-build.80",
+  "version": "0.2.33-build.81",
   "description": "React components and hooks for Semiont",
   "main": "./dist/index.mjs",
   "types": "./dist/index.d.mts",

package/src/components/LiveRegion.tsx

@@ -76,11 +76,11 @@ export function useSearchAnnouncements() {
     } else {
       announce(`${count} result${count === 1 ? '' : 's'} found for ${query}`, 'polite');
     }
-  }, []);
+  }, [announce]);
 
   const announceSearching = useCallback(() => {
     announce('Searching...', 'polite');
-  }, []);
+  }, [announce]);
 
   return {
     announceSearchResults,
@@ -93,31 +93,31 @@ export function useDocumentAnnouncements(annotators?: Record<string, Annotator>)
 
   const announceDocumentSaved = useCallback(() => {
     announce('Document saved successfully', 'polite');
-  }, []);
+  }, [announce]);
 
   const announceDocumentDeleted = useCallback(() => {
     announce('Document deleted', 'polite');
-  }, []);
+  }, [announce]);
 
   const announceAnnotationCreated = useCallback((annotation: Annotation) => {
     const metadata = annotators ? Object.values(annotators).find(a => a.matchesAnnotation(annotation)) : null;
     const message = metadata?.announceOnCreate ?? 'Annotation created';
     announce(message, 'polite');
-  }, [annotators]);
+  }, [announce, annotators]);
 
   const announceAnnotationDeleted = useCallback(() => {
     announce('Annotation deleted', 'polite');
-  }, []);
+  }, [announce]);
 
   const announceAnnotationUpdated = useCallback((annotation: Annotation) => {
     const metadata = annotators ? Object.values(annotators).find(a => a.matchesAnnotation(annotation)) : null;
     const message = `${metadata?.displayName ?? 'Annotation'} updated`;
     announce(message, 'polite');
-  }, [annotators]);
+  }, [announce, annotators]);
 
   const announceError = useCallback((message: string) => {
     announce(`Error: ${message}`, 'assertive');
-  }, []);
+  }, [announce]);
 
   return {
     announceDocumentSaved,
@@ -138,22 +138,22 @@ export function useResourceLoadingAnnouncements() {
       ? `Loading ${resourceName}...`
       : 'Loading resource...';
     announce(message, 'polite');
-  }, []);
+  }, [announce]);
 
   const announceResourceLoaded = useCallback((resourceName: string) => {
     announce(`${resourceName} loaded successfully`, 'polite');
-  }, []);
+  }, [announce]);
 
   const announceResourceLoadError = useCallback((resourceName?: string) => {
     const message = resourceName
       ? `Failed to load ${resourceName}`
       : 'Failed to load resource';
     announce(message, 'assertive');
-  }, []);
+  }, [announce]);
 
   const announceResourceUpdating = useCallback((resourceName: string) => {
     announce(`Updating ${resourceName}...`, 'polite');
-  }, []);
+  }, [announce]);
 
   return {
     announceResourceLoading,
@@ -169,22 +169,22 @@ export function useFormAnnouncements() {
 
   const announceFormSubmitting = useCallback(() => {
     announce('Submitting form...', 'polite');
-  }, []);
+  }, [announce]);
 
   const announceFormSuccess = useCallback((message?: string) => {
     announce(message || 'Form submitted successfully', 'polite');
-  }, []);
+  }, [announce]);
 
   const announceFormError = useCallback((message?: string) => {
     announce(message || 'Form submission failed. Please check your entries and try again.', 'assertive');
-  }, []);
+  }, [announce]);
 
   const announceFormValidationError = useCallback((fieldCount: number) => {
     const message = fieldCount === 1
       ? 'There is 1 field with an error'
       : `There are ${fieldCount} fields with errors`;
     announce(message, 'assertive');
-  }, []);
+  }, [announce]);
 
   return {
     announceFormSubmitting,
@@ -200,11 +200,11 @@ export function useLanguageChangeAnnouncements() {
 
   const announceLanguageChanging = useCallback((newLanguage: string) => {
     announce(`Changing language to ${newLanguage}...`, 'polite');
-  }, []);
+  }, [announce]);
 
   const announceLanguageChanged = useCallback((newLanguage: string) => {
     announce(`Language changed to ${newLanguage}`, 'polite');
-  }, []);
+  }, [announce]);
 
   return {
     announceLanguageChanging,

package/src/components/SessionExpiryBanner.tsx

@@ -1,13 +1,12 @@
 'use client';
 
 import { useState } from 'react';
-import { useSessionExpiry } from '@semiont/react-ui';
-import { useFormattedTime } from '@semiont/react-ui';
+import { useSessionExpiry, formatTime } from '@semiont/react-ui';
 
 export function SessionExpiryBanner() {
   const { timeRemaining, isExpiringSoon } = useSessionExpiry();
   const [dismissed, setDismissed] = useState(false);
-  const formattedTime = useFormattedTime(timeRemaining);
+  const formattedTime = formatTime(timeRemaining);
 
   // Don't show if:
   // - Session is not expiring soon

package/src/components/SessionTimer.tsx

@@ -1,11 +1,11 @@
 'use client';
 
 import { useSessionExpiry } from '../hooks/useSessionExpiry';
-import { useFormattedTime } from '../hooks/useFormattedTime';
+import { formatTime } from '../lib/formatTime';
 
 export function SessionTimer() {
   const { timeRemaining } = useSessionExpiry();
-  const formattedTime = useFormattedTime(timeRemaining);
+  const formattedTime = formatTime(timeRemaining);
 
   if (!formattedTime) return null;
 

package/src/components/__tests__/SessionTimer.test.tsx

@@ -7,18 +7,18 @@ vi.mock('../../hooks/useSessionExpiry', () => ({
   useSessionExpiry: vi.fn(),
 }));
 
-vi.mock('../../hooks/useFormattedTime', () => ({
-  useFormattedTime: vi.fn(),
+vi.mock('../../lib/formatTime', () => ({
+  formatTime: vi.fn(),
 }));
 
 import { useSessionExpiry } from '../../hooks/useSessionExpiry';
-import { useFormattedTime } from '../../hooks/useFormattedTime';
+import { formatTime } from '../../lib/formatTime';
 
 describe('SessionTimer', () => {
   describe('Rendering', () => {
     it('should render formatted time when available', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 300000 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('5:00');
+      vi.mocked(formatTime).mockReturnValue('5:00');
 
       render(<SessionTimer />);
 
@@ -28,7 +28,7 @@ describe('SessionTimer', () => {
 
     it('should have correct class name', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 300000 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('5:00');
+      vi.mocked(formatTime).mockReturnValue('5:00');
 
       const { container } = render(<SessionTimer />);
 
@@ -37,7 +37,7 @@ describe('SessionTimer', () => {
 
     it('should display complete message with formatted time', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 120000 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('2:00');
+      vi.mocked(formatTime).mockReturnValue('2:00');
 
       render(<SessionTimer />);
 
@@ -49,7 +49,7 @@ describe('SessionTimer', () => {
   describe('Null Cases', () => {
     it('should return null when formattedTime is null', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 0 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue(null);
+      vi.mocked(formatTime).mockReturnValue(null);
 
       const { container } = render(<SessionTimer />);
 
@@ -58,7 +58,7 @@ describe('SessionTimer', () => {
 
     it('should return null when formattedTime is undefined', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 0 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue(undefined as any);
+      vi.mocked(formatTime).mockReturnValue(undefined as any);
 
       const { container } = render(<SessionTimer />);
 
@@ -67,7 +67,7 @@ describe('SessionTimer', () => {
 
     it('should return null when formattedTime is empty string', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 0 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('');
+      vi.mocked(formatTime).mockReturnValue('');
 
       const { container } = render(<SessionTimer />);
 
@@ -79,31 +79,31 @@ describe('SessionTimer', () => {
     it('should call useSessionExpiry hook', () => {
       const mockUseSessionExpiry = vi.mocked(useSessionExpiry);
       mockUseSessionExpiry.mockReturnValue({ timeRemaining: 100000 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('1:40');
+      vi.mocked(formatTime).mockReturnValue('1:40');
 
       render(<SessionTimer />);
 
       expect(mockUseSessionExpiry).toHaveBeenCalled();
     });
 
-    it('should call useFormattedTime with timeRemaining', () => {
-      const mockUseFormattedTime = vi.mocked(useFormattedTime);
+    it('should call formatTime with timeRemaining', () => {
+      const mockFormatTime = vi.mocked(formatTime);
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 300000 } as any);
-      mockUseFormattedTime.mockReturnValue('5:00');
+      mockFormatTime.mockReturnValue('5:00');
 
       render(<SessionTimer />);
 
-      expect(mockUseFormattedTime).toHaveBeenCalledWith(300000);
+      expect(mockFormatTime).toHaveBeenCalledWith(300000);
     });
 
-    it('should pass correct timeRemaining to useFormattedTime', () => {
-      const mockUseFormattedTime = vi.mocked(useFormattedTime);
+    it('should pass correct timeRemaining to formatTime', () => {
+      const mockFormatTime = vi.mocked(formatTime);
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 12345 } as any);
-      mockUseFormattedTime.mockReturnValue('0:12');
+      mockFormatTime.mockReturnValue('0:12');
 
       render(<SessionTimer />);
 
-      expect(mockUseFormattedTime).toHaveBeenCalledWith(12345);
+      expect(mockFormatTime).toHaveBeenCalledWith(12345);
     });
   });
 
@@ -118,7 +118,7 @@ describe('SessionTimer', () => {
 
       testCases.forEach(({ timeRemaining, formatted }) => {
         vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining } as any);
-        vi.mocked(useFormattedTime).mockReturnValue(formatted);
+        vi.mocked(formatTime).mockReturnValue(formatted);
 
         const { unmount } = render(<SessionTimer />);
 
@@ -132,17 +132,17 @@ describe('SessionTimer', () => {
   describe('Re-rendering', () => {
     it('should update when timeRemaining changes', () => {
       const mockUseSessionExpiry = vi.mocked(useSessionExpiry);
-      const mockUseFormattedTime = vi.mocked(useFormattedTime);
+      const mockFormatTime = vi.mocked(formatTime);
 
       mockUseSessionExpiry.mockReturnValue({ timeRemaining: 60000 } as any);
-      mockUseFormattedTime.mockReturnValue('1:00');
+      mockFormatTime.mockReturnValue('1:00');
 
       const { rerender } = render(<SessionTimer />);
       expect(screen.getByText('Session: 1:00 remaining')).toBeInTheDocument();
 
       // Simulate time passing
       mockUseSessionExpiry.mockReturnValue({ timeRemaining: 30000 } as any);
-      mockUseFormattedTime.mockReturnValue('0:30');
+      mockFormatTime.mockReturnValue('0:30');
 
       rerender(<SessionTimer />);
       expect(screen.getByText('Session: 0:30 remaining')).toBeInTheDocument();
@@ -150,13 +150,13 @@ describe('SessionTimer', () => {
 
     it('should hide when formattedTime becomes null', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 60000 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('1:00');
+      vi.mocked(formatTime).mockReturnValue('1:00');
 
       const { container, rerender } = render(<SessionTimer />);
       expect(screen.getByText('Session: 1:00 remaining')).toBeInTheDocument();
 
       // Time expires
-      vi.mocked(useFormattedTime).mockReturnValue(null);
+      vi.mocked(formatTime).mockReturnValue(null);
 
       rerender(<SessionTimer />);
       expect(container.firstChild).toBeNull();
@@ -166,7 +166,7 @@ describe('SessionTimer', () => {
   describe('Edge Cases', () => {
     it('should handle zero timeRemaining', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 0 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('0:00');
+      vi.mocked(formatTime).mockReturnValue('0:00');
 
       render(<SessionTimer />);
 
@@ -175,7 +175,7 @@ describe('SessionTimer', () => {
 
     it('should handle negative timeRemaining', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: -1000 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('expired');
+      vi.mocked(formatTime).mockReturnValue('expired');
 
       render(<SessionTimer />);
 
@@ -184,7 +184,7 @@ describe('SessionTimer', () => {
 
     it('should handle very large timeRemaining', () => {
       vi.mocked(useSessionExpiry).mockReturnValue({ timeRemaining: 999999999 } as any);
-      vi.mocked(useFormattedTime).mockReturnValue('16666:39');
+      vi.mocked(formatTime).mockReturnValue('16666:39');
 
       render(<SessionTimer />);
 

package/src/components/resource/panels/AssessmentPanel.tsx

@@ -137,8 +137,8 @@ export function AssessmentPanel({
 
   const handleSaveNewAssessment = () => {
     if (pendingAnnotation) {
-      const body = newAssessmentText.trim()
-        ? [{ type: 'TextualBody', value: newAssessmentText, purpose: 'assessing' }]
+      const body: components['schemas']['AnnotationBody'][] = newAssessmentText.trim()
+        ? [{ type: 'TextualBody' as const, value: newAssessmentText, purpose: 'assessing' as const }]
         : [];
 
       eventBus.emit('annotation:create', {

package/src/components/resource/panels/DetectSection.tsx

@@ -1,6 +1,6 @@
 'use client';
 
-import { useState, useEffect } from 'react';
+import { useState, useEffect, useCallback } from 'react';
 import { useTranslations } from '../../../contexts/TranslationContext';
 import { useEventBus } from '../../../contexts/EventBusContext';
 import type { Motivation } from '@semiont/api-client';
@@ -29,6 +29,7 @@ interface DetectSectionProps {
  * - Progress display during detection
  *
  * @emits detection:start - Start detection for annotation type. Payload: { motivation: Motivation, options: { instructions?: string, tone?: string, density?: number } }
+ * @emits detection:dismiss-progress - Dismiss the detection progress display
  */
 export function DetectSection({
   annotationType,
@@ -42,7 +43,8 @@ export function DetectSection({
   const t = useTranslations(panelName);
   const eventBus = useEventBus();
   const [instructions, setInstructions] = useState('');
-  const [tone, setTone] = useState('');
+  type ToneValue = 'scholarly' | 'explanatory' | 'conversational' | 'technical' | 'analytical' | 'critical' | 'balanced' | 'constructive' | '';
+  const [tone, setTone] = useState<ToneValue>('');
   // Default density depends on annotation type
   const defaultDensity = annotationType === 'comment' ? 5 : annotationType === 'assessment' ? 4 : annotationType === 'highlight' ? 5 : 5;
   const [density, setDensity] = useState(defaultDensity);
@@ -61,7 +63,7 @@ export function DetectSection({
     localStorage.setItem(`detect-section-expanded-${annotationType}`, String(isExpanded));
   }, [isExpanded, annotationType]);
 
-  const handleDetect = () => {
+  const handleDetect = useCallback(() => {
     // Map annotation type to motivation
     const motivation: Motivation =
       annotationType === 'highlight' ? 'highlighting' :
@@ -73,7 +75,7 @@ export function DetectSection({
       motivation,
       options: {
         instructions: instructions.trim() || undefined,
-        tone: (annotationType === 'comment' || annotationType === 'assessment') && tone ? tone as any : undefined,
+        tone: (annotationType === 'comment' || annotationType === 'assessment') && tone ? tone : undefined,
         density: (annotationType === 'comment' || annotationType === 'assessment' || annotationType === 'highlight') && useDensity ? density : undefined,
       },
     });
@@ -81,7 +83,11 @@ export function DetectSection({
     setInstructions('');
     setTone('');
     // Don't reset density/useDensity - persist across detections
-  };
+  }, [annotationType, instructions, tone, useDensity, density]); // eventBus is stable singleton - never in deps
+
+  const handleDismissProgress = useCallback(() => {
+    eventBus.emit('detection:dismiss-progress', undefined);
+  }, []); // eventBus is stable singleton - never in deps
 
   return (
     <div className="semiont-panel__section">
@@ -134,7 +140,7 @@ export function DetectSection({
                 </label>
                 <select
                   value={tone}
-                  onChange={(e) => setTone(e.target.value)}
+                  onChange={(e) => setTone(e.target.value as ToneValue)}
                   className="semiont-select"
                 >
                   <option value="">Default</option>
@@ -233,7 +239,7 @@ export function DetectSection({
               {/* Close button - shown after detection completes (when not actively detecting) */}
               {!isDetecting && (
                 <button
-                  onClick={() => eventBus.emit('detection:dismiss-progress', undefined)}
+                  onClick={handleDismissProgress}
                   className="semiont-detection-progress__close"
                   aria-label={t('closeProgress')}
                   title={t('closeProgress')}

package/src/components/resource/panels/ReferenceEntry.tsx

@@ -87,7 +87,7 @@ export const ReferenceEntry = forwardRef<HTMLDivElement, ReferenceEntryProps>(
       eventBus.emit('annotation:update-body', {
         annotationUri: reference.id,
         resourceId: sourceUri.split('/resources/')[1] || '',
-        operations: [{ op: 'remove', item: null }], // Remove all body items
+        operations: [{ op: 'remove' }], // Remove all body items
       });
     }
   };

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

@@ -279,10 +279,9 @@ export function TaggingPanel({
                         selector: pendingAnnotation.selector,
                         body: [
                           {
-                            type: 'TextualBody',
+                            type: 'TextualBody' as const,
                             value: e.target.value,
-                            purpose: 'tagging',
-                            schema: selectedSchemaId,
+                            purpose: 'tagging' as const,
                           },
                         ],
                       });

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

@@ -381,7 +381,7 @@ describe('TaggingPanel Component', () => {
           e.event === 'annotation:create' &&
           e.payload?.motivation === 'tagging' &&
           e.payload?.body?.[0]?.value === 'Issue' &&
-          e.payload?.body?.[0]?.schema === 'legal-irac'
+          e.payload?.body?.[0]?.type === 'TextualBody'
         )).toBe(true);
       });
     });

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

@@ -4,8 +4,7 @@
  * Tests the COMPLETE annotation deletion flow with real component composition:
  * - EventBusProvider (REAL)
  * - ApiClientProvider (REAL, with MOCKED client)
- * - useAnnotationFlow (REAL)
- * - useEventOperations (REAL)
+ * - useDetectionFlow (REAL) — single registration point for useResolutionFlow
  * - useEventSubscriptions (REAL)
  *
  * This test focuses on ARCHITECTURE and EVENT WIRING:
@@ -16,14 +15,18 @@
  *
  * CRITICAL: This test prevents regressions where:
  * - Multiple deletion paths exist (event-driven vs direct)
- * - useEventOperations not called in useAnnotationFlow
+ * - useResolutionFlow called in more than one hook (causes duplicate subscriptions)
  * - Auth token missing from API calls (401 errors)
+ *
+ * ARCHITECTURE: useResolutionFlow is called ONLY in useDetectionFlow.
+ * useDetectionFlow handles all detection state (manual annotation selection
+ * and AI-driven SSE detection) plus all API operations via useResolutionFlow.
  */
 
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import { render, waitFor } from '@testing-library/react';
 import { act } from 'react';
-import { useAnnotationFlow } from '../../../hooks/useAnnotationFlow';
+import { useDetectionFlow } from '../../../hooks/useDetectionFlow';
 import { EventBusProvider, useEventBus, resetEventBusForTesting } from '../../../contexts/EventBusContext';
 import { ApiClientProvider } from '../../../contexts/ApiClientContext';
 import { AuthTokenProvider } from '../../../contexts/AuthTokenContext';
@@ -56,7 +59,9 @@ describe('Annotation Deletion - Feature Integration', () => {
 
     function TestComponent() {
       eventBusInstance = useEventBus();
-      useAnnotationFlow(testUri);
+      // useDetectionFlow is the single registration point for useResolutionFlow
+      // (handles annotation:delete, annotation:create, detection:start, etc.)
+      useDetectionFlow(testUri);
       return null;
     }
 
@@ -186,16 +191,14 @@ describe('Annotation Deletion - Feature Integration', () => {
     expect(deleteAnnotationSpy.mock.calls[1][0]).toContain('annotation-2');
   });
 
-  it('ARCHITECTURE: useEventOperations is called in useAnnotationFlow (not elsewhere)', async () => {
+  it('ARCHITECTURE: useResolutionFlow is called in useDetectionFlow (single registration point)', async () => {
     /**
      * This test validates that there's only ONE event-driven deletion path:
-     * - useAnnotationFlow calls useEventOperations
-     * - useEventOperations subscribes to annotation:delete
-     * - No other component/hook subscribes to annotation:delete
+     * - useDetectionFlow calls useResolutionFlow (the single registration point)
+     * - useResolutionFlow subscribes to annotation:delete
      *
-     * If this test fails, it means either:
-     * 1. useEventOperations removed from useAnnotationFlow (CRITICAL BUG)
-     * 2. Duplicate subscription added elsewhere (ARCHITECTURE VIOLATION)
+     * If this test fails with 2 API calls, it means useResolutionFlow was added
+     * to a second hook, causing duplicate subscriptions (ARCHITECTURE VIOLATION).
      */
 
     const { emitDelete } = renderAnnotationFlow();
@@ -216,7 +219,7 @@ describe('Annotation Deletion - Feature Integration', () => {
      *
      * The correct pattern is event-driven only:
      * - UI emits annotation:delete event
-     * - useEventOperations handles it
+     * - useResolutionFlow handles it
      * - No direct function calls
      */
 

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

@@ -25,7 +25,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
     vi.clearAllMocks();
 
     // Minimal mock - SSE streams not needed for this test
-    vi.spyOn(SSEClient.prototype, 'detectAnnotations').mockReturnValue({ onProgress: vi.fn().mockReturnThis(), onComplete: vi.fn().mockReturnThis(), onError: vi.fn().mockReturnThis(), close: vi.fn() } as any);
+    vi.spyOn(SSEClient.prototype, 'detectReferences').mockReturnValue({ onProgress: vi.fn().mockReturnThis(), onComplete: vi.fn().mockReturnThis(), onError: vi.fn().mockReturnThis(), close: vi.fn() } as any);
     vi.spyOn(SSEClient.prototype, 'detectHighlights').mockReturnValue({ onProgress: vi.fn().mockReturnThis(), onComplete: vi.fn().mockReturnThis(), onError: vi.fn().mockReturnThis(), close: vi.fn() } as any);
     vi.spyOn(SSEClient.prototype, 'detectComments').mockReturnValue({ onProgress: vi.fn().mockReturnThis(), onComplete: vi.fn().mockReturnThis(), onError: vi.fn().mockReturnThis(), close: vi.fn() } as any);
     vi.spyOn(SSEClient.prototype, 'detectAssessments').mockReturnValue({ onProgress: vi.fn().mockReturnThis(), onComplete: vi.fn().mockReturnThis(), onError: vi.fn().mockReturnThis(), close: vi.fn() } as any);

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

@@ -5,7 +5,7 @@
  * - EventBusProvider (REAL)
  * - ApiClientProvider (REAL, with MOCKED client)
  * - useDetectionFlow (REAL)
- * - useEventOperations (REAL)
+ * - useResolutionFlow (REAL)
  * - useEventSubscriptions (REAL)
  *
  * This test focuses on ARCHITECTURE and EVENT WIRING:
@@ -60,7 +60,7 @@ const createMockSSEStream = () => {
 
 describe('Detection Flow - Feature Integration', () => {
   let mockStream: ReturnType<typeof createMockSSEStream>;
-  let detectAnnotationsSpy: any;
+  let detectReferencesSpy: any;
   let detectHighlightsSpy: any;
   let detectCommentsSpy: any;
 
@@ -72,7 +72,7 @@ describe('Detection Flow - Feature Integration', () => {
     mockStream = createMockSSEStream();
 
     // Spy on SSEClient prototype methods
-    detectAnnotationsSpy = vi.spyOn(SSEClient.prototype, 'detectAnnotations').mockReturnValue(mockStream as any);
+    detectReferencesSpy = vi.spyOn(SSEClient.prototype, 'detectReferences').mockReturnValue(mockStream as any);
     detectHighlightsSpy = vi.spyOn(SSEClient.prototype, 'detectHighlights').mockReturnValue(mockStream as any);
     detectCommentsSpy = vi.spyOn(SSEClient.prototype, 'detectComments').mockReturnValue(mockStream as any);
     vi.spyOn(SSEClient.prototype, 'detectAssessments').mockReturnValue(mockStream as any);
@@ -82,13 +82,13 @@ describe('Detection Flow - Feature Integration', () => {
     vi.restoreAllMocks();
   });
 
-  it('should call detectAnnotations exactly ONCE when detection starts (not twice)', async () => {
+  it('should call detectReferences exactly ONCE when detection starts (not twice)', async () => {
     const testUri = resourceUri('http://localhost:4000/resources/test-resource');
 
     // Render with real component composition
     const { emitDetectionStart } = renderDetectionFlow(testUri);
 
-    // Trigger detection for linking (uses detectAnnotations)
+    // Trigger detection for linking (uses detectReferences)
     act(() => {
       emitDetectionStart('linking', {
         entityTypes: ['Person', 'Organization'],
@@ -97,13 +97,13 @@ describe('Detection Flow - Feature Integration', () => {
     });
 
     // CRITICAL ASSERTION: API called exactly once (not twice!)
-    // This would FAIL if useEventOperations was called in multiple places
+    // This would FAIL if useResolutionFlow was called in multiple places
     await waitFor(() => {
-      expect(detectAnnotationsSpy).toHaveBeenCalledTimes(1);
+      expect(detectReferencesSpy).toHaveBeenCalledTimes(1);
     });
 
     // Verify correct parameters
-    expect(detectAnnotationsSpy).toHaveBeenCalledWith(
+    expect(detectReferencesSpy).toHaveBeenCalledWith(
       testUri,
       {
         entityTypes: ['Person', 'Organization'],
@@ -128,7 +128,7 @@ describe('Detection Flow - Feature Integration', () => {
 
     // Wait for stream to be created
     await waitFor(() => {
-      expect(detectAnnotationsSpy).toHaveBeenCalled();
+      expect(detectReferencesSpy).toHaveBeenCalled();
     });
 
     // Simulate SSE progress callback being invoked
@@ -316,8 +316,8 @@ describe('Detection Flow - Feature Integration', () => {
   it('should only call API once even with multiple event listeners', async () => {
     const testUri = resourceUri('http://localhost:4000/resources/test-resource');
 
-    // This test specifically catches the duplicate useEventOperations bug
-    // If multiple components call useEventOperations, we'll see multiple API calls
+    // This test specifically catches the duplicate useResolutionFlow bug
+    // If multiple components call useResolutionFlow, we'll see multiple API calls
     const { emitDetectionStart, getEventBus } = renderDetectionFlow(testUri);
 
     // Add an additional event listener (simulating multiple subscribers)
@@ -331,11 +331,11 @@ describe('Detection Flow - Feature Integration', () => {
 
     // Wait for operation to complete
     await waitFor(() => {
-      expect(detectAnnotationsSpy).toHaveBeenCalled();
+      expect(detectReferencesSpy).toHaveBeenCalled();
     });
 
     // VERIFY: API called exactly once, even though multiple listeners exist
-    expect(detectAnnotationsSpy).toHaveBeenCalledTimes(1);
+    expect(detectReferencesSpy).toHaveBeenCalledTimes(1);
 
     // VERIFY: Our additional listener was called (events work)
     expect(additionalListener).toHaveBeenCalledTimes(1);