Skip to content

fix(build): correct docs workspace filter and add offline emoji-data …#52

Open
shaayar wants to merge 1 commit intoonce-ui-system:mainfrom
shaayar:fix/correct-docs-workspace-filter-and-add-offline-emoji-data-fallback
Open

fix(build): correct docs workspace filter and add offline emoji-data …#52
shaayar wants to merge 1 commit intoonce-ui-system:mainfrom
shaayar:fix/correct-docs-workspace-filter-and-add-offline-emoji-data-fallback

Conversation

@shaayar
Copy link
Copy Markdown

@shaayar shaayar commented Apr 1, 2026

Summary

  • fix root monorepo build filter to target the actual docs workspace package
  • make emoji data generation resilient in offline/restricted network environments
  • keep existing emoji-data.json as fallback when remote fetch is unavailable

Changes

  • updated core/package.json
    • build script changed from:
      • turbo build --filter=docs...
    • to:
      • turbo build --filter=@once-ui-system/docs...
  • updated core/packages/core/scripts/generate-emoji-data.js
    • validate HTTP status code when fetching remote emoji data
    • fallback to local src/data/emoji-data.json if fetch fails
    • only fail hard when remote fetch fails and no local fallback exists

Why

  • pnpm build in core/ failed because docs is not a workspace package name
  • build also failed in offline/restricted environments due to remote fetch dependency on raw.githubusercontent.com

Validation

  • ran pnpm --dir core build
  • confirmed @once-ui-system/core now proceeds past emoji fetch failures using local fallback
  • confirmed docs build now reaches Next.js build phase (remaining external network dependency is Google Fonts)

Notes

  • engine warning persists (node: 20.x expected, local is v24.13.0)
  • docs app still requires network access for next/font/google unless moved to local fonts

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026

@shaayar is attempting to deploy a commit to the Dopler Team on Vercel.

A member of the Team first needs to authorize it.

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