From 84fb586a1dcc3143bbda69650425d86baf04d8c0 Mon Sep 17 00:00:00 2001
From: bnyashwanth <bnyashwanth2006@gmail.com>
Date: Sat, 21 Feb 2026 19:54:51 +0530
Subject: [PATCH 1/3] fix(types): resolve all eslint errors and improve type
 safety (#722)

---
 src/contexts/MarkdownEditorContext.tsx      |   1 +
 src/tests/components/SettingsModal.test.tsx |  42 +++---
 src/utils/helpers/errorUtils.ts             |  51 ++++---
 src/utils/testing/setup.ts                  | 143 ++++++++++++--------
 4 files changed, 132 insertions(+), 105 deletions(-)

diff --git a/src/contexts/MarkdownEditorContext.tsx b/src/contexts/MarkdownEditorContext.tsx
index 50d40001..1690ac24 100644
--- a/src/contexts/MarkdownEditorContext.tsx
+++ b/src/contexts/MarkdownEditorContext.tsx
@@ -1,3 +1,4 @@
+/* eslint-disable react-refresh/only-export-components */
 import { createContext, useContext, useState, ReactNode } from "react";
 
 export interface MarkdownEditorCommands {
diff --git a/src/tests/components/SettingsModal.test.tsx b/src/tests/components/SettingsModal.test.tsx
index bade9242..068ee246 100644
--- a/src/tests/components/SettingsModal.test.tsx
+++ b/src/tests/components/SettingsModal.test.tsx
@@ -2,22 +2,33 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { render, screen } from '@testing-library/react';
 import SettingsModal from '../../components/SettingsModal';
 
-// Mock the store - use inline functions to avoid hoisting issues
+interface StoreState {
+  isSettingsOpen: boolean;
+  setSettingsOpen: () => void;
+  showLineNumbers: boolean;
+  setShowLineNumbers: () => void;
+  textColor: string;
+  backgroundColor: string;
+  toggleDarkMode: () => void;
+}
+
+// Update the mock
 vi.mock('../../store/store', () => {
   return {
-    default: vi.fn((selector) => selector({
-      isSettingsOpen: true,
-      setSettingsOpen: vi.fn(),
-      showLineNumbers: true,
-      setShowLineNumbers: vi.fn(),
-      textColor: '#121212',
-      backgroundColor: '#ffffff',
-      toggleDarkMode: vi.fn(),
-    })),
+    default: vi.fn((selector: (state: StoreState) => unknown) => 
+      selector({
+        isSettingsOpen: true,
+        setSettingsOpen: vi.fn(),
+        showLineNumbers: true,
+        setShowLineNumbers: vi.fn(),
+        textColor: '#121212',
+        backgroundColor: '#ffffff',
+        toggleDarkMode: vi.fn(),
+      })
+    ),
   };
 });
 
-// Mock react-dark-mode-toggle
 vi.mock('react-dark-mode-toggle', () => ({
   default: ({ checked, onChange }: { checked: boolean; onChange: () => void }) => (
     <button 
@@ -37,49 +48,42 @@ describe('SettingsModal', () => {
 
   it('renders the modal with title when open', () => {
     render(<SettingsModal />);
-    
     expect(screen.getByText('Settings')).toBeInTheDocument();
   });
 
   it('renders the Dark Mode setting', () => {
     render(<SettingsModal />);
-    
     expect(screen.getByText('Dark Mode')).toBeInTheDocument();
     expect(screen.getByText('Toggle between light and dark theme')).toBeInTheDocument();
   });
 
   it('renders the Show Line Numbers setting', () => {
     render(<SettingsModal />);
-    
     expect(screen.getByText('Show Line Numbers')).toBeInTheDocument();
     expect(screen.getByText('Display line numbers in code editors')).toBeInTheDocument();
   });
 
   it('renders the dark mode toggle button', () => {
     render(<SettingsModal />);
-    
     const toggle = screen.getByTestId('dark-mode-toggle');
     expect(toggle).toBeInTheDocument();
   });
 
   it('renders the line numbers toggle switch', () => {
     render(<SettingsModal />);
-    
     const toggle = screen.getByRole('switch', { name: /toggle line numbers/i });
     expect(toggle).toBeInTheDocument();
   });
 
   it('line numbers toggle is checked when showLineNumbers is true', () => {
     render(<SettingsModal />);
-    
     const toggle = screen.getByRole('switch', { name: /toggle line numbers/i });
     expect(toggle).toBeChecked();
   });
 
   it('renders divider between settings', () => {
     render(<SettingsModal />);
-    
     const divider = document.querySelector('hr');
     expect(divider).toBeInTheDocument();
   });
-});
+});
\ No newline at end of file
diff --git a/src/utils/helpers/errorUtils.ts b/src/utils/helpers/errorUtils.ts
index 19d247e9..44008ddb 100644
--- a/src/utils/helpers/errorUtils.ts
+++ b/src/utils/helpers/errorUtils.ts
@@ -1,23 +1,33 @@
-// Helper function to extract meaningful error message from complex error objects
+interface NestedError {
+  error?: {
+    message?: string;
+  };
+}
+
+interface CommonErrorStructure {
+  error?: string | {
+    message?: string;
+  };
+  detail?: string;
+  message?: string;
+}
+
 export const extractErrorMessage = (error: Error | unknown): string => {
   if (!error) return 'An unknown error occurred';
   
-  let errorMessage = error instanceof Error ? error.message : String(error);
+  const errorMessage = error instanceof Error ? error.message : String(error);
   
-  // Try to extract JSON from the message (might be prefixed with error type/code)
-  let jsonMatch = errorMessage.match(/\{.*\}/s);
+  const jsonMatch = errorMessage.match(/\{.*\}/s);
   if (jsonMatch) {
     try {
-      const parsed = JSON.parse(jsonMatch[0]);
+      const parsed = JSON.parse(jsonMatch[0]) as CommonErrorStructure;
       
-      // Handle Google error structure: {"error":{"message":"..."}}
       if (parsed.error) {
         if (typeof parsed.error === 'object') {
-          // Handle nested error with message
           if (parsed.error.message) {
             if (typeof parsed.error.message === 'string') {
               try {
-                const inner = JSON.parse(parsed.error.message);
+                const inner = JSON.parse(parsed.error.message) as NestedError;
                 if (inner?.error?.message) {
                   return inner.error.message;
                 }
@@ -26,39 +36,31 @@ export const extractErrorMessage = (error: Error | unknown): string => {
               }
             }
           }
-          // Handle error as string
-          if (typeof parsed.error === 'string') {
-            return parsed.error;
-          }
+        } else if (typeof parsed.error === 'string') {
+          return parsed.error;
         }
       }
       
-      // Handle Mistral error structure: {"detail":"..."}
       if (parsed.detail) {
         return parsed.detail;
       }
       
-      // Handle direct error message
       if (parsed.message) {
         return parsed.message;
       }
     } catch {
-      // JSON parsing failed, continue
+      // Ignore
     }
   }
   
-  // Try to parse the entire message as JSON (for cleanly formatted JSON errors)
   try {
-    const parsed = JSON.parse(errorMessage);
+    const parsed = JSON.parse(errorMessage) as CommonErrorStructure;
     
-    // Handle nested error structures
     if (parsed.error) {
       if (typeof parsed.error === 'object') {
-        // Handle Google nested structure
         if (parsed.error.message) {
-          // Check if the inner message is also JSON
           try {
-            const innerParsed = JSON.parse(parsed.error.message);
+            const innerParsed = JSON.parse(parsed.error.message) as NestedError;
             if (innerParsed.error && innerParsed.error.message) {
               return innerParsed.error.message;
             }
@@ -66,23 +68,20 @@ export const extractErrorMessage = (error: Error | unknown): string => {
             return parsed.error.message;
           }
         }
-      }
-      if (typeof parsed.error === 'string') {
+      } else if (typeof parsed.error === 'string') {
         return parsed.error;
       }
     }
     
-    // Handle Mistral error structure: {"detail":"..."}
     if (parsed.detail) {
       return parsed.detail;
     }
     
-    // Handle direct error message
     if (parsed.message) {
       return parsed.message;
     }
   } catch {
-    // Not JSON, return as is
+    // Ignore
   }
   
   return errorMessage;
diff --git a/src/utils/testing/setup.ts b/src/utils/testing/setup.ts
index 39126822..becb5462 100644
--- a/src/utils/testing/setup.ts
+++ b/src/utils/testing/setup.ts
@@ -8,13 +8,9 @@ afterEach(() => {
   cleanup();
 });
 
-// Mock getComputedStyle for Ant Design components that use scroll locking
-// jsdom doesn't fully support getComputedStyle with pseudo-elements
-// rc-util's getScrollBarSize calls .match() on style properties
 const originalGetComputedStyle = window.getComputedStyle;
 window.getComputedStyle = (elt: Element, pseudoElt?: string | null) => {
   if (pseudoElt) {
-    // Return a mock CSSStyleDeclaration with string properties for .match() calls
     return {
       width: '0px',
       height: '0px',
@@ -30,62 +26,89 @@ Object.defineProperty(window, "matchMedia", {
     matches: false,
     media: query,
     onchange: null,
-    addListener: () => {
-      // Mock implementation for tests
-    },
-    removeListener: () => {
-      // Mock implementation for tests
-    },
-    addEventListener: () => {
-      // Mock implementation for tests
-    },
-    removeEventListener: () => {
-      // Mock implementation for tests
-    },
-    dispatchEvent: () => {
-      return false;
-    },
+    addListener: () => { /* mock */ },
+    removeListener: () => { /* mock */ },
+    addEventListener: () => { /* mock */ },
+    removeEventListener: () => { /* mock */ },
+    dispatchEvent: () => { return false; },
   }),
 });
 
-// Mock HTMLCanvasElement.getContext for lottie-web library
-// jsdom doesn't implement canvas 2D context
-// @ts-expect-error: Mock implementation has simplified types
-HTMLCanvasElement.prototype.getContext = ((originalGetContext) => {
-  return function (
-    this: HTMLCanvasElement,
-    contextId: string,
-    options?: unknown
-  ) {
-    if (contextId === '2d') {
-      return {
-        fillStyle: '',
-        fillRect: () => {},
-        clearRect: () => {},
-        getImageData: () => ({ data: [] }),
-        putImageData: () => {},
-        createImageData: () => ({ data: [] }),
-        setTransform: () => {},
-        drawImage: () => {},
-        save: () => {},
-        restore: () => {},
-        beginPath: () => {},
-        moveTo: () => {},
-        lineTo: () => {},
-        closePath: () => {},
-        stroke: () => {},
-        fill: () => {},
-        translate: () => {},
-        scale: () => {},
-        rotate: () => {},
-        arc: () => {},
-        measureText: () => ({ width: 0 }),
-        transform: () => {},
-        rect: () => {},
-        clip: () => {},
-        canvas: this,
-      } as unknown as CanvasRenderingContext2D;
-    }
-    return originalGetContext.call(this, contextId as any, options);
-  };
-})(HTMLCanvasElement.prototype.getContext);
+interface MockCanvasContext {
+  fillStyle: string;
+  fillRect: () => void;
+  clearRect: () => void;
+  getImageData: () => { data: number[] };
+  putImageData: () => void;
+  createImageData: () => { data: number[] };
+  setTransform: () => void;
+  drawImage: () => void;
+  save: () => void;
+  restore: () => void;
+  beginPath: () => void;
+  moveTo: () => void;
+  lineTo: () => void;
+  closePath: () => void;
+  stroke: () => void;
+  fill: () => void;
+  translate: () => void;
+  scale: () => void;
+  rotate: () => void;
+  arc: () => void;
+  measureText: () => { width: number };
+  transform: () => void;
+  rect: () => void;
+  clip: () => void;
+  canvas: HTMLCanvasElement;
+}
+
+// capture original method so we can delegate for non-2d calls
+// use a loose any type to avoid lib.dom union conflicts
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+const originalGetContext: any = HTMLCanvasElement.prototype.getContext.bind(
+  HTMLCanvasElement.prototype
+);
+
+HTMLCanvasElement.prototype.getContext = function (
+  this: HTMLCanvasElement,
+  contextId: string,
+  options?: unknown
+) {
+  if (contextId === '2d') {
+    const mockContext: MockCanvasContext = {
+      fillStyle: '',
+      fillRect: () => { /* mock */ },
+      clearRect: () => { /* mock */ },
+      getImageData: () => ({ data: [] }),
+      putImageData: () => { /* mock */ },
+      createImageData: () => ({ data: [] }),
+      setTransform: () => { /* mock */ },
+      drawImage: () => { /* mock */ },
+      save: () => { /* mock */ },
+      restore: () => { /* mock */ },
+      beginPath: () => { /* mock */ },
+      moveTo: () => { /* mock */ },
+      lineTo: () => { /* mock */ },
+      closePath: () => { /* mock */ },
+      stroke: () => { /* mock */ },
+      fill: () => { /* mock */ },
+      translate: () => { /* mock */ },
+      scale: () => { /* mock */ },
+      rotate: () => { /* mock */ },
+      arc: () => { /* mock */ },
+      measureText: () => ({ width: 0 }),
+      transform: () => { /* mock */ },
+      rect: () => { /* mock */ },
+      clip: () => { /* mock */ },
+      canvas: this,
+    };
+    // cast through unknown since the mock only implements a subset of methods
+    return mockContext as unknown as CanvasRenderingContext2D;
+  }
+
+  // for other context types, delegate to original
+  // originalGetContext is typed as `any` above, so suppress safety checks
+  /* eslint-disable @typescript-eslint/no-unsafe-return, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access */
+  return originalGetContext.call(this, contextId, options);
+  /* eslint-enable @typescript-eslint/no-unsafe-return, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access */
+};
\ No newline at end of file

From c1320433551d26014ed6970bf49c14bd01ddd9d2 Mon Sep 17 00:00:00 2001
From: bnyashwanth <bnyashwanth2006@gmail.com>
Date: Wed, 4 Mar 2026 09:26:06 +0530
Subject: [PATCH 2/3] DCO Remediation Commit for bnyashwanth
 <bnyashwanth2006@gmail.com>

I, bnyashwanth <bnyashwanth2006@gmail.com>, hereby add my Signed-off-by to this commit: 84fb586a1dcc3143bbda69650425d86baf04d8c0

Signed-off-by: bnyashwanth <bnyashwanth2006@gmail.com>
---
 .gitignore | Bin 336 -> 372 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)

diff --git a/.gitignore b/.gitignore
index 04471798a4133cd0b0b0b3b4761d38b2f4685ae3..757c1ca4df08e6e89c04489689f5f52432f40479 100644
GIT binary patch
literal 372
zcmZWjNp8b14D?xm{-7ea0G6N7OMB~~Kpqgt66?rNM1hpE{JxaaCS5KZ&Txk0#=POg
zTBWsat)N!99Qy#{GBvu@>gp)}7k~)q%TfMD5>R%(GO`u}?2j0h1adJ%uD9AVlUD7v
zcS$R)Hs&?Niey40@X|zJ<8m+`agEv&d)YO}Yfs-jTq<OanAy&ZSt}a@IDwblekUuS
zUEl>9<a%PsN;7jIoYzs3j5yBEw-4TvyPhdd2YGU+z_lmfB~>1Tm-|rEf>fJ0CLDjz
Qhi9<f`2&c@bh=aBCru!8wEzGB

literal 336
zcmZWi%WlIk47=+qT=W(o_9yJJ-Fhg{4=5a4OC7k9Avt~g{p=1~hxI~<lt^yPj%S73
zVX$ogmGc;JTIbFNxO<@&jv_4I_ww&Qqmch=0OE+pGsiWfLQGm8FsjO<G58FyH6Jlm
z788?TXiGvho}&3o-DNKtT8bNB^RwcuB57h$yC@Ub6w!lyKO7IujxAbOZb<c_L!XNl
x;ic=DI$qy;?LPbaRnmN_;3{ez1$E6;9k}v)YGPrjO`9@LPxkz9Hu!Kt2Vdd$argiL


From 62c5faa4935a744b61eed1ba62c7538069573d00 Mon Sep 17 00:00:00 2001
From: bnyashwanth <bnyashwanth2006@gmail.com>
Date: Sun, 15 Mar 2026 20:14:49 +0530
Subject: [PATCH 3/3] fix(lint): resolve follow-up review findings from #825

Signed-off-by: bnyashwanth <bnyashwanth2006@gmail.com>
---
 src/contexts/MarkdownEditorContext.tsx      | 1 -
 src/tests/components/SettingsModal.test.tsx | 2 +-
 src/utils/helpers/errorUtils.ts             | 2 --
 3 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/contexts/MarkdownEditorContext.tsx b/src/contexts/MarkdownEditorContext.tsx
index d4fdde9a..c9907035 100644
--- a/src/contexts/MarkdownEditorContext.tsx
+++ b/src/contexts/MarkdownEditorContext.tsx
@@ -33,7 +33,6 @@ export const MarkdownEditorProvider = ({ children }: { children: ReactNode }) =>
   );
 };
 // eslint-disable-next-line react-refresh/only-export-components
-
 export const useMarkdownEditorContext = () => {
   const context = useContext(MarkdownEditorContext);
   if (context === undefined) {
diff --git a/src/tests/components/SettingsModal.test.tsx b/src/tests/components/SettingsModal.test.tsx
index cc618132..3e96a14b 100644
--- a/src/tests/components/SettingsModal.test.tsx
+++ b/src/tests/components/SettingsModal.test.tsx
@@ -12,7 +12,7 @@ interface StoreState {
   toggleDarkMode: () => void;
 }
 
-// Update the mock
+// Mock the Zustand store selector with a fixed settings state for isolated SettingsModal tests
 vi.mock('../../store/store', () => {
   return {
     default: vi.fn((selector: (state: StoreState) => unknown) => 
diff --git a/src/utils/helpers/errorUtils.ts b/src/utils/helpers/errorUtils.ts
index a211f02a..2fb77ad7 100644
--- a/src/utils/helpers/errorUtils.ts
+++ b/src/utils/helpers/errorUtils.ts
@@ -36,8 +36,6 @@ export const extractErrorMessage = (error: Error | unknown): string => {
               }
             }
           }
-        } else if (typeof parsed.error === 'string') {
-
         }
         // Handle error as string
         if (typeof parsed.error === 'string') {