fix: add null safety checks for terminal scroll refs in tests

- Add requestAnimationFrame mock in Hero tests
- Add null checks before accessing scrollHeight in TerminalPopup
- Prevents 'Cannot read properties of null' error in CI/CD
- All 278 tests now passing including Hero component tests

Author: rudra-sah00

src/components/TerminalPopup.tsx

@@ -623,8 +623,10 @@ const TerminalPopup = ({
   useEffect(() => {
     if (terminalBodyRef.current) {
       const scrollToBottom = () => {
-        terminalBodyRef.current!.scrollTop =
-          terminalBodyRef.current!.scrollHeight;
+        if (terminalBodyRef.current) {
+          terminalBodyRef.current.scrollTop =
+            terminalBodyRef.current.scrollHeight;
+        }
       };
 
       // Use requestAnimationFrame for smooth scrolling
@@ -636,10 +638,12 @@ const TerminalPopup = ({
   useEffect(() => {
     if (terminalBodyRef.current && isTyping) {
       const scrollToBottom = () => {
-        terminalBodyRef.current!.scrollTo({
-          top: terminalBodyRef.current!.scrollHeight,
-          behavior: "smooth",
-        });
+        if (terminalBodyRef.current) {
+          terminalBodyRef.current.scrollTo({
+            top: terminalBodyRef.current.scrollHeight,
+            behavior: "smooth",
+          });
+        }
       };
 
       // Small delay to ensure content is rendered

src/components/__tests__/Hero.test.tsx

@@ -35,6 +35,15 @@ describe("Hero Component", () => {
 
   beforeEach(() => {
     jest.clearAllMocks();
+    // Mock requestAnimationFrame for tests
+    jest.spyOn(window, "requestAnimationFrame").mockImplementation((cb) => {
+      cb(0);
+      return 0;
+    });
+  });
+
+  afterEach(() => {
+    jest.restoreAllMocks();
   });
 
   it("should render hero section", () => {