fix(publish): use package.json from --contents

[aleclarson]

Description

This PR lets any Command subclass intercept the Package loader by defining a redirectPackages method on itself. The publish command uses this to ensure --contents is respected when re-serializing any package.json modules before calling npm publish.

Motivation and Context

See here: #2113 (comment)

How Has This Been Tested?

I tested this manually on the monorepo for react-spring.

I don't have time to write any tests, currently.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[evocateur]

The sheer number of test failures indicates that this is not the appropriate fix. There's not even a test to demonstrate the case you're trying to account for. I would expect most of the fix to be in and around packDirectory().

[aleclarson]

The failed tests are mostly to do with --canary (which I don't use or understand). The other failed tests are just checking the wrong package.json for changes.

Could you take over this PR for me? 😅 Or will it be much more involved than I expect?

[aleclarson]

Note: I won't have time in the near future to properly finish this PR. I'm using patch-package in my monorepo for now.

[evocateur]

Closing in favor of b922766