Fix lint errors and warnings

- Fixed unnecessary escape characters in detector.ts URL pattern
- Fixed DOB pattern escape characters
- Added eslint-disable for security.ts null byte check (intentional)
- Added test-samples to eslint ignores (legacy test folder)
- Removed unused readFileSync import from web-app.spec.ts
- Fixed prefer-const warnings in test files

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: iYassr

electron/services/detector.ts

@@ -1005,7 +1005,7 @@ function detectURLs(
   addEntity: (entity: NEREntity) => void
 ): void {
   // URL pattern - matches http://, https://, ftp://, etc.
-  const urlPattern = /\b(?:https?|ftp):\/\/[^\s<>\[\]"'`,;)]+/gi
+  const urlPattern = /\b(?:https?|ftp):\/\/[^\s<>[\]"'`,;)]+/gi
 
   let match: RegExpExecArray | null
   while ((match = urlPattern.exec(text)) !== null) {
@@ -1239,8 +1239,8 @@ function detectDOB(
   const dobPatterns = [
     // With DOB context - match any year
     // "Date of Birth: 15/03/1985" or "DOB: 1990-07-22"
-    /\b(?:date\s+of\s+birth|dob|birth\s*date|born|birthday)[:\s]+(\d{1,2}[-/\.]\d{1,2}[-/\.]\d{4})\b/gi,
-    /\b(?:date\s+of\s+birth|dob|birth\s*date|born|birthday)[:\s]+(\d{4}[-/\.]\d{1,2}[-/\.]\d{1,2})\b/gi,
+    /\b(?:date\s+of\s+birth|dob|birth\s*date|born|birthday)[:\s]+(\d{1,2}[-/.]\d{1,2}[-/.]\d{4})\b/gi,
+    /\b(?:date\s+of\s+birth|dob|birth\s*date|born|birthday)[:\s]+(\d{4}[-/.]\d{1,2}[-/.]\d{1,2})\b/gi,
     /\b(?:date\s+of\s+birth|dob|birth\s*date|born|birthday)[:\s]+([A-Za-z]+\s+\d{1,2},?\s+\d{4})\b/gi,
     /\b(?:date\s+of\s+birth|dob|birth\s*date|born|birthday)[:\s]+(\d{1,2}\s+[A-Za-z]+\s+\d{4})\b/gi,
 

electron/services/security.ts

@@ -354,8 +354,10 @@ export function sanitizeFilename(filename: string): string {
     return 'document'
   }
 
-  // Remove path separators and null bytes
+  // Remove path separators and null bytes (security measure)
+  // eslint-disable-next-line no-control-regex -- null byte check is intentional for security
+  const sanitizePattern = /[/\\:*?"<>|\x00]/g
   return filename
-    .replace(/[/\\:*?"<>|\x00]/g, '_')
+    .replace(sanitizePattern, '_')
     .slice(0, 255)
 }

eslint.config.js

@@ -5,7 +5,7 @@ import reactRefresh from 'eslint-plugin-react-refresh'
 import tseslint from 'typescript-eslint'
 
 export default tseslint.config(
-  { ignores: ['dist', 'dist-electron', 'release', 'node_modules'] },
+  { ignores: ['dist', 'dist-electron', 'release', 'node_modules', 'test-samples'] },
   {
     extends: [js.configs.recommended, ...tseslint.configs.recommended],
     files: ['**/*.{ts,tsx}'],

test-samples/e2e-test.spec.ts

@@ -162,7 +162,7 @@ export async function verifyOutputFiles() {
     console.log(`\nChecking: ${file}`)
     console.log('-'.repeat(40))
 
-    let issues = []
+    const issues = []
 
     // Check for remaining PII
     for (const email of expectedPII.emails) {

tests/all-formats.spec.ts

@@ -374,23 +374,23 @@ test.describe('maskr All Formats E2E Tests', () => {
     await personalTab.click()
     await page.waitForTimeout(500)
 
-    let emailVisible = await page.locator('table').getByText('ahmed.rashid@acme-corp.com').first().isVisible().catch(() => false)
+    const emailVisible = await page.locator('table').getByText('ahmed.rashid@acme-corp.com').first().isVisible().catch(() => false)
     console.log('Personal tab - email visible:', emailVisible)
 
     // Test Financial tab
     const financialTab = page.locator('button', { hasText: 'Financial' })
     await financialTab.click()
     await page.waitForTimeout(500)
 
-    let creditCardVisible = await page.locator('table').getByText('4532015112830366').first().isVisible().catch(() => false)
+    const creditCardVisible = await page.locator('table').getByText('4532015112830366').first().isVisible().catch(() => false)
     console.log('Financial tab - credit card visible:', creditCardVisible)
 
     // Test Technical tab
     const technicalTab = page.locator('button', { hasText: 'Technical' })
     await technicalTab.click()
     await page.waitForTimeout(500)
 
-    let ipVisible = await page.locator('table').getByText('192.168.1.100').first().isVisible().catch(() => false)
+    const ipVisible = await page.locator('table').getByText('192.168.1.100').first().isVisible().catch(() => false)
     console.log('Technical tab - IP visible:', ipVisible)
   })
 

tests/web-app.spec.ts

@@ -1,5 +1,4 @@
 import { test, expect } from '@playwright/test'
-import { readFileSync } from 'fs'
 import { join, dirname } from 'path'
 import { fileURLToPath } from 'url'