The value of thefolder URI parameter is passed to the IDE&Project dialog when handling URIs. The folder value will be rendered instead of the default home folder if the value is not blank.

I've also changed the list of Gateway versions we run the verifier against to include Gateway 2025.1 while excluding some ancient releases.Without this commit the PR will fail to build, the declared snapshots are no longer available in the repository
-resolves#466

fioan89 added3 commits

May 16, 2025 00:28

chore: use newer snapshot build for testing and building

92234b3

I've also changed the list of Gateway versions we run the verifier againstto include Gateway 2025.1 while excluding some ancient releases.Without this commit the PR will fail to build, the declared snapshots areno longer available in the repository

fix: pass the folder parameter to the IDE&Project dialog

e4d9fbf

The value of the `folder` URI parameter is passed to the IDE&Projectdialog when handling URIs. The folder value will be rendered insteadof the default home folder if the value is not blank.-resolves#466

chore: next version is 2.20.1

3e3ce35

fioan89 requested review frommatifali andf0ssel

May 15, 2025 21:35

Copy link

CollaboratorAuthor

fioan89 commentedMay 15, 2025

I'll also push a new release once this PR is merged.

Copy link

Contributor

github-actionsbot commentedMay 15, 2025

Qodana Community for JVM

23 new problems were found

Inspection name	Severity	Problems
`Usage of API marked for removal`	🔴 Failure	13
`Local 'var' is never modified and can be declared as 'val'`	🔶 Warning	1
`Incorrect string capitalization`	🔶 Warning	1
`Constant conditions`	🔶 Warning	1
`Usage of redundant or deprecated syntax or deprecated symbols`	🔶 Warning	1
`Redundant nullable return type`	🔶 Warning	1
`String concatenation that can be converted to string template`	◽️ Notice	2
`Argument could be converted to 'Set' to improve performance`	◽️ Notice	1
`Class member can have 'private' visibility`	◽️ Notice	1
`Redundant lambda arrow`	◽️ Notice	1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked

View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

Register atQodana Cloud andconfigure the action
UseGitHub Code Scanning with Qodana
HostQodana report at GitHub Pages
Inspect and useqodana.sarif.json (seethe Qodana SARIF format for details)

To get*.log files or any other Qodana artifacts, run the action withupload-result option set totrue,
so that the action will upload the files as the job artifacts:

      -name:'Qodana Scan'uses:JetBrains/qodana-action@v2023.3.2with:upload-result:true

Contact Qodana team

Or via our issue tracker:https://jb.gg/qodana-issue
Or share your feedback:https://jb.gg/qodana-discussions

matifali reviewed

May 16, 2025

View reviewed changes

Copy link

Member

matifali left a comment

Choose a reason for hiding this comment

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

LGTM. There are a few warnings saying, we are using some marked for removal libraries

gradle.properties

		@@ -4,10 +4,10 @@ pluginGroup=com.coder.gateway
		# Zip file name.
		pluginName=coder-gateway
		# SemVer format -> https://semver.org
		pluginVersion=2.20.0
		pluginVersion=2.20.1

Copy link

Member

matifaliMay 16, 2025

Choose a reason for hiding this comment

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

As we are dropping support for the minimum version, what do you think about bumping the minor version?
@code-asher, how have we done it historically?

Copy link

CollaboratorAuthor

fioan89May 16, 2025

Choose a reason for hiding this comment

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

We're not dropping support for the minimum version — we're just building with a newer snapshot because that's what's available in the snapshot repositories. I've updated the versions we test the plugin against in GitHub workflows and locally. Since we’re no longer using any new Gateway APIs, I think it’s unnecessary to keep verifying against so many older versions.