Merge pull request #25452 from ksugawara61/case-insensitive-trailing-…

…duplicate AutoTitle: Fix case-insensitive trailing duplicate
storybookjs · Jan 8, 2024 · 898557a · 898557a
2 parents 6302cc6 + d2c59b3
commit 898557a
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 4 deletions.
diff --git a/code/lib/preview-api/src/modules/store/autoTitle.test.ts b/code/lib/preview-api/src/modules/store/autoTitle.test.ts
@@ -177,6 +177,16 @@ describe('userOrAutoTitleFromSpecifier', () => {
         ).toMatchInlineSnapshot(`to/button`);
       });
 
+      it('match with case-insensitive trailing duplicate', () => {
+        expect(
+          userOrAuto(
+            './path/to/button/Button.stories.js',
+            normalizeStoriesEntry({ directory: './path' }, options),
+            undefined
+          )
+        ).toMatchInlineSnapshot(`to/Button`);
+      });
+
       it('match with trailing index', () => {
         expect(
           userOrAuto(

diff --git a/code/lib/preview-api/src/modules/store/autoTitle.ts b/code/lib/preview-api/src/modules/store/autoTitle.ts
@@ -15,11 +15,12 @@ const sanitize = (parts: string[]) => {
   if (parts.length === 1) return [lastStripped];
 
   const nextToLast = parts[parts.length - 2];
+  if (lastStripped && nextToLast && lastStripped.toLowerCase() === nextToLast.toLowerCase()) {
+    return [...parts.slice(0, -2), lastStripped];
+  }
+
   return lastStripped &&
-    nextToLast &&
-    (lastStripped === nextToLast ||
-      /^(story|stories)([.][^.]+)$/i.test(last) ||
-      /^index$/i.test(lastStripped))
+    (/^(story|stories)([.][^.]+)$/i.test(last) || /^index$/i.test(lastStripped))
     ? parts.slice(0, -1)
     : [...parts.slice(0, -1), lastStripped];
 };