More refactor cleanup.

Author: ntwigg

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

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

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

@@ -13,7 +13,7 @@ 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;
+  identifyContextOf(textarea: HTMLTextAreaElement): T | null;
 
   // Popup functionality helpers
   generateDisplayTitle(context: T): string;

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

@@ -30,7 +30,7 @@ export class GitHubHandler implements TextareaHandler<GitHubContext> {
     ];
   }
 
-  identifyContextOf(textarea: HTMLTextAreaElement): TextareaInfo<GitHubContext> | null {
+  identifyContextOf(textarea: HTMLTextAreaElement): GitHubContext | null {
     // Only handle GitHub domains
     if (!window.location.hostname.includes('github')) {
       return null;
@@ -102,7 +102,7 @@ export class GitHubHandler implements TextareaHandler<GitHubContext> {
       commentId: commentId || undefined
     };
 
-    return { element: textarea, context };
+    return context;
   }
 
   generateDisplayTitle(context: GitHubContext): string {

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

@@ -21,7 +21,7 @@ export class RedditHandler implements TextareaHandler<RedditContext> {
     ];
   }
 
-  identifyContextOf(textarea: HTMLTextAreaElement): TextareaInfo<RedditContext> | null {
+  identifyContextOf(textarea: HTMLTextAreaElement): RedditContext | null {
     // Only handle Reddit domains
     if (!window.location.hostname.includes('reddit')) {
       return null;
@@ -87,7 +87,7 @@ export class RedditHandler implements TextareaHandler<RedditContext> {
       commentId: commentId || undefined
     };
 
-    return { element: textarea, context };
+    return context;
   }
 
   generateDisplayTitle(context: RedditContext): string {