feat: add violation summary UI and adapt standalone deployment backend#90
Open
dalu-wins wants to merge 20 commits intoDataFlowAnalysis:mainfrom
Open
feat: add violation summary UI and adapt standalone deployment backend#90dalu-wins wants to merge 20 commits intoDataFlowAnalysis:mainfrom
dalu-wins wants to merge 20 commits intoDataFlowAnalysis:mainfrom
Conversation
…to DataflowAnalysis repo
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.
Overview
This PR adds a violation summary display to the web editor frontend and adapts the standalone WebSocket backend for headless deployment using a Tycho/Maven build.
Frontend Changes
Backend Changes
Build Approach
I opted for a self-contained Tycho product build instead of the existing AnalysisBackendServerProduct approach. This produces a standalone executable with all dependencies bundled, requiring only Java 17 on the target machine. See backend/analysisBackendServer/violationSummarySetup.md for full build and deployment instructions.
The maintainers may want to decide if they want to go this way for future deployment or stay on the current strategy.
Notes
The standalone backend depends on modified bundles from my DataFlowAnalysis fork. A separate PR will be opened there alongside this one.
Once the DataFlowAnalysis PR is merged upstream, the file:// local path dependency in the target platform can be replaced with the nightly update site URL, and the README updated to point to the official repository.