Add ES module support by specifying import path in package.json

[narekhovhannisyan]

Motivation

Changes

• Change 1
• Change 2

How to test

• Test 1
• Test 2

Images and GIFs

Before	After
link1	link2
link3	link4
link5	link6
link7	link8
link9	link10

Summary by CodeRabbit

• Chores
  • Simplified package exports to expose a single public entry point, streamlining module imports and improving compatibility.
  • Removed the separate declaration export mapping — this may affect TypeScript declaration resolution in some setups; consumers relying on separate declaration exports should verify their build or editor behavior.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The package.json exports map for "." was simplified: the previous object with "require" and "types" entries was replaced by a single string pointing to "./dist/index.js", removing the explicit "types": "./dist/index.d.ts" export.

Changes

Cohort / File(s)	Summary
Exports simplification package.json	Replaced the "." export object ({ "require": "./dist/index.js", "types": "./dist/index.d.ts" }) with a single string export "./dist/index.js", removing the "types" mapping under "."

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Review focus: confirm intended TypeScript declaration exposure (removal of "types" under ".") and that consumers relying on that mapping are unaffected.

Possibly related PRs

• Enhance package configuration with module exports and TypeScript settings #81 — Modified the same package.json "." exports (added/changed require and types) and is directly related.
• Fix #82 #97 — Changes to build/publish ensuring dist/index.js is produced, which this export references.

Suggested reviewers

• IgorDobryn
• VladimirTaytor
• mklocek

Poem

🐇 I hopped through package.json tonight,
Trimmed a map to keep the path light,
One dot, one file, tidy and neat,
Dist-bound footsteps soft and fleet. 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title claims to add ES module support by specifying import path, but the actual change removes the types path from the main export and simplifies the export structure, which doesn't align with adding ES module support.	Update the title to accurately reflect the actual change, such as 'Simplify main export in package.json to single entry point' or provide clarification on how this enables ES module support.
Description check	⚠️ Warning	The pull request description is a template with placeholder content (Change 1, Change 2, Test 1, Test 2, and dummy image links) that has not been filled in with actual details about the changes, motivation, or testing steps.	Replace placeholder content with actual motivation, specific changes made, detailed testing instructions, and relevant before/after images or remove unused sections.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-import-issue

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a5f1421 and 7703427.

📒 Files selected for processing (1)

• package.json (1 hunks)

---

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.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c72c713 and a5f1421.

📒 Files selected for processing (1)

• package.json (1 hunks)

🔇 Additional comments (1)

package.json (1)

43-50: Verify export field configuration aligns with package resolution standards.

The "import" field has been added, which enables ESM consumers to import this package. However, ensure that:

1. The build output (dist/index.js) is actually transpiled to ESM format by the TypeScript compiler
2. If both "import" and "require" must point to the same file, confirm it's in a universally compatible format
3. Consider whether separate CJS and ESM builds might be needed in the future for better tree-shaking and optimization

You can verify the actual module format by inspecting the generated dist/index.js file after running yarn build.

[narekhovhannisyan]

@coderabbitai check

[coderabbitai]

@narekhovhannisyan: I'll review the changes in this PR for you.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[narekhovhannisyan]

@coderabbitai check

[coderabbitai]

@narekhovhannisyan: I'll review the changes in this PR for you.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.