-
Notifications
You must be signed in to change notification settings - Fork 53
Add experimentalFeatures to initialization options #287
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Introduce a feature flagging system for opt-in experimental features. Clients can enable features via initializationOptions.experimentalFeatures with granular per-feature control or an 'all' flag to enable everything. First experimental feature: missingInputsQuickfix (for upcoming code actions)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR introduces a feature flagging system for experimental features in the language server. Clients can enable features via initializationOptions.experimentalFeatures with granular per-feature control or an all flag to enable everything. The first experimental feature defined is missingInputsQuickfix for an upcoming code actions feature.
Key changes:
- Added
ExperimentalFeaturesinterface to define feature flags with precedence rules - Implemented
FeatureFlagsclass to check and list enabled experimental features - Integrated feature flags into the connection initialization with informational logging
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| languageserver/src/initializationOptions.ts | Adds ExperimentalFeatures interface with all and missingInputsQuickfix flags, and includes it in InitializationOptions |
| languageserver/src/features.ts | Implements FeatureFlags class to manage feature flag resolution with proper precedence handling |
| languageserver/src/features.test.ts | Adds comprehensive test coverage for feature flag resolution logic and precedence rules |
| languageserver/src/connection.ts | Integrates feature flags initialization and logs enabled experimental features on server startup |
| languageserver/README.md | Documents the experimental features system with usage examples and available features table |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Introduce a feature flagging system for opt-in experimental features. Clients can enable features via initializationOptions.experimentalFeatures with granular per-feature control or an 'all' flag to enable everything.
This includes the first experimental feature: missingInputsQuickfix (for upcoming code actions/first quick fix).