windup / rhamt-vscode-extension

Windup

Windup is an assembly of tools that support large-scale Java application modernization and migration projects across a broad range of transformations and use cases. It accelerates application code analysis, supports effort estimation, accelerates code migration, and helps you move applications to the cloud and containers. If you want to contribute or participate in developer discussions please join the Konveyor community and ping us on slack.

Installation and Usage

Download the latest Windup release from here and refer to the getting started blog to run your first code analysis.

Issue Tracker

https://issues.redhat.com/projects/WINDUP/issues

Building from source

Ensure you use the provided Maven Wrapper utility to build the project, as Windup requires Maven 3.8.8:

./mvnw verify

Project Source Code Format

This project has an agreed upon source format style documented via an Eclipse code formatter profile. If using an alternate IDE, plugins are available to allow use of this profile (see below). All contributions should be formatted using this profile before submission.

• Netbeans: http://plugins.netbeans.org/plugin/50877/eclipse-code-formatter-for-java

• IntelliJ: http://plugins.jetbrains.com/plugin/?id=6546

rhamt-vscode-extension's Issues

Only first `--target` option is included in `kantra analyze` command

When selecting multiple targets, only the first one is added to the analyze command.

Recreation steps:

1. Activate the extension
2. Open the MTA: Explorer
3. Create a new configuration
4. Select multiple targets (ie. cloud-readiness and jakarta-ee)
5. Run the configuration

Expected results:
Both targets included

Actual results:

time="2023-12-05T13:41:59-05:00" level=info msg="running source code analysis" args="--provider-settings=/opt/input/config/settings.json --rules=/opt/rulesets/ --output-file=/opt/output/output.yaml --context-lines=100 --dep-label-selector=(!konveyor.io/dep-source=open-source) --verbose=4 --label-selector=((konveyor.io/target=cloud-readiness) && konveyor.io/source) || (discovery)" input=/Users/dzager/Workspace/kyma_poc/data/coolstuff-javaee log=/Users/dzager/.windup/tooling/vscode/xjmsw61l5-nos88yyps/analysis.log output=/Users/dzager/.windup/tooling/vscode/xjmsw61l5-nos88yyps volumes="{\"/Users/dzager/.windup/tooling/vscode/xjmsw61l5-nos88yyps\":\"/opt/output\",\"/Users/dzager/Workspace/kyma_poc/data/coolstuff-javaee\":\"/opt/input/source\",\"/var/folders/rm/h4pf5h4j1z39vkywmhyfcgrh0000gn/T/analyze-config-196555721\":\"/opt/input/config\"}"

Report URL opened from MTA extension breaks CSS/JS URLs

• OS: Windows 10 21H1
• Java version: OpenJDK Runtime Environment AdoptOpenJDK-11.0.11+9 (build 11.0.11+9)
• VSCode Version: 1.67
  • Node.js: 16.13.0
• Extension version: Migration Toolkit for Applications v0.0.98 (preview)
• Browsers:
  • Microsoft Edge 101.0.1210.32 (Official build) (64-bit)
  • Google Chrome 100.0.4896.127 (Official Build) (64-bit)

I tried the extension today and I was able to see the issues reported inside VSCode for a sample application, but when opening the report from VSCode it looks broken because the CSS/JS are not found, as in the screenshot below:

Opening the report in VSCode:

Website open in Edge (with the Developer Tools panel enabled):

The following URLs were broken (HTTP 404):

• http://localhost:61435/reports/resources/css/bootstrap.min.css
• http://localhost:61435/reports/resources/css/font-awesome.min.css
• http://localhost:61435/reports/resources/css/windup.css
• http://localhost:61435/reports/resources/js/jquery-3.3.1.min.js
• ... and so on

Likely cause

The problem seems to be that there is a / encoded in the URL used to open the report. The extension tries to open the address http://localhost:61435/54fkx96i7-lqqqufhqb%5Cindex.html but it should be http://localhost:61435/54fkx96i7-lqqqufhqb/index.html instead (note the %5C is the / encoded). This encoded character causes the paths to all the CSS/JS resources to be computed as relative to the root of the web address, instead of relative to the folder (54fkx96i7-lqqqufhqb in this case).

When the correct URL is used, the report works correctly:

The correct location of the resources have the unique identifier in them, like this:

• http://localhost:61435/54fkx96i7-lqqqufhqb/reports/resources/css/bootstrap.min.css

MTA fails silently VSCode+OSX+OpenJDK 16.0.1

Because of this issue: windup/windup-distribution#73 mat-client doesn't run at all with the VS Code console showing only this:

Using MTA at /Users/dd/Library/Application Support/Code/User/globalStorage/redhat.mta-vscode-extension/.mta/tooling/vscode/mta-cli/mta-cli-5.2.0.Final

The extension should capture stderr and include that into the VS Code output console, so users can see there is an issue rather than letting us just sit there scratching their heads wondering if it ran, is still running or did anything.

Add way for user to undo "Mark as Complete"

If a user right-clicks an issue and selects "Mark as Complete" by mistake, there is no way to undo the change.

Would be good to have some way to undo this, perhaps by having the menu item change to "Mark as Incomplete" if the issue has been marked by completed.

Improve release process to include sources when publishing vsix

Similar to discussions in...

• camel-tooling/vscode-wsdl2rest#142
• redhat-developer/vscode-java#892
• redhat-developer/vscode-project-initializer#40
• redhat-developer/vscode-quarkus#177
• camel-tooling/vscode-camelk#243
• https://issues.redhat.com/browse/FUSETOOLS2-80

... we'd like that each release of this .vsix file to dl.jb.o be accompanied by a matching sources.tar.gz

I don't care if you push the .vsix to GH releases page, or tar up sources and push that to dl.jb.o, just that we have binary and sources in the same place.

Thanks!

This is a RH product security requirement in that we need to either build everything used in CodeReady Workspaces from scratch as part of our build, or else provide matching sources for upstream RH-built binaries.

`--overwrite` configuration option not respected

When attempting to run a configuration from the MTA: Explorer that has previously been run, I get the error:

time="2023-12-05T13:35:01-05:00" level=error msg="failed to validate flags" error="output dir /Users/dzager/.windup/tooling/vscode/2pkzd8cc9-7x3l4humx already exists and --overwrite not set"

Recreation Steps:

1. Activate the extension
2. Open the MTA: Explorer and create a new configuration
3. Run the configuration (cancel if you don't want to wait)
4. Select --overwrite
5. Run the configuration

Expected Results:
Analyzer to run

Actual Results:

time="2023-12-05T13:35:01-05:00" level=error msg="failed to validate flags" error="output dir /Users/dzager/.windup/tooling/vscode/2pkzd8cc9-7x3l4humx already exists and --overwrite not set"