Skip to content

feat: add drive to iag-mcp-demo#80

Merged
cowan-macady merged 1 commit into
masterfrom
upd-iag-mcp
Jul 17, 2026
Merged

feat: add drive to iag-mcp-demo#80
cowan-macady merged 1 commit into
masterfrom
upd-iag-mcp

Conversation

@cowan-macady

@cowan-macady cowan-macady commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

implement ENG-8488

  • add weather
  • fix issues

implement [ENG-8488]
@cowan-macady
cowan-macady requested a review from a team as a code owner July 17, 2026 20:28
Copilot AI review requested due to automatic review settings July 17, 2026 20:28
@deepsource-io

deepsource-io Bot commented Jul 17, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in ff54bba...8e2868a on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Secrets Jul 17, 2026 8:28p.m. Review ↗
Python Jul 17, 2026 8:28p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

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/updates relationship graph wiring and demo stack configuration for the iag-mcp-demo to support additional workflow routing (notably weather → MCP via wf2) and to improve MCP proxy troubleshooting by preserving error bodies and redacting Authorization headers in debug output.

Changes:

  • Extend relationship definitions to allow support to trigger wf2 and to route wf2 through the MCP gateway/agent chain.
  • Pin the Agent Gateway image to indykite/agent-gateway:2.21.1 and simplify compose by inheriting the pinned tag everywhere; configure weather gateway to allow wf2.
  • Improve MCP error diagnostics in the analyst/retriever agents (buffer error bodies before stream close; redact Authorization header in formatted errors).

Reviewed changes

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

Show a summary per file
File Description
canbank-iag/data/relationships/relationships_iag.json Adds wf2 trigger + INVOKES edges to wire wf2 into the agent/MCP chain.
a2a/iag-mcp-demo/retriever_agent/retriever_agent.py Buffers error bodies for streaming HTTP failures; redacts Authorization in formatted error output.
a2a/iag-mcp-demo/analyst_agent/analyst_agent.py Same MCP streaming/error-formatting improvements as retriever agent.
a2a/iag-mcp-demo/README.md Updates documentation to pin Agent Gateway to 2.21.1 and clarifies MCP proxying requirement.
a2a/iag-mcp-demo/iag-base-docker.yaml Pins gateway image to 2.21.1 with updated rationale.
a2a/iag-mcp-demo/docker-compose.yaml Sets weather gateway allowed workflow to WEATHER_WORKFLOW_ID; removes per-service image overrides now that base is pinned.
a2a/iag-mcp-demo/bruno/iag-demo/ingest/demo-data/canbank/create-canbank-rels.yml Adds department → wf2 relationship for the demo data ingest.
a2a/iag-mcp-demo/bruno/iag-demo/ingest/agent-worfklow/create-relationships.yml Adds/updates wf1/wf2 routing edges in the agent-workflow relationship ingest payload.

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

@cowan-macady
cowan-macady merged commit a69f0aa into master Jul 17, 2026
10 of 11 checks passed
@cowan-macady
cowan-macady deleted the upd-iag-mcp branch July 17, 2026 23:51
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.

3 participants