[Power Pages][Actions Hub] Update command references for authentication profile #1132
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
priyanshu92
approved these changes
Feb 19, 2025
…TreeDataProvider test
…thub.com/microsoft/powerplatform-vscode into users/amitjoshi/newAuthProfileCmdNameFix
MrRishabhJain
approved these changes
Feb 19, 2025
priyanshu92
approved these changes
Feb 19, 2025
MrRishabhJain
approved these changes
Feb 19, 2025
sanxsth
approved these changes
Feb 20, 2025
sanxsth
added a commit
that referenced
this pull request
Feb 24, 2025
* Refactor PowerPagesChatParticipant and add CommandRegistry (#1050) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * [PowerPage][Copilot] Nl2Site and Nl2Page Service Integration for Site Create (#1053) * Refactor PowerPagesChatParticipantConstants and add NL2PAGE and NL2SITE constants * Refactor PowerPagesChatParticipantConstants and add new page types * Update constants and move telemetry to different file * Refactor and add getCommonHeaders function --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Corrected scope of PPAPI access token for preprod env (#1055) Co-authored-by: Ashwani Kumar <ashwanikumar@microsoft.com> * Hook for create site command with nl2page&site (#1057) * Hook for create site command with nl2page&site * Disable any type validation * Refactor PowerPagesChatParticipantConstants and CreateSiteCommand - Add NL2SITE_GENERATING_SITE constant for generating a new Power Pages site - Update progress message in CreateSiteCommand to use NL2SITE_GENERATING_SITE constant - Remove unnecessary markdown formatting in CreateSiteHelper error handling * Refactor localization files and update Power Pages chat participant utils * Refactor NL2SiteService and Nl2PageService to include additional telemetry logging --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> Co-authored-by: tyaginidhi <tyaginidhi@users.noreply.github.com> * Add environment list retrieval and refactor BAP endpoint logic (#1060) * Add environment list retrieval and refactor BAP endpoint logic * Add type annotation for environment list retrieval in Utils.ts * Update src/common/utilities/Utils.ts Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * Refactor BAP environment list URL to use a variable for API version and update imports in Utils.ts --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * [PowerPages][create-site] Preview and Edit Site Page and Command Registration (#1061) * Enhance CreateSiteCommand to include extension context and add ReadonlyFileSystemProvider for site page previews * Implement EditableFileSystemProvider for site page editing and update CreateSiteHelper to utilize it * Integrate CreateSiteCommand into CommandRegistry and update related components for site creation functionality * Disable copy functionality in EditableFileSystemProvider implementation * Remove ReadonlyFileSystemProvider implementation * Add telemetry constant for previewing site pages and refactor related components * Refactor CommandRegistry and add command registration utility for chat participants * Add constants for site creation parameters and refactor NL2SiteService to use them * Refactor CreateSiteCommand and CreateSiteHelper to use structured options and improve readability; add CreateSiteTypes for better type management * Add error telemetry constant for previewing site pages and handle errors in previewSitePagesContent function * Rename fileContentMap to _fileContentMap for consistency and clarity in EditableFileSystemProvider * Remove unused getUpdatedPageContent function from CreateSiteHelper to streamline code * Add ESLint disable comments for any type usage in CreateSiteHelper and CreateSiteTypes --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Bump cross-spawn (#1059) Bumps and [cross-spawn](https://github.com/moxystudio/node-cross-spawn). These dependencies needed to be updated together. Updates `cross-spawn` from 7.0.3 to 7.0.6 - [Changelog](https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.6) Updates `cross-spawn` from 6.0.5 to 7.0.6 - [Changelog](https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.6) --- updated-dependencies: - dependency-name: cross-spawn dependency-type: indirect - dependency-name: cross-spawn dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump CLI version from 1.35.1 to 1.37.4 (#1065) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * December Release Notes (#1066) * add release note * fix typo --------- Co-authored-by: Sandeep Satheesh <ssatheesh@microsoft.com> * Remove unused command from package.json (#1070) Removing the command `extension.createChatView` from package.json as it is registered but there is no handler associated with it. * Fix error type for concurrency handler (#1071) Fixes the error throws in case of bulk header rejection * Add code coverage output folder in gitignore (#1072) Adding the code coverage output folder `.nyc_ouput` folder in `.gitignore` so that it doesn't show up in `git diff` after running code coverage command. * Refactor CLI Acquisition Context (#1074) * Refactor CLI Acquisition Context * Fix import * Add tests * [PowerPages] [Create-Site] Implement multi-step site creation with enhanced input handling (#1063) * Enhance CreateSiteCommand to include extension context and add ReadonlyFileSystemProvider for site page previews * Implement EditableFileSystemProvider for site page editing and update CreateSiteHelper to utilize it * Integrate CreateSiteCommand into CommandRegistry and update related components for site creation functionality * Disable copy functionality in EditableFileSystemProvider implementation * Remove ReadonlyFileSystemProvider implementation * Add telemetry constant for previewing site pages and refactor related components * Refactor CommandRegistry and add command registration utility for chat participants * Add constants for site creation parameters and refactor NL2SiteService to use them * Refactor CreateSiteCommand and CreateSiteHelper to use structured options and improve readability; add CreateSiteTypes for better type management * Add error telemetry constant for previewing site pages and handle errors in previewSitePagesContent function * Rename fileContentMap to _fileContentMap for consistency and clarity in EditableFileSystemProvider * Implement multi-step input for site creation and register command for user inputs in CreateSiteHelper * Enhance copilot availability checks and update response structure in ArtemisService * Add site creation inputs and environment info interfaces; refactor CreateSiteCommand * Update src/common/chat-participants/powerpages/commands/create-site/CreateSiteTypes.ts Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * Update src/common/chat-participants/powerpages/commands/create-site/CreateSiteHelper.ts Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * Update src/common/chat-participants/powerpages/commands/create-site/CreateSiteHelper.ts Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * Add eslint directives to suppress no-explicit-any warnings in site page handling --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * Fix accessibility issue in Pages Copilot pane (#1076) Fix accessibility issue in `Send` button in Power Pages Copilot pane. Updated the `aria-label` to `Send` instead of `Match Case` * Fix casing of OrgUrl to orgUrl in site creation input handling (#1077) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Add test scripts in .vscode\tasks.json (#1078) To be able to easily run tests from VS Code, added test scripts in `tasks.json` * Remove redundant step from gulp file (#1079) The step `testWeb` is redundant as the step `testUnitTests` already includes the test pattern defined in `testWeb` step. Hence, removing it from gulp config. * Added site runtime preview code behind ECS Config (#1052) * Added site runtime preview code behind ECS Config * localization added and moved preview site code to separate file * fixed build failures and handled empty website recordID * Corrected website details ppapi service for runtime preview * merged main branch to runtimePreviewECS * correcting function call for runtime preview * corrected localised error * removed bug of no show command found * Launch Edge new tab instead of launching project --------- Co-authored-by: Ashwani Kumar <ashwanikumar@microsoft.com> Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * Hide preview site command from palette when feature is disabled (#1080) * Enhance feedback interaction by adding keyboard accessibility for thumbs up/down buttons (#1082) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Add conditional visibility for Current Active Users command in web virtual workspace and remove alt cmds (#1083) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Handle null language code for content snippets and update query parameters in portal schema (#1086) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Enhance accessibility by adding ARIA attributes and keyboard navigation support (#1085) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Enhance accessibility by adding ARIA attributes and making thinking message focusable (#1087) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Add sovereign cloud endpoints for PPAPI service (#1081) * Site Preview Enhancements: Clear cache and telemetry logging (#1088) * Enhance accessibility by improving focus styles and adding ARIA attributes in notification panel (#1089) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Actions Hub: Implement base classes to show tree structure (#1090) * Add models for Actions Hub * Basic wireup * Add models * ToDo * Remove foo environment * Remove error * Refactor * Add translations * Add tests * Add tree item for No sites found * Add translations * Added changes for orgGeo (#1093) * Refactor: Move actions-hub and preview-site folder under power-pages (#1092) * Refactor: Move actions-hub and preview-site folder under power-pages * Remove unwanted file * Refactor * Add PPAPI to known words * Disable actions hub by default * Web Extension: Remove dependency on website preview URL (#1091) * Web Extension: Remove dependency on website preview URL * Translations * Web Extension: Fix file opening for media files (#1094) Using `vscode.open` API to open the default file instead of `window.showTextDocument` because `showTextDocument` doesn't open binary files to it was failing to open non-text files. * Bump CLI version from 1.37.4 to 1.39.3 (#1095) * Fix environment icon for dark mode (#1097) * Fix environment icon for dark mode * Refactor * Fix test * fix screen reader issue in copilot chat (#1100) Co-authored-by: Sandeep Satheesh <ssatheesh@microsoft.com> * add focus ring (#1099) Co-authored-by: Sandeep Satheesh <ssatheesh@microsoft.com> * add release notes (#1101) Co-authored-by: Sandeep Satheesh <ssatheesh@microsoft.com> * Site Preview: Initialize after ECS fetch (#1102) * Site Preview: Initialize after ECS fetch * Set context after loading website URL * Add source attribute handling and telemetry for invalid attributes in web extension (#1105) * Add source attribute handling and telemetry for invalid attributes in web extension * Refactor source attribute handling in processDataAndCreateFile function --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Enhance Actions Hub with environment support and unit tests (#1098) * Enhance Actions Hub: Add support for environment retrieval and localization updates * Add unit tests for ActionsHubTreeDataProvider functionality --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Fix A11y bug (#1108) * Actions Hub: Add toggle based on ECS (#1107) * Add feature flag for Actions Hub * Refactor * Fix errors * Fix errors * Add unit tests * [PowerPages] [Site Preview] Simplify preview functionality (#1109) * Refactor telemetry handling by removing ITelemetry parameter from functions (#1110) * Refactor telemetry handling by removing ITelemetry parameter from sendTelemetryEvent and related functions * Refactor: Remove telemetry parameter from getEndpoint and related functions * Refactor: Remove telemetry parameter from authentication functions and related calls * Refactor: Remove telemetry parameter from getArtemisResponse call in activate function * Refactor: Remove telemetry parameter from various authentication and service calls --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Webpack: Ignore warnings (#1113) * Run desktop integration tests in CI (#1114) * Run desktop integration tests in CI * Run desktop test only on Windows * Enhance Actions Hub with environment retrieval and refresh functionality (#1111) * Enhance Actions Hub: Add support for environment retrieval and localization updates * Add unit tests for ActionsHubTreeDataProvider functionality * Add AuthManager class and AuthInfo interface for authentication handling * Enhance authentication handling: Add auth info extraction and update environment info retrieval in Actions Hub * Add refresh command and event handling for Actions Hub environment changes * Refactor ActionsHubTreeDataProvider: Manual refresh should trigger Pac * Remove AuthManager class and related authentication logic * Add error handling to refresh command in ActionsHubTreeDataProvider * RemoveDependencyOnOldTelemetryCluster * Refactor ActionsHubTreeDataProvider tests to include PacTerminal in initialization * Remove telemetry dependency from intelligence API endpoint retrieval * Refactor telemetry handling by removing ITelemetry parameter from sendTelemetryEvent and related functions * Refactor: Remove telemetry parameter from getEndpoint and related functions * Refactor: Remove telemetry parameter from authentication functions and related calls * Refactor telemetry event assertion in AuthenticationProvider tests * Refactor: Remove telemetry parameter from getArtemisResponse call in activate function * Refactor: Remove telemetry parameter from various authentication and service calls * Refactor: Rename AuthManager to PacAuthManager for consistency * Refactor: Use pacWrapper for activeAuth retrieval in ActionsHubTreeDataProvider * Add ACTIONS_HUB_REFRESH_FAILED constant and update error logging in ActionsHubTreeDataProvider * Remove unused AuthInfo interface from Constants.ts * Refactor ActionsHub initialization by removing authentication handling and simplifying the process * Add localization for expired environment message and update refresh title --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Remove old telemetry usage (#1112) * Remove old telemetry usage * Remove more usage * Remove more usage * Remove more usage * Remove more usage * Fix tests * Fix typo * Comment tests * Refactor Actions Hub and Preview Site initialization logic (#1115) * [PowerPages][Actions Hub] Toggle to Change Environment (#1119) * Add switch environment functionality to Actions Hub * Update icon for switch environment command in Actions Hub * Update translation for switch environment title to "Change Environment" * Add tests for switch environment command registration and error handling * Remove redundant error handling tests for switch environment and refresh commands * Refactor command names to include namespace for actions in Actions Hub * Update refresh command registration to include namespace in Actions Hub tests --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Bump serialize-javascript, gulp-mocha and mocha (#1120) Bumps [serialize-javascript](https://github.com/yahoo/serialize-javascript) to 6.0.2 and updates ancestor dependencies [serialize-javascript](https://github.com/yahoo/serialize-javascript), [gulp-mocha](https://github.com/sindresorhus/gulp-mocha) and [mocha](https://github.com/mochajs/mocha). These dependencies need to be updated together. Updates `serialize-javascript` from 5.0.1 to 6.0.2 - [Release notes](https://github.com/yahoo/serialize-javascript/releases) - [Commits](yahoo/serialize-javascript@v5.0.1...v6.0.2) Updates `gulp-mocha` from 8.0.0 to 10.0.1 - [Release notes](https://github.com/sindresorhus/gulp-mocha/releases) - [Commits](sindresorhus/gulp-mocha@v8.0.0...v10.0.1) Updates `mocha` from 9.2.2 to 11.1.0 - [Release notes](https://github.com/mochajs/mocha/releases) - [Changelog](https://github.com/mochajs/mocha/blob/main/CHANGELOG.md) - [Commits](mochajs/mocha@v9.2.2...v11.1.0) --- updated-dependencies: - dependency-name: serialize-javascript dependency-type: indirect - dependency-name: gulp-mocha dependency-type: direct:development - dependency-name: mocha dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [Power Pages][Actions Hub] Add "Show Environment Details" to Actions Hub (#1118) * [Power Pages][Actions Hub] Add "Show Environment Details" to Actions Hub * [PowerPages][Actions Hub] Update icon for switch environment command * [PowerPages][Actions Hub] Remove refresh command from Actions Hub navigation * Fix order * Update translation identifiers and fix newline in localization files * Refactor Actions Hub commands and extract command handlers into a new file * Add tests for switchEnvironment command * [PowerPages][ActionsHub] - Show Active and Inactive Sites (#1117) * show active and inactive sites in actions hub view * add integration tests for actions hub * [Power Pages][Actions Hub] Add NoSitesTreeItem to handle empty site groups * [Power Pages][Actions Hub] Update website details property names to PascalCase and change collapsible state to expanded * Refactor environment handling: rename method for clarity, improve error handling, and streamline environment selection process * Update localization strings for environment selection and session details * Refactor authentication and organization data structures; update TypeScript configuration and remove unused PacAuthManager * Add ArtemisContext for managing service responses; update ActionsHub and related components to utilize new context --------- Co-authored-by: Sandeep Satheesh <ssatheesh@microsoft.com> Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * [PowerPages][Actions Hub] Add Instance URL to Environment Details (#1122) * Add 'isCurrent' property and description to site tree items (#1123) * Update ArtemisService URLs to use production segment for GCC, high, mooncake, and DOD environments (#1121) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * [PowerPages][Actions Hub] Add commands to open active and inactive site in PP Studio (#1124) * [PowerPages][Actions Hub] Add commands to open active and inactive sites in Power Pages Studio * Refactor studio URL handling to use constants for endpoints * Add localization entries for "Current" and "Instance url" in JSON and XLF files * Refactor studio URL generation to include environment ID and add openInactiveSitesInStudio function * [Power Pages][Actions Hub] Add ability to preview active sites in VS Code (#1127) * Add preview command and siteInfo property to SiteTreeItem * Add unit tests for PreviewSite.isSiteRuntimePreviewEnabled method * Refactor PreviewSite initialization and loadSiteUrl method; update tests for loadSiteUrl functionality * Add tests for ActionsHubTreeDataProvider.getChildren method; mock website data for active and inactive sites * Add mocks for active and all websites in ActionsHubTreeDataProvider tests * Refactor site context values to distinguish between current and non-current active sites; update related tests * Add websiteId property to IWebsiteInfo interface and update related tree item implementations * Update walkthrough title to use h4 and enhance focus indicator for accessibility (#1125) Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * Add new command to create authentication profile and handle errors in actions hub (#1128) * Add new command to create authentication profile and handle errors in actions hub * Refactor tests for ActionsHubCommandHandlers and ActionsHubTreeDataProvider to improve argument validation and remove unnecessary assertions * Stub AuthInfo and OrgInfo getters in ActionsHubTreeDataProvider tests to improve test reliability * Remove redundant test for creating new auth profile in ActionsHubCommandHandlers * Fix async initialization in ActionsHub and streamline environment info handling in ActionsHubTreeDataProvider * Implement createAuthProfileExp function and add tests for authentication profile creation * Refactor createAuthProfileExp function import path and add new utility file for authentication profile creation * Add tests for ActionsHubTreeDataProvider to validate site retrieval and error handling * Remove redundant tests from ActionsHubTreeDataProvider for getChildren method * Refactor import path for createAuthProfileExp to new utility file in PacAuthUtil * Remove tests for PacAuthUtils as part of code cleanup * Add comment to clarify DV authentication requirement in createNewAuthProfile --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> * [Power Pages][Actions Hub] Show current site (#1129) * Refactor site context handling and update site data types for improved clarity * Refactor website data handling to use IWebsiteDetails and streamline site fetching logic * Refactor import statements in ActiveGroupTreeItem, EnvironmentGroupTreeItem, and InactiveGroupTreeItem for improved organization and clarity * Refactor import statements in InactiveGroupTreeItem for improved organization * Add tests for PacContext and improve context update logic * Add tests for previewSite functionality in ActionsHubCommandHandlers * Add fetchWebsites function and related tests for website retrieval logic * Add tests for ActiveGroupTreeItem and update siteInfo assertions * Add tests for ActionsHubTreeDataProvider to verify website loading logic * Add websiteid to VSCode settings for improved configuration * Refactor ActionsHubTreeDataProvider to simplify loadWebsites method and improve refresh logic * Refactor CurrentSiteContext to utilize utility functions for website YAML handling and improve code clarity * Refactor ActionsHubTreeDataProvider to streamline website loading logic and improve refresh handling * [Power Pages][Actions Hub] Update "Change environment" option to match Figma (#1130) * [Power Pages][Actions Hub] Update "Change environment" option to match Figma * Add test for environment switching in Actions Hub command handlers * Refactor environment switching to use detailed description and enhance test coverage for environment selection * Refactor tests for ActionsHubTreeDataProvider by removing redundant cases and ensuring proper stubbing of fetchWebsites * Disable website loading in ActionsHubTreeDataProvider tests to ensure empty result for specific scenarios * [Power Pages][Actions Hub] Add revealInOS command for current active site and update translations (#1131) * Refactor website details to use camelCase properties and add siteVisibility field (#1133) * [Power Pages][Actions Hub] Update command references for authentication profile (#1132) * [Power Pages][Actions Hub] Update command references for authentication profile * Update command reference for new authentication profile in ActionsHubTreeDataProvider test --------- Co-authored-by: amitjoshi <amitjoshi@microsoft.com> Co-authored-by: Priyanshu Agrawal <priyanshuag@microsoft.com> Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> * [Power Pages][Actions Hub] Add open site management functionality (#1136) * [Power Pages][Actions Hub] Add open site management functionality * Update translation for "Open Site Management" to use lowercase in localization files * remove unused methods --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: amitjoshi438 <54068463+amitjoshi438@users.noreply.github.com> Co-authored-by: amitjoshi <amitjoshi@microsoft.com> Co-authored-by: Ashwani Kumar <ashwanikumar1303@gmail.com> Co-authored-by: Ashwani Kumar <ashwanikumar@microsoft.com> Co-authored-by: tyaginidhi <tyaginidhi@users.noreply.github.com> Co-authored-by: Priyanshu Agrawal <priyanshuagrawal@outlook.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Sandeep Satheesh <ssatheesh@microsoft.com> Co-authored-by: Priyanshu Agrawal <priyanshuag@microsoft.com> Co-authored-by: BidishaMS <97606074+BidishaMS@users.noreply.github.com>
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.