Address source path review feedback

Use a null-safe SourceType comparison and convert sourcePaths matches to the client's expected path format.

Extend regression coverage for URI clients and null source types.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: wenytang-ms

com.microsoft.java.debug.core/src/main/java/com/microsoft/java/debug/core/adapter/handler/StackTraceRequestHandler.java

@@ -302,13 +302,18 @@ public static Types.Source convertDebuggerSourceToClient(String fullyQualifiedNa
     }
 
     private static int getSourceReference(Source source, IDebugAdapterContext context) {
-        return source.getType().equals(SourceType.REMOTE) ? context.createSourceReference(source.getUri()) : 0;
+        return SourceType.REMOTE == source.getType() ? context.createSourceReference(source.getUri()) : 0;
     }
 
     private static Types.Source resolveSourceFromSourcePaths(String sourceName, String relativeSourcePath,
             IDebugAdapterContext context) {
         String absoluteSourcepath = AdapterUtils.sourceLookup(context.getSourcePaths(), relativeSourcePath);
-        return absoluteSourcepath == null ? null : new Types.Source(sourceName, absoluteSourcepath, 0);
+        if (absoluteSourcepath == null) {
+            return null;
+        }
+
+        String clientPath = AdapterUtils.convertPath(absoluteSourcepath, false, context.isClientPathsAreUri());
+        return new Types.Source(sourceName, clientPath, 0);
     }
 
     private String formatMethodName(String methodName, List<String> argumentTypeNames, String fqn, boolean showContextClass, boolean showParameter) {

com.microsoft.java.debug.core/src/test/java/com/microsoft/java/debug/core/adapter/handler/StackTraceRequestHandlerTest.java

@@ -67,6 +67,37 @@ public void convertDebuggerSourceShouldPreferSourcePathsOverCachedJdtUri() throw
 
         assertEquals(sourceFile.toString(), localSource.path);
         assertEquals(0, localSource.sourceReference);
+
+        context.setClientPathsAreUri(true);
+        Types.Source localUriSource = StackTraceRequestHandler.convertDebuggerSourceToClient(fullyQualifiedName,
+                sourceName, relativeSourcePath, context);
+
+        assertEquals(sourceFile.toUri().toString(), localUriSource.path);
+        verifyAll();
+    }
+
+    @Test
+    public void convertDebuggerSourceShouldHandleSourceWithNullType() throws Exception {
+        String fullyQualifiedName = "com.example.Bar";
+        String sourceName = "Bar.java";
+        Path sourceFile = tempFolder.newFile(sourceName).toPath();
+        String sourceUri = sourceFile.toUri().toString();
+
+        ISourceLookUpProvider sourceProvider = createMock(ISourceLookUpProvider.class);
+        expect(sourceProvider.getSource(fullyQualifiedName, sourceName))
+                .andReturn(new Source(sourceUri, null))
+                .once();
+        replayAll();
+
+        ProviderContext providerContext = new ProviderContext();
+        providerContext.registerProvider(ISourceLookUpProvider.class, sourceProvider);
+        DebugAdapterContext context = new DebugAdapterContext(null, providerContext);
+
+        Types.Source source = StackTraceRequestHandler.convertDebuggerSourceToClient(fullyQualifiedName, sourceName,
+                sourceName, context);
+
+        assertEquals(sourceFile.toString(), source.path);
+        assertEquals(0, source.sourceReference);
         verifyAll();
     }
 }