identifyContextOf

Author: ntwigg

browser-extension/src/datamodel/handler-registry.ts

@@ -24,6 +24,20 @@ export class HandlerRegistry {
     return null;
   }
 
+  identifyTextarea(textarea: HTMLTextAreaElement): TextareaInfo<any> | null {
+    for (const handler of this.handlers) {
+      try {
+        const result = handler.identifyContextOf(textarea);
+        if (result) {
+          return result;
+        }
+      } catch (error) {
+        console.warn('Handler failed to identify textarea:', error);
+      }
+    }
+    return null;
+  }
+
   identifyAll(): TextareaInfo<any>[] {
     const allTextareas: TextareaInfo<any>[] = [];
 

browser-extension/src/datamodel/textarea-handler.ts

@@ -12,6 +12,8 @@ export interface TextareaInfo<T extends CommentContext = CommentContext> {
 export interface TextareaHandler<T extends CommentContext = CommentContext> {
   // Handler metadata
   forCommentTypes(): string[];
+  // whenever a new `textarea` is added to any webpage, this method is called to try to find a handler for it
+  identifyContextOf(textarea: HTMLTextAreaElement): TextareaInfo | null;
 
   // Content script functionality
   identify(): TextareaInfo<T>[];
@@ -37,6 +39,7 @@ export abstract class BaseTextareaHandler<T extends CommentContext = CommentCont
   }
 
   abstract forCommentTypes(): string[];
+  abstract identifyContextOf(textarea: HTMLTextAreaElement): TextareaInfo<T> | null;
   abstract identify(): TextareaInfo<T>[];
   abstract extractContext(textarea: HTMLTextAreaElement): T | null;
   abstract determineType(textarea: HTMLTextAreaElement): string | null;

browser-extension/src/entrypoints/content.ts

@@ -48,11 +48,13 @@ function initializeMaybe(textarea: HTMLTextAreaElement) {
     injectStyles()
     textarea.classList.add(CONFIG.ADDED_OVERTYPE_CLASS)
 
-    // Use registry to identify and handle this textarea
-    const textareaInfos = registry.identifyAll().filter(info => info.element === textarea)
-    for (const info of textareaInfos) {
-      logger.debug('Identified textarea:', info.type, info.context.unique_key)
+    // Use registry to identify and handle this specific textarea
+    const textareaInfo = registry.identifyTextarea(textarea)
+    if (textareaInfo) {
+      logger.debug('Identified textarea:', textareaInfo.type, textareaInfo.context.unique_key)
       // TODO: Set up textarea monitoring and draft saving
+    } else {
+      logger.debug('No handler found for textarea')
     }
   } else {
     logger.debug('already activated textarea {}', textarea)

browser-extension/src/handlers/github-handler.ts

@@ -33,6 +33,22 @@ export class GitHubHandler extends BaseTextareaHandler<GitHubContext> {
     ];
   }
 
+  identifyContextOf(textarea: HTMLTextAreaElement): TextareaInfo<GitHubContext> | null {
+    // Only handle GitHub domains
+    if (!window.location.hostname.includes('github')) {
+      return null;
+    }
+
+    const type = this.determineType(textarea);
+    const context = this.extractContext(textarea);
+    
+    if (type && context) {
+      return { element: textarea, type, context };
+    }
+    
+    return null;
+  }
+
   identify(): TextareaInfo<GitHubContext>[] {
     const textareas = document.querySelectorAll<HTMLTextAreaElement>('textarea');
     const results: TextareaInfo<GitHubContext>[] = [];

browser-extension/src/handlers/reddit-handler.ts

@@ -24,6 +24,22 @@ export class RedditHandler extends BaseTextareaHandler<RedditContext> {
     ];
   }
 
+  identifyContextOf(textarea: HTMLTextAreaElement): TextareaInfo<RedditContext> | null {
+    // Only handle Reddit domains
+    if (!window.location.hostname.includes('reddit')) {
+      return null;
+    }
+
+    const type = this.determineType(textarea);
+    const context = this.extractContext(textarea);
+    
+    if (type && context) {
+      return { element: textarea, type, context };
+    }
+    
+    return null;
+  }
+
   identify(): TextareaInfo<RedditContext>[] {
     const textareas = document.querySelectorAll<HTMLTextAreaElement>('textarea');
     const results: TextareaInfo<RedditContext>[] = [];