Skip to content

feat(forge): add Solar LSP command - #15873

Closed
decofe wants to merge 1 commit into
foundry-rs:masterfrom
decofe:centaur/integrate-solar-lsp-1784841400
Closed

feat(forge): add Solar LSP command#15873
decofe wants to merge 1 commit into
foundry-rs:masterfrom
decofe:centaur/integrate-solar-lsp-1784841400

Conversation

@decofe

@decofe decofe commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Motivation

Expose Solar’s language server through forge lsp, so editors can use the same Solar version bundled with Foundry. This supersedes the earlier attempts in #11619, #11277, and #11187 now that solar-lsp is released on crates.io.

Closes #11448.

Changes

  • Add the forge lsp subcommand.
  • Run Solar 0.2.0’s language server over stdin/stdout.
  • Pin solar-lsp alongside Foundry’s existing Solar dependencies.

Testing

  • cargo +nightly fmt --all -- --check
  • cargo metadata --locked --no-deps --format-version 1
  • git diff --check

A full cargo check -p forge was attempted, but the sandbox could not authenticate to the workspace’s patched foundry-rs/optimism Git dependency.

AI assistance

This PR was implemented and drafted with Codex.

Prompted by: @DaniPopes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

feat(forge): Language Server Protocol (LSP)

2 participants