Shopify / Shopify/shopify-app-template-react-router
.npmrc ships pnpm-only shamefully-hoist=true — npm warns on every command
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 197
- Forks
- 68
- Avg merge
- 4d 21h
- Merged PRs (30d)
- 4
Description
Every npm command in a fresh app from this template prints:
npm warn Unknown project config "shamefully-hoist". This will stop working in the next major version of npm.
The template's .npmrc sets shamefully-hoist=true, which is a pnpm-specific option — npm doesn't recognise it, warns on every single command, and the warning says the next npm major will stop tolerating unknown config keys entirely.
Suggestion: move the setting to pnpm-workspace.yaml, which pnpm reads for settings — same behaviour for pnpm users, silence for npm users, and no breakage when npm's next major lands. Happy to open the PR if that direction works for you.
Seen with npm 11.9.0 / Node 24 on Windows 11.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect the template's .npmrc and the proposed pnpm-workspace.yaml location for the shamefully-hoist setting. Run an npm command in a fresh app to confirm the warning, then verify pnpm retains the intended behavior. Done means npm is silent while pnpm still uses the setting.
Written by the indexing model from the issue text.
Assessment
- Domain
- build-system, tooling
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 76/100