Skip to content

Fix: markdown table double extraction in parser#13892

Merged
asiroliu merged 2 commits intoinfiniflow:mainfrom
6ba3i:fix/duplicate_mdtables
Apr 2, 2026
Merged

Fix: markdown table double extraction in parser#13892
asiroliu merged 2 commits intoinfiniflow:mainfrom
6ba3i:fix/duplicate_mdtables

Conversation

@6ba3i
Copy link
Copy Markdown
Contributor

@6ba3i 6ba3i commented Apr 2, 2026

What problem does this PR solve?

Fixes markdown tables being parsed twice (once as markdown and again as generated HTML), which caused duplicate table chunks in the chunk list UI.

Type of change

  • Bug Fix (non-breaking change which fixes an issue)

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. 🐞 bug Something isn't working, pull request that fix bug. labels Apr 2, 2026
@6ba3i 6ba3i added the ci Continue Integration label Apr 2, 2026
@6ba3i 6ba3i marked this pull request as draft April 2, 2026 02:25
@6ba3i 6ba3i marked this pull request as ready for review April 2, 2026 02:25
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.52%. Comparing base (af40be6) to head (f64d214).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13892      +/-   ##
==========================================
- Coverage   98.11%   96.52%   -1.60%     
==========================================
  Files          10       10              
  Lines         690      690              
  Branches      108      108              
==========================================
- Hits          677      666      -11     
- Misses          4        8       +4     
- Partials        9       16       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Apr 2, 2026
@asiroliu asiroliu merged commit dd52913 into infiniflow:main Apr 2, 2026
1 check passed
@6ba3i 6ba3i deleted the fix/duplicate_mdtables branch April 2, 2026 05:33
sirj0k3r pushed a commit to sirj0k3r/ragflow that referenced this pull request Apr 2, 2026
### What problem does this PR solve?

Fixes markdown tables being parsed twice (once as markdown and again as
generated HTML), which caused duplicate table chunks in the chunk list
UI.

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug Something isn't working, pull request that fix bug. ci Continue Integration size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants