Merge pull request #4027 from storybooks/core/presets

Presets support

app/angular/src/server/framework-preset-angular-cli.js

@@ -0,0 +1,17 @@
+import { logger } from '@storybook/node-logger';
+
+import {
+  getAngularCliWebpackConfigOptions,
+  applyAngularCliWebpackConfig,
+} from './angular-cli_config';
+
+export function webpack(config) {
+  const cwd = process.cwd();
+  const cliWebpackConfigOptions = getAngularCliWebpackConfigOptions(cwd);
+
+  if (cliWebpackConfigOptions) {
+    logger.info('=> Loading angular-cli config.');
+  }
+
+  return applyAngularCliWebpackConfig(config, cliWebpackConfigOptions);
+}

app/angular/src/server/framework-preset-angular.js

@@ -0,0 +1,50 @@
+import path from 'path';
+import { ContextReplacementPlugin } from 'webpack';
+import loadTsConfig from './ts_config';
+
+export function webpack(config, { configDir }) {
+  return {
+    ...config,
+    module: {
+      ...config.module,
+      rules: [
+        ...config.module.rules,
+        {
+          test: /\.tsx?$/,
+          use: [
+            {
+              loader: require.resolve('ts-loader'),
+              options: loadTsConfig(configDir),
+            },
+            require.resolve('angular2-template-loader'),
+          ],
+        },
+        {
+          test: /[/\\]@angular[/\\]core[/\\].+\.js$/,
+          parser: { system: true },
+        },
+        {
+          test: /\.html$/,
+          loader: 'raw-loader',
+          exclude: /\.async\.html$/,
+        },
+        {
+          test: /\.scss$/,
+          use: [require.resolve('raw-loader'), require.resolve('sass-loader')],
+        },
+      ],
+    },
+    resolve: {
+      ...config.resolve,
+      extensions: [...config.resolve.extensions, '.ts', '.tsx'],
+    },
+    plugins: [
+      ...config.plugins,
+      // See https://github.com/angular/angular/issues/11580#issuecomment-401127742
+      new ContextReplacementPlugin(
+        /@angular(\\|\/)core(\\|\/)fesm5/,
+        path.resolve(__dirname, '..')
+      ),
+    ],
+  };
+}

app/angular/src/server/options.js

@@ -1,24 +1,10 @@
-import { logger } from '@storybook/node-logger';
-
 import packageJson from '../../package.json';
 
-import wrapInitialConfig from './wrapInitialConfig';
-
-import {
-  getAngularCliWebpackConfigOptions,
-  applyAngularCliWebpackConfig,
-} from './angular-cli_config';
-
-const cliWebpackConfigOptions = getAngularCliWebpackConfigOptions(process.cwd());
-
-if (cliWebpackConfigOptions) {
-  logger.info('=> Loading angular-cli config.');
-}
-
 export default {
   packageJson,
   defaultConfigName: 'angular-cli',
-  wrapInitialConfig,
-  wrapDefaultConfig: config => applyAngularCliWebpackConfig(config, cliWebpackConfigOptions),
-  wrapBasicConfig: config => applyAngularCliWebpackConfig(config, cliWebpackConfigOptions),
+  frameworkPresets: [
+    require.resolve('./framework-preset-angular.js'),
+    require.resolve('./framework-preset-angular-cli.js'),
+  ],
 };

app/html/src/server/framework-preset-html.js

@@ -0,0 +1,19 @@
+export function webpack(config) {
+  return {
+    ...config,
+    module: {
+      ...config.module,
+      rules: [
+        ...config.module.rules,
+        {
+          test: /\.html$/,
+          use: [
+            {
+              loader: require.resolve('html-loader'),
+            },
+          ],
+        },
+      ],
+    },
+  };
+}

app/html/src/server/options.js

@@ -1,8 +1,6 @@
 import packageJson from '../../package.json';
 
-import wrapInitialConfig from './wrapInitialConfig';
-
 export default {
   packageJson,
-  wrapInitialConfig,
+  frameworkPresets: [require.resolve('./framework-preset-html.js')],
 };

app/marko/src/server/framework-preset-marko.js

@@ -0,0 +1,19 @@
+export function webpack(config) {
+  return {
+    ...config,
+    module: {
+      ...config.module,
+      rules: [
+        ...config.module.rules,
+        {
+          test: /\.marko$/,
+          loader: require.resolve('marko-loader'),
+        },
+      ],
+    },
+    resolve: {
+      ...config.resolve,
+      extensions: [...config.resolve.extensions, '.marko'],
+    },
+  };
+}

app/marko/src/server/options.js

@@ -1,8 +1,6 @@
 import packageJson from '../../package.json';
 
-import wrapInitialConfig from './wrapInitialConfig';
-
 export default {
   packageJson,
-  wrapInitialConfig,
+  frameworkPresets: [require.resolve('./framework-preset-marko.js')],
 };

app/mithril/src/server/framework-preset-mithril.js

@@ -0,0 +1,9 @@
+import { mergeBabel } from '@storybook/core/server';
+
+export function babel(config) {
+  const patch = {
+    plugins: [require.resolve('@babel/plugin-transform-react-jsx')],
+  };
+
+  return mergeBabel(patch, config);
+}

app/mithril/src/server/options.js

@@ -1,8 +1,6 @@
 import packageJson from '../../package.json';
 
-import wrapDefaultBabelConfig from './wrapDefaultBabelConfig';
-
 export default {
   packageJson,
-  wrapDefaultBabelConfig,
+  frameworkPresets: [require.resolve('./framework-preset-mithril.js')],
 };

app/polymer/src/server/framework-preset-polymer.js

@@ -0,0 +1,28 @@
+import { IgnorePlugin } from 'webpack';
+
+export function webpack(config) {
+  return {
+    ...config,
+    module: {
+      ...config.module,
+      rules: [
+        ...config.module.rules,
+        {
+          test: /\.html$/,
+          use: [
+            ...config.module.rules[0].use,
+            {
+              loader: require.resolve('polymer-webpack-loader'),
+              options: { processStyleLinks: true },
+            },
+          ],
+        },
+      ],
+    },
+    plugins: [
+      ...config.plugins,
+      // See https://github.com/webcomponents/webcomponentsjs/issues/794#issuecomment-386554298
+      new IgnorePlugin(/^vertx$/),
+    ],
+  };
+}

app/polymer/src/server/options.js

@@ -1,8 +1,6 @@
 import packageJson from '../../package.json';
 
-import wrapInitialConfig from './wrapInitialConfig';
-
 export default {
   packageJson,
-  wrapInitialConfig,
+  frameworkPresets: [require.resolve('./framework-preset-polymer.js')],
 };

app/react/src/server/wrapBabelConfig.js → app/react/src/server/framework-preset-react-docgen.js

@@ -1,4 +1,4 @@
-export default function(config) {
+export function babel(config) {
   // Ensure plugins are defined or fallback to an array to avoid empty values.
   const babelConfigPlugins = config.plugins || [];
 

app/react/src/server/wrapBabelConfig.test.js → app/react/src/server/framework-preset-react-docgen.test.js

@@ -1,6 +1,6 @@
-import wrapBabelConfig from './wrapBabelConfig';
+import * as preset from './framework-preset-react-docgen';
 
-describe('babel_config', () => {
+describe('framework-preset-react-docgen', () => {
   const babelPluginReactDocgenPath = require.resolve('babel-plugin-react-docgen');
 
   it('should return the config with the extra plugins when `plugins` is an array.', () => {
@@ -10,7 +10,7 @@ describe('babel_config', () => {
       plugins: ['foo-plugin'],
     };
 
-    const config = wrapBabelConfig(babelConfig);
+    const config = preset.babel(babelConfig);
 
     expect(config).toEqual({
       babelrc: false,
@@ -34,7 +34,7 @@ describe('babel_config', () => {
       plugins: 'bar-plugin',
     };
 
-    const config = wrapBabelConfig(babelConfig);
+    const config = preset.babel(babelConfig);
 
     expect(config).toEqual({
       babelrc: false,
@@ -57,7 +57,7 @@ describe('babel_config', () => {
       presets: ['env', 'foo-preset'],
     };
 
-    const config = wrapBabelConfig(babelConfig);
+    const config = preset.babel(babelConfig);
 
     expect(config).toEqual({
       babelrc: false,

app/react/src/server/framework-preset-react.js

@@ -0,0 +1,9 @@
+import { mergeBabel } from '@storybook/core/server';
+
+export function babel(config) {
+  const patch = {
+    presets: [require.resolve('@babel/preset-react'), require.resolve('@babel/preset-flow')],
+  };
+
+  return mergeBabel(patch, config);
+}

app/react/src/server/options.js

@@ -1,11 +1,10 @@
 import packageJson from '../../package.json';
 
-import wrapBabelConfig from './wrapBabelConfig';
-import wrapDefaultBabelConfig from './wrapDefaultBabelConfig';
-
 export default {
   packageJson,
   defaultConfigName: 'create-react-app',
-  wrapDefaultBabelConfig,
-  wrapBabelConfig,
+  frameworkPresets: [
+    require.resolve('./framework-preset-react.js'),
+    require.resolve('./framework-preset-react-docgen.js'),
+  ],
 };