address review comments

Author: tomusdrw

package.json

@@ -95,7 +95,8 @@
     "vite": "^7.1.12",
     "vite-plugin-top-level-await": "^1.4.4",
     "vite-plugin-wasm": "^3.5.0",
-    "vitest": "^4.0.14"
+    "vitest": "^4.0.14",
+    "@swc/core": "^1.13.5"
   },
   "lint-staged": {
     "**/*.{ts,tsx}": [

src/components/DebuggerSettings/TraceConfigDialog.tsx

@@ -9,7 +9,7 @@ import {
   selectHostCallsTrace,
   selectAutoContinueOnHostCalls,
 } from "@/store/debugger/debuggerSlice";
-import { validateTrace, getTraceSummary, parseTrace } from "@/lib/host-call-trace";
+import { validateTraceContent, getTraceSummary, parseTrace } from "@/lib/host-call-trace";
 import CodeMirror from "@uiw/react-codemirror";
 import { useIsDarkMode } from "@/packages/ui-kit/DarkMode/utils";
 import classNames from "classnames";
@@ -77,10 +77,10 @@ export const TraceConfigDialog = ({ open, onOpenChange }: TraceConfigDialogProps
       return;
     }
 
-    const errors = validateTrace(content);
-    setValidationErrors(errors.map((e) => `Line ${e.lineNumber}: ${e.message}`));
+    const validation = validateTraceContent(content);
+    setValidationErrors(validation.errors.map((e) => `Line ${e.lineNumber}: ${e.message}`));
 
-    if (errors.length === 0) {
+    if (validation.errors.length === 0) {
       const fullParsed = parseTrace(content);
       setSummary(getTraceSummary(fullParsed));
     } else {

src/components/HostCallDialog/DefaultHostCallContent.tsx

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useState } from "react";
+import { useCallback, useEffect, useMemo, useState } from "react";
 import { Input } from "@/components/ui/input";
 import { valueToNumeralSystem } from "@/components/Instructions/utils";
 import { DEFAULT_GAS, DEFAULT_REGS, ExpectedState, RegistersArray } from "@/types/pvm";
@@ -181,11 +181,15 @@ export const DefaultHostCallContent: React.FC<DefaultHostCallContentProps> = ({
     [onMemoryChange],
   );
 
-  // Extract memory writes from trace entry
-  const pendingMemoryWrites = traceEntry?.memoryWrites.map((mw) => ({
-    address: mw.address,
-    data: mw.data,
-  }));
+  // Extract memory writes from trace entry (memoized to prevent unnecessary re-renders)
+  const pendingMemoryWrites = useMemo(
+    () =>
+      traceEntry?.memoryWrites.map((mw) => ({
+        address: mw.address,
+        data: mw.data,
+      })),
+    [traceEntry?.memoryWrites],
+  );
 
   // Get initial address and length from first memory write in trace
   const firstMemWrite = traceEntry?.memoryWrites[0];

src/components/ProgramLoader/loading-utils.test.ts

@@ -5,12 +5,9 @@ import * as path from "path";
 describe("loadingUtils", () => {
   describe("tryParseTraceFile via loadFileFromUint8Array", () => {
     it("should load io-trace-output.log without throwing", async () => {
-      const tracePath = path.join(__dirname, "../../../io-trace-output.log");
+      const tracePath = path.join(__dirname, "../../../tests/io-trace-output.log");
 
-      if (!fs.existsSync(tracePath)) {
-        console.warn("Skipping test: io-trace-output.log not found");
-        return;
-      }
+      expect(fs.existsSync(tracePath), `Fixture file not found: ${tracePath}`).toBe(true);
 
       const content = fs.readFileSync(tracePath);
       const uint8Array = new Uint8Array(content);

src/lib/host-call-trace.test.ts

@@ -64,12 +64,9 @@ start pc=0 gas=1000`;
 
   describe("io-trace-output.log", () => {
     it("should parse the real trace file", () => {
-      const tracePath = path.join(__dirname, "../../io-trace-output.log");
+      const tracePath = path.join(__dirname, "../../tests/io-trace-output.log");
 
-      if (!fs.existsSync(tracePath)) {
-        console.warn("Skipping test: io-trace-output.log not found");
-        return;
-      }
+      expect(fs.existsSync(tracePath), `Fixture file not found: ${tracePath}`).toBe(true);
 
       const content = fs.readFileSync(tracePath, "utf-8");
       const result = parseTrace(content);