Fixed up!

Author: nedtwigg

browser-extension/src/entrypoints/content.ts

@@ -6,8 +6,8 @@ import { EnhancerRegistry, TextareaRegistry } from '../lib/registries'
 const enhancers = new EnhancerRegistry()
 const enhancedTextareas = new TextareaRegistry()
 
-// Expose for debugging in har:view
-;(window as any).gitcassoTextareaRegistry = enhancedTextareas
+  // Expose for debugging in har:view
+  ; (window as any).gitcassoTextareaRegistry = enhancedTextareas
 
 function sendEventToBackground(type: 'ENHANCED' | 'DESTROYED', spot: CommentSpot): void {
   const message: CommentEvent = {
@@ -88,16 +88,20 @@ function enhanceMaybe(textarea: HTMLTextAreaElement) {
   logger.debug('activating textarea {}', textarea)
   injectStyles()
 
-  const enhancedTextarea = enhancers.tryToEnhance(textarea)
-  if (enhancedTextarea) {
-    logger.debug(
-      'Identified textarea:',
-      enhancedTextarea.spot.type,
-      enhancedTextarea.spot.unique_key,
-    )
-    enhancedTextareas.register(enhancedTextarea)
-  } else {
-    logger.debug('No handler found for textarea')
+  try {
+    const enhancedTextarea = enhancers.tryToEnhance(textarea)
+    if (enhancedTextarea) {
+      logger.debug(
+        'Identified textarea:',
+        enhancedTextarea.spot.type,
+        enhancedTextarea.spot.unique_key,
+      )
+      enhancedTextareas.register(enhancedTextarea)
+    } else {
+      logger.debug('No handler found for textarea')
+    }
+  } catch (e) {
+    logger.error(e)
   }
 }
 

browser-extension/tests/har-view.ts

@@ -337,20 +337,10 @@ function injectGitcassoScript(key: keyof typeof PAGES, html: string) {
                 'console.warn("Webextension-polyfill check bypassed for HAR testing")'
               );
 
-              // Mock necessary APIs before executing
-              window.chrome = window.chrome || {
-                runtime: {
-                  getURL: (path) => 'chrome-extension://gitcasso-test/' + path,
-                  onMessage: { addListener: () => {} },
-                  sendMessage: () => Promise.resolve(),
-                  id: 'gitcasso-test'
-                }
-              };
-              window.browser = window.chrome;
-
-              // Execute the patched script
+              // Execute the patched script with browser API mocks prepended
+              const browserMocks = 'window.chrome=window.chrome||{runtime:{getURL:path=>"chrome-extension://gitcasso-test/"+path,onMessage:{addListener:()=>{}},sendMessage:()=>Promise.resolve(),id:"gitcasso-test"}};window.browser=window.chrome;';
               const script = document.createElement('script');
-              script.textContent = patchedCode;
+              script.textContent = browserMocks + patchedCode;
               document.head.appendChild(script);
               console.log('Gitcasso content script loaded with location patching for:', '` + urlParts.href + `');
             })