improve prompting tactics and retry logic for multiple tasks

src/aiService.js

@@ -20,6 +20,25 @@ class OpenRouterService {
     this.apiKey = key;
   }
 
+  async retryRequest(requestFn, maxRetries = 3, delayMs = 1000) {
+    for (let attempt = 1; attempt <= maxRetries; attempt++) {
+      try {
+        return await requestFn();
+      } catch (error) {
+        console.log(`API request attempt ${attempt}/${maxRetries} failed:`, error.message);
+        
+        if (attempt === maxRetries) {
+          throw error; // Final attempt failed, throw the error
+        }
+        
+        // Wait before retrying, with exponential backoff
+        const delay = delayMs * Math.pow(2, attempt - 1);
+        console.log(`Retrying in ${delay}ms...`);
+        await new Promise(resolve => setTimeout(resolve, delay));
+      }
+    }
+  }
+
   async generateContextualHint(prompt) {
     // Check for API key at runtime
     const currentKey = this.getApiKey();
@@ -43,11 +62,10 @@ class OpenRouterService {
         body: JSON.stringify({
           model: this.model,
           messages: [{
-            role: 'system',
-            content: 'You provide clues for word puzzles. You will be told the target word that players need to guess, but you must NEVER mention, spell, or reveal that word in your response. Follow the EXACT format requested. Be concise and direct about the target word without revealing it. Use plain text only - no bold, italics, asterisks, or markdown formatting. Stick to word limits.'
-          }, {
             role: 'user',
-            content: prompt
+            content: `You provide clues for word puzzles. You will be told the target word that players need to guess, but you must NEVER mention, spell, or reveal that word in your response. Follow the EXACT format requested. Be concise and direct about the target word without revealing it. Use plain text only - no bold, italics, asterisks, or markdown formatting. Stick to word limits.
+
+${prompt}`
           }],
           max_tokens: 50,
           temperature: 0.6
@@ -109,22 +127,20 @@ class OpenRouterService {
     }
 
     try {
-      const response = await fetch(this.apiUrl, {
-        method: 'POST',
-        headers: {
-          'Content-Type': 'application/json',
-          'Authorization': `Bearer ${this.apiKey}`,
-          'HTTP-Referer': window.location.origin,
-          'X-Title': 'Cloze Reader'
-        },
-        body: JSON.stringify({
-          model: this.model,
-          messages: [{
-            role: 'system',
-            content: 'You are a vocabulary selector for educational cloze exercises. Select meaningful, properly-spelled content words that appear exactly as written in the passage.'
-          }, {
-            role: 'user',
-            content: `Select exactly ${count} words from this passage for a cloze exercise.
+      return await this.retryRequest(async () => {
+        const response = await fetch(this.apiUrl, {
+          method: 'POST',
+          headers: {
+            'Content-Type': 'application/json',
+            'Authorization': `Bearer ${this.apiKey}`,
+            'HTTP-Referer': window.location.origin,
+            'X-Title': 'Cloze Reader'
+          },
+          body: JSON.stringify({
+            model: this.model,
+            messages: [{
+              role: 'user',
+              content: `You are a vocabulary selector for educational cloze exercises. Select exactly ${count} words from this passage for a cloze exercise.
 
 REQUIREMENTS:
 - Choose clear, properly-spelled words (no OCR errors like "andsatires")
@@ -136,47 +152,48 @@ REQUIREMENTS:
 Return ONLY a JSON array of the selected words.
 
 Passage: "${passage}"`
-          }],
-          max_tokens: 100,
-          temperature: 0.3
-        })
-      });
+            }],
+            max_tokens: 100,
+            temperature: 0.3
+          })
+        });
 
-      if (!response.ok) {
-        throw new Error(`API request failed: ${response.status}`);
-      }
+        if (!response.ok) {
+          throw new Error(`API request failed: ${response.status}`);
+        }
 
-      const data = await response.json();
-      
-      // Check for OpenRouter error response
-      if (data.error) {
-        console.error('OpenRouter API error for word selection:', data.error);
-        throw new Error(`OpenRouter API error: ${data.error.message || JSON.stringify(data.error)}`);
-      }
-      
-      // Check if response has expected structure
-      if (!data.choices || !data.choices[0] || !data.choices[0].message || !data.choices[0].message.content) {
-        console.error('Invalid word selection API response structure:', data);
-        throw new Error('API response missing expected content');
-      }
-      
-      const content = data.choices[0].message.content.trim();
-      
-      // Try to parse as JSON array
-      try {
-        const words = JSON.parse(content);
-        if (Array.isArray(words)) {
-          return words.slice(0, count);
+        const data = await response.json();
+        
+        // Check for OpenRouter error response
+        if (data.error) {
+          console.error('OpenRouter API error for word selection:', data.error);
+          throw new Error(`OpenRouter API error: ${data.error.message || JSON.stringify(data.error)}`);
         }
-      } catch (e) {
-        // If not valid JSON, try to extract words from the response
-        const matches = content.match(/"([^"]+)"/g);
-        if (matches) {
-          return matches.map(m => m.replace(/"/g, '')).slice(0, count);
+        
+        // Check if response has expected structure
+        if (!data.choices || !data.choices[0] || !data.choices[0].message || !data.choices[0].message.content) {
+          console.error('Invalid word selection API response structure:', data);
+          throw new Error('API response missing expected content');
         }
-      }
-      
-      throw new Error('Failed to parse AI response');
+        
+        const content = data.choices[0].message.content.trim();
+        
+        // Try to parse as JSON array
+        try {
+          const words = JSON.parse(content);
+          if (Array.isArray(words)) {
+            return words.slice(0, count);
+          }
+        } catch (e) {
+          // If not valid JSON, try to extract words from the response
+          const matches = content.match(/"([^"]+)"/g);
+          if (matches) {
+            return matches.map(m => m.replace(/"/g, '')).slice(0, count);
+          }
+        }
+        
+        throw new Error('Failed to parse AI response');
+      });
     } catch (error) {
       console.error('Error selecting words with AI:', error);
       throw error;
@@ -359,65 +376,64 @@ Return as JSON: {"passage1": {...}, "passage2": {...}}`
     }
 
     try {
-      const response = await fetch(this.apiUrl, {
-        method: 'POST',
-        headers: {
-          'Content-Type': 'application/json',
-          'Authorization': `Bearer ${this.apiKey}`,
-          'HTTP-Referer': window.location.origin,
-          'X-Title': 'Cloze Reader'
-        },
-        body: JSON.stringify({
-          model: this.model,
-          messages: [{
-            role: 'system',
-            content: 'You are a literary expert. Provide exactly 1 short, factual sentence about this classic work. Be accurate and concise. Do not add fictional details or characters.'
-          }, {
-            role: 'user',
-            content: `Write one factual sentence about "${title}" by ${author}. Focus on what type of work it is, when it was written, or its historical significance.`
-          }],
-          max_tokens: 80,
-          temperature: 0.2
-        })
-      });
+      return await this.retryRequest(async () => {
+        const response = await fetch(this.apiUrl, {
+          method: 'POST',
+          headers: {
+            'Content-Type': 'application/json',
+            'Authorization': `Bearer ${this.apiKey}`,
+            'HTTP-Referer': window.location.origin,
+            'X-Title': 'Cloze Reader'
+          },
+          body: JSON.stringify({
+            model: this.model,
+            messages: [{
+              role: 'user',
+              content: `You are a literary expert. Write one factual sentence about "${title}" by ${author}. Focus on what type of work it is, when it was written, or its historical significance. Be accurate and concise. Do not add fictional details or characters.`
+            }],
+            max_tokens: 80,
+            temperature: 0.2
+          })
+        });
 
-      if (!response.ok) {
-        const errorText = await response.text();
-        console.error('Contextualization API error:', response.status, errorText);
-        throw new Error(`API request failed: ${response.status}`);
-      }
+        if (!response.ok) {
+          const errorText = await response.text();
+          console.error('Contextualization API error:', response.status, errorText);
+          throw new Error(`API request failed: ${response.status}`);
+        }
 
-      const data = await response.json();
-      
-      // Check for OpenRouter error response
-      if (data.error) {
-        console.error('OpenRouter API error for contextualization:', data.error);
-        throw new Error(`OpenRouter API error: ${data.error.message || JSON.stringify(data.error)}`);
-      }
-      
-      // Check if response has expected structure
-      if (!data.choices || !data.choices[0] || !data.choices[0].message || !data.choices[0].message.content) {
-        console.error('Invalid contextualization API response structure:', data);
-        throw new Error('API response missing expected content');
-      }
-      
-      let content = data.choices[0].message.content.trim();
-      
-      // Clean up AI response artifacts
-      content = content
-        .replace(/^\s*["']|["']\s*$/g, '')  // Remove leading/trailing quotes
-        .replace(/^\s*[:;]+\s*/, '')        // Remove leading colons and semicolons
-        .replace(/\*+/g, '')                // Remove asterisks (markdown bold/italic)
-        .replace(/_+/g, '')                 // Remove underscores (markdown)
-        .replace(/#+\s*/g, '')              // Remove hash symbols (markdown headers)
-        .replace(/\s+/g, ' ')               // Normalize whitespace
-        .trim();
-      
-      console.log('Contextualization received:', content);
-      return content;
+        const data = await response.json();
+        
+        // Check for OpenRouter error response
+        if (data.error) {
+          console.error('OpenRouter API error for contextualization:', data.error);
+          throw new Error(`OpenRouter API error: ${data.error.message || JSON.stringify(data.error)}`);
+        }
+        
+        // Check if response has expected structure
+        if (!data.choices || !data.choices[0] || !data.choices[0].message || !data.choices[0].message.content) {
+          console.error('Invalid contextualization API response structure:', data);
+          throw new Error('API response missing expected content');
+        }
+        
+        let content = data.choices[0].message.content.trim();
+        
+        // Clean up AI response artifacts
+        content = content
+          .replace(/^\s*["']|["']\s*$/g, '')  // Remove leading/trailing quotes
+          .replace(/^\s*[:;]+\s*/, '')        // Remove leading colons and semicolons
+          .replace(/\*+/g, '')                // Remove asterisks (markdown bold/italic)
+          .replace(/_+/g, '')                 // Remove underscores (markdown)
+          .replace(/#+\s*/g, '')              // Remove hash symbols (markdown headers)
+          .replace(/\s+/g, ' ')               // Normalize whitespace
+          .trim();
+        
+        console.log('Contextualization received:', content);
+        return content;
+      });
     } catch (error) {
       console.error('Error getting contextualization:', error);
-      return `📚 Practice with classic literature from ${author}'s "${title}"`;
+      return `📜 Practice with literature from ${author}'s "${title}"`;
     }
   }