Skip to content

Conversation

@ShawkyZ
Copy link
Contributor

@ShawkyZ ShawkyZ commented Nov 5, 2025

Description

Provide description of this PR and changes, if linked Jira ticket doesn't cover it in full.

Checklist

Screenshots / GIFs

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

@snyk-io
Copy link

snyk-io bot commented Nov 5, 2025

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

Status Scanner 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.

@ShawkyZ ShawkyZ changed the title chore: wip move sat to ls refactor: wip move sat to ls Nov 5, 2025
- Removed the user parameter from the ConfigurationWatcher constructor as it is no longer needed.
- Updated the LanguageServer initialization to pass the vscodeContext directly, ensuring proper context handling.
- Simplified the return statement in the provideMcpServerDefinitions method to directly return the McpStdioServerDefinition.

These changes streamline the code and improve the clarity of the configuration handling.
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