Use package.json#exports

[Andarist]

No description provided.

[changeset-bot]

🦋 Changeset detected

Latest commit: f2112b3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@preconstruct/cli	Minor
@preconstruct/eslint-plugin-format-js-tag	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (58df49f) 92.19% compared to head (ae3c481) 92.19%.

❗ Current head ae3c481 differs from pull request most recent head f2112b3. Consider uploading reports for the commit f2112b3 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #565   +/-   ##
=======================================
  Coverage   92.19%   92.19%           
=======================================
  Files          40       40           
  Lines        1973     1973           
  Branches      595      595           
=======================================
  Hits         1819     1819           
  Misses        144      144           
  Partials       10       10           

Impacted Files	Coverage Δ
packages/cli/src/rollup-plugins/prod-dev-entry.ts	100.00% <ø> (ø)
.../src/rollup-plugins/inline-process-env-node-env.ts	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[Andarist]

is this even needed now? I'm not sure what this was about

[emmatown]

It's because some babel plugins might rely on process.env.NODE_ENV and Preconstruct is basically always meant for building for "production"

[emmatown]

I'm assuming the main reason to do this is to avoid the dev vs prod bundle so it's a bit easier to patch, yeah? it's not like the package.json#exports field really matters here?

[Andarist]

I'm assuming the main reason to do this is to avoid the dev vs prod bundle so it's a bit easier to patch, yeah?

Yeah, exactly.