fix: declare uuid and fraction.js dependencies#396
Conversation
uuid is imported directly but was never listed in package.json, and fraction.js must be hoisted for autoprefixer under pnpm so CSS builds.
|
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript | Jul 9, 2026 8:25p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
|
Visit the preview URL for this PR (updated for commit 60e6e9e): https://idc-external-006--pr396-fix-missing-uuid-fra-wvnpxlaq.web.app (expires Thu, 16 Jul 2026 20:27:48 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 88aacecd98ba54d2f9c8d201a9444e43d1ad8307 |



Summary
uuidas a direct dependency (used by Header, Description, Report, and pubSubServiceInterface).fraction.jsas a devDependency so autoprefixer/postcss-preset-env can resolve it under pnpm's layout.Without these,
pnpm startfails withCan't resolve 'uuid'andCannot find module 'fraction.js'.Test plan
rm -rf node_modules && pnpm install), runREACT_APP_CONFIG=example pnpm start