Skip to content

Commit f65f467

Browse files
committed
fix(core): add workspace path validation to getNxRequirePaths if the workspace is different then we should not include it.
1 parent ff98c24 commit f65f467

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

packages/nx/src/config/schema-utils.ts

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { resolve as resolveExports } from 'resolve.exports';
44
import { getWorkspacePackagesMetadata } from '../plugins/js/utils/packages';
55
import { registerPluginTSTranspiler } from '../project-graph/plugins';
66
import { normalizePath } from '../utils/path';
7+
import { workspaceRootInner } from '../utils/workspace-root';
78
import type { ProjectConfiguration } from './workspace-json-project-json';
89

910
/**
@@ -97,6 +98,18 @@ export function resolveImplementation(
9798
const resolved = require.resolve(maybeImplementation, {
9899
paths: [directory],
99100
});
101+
console.log(`Resolved via require.resolve: ${resolved}`);
102+
console.log(`Checking against directory: ${directory}`);
103+
const resolvedWorkspaceRoot = workspaceRootInner(resolved, null);
104+
const expectedWorkspaceRoot = workspaceRootInner(directory, null);
105+
106+
if (resolvedWorkspaceRoot !== expectedWorkspaceRoot) {
107+
console.log(
108+
`Skipping resolved path from different workspace: ${resolved} (workspace: ${resolvedWorkspaceRoot}) vs expected (${expectedWorkspaceRoot})`
109+
);
110+
continue;
111+
}
112+
100113
console.log(`Resolved via require.resolve: ${resolved}`);
101114
return resolved;
102115
} catch (e) {

0 commit comments

Comments
 (0)