storybook production build breaks with use-lockfile-v6

[mcmxcdev]

pnpm version: 7.25.1

Code to reproduce the issue:

We recently updated to the new use-lockfile-v6 format and everything works fine except our storybook production build.

This is the stacktrace:

$ nx run zorro-ui-design:build-storybook               

> nx run zorro-ui-design:build-storybook


>  NX  ui framework: @storybook/react


>  NX  Storybook builder starting ...

info => Cleaning outputDir: /home/user/Webdevelopment/Freelancing/org/repo/dist/storybook/zorro-ui-design

 >  NX   Cannot copy `/home/user/Webdevelopment/Freelancing/org/repo/node_modules/.pnpm/@storybook+core-server@6.5.15(@storybook+builder-w_2rbwioi6ggwys4pddb257x4q6a/node_modules/@storybook/core-server/public/favicon.ico`: the file doesn't exist

   Pass --verbose to see the stacktrace.


 —————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  NX   Ran target build-storybook for project repo (8s)
 
    ✖    1/1 failed
    ✔    0/1 succeeded [0 read from cache]

I have also reported this issue on storybookjs/storybook#20673 since I am not sure which repo has to make changes here to make it work.

Expected behavior:

Storybook production build should work same as before

Actual behavior:

The build runs into "file not found" issues with favicon.ico and then probably other files as well.

Additional information:

• node -v prints: 18.12.1
• Windows, macOS, or Linux?: Ubuntu 22.10

[zkochan]

How to reproduce it?

[mcmxcdev]

In the end I was able to upgrade to storybook v7 beta and there it works without a problem.

Anyways, since not everyone will be able to do that, I have created a minimal repo: https://github.com/mcmxcdev/pnpm-nx-storybook-issue

nx run ui-design:storybook will work while the nx run ui-design:build-storybook should be breaking with the error message in the original issue description.

[bvanjoi]

error report:

reducing:

// package.json
{
  "dependencies": {
    "react": "18.2.0",
    "react-dom": "18.2.0"
  },
  "devDependencies": {
    "@storybook/builder-webpack5": "6.5.15",
    "@storybook/core-server": "6.5.15",
    "@storybook/manager-webpack5": "6.5.15"
  }
}

then pnpm i you will get following under node_modules/.pnpm:

---

It seems globby can't get expected result from source:

And with use-lockfile-v6=false, the result globby is correct, the problem may be caused by (.

[zkochan]

I see 2 possible solutions. One is to replace () with []. It fixes storybook. But there's a chance that some other tool won't like it.

Another solution would be to escape () in directory names. For instance, replace them with + or _. We already use the these chars in these directories, so it should be OK. But it will be less readable.

[bvanjoi]

I see 2 possible solutions. One is to replace () with []. It fixes storybook. But there's a chance that some other tool won't like it.

Another solution would be to escape () in directory names. For instance, replace them with + or _. We already use the these chars in these directories, so it should be OK. But it will be less readable.

I think _ is better. [] also used as glob syntax