@mcp-abap-adt/adt-clients 10.0.1 → 10.1.0

package/README.md

@@ -364,7 +364,23 @@ implementations pass it internally on reads after create/update.
 See the caveat above — on the system measured, the flag had no effect on object
 reads at all.
 
-**Note:** Long polling is automatically used in `create()` and `update()` methods of all `AdtObject` implementations to ensure objects are ready before proceeding with subsequent operations.
+**Note:** `create()` and `update()` request long polling on their follow-up reads
+where the handler supports it. Read that as "asked for", not "ensured" — the
+caveat above stands, and nothing in this library can make the system answer
+sooner than it does.
+
+What the library does guarantee is that a read which came back empty is not
+written back. ADT answers a read of a not-yet-ready object with **HTTP 200 and
+an empty body**, never a 404, so a read-modify-write update used to patch that
+empty body — changing nothing, because there was nothing to change — and PUT the
+result. Since 10.1.0 that read fails with `XmlPatchError`, naming the object:
+
+```
+Cannot update domain ZAC_DOM01: the read returned an empty body.
+```
+
+A slow system therefore surfaces as a read error, not as a write the server
+rejects for a reason that points nowhere near the cause.
 
 ### Creating Behavior Implementation Classes
 

package/dist/core/accessControl/activation.d.ts

@@ -1,4 +1,7 @@
 import type { IAbapConnection, IAdtResponse } from '@mcp-abap-adt/interfaces';
+/**
+ * Parse activation response
+ */
 /**
  * Activate access control
  */

package/dist/core/accessControl/activation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/accessControl/activation.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAwD9E;;GAEG;AACH,wBAAsB,qBAAqB,CACzC,UAAU,EAAE,eAAe,EAC3B,iBAAiB,EAAE,MAAM,GACxB,OAAO,CAAC,YAAY,CAAC,CAyBvB"}
+{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/accessControl/activation.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAe9E;;GAEG;AAEH;;GAEG;AACH,wBAAsB,qBAAqB,CACzC,UAAU,EAAE,eAAe,EAC3B,iBAAiB,EAAE,MAAM,GACxB,OAAO,CAAC,YAAY,CAAC,CAoBvB"}

package/dist/core/accessControl/activation.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.activateAccessControl = activateAccessControl;
-const fast_xml_parser_1 = require("fast-xml-parser");
+const activationUtils_1 = require("../../utils/activationUtils");
 const internalUtils_1 = require("../../utils/internalUtils");
 const timeouts_1 = require("../../utils/timeouts");
 /**
@@ -16,35 +16,6 @@ function buildActivationXml(accessControlName) {
 /**
  * Parse activation response
  */
-function parseActivationResponse(response) {
-    const parser = new fast_xml_parser_1.XMLParser({
-        ignoreAttributes: false,
-        attributeNamePrefix: '',
-    });
-    try {
-        const result = parser.parse(response.data);
-        const properties = result['chkl:messages']?.['chkl:properties'];
-        if (properties) {
-            const activated = properties.activationExecuted === 'true' ||
-                properties.activationExecuted === true;
-            const checked = properties.checkExecuted === 'true' ||
-                properties.checkExecuted === true;
-            return {
-                success: activated && checked,
-                message: activated
-                    ? 'Access control activated successfully'
-                    : 'Activation failed',
-            };
-        }
-        return { success: false, message: 'Unknown activation status' };
-    }
-    catch (error) {
-        return {
-            success: false,
-            message: `Failed to parse activation response: ${error}`,
-        };
-    }
-}
 /**
  * Activate access control
  */
@@ -62,9 +33,6 @@ async function activateAccessControl(connection, accessControlName) {
         data: xmlBody,
         headers,
     });
-    const activationResult = parseActivationResponse(response);
-    if (!activationResult.success) {
-        throw new Error(`Access control activation failed: ${activationResult.message}`);
-    }
+    (0, activationUtils_1.assertActivationSucceeded)('Access control', response.data);
     return response;
 }

package/dist/core/appendStructure/activation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/appendStructure/activation.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AA8C9E,wBAAsB,uBAAuB,CAC3C,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,YAAY,CAAC,CAgBvB"}
+{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/appendStructure/activation.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAY9E,wBAAsB,uBAAuB,CAC3C,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,YAAY,CAAC,CAWvB"}

package/dist/core/appendStructure/activation.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.activateAppendStructure = activateAppendStructure;
-const fast_xml_parser_1 = require("fast-xml-parser");
+const activationUtils_1 = require("../../utils/activationUtils");
 const internalUtils_1 = require("../../utils/internalUtils");
 const timeouts_1 = require("../../utils/timeouts");
 function buildActivationXml(name) {
@@ -10,35 +10,6 @@ function buildActivationXml(name) {
   <adtcore:objectReference adtcore:uri="/sap/bc/adt/ddic/structures/${(0, internalUtils_1.encodeSapObjectName)(name.toLowerCase())}" adtcore:name="${name.toUpperCase()}"/>
 </adtcore:objectReferences>`;
 }
-function parseActivationResponse(response) {
-    const parser = new fast_xml_parser_1.XMLParser({
-        ignoreAttributes: false,
-        attributeNamePrefix: '',
-    });
-    try {
-        const result = parser.parse(response.data);
-        const properties = result['chkl:messages']?.['chkl:properties'];
-        if (properties) {
-            const activated = properties.activationExecuted === 'true' ||
-                properties.activationExecuted === true;
-            const checked = properties.checkExecuted === 'true' ||
-                properties.checkExecuted === true;
-            return {
-                success: activated && checked,
-                message: activated
-                    ? 'Append structure activated successfully'
-                    : 'Activation failed',
-            };
-        }
-        return { success: false, message: 'Unknown activation status' };
-    }
-    catch (error) {
-        return {
-            success: false,
-            message: `Failed to parse activation response: ${error}`,
-        };
-    }
-}
 async function activateAppendStructure(connection, name) {
     const url = `/sap/bc/adt/activation?method=activate&preauditRequested=true`;
     const response = await connection.makeAdtRequest({
@@ -48,9 +19,6 @@ async function activateAppendStructure(connection, name) {
         data: buildActivationXml(name),
         headers: { Accept: 'application/xml', 'Content-Type': 'application/xml' },
     });
-    const activationResult = parseActivationResponse(response);
-    if (!activationResult.success) {
-        throw new Error(`Append structure activation failed: ${activationResult.message}`);
-    }
+    (0, activationUtils_1.assertActivationSucceeded)('Append structure', response.data);
     return response;
 }

package/dist/core/authorizationField/activation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/authorizationField/activation.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAqD9E;;GAEG;AACH,wBAAsB,0BAA0B,CAC9C,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,YAAY,CAAC,CA2BvB"}
+{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/authorizationField/activation.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAc9E;;GAEG;AACH,wBAAsB,0BAA0B,CAC9C,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,YAAY,CAAC,CAsBvB"}

package/dist/core/authorizationField/activation.js

@@ -4,7 +4,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.activateAuthorizationField = activateAuthorizationField;
-const fast_xml_parser_1 = require("fast-xml-parser");
+const activationUtils_1 = require("../../utils/activationUtils");
 const internalUtils_1 = require("../../utils/internalUtils");
 const timeouts_1 = require("../../utils/timeouts");
 function buildActivationXml(name) {
@@ -15,36 +15,6 @@ function buildActivationXml(name) {
   <adtcore:objectReference adtcore:uri="/sap/bc/adt/aps/iam/auth/${encoded}" adtcore:name="${upper}"/>
 </adtcore:objectReferences>`;
 }
-function parseActivationResponse(response) {
-    const parser = new fast_xml_parser_1.XMLParser({
-        ignoreAttributes: false,
-        attributeNamePrefix: '',
-    });
-    try {
-        const result = parser.parse(response.data);
-        const properties = result['chkl:messages']?.['chkl:properties'];
-        if (properties) {
-            const activated = properties.activationExecuted === 'true' ||
-                properties.activationExecuted === true;
-            const checked = properties.checkExecuted === 'true' ||
-                properties.checkExecuted === true;
-            return {
-                success: activated && checked,
-                message: activated
-                    ? 'Authorization field activated successfully'
-                    : 'Activation failed',
-            };
-        }
-        // When the response body is empty, SAP treats activation as successful.
-        return { success: true, message: 'Authorization field activated' };
-    }
-    catch (error) {
-        return {
-            success: false,
-            message: `Failed to parse activation response: ${error}`,
-        };
-    }
-}
 /**
  * Activate an authorization field.
  */
@@ -64,9 +34,6 @@ async function activateAuthorizationField(connection, name) {
             'Content-Type': 'application/xml',
         },
     });
-    const activationResult = parseActivationResponse(response);
-    if (!activationResult.success) {
-        throw new Error(`Authorization field activation failed: ${activationResult.message}`);
-    }
+    (0, activationUtils_1.assertActivationSucceeded)('Authorization field', response.data);
     return response;
 }

package/dist/core/dataElement/activation.d.ts

@@ -2,6 +2,9 @@
  * DataElement activation operations
  */
 import type { IAbapConnection, IAdtResponse } from '@mcp-abap-adt/interfaces';
+/**
+ * Parse activation response
+ */
 /**
  * Activate data element
  * Makes data element active and usable in SAP system

package/dist/core/dataElement/activation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/dataElement/activation.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAwD9E;;;GAGG;AACH,wBAAsB,mBAAmB,CACvC,UAAU,EAAE,eAAe,EAC3B,eAAe,EAAE,MAAM,GACtB,OAAO,CAAC,YAAY,CAAC,CAyBvB"}
+{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/dataElement/activation.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAe9E;;GAEG;AAEH;;;GAGG;AACH,wBAAsB,mBAAmB,CACvC,UAAU,EAAE,eAAe,EAC3B,eAAe,EAAE,MAAM,GACtB,OAAO,CAAC,YAAY,CAAC,CAoBvB"}

package/dist/core/dataElement/activation.js

@@ -4,7 +4,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.activateDataElement = activateDataElement;
-const fast_xml_parser_1 = require("fast-xml-parser");
+const activationUtils_1 = require("../../utils/activationUtils");
 const internalUtils_1 = require("../../utils/internalUtils");
 const timeouts_1 = require("../../utils/timeouts");
 /**
@@ -19,35 +19,6 @@ function buildActivationXml(dataElementName) {
 /**
  * Parse activation response
  */
-function parseActivationResponse(response) {
-    const parser = new fast_xml_parser_1.XMLParser({
-        ignoreAttributes: false,
-        attributeNamePrefix: '',
-    });
-    try {
-        const result = parser.parse(response.data);
-        const properties = result['chkl:messages']?.['chkl:properties'];
-        if (properties) {
-            const activated = properties.activationExecuted === 'true' ||
-                properties.activationExecuted === true;
-            const checked = properties.checkExecuted === 'true' ||
-                properties.checkExecuted === true;
-            return {
-                success: activated && checked,
-                message: activated
-                    ? 'Data element activated successfully'
-                    : 'Activation failed',
-            };
-        }
-        return { success: false, message: 'Unknown activation status' };
-    }
-    catch (error) {
-        return {
-            success: false,
-            message: `Failed to parse activation response: ${error}`,
-        };
-    }
-}
 /**
  * Activate data element
  * Makes data element active and usable in SAP system
@@ -66,9 +37,6 @@ async function activateDataElement(connection, dataElementName) {
         data: xmlBody,
         headers,
     });
-    const activationResult = parseActivationResponse(response);
-    if (!activationResult.success) {
-        throw new Error(`Data element activation failed: ${activationResult.message}`);
-    }
+    (0, activationUtils_1.assertActivationSucceeded)('Data element', response.data);
     return response;
 }

package/dist/core/dataElement/update.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/core/dataElement/update.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EACV,eAAe,EACf,YAAY,EACZ,OAAO,EACR,MAAM,0BAA0B,CAAC;AAgBlC,OAAO,KAAK,EAAE,wBAAwB,EAAE,MAAM,SAAS,CAAC;AAIxD;;GAEG;AACH,wBAAsB,aAAa,CACjC,UAAU,EAAE,eAAe,EAC3B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,CAAC,CAmCjE;AAqID;;;;GAIG;AACH,wBAAsB,iBAAiB,CACrC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,wBAAwB,EAChC,UAAU,EAAE,MAAM,EAClB,MAAM,CAAC,EAAE,OAAO,GACf,OAAO,CAAC,YAAY,CAAC,CAkEvB;AAED;;GAEG;AACH,wBAAsB,yBAAyB,CAC7C,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,wBAAwB,EAC9B,UAAU,EAAE,MAAM,EAClB,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,EACnE,MAAM,CAAC,EAAE,OAAO,GACf,OAAO,CAAC,YAAY,CAAC,CAEvB"}
+{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/core/dataElement/update.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EACV,eAAe,EACf,YAAY,EACZ,OAAO,EACR,MAAM,0BAA0B,CAAC;AAgBlC,OAAO,KAAK,EAAE,wBAAwB,EAAE,MAAM,SAAS,CAAC;AAIxD;;GAEG;AACH,wBAAsB,aAAa,CACjC,UAAU,EAAE,eAAe,EAC3B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,CAAC,CAmCjE;AAqID;;;;GAIG;AACH,wBAAsB,iBAAiB,CACrC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,wBAAwB,EAChC,UAAU,EAAE,MAAM,EAClB,MAAM,CAAC,EAAE,OAAO,GACf,OAAO,CAAC,YAAY,CAAC,CAqEvB;AAED;;GAEG;AACH,wBAAsB,yBAAyB,CAC7C,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,wBAAwB,EAC9B,UAAU,EAAE,MAAM,EAClB,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,EACnE,MAAM,CAAC,EAAE,OAAO,GACf,OAAO,CAAC,YAAY,CAAC,CAEvB"}

package/dist/core/dataElement/update.js

@@ -171,7 +171,7 @@ async function updateDataElement(connection, params, lockHandle, logger) {
         timeout: (0, timeouts_1.getTimeout)('default'),
         headers: { Accept: contentTypes_1.ACCEPT_DATA_ELEMENT },
     });
-    const currentXml = (0, xmlPatch_1.extractXmlString)(currentResponse.data);
+    const currentXml = (0, xmlPatch_1.extractXmlString)(currentResponse.data, `data element ${params.data_element_name}`);
     // 2. Patch only changed fields
     const updatedXml = patchDataElementXml(currentXml, params);
     // Debug: log XML when DEBUG_ADT_LIBS is enabled

package/dist/core/domain/update.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/core/domain/update.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAe9E,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,SAAS,CAAC;AAsEnD;;;;GAIG;AACH,wBAAsB,YAAY,CAChC,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,mBAAmB,EACzB,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,YAAY,CAAC,CAiCvB"}
+{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/core/domain/update.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAe9E,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,SAAS,CAAC;AAsEnD;;;;GAIG;AACH,wBAAsB,YAAY,CAChC,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,mBAAmB,EACzB,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,YAAY,CAAC,CAoCvB"}

package/dist/core/domain/update.js

@@ -69,7 +69,7 @@ async function updateDomain(connection, args, lockHandle) {
         timeout: (0, timeouts_1.getTimeout)('default'),
         headers: { Accept: contentTypes_1.ACCEPT_DOMAIN },
     });
-    const currentXml = (0, xmlPatch_1.extractXmlString)(currentResponse.data);
+    const currentXml = (0, xmlPatch_1.extractXmlString)(currentResponse.data, `domain ${args.domain_name}`);
     // 2. Patch only changed fields
     const updatedXml = patchDomainXml(currentXml, args);
     // 3. PUT

package/dist/core/featureToggle/activation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/featureToggle/activation.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAqD9E;;GAEG;AACH,wBAAsB,qBAAqB,CACzC,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,YAAY,CAAC,CA2BvB"}
+{"version":3,"file":"activation.d.ts","sourceRoot":"","sources":["../../../src/core/featureToggle/activation.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAc9E;;GAEG;AACH,wBAAsB,qBAAqB,CACzC,UAAU,EAAE,eAAe,EAC3B,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,YAAY,CAAC,CAsBvB"}

package/dist/core/featureToggle/activation.js

@@ -4,7 +4,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.activateFeatureToggle = activateFeatureToggle;
-const fast_xml_parser_1 = require("fast-xml-parser");
+const activationUtils_1 = require("../../utils/activationUtils");
 const internalUtils_1 = require("../../utils/internalUtils");
 const timeouts_1 = require("../../utils/timeouts");
 function buildActivationXml(name) {
@@ -15,36 +15,6 @@ function buildActivationXml(name) {
   <adtcore:objectReference adtcore:uri="/sap/bc/adt/sfw/featuretoggles/${encoded}" adtcore:name="${name.toUpperCase()}"/>
 </adtcore:objectReferences>`;
 }
-function parseActivationResponse(response) {
-    const parser = new fast_xml_parser_1.XMLParser({
-        ignoreAttributes: false,
-        attributeNamePrefix: '',
-    });
-    try {
-        const result = parser.parse(response.data);
-        const properties = result['chkl:messages']?.['chkl:properties'];
-        if (properties) {
-            const activated = properties.activationExecuted === 'true' ||
-                properties.activationExecuted === true;
-            const checked = properties.checkExecuted === 'true' ||
-                properties.checkExecuted === true;
-            return {
-                success: activated && checked,
-                message: activated
-                    ? 'Feature toggle activated successfully'
-                    : 'Activation failed',
-            };
-        }
-        // When the response body is empty, SAP treats activation as successful.
-        return { success: true, message: 'Feature toggle activated' };
-    }
-    catch (error) {
-        return {
-            success: false,
-            message: `Failed to parse activation response: ${error}`,
-        };
-    }
-}
 /**
  * Activate a feature toggle.
  */
@@ -64,9 +34,6 @@ async function activateFeatureToggle(connection, name) {
             'Content-Type': 'application/xml',
         },
     });
-    const activationResult = parseActivationResponse(response);
-    if (!activationResult.success) {
-        throw new Error(`Feature toggle activation failed: ${activationResult.message}`);
-    }
+    (0, activationUtils_1.assertActivationSucceeded)('Feature toggle', response.data);
     return response;
 }

package/dist/core/functionGroup/update.js

@@ -60,9 +60,7 @@ async function updateFunctionGroup(connection, params, contentTypes) {
         }
         // Get current XML
         const currentResponse = await (0, read_1.getFunctionGroup)(connection, params.function_group_name, undefined);
-        const currentXml = typeof currentResponse.data === 'string'
-            ? currentResponse.data
-            : JSON.stringify(currentResponse.data);
+        const currentXml = (0, xmlPatch_1.extractXmlString)(currentResponse.data, `function group ${params.function_group_name}`);
         // Update metadata
         const updateResponse = await updateFunctionGroupMetadata(connection, params.function_group_name, currentXml, params.description, lockHandle, params.transport_request, contentTypes);
         // Unlock if we locked it

package/dist/core/metadataExtension/activate.d.ts

@@ -8,14 +8,14 @@ import type { IAbapConnection, IAdtResponse } from '@mcp-abap-adt/interfaces';
  * Activate a metadata extension
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param sessionId - Session ID for request tracking
  * @param preaudit - Request pre-audit before activation (default: true)
  * @returns Axios response with activation result
  *
  * @example
  * ```typescript
- * await activateMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001', sessionId);
+ * await activateMetadataExtension(connection, 'ZDEMO_C_CDS_MDE', sessionId);
  * ```
  */
 export declare function activateMetadataExtension(connection: IAbapConnection, name: string, preaudit?: boolean): Promise<IAdtResponse>;

package/dist/core/metadataExtension/activate.js

@@ -11,14 +11,14 @@ const activationUtils_1 = require("../../utils/activationUtils");
  * Activate a metadata extension
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param sessionId - Session ID for request tracking
  * @param preaudit - Request pre-audit before activation (default: true)
  * @returns Axios response with activation result
  *
  * @example
  * ```typescript
- * await activateMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001', sessionId);
+ * await activateMetadataExtension(connection, 'ZDEMO_C_CDS_MDE', sessionId);
  * ```
  */
 async function activateMetadataExtension(connection, name, preaudit = true) {

package/dist/core/metadataExtension/check.d.ts

@@ -8,7 +8,7 @@ import type { IAbapConnection, IAdtResponse } from '@mcp-abap-adt/interfaces';
  * Check metadata extension syntax
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param sessionId - Session ID for request tracking
  * @param version - Version to check ('active' or 'inactive', default 'inactive')
  * @param sourceCode - Optional source code to validate before saving
@@ -16,7 +16,7 @@ import type { IAbapConnection, IAdtResponse } from '@mcp-abap-adt/interfaces';
  *
  * @example
  * ```typescript
- * const checkResult = await checkMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001', sessionId);
+ * const checkResult = await checkMetadataExtension(connection, 'ZDEMO_C_CDS_MDE', sessionId);
  * ```
  */
 export declare function checkMetadataExtension(connection: IAbapConnection, name: string, version?: 'active' | 'inactive', sourceCode?: string): Promise<IAdtResponse>;

package/dist/core/metadataExtension/check.js

@@ -11,7 +11,7 @@ const checkRun_1 = require("../../utils/checkRun");
  * Check metadata extension syntax
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param sessionId - Session ID for request tracking
  * @param version - Version to check ('active' or 'inactive', default 'inactive')
  * @param sourceCode - Optional source code to validate before saving
@@ -19,7 +19,7 @@ const checkRun_1 = require("../../utils/checkRun");
  *
  * @example
  * ```typescript
- * const checkResult = await checkMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001', sessionId);
+ * const checkResult = await checkMetadataExtension(connection, 'ZDEMO_C_CDS_MDE', sessionId);
  * ```
  */
 async function checkMetadataExtension(connection, name, version = 'inactive', sourceCode) {

package/dist/core/metadataExtension/create.d.ts

@@ -16,9 +16,9 @@ import type { IMetadataExtensionCreateParams } from './types';
  * @example
  * ```typescript
  * const response = await createMetadataExtension(connection, {
- *   name: 'ZOK_C_CDS_TEST_0001',
+ *   name: 'ZDEMO_C_CDS_MDE',
  *   description: 'First metadata extension',
- *   packageName: 'ZOK_TEST_000222',
+ *   packageName: 'ZDEMO_PKG',
  *   transportRequest: 'TRLK900123'
  * }, sessionId);
  * ```

package/dist/core/metadataExtension/create.js

@@ -20,9 +20,9 @@ const timeouts_1 = require("../../utils/timeouts");
  * @example
  * ```typescript
  * const response = await createMetadataExtension(connection, {
- *   name: 'ZOK_C_CDS_TEST_0001',
+ *   name: 'ZDEMO_C_CDS_MDE',
  *   description: 'First metadata extension',
- *   packageName: 'ZOK_TEST_000222',
+ *   packageName: 'ZDEMO_PKG',
  *   transportRequest: 'TRLK900123'
  * }, sessionId);
  * ```

package/dist/core/metadataExtension/delete.d.ts

@@ -8,14 +8,14 @@ import type { IAbapConnection, IAdtResponse } from '@mcp-abap-adt/interfaces';
  * Delete a metadata extension
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param transportRequest - Transport request number (optional for local objects)
  * @param sessionId - Session ID for request tracking
  * @returns Axios response
  *
  * @example
  * ```typescript
- * await deleteMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001', 'TRLK900123', sessionId);
+ * await deleteMetadataExtension(connection, 'ZDEMO_C_CDS_MDE', 'TRLK900123', sessionId);
  * ```
  */
 export declare function deleteMetadataExtension(connection: IAbapConnection, name: string, transportRequest: string | undefined): Promise<IAdtResponse>;

package/dist/core/metadataExtension/delete.js

@@ -11,14 +11,14 @@ const timeouts_1 = require("../../utils/timeouts");
  * Delete a metadata extension
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param transportRequest - Transport request number (optional for local objects)
  * @param sessionId - Session ID for request tracking
  * @returns Axios response
  *
  * @example
  * ```typescript
- * await deleteMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001', 'TRLK900123', sessionId);
+ * await deleteMetadataExtension(connection, 'ZDEMO_C_CDS_MDE', 'TRLK900123', sessionId);
  * ```
  */
 async function deleteMetadataExtension(connection, name, transportRequest) {

package/dist/core/metadataExtension/lock.d.ts

@@ -8,13 +8,13 @@ import type { IAbapConnection } from '@mcp-abap-adt/interfaces';
  * Lock a metadata extension for modification
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param sessionId - Session ID for request tracking
  * @returns Lock handle string
  *
  * @example
  * ```typescript
- * const lockHandle = await lockMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001', sessionId);
+ * const lockHandle = await lockMetadataExtension(connection, 'ZDEMO_C_CDS_MDE', sessionId);
  * ```
  */
 export declare function lockMetadataExtension(connection: IAbapConnection, name: string): Promise<string>;

package/dist/core/metadataExtension/lock.js

@@ -13,13 +13,13 @@ const timeouts_1 = require("../../utils/timeouts");
  * Lock a metadata extension for modification
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param sessionId - Session ID for request tracking
  * @returns Lock handle string
  *
  * @example
  * ```typescript
- * const lockHandle = await lockMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001', sessionId);
+ * const lockHandle = await lockMetadataExtension(connection, 'ZDEMO_C_CDS_MDE', sessionId);
  * ```
  */
 async function lockMetadataExtension(connection, name) {

package/dist/core/metadataExtension/read.d.ts

@@ -10,12 +10,12 @@ import type { IReadOptions } from '../shared/types';
  * Read metadata extension metadata
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @returns Axios response with metadata extension metadata
  *
  * @example
  * ```typescript
- * const metadata = await readMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001');
+ * const metadata = await readMetadataExtension(connection, 'ZDEMO_C_CDS_MDE');
  * ```
  */
 export declare function readMetadataExtension(connection: IAbapConnection, name: string, options?: IReadOptions, logger?: ILogger): Promise<IAdtResponse>;
@@ -23,13 +23,13 @@ export declare function readMetadataExtension(connection: IAbapConnection, name:
  * Read metadata extension source code
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param version - Version to read ('active' or 'inactive', default 'active')
  * @returns Axios response with source code as string
  *
  * @example
  * ```typescript
- * const response = await readMetadataExtensionSource(connection, 'ZOK_C_CDS_TEST_0001');
+ * const response = await readMetadataExtensionSource(connection, 'ZDEMO_C_CDS_MDE');
  * const sourceCode = response.data;
  * ```
  */

package/dist/core/metadataExtension/read.js

@@ -16,12 +16,12 @@ const timeouts_1 = require("../../utils/timeouts");
  * Read metadata extension metadata
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @returns Axios response with metadata extension metadata
  *
  * @example
  * ```typescript
- * const metadata = await readMetadataExtension(connection, 'ZOK_C_CDS_TEST_0001');
+ * const metadata = await readMetadataExtension(connection, 'ZDEMO_C_CDS_MDE');
  * ```
  */
 async function readMetadataExtension(connection, name, options, logger) {
@@ -42,13 +42,13 @@ async function readMetadataExtension(connection, name, options, logger) {
  * Read metadata extension source code
  *
  * @param connection - ABAP connection instance
- * @param name - Metadata extension name (e.g., 'ZOK_C_CDS_TEST_0001')
+ * @param name - Metadata extension name (e.g., 'ZDEMO_C_CDS_MDE')
  * @param version - Version to read ('active' or 'inactive', default 'active')
  * @returns Axios response with source code as string
  *
  * @example
  * ```typescript
- * const response = await readMetadataExtensionSource(connection, 'ZOK_C_CDS_TEST_0001');
+ * const response = await readMetadataExtensionSource(connection, 'ZDEMO_C_CDS_MDE');
  * const sourceCode = response.data;
  * ```
  */