Merge pull request #98 from bocan:just-a-friday

Improve type safety and update dependencies

Author: bocan

client/package.json

@@ -20,7 +20,7 @@
     "react": "^19.2.4",
     "react-dom": "^19.2.4",
     "react-markdown": "^10.1.0",
-    "react-syntax-highlighter": "^16.1.0",
+    "react-syntax-highlighter": "^16.1.1",
     "rehype-autolink-headings": "^7.1.0",
     "rehype-slug": "^6.0.0",
     "remark-gfm": "^4.0.0",

client/src/App.tsx

@@ -271,10 +271,10 @@ function App() {
       if (!selectedFolder) {
         setSelectedFolder(tree.path);
       }
-    } catch (error: any) {
+    } catch (error: unknown) {
       console.error("Failed to load folder tree:", error);
       const message =
-        error.response?.status === 401
+        (error as { response?: { status?: number } })?.response?.status === 401
           ? "Session expired. Please log in again."
           : "Failed to load folders. Please try refreshing the page.";
       setError(message);
@@ -302,10 +302,10 @@ function App() {
         if (readme) {
           setSelectedPage(readme.path);
         }
-      } catch (error: any) {
+      } catch (error: unknown) {
         console.error("Failed to check for README.md:", error);
         const message =
-          error.response?.status === 401
+          (error as { response?: { status?: number } })?.response?.status === 401
             ? "Session expired. Please log in again."
             : `Failed to load pages from folder: ${path}`;
         setError(message);

client/src/components/Attachments.tsx

@@ -35,7 +35,7 @@ export const Attachments: React.FC<AttachmentsProps> = ({
     setError(null);
     try {
       const data = await api.getAttachments(folderPath);
-      setAttachments(data);
+      setAttachments(data as Attachment[]);
     } catch (err) {
       console.error("Failed to load attachments:", err);
       setError("Failed to load attachments");

client/src/components/Editor.tsx

@@ -146,9 +146,15 @@ export const Editor: React.FC<EditorProps> = ({ pagePath, onClose }) => {
 
   // Speech recognition functions
   const startListening = () => {
+    interface WindowWithSpeech extends Window {
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      SpeechRecognition?: { new(): any };
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      webkitSpeechRecognition?: { new(): any };
+    }
     const SpeechRecognition =
-      (window as any).SpeechRecognition ||
-      (window as any).webkitSpeechRecognition;
+      (window as unknown as WindowWithSpeech).SpeechRecognition ||
+      (window as unknown as WindowWithSpeech).webkitSpeechRecognition;
     if (!SpeechRecognition) {
       alert(
         "Speech recognition is not supported in this browser. Try Chrome or Edge.",
@@ -163,10 +169,18 @@ export const Editor: React.FC<EditorProps> = ({ pagePath, onClose }) => {
 
     let finalTranscript = "";
 
-    recognition.onresult = (event: any) => {
-      for (let i = event.resultIndex; i < event.results.length; i++) {
-        const transcript = event.results[i][0].transcript;
-        if (event.results[i].isFinal) {
+    recognition.onresult = (event: unknown) => {
+      const typedEvent = event as {
+        resultIndex: number;
+        results: { isFinal: boolean; [index: number]: { transcript: string } }[];
+      };
+      for (
+        let i = typedEvent.resultIndex;
+        i < typedEvent.results.length;
+        i++
+      ) {
+        const transcript = typedEvent.results[i][0].transcript;
+        if (typedEvent.results[i].isFinal) {
           finalTranscript += transcript + " ";
         }
       }
@@ -197,9 +211,10 @@ export const Editor: React.FC<EditorProps> = ({ pagePath, onClose }) => {
       }
     };
 
-    recognition.onerror = (event: any) => {
-      console.error("Speech recognition error:", event.error);
-      if (event.error === "not-allowed") {
+    recognition.onerror = (event: unknown) => {
+      const typedEvent = event as { error?: string };
+      console.error("Speech recognition error:", typedEvent.error);
+      if (typedEvent.error === "not-allowed") {
         alert(
           "Microphone access denied. Please allow microphone access and try again.",
         );
@@ -240,7 +255,6 @@ export const Editor: React.FC<EditorProps> = ({ pagePath, onClose }) => {
     textarea.focus();
 
     // execCommand preserves native undo stack
-    // eslint-disable-next-line @typescript-eslint/no-deprecated
     const success = document.execCommand("insertText", false, text);
 
     if (!success) {

client/src/components/Login.tsx

@@ -25,9 +25,11 @@ export const Login: React.FC<LoginProps> = ({ onLoginSuccess }) => {
     try {
       await api.login(password);
       onLoginSuccess();
-    } catch (err: any) {
+    } catch (err: unknown) {
       console.error("Login failed:", err);
-      setError(err.response?.data?.error || "Invalid password");
+      setError(
+        (err as { response?: { data?: { error?: string } } })?.response?.data?.error || "Invalid password"
+      );
     } finally {
       setIsLoading(false);
     }

client/src/components/PageList.tsx

@@ -339,10 +339,10 @@ export const PageList: React.FC<PageListProps> = ({
         onSelectPage(result.newPath);
       }
       setMovingPage(null);
-    } catch (err: any) {
+    } catch (err: unknown) {
       console.error("Failed to move page:", err);
       setError(
-        err.response?.data?.error || "Failed to move page. Please try again.",
+        (err as { response?: { data?: { error?: string } } })?.response?.data?.error || "Failed to move page. Please try again.",
       );
       setMovingPage(null);
     } finally {

client/src/components/Preview.tsx

@@ -1450,7 +1450,7 @@ ${htmlContent}
     const blob =
       docxBlob instanceof Blob
         ? docxBlob
-        : new Blob([new Uint8Array(docxBlob as any)], {
+        : new Blob([new Uint8Array((docxBlob as unknown) as ArrayBuffer)], {
             type: "application/vnd.openxmlformats-officedocument.wordprocessingml.document",
           });
 

client/src/services/api.ts

@@ -149,7 +149,7 @@ export const api = {
     });
   },
 
-  getAttachments: async (folderPath: string): Promise<any[]> => {
+  getAttachments: async (folderPath: string): Promise<unknown[]> => {
     const response = await axios.get(`${API_BASE}/attachments`, {
       params: { folder: folderPath },
     });