Passing Active Editor Info #1015
Merged
Conversation
This file contains 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
* Moving auth create flow to utils (#926) Co-authored-by: amitjoshi <[email protected]> * Making auth provider a common module flow (#928) * refactor: Update QuickPickProvider to handle tab input changes (#930) The code changes in the QuickPickProvider class handle tab input changes by checking if the active tab is an instance of vscode.TabInputCustom or vscode.TabInputText. If it is, the QuickPickProvider updates the quick pick items based on the file path of the tab input. Note: This commit message follows the convention used in recent repository commits. * Updates to Artemis service & telemetry endpoints for gov clouds (#906) * Updates to artemis for gov clouds * change * changes --------- Co-authored-by: Shivika Gupta <[email protected]> --------- Co-authored-by: amitjoshi438 <[email protected]> Co-authored-by: amitjoshi <[email protected]> Co-authored-by: Ritik Ramuka <[email protected]> Co-authored-by: gshivi <[email protected]> Co-authored-by: Shivika Gupta <[email protected]>
* chore: Update package.json with extension dependencies and enabled API proposals for chat participants * feat: Add PowerPagesChatParticipant for chat functionality * feat: Add logic to handle chat requests in PowerPagesChatParticipant * feat: Removed logs * TODO * formatting change * Remove unused extension dependency * feat: Add handler for chat requests in PowerPagesChatParticipant --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]> Co-authored-by: Nidhi Tyagi 🌟🐇🌴❄️ <[email protected]>
* chore: Update package.json with extension dependencies and enabled API proposals for chat participants * feat: Add PowerPagesChatParticipant for chat functionality * feat: Add logic to handle chat requests in PowerPagesChatParticipant * feat: Removed logs * TODO * feat: Add logic to handle chat requests in PowerPagesChatParticipant(correct response format) * chore: Remove unused code and update PowerPagesChatParticipant initialization * chore: Update PowerPagesChatParticipant initialization and pac integration * feat: Initialize organization details in PowerPagesChatParticipant * chore: Remove unnecessary code and update OrgChangeNotifier initialization * Fix lint warnings * feat: Refactor PowerPagesChatParticipant class The code changes refactor the PowerPagesChatParticipant class in the `PowerPagesChatParticipant.ts` file. The changes include: - Fixing a typo in the `instance` property declaration - Updating the constructor parameters to have consistent spacing - Adding a comment to handle chat requests - Removing a console.log statement - Updating the `intializeOrgDetails` method to have consistent spacing - Updating the `intializeOrgDetails` method to use destructuring assignment - Updating the `intializeOrgDetails` method to update the `orgDetails` in the global state - Removing unused code Co-authored-by: amitjoshi <[email protected]> * Added response for error scenarios * Added & removed comments * refactor: Update PowerPagesCopilot name in package.json The code changes in the package.json file update the "name" property for the "powerpages" module to "Power Pages Copilot". This change reflects the updated name for the module. Co-authored-by: amitjoshi <[email protected]> * Code refactoring for utils and constants * Enhanced pac auth handling * refactor: Update PowerPagesChatParticipantConstants The code changes in the `PowerPagesChatParticipantConstants.ts` file refactor the constants used in the PowerPages chat participant. The changes include: - Cleaning up the code formatting Co-authored-by: amitjoshi <[email protected]> --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]> Co-authored-by: Nidhi Tyagi 🌟🐇🌴❄️ <[email protected]>
* chore: Update package.json with extension dependencies and enabled API proposals for chat participants * feat: Add PowerPagesChatParticipant for chat functionality * feat: Add logic to handle chat requests in PowerPagesChatParticipant * feat: Removed logs * TODO * feat: Add logic to handle chat requests in PowerPagesChatParticipant(correct response format) * feat: Add getActiveEditorContent function for retrieving active editor data The code changes add a new function called getActiveEditorContent to the Utils module. This function retrieves the content and parameters of the active editor in Visual Studio Code. It is used in the PowerPagesCopilot module to get the active file content and its associated parameters. Co-authored-by: amitjoshi <[email protected]> * chore: Remove unused code and update PowerPagesChatParticipant initialization * chore: Update PowerPagesChatParticipant initialization and pac integration * feat: Initialize organization details in PowerPagesChatParticipant * @powerpages aib endpoint handling * feat: Add getActiveEditorContent function to retrieve active editor data The code changes introduce a new function called getActiveEditorContent in the Utils module. This function allows retrieving the content and parameters of the active editor in Visual Studio Code. It is utilized in the PowerPagesCopilot module to obtain the active file content and its associated parameters. Co-authored-by: amitjoshi <[email protected]> * Multiline response handling * Todo comment for auth create exp * feat: Handle Copilot Unavailable scenario with response to user * Update in package.json * chore: Remove extension dependency * Full name for @powerpages * refactor: Refactor PowerPagesChatParticipant class - Removing unused code * chat api insider version * refactor: Refactor PowerPagesChatParticipant import paths * Update package.json version to 1.0.1-dev and other refactoring * refactor: Remove unused code in PowerPagesChatParticipant class * refactor: Moved Dataverse details fetch to utils * refactor: Added entity check to utils * refactor: Update localization strings and package version * chore: Remove @types/vscode dependency and update package-lock.json and package.json * chore: Update @types/vscode dependency to version 1.89.0 * chore: Remove @types/vscode dependency and update package-lock.json and package.json --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]>
* feat: Add environmentID field to OrgDetails interface The code changes add an optional `environmentID` field to the `OrgDetails` interface in the `PowerPagesChatParticipantTypes.ts` file. This field represents the ID of the environment associated with the organization. It is used in the `getEndpoint` function in the `PowerPagesChatParticipantUtils.ts` file to retrieve the intelligence endpoint and geo name for the specified organization and environment. Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]> * feat: Update PowerPagesChatParticipantUtils to return componentInfo and entityName The code changes in PowerPagesChatParticipantUtils modify the getComponentInfo function to return an object with two properties: componentInfo and entityName. This change allows for better handling of component information and entity details for the active file. The updated function is used in PowerPagesChatParticipant to retrieve the necessary data for sending API requests. Co-authored-by: amitjoshi <[email protected]> * feat: Modify getComponentInfo to return componentInfo and entityName The code changes in PowerPagesChatParticipantUtils modify the getComponentInfo function to return an object with two properties: componentInfo and entityName. This change improves the handling of component information and entity details for the active file. The updated function is used in PowerPagesChatParticipant to retrieve the necessary data for sending API requests. --------- Co-authored-by: amitjoshi <[email protected]>
* feat: Add telemetry event names for GitHub Power Pages agent * chore: Update telemetry event names for GitHub Power Pages agent --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]>
The code changes in the `getActiveEditorContent` function of the `Utils.ts` file add support for retrieving the content and parameters of the active editor in Visual Studio Code. This function now handles selected code, visible code, and full file content scenarios. It also updates the `activeFileData` object with the appropriate content and line range information. This change improves the functionality of the `getActiveEditorContent` function and enhances the overall user experience. Co-authored-by: amitjoshi <[email protected]>
…, visible code, and full file content scenario Co-authored-by: amitjoshi <[email protected]>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
gshivi
reviewed
Aug 13, 2024
gshivi
approved these changes
Aug 13, 2024
tyaginidhi
approved these changes
Aug 14, 2024
src/common/chat-participants/powerpages/PowerPagesChatParticipant.ts
Outdated
Show resolved
Hide resolved
src/common/chat-participants/powerpages/PowerPagesChatParticipantUtils.ts
Outdated
Show resolved
Hide resolved
… of https://github.com/microsoft/powerplatform-vscode into user/amitjoshi/powerpagesGithubCopilotActiveEditorInfo
amitjoshi438
deleted the
user/amitjoshi/powerpagesGithubCopilotActiveEditorInfo
branch
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.
This pull request includes various updates to enhance functionality, improve code structure, and support new features. The changes focus on adding new API proposals, modifying constructors and methods for better context handling, and integrating new utility functions.
Enhancements and New Features:
package.json: Added support for new API proposals includingchatParticipant,chatVariableResolver, andlanguageModels.PowerPagesChatParticipant.ts: Introduced a new utility functioncreateAndReferenceLocationfor referencing code locations in chat responses. [1] [2]Code Structure Improvements:
OrgChangeNotifier.ts: Updated the constructor andcreateOrgChangeNotifierInstancemethod to includevscode.ExtensionContextfor better context management.PowerPagesCopilot.ts: Refactored to include active file content and parameters in API requests, improving the relevance of responses. [1] [2]Utility and Configuration Updates:
IntelligenceApiService.ts: Added support for testing with localhost by configuring an HTTPS agent and setting appropriate headers.model.ts: ExpandedIActiveFileDatainterface to includeactiveFileUri,startLine, andendLinefor detailed file context.These changes collectively enhance the functionality and maintainability of the codebase, ensuring better integration and context handling across different components.