Comparing snjeza:main...microsoft:main · snjeza/java-debug · GitHub
Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: snjeza/java-debug
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: microsoft/java-debug
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 8 commits
  • 11 files changed
  • 8 contributors

Commits on Mar 12, 2026

  1. Configuration menu
    Copy the full SHA
    b35f033 View commit details
    Browse the repository at this point in the history

Commits on Mar 30, 2026

  1. Exclude Map.Entry from lazy loading to show key:value inline (microso…

    …ft#621)
    
    Map.Entry objects were incorrectly treated as lazy-loading candidates
    because they are included in COLLECTION_TYPES. This caused the debugger
    to display entries as 'HashMap$Node@id' instead of showing the actual
    key:value details inline.
    
    Map.Entry's details computation (getKey + getValue) is lightweight,
    so eager evaluation is safe and significantly improves UX when debugging
    large Maps.
    
    Fixes microsoft/vscode-java-debug#1605
    
    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    wenytang-ms and Copilot authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    26b05eb View commit details
    Browse the repository at this point in the history
  2. Use JDTUtils.toUri() for decompiled class file URIs (microsoft#624)

    Replace manual jdt:// URI construction in getFileURI(IClassFile) with
    JDTUtils.toUri(classFile) to align with eclipse.jdt.ls PR #3666.
    
    This ensures the debugger generates the same URI format as the language
    server, fixing the issue where classes with source code are opened twice
    (once as .java, once as .class) during debugging.
    
    Fixes microsoft#623
    Related: eclipse-jdtls/eclipse.jdt.ls#3729
    
    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    wenytang-ms and Copilot authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    07b1e4c View commit details
    Browse the repository at this point in the history

Commits on Mar 31, 2026

  1. fix: handle NoSuchMethodError for isMainMethodCandidate() on older JD…

    …T Core (microsoft#622)
    
    Catch NoSuchMethodError when calling SourceMethod.isMainMethodCandidate()
    in ResolveMainClassHandler and ResolveMainMethodHandler. This method was
    added in JDT Core 3.36 and is unavailable on older versions, causing
    the debugger to fail entirely when resolving main classes.
    
    Fixes microsoft/vscode-java-debug#1598
    
    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    Co-authored-by: Changyong Gong <shawn.gong@hotmail.com>
    3 people authored Mar 31, 2026
    Configuration menu
    Copy the full SHA
    a4c27fb View commit details
    Browse the repository at this point in the history

Commits on Apr 20, 2026

  1. Added attributes to VariablePresentationHint as specified in protocol…

    … definition (microsoft#625)
    
    * added attributes to VariablePresentationHint as specified in protocol definition
    
    * Fixed issue with lazy VariablePresentationHint.
    Restricted constant to Primitive & StringRefs
    Daniel-Pfeffer authored Apr 20, 2026
    Configuration menu
    Copy the full SHA
    ae2847e View commit details
    Browse the repository at this point in the history

Commits on Apr 30, 2026

  1. Handle VM disconnection gracefully and reorder termination logic (mic…

    …rosoft#626)
    
    * Ignore VMDisconnected on detach; reorder terminate
    
    Wrap vm.dispose() in a try/catch to ignore VMDisconnectedException when the VM has already disconnected (e.g. process terminated) to avoid spurious errors during detach. Reorder calls in DisconnectRequestHandler to invoke terminate() before detach() when terminateDebuggee is true and the session is not attached, ensuring the debuggee is terminated prior to detaching.
    
    * Always detach debug session on terminate
    
    Wrap debugSession.terminate() in a try/finally so debugSession.detach() is always invoked when terminateDebuggee is true and the session is not attached. This ensures the session is detached even if terminate() throws, avoiding potential dangling sessions. Other detach logic is unchanged.
    zepedrocosta authored Apr 30, 2026
    Configuration menu
    Copy the full SHA
    3adbc03 View commit details
    Browse the repository at this point in the history

Commits on Jun 9, 2026

  1. fix: prefer sourcePaths before JDT source URIs (microsoft#629)

    * Prefer sourcePaths before JDT source URIs
    
    When JDT source lookup returns a non-file URI, check debug sourcePaths before returning that URI to the client.
    
    Add a regression test covering cached JDT URI results being overridden by sourcePaths.
    
    * 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.
    wenytang-ms authored Jun 9, 2026
    Configuration menu
    Copy the full SHA
    3f03453 View commit details
    Browse the repository at this point in the history

Commits on Jun 11, 2026

  1. fix: correct method location typo (microsoft#630)

    Co-authored-by: titanniya542-spec <titanniya542-spec@users.noreply.github.com>
    titanniya542-spec and titanniya542-spec authored Jun 11, 2026
    Configuration menu
    Copy the full SHA
    d6a66c8 View commit details
    Browse the repository at this point in the history
Loading