add codemod for deprecated addon-links and addon-actions from app

[kebot]

Issue:

Part of #1268

What I did

add a Codemod to help developers easily migration to v3.0

How to test

You can play around the codemod here:
https://astexplorer.net/#/gist/0f953dcda14eaed7df453ead00a25479/318680017de50a980ff95bd83c310bbb58631090

[usulpro]

Thanks @kebot!
But looks like we have migration tool here: https://github.com/storybooks/storybook/blob/master/lib/cli/generators/UPDATE_PACKAGE_ORGANIZATIONS/index.js
(see MIGRATION.md)
Could you check is this feature already included in it?

[kebot]

@usulpro No, it only changes the org name from "@kadira" to "@storybook".
My PR is part of the migration tool from v2.0 to v3.0, not familiar with the CLI tool. Maybe @tmeasday can help.

[tmeasday]

I think @ndelangen and @aaronmcadam wrote the original codemod, so they can correct me if I am wrong, but my understanding is that it just changes the names of the packages you import from, rather than any code changes like @kebot proposes here.

This looks great!

[ndelangen]

This is fantastic!!!
Sorry this went unnoticed by me for so long, my github notifications are a mess 😓

[ndelangen]

I love the tests mate, WOW!

[ndelangen]

I'm missing some documentation on this change, do you think you can add it?

How does a user execute this? or is this an unsolved problem right now?

If not, that's ok, I can write something then, thank you @kebot

[codecov]

Codecov Report

Merging #1368 into master will increase coverage by 0.15%.
The diff coverage is 65.21%.

@@            Coverage Diff             @@
##           master    #1368      +/-   ##
==========================================
+ Coverage   21.16%   21.32%   +0.15%     
==========================================
  Files         252      257       +5     
  Lines        5711     5739      +28     
  Branches      671      685      +14     
==========================================
+ Hits         1209     1224      +15     
+ Misses       3997     3995       -2     
- Partials      505      520      +15

Impacted Files	Coverage Δ
...stfixtures__/move-buildin-addons/default.output.js	0% <0%> (ø)
...estfixtures__/move-buildin-addons/default.input.js	0% <0%> (ø)
...ures__/move-buildin-addons/with-no-change.input.js	0% <0%> (ø)
...res__/move-buildin-addons/with-no-change.output.js	0% <0%> (ø)
lib/codemod/src/transforms/move-buildin-addons.js	100% <100%> (ø)
app/vue/src/server/config/babel.js	0% <0%> (-100%)	⬇️
app/vue/src/server/babel_config.js	0% <0%> (-77.42%)	⬇️
lib/components/src/navigation/menu_link.js	0% <0%> (ø)	⬆️
lib/ui/src/modules/ui/configs/init_panels.js	25% <0%> (ø)	⬆️
.../src/manager/containers/CommentsPanel/dataStore.js	34.97% <0%> (ø)	⬆️
... and 28 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9b34c32...54c7dc9. Read the comment docs.

[ndelangen]

@aaronmcadam Do you have some thoughts on how to enable this from the CLI?

[aaronmcadam]

I don't know how the CLI works @ndelangen, I've not had a chance to look at it. Would it make sense to add this as an extra step in the codemod we already have?

[tmeasday]

We may as well continue in the pattern from 3.0 and make getstorybook run the latest set of codemods if you already have storybook installed when you run it?

We may need to update the CLI to understand which codemods have been run before (maybe it just looks at storybook version before and after), but it seems sensible?