[architecture] Update architecture diagram - 2026-04-22#27792
Conversation
Full rebuild of architecture diagram reflecting actual package dependency graph as of commit 27e6b35. Improved diagram layout shows clear three-layer structure with real dependency arrows between cli → workflow → parser → console chain. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
github-actions
Bot
added
architecture
diagram
documentation
There was a problem hiding this comment.
Pull request overview
Updates scratchpad/architecture.md to refresh the ASCII architecture diagram and package reference to reflect current layering and dependencies.
Changes:
- Rebuilt the ASCII diagram with a clearer Entry Points → Core → Utility layout and updated dependency arrows.
- Updated the package reference table to match the new layering and descriptions.
- Updated the “Last updated” header metadata.
Show a summary per file
| File | Description |
|---|---|
| scratchpad/architecture.md | Refreshes the architecture diagram and package reference to reflect current package relationships and layering. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 3
| # Architecture Diagram | ||
|
|
||
| > Last updated: 2026-04-21 · [Run §24714109908](https://github.com/github/gh-aw/actions/runs/24714109908) | ||
| > Last updated: 2026-04-22 · Source: [Issue #27791](https://github.com/github/gh-aw/issues) |
There was a problem hiding this comment.
The Issue link text indicates a specific issue (#27791), but the URL points to the generic issues list (.../issues) instead of the issue itself. Update the link target to include the issue number (or adjust the link text if the intent is to link to the list).
| > Last updated: 2026-04-22 · Source: [Issue #27791](https://github.com/github/gh-aw/issues) | |
| > Last updated: 2026-04-22 · Source: [Issue #27791](https://github.com/github/gh-aw/issues/27791) |
| # Architecture Diagram | ||
|
|
||
| > Last updated: 2026-04-21 · [Run §24714109908](https://github.com/github/gh-aw/actions/runs/24714109908) | ||
| > Last updated: 2026-04-22 · Source: [Issue #27791](https://github.com/github/gh-aw/issues) |
There was a problem hiding this comment.
PR description cites the architecture diagram source as workflow run §24769989505, but the document header now cites an issue as the source. Align the header with the stated source (e.g., reference the workflow run, or update the PR description if the issue is the actual source).
| │ │ cmd/gh-aw │ │ cmd/gh-aw-wasm │ │ | ||
| │ │ Main CLI binary │ │ WebAssembly / JS target │ │ | ||
| │ └──────────────────────┬───────┘ └────────┬─────────────────────┘ │ | ||
| │ (cli,workflow,parser,│console,constants) (parser, │workflow) │ |
There was a problem hiding this comment.
The diagram line listing entry-point imports includes box-border characters (│) inside the parenthesized text, which breaks readability and appears to be an alignment artifact (e.g., parser,│console,constants and parser, │workflow). Remove the stray border characters and reflow spacing so the import lists render cleanly within the diagram.
| │ (cli,workflow,parser,│console,constants) (parser, │workflow) │ | |
| │ (cli,workflow,parser,console,constants) (parser,workflow) │ |
Updates
scratchpad/architecture.mdwith a refreshed ASCII architecture diagram.What changed
cli → workflow → parserchainpkg/consoleshown as shared consumer ofpkg/styles,pkg/tty,pkg/loggerpkg/actionpinssurfaced as a direct dependency ofpkg/workflowSource
Generated from workflow run §24769989505 · Base commit:
27e6b35