fix(cursor): address PR review feedback

- Remove redundant Array.isArray guards in list_artifacts.ts
- Pass through actual HTTP status on presigned URL download failure instead of hardcoded 400

Author: waleedlatif1

apps/sim/app/api/tools/cursor/download-artifact/route.ts

@@ -103,8 +103,11 @@ export async function POST(request: NextRequest) {
         statusText: downloadResponse.statusText,
       })
       return NextResponse.json(
-        { success: false, error: 'Failed to download artifact content' },
-        { status: 400 }
+        {
+          success: false,
+          error: `Failed to download artifact content (${downloadResponse.status}: ${downloadResponse.statusText})`,
+        },
+        { status: downloadResponse.status }
       )
     }
 

apps/sim/tools/cursor/list_artifacts.ts

@@ -41,9 +41,9 @@ export const listArtifactsTool: ToolConfig<ListArtifactsParams, ListArtifactsRes
     return {
       success: true,
       output: {
-        content: `Found ${Array.isArray(artifacts) ? artifacts.length : 0} artifact(s)`,
+        content: `Found ${artifacts.length} artifact(s)`,
         metadata: {
-          artifacts: Array.isArray(artifacts) ? artifacts : [],
+          artifacts,
         },
       },
     }