fix(workday): coerce SOAP scalar strings to typed booleans/numbers

- XML parser returns leaf text as strings; `!"false"` evaluated to
  `false`, causing all organizations to report `isActive: false`
- Add parseSoapBoolean and parseSoapNumber helpers and apply at consumer
  sites (Inactive, Total_Results)
- Drop unused service/soapAction fields from WD_OPERATIONS map

Author: waleedlatif1

apps/sim/app/api/tools/workday/get-organizations/route.ts

@@ -9,6 +9,8 @@ import {
   createWorkdaySoapClient,
   extractRefId,
   normalizeSoapArray,
+  parseSoapBoolean,
+  parseSoapNumber,
   type WorkdayOrganizationSoap,
 } from '@/tools/workday/soap'
 
@@ -63,15 +65,18 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
         | undefined
     )
 
-    const organizations = orgsArray.map((o) => ({
-      id: extractRefId(o.Organization_Reference) ?? null,
-      descriptor: o.Organization_Descriptor ?? null,
-      type: extractRefId(o.Organization_Data?.Organization_Type_Reference) ?? null,
-      subtype: extractRefId(o.Organization_Data?.Organization_Subtype_Reference) ?? null,
-      isActive: o.Organization_Data?.Inactive != null ? !o.Organization_Data.Inactive : null,
-    }))
+    const organizations = orgsArray.map((o) => {
+      const inactive = parseSoapBoolean(o.Organization_Data?.Inactive)
+      return {
+        id: extractRefId(o.Organization_Reference) ?? null,
+        descriptor: o.Organization_Descriptor ?? null,
+        type: extractRefId(o.Organization_Data?.Organization_Type_Reference) ?? null,
+        subtype: extractRefId(o.Organization_Data?.Organization_Subtype_Reference) ?? null,
+        isActive: inactive == null ? null : !inactive,
+      }
+    })
 
-    const total = result?.Response_Results?.Total_Results ?? organizations.length
+    const total = parseSoapNumber(result?.Response_Results?.Total_Results) ?? organizations.length
 
     return NextResponse.json({
       success: true,

apps/sim/app/api/tools/workday/list-workers/route.ts

@@ -9,6 +9,7 @@ import {
   createWorkdaySoapClient,
   extractRefId,
   normalizeSoapArray,
+  parseSoapNumber,
   type WorkdayWorkerSoap,
 } from '@/tools/workday/soap'
 
@@ -61,7 +62,7 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
       employmentData: w.Worker_Data?.Employment_Data ?? null,
     }))
 
-    const total = result?.Response_Results?.Total_Results ?? workers.length
+    const total = parseSoapNumber(result?.Response_Results?.Total_Results) ?? workers.length
 
     return NextResponse.json({
       success: true,

apps/sim/tools/workday/soap.ts

@@ -15,10 +15,10 @@ export type WorkdayServiceKey = keyof typeof WORKDAY_SERVICES
 export interface WorkdaySoapResult {
   Response_Data?: Record<string, unknown>
   Response_Results?: {
-    Total_Results?: number
-    Total_Pages?: number
-    Page_Results?: number
-    Page?: number
+    Total_Results?: number | string
+    Total_Pages?: number | string
+    Page_Results?: number | string
+    Page?: number | string
   }
   Event_Reference?: WorkdayReference
   Employee_Reference?: WorkdayReference
@@ -98,7 +98,7 @@ export interface WorkdayOrganizationSoap {
 export interface WorkdayOrganizationDataSoap {
   Organization_Type_Reference?: WorkdayReference
   Organization_Subtype_Reference?: WorkdayReference
-  Inactive?: boolean
+  Inactive?: boolean | string
 }
 
 /**
@@ -110,24 +110,51 @@ export function normalizeSoapArray<T>(value: T | T[] | undefined): T[] {
   return Array.isArray(value) ? value : [value]
 }
 
-const WD_OPERATIONS = {
-  Get_Workers: { service: 'humanResources', soapAction: 'Get_Workers' },
-  Get_Organizations: { service: 'humanResources', soapAction: 'Get_Organizations' },
-  Put_Applicant: { service: 'recruiting', soapAction: 'Put_Applicant' },
-  Hire_Employee: { service: 'staffing', soapAction: 'Hire_Employee' },
-  Change_Job: { service: 'staffing', soapAction: 'Change_Job' },
-  Terminate_Employee: { service: 'staffing', soapAction: 'Terminate_Employee' },
-  Change_Personal_Information: {
-    service: 'humanResources',
-    soapAction: 'Change_Personal_Information',
-  },
-  Put_Onboarding_Plan_Assignment: {
-    service: 'humanResources',
-    soapAction: 'Put_Onboarding_Plan_Assignment',
-  },
-} as const satisfies Record<string, { service: WorkdayServiceKey; soapAction: string }>
-
-type WorkdayOperation = keyof typeof WD_OPERATIONS
+/**
+ * Coerces a SOAP scalar to a boolean. The XML parser returns leaf text as strings,
+ * so `"true"`/`"false"` must be normalized before boolean operations like negation.
+ * Returns null when the value is null/undefined or unrecognized.
+ */
+export function parseSoapBoolean(value: unknown): boolean | null {
+  if (value == null) return null
+  if (typeof value === 'boolean') return value
+  if (typeof value === 'string') {
+    const trimmed = value.trim().toLowerCase()
+    if (trimmed === 'true' || trimmed === '1') return true
+    if (trimmed === 'false' || trimmed === '0') return false
+  }
+  return null
+}
+
+/**
+ * Coerces a SOAP scalar to a number. The XML parser returns leaf text as strings,
+ * so numeric fields like `Total_Results` must be normalized before arithmetic.
+ * Returns null when the value is null/undefined or not a finite number.
+ */
+export function parseSoapNumber(value: unknown): number | null {
+  if (value == null) return null
+  if (typeof value === 'number') return Number.isFinite(value) ? value : null
+  if (typeof value === 'string') {
+    const trimmed = value.trim()
+    if (trimmed === '') return null
+    const n = Number(trimmed)
+    return Number.isFinite(n) ? n : null
+  }
+  return null
+}
+
+const WD_OPERATIONS = [
+  'Get_Workers',
+  'Get_Organizations',
+  'Put_Applicant',
+  'Hire_Employee',
+  'Change_Job',
+  'Terminate_Employee',
+  'Change_Personal_Information',
+  'Put_Onboarding_Plan_Assignment',
+] as const
+
+type WorkdayOperation = (typeof WD_OPERATIONS)[number]
 
 type SoapOperationFn = (
   args: Record<string, unknown>