Add condition, hitCondition, and logMessage to the Breakpoint type
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 48/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Stale
- Domain
- api
Research direction
Start by locating the Breakpoint and SourceBreakpoint type definitions and compare their existing fields, using the linked Debug Adapter Protocol specification for context. Add the three optional fields proposed in the issue and verify that the protocol definition remains consistent with the documented request and response types.
Written by the indexing model from the issue text.
Description
Problem
The SourceBreakpoint type (request side) includes condition, hitCondition, and logMessage, but the Breakpoint type (response/event side) does not. This means when a debug adapter sends a breakpoint event, the client cannot determine whether the breakpoint is a logpoint or conditional breakpoint.
Consequence: Logpoints sent via breakpoint events render as regular breakpoints in the UI — losing the logpoint icon and confusing users. Conditional breakpoints similarly lose their condition decoration.
Proposed Change
Add three optional fields to the Breakpoint type, mirroring SourceBreakpoint:
interface Breakpoint {
// ... existing fields ...
condition?: string;
hitCondition?: string;
logMessage?: string;
}
Rationale
Symmetry: line and column already round-trip between SourceBreakpoint and Breakpoint. condition, hitCondition, and logMessage should too.
Backward compatible: All fields are optional — existing adapters and clients are unaffected.
Enables correct rendering: Clients can use logMessage presence to render logpoints (◇) vs breakpoints (●), and display conditions in hover/detail views.
Unblocks AI usecase: In developing AI capabilities for debugging, logpoints and conditional breakpoints are particularly requested features as agents can easily leverage advanced breakpointing techniques.
- Dominant language
- HTML
- Stars
- 1.8k
- Forks
- 173
- Avg merge
- 7d 7h
- Merged PRs (30d)
- 2
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from microsoft/debug-adapter-protocol
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
microsoft/debug-adapter-protocol#633 · 1 comment ·
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
microsoft/debug-adapter-protocol#599 · 8 reactions ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
microsoft/debug-adapter-protocol#588 · 1 comment ·
All issues in microsoft/debug-adapter-protocol
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
external-issue to-triage
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
googleapis/google-cloud-swift#1151 ·
-
external
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
langchain-ai/langgraph#9074 · 1 comment ·
-
documentation specification
Difficulty 1/5 Under an hour Newbie friendliness 90/100
openai/openai-openapi#584 ·