fix(tools): restore data.error fallback with OAuth guard to avoid regression

Add data.error string check back to atlassian-errors extractor (needed for
all Jira/Confluence proxy routes that return { error: "..." }), but skip it
when error_description is present to avoid preempting OAuth error extraction
in the backwards-compat loop. Revert JSM Forms routes to original error field.

Author: waleedlatif1

apps/sim/app/api/tools/jsm/forms/issue/route.ts

@@ -70,8 +70,8 @@ export async function POST(request: NextRequest) {
 
       return NextResponse.json(
         {
-          message: parseJsmErrorMessage(response.status, response.statusText, errorText),
           error: parseJsmErrorMessage(response.status, response.statusText, errorText),
+          details: errorText,
         },
         { status: response.status }
       )

apps/sim/app/api/tools/jsm/forms/structure/route.ts

@@ -80,8 +80,8 @@ export async function POST(request: NextRequest) {
 
       return NextResponse.json(
         {
-          message: parseJsmErrorMessage(response.status, response.statusText, errorText),
           error: parseJsmErrorMessage(response.status, response.statusText, errorText),
+          details: errorText,
         },
         { status: response.status }
       )

apps/sim/app/api/tools/jsm/forms/templates/route.ts

@@ -70,8 +70,8 @@ export async function POST(request: NextRequest) {
 
       return NextResponse.json(
         {
-          message: parseJsmErrorMessage(response.status, response.statusText, errorText),
           error: parseJsmErrorMessage(response.status, response.statusText, errorText),
+          details: errorText,
         },
         { status: response.status }
       )

apps/sim/tools/error-extractors.ts

@@ -74,6 +74,10 @@ const ERROR_EXTRACTORS: ErrorExtractorConfig[] = [
       if (errorInfo?.data?.message) {
         return errorInfo.data.message
       }
+      // Proxy route error field fallback (skip if error_description present to avoid preempting OAuth)
+      if (typeof errorInfo?.data?.error === 'string' && !errorInfo?.data?.error_description) {
+        return errorInfo.data.error
+      }
       return undefined
     },
   },