Skip to content

feat: add new connectivity & directory check commands [IDE-1557]#690

Merged
rrama merged 10 commits intomainfrom
feat/IDE-1557_connectivity-check-tool
Mar 23, 2026
Merged

feat: add new connectivity & directory check commands [IDE-1557]#690
rrama merged 10 commits intomainfrom
feat/IDE-1557_connectivity-check-tool

Conversation

@rrama
Copy link
Copy Markdown
Contributor

@rrama rrama commented Dec 2, 2025

Description

Also refactor test mocks to be non-global mocks so we ensure no leaks between tests.

Relies on LS PR: snyk/snyk-ls#1080

Checklist

Screenshots / GIFs

Visuals that may help the reviewer. Please add screenshots for any UI change. GIFs are most welcome!

Refactor test mocks to be non-global mocks so we ensure no leaks between tests.
@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented Dec 2, 2025

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@rrama rrama marked this pull request as ready for review December 4, 2025 10:16
@rrama rrama requested review from a team as code owners December 4, 2025 10:16
@rrama rrama changed the title feat: add new connectivity check command [IDE-1557] feat: add new connectivity & directory check commands [IDE-1557] Jan 26, 2026
@rrama rrama added the ⚠️ DON'T MERGE Shouldn't be merged yet. label Jan 30, 2026
Comment thread src/snyk/common/commands/commandController.ts Outdated
Comment thread src/snyk/common/commands/commandController.ts Outdated
@snyk-pr-review-bot
Copy link
Copy Markdown

PR Reviewer Guide 🔍

🧪 PR contains tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Missing Command Palette Entry 🟠 [major]

The new commands snyk.collectConnectivityDiagnostics and snyk.collectDirectoryDiagnostics are registered in the commands contribution point but are not added to the menus.commandPalette section. This means the tools will not appear in the Command Palette (Cmd+Shift+P), making them undiscoverable and unreachable for regular users without manually defining shortcuts or using the debugger.

{
  "command": "snyk.collectConnectivityDiagnostics",
  "title": "Collect Connectivity Diagnostics",
  "category": "Snyk"
},
{
  "command": "snyk.collectDirectoryDiagnostics",
  "title": "Collect Directory Diagnostics",
  "category": "Snyk"
📚 Repository Context Analyzed

This review considered 37 relevant code sections from 15 files (average relevance: 0.77)

@rrama rrama removed the ⚠️ DON'T MERGE Shouldn't be merged yet. label Mar 23, 2026
@rrama rrama merged commit 774e9ce into main Mar 23, 2026
12 checks passed
@rrama rrama deleted the feat/IDE-1557_connectivity-check-tool branch March 23, 2026 18:23
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.

2 participants