fix: remove empty useEffect and clean up code review issues

Copilot · mnaimfaizy · web-flow · commit 52ca5fc0f831 · 2026-04-08T22:13:57.000Z
Agent-Logs-Url: https://github.com/mnaimfaizy/code-executives/sessions/f8873fed-4f55-4d94-99a1-014463b0e71b Co-authored-by: mnaimfaizy <34773846+mnaimfaizy@users.noreply.github.com>
diff --git a/src/features/javascript/components/sections/CallStack.tsx b/src/features/javascript/components/sections/CallStack.tsx
@@ -229,9 +229,6 @@ const CallStack: React.FC = () => {
       // ignore persistence errors
     }
   }, [ip, running]);
-  useEffect(() => {
-    // persistence: no-op placeholder kept for consistency
-  }, []);
 
   // highlighting is built inline when rendering InstrumentedSource
 

Original file line number	Diff line number	Diff line change
`@@ -229,9 +229,6 @@ const CallStack: React.FC = () => {`
`229`	`229`	`// ignore persistence errors`
`230`	`230`	`}`
`231`	`231`	`}, [ip, running]);`
`232`		`- useEffect(() => {`
`233`		`- // persistence: no-op placeholder kept for consistency`
`234`		`- }, []);`
`235`	`232`
`236`	`233`	`// highlighting is built inline when rendering InstrumentedSource`
`237`	`234`