feat: add architecture page - #112
Conversation
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| Secrets | Sep 1, 2026 11:37a.m. | Review ↗ | |
| Python | Sep 1, 2026 11:37a.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 a new “Architecture” static page to the a2a/iag-mcp-demo console and updates the chatbot UI to better support streaming/chat readability and multi-line prompts.
Changes:
- Introduces
/static/architecture.htmlwith an inline SVG diagram and lightweight JS to badge the active usecase and dim optional profile-gated groups. - Exposes
usecaseandprofilesvia/api/config(fed fromUSECASE/COMPOSE_PROFILES) and wires those env vars intodocker-compose.yaml. - Updates the console UI (new header link + user chip, textarea input with autosize, message metadata, style tweaks).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| a2a/iag-mcp-demo/README.md | Documents the new Architecture page behavior and how it reflects /api/config. |
| a2a/iag-mcp-demo/docker-compose.yaml | Adds USECASE / COMPOSE_PROFILES env for the console to report active context/profiles. |
| a2a/iag-mcp-demo/chatbot/static/styles.css | UI/layout tweaks (wider container, sidebar sizing, message meta, textarea styling, loading indicator styles). |
| a2a/iag-mcp-demo/chatbot/static/index.html | Adds Architecture link + user chip; textarea + autosize; message meta labels/timestamps; streaming safeguards. |
| a2a/iag-mcp-demo/chatbot/static/architecture.html | New self-contained architecture diagram page with profile dimming via /api/config. |
| a2a/iag-mcp-demo/chatbot/app.py | Adds USECASE / COMPOSE_PROFILES to config output and returns a display username in auth status. |
Suppressed comments (1)
a2a/iag-mcp-demo/chatbot/static/index.html:334
keypressis deprecated and can behave inconsistently for Enter on some platforms/IME input. Preferkeydown, and skip handling while the user is composing text (IME) to avoid accidental sends.
messageInput.addEventListener('keypress', (e) => {
if (e.key === 'Enter' && !e.shiftKey) {
e.preventDefault();
sendMessage();
}
});
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
3633790 to
10eb51b
Compare
implement [ENG-8488]
implement ENG-8488
add architecture page
add some style modifications