fix(github): guard new tools against non-2xx GitHub responses

list_tags, get_readme, get_latest_release, and create_pr_review
(v1 + v2) now check response.ok before parsing the payload as
success data, returning success:false with a real error message
instead of crashing on .map() or silently returning undefined
fields when GitHub returns a 404/422/etc.

Author: waleedlatif1

apps/sim/tools/github/create_pr_review.ts

@@ -74,6 +74,18 @@ export const createPRReviewTool: ToolConfig<CreatePRReviewParams, PRReviewRespon
   },
 
   transformResponse: async (response) => {
+    if (!response.ok) {
+      const error = await response.json().catch(() => ({}))
+      return {
+        success: false,
+        error: error.message || `Failed to submit PR review (HTTP ${response.status})`,
+        output: {
+          content: '',
+          metadata: { id: 0, state: '', body: '', html_url: '', commit_id: '' },
+        },
+      }
+    }
+
     const review = await response.json()
 
     const content = `Review submitted for PR #${review.pull_request_url?.split('/').pop() ?? ''}
@@ -123,6 +135,24 @@ export const createPRReviewV2Tool: ToolConfig<CreatePRReviewParams, any> = {
   request: createPRReviewTool.request,
 
   transformResponse: async (response: Response) => {
+    if (!response.ok) {
+      const error = await response.json().catch(() => ({}))
+      return {
+        success: false,
+        error: error.message || `Failed to submit PR review (HTTP ${response.status})`,
+        output: {
+          id: 0,
+          user: null,
+          body: null,
+          state: '',
+          html_url: '',
+          pull_request_url: '',
+          commit_id: '',
+          submitted_at: null,
+        },
+      }
+    }
+
     const review = await response.json()
     return {
       success: true,

apps/sim/tools/github/get_latest_release.ts

@@ -45,6 +45,29 @@ export const getLatestReleaseTool: ToolConfig<GetLatestReleaseParams, ReleaseRes
   },
 
   transformResponse: async (response) => {
+    if (!response.ok) {
+      const error = await response.json().catch(() => ({}))
+      return {
+        success: false,
+        error: error.message || `Failed to get latest release (HTTP ${response.status})`,
+        output: {
+          content: '',
+          metadata: {
+            id: 0,
+            tag_name: '',
+            name: '',
+            html_url: '',
+            tarball_url: '',
+            zipball_url: '',
+            draft: false,
+            prerelease: false,
+            created_at: '',
+            published_at: '',
+          },
+        },
+      }
+    }
+
     const data = await response.json()
 
     const assetsInfo =
@@ -116,6 +139,30 @@ export const getLatestReleaseV2Tool: ToolConfig<GetLatestReleaseParams, any> = {
   request: getLatestReleaseTool.request,
 
   transformResponse: async (response: Response) => {
+    if (!response.ok) {
+      const error = await response.json().catch(() => ({}))
+      return {
+        success: false,
+        error: error.message || `Failed to get latest release (HTTP ${response.status})`,
+        output: {
+          id: 0,
+          tag_name: '',
+          name: '',
+          body: null,
+          html_url: '',
+          tarball_url: '',
+          zipball_url: '',
+          draft: false,
+          prerelease: false,
+          author: null,
+          assets: [],
+          created_at: '',
+          published_at: null,
+          target_commitish: '',
+        },
+      }
+    }
+
     const data = await response.json()
     return {
       success: true,

apps/sim/tools/github/get_readme.ts

@@ -50,6 +50,18 @@ export const getReadmeTool: ToolConfig<GetReadmeParams, ReadmeResponse> = {
   },
 
   transformResponse: async (response) => {
+    if (!response.ok) {
+      const error = await response.json().catch(() => ({}))
+      return {
+        success: false,
+        error: error.message || `Failed to get README (HTTP ${response.status})`,
+        output: {
+          content: '',
+          metadata: { name: '', path: '', sha: '', size: 0, html_url: '', download_url: '' },
+        },
+      }
+    }
+
     const data = await response.json()
 
     let decodedContent = ''
@@ -109,6 +121,24 @@ export const getReadmeV2Tool: ToolConfig<GetReadmeParams, any> = {
   request: getReadmeTool.request,
 
   transformResponse: async (response: Response) => {
+    if (!response.ok) {
+      const error = await response.json().catch(() => ({}))
+      return {
+        success: false,
+        error: error.message || `Failed to get README (HTTP ${response.status})`,
+        output: {
+          name: '',
+          path: '',
+          sha: '',
+          size: 0,
+          encoding: '',
+          html_url: '',
+          download_url: null,
+          content: '',
+        },
+      }
+    }
+
     const data = await response.json()
 
     let decodedContent = ''

apps/sim/tools/github/list_tags.ts

@@ -63,6 +63,15 @@ export const listTagsTool: ToolConfig<ListTagsParams, TagsListResponse> = {
   },
 
   transformResponse: async (response) => {
+    if (!response.ok) {
+      const error = await response.json().catch(() => ({}))
+      return {
+        success: false,
+        error: error.message || `Failed to list tags (HTTP ${response.status})`,
+        output: { content: '', metadata: { total_count: 0, tags: [] } },
+      }
+    }
+
     const tags = await response.json()
 
     const tagsList = tags
@@ -126,6 +135,15 @@ export const listTagsV2Tool: ToolConfig<ListTagsParams, any> = {
   request: listTagsTool.request,
 
   transformResponse: async (response: Response) => {
+    if (!response.ok) {
+      const error = await response.json().catch(() => ({}))
+      return {
+        success: false,
+        error: error.message || `Failed to list tags (HTTP ${response.status})`,
+        output: { items: [], count: 0 },
+      }
+    }
+
     const tags = await response.json()
     return {
       success: true,