Merge pull request #49 from typelets/chore/update-dependencies

Chore/update dependencies

Author: typelets

.husky/pre-commit

@@ -1,6 +1,3 @@
-#!/usr/bin/env sh
-. "$(dirname "$0")/_/husky.sh"
-
 # Load nvm
 export NVM_DIR="$HOME/.nvm"
 [ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"

.husky/pre-push

@@ -1,6 +1,3 @@
-#!/usr/bin/env sh
-. "$(dirname "$0")/_/husky.sh"
-
 # Load nvm
 export NVM_DIR="$HOME/.nvm"
 [ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"

package.json

@@ -56,7 +56,9 @@
   },
   "pnpm": {
     "overrides": {
-      "tmp": ">=0.2.4"
+      "tmp": ">=0.2.4",
+      "glob": ">=11.1.0",
+      "js-yaml": ">=4.1.1"
     }
   },
   "dependencies": {
@@ -131,7 +133,7 @@
     "lint-staged": "^15.5.2",
     "prettier": "^3.6.2",
     "prettier-plugin-tailwindcss": "^0.6.14",
-    "rimraf": "^6.0.1",
+    "rimraf": "^6.1.2",
     "semantic-release": "^24.2.9",
     "tw-animate-css": "^1.3.8",
     "typescript": "~5.9.2",

pnpm-lock.yaml

@@ -1,4 +1,4 @@
-import { useState, useEffect } from 'react';
+import { useState } from 'react';
 import { X, Smartphone } from 'lucide-react';
 import { Button } from '@/components/ui/button';
 import { useMobilePlatform } from '@/hooks/useIsMobile';
@@ -10,15 +10,11 @@ const BANNER_DISMISSED_KEY = 'typelets_mobile_banner_dismissed';
 
 export function MobileAppBanner() {
   const platform = useMobilePlatform();
-  const [isDismissed, setIsDismissed] = useState(true);
-
-  useEffect(() => {
-    // Check if banner was previously dismissed
+  // Initialize state based on localStorage to avoid setState in effect
+  const [isDismissed, setIsDismissed] = useState(() => {
     const dismissed = localStorage.getItem(BANNER_DISMISSED_KEY);
-    if (!dismissed && platform) {
-      setIsDismissed(false);
-    }
-  }, [platform]);
+    return !!dismissed || !platform;
+  });
 
   const handleDismiss = () => {
     setIsDismissed(true);

src/components/common/MobileAppBanner/index.tsx

@@ -59,10 +59,12 @@ export const NoteLinkSuggestionList = forwardRef<
     );
   }, [filteredItems.length]);
 
-  // Reset selection when items change
+  // Reset selection when filtered items change
+  /* eslint-disable react-hooks/set-state-in-effect -- Legitimate reset on data change */
   useEffect(() => {
     setSelectedIndex(0);
   }, [filteredItems]);
+  /* eslint-enable react-hooks/set-state-in-effect */
 
   useImperativeHandle(
     ref,

src/components/editor/extensions/NoteLinkSuggestionList.tsx

@@ -1,7 +1,7 @@
 import { Node, mergeAttributes } from '@tiptap/core';
 import { ReactNodeViewRenderer } from '@tiptap/react';
 import { NodeViewWrapper } from '@tiptap/react';
-import { useState, useRef, useEffect } from 'react';
+import { useState, useEffect, useRef } from 'react';
 import { X, Maximize2 } from 'lucide-react';
 
 const ImageComponent = (props: {
@@ -19,15 +19,16 @@ const ImageComponent = (props: {
   const { node, deleteNode, updateAttributes } = props;
   const [showControls, setShowControls] = useState(false);
   const [isResizing, setIsResizing] = useState(false);
+  const [width, setWidth] = useState(node.attrs.width || 'auto');
   const imageRef = useRef<HTMLImageElement>(null);
   const containerRef = useRef<HTMLDivElement>(null);
 
-  const initialWidth = node.attrs.width || 'auto';
-  const [width, setWidth] = useState(initialWidth);
-
+  // Sync width with node attrs
+  /* eslint-disable react-hooks/set-state-in-effect -- Sync external prop to local state */
   useEffect(() => {
     setWidth(node.attrs.width || 'auto');
   }, [node.attrs.width]);
+  /* eslint-enable react-hooks/set-state-in-effect */
 
   const handleResize = (e: React.MouseEvent) => {
     e.preventDefault();

src/components/editor/extensions/ResizableImage.tsx

@@ -1,6 +1,5 @@
 import {
   useState,
-  useEffect,
   useImperativeHandle,
   forwardRef,
   useCallback,
@@ -277,10 +276,6 @@ export const SlashCommandsList = forwardRef<
     setSelectedIndex((selectedIndex + 1) % commands.length);
   }, [selectedIndex]);
 
-  useEffect(() => {
-    setSelectedIndex(0);
-  }, []);
-
   useImperativeHandle(
     ref,
     () => ({

src/components/editor/extensions/SlashCommands.tsx

@@ -151,6 +151,7 @@ export function useEditorEffects({
 
       const editorElement = editor.view.dom as HTMLElement;
 
+      /* eslint-disable react-hooks/immutability -- Modifying DOM element style, not React state */
       if (zoomLevel === 100) {
         // At 100%, use the original font size
         editorElement.style.fontSize = baseFontSize;
@@ -161,6 +162,7 @@ export function useEditorEffects({
         const newSize = baseSize * scaleFactor;
         editorElement.style.fontSize = `${newSize}px`;
       }
+      /* eslint-enable react-hooks/immutability */
     } catch {
       // Silently ignore zoom level application errors
     }

src/components/editor/hooks/useEditorEffects.ts

@@ -34,12 +34,18 @@ export default function MainLayout() {
   const [activeTabId, setActiveTabId] = useState<string | null>(null);
 
   // Sync responsive panel states with local states for desktop
+  /* eslint-disable react-hooks/set-state-in-effect -- Sync external state to local state */
   useEffect(() => {
     if (!isMobile) {
-      setFolderSidebarOpen(responsiveFolderPanel.isOpen);
-      setFilesPanelOpen(responsiveNotesPanel.isOpen);
+      if (folderSidebarOpen !== responsiveFolderPanel.isOpen) {
+        setFolderSidebarOpen(responsiveFolderPanel.isOpen);
+      }
+      if (filesPanelOpen !== responsiveNotesPanel.isOpen) {
+        setFilesPanelOpen(responsiveNotesPanel.isOpen);
+      }
     }
-  }, [isMobile, responsiveFolderPanel.isOpen, responsiveNotesPanel.isOpen]);
+  }, [isMobile, responsiveFolderPanel.isOpen, responsiveNotesPanel.isOpen, folderSidebarOpen, filesPanelOpen]);
+  /* eslint-enable react-hooks/set-state-in-effect */
 
   const {
     notes,
@@ -280,17 +286,19 @@ export default function MainLayout() {
   }, [selectedNote]);
 
   // Update tab properties when note changes
+  /* eslint-disable react-hooks/set-state-in-effect -- Sync note properties to tab state */
   useEffect(() => {
-    if (!selectedNote?.id) return;
-
-    setOpenTabs(tabs =>
-      tabs.map(tab =>
-        tab.noteId === selectedNote.id
-          ? { ...tab, title: selectedNote.title || 'Untitled', type: selectedNote.type || 'note', isPublished: selectedNote.isPublished }
-          : tab
-      )
-    );
-  }, [selectedNote]);
+    if (selectedNote?.id) {
+      setOpenTabs(tabs =>
+        tabs.map(tab =>
+          tab.noteId === selectedNote.id
+            ? { ...tab, title: selectedNote.title || 'Untitled', type: selectedNote.type || 'note', isPublished: selectedNote.isPublished }
+            : tab
+        )
+      );
+    }
+  }, [selectedNote?.id, selectedNote?.title, selectedNote?.type, selectedNote?.isPublished]);
+  /* eslint-enable react-hooks/set-state-in-effect */
 
   const handlePasswordChange = useCallback(() => {
     setSelectedNote(null);