Skip to content

feat: add tool execution timeout guard#38

Merged
anurag629 merged 1 commit intocodercops:devfrom
Ashok161:feat/issue-37-tool-timeout
Mar 25, 2026
Merged

feat: add tool execution timeout guard#38
anurag629 merged 1 commit intocodercops:devfrom
Ashok161:feat/issue-37-tool-timeout

Conversation

@Ashok161
Copy link
Copy Markdown
Contributor

Summary

  • add a configurable timeout guard around tool execution in core
  • plumb oolTimeoutMs through server config into provider chat params
  • add focused timeout tests in core and handler coverage in server

Verification

  • corepack pnpm --filter @chatcops/core test -- tests/providers/tool-execution.test.ts
  • corepack pnpm --filter @chatcops/server test -- tests/handler.test.ts

Closes #37

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 23, 2026

@Ashok161 is attempting to deploy a commit to the CODERCOPS Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
chatcops-website Ready Ready Preview, Comment Mar 25, 2026 1:56am

Request Review

Copy link
Copy Markdown
Member

@anurag629 anurag629 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean and focused. Timeout with Promise.race, timer cleanup in finally, fake timer tests hitting the exact boundary. Looks good @Ashok161.

@anurag629 anurag629 merged commit 0d68b22 into codercops:dev Mar 25, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: add timeout guard for tool execution

2 participants