Add file upload support for chat sessions (#19)

[GunaPalanivel]

Summary

Adds the ability for users to attach files when sending messages to the chatbot, giving the LLM richer context for troubleshooting Jenkins issues.

Changes

Backend (Python/FastAPI)

• New file_service.py module for file processing and validation
• Added multipart form upload endpoint POST /sessions/{session_id}/message/upload
• Added GET /files/supported-extensions endpoint for frontend validation
• Updated chat_service.py to incorporate file content into LLM prompts
• New Pydantic models: FileAttachment, ChatRequestWithFiles, SupportedExtensionsResponse

Frontend (React/TypeScript)

• Added attach button (📎) to the input component
• File preview chips showing name, size, and remove option
• Image thumbnails and file icons displayed in chat messages
• Client-side validation for file size and extension

Supported File Types

• Logs/Text: .txt, .log, .md
• Code: .py, .js, .ts, .tsx, .java, .groovy, .sh
• Config: .json, .xml, .yaml, .yml, .properties, .toml
• Jenkins: Jenkinsfile, .groovy
• Images: .png, .jpg, .jpeg, .gif, .webp, .bmp

Limits

• Max file size: 10MB per file
• Images are base64 encoded for LLM vision capabilities

Testing

• Added unit tests for file service (test_file_service.py)
• Added route tests for upload endpoint (test_file_upload.py)
• Updated frontend tests for Input component

How to Test

1. Start the backend and frontend servers
2. Click the 📎 button or drag files into the input area
3. Attach a log file or screenshot and ask a question
4. Verify the file content appears in the chat and the LLM references it

Closes #19

[cnu1812]

Also, a few checks are failing. Take a note!

[GunaPalanivel]

Also, a few checks are failing. Take a note!

CI Failures Fixed

Backend Tests Failure

Form data requires "python-multipart" to be installed.

Resolution: Added python-multipart==0.0.20 to requirements.txt

---

Frontend Tests Failure

TypeError: (0 , chatbot_1.fetchSupportedExtensions) is not a function

Resolution: Added missing mocks in Chatbot.test.tsx:

jest.mock("../api/chatbot", () => ({
  fetchChatbotReply: jest.fn()...,
  fetchChatbotReplyWithFiles: jest.fn()...,
  fetchSupportedExtensions: jest.fn().mockResolvedValue({...}),
  validateFile: jest.fn().mockReturnValue({ isValid: true }),
  fileToAttachment: jest.fn()...,
  // ... other mocks
}));

---

Pylint Failures

Fixed all pylint issues:

• Trailing whitespace (C0303)
• Line too long (C0301)
• Unused imports (W0611)
• raise-missing-from (W0707)
• Unnecessary pass statement (W0107)
• Wrong import order (C0411)

[berviantoleo]

Please see my comments. Some of them are required changes to strengthen the validation.

[berviantoleo]

Please have a look at the failed CIs

[GunaPalanivel]

Hey @berviantoleo, thanks for the review! We’ve been hitting some stubborn CI fails—mostly due to PyTorch and CUDA version conflicts on the latest Python. Our plan is to relax the strict version pins and let pip resolve the right dependencies, following PyTorch’s official guidance. If you’re good with this approach, we’ll go ahead and update the requirements accordingly. Let us know your thoughts!

[berviantoleo]

Hey @berviantoleo, thanks for the review! We’ve been hitting some stubborn CI fails—mostly due to PyTorch and CUDA version conflicts on the latest Python. Our plan is to relax the strict version pins and let pip resolve the right dependencies, following PyTorch’s official guidance. If you’re good with this approach, we’ll go ahead and update the requirements accordingly. Let us know your thoughts!

Good to go. However, @GunaPalanivel, please focusing to fix 3.12 and 3.13. I'm just experimenting with 3.14, so you can ignore the install dependencies error at the moment.

[GunaPalanivel]

@berviantoleo Done.