Skip to content

Show infoset actions throughout active debug sessions#1705

Open
999axel999 wants to merge 2 commits into
apache:mainfrom
999axel999:zerocracy/1703-1779465112188
Open

Show infoset actions throughout active debug sessions#1705
999axel999 wants to merge 2 commits into
apache:mainfrom
999axel999:zerocracy/1703-1779465112188

Conversation

@999axel999

@999axel999 999axel999 commented May 22, 2026

Copy link
Copy Markdown

Closes #1703

Description

  • Relax the editor title menu conditions for the infoset actions.
  • Show infoset.display and infoset.diff whenever a debug session is active, regardless of the focused editor language.
  • Keep the change scoped to the existing inDebugMode context used by VS Code.

Wiki

  • I have determined that no documentation updates are needed for these changes
  • I have added the following documentation for these changes

Review Instructions including Screenshots

  • node -e "const p=require('./package.json'); const items=p.contributes.menus['editor/title'].filter(x=>x.command==='infoset.display'||x.command==='infoset.diff'); if(items.some(x=>x.when!=='inDebugMode')) process.exit(1);"

Prepared with local automation assistance and reviewed before submission.

@999axel999 999axel999 changed the title Show infoset actions whenever debugging is active Show infoset actions throughout active debug sessions May 22, 2026
@999axel999 999axel999 marked this pull request as ready for review May 22, 2026 15:54
@scholarsmate scholarsmate requested a review from Copilot May 29, 2026 14:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the extension’s VS Code menu contributions so the infoset-related editor title actions are visible throughout an active debug session, addressing issue #1703 where those actions were missing when focus wasn’t on a DFDL editor.

Changes:

  • Relax infoset.display and infoset.diff editor/title menu visibility conditions from inDebugMode && editorLangId == 'dfdl' to inDebugMode.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Comment on lines 223 to 230
"command": "infoset.display",
"when": "inDebugMode && editorLangId == 'dfdl'",
"when": "inDebugMode",
"group": "navigation@2"
},
{
"command": "infoset.diff",
"when": "inDebugMode && editorLangId == 'dfdl'",
"when": "inDebugMode",
"group": "navigation@3"
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.

Unable to open infoset or infoset-diff-view from Daffodil helper tab

2 participants