- 
                Notifications
    
You must be signed in to change notification settings  - Fork 2.4k
 
add optional timeout field to mcp config to allow users to use responding servers #3558
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| 
           @mattgillard can we instead make timeout an optional param for an mcp server in the config? We like to error earlier if possible and 10 seconds is a long time  | 
    
          
 sure i can have a go at that  | 
    
…d remote mcp servers and updated docs
          
 this is done and tested  | 
    
| 
           Thanks!  | 
    
| 
           /oc in this PR reword all "Timeout in milliseconds for tools....." to say "Timeout in ms for fetching tools from the MCP server."  | 
    
| 
           Perfect! I've successfully updated all 4 occurrences of the timeout description text: 
 All instances now say "Timeout in ms for fetching tools from the MCP server." instead of "Timeout in milliseconds for tools verification." as requested.  | 
    
Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com>
| 
           Thanks Matt!  | 
    
…ding servers (sst#3558) Co-authored-by: Matt Gillard <matt-github@gillard.biz> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com>
🔥 CRITICAL BUGFIXES: - Fix hanging permission prompts in headless mode (sst#3522) - Add doom loop detection (sst#3445) - Fix MCP metadata preservation (sst#3573) - Fix agent model selection priority (sst#3572) - Fix TUI error display - Fix parcel watcher issue (sst#3544) 🚀 NEW FEATURES: - Add MCP server timeout configuration (sst#3558) - Add positional arguments for slash commands (sst#3456) - Add session ID to agent context metadata (sst#3559) - Add session.started event hook (sst#3413) - Add SNOWCODE_CONFIG_DIR support (rebranded from OPENCODE_CONFIG_DIR) 🔧 IMPROVEMENTS: - Add Zod validation for tool inputs - Fix MCP status() overwrite issue - Add bash timeout validation - Use stdout.write instead of console.log (sst#3508) - Fix SSE buffer overflow (sst#3531) - Make build script cross-platform (sst#3430) 🎨 UX ENHANCEMENTS: - Fix editor paste functionality (sst#3489) - Better frontmatter error messages - Configurable provider timeout (sst#3443) 📝 CHANGES: - 32 files changed, +1,668/-874 lines - Merged from OpenCode production branch (week 44) - Bun version check relaxed to 1.3.x Co-authored-by: OpenCode Contributors
…ding servers (sst#3558) Co-authored-by: Matt Gillard <matt-github@gillard.biz> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com>

This issue was blocking my work so contributing a fix.
I needed to set this slightly higher timeout so opencode registers the atlassian tools. Otherwise i get this error in the logs:
With this slightly higher timeout its fine - there is still a weird delay but I have not debugged that yet:
My MCP config: