accordproject / markdown-transform / 26867278460

79%

push

github

fix(table): handle HTML table captions correctly (#640)

* fix(table): handle HTML table captions correctly

Signed-off-by: Abdelrahman Emad <abdoemad0120@gmail.com>

* fix(html): add table caption regression tests

Signed-off-by: Abdelrahman Emad <abdoemad0120@gmail.com>

* fix(html): keep table caption CommonMark valid

Address Copilot review feedback on PR #640:

- Flatten <caption> content to inline-only nodes before wrapping in
  Strong, so flow content (e.g. nested <p>, lists) can no longer become
  invalid block children of a Strong node.
- Handle <caption> inside the table branch instead of a standalone rule,
  so a caption appearing outside a table no longer leaks a node without
  a $class into the output.
- Emit the bolded caption as its own Paragraph block and drop the inline
  '\n\n' Text hack, leaving block-level spacing to the Markdown serializer.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Signed-off-by: Matt Roberts <code@rbrts.uk>

---------

Signed-off-by: Abdelrahman Emad <abdoemad0120@gmail.com>
Signed-off-by: Matt Roberts <code@rbrts.uk>
Co-authored-by: Matt Roberts <code@rbrts.uk>
Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

820 of 1221 branches covered (67.16%)

Branch coverage included in aggregate %.

100 of 104 new or added lines in 2 files covered. (96.15%)

2077 of 2736 relevant lines covered (75.91%)

977.06 hits per line