Skip to content

fix(satteri): reset heading slugs between renders - #18046

Merged
Princesseuh merged 1 commit into
mainfrom
fix/18045-satteri-heading-slugger-reset
Sep 17, 2026
Merged

Princesseuh merged 1 commit into
mainfrom
fix/18045-satteri-heading-slugger-reset

Conversation

@Princesseuh

@Princesseuh Princesseuh commented Sep 17, 2026 •

Copy link
Copy Markdown
Member

Changes

As suggested by @PFiS1737, we need to reset the slugger between invocations to make sure slugs are unique per document and not per build.

Fixes #18045

Testing

Added a test

Documentation

N/A

@changeset-bot

changeset-bot Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4cfe216

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@astrojs/markdown-satteri Patch
astro Patch
@astrojs/mdx Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions Bot added the feat: markdown Related to Markdown (scope) label Sep 17, 2026
Co-authored-by: PFiS1737 <69380732+PFiS1737@users.noreply.github.com>
@Princesseuh
Princesseuh force-pushed the fix/18045-satteri-heading-slugger-reset branch from 940ac58 to 4cfe216 Compare September 17, 2026 16:25
@Princesseuh
Princesseuh merged commit 8358d59 into main Sep 17, 2026
24 checks passed
@Princesseuh
Princesseuh deleted the fix/18045-satteri-heading-slugger-reset branch September 17, 2026 16:37
@astrobot-houston astrobot-houston mentioned this pull request Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat: markdown Related to Markdown (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] satteriHeadingIdsPlugin uses a non-reset Slugger instance across multiple posts

2 participants