-
Notifications
You must be signed in to change notification settings - Fork 0
feat: Enhance C# Package Autocomplete with SDK and Property directives support #1
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
…s support - Updated package.json to reflect new version and expanded description. - Introduced new interfaces for SDK and Property information. - Refactored completion provider to handle #:sdk and #:property directives. - Added common SDKs and MSBuild properties for autocompletion. - Improved hover provider to display detailed information for SDKs and properties. - Updated activation events to trigger on new directive formats. - Enhanced documentation and examples for better user guidance.
|
Caution Review failedThe pull request is closed. WalkthroughAdds directive-based functionality to the VS Code extension, expanding from package-only to support for #:package, #:sdk, and #:property completions and hovers. Updates README to document new directives and workflows. Adjusts package metadata and tooling versions. Activation now registers new directive-aware completion and hover providers with caching and timeout handling. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor U as User
participant VS as VS Code
participant EP as Extension (Completion Provider)
participant NU as NuGet API
note over U,VS: Typing in C# file-based app
U->>VS: Type "#:..." or continue after directive
VS->>EP: Request completions (position, line)
EP->>EP: Parse directive (#:package|#:sdk|#:property)
alt #:package
EP->>EP: If after package name: suggest versions
EP->>NU: Search packages (with cache, 5s timeout)
NU-->>EP: Package list
EP-->>VS: Completion items (sorted by downloads)
else #:sdk
EP->>EP: Lookup COMMON_SDKS
EP-->>VS: SDK completion items (static docs)
else #:property
EP->>EP: If name context → COMMON_PROPERTIES
EP->>EP: If value context → possible values/default
EP-->>VS: Property name/value completion items
end
VS-->>U: Show suggestions
sequenceDiagram
autonumber
actor U as User
participant VS as VS Code
participant HP as Extension (Hover Provider)
participant NU as NuGet API
U->>VS: Hover over directive/value
VS->>HP: Hover request (range, text)
HP->>HP: Identify directive context
alt #:package
HP->>NU: Fetch package metadata (5s timeout)
NU-->>HP: Description, version, downloads, link
HP-->>VS: Hover markdown with package info
else #:sdk
HP->>HP: findSdkInfo()
HP-->>VS: Hover markdown with SDK metadata
else #:property
HP->>HP: findPropertyInfo()
HP-->>VS: Hover markdown with property info
end
VS-->>U: Show hover
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. 📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
✨ Finishing Touches
🧪 Generate 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
feat: Enhance C# Package Autocomplete with SDK and Property directives support
Summary by CodeRabbit
New Features
Documentation
Chores