Skip to content

fix(mdx): declare satteri as a direct dependency#17372

Open
grant wants to merge 1 commit into
withastro:mainfrom
grant:fix/mdx-declare-satteri-dependency
Open

fix(mdx): declare satteri as a direct dependency#17372
grant wants to merge 1 commit into
withastro:mainfrom
grant:fix/mdx-declare-satteri-dependency

Conversation

@grant

@grant grant commented Jul 13, 2026

Copy link
Copy Markdown

Changes

  • Moves satteri from @astrojs/mdx devDependencies to dependencies so the bare import … from 'satteri' in the Sätteri MDX entrypoints is declared for consumers
  • Adds a changeset

@astrojs/mdx statically imports satteri from src/satteri/*, but previously relied on it only transitively via the optional @astrojs/markdown-satteri peer (or a devDependency that is not published to consumers). Under pnpm’s isolated node_modules, that can fail with ERR_MODULE_NOT_FOUND for the bare satteri import during astro build of MDX pages.

Fixes #17371

Testing

@github-actions github-actions Bot added the pkg: integration Related to any renderer integration (scope) label Jul 13, 2026
@grant

grant commented Jul 14, 2026

Copy link
Copy Markdown
Author

@astrobot-houston

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

Labels

pkg: integration Related to any renderer integration (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@astrojs/mdx imports satteri but does not declare it (pnpm ERR_MODULE_NOT_FOUND)

1 participant