Skip to content

Re-added support for 'Jump to Cursor' command#1227

Closed
basit-s wants to merge 1 commit into
Marus:masterfrom
basit-s:feature/bsheikh-jump-to-cursor
Closed

Re-added support for 'Jump to Cursor' command#1227
basit-s wants to merge 1 commit into
Marus:masterfrom
basit-s:feature/bsheikh-jump-to-cursor

Conversation

@basit-s
Copy link
Copy Markdown

@basit-s basit-s commented May 15, 2026

This PR restores Jump to Cursor behavior and aligns the adapter with expected DAP request flow, see Issue #1226

I found the original feature implementation for 'Jump to Cursor' in PR #417 and Issue #418, but it seems to have been broken during a refactor in commit 1ff878c.

  • 6ac5c31, “Jump to cursor, PR 417”, added Jump to Cursor support
  • 1ff878c, “hardwareBreakpoints to force HW only bpts”, removed the low-level MI goto helper and changed gotoTargetsRequest to create a temporary breakpoint instead

Current master still advertises supportsGotoTargetsRequest, and the changelog still says Jump to Cursor is supported, but I cannot find a gotoRequest implementation in the adapter anymore, and the MI backend no longer has the exec-jump path that PR #417 added. So this looks like a partial DAP implementation now: gotoTargetsRequest exists, but the actual goto step appears to be missing. When running the code, Jump to Cursor returns the error message shown in the image below.

@basit-s
Copy link
Copy Markdown
Author

basit-s commented May 19, 2026

Marking this as closed - it seems like there was some issue with my setup as I can't reproduce the issue anymore. I tried it on my primary machine which had the issue as well as two other machines.

@basit-s basit-s closed this May 19, 2026
@haneefdm
Copy link
Copy Markdown
Collaborator

Thank you for the follow-up. But if you find it again, we can reopen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants