Skip to content

[POM-leading for DSLs] Routing to dedicated remote language servers#1069

Merged
toinehartman merged 112 commits into
feature/1010-pom-leading-for-dsls/finalfrom
feature/1010-pom-leading-for-dsls/remote-routing
May 27, 2026
Merged

[POM-leading for DSLs] Routing to dedicated remote language servers#1069
toinehartman merged 112 commits into
feature/1010-pom-leading-for-dsls/finalfrom
feature/1010-pom-leading-for-dsls/remote-routing

Conversation

@toinehartman
Copy link
Copy Markdown
Member

@toinehartman toinehartman commented Apr 28, 2026

This PR is the first of a series. gradually implementing POM-leading behavior in DSLs/the parametric server.
This PR introduces a routing language server, which delegates incoming LSP requests to the appropriate parametric language server, dedicated to a single language. A dedicated server is started upon initial language registration.

Closes #1076

@toinehartman toinehartman self-assigned this Apr 28, 2026
@toinehartman toinehartman added the enhancement New feature or request label Apr 28, 2026
@toinehartman toinehartman changed the base branch from main to feature/1010-pom-leading-for-dsls/final April 28, 2026 16:23
@toinehartman toinehartman force-pushed the feature/1010-pom-leading-for-dsls/remote-routing branch from b82310c to 756576e Compare May 21, 2026 09:48
Copy link
Copy Markdown
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.

Yes, this looks good, my only consern is around the log forwarding code.

Comment thread rascal-lsp/src/main/java/org/rascalmpl/vscode/lsp/BaseLanguageServer.java Outdated
@toinehartman toinehartman force-pushed the feature/1010-pom-leading-for-dsls/remote-routing branch from 548183e to 766bbe5 Compare May 21, 2026 14:00
@toinehartman toinehartman requested a review from DavyLandman May 26, 2026 09:26
@toinehartman toinehartman merged commit 82b3d8c into feature/1010-pom-leading-for-dsls/final May 27, 2026
12 checks passed
@sonarqubecloud
Copy link
Copy Markdown

toinehartman added a commit that referenced this pull request May 27, 2026
…e/feature/1010-pom-leading-for-dsls/remote-routing
@toinehartman
Copy link
Copy Markdown
Member Author

toinehartman commented May 27, 2026

I squashed this PR initially, while it should have been a regular merged. Manually reverted/re-did the merge in 900d818.

toinehartman added a commit that referenced this pull request May 27, 2026
…e/feature/1010-pom-leading-for-dsls/remote-routing

[POM-leading for DSLs] Routing to dedicated remote language servers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Parametric routing layer with dedicated servers

2 participants