fix: handle null from pnp resolver

src/parse-tsconfig/resolve-extends.ts

@@ -59,13 +59,13 @@ export function resolveExtends(
 
 	const pnpApi = getPnpApi();
 	if (pnpApi) {
-		const { resolveRequest } = pnpApi;
+		const { resolveRequest: resolveWithPnp } = pnpApi;
 		const [first, second] = filePath.split('/');
 		const packageName = first.startsWith('@') ? `${first}/${second}` : first;
 
 		try {
 			if (packageName === filePath) {
-				const packageJsonPath = resolveRequest(
+				const packageJsonPath = resolveWithPnp(
 					path.join(packageName, 'package.json'),
 					directoryPath,
 				);
@@ -78,18 +78,23 @@ export function resolveExtends(
 					}
 				}
 			} else {
+				let resolved: string | null;
 				try {
-					return resolveRequest(
+					resolved = resolveWithPnp(
 						filePath,
 						directoryPath,
 						{ extensions: ['.json'] },
 					);
 				} catch {
-					return resolveRequest(
+					resolved = resolveWithPnp(
 						path.join(filePath, 'tsconfig.json'),
 						directoryPath,
 					);
 				}
+
+				if (resolved) {
+					return resolved;
+				}
 			}
 		} catch {}
 	}

tests/fixtures/yarn-pnp/index.js

@@ -4,13 +4,15 @@ const tests = [
 	() => parseTsconfig('./tsconfig.package-path.json'),
 	() => parseTsconfig('./tsconfig.package-path-directory.json'),
 	() => parseTsconfig('./tsconfig.org-package.json'),
+	() => parseTsconfig('./tsconfig.missing-extends.json'),
+	() => parseTsconfig('./tsconfig.invalid-extends.json'),
 ];
 
 for (const test of tests) {
 	try {
 		console.log(test());
 	} catch (error) {
-		console.log(error.message);
+		console.log('Error:', error.message);
 		process.exitCode = 1;
 	}
 }

tests/fixtures/yarn-pnp/tsconfig.invalid-extends.json

@@ -0,0 +1,3 @@
+{
+	"extends": "fs/promises"
+}

tests/fixtures/yarn-pnp/tsconfig.missing-extends.json

@@ -0,0 +1,3 @@
+{
+	"extends": "non-existent-package"
+}

tests/specs/parse-tsconfig/extends/resolves.spec.ts

@@ -364,13 +364,16 @@ export default testSuite(({ describe }) => {
 			const { stdout } = await execaNode('./index.js', [], {
 				nodeOptions: ['--require', './.pnp.cjs'],
 				cwd: './tests/fixtures/yarn-pnp',
+				reject: false,
 			});
 
 			expect(stdout).toBe([
 				'{ compilerOptions: { strict: true, jsx: \'react\' } }',
 				'{ compilerOptions: { strict: true, jsx: \'react\' } }',
 				'{ compilerOptions: { strict: true, jsx: \'react\' } }',
 				'{ compilerOptions: { strict: true, jsx: \'react\' } }',
+				'Error: File \'non-existent-package\' not found.',
+				'Error: File \'fs/promises\' not found.',
 			].join('\n'));
 		});
 	});