Skip to content

Add a tool to get the active cursor location #116

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
May 8, 2025
Merged

Conversation

jakemac53
Copy link
Contributor

@jakemac53 jakemac53 commented May 7, 2025

Similar to the selected widget tool this should help the AI get some context as to what the user is talking about.

Just directly forwards the last event received from DTD.

@@ -220,8 +225,10 @@ base mixin DartToolingDaemonSupport
await updateActiveVmServices();
case 'debugSessionStopped':
await activeVmServices
.remove((e.data['debugSession'] as DebugSession).id)
.remove(e.data['debugSessionId'] as String)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an unrelated bug fix that I snuck in here, I can separate it out if we want

Copy link

github-actions bot commented May 7, 2025

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

Copy link
Contributor

@kenzieschmoll kenzieschmoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add to list of tools in the readme

Copy link

auto-submit bot commented May 8, 2025

autosubmit label was removed for dart-lang/ai/116, because - The status or check suite build (stable) has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit bot removed the autosubmit label May 8, 2025
@jakemac53 jakemac53 merged commit 80a0e3f into main May 8, 2025
13 checks passed
@jakemac53 jakemac53 deleted the active-locatoin branch May 8, 2025 15:28
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.

2 participants