Only mutate arg keys when writable

code/addons/interactions/src/preview.test.ts

@@ -0,0 +1,34 @@
+import { expect, test } from 'vitest';
+import { fn, isMockFunction } from '@storybook/test';
+import { action } from '@storybook/addon-actions';
+
+import { traverseArgs } from './preview';
+
+test('traverseArgs', () => {
+  const args = {
+    deep: {
+      deeper: {
+        fnKey: fn(),
+        actionKey: action('name'),
+      },
+    },
+    arg2: Object.freeze({ frozen: true }),
+  };
+
+  expect(args.deep.deeper.fnKey.getMockName()).toEqual('spy');
+
+  const traversed = traverseArgs(args) as typeof args;
+  expect(traversed).toEqual({
+    deep: {
+      deeper: {
+        fnKey: args.deep.deeper.fnKey,
+        actionKey: args.deep.deeper.actionKey,
+      },
+    },
+    arg2: args.arg2,
+  });
+
+  expect(traversed.deep.deeper.fnKey.getMockName()).toEqual('fnKey');
+  const actionFn = traversed.deep.deeper.actionKey;
+  expect(isMockFunction(actionFn) && actionFn.getMockName()).toEqual('actionKey');
+});

code/addons/interactions/src/preview.ts

@@ -16,7 +16,7 @@ export const { step: runStep } = instrument(
   { intercept: true }
 );
 
-const traverseArgs = (value: unknown, depth = 0, key?: string): any => {
+export const traverseArgs = (value: unknown, depth = 0, key?: string): unknown => {
   // Make sure to not get in infinite loops with self referencing args
   if (depth > 5) return value;
   if (value == null) return value;
@@ -45,9 +45,11 @@ const traverseArgs = (value: unknown, depth = 0, key?: string): any => {
 
   if (typeof value === 'object' && value.constructor === Object) {
     depth++;
-    // We have to mutate the original object for this to survive HMR.
     for (const [k, v] of Object.entries(value)) {
-      (value as Record<string, unknown>)[k] = traverseArgs(v, depth, k);
+      if (Object.getOwnPropertyDescriptor(value, k).writable) {
+        // We have to mutate the original object for this to survive HMR.
+        (value as Record<string, unknown>)[k] = traverseArgs(v, depth, k);
+      }
     }
     return value;
   }