Ensure build is run on prepack

[frederikprijck]

Description

Ensure to build before running npm pack or npm publish.

References

See: #151 (comment)

Checklist

• I have added documentation for new/changed functionality in this PR or in auth0.com/docs
• All active GitHub checks for tests, formatting, and security are passing
• The correct base branch is being used, if not the default branch

[frederikprijck]

Force merging this to fix the broken beta release.

[jonkoops]

Any particular reason why prepack was used rather than prepublishOnly? The reason I am asking is that prepack will not run on npm publish, so I imagine the publishing process does not make use of this command?

[frederikprijck]

It does, we use npm publish (see: https://github.com/auth0/jwt-decode/blob/beta/.circleci/config.yml#L39), and prepack solved it (it's what's we use on all SDKs, no particular reason). See https://docs.npmjs.com/cli/v9/using-npm/scripts#life-cycle-operation-order

[frederikprijck]

The main benefit is it also runs on npm pack, which we use to try the tarball locally, so just a little bit of convenience over prePublishOnly

[jonkoops]

Strange, when I run npm publish --dry-run locally, the prepack step never runs:

I'm on NPM 9.8.0, not sure if this has changed between versions.

[frederikprijck]

The page mentions that prepare isnt executed when adding --dry-run. Prepack is done before, and postpack after prepare. So i assume its expected with --dry-run.

[jonkoops]

I'd still expect it to run either way, even if I omit --dry-run I am still not seeing prepack being run. But perhaps it's something about my NPM setup 🤷‍♂️