Skip to content

Commit a498c47

Browse files
Move bundles out of js folder (#31681)
1 parent e5a32c3 commit a498c47

26 files changed

+34
-21
lines changed

packages/devextreme-themebuilder/src/metadata/dependency-collector.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,12 @@ export default class DependencyCollector {
9898
es6: { mixedImports: true },
9999
ts: { skipTypeImports : true }
100100
});
101+
const isBundleFile = filePath.includes('bundle-templates');
101102

102-
const deps = result.map((relativeDependency: string): string => {
103+
const deps = result.map((dependency: string): string => {
104+
const relativeDependency = isBundleFile
105+
? dependency.replace(/^((?:\.\.\/)+)(?!bundles)/, '$1../js/')
106+
: dependency;
103107
let absDepPath = relativeDependency
104108

105109
if (relativeDependency.startsWith('.')) {
@@ -176,7 +180,7 @@ export default class DependencyCollector {
176180
}
177181

178182
collect(): void {
179-
const fullDependencyTree = this.getFullDependencyTree(path.resolve(__dirname, '../../../devextreme/js/bundles/dx.all.js'));
183+
const fullDependencyTree = this.getFullDependencyTree(path.resolve(__dirname, '../../../devextreme/build/bundle-templates/dx.all.js'));
180184

181185
this.treeProcessor(fullDependencyTree);
182186
this.validate();

packages/devextreme-themebuilder/tests/metadata/dependency-collector.test.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,9 @@ const filesContent: { [key: string]: string } = {
105105
[path.resolve(__dirname, '../../../devextreme-scss/scss/widgets/lesstheme/_index.scss')]: '// public widgets\n@use "./toolbar";',
106106

107107
// collect test
108-
[path.resolve(__dirname, '../../../devextreme/js/bundles/dx.all.js')]: 'import t from \'./toolbar\';import b from \'./button\';',
108+
[path.resolve(__dirname, '../../../devextreme/build/bundle-templates/dx.all.js')]: 'import t from \'../toolbar\';import b from \'../button\';',
109+
'../../js/toolbar.js': 'import m from \'./menu\';import u from \'./utils\';\n// STYLE toolbar',
110+
'../../js/button.js': 'import u from \'./utils\';\n// STYLE button',
109111
[path.resolve(__dirname, '../../../devextreme-scss/scss/widgets/generic/_index.scss')]: '// public widgets\n@use "./toolbar";@use "./button";@use "./icon";@use "./menu";',
110112
[path.resolve(__dirname, '../../../devextreme-scss/scss/widgets/material/_index.scss')]: '// public widgets\n@use "./toolbar";@use "./button";@use "./icon";@use "./menu";',
111113
};
@@ -133,7 +135,7 @@ jest.mock('fs', () => ({
133135
jest.mock('filing-cabinet', () => ({
134136
__esModule: true,
135137
default: (options: cabinet.Options): string => {
136-
const normalizedPartial = options.partial.replace('./', '');
138+
const normalizedPartial = options.partial.replace(/^\.\//, '');
137139
return `${normalizedPartial}.${tsFilesSet.has(normalizedPartial) ? 'ts' : 'js'}`;
138140
},
139141
}));
@@ -293,3 +295,4 @@ describe('Integration test', () => {
293295
expect(builtDependenciesCopy).toEqual(idealDependenciesCopy);
294296
});
295297
});
298+

packages/devextreme/js/bundles/dx.ai-integration.js renamed to packages/devextreme/build/bundle-templates/dx.ai-integration.js

File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

packages/devextreme/js/bundles/modules/common.charts.js renamed to packages/devextreme/build/bundle-templates/modules/common.charts.js

File renamed without changes.
File renamed without changes.

packages/devextreme/js/bundles/modules/core.legacy.js renamed to packages/devextreme/build/bundle-templates/modules/core.legacy.js

File renamed without changes.
File renamed without changes.

0 commit comments

Comments
 (0)