Skip to content

Fix issues in the MI Extension - #1568

Merged
ChinthakaJ98 merged 4 commits into
wso2:release-4.2.xfrom
ChinthakaJ98:issue-fix-03
Aug 6, 2026
Merged

Fix issues in the MI Extension#1568
ChinthakaJ98 merged 4 commits into
wso2:release-4.2.xfrom
ChinthakaJ98:issue-fix-03

Conversation

@ChinthakaJ98

Copy link
Copy Markdown
Contributor

This PR will fix the below mentioned issues.

  • JDK 25 build issue with class mediators
  • Issues in docker-build configuration changes
  • No option to edit the pom.xml of the docker-build module in a consolidated project

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 540c036e-6547-4c9a-9146-4b4763114b7f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • ✅ Review completed - (🔄 Check again to review again)

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/mi-extension/src/project-explorer/project-explorer-provider.ts`:
- Around line 219-223: In the hasPom branch of the project explorer provider,
replace buildReadOnlyFileEntry for pomPath with the standard editable file-entry
builder so pom.xml is editable and does not receive the readOnlyFile context.
Keep buildReadOnlyFolderChildren unchanged so the docker-build folder remains
read-only.

In `@packages/mi-extension/src/util/onboardingUtils.ts`:
- Around line 806-813: Update the updatePomXml call guarded by
hasSynapseCoreDependency to match both groupId org.apache.synapse and artifactId
synapse-core before selecting version. Preserve the existing synapseCoreVersion
update while preventing dependencies with the same artifactId from other groups
from being modified.

In `@packages/mi-extension/src/util/templates.ts`:
- Around line 496-507: Update libLineRegex in the existing-content parsing flow
to accept one or more whitespace characters between COPY, the library path, and
${WSO2_SERVER_HOME}, while preserving the matched original line in
existingLibLines. Add regression coverage for multiple spaces and tab-separated
valid COPY directives to ensure libsCopyLines reuses them instead of emitting
commented defaults.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 258d078d-5b7e-4f45-9f9e-556bbc5e7c13

📥 Commits

Reviewing files that changed from the base of the PR and between 77e19b1 and 4b7f96b.

📒 Files selected for processing (5)
  • packages/mi-extension/src/debugger/pomResolver.ts
  • packages/mi-extension/src/project-explorer/project-explorer-provider.ts
  • packages/mi-extension/src/stateMachine.ts
  • packages/mi-extension/src/util/onboardingUtils.ts
  • packages/mi-extension/src/util/templates.ts

Comment thread packages/mi-extension/src/util/onboardingUtils.ts Outdated
Comment thread packages/mi-extension/src/util/templates.ts Outdated
@ChinthakaJ98
ChinthakaJ98 merged commit 392c2a2 into wso2:release-4.2.x Aug 6, 2026
6 checks passed
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