Skip to content

Update plugin to support 2026.1 releases#207

Merged
charlieegan3 merged 3 commits into
open-policy-agent:masterfrom
idexter:master
May 27, 2026
Merged

Update plugin to support 2026.1 releases#207
charlieegan3 merged 3 commits into
open-policy-agent:masterfrom
idexter:master

Conversation

@idexter
Copy link
Copy Markdown
Contributor

@idexter idexter commented May 19, 2026

Description

Update plugin to work with 2026.* versions of IDE.

Tested on Goland 2026.1.2

Screenshot 2026-05-20 at 00 06 51

Tested using environment:

$ uname -a
Darwin dextermac.local 24.6.0 Darwin Kernel Version 24.6.0: Mon Jan 19 22:00:10 PST 2026; root:xnu-11417.140.69.708.3~1/RELEASE_X86_64 x86_64

$ opa version

Version: 1.16.2
Build Commit:
Build Timestamp:
Build Hostname:
Go Version: go1.26.3
Platform: darwin/amd64
Rego Version: v1
WebAssembly: unavailable

$ regal version

Version:       0.40.0
Go Version:    go1.26.2
OPA Version:   1.15.2
Platform:      darwin/amd64
Commit:        Homebrew
Timestamp:     2026-04-21T13:07:38Z
Installed via: Homebrew

Fixes #196

@idexter idexter mentioned this pull request May 19, 2026
@charlieegan3
Copy link
Copy Markdown
Contributor

Hey, are you still having issues on the 0.17.0 release with this goland version?

@idexter
Copy link
Copy Markdown
Contributor Author

idexter commented May 21, 2026

Hey, are you still having issues on the 0.17.0 release with this goland version?

Sure, 0.17.0 is not compatible with 2026.* branch...

Screenshot 2026-05-21 at 19 52 05

Due to Compatibility docs it should be built with 261 tree like here in this PR

@charlieegan3
Copy link
Copy Markdown
Contributor

I see, looking at the previous times we've update this we also need to do update:

  • Kotlin JVM plugin
  • IntelliJ platform plugin
  • psiViewerPlugin

Are you able to get the plugin running with this version with ./gradlew :plugin:runIde?

@idexter
Copy link
Copy Markdown
Contributor Author

idexter commented May 26, 2026

Are you able to get the plugin running with this version with ./gradlew :plugin:runIde?

If I don't change anything it builds and runs sucessfully. But with IDEA 2025.3.1

Screenshot 2026-05-26 at 22 52 49
  • When I change platformVersion=2026.1.2 in gradle.properties it doesn't build and ask for update Kotlin at least to 2.3.0.
  • Using Kotlin 2.3.0 it fail to compile with some errors:
> Task :compileKotlin FAILED
e: file:///home/open-policy-agent/opa-idea-plugin/src/main/kotlin/org/openpolicyagent/ideaplugin/ide/actions/utils.kt:52:27 Only safe (?.) or non-null asserted (!!.) calls are allowed on a nullable receiver of type 'PsiFile?'.
e: file:///home/open-policy-agent/opa-idea-plugin/src/main/kotlin/org/openpolicyagent/ideaplugin/ide/actions/utils.kt:68:27 Only safe (?.) or non-null asserted (!!.) calls are allowed on a nullable receiver of type 'PsiFile?'.

😢

@idexter
Copy link
Copy Markdown
Contributor Author

idexter commented May 26, 2026

I've try to change Kotlin version and plantformVersion and fix suggestions. Now it works using ./gradlew :plugin:runIde

Screenshot 2026-05-26 at 23 25 31

@charlieegan3
Copy link
Copy Markdown
Contributor

Hey, thanks for your work on this. I am just trying to make sure I can test locally the same and then we can get this in.

idexter added 2 commits May 27, 2026 15:38
Signed-off-by: Anton Kucherov <idexter.ru@gmail.com>
Signed-off-by: Anton Kucherov <idexter.ru@gmail.com>
@charlieegan3
Copy link
Copy Markdown
Contributor

I've been able to test this locally, let's see if we can get the tests running here and then we can merge.

- replace runReadAction with ApplicationManager pattern
- migrate ProcessAdapter to ProcessListener
- update FileTypeDescriptor to FileChooserDescriptor API
- isolate fastutil in grammarKitClasspath to prevent runtime conflicts with the platform's bundled version.
- Add -jvm-default=no-compatibility to prevent bridge method issues.
- Remove redundant null operators.

Signed-off-by: Charlie Egan <charlie_egan@apple.com>
Copy link
Copy Markdown
Contributor

@charlieegan3 charlieegan3 left a comment

Choose a reason for hiding this comment

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

I had a number of issues and deprecations in the tests, see 2f4007d. But I think we should be good now!

@charlieegan3 charlieegan3 merged commit 51f698b into open-policy-agent:master May 27, 2026
6 checks passed
@charlieegan3
Copy link
Copy Markdown
Contributor

Thanks for the work on this, I'll try and get a release out shortly.

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.

Support for 2026

2 participants