@semiont/react-ui 0.2.35-build.101 → 0.2.35-build.103

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

@@ -4,7 +4,7 @@
  * Tests the COMPLETE annotation deletion flow with real component composition:
  * - EventBusProvider (REAL)
  * - ApiClientProvider (REAL, with MOCKED client)
- * - useAnnotationFlow (REAL) — single registration point for useResolutionFlow
+ * - useMarkFlow (REAL) — single registration point for useBindFlow
  * - useEventSubscriptions (REAL)
  *
  * This test focuses on ARCHITECTURE and EVENT WIRING:
@@ -15,18 +15,18 @@
  *
  * CRITICAL: This test prevents regressions where:
  * - Multiple deletion paths exist (event-driven vs direct)
- * - useResolutionFlow called in more than one hook (causes duplicate subscriptions)
+ * - useBindFlow called in more than one hook (causes duplicate subscriptions)
  * - Auth token missing from API calls (401 errors)
  *
- * ARCHITECTURE: useResolutionFlow is called ONLY in useAnnotationFlow.
- * useAnnotationFlow handles all detection state (manual annotation selection
- * and AI-driven SSE detection) plus all API operations via useResolutionFlow.
+ * ARCHITECTURE: useBindFlow is called ONLY in useMarkFlow.
+ * useMarkFlow handles all detection state (manual annotation selection
+ * and AI-driven SSE detection) plus all API operations via useBindFlow.
  */
 
 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 { useMarkFlow } from '../../../hooks/useMarkFlow';
 import { EventBusProvider, useEventBus, resetEventBusForTesting } from '../../../contexts/EventBusContext';
 
 // Mock Toast module to prevent "useToast must be used within a ToastProvider" errors
@@ -70,9 +70,9 @@ describe('Annotation Deletion - Feature Integration', () => {
 
     function TestComponent() {
       eventBusInstance = useEventBus();
-      // useAnnotationFlow is the single registration point for useResolutionFlow
-      // (handles annotate:delete annotate:create annotate:detect-request, etc.)
-      useAnnotationFlow(testUri);
+      // useMarkFlow is the single registration point for useBindFlow
+      // (handles mark:delete mark:create annotate:detect-request, etc.)
+      useMarkFlow(testUri);
       return null;
     }
 
@@ -89,14 +89,14 @@ describe('Annotation Deletion - Feature Integration', () => {
     return {
       emitDelete: (annotationId: string) => {
         act(() => {
-          eventBusInstance!.get('annotate:delete').next({ annotationId });
+          eventBusInstance!.get('mark:delete').next({ annotationId });
         });
       },
       eventBus: eventBusInstance!,
     };
   }
 
-  it('should call deleteAnnotation API exactly ONCE when annotate:deleteevent is emitted', async () => {
+  it('should call deleteAnnotation API exactly ONCE when mark:deleteevent is emitted', async () => {
     const { emitDelete } = renderAnnotationFlow();
     const annotationId = 'annotation-123';
 
@@ -133,12 +133,12 @@ describe('Annotation Deletion - Feature Integration', () => {
     expect(callArgs[1].auth).toBe(accessToken(testToken));
   });
 
-  it('should emit annotate:deleted event on successful deletion', async () => {
+  it('should emit mark:deleted event on successful deletion', async () => {
     const { emitDelete, eventBus } = renderAnnotationFlow();
     const deletedListener = vi.fn();
 
     // Subscribe to success event
-    eventBus.get('annotate:deleted').subscribe(deletedListener);
+    eventBus.get('mark:deleted').subscribe(deletedListener);
 
     emitDelete('annotation-789');
 
@@ -155,7 +155,7 @@ describe('Annotation Deletion - Feature Integration', () => {
     });
   });
 
-  it('should emit annotate:delete-failed event on API error', async () => {
+  it('should emit mark:delete-failed event on API error', async () => {
     // Make API call fail
     deleteAnnotationSpy.mockRejectedValue(new Error('Network error'));
 
@@ -163,7 +163,7 @@ describe('Annotation Deletion - Feature Integration', () => {
     const failedListener = vi.fn();
 
     // Subscribe to failure event
-    eventBus.get('annotate:delete-failed').subscribe(failedListener);
+    eventBus.get('mark:delete-failed').subscribe(failedListener);
 
     emitDelete('annotation-error');
 
@@ -202,13 +202,13 @@ describe('Annotation Deletion - Feature Integration', () => {
     expect(deleteAnnotationSpy.mock.calls[1][0]).toContain('annotation-2');
   });
 
-  it('ARCHITECTURE: useResolutionFlow is called in useAnnotationFlow (single registration point)', async () => {
+  it('ARCHITECTURE: useBindFlow is called in useMarkFlow (single registration point)', async () => {
     /**
      * This test validates that there's only ONE event-driven deletion path:
-     * - useAnnotationFlow calls useResolutionFlow (the single registration point)
-     * - useResolutionFlow subscribes to annotation:delete
+     * - useMarkFlow calls useBindFlow (the single registration point)
+     * - useBindFlow subscribes to annotation:delete
      *
-     * If this test fails with 2 API calls, it means useResolutionFlow was added
+     * If this test fails with 2 API calls, it means useBindFlow was added
      * to a second hook, causing duplicate subscriptions (ARCHITECTURE VIOLATION).
      */
 
@@ -229,8 +229,8 @@ describe('Annotation Deletion - Feature Integration', () => {
      * that bypassed the event bus.
      *
      * The correct pattern is event-driven only:
-     * - UI emits annotate:deleteevent
-     * - useResolutionFlow handles it
+     * - UI emits mark:deleteevent
+     * - useBindFlow handles it
      * - No direct function calls
      */
 

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

@@ -8,20 +8,20 @@
  * 4. BUG: Progress modal stays visible showing "Processing: Location" indefinitely
  *
  * ROOT CAUSE:
- * - useAnnotationFlow.ts (line 54-62): annotate:assist-finished clears `assistingMotivation` but keeps `progress`
+ * - useMarkFlow.ts (line 54-62): mark:assist-finished clears `assistingMotivation` but keeps `progress`
  * - AssistSection.tsx (line 214): Shows progress UI whenever `progress` is not null
  * - No mechanism to auto-dismiss or manually close the progress display
  *
  * FIX OPTIONS:
  * A) Auto-dismiss after timeout (3s after completion)
  * B) Add "Close" button to progress display
- * C) Clear progress on next annotate:assist-request (already works but not ideal UX)
+ * C) Clear progress on next mark:assist-request (already works but not ideal UX)
  */
 
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import { render, screen, waitFor } from '@testing-library/react';
 import { act } from 'react';
-import { useAnnotationFlow } from '../../../hooks/useAnnotationFlow';
+import { useMarkFlow } from '../../../hooks/useMarkFlow';
 import { EventBusProvider, resetEventBusForTesting, useEventBus } from '../../../contexts/EventBusContext';
 import { ApiClientProvider } from '../../../contexts/ApiClientContext';
 import { AuthTokenProvider } from '../../../contexts/AuthTokenContext';
@@ -65,7 +65,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     function TestHarness() {
       eventBusInstance = useEventBus();
-      const { assistingMotivation, progress } = useAnnotationFlow(rUri);
+      const { assistingMotivation, progress } = useMarkFlow(rUri);
 
       return (
         <div>
@@ -91,9 +91,9 @@ describe('Detection Progress Dismissal Bug', () => {
     expect(screen.getByTestId('detecting')).toHaveTextContent('none');
     expect(screen.getByTestId('progress')).toHaveTextContent('no progress');
 
-    // User clicks detect button (emits annotate:assist-request)
+    // User clicks detect button (emits mark:assist-request)
     act(() => {
-      eventBusInstance.get('annotate:assist-request').next({
+      eventBusInstance.get('mark:assist-request').next({
         motivation: 'linking',
         options: { entityTypes: ['Location'] }
       });
@@ -106,7 +106,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // SSE sends progress update
     act(() => {
-      eventBusInstance.get('annotate:progress').next({
+      eventBusInstance.get('mark:progress').next({
         status: 'scanning',
         message: 'Processing: Location',
         currentEntityType: 'Location',
@@ -118,9 +118,9 @@ describe('Detection Progress Dismissal Bug', () => {
       expect(screen.getByTestId('progress')).toHaveTextContent('Processing: Location');
     });
 
-    // Detection completes (SSE finishes, backend emits annotate:assist-finished)
+    // Detection completes (SSE finishes, backend emits mark:assist-finished)
     act(() => {
-      eventBusInstance.get('annotate:assist-finished').next({ motivation: 'linking' });
+      eventBusInstance.get('mark:assist-finished').next({ motivation: 'linking' });
     });
 
     // assistingMotivation cleared immediately
@@ -138,7 +138,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     function TestHarness() {
       eventBusInstance = useEventBus();
-      const { progress } = useAnnotationFlow(rUri);
+      const { progress } = useMarkFlow(rUri);
 
       return (
         <div data-testid="progress">
@@ -159,15 +159,15 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // First detection with stuck progress
     act(() => {
-      eventBusInstance.get('annotate:assist-request').next({ motivation: 'linking', options: {} });
+      eventBusInstance.get('mark:assist-request').next({ motivation: 'linking', options: {} });
     });
 
     act(() => {
-      eventBusInstance.get('annotate:progress').next({ message: 'Old progress stuck here' });
+      eventBusInstance.get('mark:progress').next({ message: 'Old progress stuck here' });
     });
 
     act(() => {
-      eventBusInstance.get('annotate:assist-finished').next({ motivation: 'linking' });
+      eventBusInstance.get('mark:assist-finished').next({ motivation: 'linking' });
     });
 
     await waitFor(() => {
@@ -176,7 +176,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // WORKAROUND: Start new detection clears old progress
     act(() => {
-      eventBusInstance.get('annotate:assist-request').next({ motivation: 'highlighting', options: {} });
+      eventBusInstance.get('mark:assist-request').next({ motivation: 'highlighting', options: {} });
     });
 
     await waitFor(() => {
@@ -191,7 +191,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     function TestHarness() {
       eventBusInstance = useEventBus();
-      const { progress } = useAnnotationFlow(rUri);
+      const { progress } = useMarkFlow(rUri);
 
       return (
         <div data-testid="progress">
@@ -212,18 +212,18 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // Show progress
     act(() => {
-      eventBusInstance.get('annotate:assist-request').next({ motivation: 'linking', options: {} });
+      eventBusInstance.get('mark:assist-request').next({ motivation: 'linking', options: {} });
     });
 
     act(() => {
-      eventBusInstance.get('annotate:progress').next({
+      eventBusInstance.get('mark:progress').next({
         status: 'complete',
         message: 'Complete! Created 5 annotations'
       });
     });
 
     act(() => {
-      eventBusInstance.get('annotate:assist-finished').next({ motivation: 'linking' });
+      eventBusInstance.get('mark:assist-finished').next({ motivation: 'linking' });
     });
 
     // Progress visible initially
@@ -242,7 +242,7 @@ describe('Detection Progress Dismissal Bug', () => {
   it('FIXED: SSE emits final completion chunk data as annotate:assist-progress', async () => {
     /**
      * This test verifies that SSE emits the final chunk as annotate:assist-progress
-     * BEFORE emitting annotate:assist-finished.
+     * BEFORE emitting mark:assist-finished.
      *
      * This ensures the UI can display the final completion message with status:'complete'.
      */
@@ -251,7 +251,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     function TestHarness() {
       eventBusInstance = useEventBus();
-      const { progress } = useAnnotationFlow(rUri);
+      const { progress } = useMarkFlow(rUri);
 
       return (
         <div>
@@ -273,7 +273,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // Start detection (triggers SSE stream creation)
     act(() => {
-      eventBusInstance.get('annotate:assist-request').next({
+      eventBusInstance.get('mark:assist-request').next({
         motivation: 'linking',
         options: { entityTypes: ['Location'] }
       });
@@ -281,7 +281,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // Simulate SSE scanning chunk
     act(() => {
-      eventBusInstance.get('annotate:progress').next({
+      eventBusInstance.get('mark:progress').next({
         status: 'scanning',
         message: 'Processing: Location',
       });
@@ -293,7 +293,7 @@ describe('Detection Progress Dismissal Bug', () => {
 
     // Simulate SSE emitting final chunk as annotate:assist-progress
     act(() => {
-      eventBusInstance.get('annotate:progress').next({
+      eventBusInstance.get('mark:progress').next({
         status: 'complete',
         message: 'Complete! Found 5 entities',
         foundCount: 5,

package/src/features/resource-viewer/__tests__/{ResolutionFlowIntegration.test.tsx → BindFlowIntegration.test.tsx}

@@ -1,16 +1,16 @@
 /**
  * Layer 3: Feature Integration Test - Resolution Flow (search modal & body update)
  *
- * Tests the UNCOVERED half of useResolutionFlow:
- * - resolve:link → emits resolve:search-requested
- * - resolve:search-requested → opens search modal with pendingReferenceId
+ * Tests the UNCOVERED half of useBindFlow:
+ * - bind:link → emits bind:search-requested
+ * - bind:search-requested → opens search modal with pendingReferenceId
  * - onCloseSearchModal → closes modal
- * - resolve:update-body → calls updateAnnotationBody API
- * - resolve:update-body → emits resolve:body-updated on success
- * - resolve:update-body → emits resolve:body-update-failed on error
+ * - bind:update-body → calls updateAnnotationBody API
+ * - bind:update-body → emits bind:body-updated on success
+ * - bind:update-body → emits bind:body-update-failed on error
  * - auth token passed to updateAnnotationBody
  *
- * The deletion half of useResolutionFlow is covered by AnnotationDeletionIntegration.test.tsx.
+ * The deletion half of useBindFlow is covered by AnnotationDeletionIntegration.test.tsx.
  *
  * Uses real providers (EventBus, ApiClient, AuthToken) with mocked API boundary.
  */
@@ -18,7 +18,7 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import { render, waitFor } from '@testing-library/react';
 import { act } from 'react';
-import { useResolutionFlow } from '../../../hooks/useResolutionFlow';
+import { useBindFlow } from '../../../hooks/useBindFlow';
 import { EventBusProvider, useEventBus, resetEventBusForTesting } from '../../../contexts/EventBusContext';
 import { ApiClientProvider } from '../../../contexts/ApiClientContext';
 import { AuthTokenProvider } from '../../../contexts/AuthTokenContext';
@@ -55,13 +55,13 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
 
   // ─── Render helper ──────────────────────────────────────────────────────────
 
-  function renderResolutionFlow() {
+  function renderBindFlow() {
     let eventBusInstance: ReturnType<typeof useEventBus> | null = null;
-    let lastState: ReturnType<typeof useResolutionFlow> | null = null;
+    let lastState: ReturnType<typeof useBindFlow> | null = null;
 
     function TestComponent() {
       eventBusInstance = useEventBus();
-      lastState = useResolutionFlow(testUri);
+      lastState = useBindFlow(testUri);
       return null;
     }
 
@@ -84,19 +84,19 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
   // ─── Initial state ──────────────────────────────────────────────────────────
 
   it('starts with search modal closed and no pending reference', () => {
-    const { getState } = renderResolutionFlow();
+    const { getState } = renderBindFlow();
     expect(getState().searchModalOpen).toBe(false);
     expect(getState().pendingReferenceId).toBeNull();
   });
 
-  // ─── resolve:link ─────────────────────────────────────────────────────────
+  // ─── bind:link ─────────────────────────────────────────────────────────
 
-  it('resolve:link emits resolve:search-requested with referenceId and searchTerm', () => {
-    const { getEventBus } = renderResolutionFlow();
+  it('bind:link emits bind:search-requested with referenceId and searchTerm', () => {
+    const { getEventBus } = renderBindFlow();
     const searchRequestedSpy = vi.fn();
 
-    const subscription = getEventBus().get('resolve:search-requested').subscribe(searchRequestedSpy);
-    act(() => { getEventBus().get('resolve:link').next({ annotationUri: 'ann-uri-123', searchTerm: 'climate change' }); });
+    const subscription = getEventBus().get('bind:search-requested').subscribe(searchRequestedSpy);
+    act(() => { getEventBus().get('bind:link').next({ annotationUri: 'ann-uri-123', searchTerm: 'climate change' }); });
     subscription.unsubscribe();
 
     expect(searchRequestedSpy).toHaveBeenCalledTimes(1);
@@ -106,35 +106,35 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
     });
   });
 
-  // ─── resolve:search-requested ────────────────────────────────────────────
+  // ─── bind:search-requested ────────────────────────────────────────────
 
-  it('resolve:search-requested opens the search modal', async () => {
-    const { getState, getEventBus } = renderResolutionFlow();
+  it('bind:search-requested opens the search modal', async () => {
+    const { getState, getEventBus } = renderBindFlow();
 
     expect(getState().searchModalOpen).toBe(false);
 
-    act(() => { getEventBus().get('resolve:search-requested').next({ referenceId: 'ref-abc', searchTerm: 'oceans' }); });
+    act(() => { getEventBus().get('bind:search-requested').next({ referenceId: 'ref-abc', searchTerm: 'oceans' }); });
 
     await waitFor(() => {
       expect(getState().searchModalOpen).toBe(true);
     });
   });
 
-  it('resolve:search-requested sets pendingReferenceId', async () => {
-    const { getState, getEventBus } = renderResolutionFlow();
+  it('bind:search-requested sets pendingReferenceId', async () => {
+    const { getState, getEventBus } = renderBindFlow();
 
-    act(() => { getEventBus().get('resolve:search-requested').next({ referenceId: 'ref-xyz', searchTerm: 'forests' }); });
+    act(() => { getEventBus().get('bind:search-requested').next({ referenceId: 'ref-xyz', searchTerm: 'forests' }); });
 
     await waitFor(() => {
       expect(getState().pendingReferenceId).toBe('ref-xyz');
     });
   });
 
-  it('resolve:link → resolve:search-requested chain opens modal end-to-end', async () => {
-    const { getState, getEventBus } = renderResolutionFlow();
+  it('bind:link → bind:search-requested chain opens modal end-to-end', async () => {
+    const { getState, getEventBus } = renderBindFlow();
 
     // Simulate the full user journey: user clicks "Link Document" on a reference entry
-    act(() => { getEventBus().get('resolve:link').next({ annotationUri: 'ann-full-chain', searchTerm: 'biodiversity' }); });
+    act(() => { getEventBus().get('bind:link').next({ annotationUri: 'ann-full-chain', searchTerm: 'biodiversity' }); });
 
     await waitFor(() => {
       expect(getState().searchModalOpen).toBe(true);
@@ -145,9 +145,9 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
   // ─── onCloseSearchModal ──────────────────────────────────────────────────────
 
   it('onCloseSearchModal closes the search modal', async () => {
-    const { getState, getEventBus } = renderResolutionFlow();
+    const { getState, getEventBus } = renderBindFlow();
 
-    act(() => { getEventBus().get('resolve:search-requested').next({ referenceId: 'ref-close', searchTerm: 'test' }); });
+    act(() => { getEventBus().get('bind:search-requested').next({ referenceId: 'ref-close', searchTerm: 'test' }); });
 
     await waitFor(() => expect(getState().searchModalOpen).toBe(true));
 
@@ -159,9 +159,9 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
   });
 
   it('onCloseSearchModal does not clear pendingReferenceId (preserves for re-open)', async () => {
-    const { getState, getEventBus } = renderResolutionFlow();
+    const { getState, getEventBus } = renderBindFlow();
 
-    act(() => { getEventBus().get('resolve:search-requested').next({ referenceId: 'ref-persist', searchTerm: 'test' }); });
+    act(() => { getEventBus().get('bind:search-requested').next({ referenceId: 'ref-persist', searchTerm: 'test' }); });
     await waitFor(() => expect(getState().searchModalOpen).toBe(true));
 
     act(() => { getState().onCloseSearchModal(); });
@@ -171,12 +171,12 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
     expect(getState().pendingReferenceId).toBe('ref-persist');
   });
 
-  // ─── resolve:update-body ──────────────────────────────────────────────────
+  // ─── bind:update-body ──────────────────────────────────────────────────
 
-  it('resolve:update-body calls updateAnnotationBody API', async () => {
-    const { getEventBus } = renderResolutionFlow();
+  it('bind:update-body calls updateAnnotationBody API', async () => {
+    const { getEventBus } = renderBindFlow();
 
-    act(() => { getEventBus().get('resolve:update-body').next({
+    act(() => { getEventBus().get('bind:update-body').next({
       annotationUri: 'http://localhost:4000/resources/test-resource/annotations/ann-body-1',
       resourceId: 'linked-resource-id',
       operations: [{ op: 'add', item: { id: 'linked-resource-id' } }],
@@ -187,10 +187,10 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
     });
   });
 
-  it('resolve:update-body passes auth token to API call', async () => {
-    const { getEventBus } = renderResolutionFlow();
+  it('bind:update-body passes auth token to API call', async () => {
+    const { getEventBus } = renderBindFlow();
 
-    act(() => { getEventBus().get('resolve:update-body').next({
+    act(() => { getEventBus().get('bind:update-body').next({
       annotationUri: 'http://localhost:4000/resources/test-resource/annotations/ann-auth',
       resourceId: 'resource-id',
       operations: [{ op: 'replace', newItem: { id: 'resource-id' } }],
@@ -205,13 +205,13 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
     expect(callArgs[2].auth).toBe(accessToken(testToken));
   });
 
-  it('resolve:update-body emits resolve:body-updated on success', async () => {
-    const { getEventBus } = renderResolutionFlow();
+  it('bind:update-body emits bind:body-updated on success', async () => {
+    const { getEventBus } = renderBindFlow();
     const bodyUpdatedSpy = vi.fn();
 
-    const subscription = getEventBus().get('resolve:body-updated').subscribe(bodyUpdatedSpy);
+    const subscription = getEventBus().get('bind:body-updated').subscribe(bodyUpdatedSpy);
 
-    act(() => { getEventBus().get('resolve:update-body').next({
+    act(() => { getEventBus().get('bind:update-body').next({
       annotationUri: 'http://localhost:4000/resources/test-resource/annotations/ann-success',
       resourceId: 'resource-id',
       operations: [{ op: 'add', item: { id: 'resource-id' } }],
@@ -228,15 +228,15 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
     });
   });
 
-  it('resolve:update-body emits resolve:body-update-failed on API error', async () => {
+  it('bind:update-body emits bind:body-update-failed on API error', async () => {
     updateAnnotationBodySpy.mockRejectedValue(new Error('Update failed'));
 
-    const { getEventBus } = renderResolutionFlow();
+    const { getEventBus } = renderBindFlow();
     const bodyUpdateFailedSpy = vi.fn();
 
-    const subscription = getEventBus().get('resolve:body-update-failed').subscribe(bodyUpdateFailedSpy);
+    const subscription = getEventBus().get('bind:body-update-failed').subscribe(bodyUpdateFailedSpy);
 
-    act(() => { getEventBus().get('resolve:update-body').next({
+    act(() => { getEventBus().get('bind:update-body').next({
       annotationUri: 'http://localhost:4000/resources/test-resource/annotations/ann-fail',
       resourceId: 'resource-id',
       operations: [{ op: 'remove', item: { id: 'old-id' } }],
@@ -253,10 +253,10 @@ describe('Resolution Flow - Search Modal & Body Update Integration', () => {
     });
   });
 
-  it('resolve:update-body called ONCE — no duplicate subscriptions', async () => {
-    const { getEventBus } = renderResolutionFlow();
+  it('bind:update-body called ONCE — no duplicate subscriptions', async () => {
+    const { getEventBus } = renderBindFlow();
 
-    act(() => { getEventBus().get('resolve:update-body').next({
+    act(() => { getEventBus().get('bind:update-body').next({
       annotationUri: 'http://localhost:4000/resources/test-resource/annotations/ann-dedup',
       resourceId: 'resource-id',
       operations: [{ op: 'add', item: { id: 'resource-id' } }],

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

@@ -2,18 +2,18 @@
  * FAILING TEST: Reproduces the bug where detection events fire but state doesn't update
  *
  * Based on console logs from production:
- * ✅ annotate:assist-request emitted
+ * ✅ mark:assist-request emitted
  * ✅ annotate:assist-progress emitted
  * ❌ assistingMotivation remains null
  * ❌ progress remains null
  *
- * UPDATED: Now tests useAnnotationFlow hook instead of DetectionFlowContainer
+ * UPDATED: Now tests useMarkFlow hook instead of DetectionFlowContainer
  */
 
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { render, screen, waitFor } from '@testing-library/react';
 import { act } from 'react';
-import { useAnnotationFlow } from '../../../hooks/useAnnotationFlow';
+import { useMarkFlow } from '../../../hooks/useMarkFlow';
 import { EventBusProvider, useEventBus, resetEventBusForTesting } from '../../../contexts/EventBusContext';
 import { ApiClientProvider } from '../../../contexts/ApiClientContext';
 import { AuthTokenProvider } from '../../../contexts/AuthTokenContext';
@@ -45,17 +45,17 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
     vi.restoreAllMocks();
   });
 
-  it('SHOULD update state when annotate:assist-request event is emitted', async () => {
+  it('SHOULD update state when mark:assist-request event is emitted', async () => {
     let eventBusInstance: any;
     let currentState: any;
 
     // Component to capture EventBus and hook state
     function TestComponent() {
       eventBusInstance = useEventBus();
-      const state = useAnnotationFlow('http://localhost:8080/resources/test' as any);
+      const state = useMarkFlow('http://localhost:8080/resources/test' as any);
       currentState = state;
 
-      console.log('[TEST] useAnnotationFlow state:', {
+      console.log('[TEST] useMarkFlow state:', {
         assistingMotivation: state.assistingMotivation,
         progress: state.progress,
       });
@@ -82,17 +82,17 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
     expect(screen.getByTestId('detecting')).toHaveTextContent('null');
     expect(screen.getByTestId('progress')).toHaveTextContent('null');
 
-    console.log('[TEST] Emitting annotate:assist-request event...');
+    console.log('[TEST] Emitting mark:assist-request event...');
 
-    // Emit annotate:assist-request event (exactly like production)
+    // Emit mark:assist-request event (exactly like production)
     act(() => {
-      eventBusInstance.get('annotate:assist-request').next({
+      eventBusInstance.get('mark:assist-request').next({
         motivation: 'linking',
         options: { entityTypes: ['Location'] }
       });
     });
 
-    console.log('[TEST] After annotate:assist-request, checking state...');
+    console.log('[TEST] After mark:assist-request, checking state...');
 
     // THIS SHOULD PASS but currently FAILS
     await waitFor(() => {
@@ -109,7 +109,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     function TestComponent() {
       eventBusInstance = useEventBus();
-      const state = useAnnotationFlow('http://localhost:8080/resources/test' as any);
+      const state = useMarkFlow('http://localhost:8080/resources/test' as any);
       currentState = state;
 
       return (
@@ -134,7 +134,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     // Emit annotate:assist-progress event (exactly like production)
     act(() => {
-      eventBusInstance.get('annotate:progress').next({
+      eventBusInstance.get('mark:progress').next({
         status: 'started',
         resourceId: 'test',
         totalEntityTypes: 1,
@@ -162,7 +162,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     function TestComponent() {
       eventBusInstance = useEventBus();
-      const state = useAnnotationFlow('http://localhost:8080/resources/f45fd44f9cb0b0fe1b7980d3d034bc61' as any);
+      const state = useMarkFlow('http://localhost:8080/resources/f45fd44f9cb0b0fe1b7980d3d034bc61' as any);
 
       stateSnapshots.push({
         assistingMotivation: state.assistingMotivation,
@@ -192,18 +192,18 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     // Exactly like production logs
     act(() => {
-      console.log('[EventBus] emit: annotate:assist-request {motivation: "linking", options: {...}}');
-      eventBusInstance.get('annotate:assist-request').next({
+      console.log('[EventBus] emit: mark:assist-request {motivation: "linking", options: {...}}');
+      eventBusInstance.get('mark:assist-request').next({
         motivation: 'linking',
         options: { entityTypes: ['Location'] }
       });
     });
 
-    console.log('After annotate:assist-request:', stateSnapshots[stateSnapshots.length - 1]);
+    console.log('After mark:assist-request:', stateSnapshots[stateSnapshots.length - 1]);
 
     act(() => {
       console.log('[EventBus] emit: annotate:assist-progress {status: "started", ...}');
-      eventBusInstance.get('annotate:progress').next({
+      eventBusInstance.get('mark:progress').next({
         status: 'started',
         resourceId: 'f45fd44f9cb0b0fe1b7980d3d034bc61',
         totalEntityTypes: 1,
@@ -216,7 +216,7 @@ describe('REPRODUCING BUG: Detection state not updating', () => {
 
     act(() => {
       console.log('[EventBus] emit: annotate:assist-progress {status: "scanning", ...}');
-      eventBusInstance.get('annotate:progress').next({
+      eventBusInstance.get('mark:progress').next({
         status: 'scanning',
         resourceId: 'f45fd44f9cb0b0fe1b7980d3d034bc61',
         currentEntityType: 'Location',