Migrate storybook v4 to v5

Storybook v4.0 から v5.0 にアップデートした

Storybook v5 がリリースされたので、v4 から v5 にアップデートした。
公式アナウンスのマイグレーション方法はこちら

webpack.config.js

Storybook の webpack.config.js を full-controll mode で使っている(function を exports するやつ)場合は、function の引数を Object に変更する必要がある。

- module.exports = (storybookBaseConfig, configType) => {
+ module.exports = ({config, mode}) => {

https://github.com/kzhrk-sandbox/storybook-for-styleguide/commit/26e0974708cef41d56627e95a3ea4a432effa4cc#diff-6da49b24efd584154cfb4ec759745192L3

今までは第一引数、第二引数に Storybook から値が投げ込まれていたが、引数が Object になったことで変数名(key 名)が決め打ちになった。

storiesOf の第一引数に日本語が使えない

storiesOf の第一引数のサニタイズに使われている関数が追加され下記リンクのような実装になっているので、storiesOf の第一引数に A-Za-z0-9 以外の文字が使えなくなった。

https://github.com/storybooks/storybook/commit/54faca3793338cb521c8b1aaf5801c597f037cb8#diff-18efc3f2c976feeab3887b2fd2a58f7eR15

このサニタイズは、Storybook にカテゴリ名やストーリー名を登録する際に実行される。サニタイズの結果、カテゴリ名やストーリー名がカラになったときにエラーが返却されて登録が弾かれる。

https://github.com/storybooks/storybook/commit/54faca3793338cb521c8b1aaf5801c597f037cb8#diff-18efc3f2c976feeab3887b2fd2a58f7eR21

実際にサニタイズを実行してみると、下記のような結果になる。

"ヘッドライン"
	.toLowerCase()
	.replace(/[^a-z0-9-]/g, "-")
	.replace(/-+/g, "-")
	.replace(/^-+/, "")
	.replace(/-+$/, ""); // ''

"headline ヘッドライン"
	.toLowerCase()
	.replace(/[^a-z0-9-]/g, "-")
	.replace(/-+/g, "-")
	.replace(/^-+/, "")
	.replace(/-+$/, ""); // 'headline'

頭に半角英数を入れれば Storybook への登録はできるが、サニタイズ後の名前が登録されてしまう。
そのため、icon: ログインicon: ログアウトを登録すると、両方ともiconとなるため最初に登録された Story しか有効にならない。

このサニタイズ部分を修正してテストを実行するといくつかのテストがコケたので、修正の PR を送るのには骨が折れそう。
ひとまずフォークしてぐちゃぐちゃコードをいじってみようと思う。