Skip to content

fix: move dayjs from devDependencies to dependencies - #72

Merged
svargas-dev merged 1 commit into
masterfrom
feature/by-3082
Jul 15, 2026
Merged

fix: move dayjs from devDependencies to dependencies#72
svargas-dev merged 1 commit into
masterfrom
feature/by-3082

Conversation

@svargas-dev

Copy link
Copy Markdown
Contributor

Problem

dayjs and its plugins are imported at runtime in src/components/lib/date-utils.js but are only declared as a devDependency. This means consumers of this package cannot resolve dayjs via normal dependency resolution.

This was silently masked by pnpm v10's implicit hoisting behaviour, which leaked sub-package dev-deps into the workspace. After upgrading to pnpm v11 (which enforces strict dependency isolation), dashboard-sapper began failing immediately at startup:

Error: Cannot find module 'dayjs'
Require stack:
- apps/dashboard-sapper/__sapper__/dev/server/server.js

Fix

Move dayjs from devDependencies to dependencies and bump to 13.0.5.

dayjs is imported at runtime by date-utils.js but was declared as a
devDependency. This was masked by pnpm v10's implicit hoisting but
breaks under pnpm v11's stricter dependency isolation.
@svargas-dev
svargas-dev merged commit b3ac458 into master Jul 15, 2026
2 of 3 checks passed
@svargas-dev
svargas-dev deleted the feature/by-3082 branch July 15, 2026 11:28
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