CLI: Fix skip-install for stable latest releases

code/core/src/common/js-package-manager/JsPackageManager.test.ts

@@ -0,0 +1,54 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+import { JsPackageManager } from './JsPackageManager';
+
+vi.mock('../versions', () => ({
+ default: {
+ '@storybook/react': '8.3.0',
+ },
+}));
+
+describe('JsPackageManager', () => {
+ let jsPackageManager: JsPackageManager;
+ let mockLatestVersion: ReturnType<typeof vi.fn>;
+ let mockStorybookPackagesVersions: Record<string, string>;
+
+ beforeEach(() => {
+ mockLatestVersion = vi.fn();
+ mockStorybookPackagesVersions = {
+ '@storybook/react': '8.3.0',
+ };
+
+ // @ts-expect-error Ignore abstract class error
+ jsPackageManager = new JsPackageManager();
+ jsPackageManager.latestVersion = mockLatestVersion;
+
+ vi.clearAllMocks();
+ });
+
+ describe('getVersionedPackages method', () => {
+ it('should return the latest stable release version when current version is the latest stable release', async () => {
+ mockLatestVersion.mockResolvedValue('8.3.0');
+
+ const result = await jsPackageManager.getVersionedPackages(['@storybook/react']);
+
+ expect(result).toEqual(['@storybook/react@^8.3.0']);
+ });
+
+ it('should return the current version when it is not the latest stable release', async () => {
+ mockLatestVersion.mockResolvedValue('8.3.1');
+
+ const result = await jsPackageManager.getVersionedPackages(['@storybook/react']);
+
+ expect(result).toEqual(['@storybook/[email protected]']);
+ });
+
+ it('should return the latest stable release version when there is no current version', async () => {
+ mockLatestVersion.mockResolvedValue('2.0.0');
+
+ const result = await jsPackageManager.getVersionedPackages(['@storybook/new-addon@^8.3.0']);
+
+ expect(result).toEqual(['@storybook/new-addon@^2.0.0']);
+ });
+ });
+});

code/core/src/common/js-package-manager/JsPackageManager.ts

@@ -337,13 +337,13 @@ export abstract class JsPackageManager {
  const k = packageName as keyof typeof storybookPackagesVersions;
  const currentVersion = storybookPackagesVersions[k];
 
- if (currentVersion === latestInRange) {
- return `${packageName}`;
- }
- if (currentVersion) {
- return `${packageName}@${currentVersion}`;
+ const isLatestStableRelease = currentVersion === latestInRange;
+
+ if (isLatestStableRelease || !currentVersion) {
+ return `${packageName}@^${latestInRange}`;
  }
- return `${packageName}@^${latestInRange}`;
+
+ return `${packageName}@${currentVersion}`;
  })
  );
  }