@puredesktop/platform-editor 1.0.0-beta.1 → 1.0.0-beta.2

package/src/extensions/autoReviewPrompts.test.ts

@@ -1,529 +1,529 @@
-/**
- * @vitest-environment happy-dom
- */
-import { Editor } from '@tiptap/core'
-import StarterKit from '@tiptap/starter-kit'
-import { describe, expect, it } from 'vitest'
-import { CommentMark } from './commentMark.js'
-import {
-  applyAutoReviewFindings,
-  AutoReviewPrompts,
-  buildAutoReviewPrompt,
-  collectAutoReviewParagraphs,
-  collectScopedAutoReviewParagraphs,
-  enabledEditorialTypesForProcess,
-  parseAutoReviewFindingResponse,
-  runAutoReview,
-  validateAutoReviewFindings,
-  type AutoReviewFindingInput,
-} from './autoReviewPrompts.js'
-
-describe('AutoReviewPrompts', () => {
-  it('builds separate prompts with paragraph-only payloads', () => {
-    const paragraphs = [
-      { id: 'p-1', text: 'Evidence is limited.', from: 1, to: 21 },
-    ]
-
-    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
-      'one thing only: CLAIMS',
-    )
-    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
-      'STEP 1 — GATES',
-    )
-    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
-      'GATE A — Is it a citation or reference?',
-    )
-    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
-      'ONE SENTENCE CAN CARRY TWO CLAIMS',
-    )
-    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
-      'source it, soften it, or cut it',
-    )
-    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
-      'severity            "minor" | "notable" | "significant"',
-    )
-    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
-      '"text": "Evidence is limited."',
-    )
-    expect(buildAutoReviewPrompt('fact-check', paragraphs)).toContain(
-      'one thing only: FACT-CHECK NEEDS',
-    )
-    expect(buildAutoReviewPrompt('fact-check', paragraphs)).toContain(
-      'You DETECT; you do not VERIFY',
-    )
-    expect(buildAutoReviewPrompt('fact-check', paragraphs)).toContain(
-      'ATOMIC SPANS',
-    )
-    expect(buildAutoReviewPrompt('fact-check', paragraphs)).toContain(
-      'severity            "minor" | "notable" | "significant"',
-    )
-    expect(buildAutoReviewPrompt('source', paragraphs)).toContain(
-      'one thing only: SOURCE checks',
-    )
-    expect(buildAutoReviewPrompt('source', paragraphs)).toContain(
-      'annotation_type     "source"',
-    )
-    expect(buildAutoReviewPrompt('thread', paragraphs)).toContain(
-      'You are detecting THREADS',
-    )
-    expect(buildAutoReviewPrompt('thread', paragraphs)).toContain(
-      'annotation_type     "thread"',
-    )
-    expect(buildAutoReviewPrompt('logic', paragraphs)).toContain(
-      'You are detecting LOGIC gaps',
-    )
-    expect(buildAutoReviewPrompt('logic', paragraphs)).toContain(
-      'annotation_type     "logic"',
-    )
-    expect(buildAutoReviewPrompt('copyedit', paragraphs)).toContain(
-      'You are the COPYEDIT pass',
-    )
-    expect(buildAutoReviewPrompt('copyedit', paragraphs)).toContain(
-      'annotation_type     "copyedit"',
-    )
-    expect(buildAutoReviewPrompt('copyedit', paragraphs)).toContain(
-      '{{HOUSE_STYLE}}',
-    )
-    expect(buildAutoReviewPrompt('voice', paragraphs)).toContain(
-      'You are the VOICE check',
-    )
-    expect(buildAutoReviewPrompt('voice', paragraphs)).toContain(
-      'annotation_type     "voice"',
-    )
-    expect(buildAutoReviewPrompt('voice', paragraphs)).toContain(
-      '{{VOICE_PROFILE}}',
-    )
-  })
-
-  it('enables the new editorial checks while keeping conflict disabled', () => {
-    expect(enabledEditorialTypesForProcess('source')).toEqual(['source'])
-    expect(enabledEditorialTypesForProcess('source-and-thread')).toEqual([
-      'source',
-      'thread',
-    ])
-    expect(enabledEditorialTypesForProcess('thread')).toEqual(['thread'])
-    expect(enabledEditorialTypesForProcess('logic')).toEqual(['logic'])
-    expect(enabledEditorialTypesForProcess('logic-and-conflict')).toEqual([
-      'logic',
-    ])
-    expect(enabledEditorialTypesForProcess('conflict')).toEqual([])
-    expect(enabledEditorialTypesForProcess('full-editorial')).toEqual([
-      'claim',
-      'fact-check',
-      'source',
-      'thread',
-    ])
-    expect(enabledEditorialTypesForProcess('copyedit')).toEqual(['copyedit'])
-    expect(enabledEditorialTypesForProcess('voice')).toEqual(['voice'])
-  })
-
-  it('collects only paragraph text from a TipTap document', () => {
-    const editor = new Editor({
-      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
-      content:
-        '<h1>Heading claim</h1><p>Small layout failures are easy to spot.</p><p></p><p>Footnotes can collide with running text.</p>',
-    })
-
-    expect(collectAutoReviewParagraphs(editor)).toEqual([
-      expect.objectContaining({
-        id: 'p-1',
-        text: 'Small layout failures are easy to spot.',
-      }),
-      expect.objectContaining({
-        id: 'p-2',
-        text: 'Footnotes can collide with running text.',
-      }),
-    ])
-    editor.destroy()
-  })
-
-  it('skips protected generated content when collecting paragraphs', () => {
-    const editor = new Editor({
-      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
-      content:
-        '<p>Body claim should be reviewed.</p><p>Reference text should not be reviewed.</p>',
-    })
-    const generated = editor.view.dom.querySelectorAll('p')[1]
-    generated.setAttribute('data-pm-generated-references', 'true')
-    generated.setAttribute('contenteditable', 'false')
-
-    expect(collectAutoReviewParagraphs(editor)).toEqual([
-      expect.objectContaining({
-        text: 'Body claim should be reviewed.',
-      }),
-    ])
-    editor.destroy()
-  })
-
-  it('validates anchors, filters low confidence, and prefers fact checks for duplicate spans', () => {
-    const paragraphs = [
-      {
-        id: 'p-1',
-        text: 'Small layout failures are easy to spot.',
-        from: 1,
-        to: 40,
-      },
-    ]
-    const findings: AutoReviewFindingInput[] = [
-      {
-        paragraph_id: 'p-1',
-        exact_text: 'Small layout failures',
-        annotation_type: 'claim',
-        shape: 'generalization',
-        suggested_note: 'Add support.',
-        severity: 'significant',
-        confidence: 0.9,
-      },
-      {
-        paragraph_id: 'p-1',
-        exact_text: 'Small layout failures',
-        annotation_type: 'fact_check',
-        shape: 'attribution',
-        suggested_note: 'Verify this statement.',
-        severity: 'notable',
-        confidence: 0.95,
-      },
-      {
-        paragraph_id: 'p-1',
-        exact_text: 'not in paragraph',
-        annotation_type: 'claim',
-        confidence: 0.95,
-      },
-      {
-        paragraph_id: 'p-1',
-        exact_text: 'easy to spot',
-        annotation_type: 'claim',
-        confidence: 0.1,
-      },
-    ]
-
-    expect(validateAutoReviewFindings(paragraphs, findings)).toEqual([
-      expect.objectContaining({
-        annotationType: 'fact-check',
-        exactText: 'Small layout failures',
-        severity: 'advisory',
-        from: 1,
-        to: 22,
-      }),
-    ])
-  })
-
-  it('normalizes realistic model finding shapes before anchoring them', () => {
-    const paragraphs = [
-      {
-        id: 'p-1',
-        text: 'The “quiet export” is a stronger path - by far.',
-        from: 1,
-        to: 48,
-      },
-    ]
-    const findings = parseAutoReviewFindingResponse(
-      JSON.stringify([
-        {
-          paragraphId: 'p-1',
-          span: 'the "quiet export" is a stronger path',
-          type: 'CLAIM',
-          note: '"stronger path" — by whose measure?',
-          severity: 'notable',
-          confidence: 0.92,
-        },
-        {
-          paragraph_id: 'p-1',
-          exactText: 'BY FAR',
-          annotationType: 'FACT_CHECK',
-          suggestedNote: '"by far" — what comparison establishes this?',
-          confidence: 0.91,
-        },
-      ]),
-    )
-
-    expect(findings).toEqual([
-      expect.objectContaining({
-        paragraph_id: 'p-1',
-        exact_text: 'the "quiet export" is a stronger path',
-        annotation_type: 'claim',
-      }),
-      expect.objectContaining({
-        paragraph_id: 'p-1',
-        exact_text: 'BY FAR',
-        annotation_type: 'fact_check',
-      }),
-    ])
-
-    expect(validateAutoReviewFindings(paragraphs, findings)).toEqual([
-      expect.objectContaining({
-        annotationType: 'claim',
-        exactText: 'The “quiet export” is a stronger path',
-        from: 1,
-        to: 38,
-      }),
-      expect.objectContaining({
-        annotationType: 'fact-check',
-        exactText: 'by far',
-        from: 41,
-        to: 47,
-      }),
-    ])
-  })
-
-  it('applies validated findings as claim and fact-check comment marks', () => {
-    const editor = new Editor({
-      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
-      content: '<p>Small layout failures are easy to spot.</p>',
-    })
-    const paragraphs = collectAutoReviewParagraphs(editor)
-    const result = applyAutoReviewFindings(
-      editor,
-      paragraphs,
-      [
-        {
-          paragraph_id: 'p-1',
-          exact_text: 'Small layout failures',
-          annotation_type: 'claim',
-          subject: 'Layout failures',
-          justification: 'This generalization needs evidence.',
-          suggested_note: 'Add citation or narrow the claim.',
-          severity: 'warning',
-          confidence: 0.91,
-        },
-      ],
-      {
-        author: 'Agent',
-        createdAt: '2026-06-18T12:00:00.000Z',
-        idPrefix: 'test-review',
-      },
-    )
-
-    expect(result.applied).toHaveLength(1)
-    expect(result.skipped).toBe(0)
-    expect(editor.getHTML()).toContain('data-comment-type="claim"')
-    expect(editor.getHTML()).toContain(
-      'data-comment-review-status="needs-support"',
-    )
-    expect(editor.getHTML()).toContain('data-comment-severity="advisory"')
-    expect(editor.getHTML()).toContain('Add citation or narrow the claim.')
-    editor.destroy()
-  })
-
-  it('applies source, thread, and logic findings with their default open states', () => {
-    const editor = new Editor({
-      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
-      content:
-        '<p>Smith 2019 found a 40 percent drop, and later we return to this below, so adoption rises because costs fall.</p>',
-    })
-    const paragraphs = collectAutoReviewParagraphs(editor)
-    const result = applyAutoReviewFindings(
-      editor,
-      paragraphs,
-      [
-        {
-          paragraph_id: 'p-1',
-          exact_text: 'Smith 2019 found a 40 percent drop',
-          annotation_type: 'source',
-          suggested_note:
-            '"40 percent drop" — does Smith 2019 actually make that finding?',
-          confidence: 0.95,
-        },
-        {
-          paragraph_id: 'p-1',
-          exact_text: 'we return to this below',
-          annotation_type: 'thread',
-          suggested_note:
-            '"return to this below" — does the promised payoff land later?',
-          confidence: 0.9,
-        },
-        {
-          paragraph_id: 'p-1',
-          exact_text: 'adoption rises because costs fall',
-          annotation_type: 'logic',
-          suggested_note:
-            'Costs fall, then adoption rises — is the linking step stated?',
-          confidence: 0.92,
-        },
-      ],
-      {
-        author: 'Agent',
-        createdAt: '2026-06-18T12:00:00.000Z',
-        idPrefix: 'test-review',
-      },
-    )
-
-    const html = editor.getHTML()
-    expect(result.applied).toHaveLength(3)
-    expect(html).toContain('data-comment-type="source"')
-    expect(html).toContain('data-comment-type="thread"')
-    expect(html).toContain('data-comment-type="logic"')
-    expect(html).toContain('data-comment-review-status="unverified"')
-    expect(html).toContain('data-comment-review-status="open"')
-    expect(html).toContain('data-comment-review-status="needs-support"')
-    editor.destroy()
-  })
-
-  it('applies copyedit and voice findings with their default open states', () => {
-    const editor = new Editor({
-      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
-      content:
-        '<p>This sentence have a mechanical error, and this passage suddenly sounds promotional.</p>',
-    })
-    const paragraphs = collectAutoReviewParagraphs(editor)
-    const result = applyAutoReviewFindings(
-      editor,
-      paragraphs,
-      [
-        {
-          paragraph_id: 'p-1',
-          exact_text: 'sentence have',
-          annotation_type: 'copyedit',
-          suggested_note: '"sentence have" → "sentence has".',
-          confidence: 0.95,
-        },
-        {
-          paragraph_id: 'p-1',
-          exact_text: 'suddenly sounds promotional',
-          annotation_type: 'voice',
-          suggested_note:
-            'This shifts more promotional than the supplied profile — intended?',
-          confidence: 0.91,
-        },
-      ],
-      {
-        author: 'Agent',
-        createdAt: '2026-06-18T12:00:00.000Z',
-        idPrefix: 'test-review',
-      },
-    )
-
-    const html = editor.getHTML()
-    expect(result.applied).toHaveLength(2)
-    expect(html).toContain('data-comment-type="copyedit"')
-    expect(html).toContain('data-comment-review-status="suggested"')
-    expect(html).toContain('data-comment-type="voice"')
-    expect(html).toContain('data-comment-review-status="needs-review"')
-    editor.destroy()
-  })
-
-  it('reports an unavailable provider instead of running local detection', async () => {
-    const editor = new Editor({
-      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
-      content:
-        '<p>EPUB 3 shipped in 2011 according to the publishing standard record.</p>',
-    })
-
-    const result = await runAutoReview(editor, {
-      createdAt: '2026-06-18T12:00:00.000Z',
-      idPrefix: 'test-review',
-    })
-
-    expect(result).toEqual(
-      expect.objectContaining({
-        paragraphCount: 1,
-        findingCount: 0,
-        applied: [],
-        skipped: 0,
-        unavailableReason: 'Editorial review provider is not configured.',
-      }),
-    )
-    expect(editor.getHTML()).not.toContain('data-comment-id=')
-    editor.destroy()
-  })
-
-  it('runs a configured findings provider and skips duplicate applications', async () => {
-    const editor = new Editor({
-      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
-      content: '<p>Small layout failures are easy to spot.</p>',
-    })
-
-    const first = await runAutoReview(editor, {
-      createdAt: '2026-06-18T12:00:00.000Z',
-      idPrefix: 'test-review',
-      findingsProvider: async () => [
-        {
-          paragraph_id: 'p-1',
-          exact_text: 'Small layout failures',
-          annotation_type: 'claim',
-          suggested_note: '"Small layout failures" — says who?',
-          confidence: 0.9,
-        },
-      ],
-    })
-    const second = await runAutoReview(editor, {
-      createdAt: '2026-06-18T12:01:00.000Z',
-      idPrefix: 'test-review',
-      findingsProvider: async () => [
-        {
-          paragraph_id: 'p-1',
-          exact_text: 'Small layout failures',
-          annotation_type: 'claim',
-          suggested_note: '"Small layout failures" — says who?',
-          confidence: 0.9,
-        },
-      ],
-    })
-
-    expect(first.applied).toHaveLength(1)
-    expect(second.applied).toHaveLength(0)
-    expect(editor.getHTML().match(/data-comment-id=/g)).toHaveLength(1)
-    editor.destroy()
-  })
-
-  it('passes local scope paragraphs to the findings provider', async () => {
-    const editor = new Editor({
-      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
-      content:
-        '<p>Small layout failures are easy to spot because footnotes can collide with running text.</p><p>Nature Communications reviewed this policy in 2026.</p>',
-    })
-    const allParagraphs = collectAutoReviewParagraphs(editor)
-    editor.commands.setTextSelection({
-      from: allParagraphs[0].from,
-      to: allParagraphs[0].to,
-    })
-    const localParagraphs = collectScopedAutoReviewParagraphs(editor, 'local')
-    const documentParagraphs = collectScopedAutoReviewParagraphs(
-      editor,
-      'document',
-    )
-
-    expect(localParagraphs).toHaveLength(1)
-    expect(documentParagraphs).toHaveLength(2)
-    const result = await runAutoReview(editor, {
-      scope: 'local',
-      findingsProvider: async ({ paragraphs }) => {
-        expect(paragraphs).toHaveLength(1)
-        return []
-      },
-    })
-    expect(result.paragraphCount).toBe(1)
-    editor.destroy()
-  })
-
-  it('parses the required JSON response shape and common aliases', () => {
-    expect(
-      parseAutoReviewFindingResponse(
-        '{"findings":[{"paragraph_id":"p-1","exact_text":"x","annotation_type":"claim"}]}',
-      ),
-    ).toEqual([
-      {
-        paragraph_id: 'p-1',
-        exact_text: 'x',
-        annotation_type: 'claim',
-      },
-    ])
-    expect(
-      parseAutoReviewFindingResponse(
-        '{"annotations":[{"paragraphId":"p-1","span":"y","type":"FACT_CHECK"}]}',
-      ),
-    ).toEqual([
-      {
-        paragraph_id: 'p-1',
-        exact_text: 'y',
-        annotation_type: 'fact_check',
-        shape: undefined,
-        subject: undefined,
-        justification: undefined,
-        suggested_note: undefined,
-        severity: undefined,
-        confidence: undefined,
-      },
-    ])
-  })
-})
+/**
+ * @vitest-environment happy-dom
+ */
+import { Editor } from '@tiptap/core'
+import StarterKit from '@tiptap/starter-kit'
+import { describe, expect, it } from 'vitest'
+import { CommentMark } from './commentMark.js'
+import {
+  applyAutoReviewFindings,
+  AutoReviewPrompts,
+  buildAutoReviewPrompt,
+  collectAutoReviewParagraphs,
+  collectScopedAutoReviewParagraphs,
+  enabledEditorialTypesForProcess,
+  parseAutoReviewFindingResponse,
+  runAutoReview,
+  validateAutoReviewFindings,
+  type AutoReviewFindingInput,
+} from './autoReviewPrompts.js'
+
+describe('AutoReviewPrompts', () => {
+  it('builds separate prompts with paragraph-only payloads', () => {
+    const paragraphs = [
+      { id: 'p-1', text: 'Evidence is limited.', from: 1, to: 21 },
+    ]
+
+    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
+      'one thing only: CLAIMS',
+    )
+    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
+      'STEP 1 — GATES',
+    )
+    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
+      'GATE A — Is it a citation or reference?',
+    )
+    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
+      'ONE SENTENCE CAN CARRY TWO CLAIMS',
+    )
+    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
+      'source it, soften it, or cut it',
+    )
+    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
+      'severity            "minor" | "notable" | "significant"',
+    )
+    expect(buildAutoReviewPrompt('claim', paragraphs)).toContain(
+      '"text": "Evidence is limited."',
+    )
+    expect(buildAutoReviewPrompt('fact-check', paragraphs)).toContain(
+      'one thing only: FACT-CHECK NEEDS',
+    )
+    expect(buildAutoReviewPrompt('fact-check', paragraphs)).toContain(
+      'You DETECT; you do not VERIFY',
+    )
+    expect(buildAutoReviewPrompt('fact-check', paragraphs)).toContain(
+      'ATOMIC SPANS',
+    )
+    expect(buildAutoReviewPrompt('fact-check', paragraphs)).toContain(
+      'severity            "minor" | "notable" | "significant"',
+    )
+    expect(buildAutoReviewPrompt('source', paragraphs)).toContain(
+      'one thing only: SOURCE checks',
+    )
+    expect(buildAutoReviewPrompt('source', paragraphs)).toContain(
+      'annotation_type     "source"',
+    )
+    expect(buildAutoReviewPrompt('thread', paragraphs)).toContain(
+      'You are detecting THREADS',
+    )
+    expect(buildAutoReviewPrompt('thread', paragraphs)).toContain(
+      'annotation_type     "thread"',
+    )
+    expect(buildAutoReviewPrompt('logic', paragraphs)).toContain(
+      'You are detecting LOGIC gaps',
+    )
+    expect(buildAutoReviewPrompt('logic', paragraphs)).toContain(
+      'annotation_type     "logic"',
+    )
+    expect(buildAutoReviewPrompt('copyedit', paragraphs)).toContain(
+      'You are the COPYEDIT pass',
+    )
+    expect(buildAutoReviewPrompt('copyedit', paragraphs)).toContain(
+      'annotation_type     "copyedit"',
+    )
+    expect(buildAutoReviewPrompt('copyedit', paragraphs)).toContain(
+      '{{HOUSE_STYLE}}',
+    )
+    expect(buildAutoReviewPrompt('voice', paragraphs)).toContain(
+      'You are the VOICE check',
+    )
+    expect(buildAutoReviewPrompt('voice', paragraphs)).toContain(
+      'annotation_type     "voice"',
+    )
+    expect(buildAutoReviewPrompt('voice', paragraphs)).toContain(
+      '{{VOICE_PROFILE}}',
+    )
+  })
+
+  it('enables the new editorial checks while keeping conflict disabled', () => {
+    expect(enabledEditorialTypesForProcess('source')).toEqual(['source'])
+    expect(enabledEditorialTypesForProcess('source-and-thread')).toEqual([
+      'source',
+      'thread',
+    ])
+    expect(enabledEditorialTypesForProcess('thread')).toEqual(['thread'])
+    expect(enabledEditorialTypesForProcess('logic')).toEqual(['logic'])
+    expect(enabledEditorialTypesForProcess('logic-and-conflict')).toEqual([
+      'logic',
+    ])
+    expect(enabledEditorialTypesForProcess('conflict')).toEqual([])
+    expect(enabledEditorialTypesForProcess('full-editorial')).toEqual([
+      'claim',
+      'fact-check',
+      'source',
+      'thread',
+    ])
+    expect(enabledEditorialTypesForProcess('copyedit')).toEqual(['copyedit'])
+    expect(enabledEditorialTypesForProcess('voice')).toEqual(['voice'])
+  })
+
+  it('collects only paragraph text from a TipTap document', () => {
+    const editor = new Editor({
+      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
+      content:
+        '<h1>Heading claim</h1><p>Small layout failures are easy to spot.</p><p></p><p>Footnotes can collide with running text.</p>',
+    })
+
+    expect(collectAutoReviewParagraphs(editor)).toEqual([
+      expect.objectContaining({
+        id: 'p-1',
+        text: 'Small layout failures are easy to spot.',
+      }),
+      expect.objectContaining({
+        id: 'p-2',
+        text: 'Footnotes can collide with running text.',
+      }),
+    ])
+    editor.destroy()
+  })
+
+  it('skips protected generated content when collecting paragraphs', () => {
+    const editor = new Editor({
+      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
+      content:
+        '<p>Body claim should be reviewed.</p><p>Reference text should not be reviewed.</p>',
+    })
+    const generated = editor.view.dom.querySelectorAll('p')[1]
+    generated.setAttribute('data-pm-generated-references', 'true')
+    generated.setAttribute('contenteditable', 'false')
+
+    expect(collectAutoReviewParagraphs(editor)).toEqual([
+      expect.objectContaining({
+        text: 'Body claim should be reviewed.',
+      }),
+    ])
+    editor.destroy()
+  })
+
+  it('validates anchors, filters low confidence, and prefers fact checks for duplicate spans', () => {
+    const paragraphs = [
+      {
+        id: 'p-1',
+        text: 'Small layout failures are easy to spot.',
+        from: 1,
+        to: 40,
+      },
+    ]
+    const findings: AutoReviewFindingInput[] = [
+      {
+        paragraph_id: 'p-1',
+        exact_text: 'Small layout failures',
+        annotation_type: 'claim',
+        shape: 'generalization',
+        suggested_note: 'Add support.',
+        severity: 'significant',
+        confidence: 0.9,
+      },
+      {
+        paragraph_id: 'p-1',
+        exact_text: 'Small layout failures',
+        annotation_type: 'fact_check',
+        shape: 'attribution',
+        suggested_note: 'Verify this statement.',
+        severity: 'notable',
+        confidence: 0.95,
+      },
+      {
+        paragraph_id: 'p-1',
+        exact_text: 'not in paragraph',
+        annotation_type: 'claim',
+        confidence: 0.95,
+      },
+      {
+        paragraph_id: 'p-1',
+        exact_text: 'easy to spot',
+        annotation_type: 'claim',
+        confidence: 0.1,
+      },
+    ]
+
+    expect(validateAutoReviewFindings(paragraphs, findings)).toEqual([
+      expect.objectContaining({
+        annotationType: 'fact-check',
+        exactText: 'Small layout failures',
+        severity: 'advisory',
+        from: 1,
+        to: 22,
+      }),
+    ])
+  })
+
+  it('normalizes realistic model finding shapes before anchoring them', () => {
+    const paragraphs = [
+      {
+        id: 'p-1',
+        text: 'The “quiet export” is a stronger path - by far.',
+        from: 1,
+        to: 48,
+      },
+    ]
+    const findings = parseAutoReviewFindingResponse(
+      JSON.stringify([
+        {
+          paragraphId: 'p-1',
+          span: 'the "quiet export" is a stronger path',
+          type: 'CLAIM',
+          note: '"stronger path" — by whose measure?',
+          severity: 'notable',
+          confidence: 0.92,
+        },
+        {
+          paragraph_id: 'p-1',
+          exactText: 'BY FAR',
+          annotationType: 'FACT_CHECK',
+          suggestedNote: '"by far" — what comparison establishes this?',
+          confidence: 0.91,
+        },
+      ]),
+    )
+
+    expect(findings).toEqual([
+      expect.objectContaining({
+        paragraph_id: 'p-1',
+        exact_text: 'the "quiet export" is a stronger path',
+        annotation_type: 'claim',
+      }),
+      expect.objectContaining({
+        paragraph_id: 'p-1',
+        exact_text: 'BY FAR',
+        annotation_type: 'fact_check',
+      }),
+    ])
+
+    expect(validateAutoReviewFindings(paragraphs, findings)).toEqual([
+      expect.objectContaining({
+        annotationType: 'claim',
+        exactText: 'The “quiet export” is a stronger path',
+        from: 1,
+        to: 38,
+      }),
+      expect.objectContaining({
+        annotationType: 'fact-check',
+        exactText: 'by far',
+        from: 41,
+        to: 47,
+      }),
+    ])
+  })
+
+  it('applies validated findings as claim and fact-check comment marks', () => {
+    const editor = new Editor({
+      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
+      content: '<p>Small layout failures are easy to spot.</p>',
+    })
+    const paragraphs = collectAutoReviewParagraphs(editor)
+    const result = applyAutoReviewFindings(
+      editor,
+      paragraphs,
+      [
+        {
+          paragraph_id: 'p-1',
+          exact_text: 'Small layout failures',
+          annotation_type: 'claim',
+          subject: 'Layout failures',
+          justification: 'This generalization needs evidence.',
+          suggested_note: 'Add citation or narrow the claim.',
+          severity: 'warning',
+          confidence: 0.91,
+        },
+      ],
+      {
+        author: 'Agent',
+        createdAt: '2026-06-18T12:00:00.000Z',
+        idPrefix: 'test-review',
+      },
+    )
+
+    expect(result.applied).toHaveLength(1)
+    expect(result.skipped).toBe(0)
+    expect(editor.getHTML()).toContain('data-comment-type="claim"')
+    expect(editor.getHTML()).toContain(
+      'data-comment-review-status="needs-support"',
+    )
+    expect(editor.getHTML()).toContain('data-comment-severity="advisory"')
+    expect(editor.getHTML()).toContain('Add citation or narrow the claim.')
+    editor.destroy()
+  })
+
+  it('applies source, thread, and logic findings with their default open states', () => {
+    const editor = new Editor({
+      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
+      content:
+        '<p>Smith 2019 found a 40 percent drop, and later we return to this below, so adoption rises because costs fall.</p>',
+    })
+    const paragraphs = collectAutoReviewParagraphs(editor)
+    const result = applyAutoReviewFindings(
+      editor,
+      paragraphs,
+      [
+        {
+          paragraph_id: 'p-1',
+          exact_text: 'Smith 2019 found a 40 percent drop',
+          annotation_type: 'source',
+          suggested_note:
+            '"40 percent drop" — does Smith 2019 actually make that finding?',
+          confidence: 0.95,
+        },
+        {
+          paragraph_id: 'p-1',
+          exact_text: 'we return to this below',
+          annotation_type: 'thread',
+          suggested_note:
+            '"return to this below" — does the promised payoff land later?',
+          confidence: 0.9,
+        },
+        {
+          paragraph_id: 'p-1',
+          exact_text: 'adoption rises because costs fall',
+          annotation_type: 'logic',
+          suggested_note:
+            'Costs fall, then adoption rises — is the linking step stated?',
+          confidence: 0.92,
+        },
+      ],
+      {
+        author: 'Agent',
+        createdAt: '2026-06-18T12:00:00.000Z',
+        idPrefix: 'test-review',
+      },
+    )
+
+    const html = editor.getHTML()
+    expect(result.applied).toHaveLength(3)
+    expect(html).toContain('data-comment-type="source"')
+    expect(html).toContain('data-comment-type="thread"')
+    expect(html).toContain('data-comment-type="logic"')
+    expect(html).toContain('data-comment-review-status="unverified"')
+    expect(html).toContain('data-comment-review-status="open"')
+    expect(html).toContain('data-comment-review-status="needs-support"')
+    editor.destroy()
+  })
+
+  it('applies copyedit and voice findings with their default open states', () => {
+    const editor = new Editor({
+      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
+      content:
+        '<p>This sentence have a mechanical error, and this passage suddenly sounds promotional.</p>',
+    })
+    const paragraphs = collectAutoReviewParagraphs(editor)
+    const result = applyAutoReviewFindings(
+      editor,
+      paragraphs,
+      [
+        {
+          paragraph_id: 'p-1',
+          exact_text: 'sentence have',
+          annotation_type: 'copyedit',
+          suggested_note: '"sentence have" → "sentence has".',
+          confidence: 0.95,
+        },
+        {
+          paragraph_id: 'p-1',
+          exact_text: 'suddenly sounds promotional',
+          annotation_type: 'voice',
+          suggested_note:
+            'This shifts more promotional than the supplied profile — intended?',
+          confidence: 0.91,
+        },
+      ],
+      {
+        author: 'Agent',
+        createdAt: '2026-06-18T12:00:00.000Z',
+        idPrefix: 'test-review',
+      },
+    )
+
+    const html = editor.getHTML()
+    expect(result.applied).toHaveLength(2)
+    expect(html).toContain('data-comment-type="copyedit"')
+    expect(html).toContain('data-comment-review-status="suggested"')
+    expect(html).toContain('data-comment-type="voice"')
+    expect(html).toContain('data-comment-review-status="needs-review"')
+    editor.destroy()
+  })
+
+  it('reports an unavailable provider instead of running local detection', async () => {
+    const editor = new Editor({
+      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
+      content:
+        '<p>EPUB 3 shipped in 2011 according to the publishing standard record.</p>',
+    })
+
+    const result = await runAutoReview(editor, {
+      createdAt: '2026-06-18T12:00:00.000Z',
+      idPrefix: 'test-review',
+    })
+
+    expect(result).toEqual(
+      expect.objectContaining({
+        paragraphCount: 1,
+        findingCount: 0,
+        applied: [],
+        skipped: 0,
+        unavailableReason: 'Editorial review provider is not configured.',
+      }),
+    )
+    expect(editor.getHTML()).not.toContain('data-comment-id=')
+    editor.destroy()
+  })
+
+  it('runs a configured findings provider and skips duplicate applications', async () => {
+    const editor = new Editor({
+      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
+      content: '<p>Small layout failures are easy to spot.</p>',
+    })
+
+    const first = await runAutoReview(editor, {
+      createdAt: '2026-06-18T12:00:00.000Z',
+      idPrefix: 'test-review',
+      findingsProvider: async () => [
+        {
+          paragraph_id: 'p-1',
+          exact_text: 'Small layout failures',
+          annotation_type: 'claim',
+          suggested_note: '"Small layout failures" — says who?',
+          confidence: 0.9,
+        },
+      ],
+    })
+    const second = await runAutoReview(editor, {
+      createdAt: '2026-06-18T12:01:00.000Z',
+      idPrefix: 'test-review',
+      findingsProvider: async () => [
+        {
+          paragraph_id: 'p-1',
+          exact_text: 'Small layout failures',
+          annotation_type: 'claim',
+          suggested_note: '"Small layout failures" — says who?',
+          confidence: 0.9,
+        },
+      ],
+    })
+
+    expect(first.applied).toHaveLength(1)
+    expect(second.applied).toHaveLength(0)
+    expect(editor.getHTML().match(/data-comment-id=/g)).toHaveLength(1)
+    editor.destroy()
+  })
+
+  it('passes local scope paragraphs to the findings provider', async () => {
+    const editor = new Editor({
+      extensions: [StarterKit, CommentMark, AutoReviewPrompts],
+      content:
+        '<p>Small layout failures are easy to spot because footnotes can collide with running text.</p><p>Nature Communications reviewed this policy in 2026.</p>',
+    })
+    const allParagraphs = collectAutoReviewParagraphs(editor)
+    editor.commands.setTextSelection({
+      from: allParagraphs[0].from,
+      to: allParagraphs[0].to,
+    })
+    const localParagraphs = collectScopedAutoReviewParagraphs(editor, 'local')
+    const documentParagraphs = collectScopedAutoReviewParagraphs(
+      editor,
+      'document',
+    )
+
+    expect(localParagraphs).toHaveLength(1)
+    expect(documentParagraphs).toHaveLength(2)
+    const result = await runAutoReview(editor, {
+      scope: 'local',
+      findingsProvider: async ({ paragraphs }) => {
+        expect(paragraphs).toHaveLength(1)
+        return []
+      },
+    })
+    expect(result.paragraphCount).toBe(1)
+    editor.destroy()
+  })
+
+  it('parses the required JSON response shape and common aliases', () => {
+    expect(
+      parseAutoReviewFindingResponse(
+        '{"findings":[{"paragraph_id":"p-1","exact_text":"x","annotation_type":"claim"}]}',
+      ),
+    ).toEqual([
+      {
+        paragraph_id: 'p-1',
+        exact_text: 'x',
+        annotation_type: 'claim',
+      },
+    ])
+    expect(
+      parseAutoReviewFindingResponse(
+        '{"annotations":[{"paragraphId":"p-1","span":"y","type":"FACT_CHECK"}]}',
+      ),
+    ).toEqual([
+      {
+        paragraph_id: 'p-1',
+        exact_text: 'y',
+        annotation_type: 'fact_check',
+        shape: undefined,
+        subject: undefined,
+        justification: undefined,
+        suggested_note: undefined,
+        severity: undefined,
+        confidence: undefined,
+      },
+    ])
+  })
+})