breakpoint: `remove_by_id` is O(n) — index by id for large breakpoint sets

[Timi16]

File: src/debugger/breakpoint.rs — BreakpointManager::remove_by_id

The method does a full linear scan of all breakpoints to find one by id. While unlikely to matter today, the VS Code DAP adapter may register many source breakpoints at once. A secondary HashMap<id -> function> would make this O(1).

Labels: performance, good-first-issue

[Tola-byte]

@Tola-byte has applied to work on this issue as part of the Stellar Wave Program's 3rd wave.

I would like to work on this.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @Tola-byte to this issue.

[ObaniIsola]

@ObaniIsola has applied to work on this issue as part of the Stellar Wave Program's 3rd wave.

Hi, I'm a seasoned software developer, and I would love to work on this issue.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @ObaniIsola to this issue.

[drips-wave]

Congratulations, @ObaniIsola! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on March 30, 2026.

🧑‍💻 @ObaniIsola: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.

Warning

When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.

🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.

🌊 Happy Wave 🌊