Skip to content

Keep autolinks opaque while matching outer brackets - #321

Merged
dereuromark merged 2 commits into
masterfrom
fix/opaque-autolink-brackets
Oct 18, 2026
Merged

dereuromark merged 2 commits into
masterfrom
fix/opaque-autolink-brackets

Conversation

@dereuromark

@dereuromark dereuromark commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Skip a complete autolink while matching brackets for an outer link. A ] inside <https://x]> belongs to the autolink URL.

Use the existing autolink validator so an invalid <...> sequence does not hide a bracket closer.

Verification

  • Added nested URL and quoted-email autolink regressions, plus an invalid-angle control.
  • PHPUnit: 4,606 tests, 31,186 assertions passed.
  • PHPCS passed for both changed files.

@dereuromark dereuromark added bug Something isn't working skip-changelog labels Oct 18, 2026
@codecov

codecov Bot commented Oct 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.16%. Comparing base (5942f63) to head (aacb38d).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #321      +/-   ##
============================================
+ Coverage     93.15%   93.16%   +0.01%     
- Complexity     4785     4787       +2     
============================================
  Files           118      118              
  Lines         12748    12753       +5     
============================================
+ Hits          11875    11881       +6     
+ Misses          873      872       -1     

☔ View full report in Codecov by Harness.
📢 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.

@dereuromark
dereuromark merged commit 5cd45f2 into master Oct 18, 2026
6 checks passed
@dereuromark
dereuromark deleted the fix/opaque-autolink-brackets branch October 18, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant