fix: reject null named catalogs in workspace manifest reader (#11231)

.changeset/gentle-hats-doubt.md

@@ -0,0 +1,6 @@
+---
+"@pnpm/workspace.workspace-manifest-reader": patch
+"pnpm": patch
+---
+
+Reject `null` named catalogs in workspace manifests with `InvalidWorkspaceManifestError` instead of crashing with a raw `TypeError`.

workspace/workspace-manifest-reader/src/catalogs.ts

@@ -46,6 +46,10 @@ export function assertValidWorkspaceManifestCatalogs (manifest: { packages?: rea
       throw new InvalidWorkspaceManifestError(`Expected named catalog ${catalogName} to be an object, but found - array`)
     }
 
+    if (catalog === null) {
+      throw new InvalidWorkspaceManifestError(`Expected named catalog ${catalogName} to be an object, but found - null`)
+    }
+
     if (typeof catalog !== 'object') {
       throw new InvalidWorkspaceManifestError(`Expected named catalog ${catalogName} to be an object, but found - ${typeof catalog}`)
     }

workspace/workspace-manifest-reader/test/__fixtures__/catalogs-invalid-named-catalog-null/pnpm-workspace.yaml

@@ -0,0 +1,6 @@
+packages:
+  - "packages/**"
+  - "types"
+
+catalogs:
+  foo: null

workspace/workspace-manifest-reader/test/index.ts

@@ -133,6 +133,15 @@ describe('readWorkspaceManifest() catalogs field', () => {
     ).rejects.toThrow('Expected named catalog foo to be an object, but found - number')
   })
 
+  test('throws on null named catalog', async () => {
+    await expect(
+      readWorkspaceManifest(path.join(import.meta.dirname, '__fixtures__/catalogs-invalid-named-catalog-null'))
+    ).rejects.toMatchObject({
+      code: 'ERR_PNPM_INVALID_WORKSPACE_CONFIGURATION',
+      message: expect.stringContaining('Expected named catalog foo to be an object, but found - null'),
+    })
+  })
+
   test('throws on invalid named catalog specifier', async () => {
     await expect(
       readWorkspaceManifest(path.join(import.meta.dirname, '__fixtures__/catalogs-invalid-named-catalog-specifier'))