@bbearai/react-native 0.5.6 → 0.5.7

package/dist/index.js

@@ -11829,7 +11829,13 @@ var BugBearClient = class {
         console.error("BugBear: Failed to fetch assignments", formatPgError(error));
         return [];
       }
-      const mapped = (data || []).map((item) => ({
+      const mapped = (data || []).filter((item) => {
+        if (!item.test_case) {
+          console.warn("BugBear: Assignment returned without test_case", { id: item.id });
+          return false;
+        }
+        return true;
+      }).map((item) => ({
         id: item.id,
         status: item.status,
         startedAt: item.started_at,
@@ -12086,6 +12092,16 @@ var BugBearClient = class {
       if (feedback.rating < 1 || feedback.rating > 5) {
         return { success: false, error: "Rating must be between 1 and 5" };
       }
+      const optionalRatings = [
+        { name: "clarityRating", value: feedback.clarityRating },
+        { name: "stepsRating", value: feedback.stepsRating },
+        { name: "relevanceRating", value: feedback.relevanceRating }
+      ];
+      for (const { name, value } of optionalRatings) {
+        if (value !== void 0 && value !== null && (value < 1 || value > 5)) {
+          return { success: false, error: `${name} must be between 1 and 5` };
+        }
+      }
       const { error: feedbackError } = await this.supabase.from("test_feedback").insert({
         project_id: this.config.projectId,
         test_case_id: testCaseId,
@@ -14648,6 +14664,7 @@ function TestFeedbackScreen({ status, assignmentId, nav }) {
   const assignment = assignments.find((a) => a.id === assignmentId);
   const showFlags = rating < 4;
   const handleSubmit = async () => {
+    if (submitting || images.isUploading) return;
     setSubmitting(true);
     if (client && assignment) {
       const screenshotUrls = images.getScreenshotUrls();
@@ -14938,7 +14955,7 @@ function ReportScreen({ nav, prefill }) {
     }
   }, [reportType]);
   const handleSubmit = async () => {
-    if (!client || !description.trim()) return;
+    if (!client || !description.trim() || images.isUploading) return;
     if (submittingRef.current) return;
     submittingRef.current = true;
     setSubmitting(true);
@@ -15186,15 +15203,28 @@ function ThreadDetailScreen({ thread, nav }) {
   const [sendError, setSendError] = (0, import_react14.useState)(false);
   const replyImages = useImageAttachments(uploadImage, 3, "discussion-attachments");
   (0, import_react14.useEffect)(() => {
+    let cancelled = false;
+    setLoading(true);
     (async () => {
-      setLoading(true);
-      const msgs = await getThreadMessages(thread.id);
-      setMessages(msgs);
-      setLoading(false);
-      if (thread.unreadCount > 0) {
-        await markAsRead(thread.id);
+      try {
+        const msgs = await getThreadMessages(thread.id);
+        if (!cancelled) {
+          setMessages(msgs);
+        }
+        if (thread.unreadCount > 0) {
+          await markAsRead(thread.id);
+        }
+      } catch (err) {
+        console.error("BugBear: Failed to load thread messages", err);
+      } finally {
+        if (!cancelled) {
+          setLoading(false);
+        }
       }
     })();
+    return () => {
+      cancelled = true;
+    };
   }, [thread.id]);
   const handleSend = async () => {
     if (!replyText.trim() && replyImages.images.length === 0 || sending || replyImages.isUploading) return;
@@ -15241,9 +15271,9 @@ function ThreadDetailScreen({ thread, nav }) {
   ), /* @__PURE__ */ import_react14.default.createElement(
     import_react_native13.TouchableOpacity,
     {
-      style: [styles11.sendBtn, (!replyText.trim() || sending || replyImages.isUploading) && styles11.sendBtnDisabled],
+      style: [styles11.sendBtn, (!replyText.trim() && replyImages.images.length === 0 || sending || replyImages.isUploading) && styles11.sendBtnDisabled],
       onPress: handleSend,
-      disabled: !replyText.trim() || sending || replyImages.isUploading
+      disabled: !replyText.trim() && replyImages.images.length === 0 || sending || replyImages.isUploading
     },
     /* @__PURE__ */ import_react14.default.createElement(import_react_native13.Text, { style: styles11.sendBtnText }, sending ? "..." : "Send")
   )));
@@ -15289,7 +15319,7 @@ function ComposeMessageScreen({ nav }) {
   const [sending, setSending] = (0, import_react15.useState)(false);
   const images = useImageAttachments(uploadImage, 3, "discussion-attachments");
   const handleSend = async () => {
-    if (!subject.trim() || !message.trim()) return;
+    if (!subject.trim() || !message.trim() || sending || images.isUploading) return;
     setSending(true);
     const attachments = images.getAttachments();
     const result = await createThread({
@@ -15375,6 +15405,7 @@ function ProfileScreen({ nav }) {
     }
   }, [testerInfo]);
   const handleSave = async () => {
+    if (saving) return;
     setSaving(true);
     const updates = {
       name: name.trim(),
@@ -15503,11 +15534,21 @@ function IssueListScreen({ nav, category }) {
     let cancelled = false;
     setLoading(true);
     (async () => {
-      if (!client) return;
-      const data = await client.getIssues(category);
-      if (!cancelled) {
-        setIssues(data);
+      if (!client) {
         setLoading(false);
+        return;
+      }
+      try {
+        const data = await client.getIssues(category);
+        if (!cancelled) {
+          setIssues(data);
+        }
+      } catch (err) {
+        console.error("BugBear: Failed to load issues", err);
+      } finally {
+        if (!cancelled) {
+          setLoading(false);
+        }
       }
     })();
     return () => {

package/dist/index.mjs

@@ -11796,7 +11796,13 @@ var BugBearClient = class {
         console.error("BugBear: Failed to fetch assignments", formatPgError(error));
         return [];
       }
-      const mapped = (data || []).map((item) => ({
+      const mapped = (data || []).filter((item) => {
+        if (!item.test_case) {
+          console.warn("BugBear: Assignment returned without test_case", { id: item.id });
+          return false;
+        }
+        return true;
+      }).map((item) => ({
         id: item.id,
         status: item.status,
         startedAt: item.started_at,
@@ -12053,6 +12059,16 @@ var BugBearClient = class {
       if (feedback.rating < 1 || feedback.rating > 5) {
         return { success: false, error: "Rating must be between 1 and 5" };
       }
+      const optionalRatings = [
+        { name: "clarityRating", value: feedback.clarityRating },
+        { name: "stepsRating", value: feedback.stepsRating },
+        { name: "relevanceRating", value: feedback.relevanceRating }
+      ];
+      for (const { name, value } of optionalRatings) {
+        if (value !== void 0 && value !== null && (value < 1 || value > 5)) {
+          return { success: false, error: `${name} must be between 1 and 5` };
+        }
+      }
       const { error: feedbackError } = await this.supabase.from("test_feedback").insert({
         project_id: this.config.projectId,
         test_case_id: testCaseId,
@@ -14630,6 +14646,7 @@ function TestFeedbackScreen({ status, assignmentId, nav }) {
   const assignment = assignments.find((a) => a.id === assignmentId);
   const showFlags = rating < 4;
   const handleSubmit = async () => {
+    if (submitting || images.isUploading) return;
     setSubmitting(true);
     if (client && assignment) {
       const screenshotUrls = images.getScreenshotUrls();
@@ -14920,7 +14937,7 @@ function ReportScreen({ nav, prefill }) {
     }
   }, [reportType]);
   const handleSubmit = async () => {
-    if (!client || !description.trim()) return;
+    if (!client || !description.trim() || images.isUploading) return;
     if (submittingRef.current) return;
     submittingRef.current = true;
     setSubmitting(true);
@@ -15168,15 +15185,28 @@ function ThreadDetailScreen({ thread, nav }) {
   const [sendError, setSendError] = useState8(false);
   const replyImages = useImageAttachments(uploadImage, 3, "discussion-attachments");
   useEffect7(() => {
+    let cancelled = false;
+    setLoading(true);
     (async () => {
-      setLoading(true);
-      const msgs = await getThreadMessages(thread.id);
-      setMessages(msgs);
-      setLoading(false);
-      if (thread.unreadCount > 0) {
-        await markAsRead(thread.id);
+      try {
+        const msgs = await getThreadMessages(thread.id);
+        if (!cancelled) {
+          setMessages(msgs);
+        }
+        if (thread.unreadCount > 0) {
+          await markAsRead(thread.id);
+        }
+      } catch (err) {
+        console.error("BugBear: Failed to load thread messages", err);
+      } finally {
+        if (!cancelled) {
+          setLoading(false);
+        }
       }
     })();
+    return () => {
+      cancelled = true;
+    };
   }, [thread.id]);
   const handleSend = async () => {
     if (!replyText.trim() && replyImages.images.length === 0 || sending || replyImages.isUploading) return;
@@ -15223,9 +15253,9 @@ function ThreadDetailScreen({ thread, nav }) {
   ), /* @__PURE__ */ React12.createElement(
     TouchableOpacity10,
     {
-      style: [styles11.sendBtn, (!replyText.trim() || sending || replyImages.isUploading) && styles11.sendBtnDisabled],
+      style: [styles11.sendBtn, (!replyText.trim() && replyImages.images.length === 0 || sending || replyImages.isUploading) && styles11.sendBtnDisabled],
       onPress: handleSend,
-      disabled: !replyText.trim() || sending || replyImages.isUploading
+      disabled: !replyText.trim() && replyImages.images.length === 0 || sending || replyImages.isUploading
     },
     /* @__PURE__ */ React12.createElement(Text11, { style: styles11.sendBtnText }, sending ? "..." : "Send")
   )));
@@ -15271,7 +15301,7 @@ function ComposeMessageScreen({ nav }) {
   const [sending, setSending] = useState9(false);
   const images = useImageAttachments(uploadImage, 3, "discussion-attachments");
   const handleSend = async () => {
-    if (!subject.trim() || !message.trim()) return;
+    if (!subject.trim() || !message.trim() || sending || images.isUploading) return;
     setSending(true);
     const attachments = images.getAttachments();
     const result = await createThread({
@@ -15357,6 +15387,7 @@ function ProfileScreen({ nav }) {
     }
   }, [testerInfo]);
   const handleSave = async () => {
+    if (saving) return;
     setSaving(true);
     const updates = {
       name: name.trim(),
@@ -15485,11 +15516,21 @@ function IssueListScreen({ nav, category }) {
     let cancelled = false;
     setLoading(true);
     (async () => {
-      if (!client) return;
-      const data = await client.getIssues(category);
-      if (!cancelled) {
-        setIssues(data);
+      if (!client) {
         setLoading(false);
+        return;
+      }
+      try {
+        const data = await client.getIssues(category);
+        if (!cancelled) {
+          setIssues(data);
+        }
+      } catch (err) {
+        console.error("BugBear: Failed to load issues", err);
+      } finally {
+        if (!cancelled) {
+          setLoading(false);
+        }
       }
     })();
     return () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bbearai/react-native",
-  "version": "0.5.6",
+  "version": "0.5.7",
   "description": "BugBear React Native components for mobile apps",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",