[architecture] Update architecture diagram - 2026-05-04#30117
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
github-actions
Bot
added
architecture
diagram
documentation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the scratchpad/architecture.md architecture diagram and package reference to reflect the latest generated view of the gh-aw package structure and dependencies (dated 2026-05-04, linked to workflow run §25311481444).
Changes:
- Refreshes the ASCII architecture diagram with more detailed dependency annotations.
- Updates the “Last updated” metadata to the new workflow run.
- Revises the package reference table (layers/descriptions and ordering).
Show a summary per file
| File | Description |
|---|---|
scratchpad/architecture.md |
Updates the generated architecture diagram and package reference table to the latest snapshot. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comments suppressed due to low confidence (2)
scratchpad/architecture.md:77
- In the package reference table,
pkg/statsis listed as a Core package. Givenpkg/statsis a standalone descriptive-statistics helper (seepkg/stats/README.md) and is only imported by core packages, it seems to fit the “Utility” layer definition better. Suggest changing its Layer to Utility (or adjust the layer definitions to explain why it’s considered Core).
| `pkg/agentdrain` | Core | Agent log draining and streaming |
| `pkg/actionpins` | Core | GitHub Actions pin resolution |
| `pkg/stats` | Core | Numerical statistics for metric collection |
| `pkg/constants` | Utility | Shared constants and semantic type aliases |
scratchpad/architecture.md:91
- The table description for
pkg/testutilsays “(test builds only)”, butpkg/testutilcontains regular.gofiles without build tags and therefore can be compiled into non-test binaries if imported. Suggest changing this wording to reflect “intended for tests” rather than “builds only”.
| `pkg/timeutil` | Utility | Time helper utilities |
| `pkg/tty` | Utility | TTY detection utilities |
| `pkg/testutil` | Utility | Test helper utilities (test builds only) |
- Files reviewed: 1/1 changed files
- Comments generated: 2
Comment on lines
+37
to
+39
| │ ┌───────────────────────────────────────────────────────────────────────────────────────────────┐ │ | ||
| │ │ pkg/actionpins — GitHub Actions pin resolution pkg/stats — Metrics & statistics │ │ | ||
| │ └───────────────────────────────────────────────────────────────────────────────────────────────┘ │ |
Comment on lines
+56
to
+60
| │ ┌───────────────┐ ┌──────────────┐ ┌──────────────┐ ┌────────────┐ ┌──────────────────────┐ │ | ||
| │ │ pkg/typeutil │ │pkg/semverutil│ │ pkg/timeutil │ │ pkg/tty │ │ pkg/testutil │ │ | ||
| │ │ Type conversion│ │ Semantic │ │ Time helpers │ │TTY detect │ │ Test helpers │ │ | ||
| │ │ utilities │ │ versioning │ │ │ │ │ │ (test builds only) │ │ | ||
| │ └───────────────┘ └──────────────┘ └──────────────┘ └────────────┘ └──────────────────────┘ │ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates
scratchpad/architecture.mdwith the latest full-rebuild architecture diagram.Generated by workflow run §25311481444.