feat: add drive to iag-mcp-demo#80
Conversation
implement [ENG-8488]
|
|
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.
There was a problem hiding this comment.
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
supportto triggerwf2and to routewf2through the MCP gateway/agent chain. - Pin the Agent Gateway image to
indykite/agent-gateway:2.21.1and simplify compose by inheriting the pinned tag everywhere; configure weather gateway to allowwf2. - 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.
implement ENG-8488