[manuzhang]

Add automatic HTML comment removal from all feed item descriptions to ensure clean content in RSS and JSON feeds. Comments are stripped from all content sources including markdown, page.meta descriptions, and full page content.

• Add strip_html_comments() static method to Util class
• Update get_description_or_abstract() to strip comments in all code paths
• Handle both inline and multiline HTML comments
• Preserve content structure while removing comment artifacts

This prevents HTML comments (including TODO notes, analytics codes, and other internal annotations) from appearing in published feeds.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[manuzhang]

@Guts Please help review, thanks!

[Guts]

Pfiou huge delay here, sorry!

Thanks for the PR, stripping internal comments from feed descriptions is a useful feature. The regex approach works for the simple cases, but there are two edge cases in the truncation branch (chars_count > 0), because the markdown is cut before rendering and the comments are only stripped afterwards.

1. Comment leaks when truncation falls inside it. An unclosed <!-- is escaped by Python-Markdown to <!--, so the regex no longer matches:

md = "Début. <!-- note interne très longue qui sera coupée -->\n\nFin."
# chars_count=25 -> '<p>Début. &lt;!-- note inter...</p>'

The internal note ends up in the feed, which is what this PR aims to prevent.

2. Comments eat the character length. With an 80-character comment at the top of a page and chars_count=50, the output is only <!-- xxxx... and contains no visible text.

Both would be fixed by stripping comments from in_page.markdown before computing lengths and cutting. Please make sure the excerpt delimiter (<!-- more --> by default) is still located first. Comments inside fenced code blocks would then be stripped too, so that case should be handled or documented.

Other points:

• tests are missing
• strip_html_comments() is called in six places. Computing the result in the branches and stripping once would be more concise. The regex could also be compiled once at module level.

Can you also rebase please?

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.34%. Comparing base (8e06121) to head (aabb23c).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #421      +/-   ##
==========================================
+ Coverage   83.58%   84.34%   +0.75%     
==========================================
  Files          11       11              
  Lines         786      792       +6     
  Branches      133      132       -1     
==========================================
+ Hits          657      668      +11     
+ Misses         84       81       -3     
+ Partials       45       43       -2     

Flag	Coverage Δ
unittests	84.34% <100.00%> (+0.75%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
mkdocs_rss_plugin/util.py	77.58% <100.00%> (+2.30%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.