Anshumancanrock
diff --git a/‎.github/workflows/cubic-devin-review.yml‎
Lines changed: 8 additions & 2 deletions b/‎.github/workflows/cubic-devin-review.yml‎
Lines changed: 8 additions & 2 deletions
diff --git a/‎.github/workflows/devin-conflict-resolver.yml‎
Lines changed: 2 additions & 2 deletions b/‎.github/workflows/devin-conflict-resolver.yml‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎.github/workflows/release-docker.yaml‎
Lines changed: 4 additions & 2 deletions b/‎.github/workflows/release-docker.yaml‎
Lines changed: 4 additions & 2 deletions
diff --git a/‎apps/web/public/static/locales/en/common.json‎
Lines changed: 3 additions & 3 deletions b/‎apps/web/public/static/locales/en/common.json‎
Lines changed: 3 additions & 3 deletions
diff --git a/‎companion/app/(tabs)/(bookings)/booking-detail.ios.tsx‎
Lines changed: 4 additions & 4 deletions b/‎companion/app/(tabs)/(bookings)/booking-detail.ios.tsx‎
Lines changed: 4 additions & 4 deletions
diff --git a/‎companion/app/(tabs)/(bookings)/booking-detail.tsx‎
Lines changed: 9 additions & 9 deletions b/‎companion/app/(tabs)/(bookings)/booking-detail.tsx‎
Lines changed: 9 additions & 9 deletions
@@ -187,7 +187,7 @@ jobs:
           4. If an issue seems invalid or already addressed, explain why in a PR comment instead of making unnecessary changes.
           5. Never ask for user confirmation. Never wait for user messages."
 
-          RESPONSE=$(curl -s -X POST "https://api.devin.ai/v1/sessions" \
+          HTTP_CODE=$(curl -s -o /tmp/devin-response.json -w "%{http_code}" -X POST "https://api.devin.ai/v1/sessions" \
             -H "Authorization: Bearer ${DEVIN_API_KEY}" \
             -H "Content-Type: application/json" \
             -d "$(jq -n \
@@ -199,7 +199,13 @@ jobs:
                 tags: ["cubic-ai-review", "pr-${{ github.event.pull_request.number }}"]
               }')")
           
-          SESSION_URL=$(echo "$RESPONSE" | jq -r '.url // .session_url // empty')
+          if [ "$HTTP_CODE" -lt 200 ] || [ "$HTTP_CODE" -ge 300 ]; then
+            echo "Failed to create Devin session: HTTP $HTTP_CODE"
+            cat /tmp/devin-response.json
+            exit 1
+          fi
+          
+          SESSION_URL=$(cat /tmp/devin-response.json | jq -r '.url // .session_url // empty')
           if [ -n "$SESSION_URL" ]; then
             echo "Devin session created: $SESSION_URL"
             echo "SESSION_URL=$SESSION_URL" >> $GITHUB_ENV
 
@@ -314,10 +314,10 @@ jobs:
             If you prefer to resolve conflicts manually, you can close the Devin session and handle it yourself.`
                   });
                 } else {
-                  console.log(`Failed to get session URL for PR #${pr.number}:`, data);
+                  console.log(`Failed to get session URL for PR #${pr.number}`);
                 }
               } catch (error) {
-                console.error(`Error creating Devin session for PR #${pr.number}:`, error);
+                console.error(`Error creating Devin session for PR #${pr.number}: ${error.message}`);
               }
               
               await new Promise(resolve => setTimeout(resolve, 1000));
 
@@ -65,8 +65,10 @@ jobs:
             if [ "${{ inputs.BUILD_FROM_BRANCH }}" = "true" ]; then
               # Extract branch name and short SHA for the tag
               BRANCH_NAME="${GITHUB_REF#refs/heads/}"
-              # Sanitize branch name: replace slashes with dashes for valid Docker tag
-              SANITIZED_BRANCH=$(echo "$BRANCH_NAME" | tr '/' '-')
+              # Sanitize branch name: replace all invalid Docker tag characters with dashes
+              # Docker tags can only contain lowercase/uppercase letters, digits, underscores, periods, and dashes
+              # Also remove any leading dashes or periods
+              SANITIZED_BRANCH=$(echo "$BRANCH_NAME" | sed 's/[^a-zA-Z0-9_.-]/-/g' | sed 's/^[-.]*//')
               SHORT_SHA=$(git rev-parse --short HEAD)
               echo "release_tag=${SANITIZED_BRANCH}-${SHORT_SHA}" >> $GITHUB_OUTPUT
             else
 
@@ -2878,9 +2878,9 @@
   "time_range": "Time range",
   "automatically_add_all_team_members": "Add all team members, including future members",
   "redirect_to": "Redirect to",
-  "having_trouble_finding_time": "Having trouble finding a time?",
-  "show_more": "Show more",
-  "forward_params_redirect": "Forward parameters such as ?email=...&name=.... and more",
+    "having_trouble_finding_time": "Having trouble finding a time?",
+    "show_more": "Show more",
+    "forward_params_redirect": "Forward parameters such as ?email=...&name=.... and more",
   "assignment_description": "Schedule meetings when everyone is available or rotate through members on your team",
   "lowest": "lowest",
   "low": "low",
 
@@ -1,11 +1,11 @@
 import * as Clipboard from "expo-clipboard";
 import { Stack, useLocalSearchParams } from "expo-router";
 import { useCallback, useMemo, useRef } from "react";
-import { Alert } from "react-native";
 import { BookingDetailScreen } from "@/components/screens/BookingDetailScreen";
 import { useAuth } from "@/contexts/AuthContext";
 import { useBookingByUid } from "@/hooks/useBookings";
 import type { Booking } from "@/services/calcom";
+import { showErrorAlert, showInfoAlert, showSuccessAlert } from "@/utils/alerts";
 import { type BookingActionsResult, getBookingActions } from "@/utils/booking-actions";
 import { openInAppBrowser } from "@/utils/browser";
 
@@ -94,7 +94,7 @@ export default function BookingDetailIOS() {
   const handleCopyMeetingLink = useCallback(async () => {
     if (meetingUrl) {
       await Clipboard.setStringAsync(meetingUrl);
-      Alert.alert("Copied", "Meeting link copied to clipboard");
+      showSuccessAlert("Copied", "Meeting link copied to clipboard");
     }
   }, [meetingUrl]);
 
@@ -117,7 +117,7 @@ export default function BookingDetailIOS() {
     if (handlers?.[handlerName]) {
       (handlers[handlerName] as () => void)();
     } else {
-      Alert.alert("Error", errorMessage);
+      showErrorAlert("Error", errorMessage);
     }
   }, []);
 
@@ -187,7 +187,7 @@ export default function BookingDetailIOS() {
         handlerName: null,
         errorMessage: null,
         customHandler: () => {
-          Alert.alert("Report Booking", "Report booking functionality is not yet available");
+          showInfoAlert("Report Booking", "Report booking functionality is not yet available");
         },
         destructive: true,
         gatingKey: null,
 
@@ -1,9 +1,9 @@
 import { Stack, useLocalSearchParams } from "expo-router";
 import { useCallback, useMemo, useRef } from "react";
-import { Alert } from "react-native";
 import { BookingDetailScreen } from "@/components/screens/BookingDetailScreen";
 import { useAuth } from "@/contexts/AuthContext";
 import { useBookingByUid } from "@/hooks/useBookings";
+import { showErrorAlert, showInfoAlert } from "@/utils/alerts";
 import { type BookingActionsResult, getBookingActions } from "@/utils/booking-actions";
 
 // Empty actions result for when no booking is loaded
@@ -64,59 +64,59 @@ export default function BookingDetail() {
     if (actionHandlersRef.current?.openRescheduleModal) {
       actionHandlersRef.current.openRescheduleModal();
     } else {
-      Alert.alert("Error", "Unable to reschedule. Please try again.");
+      showErrorAlert("Error", "Unable to reschedule. Please try again.");
     }
   }, []);
 
   const handleEditLocation = useCallback(() => {
     if (actionHandlersRef.current?.openEditLocationModal) {
       actionHandlersRef.current.openEditLocationModal();
     } else {
-      Alert.alert("Error", "Unable to edit location. Please try again.");
+      showErrorAlert("Error", "Unable to edit location. Please try again.");
     }
   }, []);
 
   const handleAddGuests = useCallback(() => {
     if (actionHandlersRef.current?.openAddGuestsModal) {
       actionHandlersRef.current.openAddGuestsModal();
     } else {
-      Alert.alert("Error", "Unable to add guests. Please try again.");
+      showErrorAlert("Error", "Unable to add guests. Please try again.");
     }
   }, []);
 
   const handleViewRecordings = useCallback(() => {
     if (actionHandlersRef.current?.openViewRecordingsModal) {
       actionHandlersRef.current.openViewRecordingsModal();
     } else {
-      Alert.alert("Error", "Unable to view recordings. Please try again.");
+      showErrorAlert("Error", "Unable to view recordings. Please try again.");
     }
   }, []);
 
   const handleSessionDetails = useCallback(() => {
     if (actionHandlersRef.current?.openMeetingSessionDetailsModal) {
       actionHandlersRef.current.openMeetingSessionDetailsModal();
     } else {
-      Alert.alert("Error", "Unable to view session details. Please try again.");
+      showErrorAlert("Error", "Unable to view session details. Please try again.");
     }
   }, []);
 
   const handleMarkNoShow = useCallback(() => {
     if (actionHandlersRef.current?.openMarkNoShowModal) {
       actionHandlersRef.current.openMarkNoShowModal();
     } else {
-      Alert.alert("Error", "Unable to mark no-show. Please try again.");
+      showErrorAlert("Error", "Unable to mark no-show. Please try again.");
     }
   }, []);
 
   const handleReport = useCallback(() => {
-    Alert.alert("Report Booking", "Report booking functionality is not yet available");
+    showInfoAlert("Report Booking", "Report booking functionality is not yet available");
   }, []);
 
   const handleCancel = useCallback(() => {
     if (actionHandlersRef.current?.handleCancelBooking) {
       actionHandlersRef.current.handleCancelBooking();
     } else {
-      Alert.alert("Error", "Unable to cancel. Please try again.");
+      showErrorAlert("Error", "Unable to cancel. Please try again.");
     }
   }, []);
Original file line number	Diff line number	Diff line change
`@@ -314,10 +314,10 @@ jobs:`
`314`	`314`	If you prefer to resolve conflicts manually, you can close the Devin session and handle it yourself.`
`315`	`315`	`});`
`316`	`316`	`} else {`
`317`		- console.log(`Failed to get session URL for PR #${pr.number}:`, data);
	`317`	+ console.log(`Failed to get session URL for PR #${pr.number}`);
`318`	`318`	`}`
`319`	`319`	`} catch (error) {`
`320`		- console.error(`Error creating Devin session for PR #${pr.number}:`, error);
	`320`	+ console.error(`Error creating Devin session for PR #${pr.number}: ${error.message}`);
`321`	`321`	`}`
`322`	`322`
`323`	`323`	`await new Promise(resolve => setTimeout(resolve, 1000));`