Angular 6 Expected 'styles' to be an array of strings. #3593

wuyuanyi135 · 2018-05-16T06:16:39Z

If you are reporting a bug or requesting support, start here:

Bug or support request summary

I want to use storybook with Angular 6. It turns out I have to use the 4.0.0 alpha branch to make it through (#3044 ). After I managed to make the story server run, I got another error Expected 'styles' to be an array of strings., which I have no idea how to get around.

Steps to reproduce

I followed the instructions in https://storybook.js.org/basics/guide-angular/#docs-content but at the beginning the compilation failed:

node_modules/webpack/lib/DefinePlugin.js:42
		compiler.hooks.compilation.tap(
		               ^

TypeError: Cannot read property 'compilation' of undefined

Then I installed @storybook/angular@4.0.0-alpha.6. I managed to run the server, but when I load my component, it shows

Expected 'styles' to be an array of strings.
            Error: Expected 'styles' to be an array of strings.
    at assertArrayOfStrings (http://localhost:9001/static/preview.bundle.js:8372:19)
    at CompileMetadataResolver../node_modules/@angular/compiler/fesm5/compiler.js.CompileMetadataResolver.getNonNormalizedDirectiveMetadata (http://localhost:9001/static/preview.bundle.js:17242:13)
    at CompileMetadataResolver../node_modules/@angular/compiler/fesm5/compiler.js.CompileMetadataResolver._getEntryComponentMetadata (http://localhost:9001/static/preview.bundle.js:17887:28)
    at http://localhost:9001/static/preview.bundle.js:17536:53
    at Array.map (<anonymous>)
    at CompileMetadataResolver../node_modules/@angular/compiler/fesm5/compiler.js.CompileMetadataResolver.getNgModuleMetadata (http://localhost:9001/static/preview.bundle.js:17536:18)
    at JitCompiler../node_modules/@angular/compiler/fesm5/compiler.js.JitCompiler._loadModules (http://localhost:9001/static/preview.bundle.js:29551:51)
    at JitCompiler../node_modules/@angular/compiler/fesm5/compiler.js.JitCompiler._compileModuleAndComponents (http://localhost:9001/static/preview.bundle.js:29532:36)
    at JitCompiler../node_modules/@angular/compiler/fesm5/compiler.js.JitCompiler.compileModuleAsync (http://localhost:9001/static/preview.bundle.js:29492:37)
    at CompilerImpl../node_modules/@angular/platform-browser-dynamic/fesm5/platform-browser-dynamic.js.CompilerImpl.compileModuleAsync (http://localhost:9001/static/preview.bundle.js:54699:31)

Please specify which version of Storybook and optionally any affected addons that you're running

"@ storybook/angular": "4.0.0-alpha.6",

The text was updated successfully, but these errors were encountered:

wuyuanyi135 · 2018-05-19T04:23:15Z

This is an update for this issue.

In alpha.6, the aforementioned issues can be resolved by extending the webpack.config.json in .storybook directory:
note that the magic is the to-string-loader that adapt the css to what Angular 6 want.

const path = require("path");

module.exports = (baseConfig) => {
  // Extend defaultConfig as you need.

  // For example, add typescript loader:
  baseConfig.module.rules.push({
    test: [/\.stories\.tsx?$/, /index\.ts$/],
    loaders: [
      {
        loader: require.resolve('@storybook/addon-storysource/loader'),
        options: {
          parser: 'typescript',
        },
      },
    ],
    include: [path.resolve(__dirname, '../src')],
    enforce: 'pre',
  });

  baseConfig.module.rules.push({
    test: /\.css$/,
    use: ['to-string-loader', 'css-loader']
  });
  return baseConfig;
};

After getting this config file, the error becomes to failed to load XXXX.html. It seems like storybook still want to load the template page even though it has been bundled. To resolve this issue, add moduleId = module.id to each of the @Component in your Angular components.
I don't really like this workaround. Thankfully, the failed to load html problem seems to be fixed in alpha.7. But still the css files need the to-string-loader to work.

nglazov · 2018-05-19T05:20:37Z

It fixes "Expected 'styles' to be an array of strings." error, but works not correct.
Components from angular-material (eg form controls) looks bad in this configuration.

wuyuanyi135 · 2018-05-19T23:50:25Z

@nglazov I suppose that will be taken care of by the dev teams. :) Really hope it to work soon.

enten · 2018-05-27T18:36:09Z

Does interface NgStory correct? How about templateUrl and stylesUrl?

// app/angular/src/client/preview/angular/types.ts

export interface NgStory {
  component?: any;
  props: ICollection;
  propsMeta?: ICollection;
  moduleMetadata?: NgModuleMetadata;
  template?: string;
  styles?: string[];
}

Should we update initModule too? It doesn't seem to handle components which use templateUrl or styleUrls.

// app/angular/src/client/preview/angular/helpers.ts

const initModule = (currentStory: IGetStory): Function => {
  const storyObj = currentStory();
  const { component, template, props, styles, moduleMetadata = {} } = storyObj;

  let AnnotatedComponent;

  if (template) {
    AnnotatedComponent = createComponentFromTemplate(template, styles);
  } else {
    AnnotatedComponent = component;
  }

  const story = {
    component: AnnotatedComponent,
    props,
  };

  return getModule(
    [AppComponent, AnnotatedComponent],
    [AnnotatedComponent],
    [AppComponent],
    story,
    moduleMetadata
  );
};

agalazis · 2018-06-06T02:41:17Z

@wuyuanyi135 your workaround seems to work that's the way it should be shipped (In my case I didn't need the id part but your webpack config is a must)

wuyuanyi135 · 2018-06-06T06:16:30Z

@agalazis True. I believe it was after alpha 7 where the module.id line was no longer necessary. I haven't test the later versions since I switch to Angular playground before storybook is stable for Angular.

igor-dv · 2018-06-14T16:28:58Z

Has anyone tried the latest alpha? it supports Angular 6.

DmitryEfimenko · 2018-06-25T00:11:50Z

I'm still struggling to make it work. A part of the issue might be the fact that I'm using .scss files. @wuyuanyi135 , do you have an idea what webpack configuration to use in this case? I tried the suggested loader, but that didn't work:

{
  test: /\.scss$/,
  use: [
    'style-loader', // creates style nodes from JS strings
    'css-loader', // translates CSS into CommonJS
    'sass-loader' // compiles Sass to CSS
  ]
}

igor-dv · 2018-06-27T11:28:12Z

If you are using scss and angular-cli, storybook should work without configuring anything.

ryaninvents · 2018-07-02T22:53:32Z

I fixed this issue by using text-loader. I'm not using a CSS preprocessor though.

igor-dv · 2018-07-12T15:35:21Z

I assume we are ok here, right?

MatissJanis · 2018-07-16T07:19:31Z

I'm using alpha.14 and experiencing the same issue, but with LESS files rather than SASS.

Weback config:

const path = require('path');

module.exports = {
  module: {
    rules: [
      {
        test: /\.less$/,
        loaders: ['style-loader', 'css-loader', 'less-loader'],
        include: path.resolve(__dirname, '../'),
      },
      {
        test: /\.(jpe?g|png|gif)$/,
        use: 'file-loader?name=img/[name]-[hash].[ext]'
      },
      {
        test: /\.(eot|woff2?|svg|ttf)([\?]?.*)$/,
        use: 'file-loader',
      },
    ],
  },
};

igor-dv · 2018-07-16T15:36:09Z

AFAIR, less is also treated out-of-the-box by angular-cli, if so, you don't need to define any extra rules for the .less files.

MatissJanis · 2018-07-17T05:32:59Z

Sadly, removing the .less rule does not resolve the issue, but introduces a new one:

ERROR in ./.storybook/container.less 1:0
Module parse failed: Unexpected character '@' (1:0)
You may need an appropriate loader to handle this file type.
> @import '~@org/fe-webapp-styling/src/main.less';
|
 @ ./.storybook/config.js 10:0-26
 @ multi ./node_modules/@storybook/core/dist/server/config/polyfills.js ./node_modules/@storybook/core/dist/server/config/globals.js ./.storybook/config.js (webpack)-hot-middleware/client.js?reload=true

ERROR in ./src/components/molecules/progress-bar/progress-bar.component.less 1:0
Module parse failed: Unexpected token (1:0)
You may need an appropriate loader to handle this file type.
> .progress-bar-warning {
|   width: 0;
| }
 @ ./src/components/molecules/progress-bar/progress-bar.component.ts 61:21-61
 @ ./src/components/molecules/progress-bar/index.ts
 @ ./src/components/molecules/index.ts
 @ ./src/components/index.ts
 @ ./src/components.module.ts
 @ ./.storybook/config.js
 @ multi ./node_modules/@storybook/core/dist/server/config/polyfills.js ./node_modules/@storybook/core/dist/server/config/globals.js ./.storybook/config.js (webpack)-hot-middleware/client.js?reload=true

By the looks of it, angular-cli does not add the less loader to storybook, but it does work when running ng build.

igor-dv · 2018-07-17T12:46:54Z

I've added this 👇 to an official storybook angular example

The only error I got was about the lack of less-loader, because we didn't choose this option when created our angular-cli example. After manually installing a less-loader, everything started working for me.

MatissJanis · 2018-07-18T07:19:07Z

Could you please give me a URL to the example commit? I'm trying to reproduce it locally, but failing.

MatissJanis · 2018-07-18T07:53:09Z

I fixed the issue by changing the LESS loader to this:

{
	test: /\.less$/,
	loaders: ['text-loader', 'style-loader', 'css-loader', 'less-loader'],
	include: path.resolve(__dirname, '../'),
}

Honestly - no idea why it works now.

igor-dv · 2018-07-18T09:31:49Z

I didn't commit this specific example. since there was nothing special. The official example app is here , though, you need to bootstrap the whole monorepo to be able to run it.

Anyway, to understand what is wrong in your setup we need a reproduction 🤷‍♂️

MatissJanis · 2018-07-19T05:35:06Z

Yeah, that's completely understandable. Since I somehow managed to solve the issue on my project, I'll get out of your hair. Thanks for the patience! :)

jkyoutsey · 2018-09-25T13:26:35Z

FYI, this issue will arise if you misspell any part of the file name as well. Like, leaving off the .scss extension.

westandy-dcp · 2018-10-05T21:51:08Z

I followed @wuyuanyi135 solution.

I added a .storybook/webpack.config.js file. The code is below.
I added to-string-loader and @storybook/addon-storysource (alpha 4.0.0 version)

Here's my webpack code (same as @wuyuanyi135):

const path = require('path');

module.exports = baseConfig => {
  // Extend defaultConfig as you need.

  baseConfig.module.rules = [
    ...baseConfig.module.rules,
    {
      test: [/\.stories\.tsx?$/, /index\.ts$/],
      loaders: [
        {
          loader: require.resolve('@storybook/addon-storysource/loader'),
          options: {
            parser: 'typescript'
          }
        }
      ],
      include: [path.resolve(__dirname, '../src')],
      enforce: 'pre'
    },
    {
      test: /\.css$/,
      use: ['to-string-loader', 'css-loader']
    }
  ];

  return baseConfig;
};

Adding .less to my project looks straightforward now as well. Thank you, @wuyuanyi135 !

iftkharhussain · 2018-11-02T08:51:04Z

i have fixed this issue with changing
styleUrls: ['./app.component.CSS'] to styleUrls: ['./app.component.css']

just lower case styles extensions
now it works fine :)

haskelcurry · 2019-03-23T17:55:04Z

@DmitryEfimenko Hi Dmitry! How did you solve your issue with .scss back then? I know that it should "just work", but it doesn't in my case, just like in yours

jlowcs · 2019-06-26T13:56:18Z

This error could be caused by a improper (ie too new) version of raw-loader.

See TheLarkInn/angular2-template-loader#86

Setting it to 1.0.0 fixed the issue in my project.

Aviw13 · 2019-07-05T13:04:20Z

This error could be caused by a improper (ie too new) version of raw-loader.

See TheLarkInn/angular2-template-loader#86

Setting it to 1.0.0 fixed the issue in my project.

Thanx man! You saved my day

raw-loader v. 3.0.0 also has the same issue

omares · 2019-08-05T17:20:10Z

To fix the Expected 'styles' to be an array of strings error, i followed the Custom Webpack Config chapter of the storybook documentation and added a custom rule that handles scss parsing.

The debug flag --debug-webpack revealed that storybook adds a scss rule which needs to removed, as that rule would still take priority of our custom one.

Using the angular-cli setup was not an option for us as we run a custom webpack build.

The final version of the customization looks as follow.

JavaScript (webpack.config.js) version

// Export a function. Accept the base config as the only param.
module.exports = async ({config, mode}) => {
    // `mode` has a value of 'DEVELOPMENT' or 'PRODUCTION'
    // You can change the configuration based on that.
    // 'PRODUCTION' is used when building the static version of storybook.

    r = config.module.rules.filter(rule => rule.test != '/\\.s(c|a)ss$/');

    // Make whatever fine-grained changes you need
    r.push({
        test: /\.scss$/,
        // test: /\.s(c|a)ss$/,
        use: [
            'to-string-loader',
            {
                loader: 'style-loader',
                options: {
                    sourceMap: true,
                },
            },
            {
                loader: 'css-loader',
                options: {
                    sourceMap: true,
                },
            },
            {
                loader: 'sass-loader',
                options: {
                    sourceMap: true,
                },
            },
        ],
    });

    config.module.rules = r;

    // Return the altered config
    return config;
};

TypeScript (webpack.config.ts) version with type hints

import {Configuration, RuleSetRule} from 'webpack';

module.exports = async ({config, mode}: {config: Configuration, mode: string}) => {
    // `mode` has a value of 'DEVELOPMENT' or 'PRODUCTION'
    // You can change the configuration based on that.
    // 'PRODUCTION' is used when building the static version of storybook.

    if (!config.module) {
        return config;
    }

    var rules = config.module.rules.filter((rule: RuleSetRule) => rule.test != '/\\.s(c|a)ss$/');

    rules.push({
        test: /\.scss$/,
        use: [
            'to-string-loader',
            {
                loader: 'style-loader',
                options: {
                    sourceMap: true,
                },
            },
            {
                loader: 'css-loader',
                options: {
                    sourceMap: true,
                },
            },
            {
                loader: 'sass-loader',
                options: {
                    sourceMap: true,
                },
            },
        ],
    });

    config.module.rules = rules;

    // Return the altered config
    return config;
};

gn-sky · 2020-01-06T18:19:46Z

Instead of ‘style-loader’, ‘css-loader’, ‘sass-loader’
I went with ‘to-string-loader’, ‘css-loader’, ‘sass-loader’
I used this article:
https://www.freecodecamp.org/news/how-to-configure-webpack-4-with-angular-7-a-complete-guide-9a23c879f471/

Keraito added angular question / support labels May 16, 2018

wuyuanyi135 mentioned this issue May 19, 2018

Styles no longer imported after updating to v4.0.0-alpha - Angular 6 #3605

Closed

wuyuanyi135 mentioned this issue Jun 6, 2018

Angular 6 readme #3713

Closed

igor-dv closed this as completed Jul 12, 2018

isaacplmann mentioned this issue Aug 16, 2019

feat: Storybook schematics nrwl/nx#1582

Merged

christianliebel mentioned this issue Aug 27, 2019

@storybook/angular: Expected 'styles' to be an array of strings #7877

Closed

ciekawy mentioned this issue Oct 16, 2019

fix styleUrls loading meDavid/cypress-angular-unit-test#2

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Angular 6 Expected 'styles' to be an array of strings. #3593

Angular 6 Expected 'styles' to be an array of strings. #3593

wuyuanyi135 commented May 16, 2018 •

edited

Loading

wuyuanyi135 commented May 19, 2018 •

edited

Loading

nglazov commented May 19, 2018

wuyuanyi135 commented May 19, 2018

enten commented May 27, 2018 •

edited

Loading

agalazis commented Jun 6, 2018 •

edited

Loading

wuyuanyi135 commented Jun 6, 2018 •

edited

Loading

igor-dv commented Jun 14, 2018

DmitryEfimenko commented Jun 25, 2018

igor-dv commented Jun 27, 2018

ryaninvents commented Jul 2, 2018

igor-dv commented Jul 12, 2018

MatissJanis commented Jul 16, 2018 •

edited

Loading

igor-dv commented Jul 16, 2018

MatissJanis commented Jul 17, 2018

igor-dv commented Jul 17, 2018 •

edited

Loading

MatissJanis commented Jul 18, 2018

MatissJanis commented Jul 18, 2018

igor-dv commented Jul 18, 2018

MatissJanis commented Jul 19, 2018

jkyoutsey commented Sep 25, 2018

westandy-dcp commented Oct 5, 2018 •

edited

Loading

iftkharhussain commented Nov 2, 2018

haskelcurry commented Mar 23, 2019

jlowcs commented Jun 26, 2019

Aviw13 commented Jul 5, 2019 •

edited

Loading

omares commented Aug 5, 2019 •

edited

Loading

gn-sky commented Jan 6, 2020

Angular 6 Expected 'styles' to be an array of strings. #3593

Angular 6 Expected 'styles' to be an array of strings. #3593

Comments

wuyuanyi135 commented May 16, 2018 • edited Loading

Bug or support request summary

Steps to reproduce

Please specify which version of Storybook and optionally any affected addons that you're running

wuyuanyi135 commented May 19, 2018 • edited Loading

nglazov commented May 19, 2018

wuyuanyi135 commented May 19, 2018

enten commented May 27, 2018 • edited Loading

agalazis commented Jun 6, 2018 • edited Loading

wuyuanyi135 commented Jun 6, 2018 • edited Loading

igor-dv commented Jun 14, 2018

DmitryEfimenko commented Jun 25, 2018

igor-dv commented Jun 27, 2018

ryaninvents commented Jul 2, 2018

igor-dv commented Jul 12, 2018

MatissJanis commented Jul 16, 2018 • edited Loading

igor-dv commented Jul 16, 2018

MatissJanis commented Jul 17, 2018

igor-dv commented Jul 17, 2018 • edited Loading

MatissJanis commented Jul 18, 2018

MatissJanis commented Jul 18, 2018

igor-dv commented Jul 18, 2018

MatissJanis commented Jul 19, 2018

jkyoutsey commented Sep 25, 2018

westandy-dcp commented Oct 5, 2018 • edited Loading

iftkharhussain commented Nov 2, 2018

haskelcurry commented Mar 23, 2019

jlowcs commented Jun 26, 2019

Aviw13 commented Jul 5, 2019 • edited Loading

omares commented Aug 5, 2019 • edited Loading

gn-sky commented Jan 6, 2020

wuyuanyi135 commented May 16, 2018 •

edited

Loading

wuyuanyi135 commented May 19, 2018 •

edited

Loading

enten commented May 27, 2018 •

edited

Loading

agalazis commented Jun 6, 2018 •

edited

Loading

wuyuanyi135 commented Jun 6, 2018 •

edited

Loading

MatissJanis commented Jul 16, 2018 •

edited

Loading

igor-dv commented Jul 17, 2018 •

edited

Loading

westandy-dcp commented Oct 5, 2018 •

edited

Loading

Aviw13 commented Jul 5, 2019 •

edited

Loading

omares commented Aug 5, 2019 •

edited

Loading