fix(x): add error handling and logger to all X tools

Author: waleedlatif1

apps/sim/tools/x/create_bookmark.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import type { ToolConfig } from '@/tools/types'
 import type { XCreateBookmarkParams, XCreateBookmarkResponse } from '@/tools/x/types'
 
+const logger = createLogger('XCreateBookmarkTool')
+
 export const xCreateBookmarkTool: ToolConfig<XCreateBookmarkParams, XCreateBookmarkResponse> = {
   id: 'x_create_bookmark',
   name: 'X Create Bookmark',
@@ -47,10 +50,22 @@ export const xCreateBookmarkTool: ToolConfig<XCreateBookmarkParams, XCreateBookm
 
   transformResponse: async (response) => {
     const data = await response.json()
+
+    if (!data.data) {
+      logger.error('X Create Bookmark API Error:', JSON.stringify(data, null, 2))
+      return {
+        success: false,
+        error: data.errors?.[0]?.detail || 'Failed to bookmark tweet',
+        output: {
+          bookmarked: false,
+        },
+      }
+    }
+
     return {
       success: true,
       output: {
-        bookmarked: data.data?.bookmarked ?? false,
+        bookmarked: data.data.bookmarked ?? false,
       },
     }
   },

apps/sim/tools/x/create_tweet.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import type { ToolConfig } from '@/tools/types'
 import type { XCreateTweetParams, XCreateTweetResponse } from '@/tools/x/types'
 
+const logger = createLogger('XCreateTweetTool')
+
 export const xCreateTweetTool: ToolConfig<XCreateTweetParams, XCreateTweetResponse> = {
   id: 'x_create_tweet',
   name: 'X Create Tweet',
@@ -93,6 +96,7 @@ export const xCreateTweetTool: ToolConfig<XCreateTweetParams, XCreateTweetRespon
     const data = await response.json()
 
     if (!data.data) {
+      logger.error('X Create Tweet API Error:', JSON.stringify(data, null, 2))
       return {
         success: false,
         error: data.errors?.[0]?.detail || 'Failed to create tweet',

apps/sim/tools/x/delete_bookmark.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import type { ToolConfig } from '@/tools/types'
 import type { XDeleteBookmarkParams, XDeleteBookmarkResponse } from '@/tools/x/types'
 
+const logger = createLogger('XDeleteBookmarkTool')
+
 export const xDeleteBookmarkTool: ToolConfig<XDeleteBookmarkParams, XDeleteBookmarkResponse> = {
   id: 'x_delete_bookmark',
   name: 'X Delete Bookmark',
@@ -45,10 +48,22 @@ export const xDeleteBookmarkTool: ToolConfig<XDeleteBookmarkParams, XDeleteBookm
 
   transformResponse: async (response) => {
     const data = await response.json()
+
+    if (!data.data) {
+      logger.error('X Delete Bookmark API Error:', JSON.stringify(data, null, 2))
+      return {
+        success: false,
+        error: data.errors?.[0]?.detail || 'Failed to remove bookmark',
+        output: {
+          bookmarked: false,
+        },
+      }
+    }
+
     return {
       success: true,
       output: {
-        bookmarked: data.data?.bookmarked ?? false,
+        bookmarked: data.data.bookmarked ?? false,
       },
     }
   },

apps/sim/tools/x/delete_tweet.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import type { ToolConfig } from '@/tools/types'
 import type { XDeleteTweetParams, XDeleteTweetResponse } from '@/tools/x/types'
 
+const logger = createLogger('XDeleteTweetTool')
+
 export const xDeleteTweetTool: ToolConfig<XDeleteTweetParams, XDeleteTweetResponse> = {
   id: 'x_delete_tweet',
   name: 'X Delete Tweet',
@@ -40,6 +43,7 @@ export const xDeleteTweetTool: ToolConfig<XDeleteTweetParams, XDeleteTweetRespon
     const data = await response.json()
 
     if (!data.data) {
+      logger.error('X Delete Tweet API Error:', JSON.stringify(data, null, 2))
       return {
         success: false,
         error: data.errors?.[0]?.detail || 'Failed to delete tweet',

apps/sim/tools/x/manage_block.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import type { ToolConfig } from '@/tools/types'
 import type { XManageBlockParams, XManageBlockResponse } from '@/tools/x/types'
 
+const logger = createLogger('XManageBlockTool')
+
 export const xManageBlockTool: ToolConfig<XManageBlockParams, XManageBlockResponse> = {
   id: 'x_manage_block',
   name: 'X Manage Block',
@@ -63,6 +66,7 @@ export const xManageBlockTool: ToolConfig<XManageBlockParams, XManageBlockRespon
     const data = await response.json()
 
     if (!data.data) {
+      logger.error('X Manage Block API Error:', JSON.stringify(data, null, 2))
       return {
         success: false,
         output: {

apps/sim/tools/x/manage_follow.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import type { ToolConfig } from '@/tools/types'
 import type { XManageFollowParams, XManageFollowResponse } from '@/tools/x/types'
 
+const logger = createLogger('XManageFollowTool')
+
 export const xManageFollowTool: ToolConfig<XManageFollowParams, XManageFollowResponse> = {
   id: 'x_manage_follow',
   name: 'X Manage Follow',
@@ -63,6 +66,7 @@ export const xManageFollowTool: ToolConfig<XManageFollowParams, XManageFollowRes
     const data = await response.json()
 
     if (!data.data) {
+      logger.error('X Manage Follow API Error:', JSON.stringify(data, null, 2))
       return {
         success: false,
         output: {

apps/sim/tools/x/manage_like.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import type { ToolConfig } from '@/tools/types'
 import type { XManageLikeParams, XManageLikeResponse } from '@/tools/x/types'
 
+const logger = createLogger('XManageLikeTool')
+
 export const xManageLikeTool: ToolConfig<XManageLikeParams, XManageLikeResponse> = {
   id: 'x_manage_like',
   name: 'X Manage Like',
@@ -63,6 +66,7 @@ export const xManageLikeTool: ToolConfig<XManageLikeParams, XManageLikeResponse>
     const data = await response.json()
 
     if (!data.data) {
+      logger.error('X Manage Like API Error:', JSON.stringify(data, null, 2))
       return {
         success: false,
         output: {

apps/sim/tools/x/manage_retweet.ts

@@ -1,6 +1,9 @@
+import { createLogger } from '@sim/logger'
 import type { ToolConfig } from '@/tools/types'
 import type { XManageRetweetParams, XManageRetweetResponse } from '@/tools/x/types'
 
+const logger = createLogger('XManageRetweetTool')
+
 export const xManageRetweetTool: ToolConfig<XManageRetweetParams, XManageRetweetResponse> = {
   id: 'x_manage_retweet',
   name: 'X Manage Retweet',
@@ -63,6 +66,7 @@ export const xManageRetweetTool: ToolConfig<XManageRetweetParams, XManageRetweet
     const data = await response.json()
 
     if (!data.data) {
+      logger.error('X Manage Retweet API Error:', JSON.stringify(data, null, 2))
       return {
         success: false,
         output: {