Skip to content

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

Open
Bohreromir wants to merge 1 commit into
mainfrom
fix/llms-txt-html-entities
Open

Bohreromir wants to merge 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

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Waiting for Review

Development

Successfully merging this pull request may close these issues.

2 participants