Avoid bundling @d3/shape

[mxstbr]

Looking at @primer/components on bundlephobia, it's massive (40kB min+gzip), and a big part of that is the @d3/shape dependency that is in the core bundle due to being used in the Donut component.

We will most likely not end up using the Donut component, so shipping all that code is a bit of a pain. Is there any chance that the d3 dependency could maybe be loaded asynchronously when needed or that the Donut component gets put into its own bundle?

EDIT: Found a workaround in #456 by importing specific components. It would still be good to see this solved by default.

[shawnbot]

I think this is a good candidate for pulling out into a separate bundle. In Primer CSS land this would be a "product" style rather than a "core" style; maybe we should have a @primer/components/product bundle? /cc @broccolini

[emplums]

Crazy idea, do we need the donut in our design system right now? I believe it was just created because we wanted to replicate the merge box, but it feels a little too specific and doesn't live in Primer CSS. I'd be okay with removing it all together cc @broccolini

[shawnbot]

Nope, burn it! 🔥

[emplums]

BOOM! problem solved!

[broccolini]

Agree we don't need it in Primer, this would likely be and app specific component.

If we do end up developing components that are useful for some GitHub apps and increase bundle size but also often not needed, then we could make an "addons" library so that folks can decide what to pull in or not. I don't think we need to go down the path we did with Primer CSS but addons could be a good option to consider in future if needed.

[emplums]

Fixed in #457