What

We want to step our TS game in the monorepo and enable strict typescript in all packages!

Why

Having TS track for you if a variable might be null or not, enables us to code with much more confidence,
and also gives us quick in editor feedback, when you make assumptions that are not actually true!

How

Feel free to contribute to any of packages in the list below!

After you cloned the repo, you can best run this command:
yarn task --task check --no-link --start-from=install

This will install all dependencies, compile it (including dts files, that is what no-link does), and then check typescript errors in all packages.

To enable strict mode you will need:

1. Change in tsconfig.json "strict": false -> "strict": true
2. Run yarn check in the directory of the package.
3. Fix all TS errors.

We would like to change as little as possible of the actual runtime behavior in this migration.
However, we also don't want to simply silence the compiler in every error with !, as or ts-ignore to get this migration in.
As a rule of thumb, if the logic is easy enough, prefer improving the code (e.g. add a null check, throw when not null, possibly use https://github.com/alexreardon/tiny-invariant) over silencing the compiler.
If the change needed to do the right thing, is too risky, and not in your expertise, it is okay to silence the compiler.
It is not ideal, but we still gain the benefit that new code written will have extra typesafety.

So if you'd like to help converting any of these ♥️ ! Please mention here, which ones, you can take on, and open a PR migrating a package.

If you need any help, please reach out to storybook maintainers on the contributing channel on the Storybook discord.