Skip to content

Decode HTML entities in llms.txt titles and descriptions - #974

Merged
Bohreromir merged 1 commit into
mainfrom
fix/llms-txt-html-entities
Nov 4, 2026
Merged

Bohreromir merged 1 commit into
mainfrom
fix/llms-txt-html-entities

Conversation

@Bohreromir

@Bohreromir Bohreromir commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Description

The LLMs most likely dont care, but this decodes numeric entities and the basic named ones (amp, lt, gt, quot, apos) in a single pass, so "&lt;" correctly stays the text "<"

Inside the hub it was fine, only https://docs.stackable.tech/llms.txt did show this

Docs

Release note

Checklist

Author

  • AI review done, findings addressed
  • Tests added or updated
  • Changelog updated (if the repo keeps one)

Reviewer

  • Useful comments and logging
  • Docs added or updated per the style guide

Acceptance

  • Check for release notes and docs

Asciidoctor escapes the doctitle and header attributes for HTML, so
llms.txt showed "Configuration &amp; Environment Overrides" and
"Contributor&#8217;s guide" instead of the plain characters.

Decode numeric entities and the basic named ones (amp, lt, gt, quot,
apos) in a single pass, so "&amp;lt;" correctly stays the text "&lt;".
@netlify

netlify Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for stackable-docs ready!

Name Link
🔨 Latest commit e2ea4ca
🔍 Latest deploy log https://app.netlify.com/projects/stackable-docs/deploys/6ae9c32451fce90357047530
😎 Deploy Preview https://deploy-preview-974--stackable-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Bohreromir
Bohreromir marked this pull request as ready for review November 3, 2026 11:36
@Bohreromir Bohreromir moved this to Development: Waiting for Review in Stackable Engineering Nov 3, 2026

@labrenbe labrenbe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Bohreromir
Bohreromir added this pull request to the merge queue Nov 4, 2026
@Bohreromir Bohreromir moved this from Development: Waiting for Review to Development: Done in Stackable Engineering Nov 4, 2026
Merged via the queue into main with commit ccc31fc Nov 4, 2026
7 checks passed
@Bohreromir
Bohreromir deleted the fix/llms-txt-html-entities branch November 4, 2026 07:30
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants