catalyst-relay 0.5.7 → 0.5.8

package/dist/index.mjs

@@ -2472,45 +2472,37 @@ async function createTransport(client, config) {
   return ok(transportId);
 }
 
-// src/core/adt/transports/getTransportContents.ts
-var ACCEPT_HEADER = "application/vnd.sap.adt.transportorganizer.v1+xml";
-async function getTransportContents(client, transportId) {
-  const [response, requestErr] = await client.request({
-    method: "GET",
-    path: `/sap/bc/adt/cts/transportrequests/${transportId}`,
-    headers: {
-      "Accept": ACCEPT_HEADER
+// src/core/adt/transports/parseTransportTasks.ts
+function parseTransportTasks(doc) {
+  const tasks = [];
+  const taskElements = doc.getElementsByTagName("tm:task");
+  for (let i = 0; i < taskElements.length; i++) {
+    const taskEl = taskElements[i];
+    if (!taskEl) continue;
+    const taskId = taskEl.getAttribute("tm:number");
+    if (!taskId) continue;
+    const objects = [];
+    const objectElements = taskEl.getElementsByTagName("tm:abap_object");
+    for (let j = 0; j < objectElements.length; j++) {
+      const el = objectElements[j];
+      if (!el) continue;
+      const name = el.getAttribute("tm:name");
+      if (!name) continue;
+      objects.push({
+        name,
+        description: el.getAttribute("tm:obj_desc") || el.getAttribute("tm:obj_info") || "",
+        pgmid: el.getAttribute("tm:pgmid") || "",
+        type: el.getAttribute("tm:type") || "",
+        position: el.getAttribute("tm:position") || ""
+      });
     }
-  });
-  if (requestErr) return err(requestErr);
-  if (!response.ok) {
-    const text2 = await response.text();
-    const errorMsg = extractError(text2);
-    return err(new Error(`Failed to read transport ${transportId}: ${errorMsg}`));
-  }
-  const text = await response.text();
-  const [doc, parseErr] = safeParseXml(text);
-  if (parseErr) return err(parseErr);
-  const objects = [];
-  const elements = doc.getElementsByTagName("tm:abap_object");
-  for (let i = 0; i < elements.length; i++) {
-    const el = elements[i];
-    if (!el) continue;
-    const name = el.getAttribute("tm:name");
-    if (!name) continue;
-    objects.push({
-      name,
-      description: el.getAttribute("tm:obj_desc") || el.getAttribute("tm:obj_info") || "",
-      pgmid: el.getAttribute("tm:pgmid") || "",
-      type: el.getAttribute("tm:type") || "",
-      position: el.getAttribute("tm:position") || ""
-    });
+    tasks.push({ taskId, objects });
   }
-  return ok(objects);
+  return tasks;
 }
 
 // src/core/adt/transports/removeFromTransport.ts
-var ACCEPT_HEADER2 = "application/vnd.sap.adt.transportorganizer.v1+xml";
+var ACCEPT_HEADER = "application/vnd.sap.adt.transportorganizer.v1+xml";
 async function removeTransportEntry(client, transportId, object) {
   const body = [
     '<?xml version="1.0" encoding="ASCII"?>',
@@ -2524,7 +2516,7 @@ async function removeTransportEntry(client, transportId, object) {
     method: "PUT",
     path: `/sap/bc/adt/cts/transportrequests/${transportId}`,
     headers: {
-      "Accept": ACCEPT_HEADER2,
+      "Accept": ACCEPT_HEADER,
       "Content-Type": "text/plain"
     },
     body
@@ -2538,49 +2530,36 @@ async function removeTransportEntry(client, transportId, object) {
   return ok(void 0);
 }
 async function removeFromTransport(client, transportId, objectName) {
-  const [objects, listErr] = await getTransportContents(client, transportId);
-  if (listErr) return err(listErr);
-  const object = objects.find((o) => o.name === objectName);
-  if (!object) {
-    return err(new Error(`Object '${objectName}' not found on transport ${transportId}`));
+  const [response, requestErr] = await client.request({
+    method: "GET",
+    path: `/sap/bc/adt/cts/transportrequests/${transportId}`,
+    headers: { "Accept": ACCEPT_HEADER }
+  });
+  if (requestErr) return err(requestErr);
+  if (!response.ok) {
+    const text2 = await response.text();
+    const errorMsg = extractError(text2);
+    return err(new Error(`Failed to read transport ${transportId}: ${errorMsg}`));
+  }
+  const text = await response.text();
+  const [doc, parseErr] = safeParseXml(text);
+  if (parseErr) return err(parseErr);
+  const tasks = parseTransportTasks(doc);
+  for (const task of tasks) {
+    const object = task.objects.find((o) => o.name === objectName);
+    if (!object) continue;
+    return removeTransportEntry(client, task.taskId, object);
   }
-  return removeTransportEntry(client, transportId, object);
+  return err(new Error(`Object '${objectName}' not found on transport ${transportId}`));
 }
 
 // src/core/adt/transports/deleteTransport.ts
-var ACCEPT_HEADER3 = "application/vnd.sap.adt.transportorganizer.v1+xml";
-function parseTransportTasks(doc) {
-  const tasks = [];
-  const taskElements = doc.getElementsByTagName("tm:task");
-  for (let i = 0; i < taskElements.length; i++) {
-    const taskEl = taskElements[i];
-    if (!taskEl) continue;
-    const taskId = taskEl.getAttribute("tm:number");
-    if (!taskId) continue;
-    const objects = [];
-    const objectElements = taskEl.getElementsByTagName("tm:abap_object");
-    for (let j = 0; j < objectElements.length; j++) {
-      const el = objectElements[j];
-      if (!el) continue;
-      const name = el.getAttribute("tm:name");
-      if (!name) continue;
-      objects.push({
-        name,
-        description: el.getAttribute("tm:obj_desc") || el.getAttribute("tm:obj_info") || "",
-        pgmid: el.getAttribute("tm:pgmid") || "",
-        type: el.getAttribute("tm:type") || "",
-        position: el.getAttribute("tm:position") || ""
-      });
-    }
-    tasks.push({ taskId, objects });
-  }
-  return tasks;
-}
+var ACCEPT_HEADER2 = "application/vnd.sap.adt.transportorganizer.v1+xml";
 async function sortAndCompress(client, taskId) {
   const [response, requestErr] = await client.request({
     method: "POST",
     path: `/sap/bc/adt/cts/transportrequests/${taskId}/sortandcompress`,
-    headers: { "Accept": ACCEPT_HEADER3 }
+    headers: { "Accept": ACCEPT_HEADER2 }
   });
   if (requestErr) return err(requestErr);
   if (!response.ok) {
@@ -2594,7 +2573,7 @@ async function deleteRequest(client, id) {
   const [response, requestErr] = await client.request({
     method: "DELETE",
     path: `/sap/bc/adt/cts/transportrequests/${id}`,
-    headers: { "Accept": ACCEPT_HEADER3 }
+    headers: { "Accept": ACCEPT_HEADER2 }
   });
   if (requestErr) return err(requestErr);
   if (!response.ok) {
@@ -2608,7 +2587,7 @@ async function deleteTransport(client, transportId, removeObjects = false) {
   const [response, requestErr] = await client.request({
     method: "GET",
     path: `/sap/bc/adt/cts/transportrequests/${transportId}`,
-    headers: { "Accept": ACCEPT_HEADER3 }
+    headers: { "Accept": ACCEPT_HEADER2 }
   });
   if (requestErr) return err(requestErr);
   if (!response.ok) {
@@ -2627,7 +2606,7 @@ async function deleteTransport(client, transportId, removeObjects = false) {
       const [taskResponse, taskReadErr] = await client.request({
         method: "GET",
         path: `/sap/bc/adt/cts/transportrequests/${task.taskId}`,
-        headers: { "Accept": ACCEPT_HEADER3 }
+        headers: { "Accept": ACCEPT_HEADER2 }
       });
       if (taskReadErr) return err(taskReadErr);
       const taskText = await taskResponse.text();