Increase server request timeout for long prompts

[zerone0x]

Summary

• increase the local REST server request/response timeout from 5 minutes to 30 minutes
• move the timeout into a small exported constant so it is covered by a focused test

Fixes #858

Tests

• npm run test:node -- src/main/server/tests/timeouts.test.ts
• npx eslint src/main/server/server-controller.ts src/main/server/timeouts.ts src/main/server/tests/timeouts.test.ts

Notes

• npm run typecheck:node currently fails on main with src/main/models/providers/claude-agent-sdk.ts(243,14): Property 'aiSdkTools' does not exist on type 'ClaudeCodeSettings'.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)

• src/main/server/__tests__/timeouts.test.ts - New test file, well-structured
• src/main/server/timeouts.ts - New constants module with clear documentation
• src/main/server/server-controller.ts - Refactored to use imported constant

Review Notes

This PR makes a clean refactoring that:

• Increases the server request timeout from 5 minutes to 30 minutes to handle long-running prompts
• Extracts the timeout constant to a dedicated module for better testability
• Adds appropriate unit test coverage for the timeout value

The changes are straightforward, well-documented, and follow good software engineering practices.

---

_{Reviewed by claude-4.5-sonnet-20250929 · Input: 37 · Output: 2.4K · Cached: 221.4K}

[gemini-code-assist]

Code Review

This pull request extracts the server request timeout constant into a separate timeouts.ts file, increases its value from 5 minutes to 30 minutes to support long-running requests, and adds a corresponding unit test. Feedback was provided regarding a potential server crash in the timeout middleware due to duplicate timeout callbacks attempting to send headers twice.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[wladimiiir]

This test does not really test anything and should be removed.

[wladimiiir]

No reason to have a separate file just for this constant. The constant should be eventually moved into src/main/constants.ts, but does not need to be as part of this change.