Skip to content

Conversation

@rodinaarssen
Copy link
Member

No description provided.

@rodinaarssen rodinaarssen marked this pull request as ready for review November 25, 2025 12:59
Copy link
Member

@DavyLandman DavyLandman left a comment

Choose a reason for hiding this comment

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

Some small fixes, but looks good.

@sonarqubecloud
Copy link

@DavyLandman DavyLandman merged commit 4511da3 into main Nov 25, 2025
14 checks passed
@rodinaarssen rodinaarssen deleted the remote-ideservices branch November 25, 2025 15:52
@urbanfly
Copy link
Contributor

@rodinaarssen @DavyLandman What is this PR? "Remote IDE services" sounds interesting, but I can't tell if it really is interesting 😝

@DavyLandman
Copy link
Member

@rodinaarssen @DavyLandman What is this PR? "Remote IDE services" sounds interesting, but I can't tell if it really is interesting 😝

This is part of a large project where we want to disconnect the version of the Rascasl REPL from the rascal version in the VS Code extension. Right now you always get the packaged rascal version. But we're working towards a setup where the extension respects your pom.xml. Read more about it here: usethesource/rascal#2171

But to enable that we've had to disconnect some hard dependencies in the REPL. One of the last ones (only VFS is remaining) was this IDE Service, that now instead of being a java connection between the REPL & rascal-lsp server, it's now a jsonrcp connection.

I think in summary: not so interesting, mainly internal engineering to prepare for a future where we can do more interesting stuff such as upgrading the vs code extension without forcing the users to also upgrade their rascal version.

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.

4 participants