Skip to content

Conversation

@richhankins
Copy link
Contributor

@richhankins richhankins commented Jan 21, 2026

Increase timeout for filesystem context test from 2 minutes to 5 minutes to accommodate multiple LLM API calls made during search_and_ask operations.

The previous 120-second timeout was insufficient for the test execution, which involves multiple sequential LLM API calls that can take longer than expected, especially under varying network conditions or API response times.


🤖 This description was generated automatically. Please react with 👍 if it's helpful or 👎 if it needs improvement.

@augment-app-staging
Copy link

🤖 Augment PR Summary

Summary: Extends the execution timeout for the filesystem_context example test to reduce flakiness when multiple LLM-backed calls are performed.
Changes: Raises the subprocess timeout from 120s to 300s and documents the rationale inline.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augment-app-staging augment-app-staging bot left a comment

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

@richhankins richhankins merged commit dbb860d into main Jan 21, 2026
2 checks passed
@richhankins richhankins deleted the rich-increase-timeout branch January 21, 2026 19:35
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.

2 participants