feat: show the base commit SHA in the PR comment's 'compared to' line#29
Open
brovatten wants to merge 2 commits into
Open
feat: show the base commit SHA in the PR comment's 'compared to' line#29brovatten wants to merge 2 commits into
brovatten wants to merge 2 commits into
Conversation
The base branch tip moves while a PR is open (other merges, the sync baseline refresh), so naming only the ref is ambiguous about which snapshot the diff was computed against. Append the exact base commit's 7-char SHA, linked to it: 'compared to `main` (`a1b2c3d`)'. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Architecture review · no architectural changesgraph LR
n_Orchestration_Lifecycle_Manager["Orchestration #amp; Lifecycle Manager"]
n_Visual_Rendering_Diff_Engine["Visual Rendering #amp; Diff Engine"]
n_User_Engagement_Integration["User Engagement #amp; Integration"]
n_Orchestration_Lifecycle_Manager -- "Triggers structural analysis and diagram genera…" --> n_Visual_Rendering_Diff_Engine
n_Orchestration_Lifecycle_Manager -- "Passes execution metadata and analysis status" --> n_User_Engagement_Integration
n_Visual_Rendering_Diff_Engine -- "Provides generated Mermaid diagrams and documen…" --> n_User_Engagement_Integration
classDef added fill:#1f883d,stroke:#0b5d23,color:#ffffff;
classDef modified fill:#bf8700,stroke:#7d4e00,color:#ffffff;
classDef deleted fill:#cf222e,stroke:#82071e,color:#ffffff,stroke-dasharray:5 3;
Colors indicate component changes compared to 🌐 Explore this PR’s architecture in your browser → See this architecture in your editor: Open in VS Code → codeboarding-action · run 27480659008 |
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.
The PR review comment names the base only by ref ("compared to
main"), but the base branch tip moves while a PR is open (other merges, the sync baseline refresh), so it's ambiguous whichmainsnapshot the diff was actually computed against. This appends the exact base commit's 7-char SHA as a clickable commit link —compared tomain(a1b2c3d)— in all three base-reference phrasings.Testing: Simulated the comment-body shell with sample values to confirm the rendered markdown/link, plus actionlint clean and the full unit suite green; the change is isolated to the review comment's base label.
🤖 Generated with Claude Code