@codeyam/codeyam-cli 0.1.0-staging.596f0eb → 0.1.0-staging.76566f9

package/analyzer-template/project/writeSimpleRoot.ts

@@ -255,19 +255,19 @@ export default async function writeSimpleRoot(
 
               if (!newContent.match(importRegex)) continue;
 
-              // Find the correct variable-qualified mock key for ROOT's usage
-              // The mock data is stored under keys like "variableName <- functionName"
-              // For root.tsx, we need to find the key that corresponds to ROOT's usage,
-              // NOT the route's (Settings') usage.
+              // Find the correct mock key for ROOT's usage
+              // The mock data is stored under call signature keys like "useLoaderData()"
               const dataForMocks =
                 options.routeAnalysis?.metadata?.scenariosDataStructure
                   ?.dataForMocks;
 
-              // Find all keys that end with this entity name
+              // Find all keys that match this entity/hook by base name
+              const entityBaseName = entityName.split(/[<(]/)[0];
               const matchingKeys = dataForMocks
-                ? Object.keys(dataForMocks).filter((key) =>
-                    key.endsWith(` <- ${entityName}`),
-                  )
+                ? Object.keys(dataForMocks).filter((key) => {
+                    const keyBaseName = key.split(/[<(]/)[0];
+                    return keyBaseName === entityBaseName;
+                  })
                 : [];
 
               // If there are multiple keys (e.g., one for root.tsx and one for Settings),
@@ -302,11 +302,9 @@ export default async function writeSimpleRoot(
                   }
                 }
 
-                // Find a key that uses one of root's property names
-                mockKey = matchingKeys.find((key) => {
-                  const varName = key.split(' <- ')[0];
-                  return rootPropertyNames.includes(varName);
-                });
+                // Find a key that matches by call signature
+                // For call signature format (e.g., "useLoaderData()"), just use the first match
+                mockKey = matchingKeys[0];
 
                 // Fallback: if no match found but there are multiple keys,
                 // exclude the route's key (which we can identify from mockedImports variable names)

package/background/src/lib/virtualized/project/constructMockCode.js

@@ -1,12 +1,93 @@
 import { joinParenthesesAndArrays, splitOutsideParenthesesAndArrays, functionArguments, cleanOutBoundary, fillInDirectSchemaGapsAndUnknowns, removeDuplicateFunctionCalls, } from "../../../../../packages/ai/index.js";
 /**
- * Converts a variable qualifier to a safe identifier for use in function/const names.
- * Handles indexed format like "data[1]" -> "data_1"
+ * Converts a call signature to a valid JavaScript identifier (function name).
+ * The original signature is preserved for data access - this only creates the function name.
+ *
+ * Examples:
+ * - "useAuth()" → "useAuth"
+ * - "db.select(usersQuery)" → "db_select_usersQuery"
+ * - "db.select(postsQuery)" → "db_select_postsQuery"
+ * - "useFetcher<User>()" → "useFetcher_User"
+ * - "useFetcher<{ data: UserData | null }>()" → "useFetcher_data_UserData_null"
+ * - "eq('user_id', value)" → "eq_user_id_value"
+ * - "from('workouts')" → "from_workouts"
  */
-function safeVariableQualifier(qualifier) {
-    // Replace brackets with underscores for valid JS identifier
-    // "data[1]" -> "data_1"
-    return qualifier.replace(/\[(\d+)\]/g, '_$1');
+function callSignatureToFunctionName(signature) {
+    // Extract components from the signature
+    const components = [];
+    // 1. Extract function path (parts separated by dots outside parens/brackets)
+    const pathMatch = signature.match(/^([^<(]+)/);
+    if (pathMatch) {
+        const path = pathMatch[1];
+        // Split on dots but preserve the parts
+        components.push(...path.split('.').filter(Boolean));
+    }
+    // 2. Extract generic type parameters (content between < and >)
+    const genericMatch = signature.match(/<([^>]+)>/);
+    if (genericMatch) {
+        const genericContent = genericMatch[1];
+        // Extract meaningful identifiers from generic type
+        // Handle complex types like "{ data: UserData | null }"
+        const typeIdentifiers = genericContent
+            .replace(/[{}:;,]/g, ' ') // Remove structural chars
+            .replace(/\|/g, ' ') // Handle union types
+            .split(/\s+/)
+            .filter(Boolean)
+            .filter((s) => /^[A-Za-z_][A-Za-z0-9_]*$/.test(s)) // Only valid identifiers
+            .filter((s) => ![
+            'null',
+            'undefined',
+            'void',
+            'never',
+            'any',
+            'unknown',
+            'data',
+            'typeof',
+        ].includes(s)); // Skip common non-meaningful keywords
+        if (typeIdentifiers.length > 0) {
+            components.push(...typeIdentifiers.slice(0, 2)); // Limit to first 2 for reasonable length
+        }
+    }
+    // 3. Extract function arguments (first 2 for disambiguation)
+    const argsMatch = signature.match(/\(([^)]*)\)/);
+    if (argsMatch && argsMatch[1]) {
+        const argsContent = argsMatch[1].trim();
+        if (argsContent) {
+            const args = argsContent.split(',').map((arg) => arg.trim());
+            for (const arg of args.slice(0, 2)) {
+                // For quoted strings, extract the content
+                const stringMatch = arg.match(/^['"`](.+)['"`]$/);
+                if (stringMatch) {
+                    // Split on dots for string paths like 'users.id'
+                    const parts = stringMatch[1].split('.').filter(Boolean);
+                    components.push(...parts);
+                }
+                else if (/^[A-Za-z_][A-Za-z0-9_]*$/.test(arg)) {
+                    // Valid identifier - use as-is
+                    components.push(arg);
+                }
+                else if (/^\d+$/.test(arg)) {
+                    // Number - use as-is
+                    components.push(arg);
+                }
+                // Skip complex expressions
+            }
+        }
+    }
+    // Build the function name from components
+    const functionName = components
+        .join('_')
+        .replace(/[^a-zA-Z0-9_]/g, '_') // Sanitize special chars
+        .replace(/_+/g, '_') // Collapse multiple underscores
+        .replace(/^_|_$/g, ''); // Trim underscores
+    return functionName || 'mock';
+}
+/**
+ * Check if a mock name is a call signature (e.g., "useFetcher<User>()", "db.select(query)")
+ */
+function isCallSignature(mockName) {
+    // Call signatures contain parentheses (function calls)
+    return mockName.includes('(');
 }
 /**
  * Extract property names that are jsx-components and should be preserved from original.
@@ -140,17 +221,20 @@ function isValidKey(key) {
     const keyWithOutArguments = key.split('(')[0];
     return !/\s/.test(keyWithOutArguments);
 }
-export default function constructMockCode(mockName, dependencySchemas, entityType, options) {
-    // Check for variable-qualified mock name (e.g., "entityDiffFetcher <- useFetcher")
-    // Format: "variableName <- functionName" which reads naturally as "variableName receives from functionName"
-    // Also handles indexed format: "data[1] <- useFetch" for disambiguating reused variable names
-    const variableQualifierMatch = mockName.match(/^([a-zA-Z_][a-zA-Z0-9_]*(?:\[\d+\])?)\s*<-\s*/);
-    let variableQualifier = variableQualifierMatch
-        ? variableQualifierMatch[1]
+export default function constructMockCode(mockName, dependencySchemas, entityType, _canonicalKey, // DEPRECATED: No longer used, kept for API compatibility
+options) {
+    // Check if mockName is a call signature (e.g., "useFetcher<User>()", "db.select(query)")
+    const mockNameIsCallSignature = isCallSignature(mockName);
+    // For call signatures, use the original signature for data access but generate
+    // a valid JS function name from it
+    const derivedFunctionName = mockNameIsCallSignature
+        ? callSignatureToFunctionName(mockName)
         : null;
-    const baseMockName = variableQualifier
-        ? mockName.slice(variableQualifierMatch[0].length)
-        : mockName;
+    // The baseMockName is the function name without type params and args
+    // e.g., "useFetcher<User>()" -> "useFetcher", "db.select(query)" -> "db"
+    const baseMockName = mockName.split(/[<(]/)[0];
+    // The data key is the mockName (call signature) for data access
+    let dataKey;
     const mockNameParts = splitOutsideParenthesesAndArrays(baseMockName);
     let relevantReturnValueSchema;
     let dataStructurePath;
@@ -159,26 +243,9 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
     let signatureSchema;
     for (const filePath in dependencySchemas) {
         for (const entityName in dependencySchemas[filePath]) {
-            // For variable-qualified mocks, look for the full qualified name first
-            // Format: "variableName <- functionName" (e.g., "entityDiffFetcher <- useFetcher")
-            const targetEntityName = variableQualifier
-                ? `${variableQualifier} <- ${baseMockName}`
-                : mockNameParts[0];
-            // Check for direct match
-            let matches = entityName === targetEntityName || entityName === mockNameParts[0];
-            // If no direct match and no qualifier was provided, check if the entity
-            // is stored under a variable-qualified key (e.g., "stateBadge <- getStateBadge")
-            // This handles the case where gatherDataForMocks stored the entity with a variable
-            // qualifier but writeScenarioComponents called constructMockCode without one.
-            if (!matches && !variableQualifier) {
-                const qualifiedKeyMatch = entityName.match(new RegExp(`^([a-zA-Z_][a-zA-Z0-9_]*)\\s*<-\\s*${mockNameParts[0]}$`));
-                if (qualifiedKeyMatch) {
-                    matches = true;
-                    // Extract the variable qualifier from the entity name so we can use
-                    // it for the data lookup key later
-                    variableQualifier = qualifiedKeyMatch[1];
-                }
-            }
+            // Match entity by base name (without generics/args)
+            const entityBaseName = entityName.split(/[<(]/)[0];
+            const matches = entityBaseName === baseMockName || entityName === mockNameParts[0];
             if (!matches)
                 continue;
             // Track if we found the entity and it has a signature (is a function)
@@ -210,6 +277,37 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
             }
         }
     }
+    // Check if the entity is used as a function (called with ()) vs an object/namespace.
+    // Look for paths in the schema that start with "baseMockName(" or "baseMockName<" indicating function calls.
+    // The "<" handles generic type parameters like useLoaderData<T>().
+    // Also check dataStructurePath === 'returnValue' which indicates a function return value.
+    const entityIsFunction = foundEntityWithSignature ||
+        dataStructurePath === 'returnValue' ||
+        Object.keys(relevantReturnValueSchema ?? {}).some((key) => key.startsWith(`${baseMockName}(`) ||
+            key.startsWith(`${baseMockName}<`));
+    // Calculate the data key - use the call signature (mockName) for data access
+    // For simple names without parentheses:
+    // - Append () ONLY if the entity is a function/hook (detected above)
+    // - Don't append () for object/namespace mocks like "supabase"
+    if (mockNameIsCallSignature || mockName.includes('(')) {
+        dataKey = mockName;
+    }
+    else if (entityIsFunction) {
+        // Entity is a function/hook - append () to match call signature format
+        dataKey = `${mockName}()`;
+    }
+    else {
+        // Entity is an object/namespace - use bare name as key
+        dataKey = mockName;
+    }
+    // LOGGING: Track data key generation for debugging key mismatch issues
+    console.log('CODEYAM DEBUG [constructMockCode]: Generated dataKey:', JSON.stringify({
+        mockName,
+        dataKey,
+        mockNameIsCallSignature,
+        entityIsFunction,
+        baseMockName,
+    }));
     // Check if the return value schema only contains function type markers
     // (e.g., "validateInputs()": "function") without actual return data
     // (no functionCallReturnValue entries)
@@ -287,17 +385,16 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
             // Strip type parameters like <typeof loader> from function names
             // so "useLoaderData<typeof loader>()" becomes "useLoaderData()"
             name = cleanOutTypes(name);
-            // For variable-qualified mocks at root level, use the "variableName <- functionName" format
-            // This matches the format generated by gatherDataForMocks and is more intuitive for LLMs
-            // e.g., "useFetcher()" becomes "entityDiffFetcher <- useFetcher"
-            if (isRootAccess && variableQualifier) {
-                // Remove trailing () from the function name to match gatherDataForMocks key format
-                const baseName = name.replace(/\(\)$/, '');
-                name = `${variableQualifier} <- ${baseName}`;
+            // For root data access, use the dataKey (original call signature or canonical key)
+            // This preserves the original call signature for LLM clarity
+            if (isRootAccess) {
+                // For call signature format, use the original mockName as the data key
+                // e.g., scenarios().data()?.["useFetcher<User>()"]
+                // e.g., scenarios().data()?.["db.select(usersQuery)"]
+                return `?.["${dataKey.replace(/\n/g, '\\n')}"]`;
             }
             // Only use unquoted array access syntax for pure array indices like [0], [1]
-            // NOT for variable-qualified names like "fetcher[1] <- useFetcher" which need quotes
-            if (name.match(/^\[\d+\]$/) && !name.includes(' <- ')) {
+            if (name.match(/^\[\d+\]$/)) {
                 return `?.${name}`;
             }
             return `?.["${name.replace(/\n/g, '\\n')}"]`;
@@ -499,9 +596,17 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
                     // When GENERIC array (using []) has nested content (like functions that need wrapping),
                     // use .map() to transform ALL elements instead of just creating [0]
                     // For DIFFERENTIATED arrays (using [0], [1], etc.), keep the static array structure
+                    //
+                    // IMPORTANT: If the nested content contains differentiated indices like [0], [1],
+                    // we MUST use static array pattern, not .map(). The presence of differentiated
+                    // indices means the array elements have different types/structures, so .map()
+                    // would generate invalid code trying to treat them uniformly.
+                    const hasDifferentiatedIndices = nested &&
+                        nested.some((n) => n.name.match(/^\[\d+\]$/) && n.name !== '[0]');
                     if (isGenericArray &&
                         nestedContent.length > 0 &&
-                        dataPaths.length > 0) {
+                        dataPaths.length > 0 &&
+                        !hasDifferentiatedIndices) {
                         // Get the array base path (without the [0])
                         const arrayBasePath = dataPaths[0].replace(/\?\.\[0\]$/, '');
                         // Replace [0] references with [__idx__] in level contents
@@ -531,6 +636,12 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
             if (args && args.length > 0) {
                 if (!isValidKey(name))
                     return;
+                // Skip array index patterns like [], [0], [1] when they have args
+                // These represent function calls on array elements, not property keys
+                // e.g., customSizes[].(args) means each array element is callable, not a property named "[]"
+                if (name.match(/^\[\d*\]$/)) {
+                    return;
+                }
                 const mostArgs = args.sort((a, b) => b.length - a.length)[0];
                 const argsString = mostArgs
                     .map((_, index) => `arg${index + 1}`)
@@ -570,8 +681,19 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
                         fallbackContent = `return ${returnValueContents}`;
                     }
                     else {
-                        // Use all paths as fallback for unknown arguments
-                        fallbackContent = `return ${returnValueContents}`;
+                        // No explicit fallback paths - return the first literal's value as default
+                        // Returning spread of all values is dangerous because if values are primitives (strings),
+                        // spreading them creates objects with numeric keys like {0:'a', 1:'b', ...}
+                        // which causes "Objects are not valid as React child" errors
+                        const firstLiteralValue = literalKeys[0];
+                        const firstGroupPaths = argGroups.get(firstLiteralValue);
+                        if (firstGroupPaths && firstGroupPaths.length === 1) {
+                            fallbackContent = `return ${firstGroupPaths[0]}`;
+                        }
+                        else {
+                            // Multiple paths for first literal - return undefined as safe fallback
+                            fallbackContent = `return undefined`;
+                        }
                     }
                     const funcContents = conditionalBranches.join('\n') +
                         '\n// Fallback for unmatched arguments\n' +
@@ -653,8 +775,8 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
                 parts.splice(i, 1);
             }
         }
-        // For variable-qualified mocks like "revalidator <- useRevalidator",
-        // compare against baseMockName ("useRevalidator"), not the full mockName
+        // Compare against baseMockName (without generics/args), not the full mockName
+        // e.g., for "useFetcher<User>()", baseMockName is "useFetcher"
         if (parts[0].split('(')[0] !== baseMockName)
             continue;
         // Include paths with functionCallReturnValue OR function-typed paths that need mocking
@@ -1047,16 +1169,17 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
         // For headers() and cookies() from next/headers, add common iterator methods
         // These are needed when the mock is passed to functions that use .entries(), .keys(), etc.
         // (e.g., Object.fromEntries(headers.entries()) in buildLegacyHeaders)
-        const needsIteratorMethods = mockName === 'headers' || mockName === 'cookies';
+        const needsIteratorMethods = baseMockName === 'headers' || baseMockName === 'cookies';
         let enhancedContents = contents;
         if (needsIteratorMethods && contents.trim().startsWith('{')) {
             // Add iterator methods that operate on the scenario data
+            // Use the dataKey (original call signature or canonical key)
             const iteratorMethods = `,
-  entries: () => Object.entries(scenarios().data()?.["${mockName}()"] || {}),
-  keys: () => Object.keys(scenarios().data()?.["${mockName}()"] || {}),
-  values: () => Object.values(scenarios().data()?.["${mockName}()"] || {}),
-  forEach: (fn) => Object.entries(scenarios().data()?.["${mockName}()"] || {}).forEach(([k, v]) => fn(v, k)),
-  has: (key) => Object.prototype.hasOwnProperty.call(scenarios().data()?.["${mockName}()"] || {}, key)`;
+  entries: () => Object.entries(scenarios().data()?.["${dataKey}"] || {}),
+  keys: () => Object.keys(scenarios().data()?.["${dataKey}"] || {}),
+  values: () => Object.values(scenarios().data()?.["${dataKey}"] || {}),
+  forEach: (fn) => Object.entries(scenarios().data()?.["${dataKey}"] || {}).forEach(([k, v]) => fn(v, k)),
+  has: (key) => Object.prototype.hasOwnProperty.call(scenarios().data()?.["${dataKey}"] || {}, key)`;
             // Insert before the closing brace (handle trailing whitespace)
             enhancedContents = contents.replace(/\}\s*$/, iteratorMethods + '\n}');
         }
@@ -1066,36 +1189,103 @@ export default function constructMockCode(mockName, dependencySchemas, entityTyp
         // `new ClassName("arg")` wouldn't create the expected instance.
         // For Error subclasses (detected by name ending in "Error"), extend Error for proper error handling.
         if (entityType === 'class') {
-            const isErrorSubclass = mockName.endsWith('Error');
-            const baseClass = isErrorSubclass ? 'Error' : 'Object';
+            const isErrorSubclass = baseMockName.endsWith('Error');
             const superCall = isErrorSubclass ? 'super(message);' : '';
             const nameAssignment = isErrorSubclass
-                ? `this.name = '${mockName}';`
+                ? `this.name = '${baseMockName}';`
                 : '';
-            return `class ${mockName}${isErrorSubclass ? ' extends Error' : ''} {
+            // Use the safe function name for the class definition
+            const className = mockNameIsCallSignature
+                ? derivedFunctionName
+                : baseMockName;
+            return `class ${className}${isErrorSubclass ? ' extends Error' : ''} {
   constructor(message) {
     ${superCall}
     ${nameAssignment}
-    Object.assign(this, scenarios().data()?.["${mockName}()"] || {});
+    Object.assign(this, scenarios().data()?.["${dataKey}"] || {});
   }
 }`;
         }
-        // For variable-qualified mocks, use underscore separator in the function name
-        // e.g., "entityDiffFetcher <- useFetcher" becomes "useFetcher_entityDiffFetcher"
-        // For indexed qualifiers like "data[1]", convert to safe identifier: "useFetcher_data_1"
-        // EXCEPTION: When keepOriginalFunctionName is true (for single-call cases), use the base name
-        const safeFunctionName = variableQualifier && !options?.keepOriginalFunctionName
-            ? `${baseMockName}_${safeVariableQualifier(variableQualifier)}`
-            : baseMockName;
-        return `const ${safeFunctionName}ReturnValue = ${enhancedContents};\n\n${isRootAsyncFunction ? 'async ' : ''}function ${safeFunctionName}() {\n${indent(`return ${safeFunctionName}ReturnValue;`)}\n}`;
+        // Generate safe function name:
+        // 1. For call signatures: use derivedFunctionName
+        //    e.g., "useFetcher<User>()" becomes "useFetcher_User"
+        //    e.g., "db.select(usersQuery)" becomes "db_select_usersQuery"
+        // 2. With uniqueFunctionSuffix option: append suffix for unique naming
+        //    e.g., baseMockName = "useFetcher", suffix = "entityDiffFetcher" -> "useFetcher_entityDiffFetcher"
+        // 3. EXCEPTION: When keepOriginalFunctionName is true (for single-call cases), use the base name
+        let safeFunctionName;
+        if (options?.keepOriginalFunctionName) {
+            safeFunctionName = baseMockName;
+        }
+        else if (options?.uniqueFunctionSuffix) {
+            safeFunctionName = `${baseMockName}_${options.uniqueFunctionSuffix}`;
+        }
+        else if (mockNameIsCallSignature && derivedFunctionName) {
+            safeFunctionName = derivedFunctionName;
+        }
+        else {
+            safeFunctionName = baseMockName;
+        }
+        // Check if this function returns a function (detected by double-call pattern: mockName(args)())
+        // This happens when the schema has keys like "wrapThrows(() => JSON.parse(savedFilters))()"
+        // where the function call is immediately followed by another call.
+        // Example usage: const result = wrapThrows(() => JSON.parse(x))(); // double call
+        const isHigherOrderFunction = Object.keys(relevantReturnValueSchema ?? {}).some((key) => {
+            if (!key.startsWith(baseMockName))
+                return false;
+            // Find the first ( after baseMockName (the start of the function call)
+            const firstOpenParen = key.indexOf('(', baseMockName.length);
+            if (firstOpenParen === -1)
+                return false;
+            // Skip if the ( is not immediately after the mock name
+            // (there might be type params like func<T>() - handle by checking for < or ()
+            const between = key.slice(baseMockName.length, firstOpenParen);
+            if (between.length > 0 && !between.startsWith('<'))
+                return false;
+            // Find the matching ) for the first ( using depth counting
+            let depth = 1;
+            let i = firstOpenParen + 1;
+            while (i < key.length && depth > 0) {
+                if (key[i] === '(')
+                    depth++;
+                if (key[i] === ')')
+                    depth--;
+                i++;
+            }
+            if (depth !== 0)
+                return false; // Unbalanced parentheses
+            // Now i points just after the matching )
+            // Check if there's another ( immediately (indicating double call)
+            const remaining = key.slice(i);
+            if (remaining.startsWith('('))
+                return true;
+            return false;
+        });
+        // For higher-order functions, wrap the return in an arrow function
+        // so that mockFunc(arg)() works correctly (outer call returns a function, inner call gets the data)
+        const returnValue = isHigherOrderFunction
+            ? `() => ${safeFunctionName}ReturnValue`
+            : `${safeFunctionName}ReturnValue`;
+        return `const ${safeFunctionName}ReturnValue = ${enhancedContents};\n\n${isRootAsyncFunction ? 'async ' : ''}function ${safeFunctionName}() {\n${indent(`return ${returnValue};`)}\n}`;
     }
     else {
-        // For variable-qualified mocks, use underscore separator in the const name
-        // For indexed qualifiers like "data[1]", convert to safe identifier: "useFetcher_data_1"
-        // EXCEPTION: When keepOriginalFunctionName is true (for single-call cases), use the base name
-        const safeName = variableQualifier && !options?.keepOriginalFunctionName
-            ? `${baseMockName}_${safeVariableQualifier(variableQualifier)}`
-            : baseMockName;
+        // Generate safe const name:
+        // 1. For call signatures: use derivedFunctionName
+        // 2. With uniqueFunctionSuffix option: append suffix for unique naming
+        // 3. EXCEPTION: When keepOriginalFunctionName is true (for single-call cases), use the base name
+        let safeName;
+        if (options?.keepOriginalFunctionName) {
+            safeName = baseMockName;
+        }
+        else if (options?.uniqueFunctionSuffix) {
+            safeName = `${baseMockName}_${options.uniqueFunctionSuffix}`;
+        }
+        else if (mockNameIsCallSignature && derivedFunctionName) {
+            safeName = derivedFunctionName;
+        }
+        else {
+            safeName = baseMockName;
+        }
         // Get any jsx-component properties that need to be preserved from the original
         const jsxProperties = getJsxComponentProperties(mockName, relevantReturnValueSchema);
         // If there are jsx-component properties, add them as references to the original