Skip to content

fix(tokens): use dynamic import for prettier in writeJson#709

Merged
GarthDB merged 2 commits intomainfrom
fix/tokens-prettier-dynamic-import
Feb 26, 2026
Merged

fix(tokens): use dynamic import for prettier in writeJson#709
GarthDB merged 2 commits intomainfrom
fix/tokens-prettier-dynamic-import

Conversation

@GarthDB
Copy link
Member

@GarthDB GarthDB commented Feb 26, 2026

Fixes ERR_MODULE_NOT_FOUND: Cannot find package 'prettier' when @adobe/spectrum-tokens is used outside the monorepo (e.g. via npx @adobe/spectrum-design-data-mcp).

  • Removes top-level import { format } from "prettier" from packages/tokens/index.js
  • Uses dynamic await import("prettier") inside writeJson so the module loads without prettier installed
  • MCP and other consumers that only use getTokensByFile / getAllTokens no longer fail on load

Made with Cursor

@changeset-bot
Copy link

changeset-bot bot commented Feb 26, 2026

🦋 Changeset detected

Latest commit: 548c3f5

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

This PR includes changesets to release 5 packages
Name Type
@adobe/spectrum-tokens Patch
s2-tokens-viewer Patch
markdown-generator Patch
@adobe/spectrum-design-data-mcp Patch
token-csv-generator 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

@GarthDB
Copy link
Member Author

GarthDB commented Feb 26, 2026

Run report for 548c3f5a

Total time: 45.2s | Comparison time: 45.2s | Estimated loss: 56.1ms (0.1% slower)

Action Time Status Info
⬛️ SetupProto(0.51.4) 0ms Skipped
🟩 SyncWorkspace 6.8ms Passed
🟩 SyncProject(tokens) 0.3ms Passed
🟩 SyncProject(markdown-generator) 0.3ms Passed
🟩 SyncProject(viewer) 0.3ms Passed
🟩 SyncProject(site) 0.3ms Passed
⬛️ SetupToolchain(node:20.17.0) 1.6s Skipped
🟩 InstallWorkspaceDeps(node:20.17.0) 8.2s Passed
🟩 RunTask(tokens:buildManifest) 276.3ms Passed
🟩 RunTask(site:generateToolsPage) 875.7ms Passed
🟩 RunTask(design-system-registry:test) 1.6s Passed
🟩 RunTask(componentSchemas:test) 3.2s Passed
🟩 RunTask(tokens:buildTokens) 32.5s Passed
🟩 RunTask(tokens:build) 13.7ms Passed
🟩 RunTask(viewer:prepare) 137ms Passed
🟩 RunTask(viewer:export) 62.9ms Passed
🟩 RunTask(markdown-generator:generate) 288ms Passed
🟩 RunTask(site:copyContent) 81.5ms Passed
🟩 RunTask(site:build) 2.5s Passed
🟩 RunTask(site:export) 8ms Passed
Touched files
.changeset/fix-prettier-dynamic-import.md
packages/tokens/index.js

@GarthDB GarthDB merged commit 49ad47b into main Feb 26, 2026
3 checks passed
@GarthDB GarthDB deleted the fix/tokens-prettier-dynamic-import branch February 26, 2026 19:44
@GarthDB GarthDB mentioned this pull request Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant