Skip to content

Conversation

phillip-kruger
Copy link
Member

@phillip-kruger phillip-kruger commented Sep 29, 2025

fix #50316

@myfear - Can you test against this PR ? I could not recreate the issue, I tried in the test case, but that all works.

@myfear
Copy link

myfear commented Sep 29, 2025

Dug a little deeper: cursor/cursor#3614
A spec violation by Cursor's JSON RPC implementation it seems.

Cursor sends: "parameters": "{}" (string)
Expected: "parameters": {} (object)

@phillip-kruger
Copy link
Member Author

I can not recreate this with cursor:

Version: 1.7.17
VSCode Version: 1.99.3
Commit: 34881053400013f38e2354f1479c88c9067039a0
Date: 2025-09-29T03:10:26.099Z
Electron: 34.5.8
Chromium: 132.0.6834.210
Node.js: 20.19.1
V8: 13.2.152.41-electron.0
OS: Linux x64 6.16.8-200.fc42.x86_64

My cursor successfully fetched the tool list and successfully called method with and without parameterrs.

@myfear
Copy link

myfear commented Sep 30, 2025

Still reliably can. Seems to be an ARM problem

Version: 1.7.17 (Universal)
VSCode Version: 1.99.3
Commit: 34881053400013f38e2354f1479c88c9067039a0
Date: 2025-09-29T03:10:26.099Z (1 day ago)
Electron: 34.5.8
Chromium: 132.0.6834.210
Node.js: 20.19.1
V8: 13.2.152.41-electron.0
OS: Darwin arm64 25.0.0

@phillip-kruger phillip-kruger marked this pull request as ready for review September 30, 2025 06:52

This comment has been minimized.

This comment has been minimized.

Copy link

quarkus-bot bot commented Sep 30, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit f68786b.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@phillip-kruger phillip-kruger merged commit 962e47b into quarkusio:main Sep 30, 2025
22 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.29 - main milestone Sep 30, 2025
@gsmet gsmet modified the milestones: 3.29 - main, 3.28.2 Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DevUI MCP Tool Call from Cursor fails => Cannot invoke "java.util.Map.entrySet()" because "parameters" is null
4 participants