Skip to content

fix: support relocated RD client library - #139

Open
twangodev wants to merge 4 commits into
mainfrom
fix/iu-2026-2-rd-client
Open

fix: support relocated RD client library#139
twangodev wants to merge 4 commits into
mainfrom
fix/iu-2026-2-rd-client

Conversation

@twangodev

@twangodev twangodev commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • compile against JetBrains' published RD client artifact selected for the configured IntelliJ Platform
  • declare the generated RD model through the official bundled-module dependency
  • correct the README format matrix to match the plugin descriptor

Validation

  • ./gradlew check buildPlugin
  • confirmed the plugin ZIP does not bundle RD dependencies
  • Plugin Verifier: IU 2025.3.6.1, IU 2026.1.5, and IU 2026.2.1 compatible

A live Gateway Remote Development smoke test was not run.

Copilot AI lite review requested due to automatic review settings August 17, 2026 11:08
@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a191234c-9421-42b3-8481-a1c4a9fddc32


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the :client module’s build-time dependencies to handle the IntelliJ RD client JAR relocation between 2025.3–2026.2 IDE layouts, and aligns documented format support with the frontend module descriptor.

Changes:

  • Resolve the RD client library from the configured IntelliJ Platform platformPath, supporting both the legacy and new IDE layouts.
  • Switch to the official bundled module dependency for the generated RD IDE model.
  • Update the README format matrix to match the extensions declared in dev.twango.jetplay.frontend.xml.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
README.md Aligns the format support matrix with the actual supported extensions in the frontend module descriptor.
client/build.gradle.kts Updates Gradle dependency resolution to locate the RD client library across IDE layout changes and uses the bundled generated RD model module.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI review requested due to automatic review settings August 17, 2026 11:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@github-actions

Copy link
Copy Markdown
Contributor

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 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:

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

      - name: 'Qodana Scan'
        uses: JetBrains/qodana-action@v2026.2.0
        with:
          upload-result: true
Contact Qodana team

Contact us at qodana-support@jetbrains.com

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