fix(slack): reconnect surfaces Atlassian error codes and persists name/description edits

- PUT credential route now returns the Atlassian provider code (providerErrorCode
  -> code) so reconnect failures map to specific token/domain messages, matching create
- Google/Atlassian reconnect send + seed displayName/description (parity with Slack);
  edits are no longer silently discarded, and empty fields don't clobber existing values

Author: TheodoreSpeaks

apps/sim/app/api/credentials/[id]/route.ts

@@ -99,7 +99,13 @@ export const PUT = withRouteHandler(
                 : result.errorCode === 'validation'
                   ? 400
                   : 500
-        return NextResponse.json({ error: result.error }, { status })
+        return NextResponse.json(
+          {
+            error: result.error,
+            ...(result.providerErrorCode ? { code: result.providerErrorCode } : {}),
+          },
+          { status }
+        )
       }
 
       const access = await getCredentialActorContext(id, session.user.id)

apps/sim/app/workspace/[workspaceId]/integrations/components/connect-service-account-modal/connect-service-account-modal.tsx

@@ -142,6 +142,8 @@ export function ConnectServiceAccountModal({
         serviceName={serviceName}
         serviceIcon={serviceIcon}
         credentialId={credentialId}
+        initialDisplayName={credentialDisplayName}
+        initialDescription={credentialDescription}
       />
     )
   }
@@ -153,6 +155,8 @@ export function ConnectServiceAccountModal({
       serviceName={serviceName}
       serviceIcon={serviceIcon}
       credentialId={credentialId}
+      initialDisplayName={credentialDisplayName}
+      initialDescription={credentialDescription}
     />
   )
 }
@@ -165,6 +169,9 @@ interface ProviderModalProps {
   serviceIcon: ComponentType<{ className?: string }>
   /** When set, reconnect (rotate secrets on) this credential in place. */
   credentialId?: string
+  /** Existing name/description, seeded into the fields on reconnect. */
+  initialDisplayName?: string
+  initialDescription?: string
 }
 
 /**
@@ -179,11 +186,13 @@ function GoogleServiceAccountModal({
   serviceName,
   serviceIcon: ServiceIcon,
   credentialId,
+  initialDisplayName,
+  initialDescription,
 }: ProviderModalProps) {
   const [jsonInput, setJsonInput] = useState('')
   const [uploadedFileName, setUploadedFileName] = useState<string | null>(null)
-  const [displayName, setDisplayName] = useState('')
-  const [description, setDescription] = useState('')
+  const [displayName, setDisplayName] = useState(initialDisplayName ?? '')
+  const [description, setDescription] = useState(initialDescription ?? '')
   const [error, setError] = useState<string | null>(null)
 
   const createCredential = useCreateWorkspaceCredential()
@@ -193,10 +202,10 @@ function GoogleServiceAccountModal({
     if (open) return
     setJsonInput('')
     setUploadedFileName(null)
-    setDisplayName('')
-    setDescription('')
+    setDisplayName(initialDisplayName ?? '')
+    setDescription(initialDescription ?? '')
     setError(null)
-  }, [open])
+  }, [open, initialDisplayName, initialDescription])
 
   /**
    * Try to auto-populate display name from the JSON `client_email`. Silent on
@@ -249,7 +258,12 @@ function GoogleServiceAccountModal({
     }
     try {
       if (credentialId) {
-        await updateCredential.mutateAsync({ credentialId, serviceAccountJson: trimmed })
+        await updateCredential.mutateAsync({
+          credentialId,
+          serviceAccountJson: trimmed,
+          displayName: displayName.trim() || undefined,
+          description: description.trim() || undefined,
+        })
       } else {
         await createCredential.mutateAsync({
           workspaceId,
@@ -359,11 +373,13 @@ function AtlassianServiceAccountModal({
   serviceName,
   serviceIcon: ServiceIcon,
   credentialId,
+  initialDisplayName,
+  initialDescription,
 }: ProviderModalProps) {
   const [apiToken, setApiToken] = useState('')
   const [domain, setDomain] = useState('')
-  const [displayName, setDisplayName] = useState('')
-  const [description, setDescription] = useState('')
+  const [displayName, setDisplayName] = useState(initialDisplayName ?? '')
+  const [description, setDescription] = useState(initialDescription ?? '')
   const [error, setError] = useState<string | null>(null)
 
   const createCredential = useCreateWorkspaceCredential()
@@ -373,10 +389,10 @@ function AtlassianServiceAccountModal({
     if (open) return
     setApiToken('')
     setDomain('')
-    setDisplayName('')
-    setDescription('')
+    setDisplayName(initialDisplayName ?? '')
+    setDescription(initialDescription ?? '')
     setError(null)
-  }, [open])
+  }, [open, initialDisplayName, initialDescription])
 
   const trimmedToken = apiToken.trim()
   const normalizedDomain = normalizeAtlassianDomain(domain)
@@ -395,6 +411,8 @@ function AtlassianServiceAccountModal({
           credentialId,
           apiToken: trimmedToken,
           domain: normalizedDomain,
+          displayName: displayName.trim() || undefined,
+          description: description.trim() || undefined,
         })
       } else {
         await createCredential.mutateAsync({

apps/sim/lib/credentials/orchestration/index.ts

@@ -54,6 +54,8 @@ export interface PerformCredentialResult {
   success: boolean
   error?: string
   errorCode?: CredentialOrchestrationErrorCode
+  /** Provider-specific code (e.g. Atlassian `invalid_credentials`) for client message mapping. */
+  providerErrorCode?: string
   workspaceId?: string
   updatedFields?: string[]
 }
@@ -139,7 +141,14 @@ export async function performUpdateCredential(
           return { success: false, error: error.message, errorCode: 'validation' }
         }
         if (error instanceof AtlassianValidationError) {
-          return { success: false, error: error.code, errorCode: 'validation' }
+          // Surface the provider code so the client maps it to the specific
+          // token/domain message (create returns it too).
+          return {
+            success: false,
+            error: error.code,
+            errorCode: 'validation',
+            providerErrorCode: error.code,
+          }
         }
         throw error
       }