chore(mobile): remove unnecessary comments and fix linting issues

Author: batalabs

apps/mobile/v1/src/components/FileUpload.tsx

@@ -10,7 +10,7 @@ import {
 import { Ionicons } from '@expo/vector-icons';
 import * as Haptics from 'expo-haptics';
 import { useTheme } from '../theme';
-import { useApiService, type FileAttachment, type PickedFile } from '../services/api';
+import { useApiService, type FileAttachment } from '../services/api';
 
 interface FileUploadProps {
   noteId: string;
@@ -37,7 +37,7 @@ export function FileUpload({
       const files = await api.pickFiles();
 
       if (files.length === 0) {
-        return; // User cancelled
+        return;
       }
 
       setIsUploading(true);

apps/mobile/v1/src/hooks/useNoteEditor.ts

@@ -6,7 +6,6 @@ import { useTheme } from '../theme';
 import { useApiService, type Note } from '../services/api';
 import { generateEditorStyles } from '../screens/EditNote/styles';
 
-const EDITOR_LOAD_DELAY = 500;
 const CSS_INJECTION_DELAY = 100;
 
 interface UseNoteEditorReturn {
@@ -27,29 +26,25 @@ export function useNoteEditor(noteId?: string): UseNoteEditorReturn {
   const editorReadyRef = useRef(false);
   const pendingContentRef = useRef<string | null>(null);
 
-  // Initialize editor with TenTapStartKit
   const editor = useEditorBridge({
     autofocus: false,
     avoidIosKeyboard: true,
     initialContent: '<p></p>',
     bridgeExtensions: TenTapStartKit,
   });
 
-  // Generate custom CSS memoized on theme colors
   const customCSS = useMemo(
     () => generateEditorStyles(theme.colors),
     [theme.colors]
   );
 
-  // Handler for when WebView loads - inject CSS at the right time
   const handleEditorLoad = useCallback(() => {
     editor.injectCSS(customCSS, 'theme-css');
 
-    // Also inject after a slight delay to ensure it overrides TenTap's default styles
+    // Inject after delay to override TenTap's default styles
     setTimeout(() => {
       editor.injectCSS(customCSS, 'theme-css');
 
-      // Mark editor as ready and set pending content if any
       editorReadyRef.current = true;
       if (pendingContentRef.current !== null) {
         if (__DEV__) {
@@ -61,7 +56,6 @@ export function useNoteEditor(noteId?: string): UseNoteEditorReturn {
     }, CSS_INJECTION_DELAY);
   }, [editor, customCSS]);
 
-  // Load note content if editing
   const loadNote = useCallback(async () => {
     if (!noteId) return;
 
@@ -73,7 +67,6 @@ export function useNoteEditor(noteId?: string): UseNoteEditorReturn {
 
       const content = note.content || '';
 
-      // If editor is ready, set content immediately
       if (editorReadyRef.current) {
         setTimeout(() => {
           if (__DEV__) {
@@ -82,7 +75,6 @@ export function useNoteEditor(noteId?: string): UseNoteEditorReturn {
           editor.setContent(content);
         }, 100);
       } else {
-        // Otherwise, store content to be set when editor is ready
         if (__DEV__) {
           console.log('Editor not ready yet, storing content...');
         }

apps/mobile/v1/src/screens/EditNote/index.tsx

@@ -1,5 +1,5 @@
 import React, { useState, useEffect } from 'react';
-import { View, Text, StyleSheet, Alert, TextInput, Platform, ScrollView } from 'react-native';
+import { View, Text, StyleSheet, Alert, TextInput, Platform } from 'react-native';
 import { SafeAreaView } from 'react-native-safe-area-context';
 import { useRouter, useLocalSearchParams } from 'expo-router';
 import * as Haptics from 'expo-haptics';
@@ -40,7 +40,6 @@ export default function EditNoteScreen() {
           setNoteData(note || null);
           setTitle(note?.title || '');
 
-          // Load attachments
           if (note) {
             const noteAttachments = await api.getAttachments(noteId as string);
             setAttachments(noteAttachments);
@@ -92,6 +91,7 @@ export default function EditNoteScreen() {
           starred: false,
           archived: false,
           deleted: false,
+          hidden: false,
         });
       }
 
@@ -195,7 +195,6 @@ export default function EditNoteScreen() {
           </View>
         )}
 
-        {/* File Attachments - only show when toggled */}
         {isEditing && noteId && showAttachments && (
           <View style={styles.attachmentsSection}>
             <FileUpload

apps/mobile/v1/src/screens/ViewNote/index.tsx

@@ -26,7 +26,6 @@ export default function ViewNoteScreen() {
   const { note, loading, htmlContent, handleEdit, handleToggleStar, handleToggleHidden } = useViewNote(noteId as string);
 
   useEffect(() => {
-    // Reset scroll position when note changes
     scrollY.setValue(0);
     if (scrollViewRef.current) {
       scrollViewRef.current.scrollTo({ y: 0, animated: false });
@@ -54,7 +53,6 @@ export default function ViewNoteScreen() {
     }
   }, [noteId, api]);
 
-  // Load attachments when note loads
   useEffect(() => {
     if (noteId && lastLoadedNoteId.current !== noteId) {
       loadAttachments();
@@ -69,11 +67,8 @@ export default function ViewNoteScreen() {
 
     try {
       setDownloadingId(attachment.id);
-      console.log('Starting download for:', attachment.originalName);
       const fileUri = await api.downloadFile(attachment);
-      console.log('File downloaded to:', fileUri);
       await api.shareFile(fileUri);
-      console.log('Share dialog opened');
     } catch (error) {
       console.error('Failed to download attachment:', error);
       const errorMessage = error instanceof Error ? error.message : 'Unknown error occurred';
@@ -88,19 +83,15 @@ export default function ViewNoteScreen() {
     }
   };
 
-  // Reload attachments when coming back to this screen
   useFocusEffect(
     useCallback(() => {
-      // Reset scroll position when screen comes into focus
       scrollY.setValue(0);
       if (scrollViewRef.current) {
         scrollViewRef.current.scrollTo({ y: 0, animated: false });
       }
 
-      // Force reload attachments when coming back (e.g., from EditNote)
-      // But only if this is our note and we're not already loading
       if (noteId && lastLoadedNoteId.current === noteId && !loadingRef.current) {
-        lastLoadedNoteId.current = null; // Reset to force reload
+        lastLoadedNoteId.current = null;
         loadAttachments();
       }
       // eslint-disable-next-line react-hooks/exhaustive-deps

apps/mobile/v1/src/services/fileService.ts

@@ -11,7 +11,6 @@ import forge from 'node-forge';
 import { encryptWithAESGCM, decryptWithAESGCM } from '../lib/encryption/core/aes';
 import { deriveEncryptionKey } from '../lib/encryption/core/keyDerivation';
 import { ENCRYPTION_CONFIG } from '../lib/encryption/config';
-import { encryptionService } from '../lib/encryption';
 import { getUserSecret, getMasterKey } from '../lib/encryption/storage/secureStorage';
 import type { FileAttachment } from './api/types';
 
@@ -70,7 +69,7 @@ class FileService {
    */
   private base64ToArrayBuffer(base64: string): ArrayBuffer {
     const bytes = this.base64ToUint8Array(base64);
-    return bytes.buffer.slice(bytes.byteOffset, bytes.byteOffset + bytes.byteLength);
+    return bytes.buffer.slice(bytes.byteOffset, bytes.byteOffset + bytes.byteLength) as ArrayBuffer;
   }
 
   /**