Skip to content

Commit f1e4d5a

Browse files
committed
fix(tools): resolve deps in dev server
1 parent 9ed896f commit f1e4d5a

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

.changeset/nice-cheetahs-mix.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@patternfly/pfe-tools": patch
3+
---
4+
5+
Correctly resolve all node_modules sources in dev server

tools/pfe-tools/dev-server.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,12 @@ export function resolveLocalFilesFromTypeScriptSources(options: PfeDevServerConf
108108
return {
109109
name: 'resolve-local-monorepo-packages-from-ts-sources',
110110
async transformImport({ source, context }) {
111-
// already resolved, but had `.js` appended, probably by export map
111+
const isNodeModule = source.match(/node_modules/) || context.path.match(/node_modules/);
112112
if (source.endsWith('.ts.js')) {
113-
return source.replace('.ts.js', '.ts');
114-
// don't try to resolve node_modules, they're already resolved
115-
} else if (source.match(/node_modules/) || context.path.match(/node_modules/)) {
113+
// already resolved, but had `.js` appended, probably by export map
114+
return source.replace('.ts.js', isNodeModule ? '.js' : '.ts');
115+
} else if (isNodeModule) {
116+
// don't try to resolve node_modules, they're already resolved
116117
return;
117118
} else {
118119
const resolved = tryToResolve(source, context);

0 commit comments

Comments
 (0)