Skip to content

Update main and types fields in package.json #59

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kearfy
Copy link
Member

@kearfy kearfy commented Jul 15, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings July 15, 2025 08:04
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates the package entry points to use the embedded build outputs.

  • Changed the main field to point to lib/embedded.js
  • Updated the types field to point to lib/embedded.d.ts
Comments suppressed due to low confidence (2)

package.json:13

  • [nitpick] Consider updating the README or other documentation to reflect the new default entry point (lib/embedded.js), so that code examples using require or import still work as expected.
    "main": "lib/embedded.js",

package.json:14

  • If you’re using the files field or .npmignore to control packaged files, ensure lib/embedded.js and lib/embedded.d.ts are included so the package’s entry points aren’t omitted from npm releases.
    "types": "lib/embedded.d.ts",

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