Skip to content

fix: set up analysis runtimes inside action#37

Merged
Svilen-Stefanov merged 1 commit into
mainfrom
feat/depth-auto-detect
Jun 15, 2026
Merged

fix: set up analysis runtimes inside action#37
Svilen-Stefanov merged 1 commit into
mainfrom
feat/depth-auto-detect

Conversation

@ivanmilevtues

Copy link
Copy Markdown
Member

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

@codeboarding-review

codeboarding-review Bot commented Jun 15, 2026

Copy link
Copy Markdown

Architecture review · 15 components changed

graph 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;
Loading

Colors indicate component changes compared to main: 🟩 Added · 🟨 Modified · 🟥 Removed

Download the PR analysis artifacts from this workflow artifact.

Analysis Orchestrator : 2 files changed
  • scripts/build_cta.py
  • scripts/engine_adapter.py
UX & Integration Layer : 3 files changed
  • scripts/diff_to_mermaid.py
  • scripts/engine_adapter.py
  • scripts/submit_feedback.py

⚠️ 1 architecture issue found — open CodeBoarding to explore them.

Explore this PR’s architecture in your browser or VS Code.

codeboarding-action · run 27582588500

@ivanmilevtues ivanmilevtues force-pushed the feat/depth-auto-detect branch from 133fc90 to 83af656 Compare June 15, 2026 23:15

@Svilen-Stefanov Svilen-Stefanov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! LGTM.

@Svilen-Stefanov Svilen-Stefanov merged commit a7164ab into main Jun 15, 2026
2 checks passed
@Svilen-Stefanov Svilen-Stefanov deleted the feat/depth-auto-detect branch June 15, 2026 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants