Remove unnecessary methods.

Author: ntwigg

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

@@ -38,20 +38,6 @@ export class HandlerRegistry {
     return null;
   }
 
-  identifyAll(): TextareaInfo<any>[] {
-    const allTextareas: TextareaInfo<any>[] = [];
-    
-    for (const handler of this.handlers) {
-      try {
-        const textareas = handler.identify();
-        allTextareas.push(...textareas);
-      } catch (error) {
-        console.warn('Handler failed to identify textareas:', error);
-      }
-    }
-    
-    return allTextareas;
-  }
 
   getAllHandlers(): TextareaHandler<any>[] {
     return Array.from(this.handlers);

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

@@ -14,13 +14,7 @@ export interface TextareaHandler<T extends CommentContext = CommentContext> {
   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>[];
-  readContent(textarea: HTMLTextAreaElement): string;
-  setContent(textarea: HTMLTextAreaElement, content: string): void;
-  onSubmit(textarea: HTMLTextAreaElement, callback: (success: boolean) => void): void;
-  
+    
   // Context extraction
   extractContext(textarea: HTMLTextAreaElement): T | null;
   determineType(textarea: HTMLTextAreaElement): string | null;
@@ -40,29 +34,9 @@ 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;
   abstract generateDisplayTitle(context: T): string;
   abstract generateIcon(type: string): string;
   abstract buildUrl(context: T, withDraft?: boolean): string;
-  
-  readContent(textarea: HTMLTextAreaElement): string {
-    return textarea.value;
-  }
-  
-  setContent(textarea: HTMLTextAreaElement, content: string): void {
-    textarea.value = content;
-    textarea.dispatchEvent(new Event('input', { bubbles: true }));
-    textarea.dispatchEvent(new Event('change', { bubbles: true }));
-  }
-  
-  onSubmit(textarea: HTMLTextAreaElement, callback: (success: boolean) => void): void {
-    const form = textarea.closest('form');
-    if (form) {
-      form.addEventListener('submit', () => {
-        setTimeout(() => callback(true), 100);
-      }, { once: true });
-    }
-  }
 }

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

@@ -49,21 +49,6 @@ export class GitHubHandler extends BaseTextareaHandler<GitHubContext> {
     return null;
   }
 
-  identify(): TextareaInfo<GitHubContext>[] {
-    const textareas = document.querySelectorAll<HTMLTextAreaElement>('textarea');
-    const results: TextareaInfo<GitHubContext>[] = [];
-
-    for (const textarea of textareas) {
-      const type = this.determineType(textarea);
-      const context = this.extractContext(textarea);
-      
-      if (type && context) {
-        results.push({ element: textarea, type, context });
-      }
-    }
-
-    return results;
-  }
 
   extractContext(textarea: HTMLTextAreaElement): GitHubContext | null {
     const pathname = window.location.pathname;

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

@@ -40,21 +40,6 @@ export class RedditHandler extends BaseTextareaHandler<RedditContext> {
     return null;
   }
 
-  identify(): TextareaInfo<RedditContext>[] {
-    const textareas = document.querySelectorAll<HTMLTextAreaElement>('textarea');
-    const results: TextareaInfo<RedditContext>[] = [];
-
-    for (const textarea of textareas) {
-      const type = this.determineType(textarea);
-      const context = this.extractContext(textarea);
-      
-      if (type && context) {
-        results.push({ element: textarea, type, context });
-      }
-    }
-
-    return results;
-  }
 
   extractContext(textarea: HTMLTextAreaElement): RedditContext | null {
     const pathname = window.location.pathname;