PNPM install prints "Cannot read properties of undefined (reading 'name')"

[andschub-ms]

Verify latest release

• I verified that the issue exists in the latest pnpm release

pnpm version

9.10.0

Which area(s) of pnpm are affected? (leave empty if unsure)

No response

Link to the code that reproduces this issue or a replay of the bug

No response

Reproduction steps

pnpm install

Describe the Bug

After pnpm install says it's finished,
After postinstall hook says it's finished,
usually at the part where it prints a bunch of warnings about peer dependencies, pnpm just prints the one line:
"Cannot read properties of undefined (reading 'name')"
There is no stack trace available.

We are using @pnpm/exe

The node_modules folder is populated after install -- but I'm concerned the process may have exited prematurely.
Is there a way to set verbose logging or to debug exactly how this error is being printed?

Expected Behavior

I would normally expect to not see this error printed, and instead see a bunch of warnings about peer dependencies printed at the end.

EDIT: the issue repros in 9.10.0 but not 9.9.0, so the issue must have been introduced between versions.

Which Node.js version are you using?

Whatever is shipped with @pnpm/exe@9.10.0

Which operating systems have you used?

• macOS
• Windows
• Linux

If your OS is a Linux based, which one it is? (Include the version if relevant)

No response

[VanTanev]

I can confirm the bug. It happens on pnpm versions 9.10.0 and 9.11.0. It happens on MacOS as well.

Reproduction repo: https://github.com/VanTanev/pnpm-reproduce-8538

Here is a minimum reproduction setup:

package.json

{
    "name": "test",
    "packageManager": "pnpm@9.11.0+sha512.0a203ffaed5a3f63242cd064c8fb5892366c103e328079318f78062f24ea8c9d50bc6a47aa3567cabefd824d170e78fa2745ed1f16b132e16436146b7688f19b",
    "dependencies": {
        "@udecode/plate-ui-table": "^18.13.0",
        "@udecode/plate-ui-toolbar": "^18.13.0"
    }
}

.npmrc

auto-install-peers = false

To replicate:

$ pnpm install # while no pnpm-lock.yaml exists
$ pnpm dedupe # if pnpm-lock.yaml exists

[trydofor]

also, for monorepo, in .npmrc
shared-workspace-lockfile=false ✅ No error
shared-workspace-lockfile=true ❌ Error

[GuskiS]

Experiencing same thing on Linux/WSL2. Any ideas how to debug or get stacktrace?

[VanTanev]

Hey @zkochan,
I have bisected the breaking commit to: 96aa4bc

The root cause seems to be this:

pnpm/pkg-manager/resolve-dependencies/src/resolvePeers.ts

Lines 437 to 445 in 9cfee8f

	if (ctx.peerDependencyIssues.missing[peerName] == null) {
	ctx.peerDependencyIssues.missing[peerName] = []
	}
	const { parents } = getLocationFromParentNodeIds(ctx)
	ctx.peerDependencyIssues.missing[peerName].push({
	optional,
	parents,
	wantedRange,
	})

parents can be empty, and it seems like in that case the issue should NOT be added, so maybe it needs to be:

 const { parents } = getLocationFromParentNodeIds(ctx) 
 if (!parents.length) continue

 if (ctx.peerDependencyIssues.missing[peerName] == null) { 
   ctx.peerDependencyIssues.missing[peerName] = [] 
 } 
 ...

The error is printed by this call:

pnpm/pkg-manager/core/src/install/index.ts

Lines 1361 to 1364 in 9cfee8f

	reportPeerDependencyIssues(peerDependencyIssuesByProjects, {
	lockfileDir: opts.lockfileDir,
	strictPeerDependencies: opts.strictPeerDependencies,
	})

And here is an example of the value of the peerDependencyIssuesByProjects variable that I see in my reproduction repository:
https://gist.github.com/VanTanev/3f3325091f64d6bff447509842fa22a8 - search for "parents": []

I would contribute a PR, but I honestly don't understand the issue well enough, apologies.

[VanTanev]

Since another bug introduced by 96aa4bc was recently fixed, I just wanted to say that the bug still exists in pnpm 9.12.1 and breaks install/dedupe.

[sneridagh]

Experiencing the same here.

[sneridagh]

Still present in 9.12.2

[andschub-ms]

@zkochan @KSXGitHub FYI this bug is install-blocking and affecting several users. @VanTanev was able to provide a minimal repro and isolate the part of the code as the root cause. Our organization will be prevented from upgrading PNPM from v9.9 until a fix or workaround is discovered. Thank you for your support.

[lukewhitehouse]

I'm also seeing this in 9.12.3, although the install still seems to work from what I can tell. Downgrading to 9.9.0 seems to resolve the issue for now, as I don't want these errors to cause problems in CI.

[zkochan]

This seems to be an issue with reporting only (so install actually succeeds but the warning about missing peers is not printed). Not sure how to fix it properly right now. I guess we could just skip the missing peer dependency warning in this case. It'll still be printed but not for all occurrences.

I'll make a PR.

[zkochan]

PR: #8745

[VanTanev]

Thank you @zkochan!

[sneridagh]

@zkochan a release with the fix would be appreciated. Thanks a lot!

[zkochan]

released

[andschub-ms]

released

Thank you for your support!