fix: address validation length mismatch and directory verification

[haasonsaas]

Summary

• Fix validation length mismatch where SafeString schema max was 1000 but usage was 2000
• Add proper file type verification in SecureCodeReader.findRelatedFiles using fs.lstat

Changes Made

1. InputValidator String Length Fix

• Updated SafeString schema max length from 1000 to 2000
• Updated validateString default maxLength parameter to 2000
• This ensures valid inputs between 1000-2000 characters are no longer rejected

2. SecureCodeReader Directory Verification

• Added fs.lstat check in findRelatedFiles to verify entries are actual files
• Skips directories and symlinks before processing
• Prevents attempting to read directories as files

3. Code Quality Improvements

• Fixed all ESLint errors (126 problems reduced to 12 warnings)
• Removed unused imports
• Fixed trailing spaces and missing commas
• Properly escaped regex patterns

Testing

• TypeScript build passes successfully
• ESLint now shows only type warnings (no errors)
• All security features remain intact

🤖 Generated with Claude Code

[Copilot]

Pull Request Overview

This PR addresses a validation length mismatch in input validation and improves file type verification by ensuring only files are processed. Key changes include updating the SafeString schema and validateString max length from 1000 to 2000, adding fs.lstat-based file checks in SecureCodeReader, and various ESLint and formatting updates.

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/utils/SecureCodeReader.ts	Added fs.lstat check to skip directories and symlinks
src/utils/PromptSanitizer.ts	Minor lint and formatting improvements
src/utils/InputValidator.ts	Updated max length and adjusted regex for safe string and filename schemas
src/utils/ErrorClassifier.ts	Added missing commas for consistency
src/utils/CodeReader.ts	Minor code formatting fixes
src/services/GeminiService.ts	Added missing commas for consistency
src/services/ConversationalGeminiService.ts	Removed unused import of SessionError
src/index.ts	Removed unused type import from models
src/analyzers/DeepCodeReasonerV2.ts	Added missing commas for consistency