Skip to content

feat(config): accept csharp settings in initializationOptions - #428

Merged
razzmatazz merged 2 commits into
mainfrom
feat/initialization-options
Oct 23, 2026
Merged

razzmatazz merged 2 commits into
mainfrom
feat/initialization-options

Conversation

@razzmatazz

@razzmatazz razzmatazz commented Oct 23, 2026 •

Copy link
Copy Markdown
Owner

Accept csharp settings in initializationOptions, in the same { "csharp": { ... } } shape as workspace/didChangeConfiguration, for clients that configure a server per connection rather than through a global workspace configuration (e.g. eglot).

  • Applied before the capabilities are computed, so razorSupport shows up in the initialize result.
  • Precedence: command line, initializationOptions, extended client capabilities, then workspace configuration.
  • ConfigurationChange is now posted before ClientCapabilityChange, which otherwise dropped the capability-derived useMetadataUris.
  • $/csharp/debugInfo reports the effective configuration; tests check both debug info and the settings' effects.

🤖 Generated with Claude Code

razzmatazz and others added 2 commits October 23, 2026 09:59
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@razzmatazz
razzmatazz merged commit 19ec0e2 into main Oct 23, 2026
2 of 3 checks passed
@razzmatazz
razzmatazz deleted the feat/initialization-options branch October 23, 2026 07:02
razzmatazz added a commit to razzmatazz/eglot-csharp that referenced this pull request Oct 23, 2026
…guration alone

eglot-csharp installed its own function as the global default of
eglot-workspace-configuration at load time, which discarded a user's
plist there, was dropped by a later setq, and stacked on every reload.
An eglot-initialization-options method on eglot-csharp--server touches
no global state, and a :csharp section the user sets still wins, as
csharp-ls pulls it after initialize.

csharp-ls reads initializationOptions from 0.29.0 on
(razzmatazz/csharp-language-server#428); older versions ignore them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

1 participant