Refactor RequestHistory and useApiTester components to simplify code; remove unused formatTime function and bodyFormat state, enhancing maintainability

Author: yceruto

spa/src/components/api-tester/RequestHistory.tsx

@@ -15,11 +15,6 @@ function statusColor(status: number): string {
   return 'text-rose-600 dark:text-rose-400';
 }
 
-function formatTime(ts: number): string {
-  const d = new Date(ts);
-  return d.toLocaleTimeString([], { hour: '2-digit', minute: '2-digit', second: '2-digit' });
-}
-
 interface RequestHistoryProps {
   history: HistoryEntry[];
   search: string;
@@ -75,14 +70,11 @@ export function RequestHistory({ history, search, onSearchChange, onSelect, acti
                   {entry.method}
                 </span>
                 <span className="flex-1 text-xs font-mono text-slate-600 dark:text-slate-400 truncate">{entry.path}</span>
-                <div className="shrink-0 flex flex-col items-end gap-0.5">
-                  {entry.response && (
-                    <span className={`text-[10px] font-semibold ${statusColor(entry.response.status)}`}>
-                      {entry.response.status || 'ERR'}
-                    </span>
-                  )}
-                  <span className="text-[10px] text-slate-400 dark:text-slate-500">{formatTime(entry.timestamp)}</span>
-                </div>
+                {entry.response && (
+                  <span className={`shrink-0 text-xs font-semibold ${statusColor(entry.response.status)}`}>
+                    {entry.response.status || 'ERR'}
+                  </span>
+                )}
               </button>
             ))}
           </div>

spa/src/components/api-tester/useApiTester.ts

@@ -9,15 +9,12 @@ export interface KeyValuePair {
   enabled: boolean;
 }
 
-export type BodyFormat = 'json' | 'xml';
-
 export interface RequestState {
   method: string;
   url: string;
   pathParams: KeyValuePair[];
   queryParams: KeyValuePair[];
   headers: KeyValuePair[];
-  bodyFormat: BodyFormat;
   body: string;
 }
 
@@ -114,7 +111,6 @@ function buildInitialRequest(endpoint: Endpoint, baseUrl: string, spec: OpenApiS
     pathParams,
     queryParams,
     headers,
-    bodyFormat: 'json',
     body,
   };
 }
@@ -174,7 +170,6 @@ export interface UseApiTesterReturn {
   setPathParams: (params: KeyValuePair[]) => void;
   setQueryParams: (params: KeyValuePair[]) => void;
   setHeaders: (headers: KeyValuePair[]) => void;
-  setBodyFormat: (format: BodyFormat) => void;
   setBody: (body: string) => void;
   setHistorySearch: (search: string) => void;
   sendRequest: () => Promise<void>;
@@ -214,10 +209,6 @@ export function useApiTester(endpoint: Endpoint, spec: OpenApiSpec): UseApiTeste
     setRequest(prev => ({ ...prev, headers }));
   }, []);
 
-  const setBodyFormat = useCallback((bodyFormat: BodyFormat) => {
-    setRequest(prev => ({ ...prev, bodyFormat }));
-  }, []);
-
   const setBody = useCallback((body: string) => {
     setRequest(prev => ({ ...prev, body }));
   }, []);
@@ -346,7 +337,6 @@ export function useApiTester(endpoint: Endpoint, spec: OpenApiSpec): UseApiTeste
     setPathParams,
     setQueryParams,
     setHeaders,
-    setBodyFormat,
     setBody,
     setHistorySearch,
     sendRequest,