Refactor code.

Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>

Author: bgravenorst

src/components/ParserOpenRPC/CollapseBox/CollapseBox.tsx

@@ -8,16 +8,16 @@ import styles from './styles.module.scss'
 
 interface CollapseBoxProps {
   children: JSX.Element
-  isInitCollapsed?: boolean
+  isInitExpanded?: boolean
 }
 
-export const CollapseBox = ({ children, isInitCollapsed = false }: CollapseBoxProps) => {
+export const CollapseBox = ({ children, isInitExpanded = false }: CollapseBoxProps) => {
   const { collapsed, toggleCollapsed } = useCollapsible({ initialState: true })
   useEffect(() => {
-    if (isInitCollapsed) {
+    if (isInitExpanded) {
       toggleCollapsed()
     }
-  }, [isInitCollapsed])
+  }, [isInitExpanded])
   return (
     <div className={clsx(styles.collapseWrapper, !collapsed && styles.collapsedWrapperView)}>
       <button

src/components/ParserOpenRPC/DetailsBox/RenderParams.tsx

@@ -45,44 +45,17 @@ const getArrayTypeDescription = (items, schemas) => {
     return 'array'
   }
 
-  // Handle basic types
-  const itemType = items.type || items.schema?.type
-  if (itemType) {
-    switch (itemType) {
-      case 'string':
-        return 'array of strings'
-      case 'number':
-        return 'array of numbers'
-      case 'integer':
-        return 'array of integers'
-      case 'boolean':
-        return 'array of booleans'
-      case 'null':
-        return 'array of null values'
-      default:
-        return 'array'
-    }
-  }
-
-  // Handle case where items is just a type string (fallback)
-  if (typeof items === 'string') {
-    switch (items) {
-      case 'string':
-        return 'array of strings'
-      case 'number':
-        return 'array of numbers'
-      case 'integer':
-        return 'array of integers'
-      case 'boolean':
-        return 'array of booleans'
-      case 'null':
-        return 'array of null values'
-      default:
-        return 'array'
-    }
+  // Handle basic types with lookup table
+  const typeLabels: Record<string, string> = {
+    string: 'array of strings',
+    number: 'array of numbers',
+    integer: 'array of integers',
+    boolean: 'array of booleans',
+    null: 'array of null values',
   }
 
-  return 'array'
+  const type = typeof items === 'string' ? items : items.type || items.schema?.type
+  return typeLabels[type] || 'array'
 }
 
 const renderSchema = (
@@ -121,7 +94,7 @@ const renderSchema = (
           showRequired={showRequired}
         />
         <div className="padding-bottom--md">
-          <CollapseBox isInitCollapsed={isExpandedByDefault}>
+          <CollapseBox isInitExpanded={isExpandedByDefault}>
             <>
               {Object.entries(item.properties).map(([key, value]: [string, SchemaPropertyType]) => (
                 <div key={key} className={styles.paramItemWrapper}>
@@ -207,7 +180,7 @@ const renderSchema = (
         />
         {shouldShowDetails && (
           <div className="padding-bottom--md">
-            <CollapseBox isInitCollapsed={isExpandedByDefault}>
+            <CollapseBox isInitExpanded={isExpandedByDefault}>
               {(() => {
                 // Check if array items are objects - if so, render properties directly (flatter structure)
                 if (arrayType === 'array of objects') {
@@ -258,7 +231,7 @@ const renderSchema = (
         showRequired={showRequired}
       />
       <div className="padding-bottom--md">
-        <CollapseBox isInitCollapsed={false}>
+        <CollapseBox isInitExpanded={false}>
           {item[type].map((option, index) => (
             <div key={`${index}`} className={styles.paramItemWrapper}>
               {renderSchema(option, schemas, option.title, showRequired, isExpandedByDefault)}