fix: Always add Storybook last (#547)

* fix: always handle storybook as the last addon

* Add changesets

* Update packages/cli/lib/install.ts

* fix lint

* fix changeset

---------

Co-authored-by: Manuel <[email protected]>
Co-authored-by: Manuel Serret <[email protected]>

Author: JReinhold

.changeset/slimy-rivers-think.md

@@ -0,0 +1,5 @@
+---
+'sv': patch
+---
+
+fix: always add `storybook` after all other add-ons

packages/cli/lib/install.ts

@@ -182,9 +182,12 @@ async function runAddon({ addon, multiple, workspace }: RunAddon) {
 
 // orders addons by putting addons that don't require any other addon in the front.
 // This is a drastic simplification, as this could still cause some inconvenient cituations,
-// but works for now in contrary to the previouse implementation
+// but works for now in contrary to the previous implementation
 function orderAddons(addons: Array<Addon<any>>, setupResults: Record<string, AddonSetupResult>) {
-	return addons.sort(
-		(a, b) => setupResults[a.id]?.dependsOn?.length - setupResults[b.id]?.dependsOn?.length
-	);
+	return addons.sort((a, b) => {
+		// Adding storybook last means it will correctly detect and integrate with other addons like vitest and eslint
+		if (a.id === 'storybook') return 1;
+		if (b.id === 'storybook') return -1;
+		return setupResults[a.id]?.dependsOn?.length - setupResults[b.id]?.dependsOn?.length;
+	});
 }