[architecture] Update architecture diagram - 2026-04-26#28564
Conversation
Full rebuild of architecture diagram showing all 21 packages across cmd/ and pkg/. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Refreshes scratchpad/architecture.md with a newly generated ASCII architecture diagram and updated package reference for the gh-aw codebase.
Changes:
- Rebuilt the ASCII architecture diagram layout (entry points, core packages, utility packages) and updated dependency annotations.
- Updated the package reference table to match the newly generated diagram content and package layering.
Show a summary per file
| File | Description |
|---|---|
scratchpad/architecture.md |
Replaces the prior diagram content with a fresh rebuild and updates the “Last updated”/package reference sections. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comments suppressed due to low confidence (1)
scratchpad/architecture.md:73
- The
agentdrainrow description doesn’t match the package documentation.pkg/agentdrain/README.mddescribes it as the Drain log template mining algorithm for structured agent pipeline events and anomaly detection, rather than “agent output streaming and draining”. Please adjust this description to avoid misleading readers.
| agentdrain | Core | Agent output streaming and draining |
| stats | Core | Numerical statistics for metric collection |
- Files reviewed: 1/1 changed files
- Comments generated: 2
| # Architecture Diagram | ||
|
|
||
| > Last updated: 2026-04-25 · Source: [Issue #28411](https://github.com/github/gh-aw/issues) | ||
| > Last updated: 2026-04-26 · Source: [Issue #🏗️ Architecture Diagram: gh-aw Package Architecture Diagram (2026-04-26)](https://github.com/github/gh-aw/issues) |
There was a problem hiding this comment.
The “Source” link text/URL doesn’t follow the documented format and currently links to the repo issues list (missing an issue number). This makes the update non-traceable. Please change it to Source: [Issue #<number>](https://github.com/github/gh-aw/issues/<number>) per .github/workflows/daily-architecture-diagram.md:195.
| > Last updated: 2026-04-26 · Source: [Issue #🏗️ Architecture Diagram: gh-aw Package Architecture Diagram (2026-04-26)](https://github.com/github/gh-aw/issues) | |
| > Last updated: 2026-04-26 · Source: [Issue #<number>](https://github.com/github/gh-aw/issues/<number>) |
| │ │ Action pin │ │ Agent output streaming │ │ struct-tag rendering, message types │ │ | ||
| │ │ resolution │ │ and draining │ └──────────────────────────────────────────┘ │ |
There was a problem hiding this comment.
The agentdrain description in the diagram appears inaccurate. The package implements the Drain log template mining algorithm for analyzing agent pipeline events / anomaly detection (see pkg/agentdrain/README.md:3), not primarily “agent output streaming and draining”. Please update the one-line description here to match the package’s documented purpose.
This issue also appears on line 72 of the same file.
| │ │ Action pin │ │ Agent output streaming │ │ struct-tag rendering, message types │ │ | |
| │ │ resolution │ │ and draining │ └──────────────────────────────────────────┘ │ | |
| │ │ Action pin │ │ Drain log template │ │ struct-tag rendering, message types │ │ | |
| │ │ resolution │ │ mining for anomalies │ └──────────────────────────────────────────┘ │ |
Updates
scratchpad/architecture.mdwith a fresh full-rebuild of the architecture diagram.What changed
cmd/andpkg/agentdrain,stats,actionpinsas distinct core packagesDiagram layers
cmd/gh-aw,cmd/gh-aw-wasmcli,workflow,parser,console,actionpins,agentdrain,statsconstants,envutil,fileutil,gitutil,logger,repoutil,semverutil,sliceutil,stringutil,styles,testutil,timeutil,tty,types,typeutil