diff --git a/packages/cli/src/migrations/rules/build-dist-docs.ts b/packages/cli/src/migrations/rules/build-dist-docs.ts index 1e68ff84576..837661a1682 100644 --- a/packages/cli/src/migrations/rules/build-dist-docs.ts +++ b/packages/cli/src/migrations/rules/build-dist-docs.ts @@ -60,7 +60,7 @@ export const buildDistDocsRule: MigrationRule = { if (isTrue) { if (propName === 'buildDist') { - targetTypesNeedingSkipInDev.push('dist-custom-elements', 'dist-hydrate-script'); + targetTypesNeedingSkipInDev.push('dist', 'dist-custom-elements', 'dist-hydrate-script'); } else if (propName === 'buildDocs') { targetTypesNeedingSkipInDev.push( 'docs-readme', diff --git a/packages/core/src/compiler/config/_test_/validate-output-dist.spec.ts b/packages/core/src/compiler/config/_test_/validate-output-dist.spec.ts index 1fb1c5067ca..34cb77e9241 100644 --- a/packages/core/src/compiler/config/_test_/validate-output-dist.spec.ts +++ b/packages/core/src/compiler/config/_test_/validate-output-dist.spec.ts @@ -39,8 +39,9 @@ describe('validateDistOutputTarget', () => { typesDir: join(rootDir, 'my-dist', 'types'), transformAliasedImportPathsInCollection: true, isPrimaryPackageOutputTarget: false, - skipInDev: false, + skipInDev: true, }, + { esmDir: join(rootDir, 'my-dist', 'my-build', 'testing'), empty: false, @@ -57,40 +58,6 @@ describe('validateDistOutputTarget', () => { file: join(rootDir, 'my-dist', 'my-build', 'testing', 'testing.css'), type: 'dist-global-styles', }, - { - dir: join(rootDir, 'my-dist'), - type: 'dist-types', - typesDir: join(rootDir, 'my-dist', 'types'), - }, - { - collectionDir: join(rootDir, 'my-dist', 'collection'), - dir: join(rootDir, '/my-dist'), - empty: false, - transformAliasedImportPaths: true, - type: 'dist-collection', - }, - { - copy: [{ src: '**/*.svg' }, { src: '**/*.js' }], - copyAssets: 'collection', - dir: join(rootDir, 'my-dist', 'collection'), - type: 'copy', - }, - { - type: 'dist-lazy', - cjsDir: join(rootDir, 'my-dist', 'cjs'), - cjsIndexFile: join(rootDir, 'my-dist', 'index.cjs.js'), - empty: false, - esmDir: join(rootDir, 'my-dist', 'esm'), - esmIndexFile: join(rootDir, 'my-dist', 'index.js'), - }, - { - cjsDir: join(rootDir, 'my-dist', 'cjs'), - componentDts: join(rootDir, 'my-dist', 'types', 'components.d.ts'), - dir: join(rootDir, 'my-dist', 'loader'), - empty: false, - esmDir: join(rootDir, 'my-dist', 'esm'), - type: 'dist-lazy-loader', - }, ]); }); @@ -135,7 +102,7 @@ describe('validateDistOutputTarget', () => { typesDir: join(rootDir, 'my-dist', 'types'), transformAliasedImportPathsInCollection: true, isPrimaryPackageOutputTarget: false, - skipInDev: false, + skipInDev: true, }, { esmDir: join(rootDir, 'my-dist', 'my-build', 'testing'), @@ -153,40 +120,6 @@ describe('validateDistOutputTarget', () => { file: join(rootDir, 'my-dist', 'my-build', 'testing', 'testing.css'), type: 'dist-global-styles', }, - { - dir: join(rootDir, 'my-dist'), - type: 'dist-types', - typesDir: join(rootDir, 'my-dist', 'types'), - }, - { - collectionDir: join(rootDir, 'my-dist', 'collection'), - dir: join(rootDir, '/my-dist'), - empty: false, - transformAliasedImportPaths: true, - type: 'dist-collection', - }, - { - copy: [{ src: '**/*.svg' }, { src: '**/*.js' }], - copyAssets: 'collection', - dir: join(rootDir, 'my-dist', 'collection'), - type: 'copy', - }, - { - type: 'dist-lazy', - cjsDir: join(rootDir, 'my-dist', 'cjs'), - cjsIndexFile: join(rootDir, 'my-dist', 'index.cjs.js'), - empty: false, - esmDir: join(rootDir, 'my-dist', 'esm'), - esmIndexFile: join(rootDir, 'my-dist', 'index.js'), - }, - { - cjsDir: join(rootDir, 'my-dist', 'cjs'), - componentDts: join(rootDir, 'my-dist', 'types', 'components.d.ts'), - dir: join(rootDir, 'my-dist', 'loader'), - empty: false, - esmDir: join(rootDir, 'my-dist', 'esm'), - type: 'dist-lazy-loader', - }, ]); }); @@ -215,7 +148,7 @@ describe('validateDistOutputTarget', () => { typesDir: join(rootDir, 'my-dist', 'types'), transformAliasedImportPathsInCollection: true, isPrimaryPackageOutputTarget: true, - skipInDev: false, + skipInDev: true, }, { esmDir: join(rootDir, 'my-dist', 'my-build', 'testing'), @@ -233,40 +166,6 @@ describe('validateDistOutputTarget', () => { file: join(rootDir, 'my-dist', 'my-build', 'testing', 'testing.css'), type: 'dist-global-styles', }, - { - dir: join(rootDir, 'my-dist'), - type: 'dist-types', - typesDir: join(rootDir, 'my-dist', 'types'), - }, - { - collectionDir: join(rootDir, 'my-dist', 'collection'), - dir: join(rootDir, '/my-dist'), - empty: false, - transformAliasedImportPaths: true, - type: 'dist-collection', - }, - { - copy: [{ src: '**/*.svg' }, { src: '**/*.js' }], - copyAssets: 'collection', - dir: join(rootDir, 'my-dist', 'collection'), - type: 'copy', - }, - { - type: 'dist-lazy', - cjsDir: join(rootDir, 'my-dist', 'cjs'), - cjsIndexFile: join(rootDir, 'my-dist', 'index.cjs.js'), - empty: false, - esmDir: join(rootDir, 'my-dist', 'esm'), - esmIndexFile: join(rootDir, 'my-dist', 'index.js'), - }, - { - cjsDir: join(rootDir, 'my-dist', 'cjs'), - componentDts: join(rootDir, 'my-dist', 'types', 'components.d.ts'), - dir: join(rootDir, 'my-dist', 'loader'), - empty: false, - esmDir: join(rootDir, 'my-dist', 'esm'), - type: 'dist-lazy-loader', - }, ]); }); }); diff --git a/packages/core/src/compiler/config/outputs/validate-dist.ts b/packages/core/src/compiler/config/outputs/validate-dist.ts index b428949bff4..6e631b18380 100644 --- a/packages/core/src/compiler/config/outputs/validate-dist.ts +++ b/packages/core/src/compiler/config/outputs/validate-dist.ts @@ -145,8 +145,8 @@ const validateOutputTargetDist = ( : true, isPrimaryPackageOutputTarget: o.isPrimaryPackageOutputTarget ?? false, cjs: isBoolean(o.cjs) ? o.cjs : false, - // dist always builds by default (skipInDev: false) - skipInDev: isBoolean(o.skipInDev) ? o.skipInDev : false, + // dist skips distribution artifacts in dev mode by default, but always builds browser/CDN output + skipInDev: isBoolean(o.skipInDev) ? o.skipInDev : true, } satisfies Required; if (!isAbsolute(outputTarget.buildDir)) { diff --git a/packages/core/src/compiler/output-targets/_test_/output-lazy-loader.spec.ts b/packages/core/src/compiler/output-targets/_test_/output-lazy-loader.spec.ts index 831fa0177c3..28b5798ce9d 100644 --- a/packages/core/src/compiler/output-targets/_test_/output-lazy-loader.spec.ts +++ b/packages/core/src/compiler/output-targets/_test_/output-lazy-loader.spec.ts @@ -23,6 +23,7 @@ function setup(configOverrides: Partial = {}) { type: DIST, dir: 'my-test-dir', cjs: true, + skipInDev: false, }, ], srcDir: '/src',