Skip to content

Commit 7495275

Browse files
refactor: optimized engines directory retrieval #1359 (#1364)
* refactor: replacing existing way to retrieve the actual node_directory especially for npm workspaces usage (aka npm 7 monorepos) * refactor: outputting the engines directory path as well for debugging * fix: we'll still need the process.cwd entry especially for development in the PL codebase itself Co-authored-by: Josef Bredreck <[email protected]> Co-authored-by: Josef Bredreck <[email protected]>
1 parent 8255872 commit 7495275

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

packages/core/src/lib/pattern_engines.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@ const engineMatcher = /^engine-(.*)$/;
99

1010
const logger = require('./log');
1111

12+
const { resolvePackageFolder } = require('@pattern-lab/core/src/lib/resolver');
13+
1214
const enginesDirectories = [
1315
{
1416
displayName: 'the core',
@@ -18,6 +20,10 @@ const enginesDirectories = [
1820
displayName: 'the edition or test directory',
1921
path: path.join(process.cwd(), 'node_modules'),
2022
},
23+
{
24+
displayName: 'the general node_modules directory',
25+
path: path.resolve(resolvePackageFolder('@pattern-lab/core'), '..', '..'),
26+
},
2127
];
2228

2329
/**
@@ -87,7 +93,9 @@ const PatternEngines = Object.create({
8793
engineDirectory.path
8894
);
8995

90-
logger.debug(`Loading engines from ${engineDirectory.displayName}...`);
96+
logger.debug(
97+
`Loading engines from ${engineDirectory.displayName}: ${engineDirectory.path} ...`
98+
);
9199

92100
// find all engine-named things in this directory and try to load them,
93101
// unless it's already been loaded.

0 commit comments

Comments
 (0)