@fatagnus/convex-feedback 0.2.1 → 0.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fatagnus/convex-feedback",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "Bug reports and feedback collection component for Convex applications with AI analysis and email notifications",
   "license": "Apache-2.0",
   "repository": {

package/src/convex/agents/feedbackInterviewAgent.ts

@@ -19,14 +19,17 @@ import { action, query, mutation, internalMutation } from "../_generated/server"
 import type { Id } from "../_generated/dataModel";
 import type { BugSeverity, FeedbackType, FeedbackPriority } from "../schema";
 
-// Create OpenRouter provider
-const openrouter = createOpenAICompatible({
-  name: "openrouter",
-  baseURL: "https://openrouter.ai/api/v1",
-  headers: {
-    Authorization: `Bearer ${process.env.OPENROUTER_API_KEY}`,
-  },
-});
+// Helper to create OpenRouter provider with a specific API key
+// This is created dynamically because Convex components don't inherit parent env vars
+function createOpenRouterProvider(apiKey: string) {
+  return createOpenAICompatible({
+    name: "openrouter",
+    baseURL: "https://openrouter.ai/api/v1",
+    headers: {
+      Authorization: `Bearer ${apiKey}`,
+    },
+  });
+}
 
 // Agent name constant
 const AGENT_NAME = "Feedback Interview Agent";
@@ -186,101 +189,6 @@ const generateFeedback = createTool({
   },
 });
 
-// ============================================================================
-// Agent Definition
-// ============================================================================
-
-/**
- * Bug Report Interview Agent
- */
-export const bugInterviewAgent = new Agent(components.agent, {
-  name: "Bug Report Interview Agent",
-  languageModel: openrouter.languageModel("anthropic/claude-sonnet-4"),
-  
-  instructions: `You are a helpful assistant that interviews users to gather detailed information about bugs they've encountered.
-
-Your goal is to understand the bug thoroughly and generate a high-quality bug report.
-
-## Interview Flow
-
-1. **Greet briefly** and ask what went wrong (the bug they encountered).
-
-2. **Understand the bug**:
-   - What happened? (the actual behavior)
-   - What did they expect to happen? (expected behavior)
-   - How often does this occur? (always, sometimes, once)
-
-3. **Get reproduction info**:
-   - What steps led to this bug?
-   - Can they reliably reproduce it?
-
-4. **Assess impact**:
-   - How does this affect their work?
-   - How urgent is this for them?
-
-5. **Generate the report** using the generateBugReport tool when you have enough information.
-
-## Guidelines
-- Be conversational but efficient - aim for 3-5 exchanges
-- Use choice inputs when there are clear options
-- Use text inputs for open-ended questions
-- Don't ask for technical details the user might not know
-- Focus on understanding the user experience
-- Generate the report as soon as you have enough info`,
-
-  tools: {
-    requestUserInput,
-    generateBugReport,
-  },
-
-  maxSteps: 30,
-});
-
-/**
- * Feedback Interview Agent
- */
-export const feedbackInterviewAgent = new Agent(components.agent, {
-  name: "Feedback Interview Agent",
-  languageModel: openrouter.languageModel("anthropic/claude-sonnet-4"),
-  
-  instructions: `You are a helpful assistant that interviews users to gather detailed feedback and suggestions.
-
-Your goal is to understand their idea thoroughly and generate well-structured feedback.
-
-## Interview Flow
-
-1. **Greet briefly** and ask about their idea or suggestion.
-
-2. **Understand the need**:
-   - What problem does this solve?
-   - What's their current workaround (if any)?
-
-3. **Clarify the request**:
-   - What would the ideal solution look like?
-   - Is this a new feature, change to existing, or general feedback?
-
-4. **Assess importance**:
-   - How important is this to them?
-   - Who else might benefit?
-
-5. **Generate the feedback** using the generateFeedback tool when you have enough information.
-
-## Guidelines
-- Be conversational but efficient - aim for 3-5 exchanges
-- Use choice inputs when there are clear options
-- Use text inputs for open-ended questions
-- Help users articulate their ideas clearly
-- Focus on understanding the value and impact
-- Generate the feedback as soon as you have enough info`,
-
-  tools: {
-    requestUserInput,
-    generateFeedback,
-  },
-
-  maxSteps: 30,
-});
-
 // ============================================================================
 // Internal Mutations
 // ============================================================================
@@ -570,6 +478,7 @@ Your goal is to understand their idea thoroughly and generate well-structured fe
  */
 export const startBugInterview = action({
   args: {
+    openRouterApiKey: v.string(),
     reporterType: v.union(v.literal("staff"), v.literal("customer")),
     reporterId: v.string(),
     reporterEmail: v.string(),
@@ -605,10 +514,8 @@ export const startBugInterview = action({
     })),
   }),
   handler: async (ctx, args) => {
-    // Check if OpenRouter API key is configured
-    if (!process.env.OPENROUTER_API_KEY) {
-      throw new Error("OPENROUTER_API_KEY not configured. Interview mode requires AI.");
-    }
+    // Create OpenRouter provider with the passed API key
+    const openrouter = createOpenRouterProvider(args.openRouterApiKey);
 
     // Create a thread for the interview
     const threadId = await createThread(ctx, components.agent, {
@@ -642,14 +549,30 @@ export const startBugInterview = action({
     });
 
     // Start the interview
-    const result = await dynamicAgent.generateText(
-      ctx,
-      { 
-        threadId,
-        customCtx: { sessionId },
-      },
-      { prompt: "Start the bug report interview. Greet the user briefly and ask what bug or issue they encountered." }
-    );
+    let result;
+    try {
+      result = await dynamicAgent.generateText(
+        ctx,
+        { 
+          threadId,
+          customCtx: { sessionId },
+        },
+        { prompt: "Start the bug report interview. Greet the user briefly and ask what bug or issue they encountered." }
+      );
+    } catch (error) {
+      // Provide more descriptive error messages based on the error type
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      if (errorMessage.includes("401") || errorMessage.includes("unauthorized")) {
+        throw new Error("AI service authentication failed. Please check your OPENROUTER_API_KEY.");
+      } else if (errorMessage.includes("429") || errorMessage.includes("rate limit")) {
+        throw new Error("AI service rate limited. Please try again in a few moments.");
+      } else if (errorMessage.includes("503") || errorMessage.includes("unavailable")) {
+        throw new Error("AI service temporarily unavailable. Please try again later.");
+      } else if (errorMessage.includes("timeout") || errorMessage.includes("ETIMEDOUT")) {
+        throw new Error("AI service request timed out. Please try again.");
+      }
+      throw new Error(`Failed to start interview: ${errorMessage}`);
+    }
 
     // Check for pending input request
     const pendingRequest = await ctx.runQuery(
@@ -686,6 +609,7 @@ export const startBugInterview = action({
  */
 export const startFeedbackInterview = action({
   args: {
+    openRouterApiKey: v.string(),
     reporterType: v.union(v.literal("staff"), v.literal("customer")),
     reporterId: v.string(),
     reporterEmail: v.string(),
@@ -721,10 +645,8 @@ export const startFeedbackInterview = action({
     })),
   }),
   handler: async (ctx, args) => {
-    // Check if OpenRouter API key is configured
-    if (!process.env.OPENROUTER_API_KEY) {
-      throw new Error("OPENROUTER_API_KEY not configured. Interview mode requires AI.");
-    }
+    // Create OpenRouter provider with the passed API key
+    const openrouter = createOpenRouterProvider(args.openRouterApiKey);
 
     // Create a thread for the interview
     const threadId = await createThread(ctx, components.agent, {
@@ -758,14 +680,30 @@ export const startFeedbackInterview = action({
     });
 
     // Start the interview
-    const result = await dynamicAgent.generateText(
-      ctx,
-      { 
-        threadId,
-        customCtx: { sessionId },
-      },
-      { prompt: "Start the feedback interview. Greet the user briefly and ask about their idea or suggestion." }
-    );
+    let result;
+    try {
+      result = await dynamicAgent.generateText(
+        ctx,
+        { 
+          threadId,
+          customCtx: { sessionId },
+        },
+        { prompt: "Start the feedback interview. Greet the user briefly and ask about their idea or suggestion." }
+      );
+    } catch (error) {
+      // Provide more descriptive error messages based on the error type
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      if (errorMessage.includes("401") || errorMessage.includes("unauthorized")) {
+        throw new Error("AI service authentication failed. Please check your OPENROUTER_API_KEY.");
+      } else if (errorMessage.includes("429") || errorMessage.includes("rate limit")) {
+        throw new Error("AI service rate limited. Please try again in a few moments.");
+      } else if (errorMessage.includes("503") || errorMessage.includes("unavailable")) {
+        throw new Error("AI service temporarily unavailable. Please try again later.");
+      } else if (errorMessage.includes("timeout") || errorMessage.includes("ETIMEDOUT")) {
+        throw new Error("AI service request timed out. Please try again.");
+      }
+      throw new Error(`Failed to start interview: ${errorMessage}`);
+    }
 
     // Check for pending input request
     const pendingRequest = await ctx.runQuery(
@@ -802,6 +740,7 @@ export const startFeedbackInterview = action({
  */
 export const continueInterview = action({
   args: {
+    openRouterApiKey: v.string(),
     threadId: v.string(),
     sessionId: v.string(),
     requestId: v.id("feedbackInputRequests"),
@@ -859,6 +798,9 @@ export const continueInterview = action({
     })),
   }),
   handler: async (ctx, args) => {
+    // Create OpenRouter provider with the passed API key
+    const openrouter = createOpenRouterProvider(args.openRouterApiKey);
+
     // Submit the response
     await ctx.runMutation(
       api.inputRequests.submitResponse,
@@ -887,18 +829,34 @@ export const continueInterview = action({
     });
 
     // Continue the agent
-    const result = await dynamicAgent.generateText(
-      ctx,
-      { 
-        threadId: args.threadId,
-        customCtx: { sessionId: args.sessionId },
-      },
-      {
-        prompt: `The user responded: ${args.response}
+    let result;
+    try {
+      result = await dynamicAgent.generateText(
+        ctx,
+        { 
+          threadId: args.threadId,
+          customCtx: { sessionId: args.sessionId },
+        },
+        {
+          prompt: `The user responded: ${args.response}
 
 Please continue the interview based on their response.`,
+        }
+      );
+    } catch (error) {
+      // Provide more descriptive error messages based on the error type
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      if (errorMessage.includes("401") || errorMessage.includes("unauthorized")) {
+        throw new Error("AI service authentication failed. Please check your OPENROUTER_API_KEY.");
+      } else if (errorMessage.includes("429") || errorMessage.includes("rate limit")) {
+        throw new Error("AI service rate limited. Please try again in a few moments.");
+      } else if (errorMessage.includes("503") || errorMessage.includes("unavailable")) {
+        throw new Error("AI service temporarily unavailable. Please try again later.");
+      } else if (errorMessage.includes("timeout") || errorMessage.includes("ETIMEDOUT")) {
+        throw new Error("AI service request timed out. Please try again.");
       }
-    );
+      throw new Error(`Failed to continue interview: ${errorMessage}`);
+    }
 
     // Check if interview is complete (session has generated report)
     const session = await ctx.runQuery(api.agents.feedbackInterviewAgent.getSessionByThread, {

package/src/react/BugReportButton.tsx

@@ -752,11 +752,12 @@ export function BugReportButton({
   };
 
   // Auto-start interview when interview mode is selected and no interview is active
+  // Important: Check for error state to prevent infinite retry loop on failure
   useEffect(() => {
-    if (opened && inputMode === 'interview' && enableInterview && !interviewState.threadId && !interviewState.isThinking) {
+    if (opened && inputMode === 'interview' && enableInterview && !interviewState.threadId && !interviewState.isThinking && !interviewState.error) {
       handleStartInterview();
     }
-  }, [opened, inputMode, enableInterview, interviewState.threadId, interviewState.isThinking, handleStartInterview]);
+  }, [opened, inputMode, enableInterview, interviewState.threadId, interviewState.isThinking, interviewState.error, handleStartInterview]);
 
   return (
     <>
@@ -844,9 +845,21 @@ export function BugReportButton({
           {interviewState.error && (
             <Alert icon={<IconAlertCircle size={16} />} color="red" title="Interview Error">
               {interviewState.error}
-              <Button variant="subtle" size="xs" mt="xs" onClick={() => setInputMode('form')}>
-                Switch to Form
-              </Button>
+              <Group gap="xs" mt="xs">
+                <Button
+                  variant="subtle"
+                  size="xs"
+                  onClick={() => {
+                    // Clear error and retry
+                    setInterviewState(prev => ({ ...prev, error: null }));
+                  }}
+                >
+                  Try Again
+                </Button>
+                <Button variant="subtle" size="xs" onClick={() => setInputMode('form')}>
+                  Switch to Form
+                </Button>
+              </Group>
             </Alert>
           )}