Enable visualizing constructs using code lenses in multi-package directory setups#986
Conversation
…s are always available, including notify and refresh commands for the Ballerina extension
…proper project type before creating test profiles
…packages using tree view
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull Request Overview
This PR enables visualizing constructs using code lenses in multi-package directory setups. The changes focus on improving cross-extension communication between the Ballerina and BI extensions, particularly for workspace scenarios with multiple packages.
- Refactored command registration to ensure core commands are always available for cross-extension calls
- Updated project root initialization to include project info for proper workspace handling
- Added defensive checks and error handling for cross-extension command execution
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
workspaces/bi/bi-extension/src/project-explorer/project-explorer-provider.ts |
Added visibility check before revealing items in tree view |
workspaces/bi/bi-extension/src/project-explorer/activate.ts |
Refactored to register core commands unconditionally for cross-extension availability |
workspaces/ballerina/ballerina-visualizer/src/components/TopNavigationBar/index.tsx |
Added "welcome" view to breadcrumb skip list |
workspaces/ballerina/ballerina-extension/src/views/visualizer/activate.ts |
Enhanced workspace handling with project info initialization and document URI tracking |
workspaces/ballerina/ballerina-extension/src/views/ai-panel/activate.ts |
Removed dead code comment |
workspaces/ballerina/ballerina-extension/src/stateMachine.ts |
Updated to pass project info with root updates, added BI extension activation check |
workspaces/ballerina/ballerina-extension/src/features/test-explorer/activator.ts |
Added guard logic to conditionally activate test explorer (contains logic bug) |
workspaces/ballerina/ballerina-extension/src/features/bi/activator.ts |
Added guard clause to prevent errors when projectPath is undefined |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Purpose
$title
Addresses: wso2/product-ballerina-integrator#1977