refactor(awsq): remove externalBrowserUtils, use openUrl() #4117

externalBrowserUtils is redundant and does not have telemetry attached.

Author: hayemaxi

src/amazonq/commons/externalBrowser/externalBrowserUtils.ts

@@ -23,9 +23,9 @@ import { AuthController } from '../../../amazonq/auth/controller'
 import { getLogger } from '../../../shared/logger'
 import { submitFeedback } from '../../../feedback/vue/submitFeedback'
 import { placeholder } from '../../../shared/vscode/commands2'
-import { ExternalBrowserUtils } from '../../../amazonq/commons/externalBrowser/externalBrowserUtils'
 import { userGuideURL } from '../../../amazonq/webview/ui/texts/constants'
 import { EditorContentController } from '../../../amazonq/commons/controllers/contentController'
+import { openUrl } from '../../../shared/utilities/vsCodeUtils'
 
 export interface ChatControllerEventEmitters {
     readonly processHumanChatMessage: EventEmitter<any>
@@ -441,7 +441,7 @@ To learn more, visit the _[Amazon Q User Guide](${userGuideURL})_.
     }
 
     private processLink(message: any) {
-        ExternalBrowserUtils.instance.openLink(message.link)
+        openUrl(vscode.Uri.parse(message.link))
     }
 
     private insertCodeAtPosition(message: any) {

src/amazonqFeatureDev/controllers/chat/controller.ts

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { Event as VSCodeEvent } from 'vscode'
+import { Event as VSCodeEvent, Uri } from 'vscode'
 import { EditorContextExtractor } from '../../editor/context/extractor'
 import { ChatSessionStorage } from '../../storages/chatSession'
 import { Messenger, StaticTextResponseType } from './messenger/messenger'
@@ -45,8 +45,8 @@ import { getLogger } from '../../../shared/logger/logger'
 import { triggerPayloadToChatRequest } from './chatRequest/converter'
 import { OnboardingPageInteraction } from '../../../amazonq/onboardingPage/model'
 import { getChatAuthState } from '../../../codewhisperer/util/authUtil'
-import { ExternalBrowserUtils } from '../../../amazonq/commons/externalBrowser/externalBrowserUtils'
 import { processTransformByQ } from '../../../amazonqGumby/entrypoint'
+import { openUrl } from '../../../shared/utilities/vsCodeUtils'
 
 export interface ChatControllerMessagePublishers {
     readonly processPromptChatMessage: MessagePublisher<PromptMessage>
@@ -191,7 +191,7 @@ export class ChatController {
         click: ResponseBodyLinkClickMessage | SourceLinkClickMessage | FooterInfoLinkClick
     ) {
         this.telemetryHelper.recordInteractWithMessage(click)
-        ExternalBrowserUtils.instance.openLink(click.link)
+        openUrl(Uri.parse(click.link))
     }
 
     private processResponseBodyLinkClick(click: ResponseBodyLinkClickMessage) {