fix: set up analysis runtimes inside action#37
Merged
Conversation
Architecture review · 15 components changedgraph LR
n_Analysis_Orchestrator["Analysis Orchestrator"]
n_Structural_Diffing_Engine["Structural Diffing Engine"]
n_Mermaid_Visualization_Engine["Mermaid Visualization Engine"]
n_UX_Integration_Layer["UX #amp; Integration Layer"]
n_Analysis_Orchestrator -- "triggers incremental or full analysis" --> n_Structural_Diffing_Engine
n_Structural_Diffing_Engine -- "provides structural diff data for rendering" --> n_Mermaid_Visualization_Engine
n_Structural_Diffing_Engine -- "reports change status and diff summaries" --> n_UX_Integration_Layer
n_Mermaid_Visualization_Engine -- "queries file-level changes for component mapping" --> n_Structural_Diffing_Engine
n_Mermaid_Visualization_Engine -- "provides rendered diagrams and change counts" --> n_UX_Integration_Layer
n_UX_Integration_Layer -- "initiates health checks and environment validat…" --> n_Analysis_Orchestrator
n_UX_Integration_Layer -- "filters visual noise for final presentation" --> n_Mermaid_Visualization_Engine
n_Analysis_Orchestrator -- "provides metadata for final reporting" --> n_UX_Integration_Layer
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_Analysis_Orchestrator,n_UX_Integration_Layer modified;
linkStyle 2,6 stroke:#0b5d23,stroke-width:2px;
linkStyle 0,4,5 stroke:#7d4e00,stroke-width:2px;
linkStyle 7 stroke:#82071e,stroke-width:2px,stroke-dasharray:5 3;
Colors indicate component changes compared to Download the PR analysis artifacts from this workflow artifact. Analysis Orchestrator : 2 files changed
UX & Integration Layer : 3 files changed
Explore this PR’s architecture in your browser or VS Code. codeboarding-action · run 27582588500 |
Amp-Thread-ID: https://ampcode.com/threads/T-019ecb81-dc76-76cb-8bf3-22a366c9be41 Co-authored-by: Amp <amp@ampcode.com>
133fc90 to
83af656
Compare
Svilen-Stefanov
approved these changes
Jun 15, 2026
Svilen-Stefanov
left a comment
Contributor
There was a problem hiding this comment.
Thank you! LGTM.
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.
Summary\n- install Java 21 inside the composite action for JDTLS\n- install .NET SDKs 8/9/10 inside the composite action for C# LSP/static analysis\n- keep setup gated by the action guard so skipped review/sync runs avoid the extra setup\n\n## Validation\n- ruby -e 'require "yaml"; YAML.load_file("action.yml")'\n- bash -n scripts/run_local.sh\n- python3 -m unittest tests.test_engine_adapter tests.test_sync_subcommands tests.test_build_cta -v\n- git diff --check