fix(netsuite): restore update location output

Author: Bill Leoutsakos

apps/docs/content/docs/en/integrations/netsuite.mdx

@@ -149,6 +149,7 @@ Update fields on an existing NetSuite record with PATCH.
 | --------- | ---- | ----------- |
 | `status` | number | HTTP status returned by NetSuite |
 | `data` | json | NetSuite response body; record fields are account-specific and dynamic |
+| `location` | string | Updated record URL from the Location response header |
 
 ### NetSuite Upsert Record
 

apps/sim/blocks/blocks/netsuite.test.ts

@@ -264,6 +264,7 @@ describe('Oracle NetSuite block', () => {
       field: 'operation',
       value: [
         'netsuite_create_record',
+        'netsuite_update_record',
         'netsuite_batch_get_records',
         'netsuite_batch_create_records',
         'netsuite_batch_update_records',

apps/sim/blocks/blocks/netsuite.ts

@@ -111,7 +111,11 @@ const BATCH_OPERATIONS = [
   'netsuite_batch_delete_records',
 ]
 
-const LOCATION_OPERATIONS = ['netsuite_create_record', ...BATCH_OPERATIONS]
+const LOCATION_OPERATIONS = [
+  'netsuite_create_record',
+  'netsuite_update_record',
+  ...BATCH_OPERATIONS,
+]
 
 const BATCH_WRITE_OPERATIONS = [
   'netsuite_batch_create_records',
@@ -832,7 +836,7 @@ Return ONLY the comma-separated sublist IDs - no explanations, no extra text.`,
     },
     location: {
       type: 'string',
-      description: 'Created resource or async job location',
+      description: 'Record resource or async job location returned by NetSuite',
       condition: { field: 'operation', value: LOCATION_OPERATIONS },
     },
     jobId: {

apps/sim/tools/generated/tool-outputs.ts

@@ -489,9 +489,11 @@ describe('NetSuite operation contracts', () => {
         const entry = SOURCE_MATRIX[apiIndex]
         apiIndex += 1
         const headers =
-          entry.successStatus === 202
-            ? { Location: `${ORIGIN}/services/rest/async/v1/job/job-${apiIndex}` }
-            : undefined
+          entry.id === 'netsuite_update_record'
+            ? { Location: `${ORIGIN}/services/rest/record/v1/customer/7` }
+            : entry.successStatus === 202
+              ? { Location: `${ORIGIN}/services/rest/async/v1/job/job-${apiIndex}` }
+              : undefined
         return entry.successStatus === 204 || entry.successStatus === 202
           ? new Response(null, { status: entry.successStatus, headers })
           : new Response(JSON.stringify({ operation: entry.id }), {
@@ -506,6 +508,11 @@ describe('NetSuite operation contracts', () => {
       expect(result.success, entry.id).toBe(true)
       expect(result.output?.status, entry.id).toBe(entry.successStatus)
       if (entry.successStatus === 204) expect(result.output?.data, entry.id).toBeNull()
+      if (entry.id === 'netsuite_update_record') {
+        expect(result.output?.location, entry.id).toBe(
+          `${ORIGIN}/services/rest/record/v1/customer/7`
+        )
+      }
       if (entry.successStatus === 202) {
         expect(result.output?.data, entry.id).toBeNull()
         expect(result.output?.location, entry.id).toContain('/services/rest/async/v1/job/')
@@ -563,7 +570,11 @@ describe('NetSuite operation contracts', () => {
 
     for (const tool of NETSUITE_TOOLS) {
       const expected = ['status', 'data']
-      if (tool.id === 'netsuite_create_record' || batchToolIds.has(tool.id)) {
+      if (
+        tool.id === 'netsuite_create_record' ||
+        tool.id === 'netsuite_update_record' ||
+        batchToolIds.has(tool.id)
+      ) {
         expected.push('location')
       }
       if (batchToolIds.has(tool.id)) expected.push('jobId')

apps/sim/tools/netsuite/netsuite.test.ts

@@ -80,5 +80,10 @@ export const netsuiteUpdateRecordTool: ToolConfig<NetSuiteUpdateRecordParams, Ne
       description: 'NetSuite response body; record fields are account-specific and dynamic',
       nullable: true,
     },
+    location: {
+      type: 'string',
+      description: 'Updated record URL from the Location response header',
+      optional: true,
+    },
   },
 }