Sync stable/mi to main after release v0.24.0-wso2v92

.github/workflows/release.yml

@@ -47,11 +47,43 @@ jobs:
         name: ${{ steps.ls.outputs.fileName }}.zip
     - name: Create a release in repo
       run: |
-        createResponse=`curl -X POST  -H "Accept: application/vnd.github.v3+json" \
-        -H "Authorization:token ${{ secrets.GIT_BOT_TOKEN }}" -d '{"tag_name":"v${{ steps.ls.outputs.version }}", \
-        "draft":false, "name": "Release v${{ steps.ls.outputs.version }}", "prerelease":true}' \
-        https://api.github.com/repos/${{ github.repository }}/releases` \
-          && id=`echo "$createResponse" | sed -n -e 's/"id":\ \([0-9]\+\),/\1/p' | head -n 1 | sed 's/[[:blank:]]//g'` && \
-          uploadResponse=`curl -X POST -H "Authorization:token ${{ secrets.GIT_BOT_TOKEN }}" -H "Content-Type:application/octet-stream" \
+        set -euo pipefail
+        createResponse=$(curl --fail-with-body -X POST \
+          -H "Accept: application/vnd.github.v3+json" \
+          -H "Authorization:token ${{ secrets.GIT_BOT_TOKEN }}" \
+          -d '{"tag_name":"v${{ steps.ls.outputs.version }}", "draft":false, "name": "Release v${{ steps.ls.outputs.version }}", "prerelease":true}' \
+          https://api.github.com/repos/${{ github.repository }}/releases)
+        id=$(echo "$createResponse" | jq -r '.id // empty')
+        if [ -z "$id" ]; then
+          echo "Failed to parse release id from response:"
+          echo "$createResponse"
+          exit 1
+        fi
+        curl --fail-with-body -X POST \
+          -H "Authorization:token ${{ secrets.GIT_BOT_TOKEN }}" \
+          -H "Content-Type:application/octet-stream" \
           --data-binary @${{ steps.ls.outputs.fileName }}.zip \
-          https://uploads.github.com/repos/${{ github.repository }}/releases/$id/assets?name=${{ steps.ls.outputs.fileName }}.zip`
+          "https://uploads.github.com/repos/${{ github.repository }}/releases/$id/assets?name=${{ steps.ls.outputs.fileName }}.zip"
+
+    - name: Create sync PR from stable/mi to main
+      if: github.ref == 'refs/heads/stable/mi'
+      env:
+        GH_TOKEN: ${{ secrets.GIT_BOT_TOKEN }}
+      run: |
+        git fetch --unshallow origin main stable/mi 2>/dev/null || git fetch origin main stable/mi
+        commits=$(git rev-list --count origin/main..origin/stable/mi)
+        if [ "$commits" -eq 0 ]; then
+          echo "No new commits on stable/mi compared to main — skipping PR creation."
+          exit 0
+        fi
+        existing=$(gh pr list --repo ${{ github.repository }} --base main --head stable/mi --state open --json number --jq '.[0].number // empty')
+        if [ -n "$existing" ]; then
+          echo "An open PR from stable/mi to main already exists: #$existing"
+          exit 0
+        fi
+        gh pr create \
+          --repo ${{ github.repository }} \
+          --base main \
+          --head stable/mi \
+          --title "Sync stable/mi to main after release v${{ steps.ls.outputs.version }}" \
+          --body "Automated PR to sync \`stable/mi\` into \`main\` following the successful release of v${{ steps.ls.outputs.version }}."

org.eclipse.lemminx/pom.xml

@@ -3,7 +3,7 @@
 	<parent>
 		<groupId>org.wso2.language.server</groupId>
 		<artifactId>mi-language-server-parent</artifactId>
-		<version>0.24.0-wso2v90</version>
+		<version>0.24.0-wso2v95</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<name>MI Language Server</name>
@@ -15,7 +15,7 @@
 		<dev.build.timestamp>${maven.build.timestamp}</dev.build.timestamp>
 		<cbi.jarsigner.skip>true</cbi.jarsigner.skip>
 		<native.maven.plugin.version>0.9.16</native.maven.plugin.version>
-		<mi.connector.generator.version>0.9.10</mi.connector.generator.version>
+		<mi.connector.generator.version>0.9.11</mi.connector.generator.version>
 		<graalvm.static />
 	</properties>
 	<build>

org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/SynapseLanguageService.java

@@ -150,6 +150,7 @@
 import org.eclipse.lemminx.customservice.synapse.idp.PdfToImagesRequest;
 import org.eclipse.lemminx.dom.DOMDocument;
 import org.eclipse.lemminx.extensions.contentmodel.settings.XMLValidationSettings;
+import org.eclipse.lemminx.extensions.synapse.SynapseDiagnosticsParticipant;
 import org.eclipse.lemminx.services.extensions.completion.ICompletionResponse;
 import org.eclipse.lemminx.settings.SharedSettings;
 import org.eclipse.lemminx.uriresolver.URIResolverExtensionManager;
@@ -346,8 +347,21 @@ private PublishDiagnosticsParams doDiagnostics(DOMDocument xmlDocument, CancelCh
     public CompletableFuture<PublishDiagnosticsParams> codeDiagnostic(CodeDiagnosticRequest param) {
 
         return CompletableFuture.supplyAsync(() -> {
-            DOMDocument xmlDocument = Utils.getDOMDocument(param.getCode(), uriResolverExtensionManager);
-            return doDiagnostics(xmlDocument, NULL_CANCEL_CHECKER);
+            // Use the real file path (when supplied) as the document URI. Several diagnostics are
+            // gated on the document path — e.g. SynapseExpressionValidator only runs for files under
+            // src/main/wso2mi/artifacts — so the literal "temp" fallback would silently drop them.
+            // Treat a blank fileName as missing, otherwise an unusable URI would skip those checks.
+            String uri = StringUtils.isBlank(param.getFileName()) ? "temp" : param.getFileName();
+            // Opt-out (default off) for cross-file reference checks: the agent validates a file
+            // before its referenced siblings are written, so those checks would fire spuriously.
+            // Set/clear around doDiagnostics on this thread; the editor never sets it.
+            try {
+                SynapseDiagnosticsParticipant.setSkipCrossFileValidation(param.isSkipCrossFileValidation());
+                DOMDocument xmlDocument = Utils.getDOMDocument(param.getCode(), uri, uriResolverExtensionManager);
+                return doDiagnostics(xmlDocument, NULL_CANCEL_CHECKER);
+            } finally {
+                SynapseDiagnosticsParticipant.clearSkipCrossFileValidation();
+            }
         });
     }
 
@@ -1044,7 +1058,10 @@ public CompletableFuture<ConnectorGeneratorResponse> generateConnector(Connector
                         connectorGenReq.connectorProjectPath, projectServerVersion, projectUri);
             }
         } catch (Exception e) {
-            log.log(Level.SEVERE, "Error occurred while generating the connector", e);
+			String errorMsg = "Error occurred while generating the connector: " + e.getMessage();
+            log.log(Level.SEVERE, errorMsg, e);
+            ConnectorGeneratorResponse errorResponse = new ConnectorGeneratorResponse(false, null, errorMsg);
+            return CompletableFuture.supplyAsync(() -> errorResponse);
         }
         ConnectorGeneratorResponse response = new ConnectorGeneratorResponse(filePath != null, filePath);
         return CompletableFuture.supplyAsync(() -> response);
@@ -1202,6 +1219,12 @@ public CompletableFuture<Either<ConnectorInfoResponse, String>> resolveConnector
         });
     }
 
+    @Override
+    public CompletableFuture<String> fetchInboundConnectors() {
+
+        return CompletableFuture.supplyAsync(() -> inboundConnectorHolder.getCustomInboundConnectors());
+    }
+
     public String getProjectUri() {
         return projectUri;
     }
@@ -1233,7 +1256,8 @@ public void dispose() {
 
     private void packHttpConnector() {
 
-        if (Utils.compareVersions(projectServerVersion, Constant.MI_440_VERSION) >= 0) {
+        if (Utils.compareVersions(projectServerVersion, Constant.MI_440_VERSION) >= 0
+                && Utils.hasDependency(projectUri, Constant.HTTP_CONNECTOR_ARTIFACT_ID)) {
             String projectId = new File(projectUri).getName() + "_" + Utils.getHash(projectUri);
             String connectorDownloadPath = Path.of(System.getProperty(Constant.USER_HOME), Constant.WSO2_MI,
                     Constant.CONNECTORS, projectId, Constant.DOWNLOADED).toString();

org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/XMLTextDocumentService.java

@@ -39,6 +39,7 @@
 import org.eclipse.lemminx.dom.DOMDocument;
 import org.eclipse.lemminx.dom.DOMParser;
 import org.eclipse.lemminx.extensions.contentmodel.settings.XMLValidationRootSettings;
+import org.eclipse.lemminx.extensions.synapse.SynapseDiagnosticsParticipant;
 import org.eclipse.lemminx.services.DocumentSymbolsResult;
 import org.eclipse.lemminx.services.SymbolInformationResult;
 import org.eclipse.lemminx.services.XMLLanguageService;
@@ -599,7 +600,14 @@ public CompletableFuture<List<ColorPresentation>> colorPresentation(ColorPresent
 	public void didSave(DidSaveTextDocumentParams params) {
 		computeAsync((monitor) -> {
 			// A document was saved, collect documents to revalidate
-			SaveContext context = new SaveContext(params.getTextDocument().getUri());
+			String savedUri = params.getTextDocument().getUri();
+			// LSP document URIs use '/', but normalize defensively so a backslash path also matches on Windows.
+			if (savedUri != null && savedUri.replace('\\', '/').contains("src/main/wso2mi")) {
+				// An artifact/resource file was saved — drop the cached cross-file index so the
+				// revalidation below (and sibling files) sees the updated set instead of stale data.
+				SynapseDiagnosticsParticipant.invalidateArtifactIndexCache();
+			}
+			SaveContext context = new SaveContext(savedUri);
 			doSave(context);
 
 			// Manage didSave document lifecycle participants

org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/XMLWorkspaceService.java

@@ -21,6 +21,7 @@
 
 import org.eclipse.lemminx.commons.WorkspaceFolders;
 import org.eclipse.lemminx.customservice.synapse.utils.Constant;
+import org.eclipse.lemminx.extensions.synapse.SynapseDiagnosticsParticipant;
 import org.eclipse.lemminx.services.extensions.commands.IXMLCommandService;
 import org.eclipse.lsp4j.DidChangeConfigurationParams;
 import org.eclipse.lsp4j.DidChangeWatchedFilesParams;
@@ -94,11 +95,19 @@ public void didChangeWatchedFiles(DidChangeWatchedFilesParams params) {
 				.getTextDocumentService();
 		List<FileEvent> changes = params.getChanges();
 		for (FileEvent change : changes) {
-			if (change.getUri().contains(Constant.INBOUND_CONNECTORS_DIR) && change.getUri().contains(".zip")) {
+			if ((change.getUri().contains(Constant.INBOUND_ENDPOINTS)
+					|| change.getUri().contains(Constant.INBOUND_CONNECTORS_DIR)) && change.getUri().contains(".zip")) {
 				((SynapseLanguageService) xmlLanguageServer.getSynapseLanguageService()).updateInboundConnectors();
 			} else if (change.getUri().contains(Constant.CONNECTORS) && change.getUri().contains(".zip")) {
 				((SynapseLanguageService) xmlLanguageServer.getSynapseLanguageService()).updateConnectors();
 			} else {
+				// LSP URIs use '/', but normalize defensively so a backslash path also matches on Windows.
+				if (change.getUri().replace('\\', '/').contains("src/main/wso2mi")) {
+					// An artifact/resource file changed on disk — drop the cached cross-file index so
+					// the next diagnostics run rebuilds it (otherwise a just-written sibling stays
+					// "unresolved" for up to the cache TTL).
+					SynapseDiagnosticsParticipant.invalidateArtifactIndexCache();
+				}
 				if (!xmlTextDocumentService.documentIsOpen(change.getUri())) {
 					xmlTextDocumentService.doSave(change.getUri());
 				}

org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/customservice/ISynapseLanguageService.java

@@ -359,4 +359,7 @@ CompletableFuture<DriverMavenCoordinatesResponse> getDriverMavenCoordinates(
 
     @JsonRequest
     CompletableFuture<Either<InboundEndpointInfo, String>> getInboundInfo(InboundInfoRequest request);
+
+    @JsonRequest
+    CompletableFuture<String> fetchInboundConnectors();
 }

org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/customservice/synapse/CodeDiagnosticRequest.java

@@ -17,6 +17,8 @@
 public class CodeDiagnosticRequest {
 
     private String code;
+    private String fileName;
+    private boolean skipCrossFileValidation;
 
     public String getCode() {
 
@@ -27,4 +29,30 @@ public void setCode(String code) {
 
         this.code = code;
     }
+
+    public String getFileName() {
+
+        return fileName;
+    }
+
+    public void setFileName(String fileName) {
+
+        this.fileName = fileName;
+    }
+
+    /**
+     * When true, cross-file reference checks (which depend on other artifact files existing) are
+     * skipped for this request. Defaults to false, so the editor and the explicit "validate all"
+     * path are unaffected. The MI Copilot agent sets it for per-file auto-validation after a write,
+     * where a referenced sibling artifact may not exist on disk yet.
+     */
+    public boolean isSkipCrossFileValidation() {
+
+        return skipCrossFileValidation;
+    }
+
+    public void setSkipCrossFileValidation(boolean skipCrossFileValidation) {
+
+        this.skipCrossFileValidation = skipCrossFileValidation;
+    }
 }