address review bot feedback (reviewloop iteration 1)

- Fix quote stripping order: use body_text when available, then strip
  quoted content on plain text (Greptile P1)
- Align test helper with production preprocessing pipeline (Greptile P2)
- Populate to_address in fetchAndCacheSentEmails rows (Devin)
- Use dedicated inferError state for style tab error display (Devin)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Ankit Gupta

src/main/services/style-profiler.ts

@@ -219,6 +219,7 @@ async function fetchAndCacheSentEmails(
       body: email.body,
       date: email.date,
       is_reply: email.subject.toLowerCase().startsWith("re:") ? 1 : 0,
+      to_address: email.to,
     });
   }
 
@@ -424,9 +425,10 @@ export async function inferStyleFromSentEmails(
   // Build the email samples for the prompt
   const emailSamples = emails
     .map((e) => {
-      // Strip quoted/forwarded content first so we only analyze the user's own writing
-      const stripped = stripQuotedContent(e.body);
-      const text = stripHtmlForSearch(stripped);
+      // Use pre-extracted plain text when available, otherwise strip HTML first
+      const plainText = e.body_text ?? stripHtmlForSearch(e.body);
+      // Strip quoted/forwarded content so we only analyze the user's own writing
+      const text = stripQuotedContent(plainText);
       const truncated = truncateBody(text, 300);
       const type = e.is_reply ? "reply" : "new";
       return `---\nTo: ${e.to_address ?? "unknown"}\nSubject: ${e.subject}\nDate: ${e.date}\nType: ${type}\n\n${truncated}\n---`;

src/renderer/components/SettingsPanel.tsx

@@ -63,6 +63,7 @@ export function SettingsPanel({ onClose, initialTab }: SettingsPanelProps) {
   const [saveResult, setSaveResult] = useState<string | null>(null);
   const [stylePrompt, setStylePrompt] = useState("");
   const [isInferring, setIsInferring] = useState(false);
+  const [inferError, setInferError] = useState<string | null>(null);
   const [agentDrafterPrompt, setAgentDrafterPrompt] = useState("");
   const [isRerunningAll, setIsRerunningAll] = useState(false);
   const [rerunResult, setRerunResult] = useState<string | null>(null);
@@ -480,7 +481,7 @@ export function SettingsPanel({ onClose, initialTab }: SettingsPanelProps) {
 
   const handleInferStyle = async () => {
     setIsInferring(true);
-    setSaveResult(null);
+    setInferError(null);
     try {
       const result = (await window.api.style.infer()) as {
         success: boolean;
@@ -490,10 +491,10 @@ export function SettingsPanel({ onClose, initialTab }: SettingsPanelProps) {
       if (result.success && result.data) {
         setStylePrompt(result.data);
       } else {
-        setSaveResult(result.error || "Failed to infer writing style");
+        setInferError(result.error || "Failed to infer writing style");
       }
     } catch {
-      setSaveResult("Failed to infer writing style");
+      setInferError("Failed to infer writing style");
     } finally {
       setIsInferring(false);
     }
@@ -2111,6 +2112,9 @@ export function SettingsPanel({ onClose, initialTab }: SettingsPanelProps) {
                   This prompt is prepended to your draft generation when style examples are
                   available. It tells the AI how to interpret the examples of your past emails.
                 </p>
+                {inferError && (
+                  <p className="text-xs text-red-600 dark:text-red-400 mt-1">{inferError}</p>
+                )}
               </div>
 
               <button

tests/unit/style-inference.spec.ts

@@ -51,10 +51,33 @@ Do NOT include example phrases or quoted text from the emails.`;
  * Build the email samples portion of the prompt, mirroring the logic
  * in inferStyleFromSentEmails.
  */
+// Simplified version of stripQuotedContent for plain text
+// (matches the production code path: body_text → stripQuotedContent → truncateBody)
+function stripQuotedContent(text: string): string {
+  if (!text) return text;
+  const lines = text.split("\n");
+  for (let i = 0; i < lines.length; i++) {
+    const line = lines[i].trim();
+    // "On ... wrote:" header
+    if (/^on\s.+wrote:\s*$/i.test(line)) {
+      return lines.slice(0, i).join("\n").trimEnd();
+    }
+    // Block of ">" quoted lines
+    if (line.startsWith(">")) {
+      const above = lines.slice(0, i).join("\n").trimEnd();
+      if (above && !above.split("\n").every((l) => l.trim().startsWith(">"))) {
+        return above;
+      }
+    }
+  }
+  return text;
+}
+
 function buildEmailSamples(emails: SentEmailRow[]): string {
   return emails
     .map((e) => {
-      const text = e.body_text ?? stripHtmlForSearch(e.body);
+      const plainText = e.body_text ?? stripHtmlForSearch(e.body);
+      const text = stripQuotedContent(plainText);
       const truncated = truncateBody(text, 300);
       const type = e.is_reply ? "reply" : "new";
       return `---\nTo: ${e.to_address ?? "unknown"}\nSubject: ${e.subject}\nDate: ${e.date}\nType: ${type}\n\n${truncated}\n---`;