Skip to content

support conflict block - #1104

Open
andreisakirkinepam wants to merge 2 commits into
epam:mainfrom
andreisakirkinepam:1075-support-conflict-block
Open

support conflict block#1104
andreisakirkinepam wants to merge 2 commits into
epam:mainfrom
andreisakirkinepam:1075-support-conflict-block

Conversation

@andreisakirkinepam

Copy link
Copy Markdown
Contributor

No description provided.

@AlexeyGirin AlexeyGirin linked an issue May 7, 2026 that may be closed by this pull request
@AlexeyGirin

Copy link
Copy Markdown
Collaborator

@copilot resolve the merge conflicts in this pull request

@AlexeyGirin
AlexeyGirin requested a review from Copilot May 20, 2026 09:05
@AlexeyGirin

Copy link
Copy Markdown
Collaborator

@copilot resolve the merge conflicts in this pull request

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds frontend support for “conflict” extraction blocks in the Extraction Editor by surfacing conflict styling and enabling switching between multiple OCR variants that share a conflict group.

Changes:

  • Add conflict-aware rendering in ExtractionBlock (conflict highlighting + variant tab switcher + visibility toggling).
  • Extend TipTap schema to preserve <span> nodes/attributes from hOCR so conflict target/comment spans can be styled.
  • Add mock hOCR content and tests covering conflict block highlighting and conflict variant tab switching.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
web/frontend/src/mocks/data/extractions/conflict_tabs.hocr Adds a mock hOCR sample containing a conflict group with multiple variants.
web/frontend/src/features/workspace/components/extraction-editor.tsx Removes now-unused conditional editability logic (customTab-based).
web/frontend/src/features/workspace/components/extraction-editor.test.tsx Adds tests for conflict highlighting and conflict-variant tab switching behavior.
web/frontend/src/features/workspace/components/extraction-editor-extensions.ts Introduces an OcrSpan TipTap node to preserve span structure/attrs from hOCR.
web/frontend/src/features/workspace/components/extraction-block.tsx Implements conflict rendering, variant discovery, and tab-driven variant visibility.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.

Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
Comment thread web/frontend/src/features/workspace/components/extraction-block.tsx
@AlexeyGirin

Copy link
Copy Markdown
Collaborator

@khyurri please have a look

@AlexeyGirin

Copy link
Copy Markdown
Collaborator

@copilot resolve the merge conflicts in this pull request

@andreisakirkinepam
andreisakirkinepam force-pushed the 1075-support-conflict-block branch from 45ef2b3 to 2828213 Compare June 11, 2026 11:49

@sem0ark sem0ark left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Retested from the previous issue, bug with removal is fixed, approved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Chat] Support conflict block

4 participants