Feature: tools realignment #297
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added SVG icon for generic tool and PNG icons for PRT and PD tools. - Implemented `registerToolsCommands` function to register commands for opening the Dataverse REST Builder (DRB), launching PRT, CMT, and PD tools. - Created `ToolsHelper` class to manage tool interactions, including opening the DRB and launching tools based on user selection. - Included error handling for command execution failures.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR implements a tools realignment feature that refactors how tools are managed in the VS Code extension. The changes consolidate tool functionality into a unified helper system, update interface property names for better consistency, and reorganize command registration.
- Refactored tool interfaces to use cleaner property names (
name,shortName,author) and added optional icon support - Consolidated tool functionality by replacing
DRBHelperwith a comprehensiveToolsHelperclass - Reorganized command registration by moving tool-specific commands to a dedicated registration module
Reviewed Changes
Copilot reviewed 10 out of 23 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/utils/Interfaces.ts | Updated IToolDetails interface with cleaner property names and added optional icon field |
| src/utils/ErrorMessages.ts | Added new error message for common tool operations |
| src/tools/toolsDataProvider.ts | Updated to use new interface properties and simplified icon path logic |
| src/tools/tools.json | Restructured tool definitions with new property names and explicit icon specifications |
| src/helpers/toolsHelper.ts | New comprehensive helper class that consolidates all tool operations |
| src/helpers/drbHelper.ts | Removed file - functionality moved to ToolsHelper |
| src/extension.ts | Added registration of new tools commands module |
| src/commands/registerToolsCommands.ts | New module for registering tool-specific commands |
| src/commands/registerCommands.ts | Removed tool commands and updated to use new ToolsHelper |
| package.json | Added "DVDT" keyword |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.