fix: add /index.js to dayjs ESM import remap

[rsbh]

Summary

• Rollup output paths rewrites dayjs / dayjs/plugin/* → dayjs/esm* in ESM builds, but emitted bare paths like dayjs/esm/plugin/customParseFormat without extension.
• dayjs has no exports map for esm subpaths, and plugin paths are directories (customParseFormat/index.js). Strict ESM resolvers (Node nodenext, host apps with "type": "module") fail to resolve directory imports without an explicit path.
• Append /index.js in the remap so output references the actual file. Works in Bun, Node strict ESM, Vite SSR, and bundlers alike.

Test plan

• pnpm build — ESM dist imports resolve to dayjs/esm/index.js and dayjs/esm/plugin/<name>/index.js
• CJS dist unchanged (require('dayjs'), require('dayjs/plugin/<name>'))
• pnpm test — 1663/1663 pass
• Verify consumer host app with "type": "module" (Bun/Next) imports DatePicker/RangePicker/DataTable date filters without resolution errors

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
apsara	Ready	Preview, Comment	Apr 23, 2026 4:28am

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5280f1bc-36ec-4bcf-8f20-11ec3f5f07ca

📥 Commits

Reviewing files that changed from the base of the PR and between 3997059 and 6139a62.

📒 Files selected for processing (1)

• packages/raystack/rollup.config.mjs

---

📝 Walkthrough

Walkthrough

This change updates the Rollup configuration to explicitly specify module entry paths for the dayjs library. The modification directs imports of dayjs to resolve to dayjs/esm/index.js instead of the directory path dayjs/esm. Additionally, plugin imports matching the dayjs/plugin/* pattern are updated to include the explicit index.js segment, resolving to dayjs/esm/plugin/*/index.js. This ensures more precise module resolution during the build process.

Suggested reviewers

• rohanchkrabrty
• paanSinghCoder

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding /index.js to dayjs ESM import remapping in the Rollup configuration.
Description check	✅ Passed	The description provides clear context about the problem, solution, and test plan, all directly related to the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}