Skip to content

Conversation

@jaalah-akamai
Copy link
Contributor

Summary

  • Removed Storybook: Removed Storybook and all React dependencies as they were not maintained and introduced significant dependencies that needed to be managed.
  • Updated Node.js requirement: Upgraded from Node 18.18.0 to Node 22.0.0 (LTS) to support style-dictionary v5
  • Major dependency updates:
    • style-dictionary: 4.3.05.1.1 (breaking changes addressed)
    • vitest: 3.2.44.0.16 (breaking changes verified)
    • @tokens-studio/sd-transforms: 1.2.02.0.3
  • Updated all devDependencies: All development dependencies updated to their latest compatible versions
  • Removed legacy resolutions: Removed unnecessary resolutions section from package.json

Why

This PR modernizes the project's dependency stack and removes unused tooling:

  1. Style-dictionary v5 compatibility: Style-dictionary v5 requires Node.js v22.0.0+ and introduces breaking changes around token references and reference syntax. This update ensures compatibility with the latest version.

  2. Removed Storybook: Storybook and React dependencies were removed as they are no longer used in this repository. The src/stories/ directory and all Storybook configuration files have been removed. This significantly simplifies the devDependencies and makes the project easier for developers to maintain. Design tokens are available through Cloud Manager's dev tools, making the Storybook setup redundant.

  3. Dependency maintenance: All devDependencies have been updated to their latest versions to benefit from bug fixes, security patches, and performance improvements.

  4. Cleanup: Removed legacy resolutions that were no longer necessary, simplifying dependency management.

Breaking Changes

Node.js Version Requirement

  • Minimum Node.js version: 18.18.022.0.0 (LTS)
  • Developers must upgrade to Node.js 22.0.0 or later
  • Updated volta configuration to reflect this requirement

Style-dictionary v5 Breaking Changes (Addressed)

  • ✅ Token references: All token references in the codebase use the correct syntax (no .value suffix)
  • ✅ Reference syntax: No custom reference syntax options found - using DTCG-compliant defaults
  • ✅ Transform hooks: Custom transform (spacingPxToRem) is compatible with new error handling

Vitest v4 Breaking Changes (Verified)

  • ✅ Test file uses only basic APIs (expect, describe, it) - no breaking changes
  • ✅ No Vitest config file - using defaults which are compatible
  • ✅ All tests passing with v4

Test Plan

  1. Verify Node.js version: node --version Should be 22.0.0 or later
  2. Install dependencies: yarn install
  3. Run tests: yarn test Expected: All 8 tests in spacingPxToRem.test.ts should pass
  4. Verify build: yarn generate Expected: Token files should be generated successfully in dist/ directory
  5. Verify no Storybook references

Files Changed

  • package.json: Updated dependencies, removed Storybook/React deps, removed resolutions, updated Node version
  • yarn.lock: Regenerated with updated dependencies
  • Removed: All .storybook/ configuration files
  • Removed: All src/stories/ files
  • Updated: dist/ files (regenerated with style-dictionary v5)

@jaalah-akamai
Copy link
Contributor Author

fyi: @abailly-akamai

@jaalah-akamai jaalah-akamai changed the base branch from main to staging December 26, 2025 20:11
@jaalah-akamai jaalah-akamai merged commit c6ec8f4 into linode:staging Dec 26, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants