storybook 10.2.0-alpha.17 → 10.2.0-alpha.18

package/dist/core-server/presets/common-preset.js

@@ -1,15 +1,15 @@
-import CJS_COMPAT_NODE_URL_w8tq4vh9mzk from 'node:url';
-import CJS_COMPAT_NODE_PATH_w8tq4vh9mzk from 'node:path';
-import CJS_COMPAT_NODE_MODULE_w8tq4vh9mzk from "node:module";
+import CJS_COMPAT_NODE_URL_bfa4dw0yocr from 'node:url';
+import CJS_COMPAT_NODE_PATH_bfa4dw0yocr from 'node:path';
+import CJS_COMPAT_NODE_MODULE_bfa4dw0yocr from "node:module";
 
-var __filename = CJS_COMPAT_NODE_URL_w8tq4vh9mzk.fileURLToPath(import.meta.url);
-var __dirname = CJS_COMPAT_NODE_PATH_w8tq4vh9mzk.dirname(__filename);
-var require = CJS_COMPAT_NODE_MODULE_w8tq4vh9mzk.createRequire(import.meta.url);
+var __filename = CJS_COMPAT_NODE_URL_bfa4dw0yocr.fileURLToPath(import.meta.url);
+var __dirname = CJS_COMPAT_NODE_PATH_bfa4dw0yocr.dirname(__filename);
+var require = CJS_COMPAT_NODE_MODULE_bfa4dw0yocr.createRequire(import.meta.url);
 
 // ------------------------------------------------------------
 // end of CJS compatibility banner, injected by Storybook's esbuild configuration
 // ------------------------------------------------------------
-import "../../_node-chunks/chunk-CCJGKJ27.js";
+import "../../_node-chunks/chunk-IHBEBZVB.js";
 import {
   doesStoryFileExist,
   generateStoryFile,
@@ -17,47 +17,47 @@ import {
   parseStaticDir,
   sendTelemetryError,
   throttle
-} from "../../_node-chunks/chunk-XTIFAWOB.js";
-import "../../_node-chunks/chunk-EG3WZ464.js";
-import "../../_node-chunks/chunk-IWM3WHZE.js";
-import "../../_node-chunks/chunk-FAARRTOD.js";
+} from "../../_node-chunks/chunk-ZUTW3EKD.js";
+import "../../_node-chunks/chunk-JK62DOZL.js";
+import "../../_node-chunks/chunk-4PWGJYBL.js";
+import "../../_node-chunks/chunk-54FLVGBQ.js";
 import {
   isI18nPackage,
   isRouterPackage,
   isStateManagementPackage,
   isStylingPackage
-} from "../../_node-chunks/chunk-GYJ7LPFJ.js";
+} from "../../_node-chunks/chunk-GJRLPTXZ.js";
 import {
   globalSettings
-} from "../../_node-chunks/chunk-VK4OWRKU.js";
+} from "../../_node-chunks/chunk-H4TYBETM.js";
 import {
   invariant
-} from "../../_node-chunks/chunk-C3FPVGL4.js";
-import "../../_node-chunks/chunk-KKNWPXMS.js";
+} from "../../_node-chunks/chunk-OZGI27C5.js";
+import "../../_node-chunks/chunk-6TOW3ZMZ.js";
 import {
   resolvePackageDir
-} from "../../_node-chunks/chunk-3D4SLKEE.js";
+} from "../../_node-chunks/chunk-DIZCVGPK.js";
 import {
   isAbsolute,
   join
-} from "../../_node-chunks/chunk-5Q5JEISY.js";
+} from "../../_node-chunks/chunk-N73BCSUL.js";
 import {
   glob
-} from "../../_node-chunks/chunk-DJ3RRSJ7.js";
-import "../../_node-chunks/chunk-VTBZVEBF.js";
+} from "../../_node-chunks/chunk-OZHRGJKY.js";
+import "../../_node-chunks/chunk-QNYGS5WG.js";
 import {
   require_dist
-} from "../../_node-chunks/chunk-G7P42ZEY.js";
-import "../../_node-chunks/chunk-IRMNO3QS.js";
-import "../../_node-chunks/chunk-MU4E5UBA.js";
+} from "../../_node-chunks/chunk-XH6HLMNG.js";
+import "../../_node-chunks/chunk-F347QVLI.js";
+import "../../_node-chunks/chunk-LMYIILVH.js";
 import {
   require_picocolors
-} from "../../_node-chunks/chunk-XI7HDOMY.js";
+} from "../../_node-chunks/chunk-A2364FS2.js";
 import {
   __commonJS,
   __require,
   __toESM
-} from "../../_node-chunks/chunk-25OWI2RR.js";
+} from "../../_node-chunks/chunk-5BYTBW23.js";
 
 // ../../node_modules/shell-quote/quote.js
 var require_quote = __commonJS({
@@ -650,7 +650,7 @@ async function searchFiles({
   ignoredFiles = IGNORED_FILES,
   fileExtensions = FILE_EXTENSIONS
 }) {
-  let { globby, isDynamicPattern } = await import("../../_node-chunks/globby-4BDMCAAD.js"), hasSearchSpecialGlobChars = isDynamicPattern(searchQuery, { cwd }), searchQueryHasExtension = /(\.[a-z]+)$/i.test(searchQuery), fileExtensionsPattern = `{${fileExtensions.join(",")}}`, globbedSearchQuery = hasSearchSpecialGlobChars ? searchQuery : searchQueryHasExtension ? [`**/*${searchQuery}*`, `**/*${searchQuery}*/**`] : [
+  let { globby, isDynamicPattern } = await import("../../_node-chunks/globby-IVEL6LAU.js"), hasSearchSpecialGlobChars = isDynamicPattern(searchQuery, { cwd }), searchQueryHasExtension = /(\.[a-z]+)$/i.test(searchQuery), fileExtensionsPattern = `{${fileExtensions.join(",")}}`, globbedSearchQuery = hasSearchSpecialGlobChars ? searchQuery : searchQueryHasExtension ? [`**/*${searchQuery}*`, `**/*${searchQuery}*/**`] : [
     `**/*${searchQuery}*.${fileExtensionsPattern}`,
     `**/*${searchQuery}*/**/*.${fileExtensionsPattern}`
   ];
@@ -841,8 +841,7 @@ async function getComponentCandidates({
       candidates,
       globMatchCount
     };
-  } catch (error) {
-    let errorMessage = error instanceof Error ? error.message : String(error);
+  } catch {
     return {
       candidates: [],
       error: "Failed to find candidates",
@@ -868,7 +867,11 @@ var ERROR_CATEGORIES = {
   MODULE_IMPORT_ERROR: "MODULE_IMPORT_ERROR",
   COMPONENT_RENDER_ERROR: "COMPONENT_RENDER_ERROR",
   SERVER_COMPONENTS_ERROR: "SERVER_COMPONENTS_ERROR",
-  UNKNOWN_ERROR: "UNKNOWN_ERROR"
+  UNKNOWN_ERROR: "UNKNOWN_ERROR",
+  // Vite related errors
+  DYNAMIC_MODULE_IMPORT_ERROR: "DYNAMIC_MODULE_IMPORT_ERROR",
+  // Vitest test run related errors
+  TEST_FILE_IMPORT_ERROR: "TEST_FILE_IMPORT_ERROR"
 };
 function buildErrorContext(message, stack) {
   let normalizedMessage = message.toLowerCase(), normalizedStack = (stack ?? "").toLowerCase(), stackDeps = /* @__PURE__ */ new Set(), stackLines = normalizedStack.split(`
@@ -891,6 +894,16 @@ var CATEGORIZATION_RULES = [
     priority: 100,
     match: (ctx) => ctx.normalizedMessage.includes("cannot find module") || ctx.normalizedMessage.includes("module not found") || ctx.normalizedMessage.includes("cannot resolve module")
   },
+  {
+    category: ERROR_CATEGORIES.TEST_FILE_IMPORT_ERROR,
+    priority: 95,
+    match: (ctx) => ctx.normalizedMessage.includes("failed to import test file")
+  },
+  {
+    category: ERROR_CATEGORIES.DYNAMIC_MODULE_IMPORT_ERROR,
+    priority: 95,
+    match: (ctx) => ctx.normalizedMessage.includes("failed to fetch dynamically imported module")
+  },
   {
     category: ERROR_CATEGORIES.HOOK_USAGE_ERROR,
     priority: 90,
@@ -958,45 +971,19 @@ function getMatchedDependencies(category, ctx) {
       return [];
   }
 }
-function getCategoryDescription(category) {
-  switch (category) {
-    case ERROR_CATEGORIES.MISSING_STATE_PROVIDER:
-      return "Component attempted to access shared state without a state management provider";
-    case ERROR_CATEGORIES.MISSING_ROUTER_PROVIDER:
-      return "Component attempted to access routing context without a router provider";
-    case ERROR_CATEGORIES.MISSING_THEME_PROVIDER:
-      return "Component attempted to access theme values without a theme provider";
-    case ERROR_CATEGORIES.MISSING_TRANSLATION_PROVIDER:
-      return "Component attempted to access translations without a translation provider";
-    case ERROR_CATEGORIES.MISSING_PROVIDER:
-      return "Component attempted to access React context without a matching provider";
-    case ERROR_CATEGORIES.MISSING_PORTAL_ROOT:
-      return "Component attempted to render a portal without a valid DOM container";
-    case ERROR_CATEGORIES.HOOK_USAGE_ERROR:
-      return "React hook was used incorrectly";
-    case ERROR_CATEGORIES.MODULE_IMPORT_ERROR:
-      return "A required dependency could not be resolved";
-    case ERROR_CATEGORIES.COMPONENT_RENDER_ERROR:
-      return "Component failed during render due to a runtime error";
-    case ERROR_CATEGORIES.SERVER_COMPONENTS_ERROR:
-      return "Server components usage in the browser";
-    default:
-      return "Error could not be categorized";
-  }
-}
 
 // src/core-server/utils/ghost-stories/parse-vitest-report.ts
 function extractCategorizedErrors(testResults) {
   let failed = testResults.filter((r) => r.status === "FAIL" && r.error), map = /* @__PURE__ */ new Map(), uniqueErrorMessages = /* @__PURE__ */ new Set();
   for (let r of failed) {
     let { category, matchedDependencies } = categorizeError(r.error, r.stack);
-    map.has(category) || map.set(category, { count: 0, matchedDependencies: /* @__PURE__ */ new Set() });
+    map.has(category) || map.set(category, { count: 0, uniqueErrors: /* @__PURE__ */ new Set(), matchedDependencies: /* @__PURE__ */ new Set() });
     let data = map.get(category);
-    data.count++, matchedDependencies.forEach((dep) => data.matchedDependencies.add(dep)), uniqueErrorMessages.add(r.error);
+    data.count++, matchedDependencies.forEach((dep) => data.matchedDependencies.add(dep)), uniqueErrorMessages.add(r.error), data.uniqueErrors.add(r.error);
   }
   let categorizedErrors = Array.from(map.entries()).reduce(
     (acc, [category, data]) => (acc[category] = {
-      description: getCategoryDescription(category),
+      uniqueCount: data.uniqueErrors.size,
       count: data.count,
       matchedDependencies: Array.from(data.matchedDependencies).sort()
     }, acc),
@@ -1008,12 +995,12 @@ function extractCategorizedErrors(testResults) {
     categorizedErrors
   };
 }
-function parseVitestResults(testResults) {
+function parseVitestResults(report) {
   let storyTestResults = [], passedButEmptyRender = 0;
-  for (let testSuite of testResults.testResults)
+  for (let testSuite of report.testResults)
     for (let assertion of testSuite.assertionResults) {
       let storyId = assertion.meta?.storyId || assertion.fullName, status = assertion.status === "passed" ? "PASS" : assertion.status === "failed" ? "FAIL" : "PENDING", hasEmptyRender = assertion.meta?.reports?.some(
-        (report) => report.type === "render-analysis" && report.result?.emptyRender === !0
+        (report2) => report2.type === "render-analysis" && report2.result?.emptyRender === !0
       );
       status === "PASS" && hasEmptyRender && passedButEmptyRender++;
       let error, stack;
@@ -1025,20 +1012,17 @@ function parseVitestResults(testResults) {
         stack
       });
     }
-  let total = testResults.numTotalTests, passed = testResults.numPassedTests, failed = testResults.numFailedTests, successRate = total > 0 ? parseFloat((passed / total).toFixed(2)) : 0, failureRate = total > 0 ? parseFloat((failed / total).toFixed(2)) : 0, successRateWithoutEmptyRender = total > 0 ? parseFloat(((passed - passedButEmptyRender) / total).toFixed(2)) : 0, errorClassification = extractCategorizedErrors(storyTestResults), categorizedErrors = errorClassification.categorizedErrors, summary = {
-    total,
-    passed,
-    passedButEmptyRender,
-    failed,
-    successRate,
-    successRateWithoutEmptyRender,
-    failureRate,
-    uniqueErrorCount: errorClassification.uniqueErrorCount,
-    categorizedErrors
-  };
+  let total = report.numTotalTests, passed = report.numPassedTests, successRate = total > 0 ? parseFloat((passed / total).toFixed(2)) : 0, successRateWithoutEmptyRender = total > 0 ? parseFloat(((passed - passedButEmptyRender) / total).toFixed(2)) : 0, errorClassification = extractCategorizedErrors(storyTestResults), categorizedErrors = errorClassification.categorizedErrors;
   return {
-    success: testResults.success,
-    summary
+    summary: {
+      total,
+      passed,
+      passedButEmptyRender,
+      successRate,
+      successRateWithoutEmptyRender,
+      uniqueErrorCount: errorClassification.uniqueErrorCount,
+      categorizedErrors
+    }
   };
 }
 
@@ -1071,32 +1055,31 @@ async function runStoryTests(componentFilePaths) {
     let duration = Date.now() - startTime;
     if (testFailureMessage)
       return {
-        success: !1,
         duration,
-        error: testFailureMessage
+        runError: testFailureMessage
       };
     if (!existsSync(outputFile))
       return {
-        success: !1,
         duration,
-        error: "JSON report not found"
+        runError: "JSON report not found"
       };
-    let testResults;
+    let vitestReport;
     try {
       let resultsJson = await readFile3(outputFile, "utf8");
-      testResults = JSON.parse(resultsJson);
+      vitestReport = JSON.parse(resultsJson);
     } catch {
       return {
-        success: !1,
         duration,
-        error: "Failed to read or parse JSON report"
+        runError: "Failed to read or parse JSON report"
       };
     }
-    return { ...parseVitestResults(testResults), duration };
+    return !vitestReport.testResults || vitestReport.testResults.length === 0 ? {
+      duration,
+      runError: "No tests found"
+    } : { ...parseVitestResults(vitestReport), duration };
   } catch {
     return {
-      success: !1,
-      error: "Uncaught error running story tests",
+      runError: "Uncaught error running story tests",
       duration: 0
     };
   }
@@ -1107,8 +1090,11 @@ function initGhostStoriesChannel(channel, options, coreOptions) {
   return coreOptions.disableTelemetry || channel.on(GHOST_STORIES_REQUEST, async () => {
     let stats = {};
     try {
-      let ghostRunStart = Date.now(), lastEvents = await getLastEvents(), sessionId = await getSessionId(), lastInit = lastEvents?.init;
-      if (lastEvents?.["ghost-stories"] || lastInit?.body?.sessionId && lastInit?.body?.sessionId !== sessionId)
+      let ghostRunStart = Date.now(), lastEvents = await getLastEvents(), lastInit = lastEvents?.init;
+      if (!lastEvents || !lastInit)
+        return;
+      let sessionId = await getSessionId();
+      if (lastEvents["ghost-stories"] || lastInit.body?.sessionId && lastInit.body.sessionId !== sessionId)
         return;
       let metadata = await getStorybookMetadata(options.configDir), isReactStorybook = metadata?.renderer?.includes("@storybook/react"), hasVitestAddon = !!metadata?.addons && Object.keys(metadata.addons).some(
         (addonKey) => addonKey.includes("@storybook/addon-vitest")
@@ -1118,33 +1104,34 @@ function initGhostStoriesChannel(channel, options, coreOptions) {
       let candidateAnalysisStart = Date.now(), candidatesResult = await getComponentCandidates();
       if (stats.candidateAnalysisDuration = Date.now() - candidateAnalysisStart, stats.globMatchCount = candidatesResult.globMatchCount, stats.analyzedCount = candidatesResult.analyzedCount ?? 0, stats.avgComplexity = candidatesResult.avgComplexity ?? 0, stats.candidateCount = candidatesResult.candidates.length, candidatesResult.error) {
         stats.totalRunDuration = Date.now() - ghostRunStart, telemetry3("ghost-stories", {
-          success: !1,
-          error: candidatesResult.error,
-          stats
+          stats,
+          runError: candidatesResult.error
         });
         return;
       }
       if (candidatesResult.candidates.length === 0) {
         stats.totalRunDuration = Date.now() - ghostRunStart, telemetry3("ghost-stories", {
-          success: !1,
-          error: "No candidates found",
-          stats
+          stats,
+          runError: "No candidates found"
         });
         return;
       }
       let testRunResult = await runStoryTests(candidatesResult.candidates);
-      stats.totalRunDuration = Date.now() - ghostRunStart, stats.testRunDuration = testRunResult.duration, telemetry3("ghost-stories", {
-        ...testRunResult.error !== void 0 ? { error: testRunResult.error } : {},
-        success: testRunResult.success,
+      if (stats.totalRunDuration = Date.now() - ghostRunStart, stats.testRunDuration = testRunResult.duration, testRunResult.runError) {
+        telemetry3("ghost-stories", {
+          stats,
+          runError: testRunResult.runError
+        });
+        return;
+      }
+      telemetry3("ghost-stories", {
         stats,
         results: testRunResult.summary
       });
-    } catch (error) {
-      let errorMessage = error instanceof Error ? error.message : String(error);
+    } catch {
       telemetry3("ghost-stories", {
-        success: !1,
-        error: errorMessage,
-        stats
+        stats,
+        runError: "Unknown error during ghost run"
       });
     } finally {
       channel.emit(GHOST_STORIES_RESPONSE);

package/dist/csf-tools/index.js

@@ -1,10 +1,10 @@
-import CJS_COMPAT_NODE_URL_w8tq4vh9mzk from 'node:url';
-import CJS_COMPAT_NODE_PATH_w8tq4vh9mzk from 'node:path';
-import CJS_COMPAT_NODE_MODULE_w8tq4vh9mzk from "node:module";
+import CJS_COMPAT_NODE_URL_bfa4dw0yocr from 'node:url';
+import CJS_COMPAT_NODE_PATH_bfa4dw0yocr from 'node:path';
+import CJS_COMPAT_NODE_MODULE_bfa4dw0yocr from "node:module";
 
-var __filename = CJS_COMPAT_NODE_URL_w8tq4vh9mzk.fileURLToPath(import.meta.url);
-var __dirname = CJS_COMPAT_NODE_PATH_w8tq4vh9mzk.dirname(__filename);
-var require = CJS_COMPAT_NODE_MODULE_w8tq4vh9mzk.createRequire(import.meta.url);
+var __filename = CJS_COMPAT_NODE_URL_bfa4dw0yocr.fileURLToPath(import.meta.url);
+var __dirname = CJS_COMPAT_NODE_PATH_bfa4dw0yocr.dirname(__filename);
+var require = CJS_COMPAT_NODE_MODULE_bfa4dw0yocr.createRequire(import.meta.url);
 
 // ------------------------------------------------------------
 // end of CJS compatibility banner, injected by Storybook's esbuild configuration
@@ -39,9 +39,9 @@ import {
   vitestTransform,
   writeConfig,
   writeCsf
-} from "../_node-chunks/chunk-VTBZVEBF.js";
-import "../_node-chunks/chunk-G7P42ZEY.js";
-import "../_node-chunks/chunk-25OWI2RR.js";
+} from "../_node-chunks/chunk-QNYGS5WG.js";
+import "../_node-chunks/chunk-XH6HLMNG.js";
+import "../_node-chunks/chunk-5BYTBW23.js";
 export {
   BadMetaError,
   ConfigFile,