Skip to content

ci: add session handling to agent#113

Merged
rahulguptajss merged 5 commits into
mainfrom
rg2-fix-ci
Apr 22, 2026
Merged

ci: add session handling to agent#113
rahulguptajss merged 5 commits into
mainfrom
rg2-fix-ci

Conversation

@rahulguptajss

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 22, 2026 07:28
@cla-bot cla-bot Bot added the cla-signed label Apr 22, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds MCP session resiliency to the integration-test “Agent” so tool calls can recover from dropped connections during CI runs.

Changes:

  • Persist MCP transport on the Agent so a new session can be established later.
  • Add reconnect logic and retry a tool call when the session appears to have been dropped.
  • Add a system prompt to guide the LLM to correct parameter-mapping issues on validation errors.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread integration/test/tools_test.go Outdated
Copilot AI review requested due to automatic review settings April 22, 2026 07:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread integration/test/tools_test.go Outdated
Comment thread integration/test/tools_test.go
Copilot AI review requested due to automatic review settings April 22, 2026 07:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread integration/test/tools_test.go Outdated
Comment thread integration/test/tools_test.go
Hardikl
Hardikl previously approved these changes Apr 22, 2026
Copilot AI review requested due to automatic review settings April 22, 2026 10:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rahulguptajss rahulguptajss merged commit 9b00b1d into main Apr 22, 2026
26 checks passed
@rahulguptajss rahulguptajss deleted the rg2-fix-ci branch April 22, 2026 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants