feat: merge the PR-comment browser + editor CTAs into one line#30
Merged
Conversation
Replace the two stacked CTA lines (a '🌐 Explore … in your browser →' line and a
separate 'See this … in your editor: Open in VS Code →' line) with a single
sentence: 'Explore this PR's architecture in your browser or VS Code.' The
browser link only joins when webview_ready (head analysis committed, non-fork
PR); 'your' rides with the browser entry alone so it still reads right when only
the editor link is present ('… in VS Code'). Multiple editors render with an
Oxford 'or' ('browser, VS Code, or Cursor'). The ⚠️ health banner and the
proxy-only 'Get the extension' line stay separate.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Architecture review · 6 components changedgraph LR
n_Action_Orchestrator["Action Orchestrator"]
n_Structural_Analyzer_Documenter["Structural Analyzer #amp; Documenter"]
n_Diffing_Visualization_Engine["Diffing #amp; Visualization Engine"]
n_Integration_Feedback_Provider["Integration #amp; Feedback Provider"]
n_Action_Orchestrator -- "triggers structural mapping and file rendering" --> n_Structural_Analyzer_Documenter
n_Action_Orchestrator -- "initiates baseline comparison and mermaid gener…" --> n_Diffing_Visualization_Engine
n_Structural_Analyzer_Documenter -- "provides filtered change sets and method metada…" --> n_Diffing_Visualization_Engine
n_Diffing_Visualization_Engine -- "requests component naming and file-level filter…" --> n_Structural_Analyzer_Documenter
n_Diffing_Visualization_Engine -- "provides visual syntax for report generation" --> n_Integration_Feedback_Provider
n_Action_Orchestrator -- "delegates final CTA and webview link creation" --> n_Integration_Feedback_Provider
n_Diffing_Visualization_Engine -- "Returns the updated analysis state to be saved…" --> n_Action_Orchestrator
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;
class n_Integration_Feedback_Provider modified;
linkStyle 1 stroke:#0b5d23,stroke-width:2px;
linkStyle 0,2,3,4,5 stroke:#7d4e00,stroke-width:2px;
linkStyle 6 stroke:#82071e,stroke-width:2px,stroke-dasharray:5 3;
Colors indicate component changes compared to Integration & Feedback Provider : 1 file changed
Explore this PR’s architecture in your browser or VS Code. codeboarding-action · run 27481078691 |
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.
Collapses the footer's two stacked call-to-action lines — a⚠️ health banner and the proxy-only "Get the extension" line stay on their own lines.
🌐 Explore … in your browser →line and a separateSee this … in your editor: Open in VS Code →line — into a single sentence: Explore this PR's architecture in your browser or VS Code. The browser link only joins whenwebview_ready(head analysis committed, non-fork PR), and "your" rides with the browser entry alone so it still reads correctly when only the editor link is present (… in VS Code); multiple detected editors render with an Oxford "or" (browser, VS Code, or Cursor), while theBefore / after:
Testing: Rewrote
tests/test_build_cta.pyfor the merged format and added cases for the join logic and each branch (browser+editor, editor-only, both editors, no-webview); full unit suite (123) green and black clean, and I rendered the real footer for the default, fork-PR, and proxy+both-editors cases.🤖 Generated with Claude Code