Merge pull request #25950 from storybookjs/jeppe/25035-bug-boolean-co…

…ntrol-in-true-state-in-url-is-ignored

Core: Fix boolean `true` args in URL getting ignored
(cherry picked from commit d77745e)

code/lib/preview-api/src/modules/store/args.test.ts

@@ -67,8 +67,9 @@ describe('mapArgsToTypes', () => {
   });
 
   it('maps booleans', () => {
+    expect(mapArgsToTypes({ a: true }, { a: { type: booleanType } })).toStrictEqual({ a: true });
     expect(mapArgsToTypes({ a: 'true' }, { a: { type: booleanType } })).toStrictEqual({ a: true });
-    expect(mapArgsToTypes({ a: 'false' }, { a: { type: booleanType } })).toStrictEqual({
+    expect(mapArgsToTypes({ a: false }, { a: { type: booleanType } })).toStrictEqual({
       a: false,
     });
     expect(mapArgsToTypes({ a: 'yes' }, { a: { type: booleanType } })).toStrictEqual({ a: false });
@@ -129,7 +130,7 @@ describe('mapArgsToTypes', () => {
         {
           key: {
             arr: ['1', '2'],
-            obj: { bool: 'true' },
+            obj: { bool: true },
           },
         },
         {
@@ -159,7 +160,7 @@ describe('mapArgsToTypes', () => {
           key: [
             {
               arr: ['1', '2'],
-              obj: { bool: 'true' },
+              obj: { bool: true },
             },
           ],
         },

code/lib/preview-api/src/modules/store/args.ts

@@ -19,7 +19,7 @@ const map = (arg: unknown, argType: InputType): any => {
     case 'number':
       return Number(arg);
     case 'boolean':
-      return arg === 'true';
+      return String(arg) === 'true';
     case 'array':
       if (!type.value || !Array.isArray(arg)) return INCOMPATIBLE;
       return arg.reduce((acc, item, index) => {