runInTerminal reverse-request missing cwd key #251

Activity

1. testforstephen commented on Feb 14, 2019

   

   Jinbo Wang (testforstephen)

   on Feb 14, 2019

   Contributor

   More actions

   Ben Jackson (@puremourning) Follow the code https://github.com/Microsoft/java-debug/blob/master/com.microsoft.java.debug.core/src/main/java/com/microsoft/java/debug/core/adapter/handler/LaunchWithDebuggingDelegate.java#L75, actually the java debug server will send cwd key from launch.json to the client.

   How did you use java debug server? VS Code Debugger for Java extension or self implemented debugger?
2. Jinbo Wang (testforstephen)

   added

   needs more info

   on Feb 14, 2019
3. puremourning commented on Sep 30, 2020

   

   Ben Jackson (puremourning)

   on Sep 30, 2020

   Author

   More actions

   Thanks

   the java debug server will send cwd key from launch.json to the client

   Is that a mandatory launch parameter ? The adapter seems to work fine without it and the docs for vscode suggest it's optional: "cwd - The working directory of the program. Defaults to ${workspaceFolder}."

   I guess the vscode-java plugin always sets it ? Is there a specification for the arguments accepted/required by this adapter when not using vscode-java ?

   How did you use java debug server? VS Code Debugger for Java extension or self implemented debugger?

   In Vimspector - another DAP client accessing java-debugger through eclipse.jdt.ls.
4. github-actions commented on Nov 14, 2025

   

   github-actionsBot

   on Nov 14, 2025 – with GitHub Actions

   More actions

   Hi Ben Jackson (@puremourning), I'm an AI Support assistant here to help with your issue. While the team reviews your request, I wanted to provide some possible tips and documentation that might help you in the meantime.

   • According to the DAP spec, the cwd field is mandatory on runInTerminal reverse-requests: https://microsoft.github.io/debug-adapter-protocol/specification#Reverse_Requests_RunInTerminal

   • Update the Java Debug adapter to populate cwd when sending the request. In LaunchRequestHandler.java, locate where you build the RunInTerminalRequestArguments and change it from:

     new RunInTerminalRequestArguments(kind, title, /* no cwd */, args);

     to something like:

     String cwd = launchConfig.getCwd() != null
         ? launchConfig.getCwd()
         : System.getProperty("user.dir");
     new RunInTerminalRequestArguments(kind, title, cwd, args);

   • Similar high-confidence reference:
     • Unknown event "processid" (Unknown event "processid" #460): notes other DAP-spec mismatches in the Java Debug server, including runInTerminal details. Unknown event "processid" #460

   Other references with low confidence

   • Definition of current working directory is missing (remove com.sun.jdi.VMDisconnectedException from reasonable places. #104): discusses setting a workspace CWD for tests, but not DAP reverse requests. Definition of current working directory is missing vscode-java-test#104
   • Can't read an input (debug console) (bump version to 0.37.0 #415): about using cwd in launch configs for input redirection. Can't read an input (debug console) redhat-developer/vscode-java#415

   The team will respond to your issue shortly. I hope these suggestions are helpful in the meantime. If this comment helped you, please give it a 👍. If the suggestion was not helpful or incorrect, please give it a 👎. Your feedback helps us improve!
5. github-actions

   added

   ai-triaged

   bug

   on Nov 14, 2025