[LindseyZ1205]

The inline walker of the Markdown backend handles marko.inline.Link, but not marko.inline.AutoLink. AutoLink is not a subclass of Link. So an angle-bracket autolink (<https://url>) and a GFM bare URL (gfm_el.Url, a subclass of AutoLink) lose their hyperlink in paragraphs, list items and headings, and export as plain text. The table cell walker already handles Link | AutoLink since #4488. This change uses the same check in the inline walker.

Issue resolved by this Pull Request:
Resolves #4790

Checklist:

• Documentation has been updated, if necessary.
• Examples have been added, if necessary.
• Tests have been added, if necessary.

Review evidence:

• Before: in a paragraph, a list item and a heading, <https://example.com>, https://example.com and www.example.com gave a TextItem without hyperlink, and the export had no link.
• After: these items have the hyperlink, and the export is [https://example.com](https://example.com/). www.example.com gets http://www.example.com/, as the GFM spec defines for this form. Explicit links and table cells give the same output as before.
• Document tree: only the hyperlink field of the text items that come from autolinks changes.
• Reference data: no change. The Markdown ground truth files do not contain autolinks, and all tests in tests/test_backend_markdown.py pass with the existing ground truth.
• New test: test_autolinks_outside_table_cells checks the hyperlinks of autolinks in a heading, a paragraph and a list item. It fails on main and passes with this change.
• Commands and results:
  • uv run --frozen pytest tests/test_backend_markdown.py: 35 passed.
  • uv run --frozen pytest tests/test_backend_markdown.py tests/test_backend_text_encoding.py tests/test_backend_optional_dependencies.py tests/test_input_doc.py tests/test_invalid_input.py tests/test_pr_topic_labels.py: the results are the same before and after this change, apart from the new test. 8 tests fail in my environment in both runs, because some optional packages (docling-parse, python-docx, pylatexenc) are not installed.
  • uvx prek run --files docling/backend/md_backend.py tests/test_backend_markdown.py: all hooks pass (Ruff, ty, Tach, max lines).
• Not run: the full test suite.

[github-actions]

✅ DCO Check Passed

Thanks @LindseyZ1205, all your commits are properly signed off. 🎉

[mergify]

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

[github-actions]

AI triage (advisory)

Checked commit ca815d24e8. The analysis read the diff only. It did not run code from this pull request.

The Markdown backend now keeps the hyperlink of angle-bracket autolinks and GFM bare URLs in paragraphs, list items and headings. The PR adds one test for this.

Topic labels added: bug, markdown.

CI safety: no concern found. A maintainer must still approve CI.

1 related pull request(s) in the same code

PR	State	Signal	Assessment
#4488	merged	referenced as #4488	related: PR 4488 is already merged. It added autolink handling in table cells. This PR applies the same check to the inline walker. The candidate diff file is empty, so this verdict uses the PR description.

_{The ai:* labels are advisory. They do not approve CI and they do not add tests:full. A new push runs this check again.}

[github-actions]

AI first review (advisory): no blocker found

This review covers the full diff at ca815d24e8. It read the code and did not run it.

No blocking problem found. The base code already handles marko.inline.AutoLink in table cells (line 517) with the same check. gfm_el.Url subclasses AutoLink in the installed marko, and both classes set dest. The regression test would fail on base because no hyperlink would be set.

_{A maintainer makes the merge decision. This review never approves the pull request.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[dolfim-ibm]

lgtm