feat(exec): support shell interpreter

[BlackHole1]

Fixed: #4320

Once this PR has been merged, the following commands will be supported:

pnpm --recursive --shell-mode exec -- echo \"\$PNPM_PACKAGE_NAME\"

{
    "scripts": {
        "check": " pnpm --recursive --shell-mode exec -- echo \"\\$PNPM_PACKAGE_NAME\""
    }
}

[zkochan]

Does it need to be configurable or should we just make it on by default?

[BlackHole1]

it need to be configurable. Otherwise the following command will report an error:

pnpm --recursive --shell-mode exec -- node -e "console.log(process.env.PNPM_PACKAGE_NAME)"

[zkochan]

Run pnpm changeset in the root of the repository and describe your changes. The resulting files should be committed as they will be used during release.

[BlackHole1]

Run pnpm changeset in the root of the repository and describe your changes. The resulting files should be committed as they will be used during release.

Okey. Done.

[zkochan]

I wonder if Lerna uses the same solution. They claim that env variables work with exec: https://github.com/lerna/lerna/tree/main/commands/exec

[zkochan]

Yes, looks like they set it to true:

https://github.com/lerna/lerna/blob/a47fc294393a3e9507a8207a5a2f07648a524722/commands/exec/index.js#L112

[BlackHole1]

lerna (shell: always is true):

yarn (shell: always is false):

[zkochan]

OK, it makes sense to follow Yarn's approach. Does Yarn have a similar option as well?

[BlackHole1]

yarn does not provide the relevant configuration, see: https://github.com/yarnpkg/yarn/blob/6db39cf0ff684ce4e7de29669046afb8103fce3d/src/cli/commands/exec.js#L23

it makes sense to follow Yarn's approach.

By default, pnpm behaves in the same way as yarn, but it would be better if we just provided a switch on top of this.

[BlackHole1]

Maybe in the future I can also provide a cli options for yarn and make yarn support shell as well. 🪄