apache / apache/daffodil-vscode

Replace instances of debug.activeDebugSession with isDFDLDebugSessionActive()

Open
#1,423 0 comments 1 reaction 0 assignees View on GitHub
bug code completion code quality data editor debugger typescript
Dominant language
TypeScript
Stars
18
Forks
33
Avg merge
3d 6h
Merged PRs (30d)
7

Description

## Description

We're currently evaluating debug.activeDebugSession as a boolean in a few places. We should investigate replacing it with the isDFDLDebugSessionActive() function call. We currently aren't testing for the debug session type, so a non-dfdl debugging session could be opened and our logic would come back as true.

## Steps to Reproduce

Difficult to reproduce - would need to debug using another extension

## Expected Behavior

I'd expect the extension to recognize that even though we're in a debugging session, it's not a dfdl session.

## Actual Behavior

I'd expect the extension to treat all debug sessions like a dfdl session.

## Additional Notes and Screenshots (optional)

## Environment

N/A

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the extension for instances of debug.activeDebugSession and inspect the existing isDFDLDebugSessionActive() function. Check how each use behaves when another extension owns the active debug session, then verify that only DFDL sessions are treated as active and run the relevant available checks.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript, vscode
Domain
devtools
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.