Skip to content

Update lsp.json in Boilerplate (#12322)#12329

Merged
msynk merged 1 commit into
bitfoundation:developfrom
zahra-yousefi:12322-update-lsp.json-boilerplate
May 16, 2026
Merged

Update lsp.json in Boilerplate (#12322)#12329
msynk merged 1 commit into
bitfoundation:developfrom
zahra-yousefi:12322-update-lsp.json-boilerplate

Conversation

@zahra-yousefi
Copy link
Copy Markdown
Member

@zahra-yousefi zahra-yousefi commented May 15, 2026

closes #12322

Summary by CodeRabbit

  • Chores
    • Simplified language server configuration for C# development. The development environment setup has been streamlined to use a single C# server with updated tooling, improving configuration clarity and reducing complexity.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: e1e10961-0a02-4b10-8d16-d2e21eafdf05

📥 Commits

Reviewing files that changed from the base of the PR and between fa5ca7c and 59a4e27.

📒 Files selected for processing (1)
  • src/Templates/Boilerplate/Bit.Boilerplate/.github/lsp.json

Walkthrough

This PR updates the Bit Boilerplate template's .github/lsp.json configuration file to align with the latest Roslyn Language Server Protocol schema. The change replaces a multi-server setup with a simplified single C# server configuration using the Roslyn LSP implementation and reduces file type mappings to C# files only.

Changes

LSP Configuration Update

Layer / File(s) Summary
LSP server configuration and command setup
src/Templates/Boilerplate/Bit.Boilerplate/.github/lsp.json
Replace multi-server setup (C#, TypeScript, CSS) with single C# server using dotnet dnx roslyn-language-server invocation and simplified .cs file extension mapping, aligning with Roslyn's authoritative LSP schema.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A config update, clean and bright,
Roslyn's schema now shines right,
One C# server, lean and true,
No more clutter to review!
The boilerplate hops forward true. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Linked Issues check ❓ Inconclusive The PR updates lsp.json to simplify the LSP server configuration by removing multi-server setup and standardizing to Roslyn schema, addressing issue #12322 requirements. Verify the simplified configuration fully aligns with Roslyn's latest .github/lsp.json schema and includes all required fields to ensure compliance with the stated objectives.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly references the updated file (lsp.json) and the target location (Boilerplate), clearly summarizing the primary change.
Out of Scope Changes check ✅ Passed All changes are confined to the lsp.json file as specified in issue #12322, with no extraneous modifications to other files or unrelated code.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@msynk msynk requested a review from yasmoradi May 15, 2026 10:09
@msynk msynk merged commit 8ed4bdb into bitfoundation:develop May 16, 2026
3 checks passed
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.

.github/lsp.json Should Be Updated to Match Latest Roslyn LSP Schema

3 participants