@mcp-abap-adt/adt-clients 0.1.0

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

@@ -0,0 +1,12 @@
+/**
+ * FunctionGroup create operations
+ */
+import { AbapConnection } from '@mcp-abap-adt/connection';
+import { AxiosResponse } from 'axios';
+import { CreateFunctionGroupParams } from './types';
+/**
+ * Create ABAP function group
+ * Full workflow: create -> activate (optional)
+ */
+export declare function createFunctionGroup(connection: AbapConnection, params: CreateFunctionGroupParams): Promise<AxiosResponse>;
+//# sourceMappingURL=create.d.ts.map

package/dist/core/functionGroup/create.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"create.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/create.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,cAAc,EAAc,MAAM,0BAA0B,CAAC;AACtE,OAAO,EAAE,aAAa,EAAE,MAAM,OAAO,CAAC;AAKtC,OAAO,EAAE,yBAAyB,EAAE,MAAM,SAAS,CAAC;AA4CpD;;;GAGG;AACH,wBAAsB,mBAAmB,CACvC,UAAU,EAAE,cAAc,EAC1B,MAAM,EAAE,yBAAyB,GAChC,OAAO,CAAC,aAAa,CAAC,CAuDxB"}

package/dist/core/functionGroup/create.js

@@ -0,0 +1,90 @@
+"use strict";
+/**
+ * FunctionGroup create operations
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.createFunctionGroup = createFunctionGroup;
+const connection_1 = require("@mcp-abap-adt/connection");
+const fast_xml_parser_1 = require("fast-xml-parser");
+const systemInfo_1 = require("../shared/systemInfo");
+const activation_1 = require("./activation");
+/**
+ * Create function group metadata via POST
+ */
+async function createFunctionGroupObject(connection, functionGroupName, description, packageName, transportRequest) {
+    const baseUrl = await connection.getBaseUrl();
+    const url = `${baseUrl}/sap/bc/adt/functions/groups${transportRequest ? `?corrNr=${transportRequest}` : ''}`;
+    // Get masterSystem and responsible (only for cloud systems)
+    // On cloud, getSystemInformation returns systemID and userName
+    // On on-premise, it returns null, so we don't add these attributes
+    const systemInfo = await (0, systemInfo_1.getSystemInformation)(connection);
+    const masterSystem = systemInfo?.systemID;
+    const username = systemInfo?.userName || process.env.SAP_USERNAME || process.env.SAP_USER || '';
+    const masterSystemAttr = masterSystem ? ` adtcore:masterSystem="${masterSystem}"` : '';
+    const responsibleAttr = username ? ` adtcore:responsible="${username}"` : '';
+    const xmlPayload = `<?xml version="1.0" encoding="UTF-8"?>
+<group:abapFunctionGroup xmlns:group="http://www.sap.com/adt/functions/groups" xmlns:adtcore="http://www.sap.com/adt/core" adtcore:description="${description}" adtcore:language="EN" adtcore:name="${functionGroupName}" adtcore:type="FUGR/F" adtcore:masterLanguage="EN"${masterSystemAttr}${responsibleAttr}>
+  <adtcore:packageRef adtcore:name="${packageName}"/>
+</group:abapFunctionGroup>`;
+    const headers = {
+        'Content-Type': 'application/vnd.sap.adt.functions.groups.v3+xml',
+        'Accept': 'application/vnd.sap.adt.functions.groups.v3+xml'
+    };
+    return connection.makeAdtRequest({
+        url,
+        method: 'POST',
+        timeout: (0, connection_1.getTimeout)('default'),
+        data: xmlPayload,
+        headers
+    });
+}
+/**
+ * Create ABAP function group
+ * Full workflow: create -> activate (optional)
+ */
+async function createFunctionGroup(connection, params) {
+    if (!params.function_group_name || !params.package_name) {
+        throw new Error('function_group_name and package_name are required');
+    }
+    if (params.function_group_name.length > 26) {
+        throw new Error('Function group name must not exceed 26 characters');
+    }
+    if (!/^[ZY]/i.test(params.function_group_name)) {
+        throw new Error('Function group name must start with Z or Y (customer namespace)');
+    }
+    try {
+        const createResponse = await createFunctionGroupObject(connection, params.function_group_name, params.description || params.function_group_name, params.package_name, params.transport_request);
+        const shouldActivate = params.activate !== false;
+        if (shouldActivate) {
+            await (0, activation_1.activateFunctionGroup)(connection, params.function_group_name);
+        }
+        // Return the real response from SAP (from initial POST)
+        return createResponse;
+    }
+    catch (error) {
+        let errorMessage = `Failed to create function group: ${error}`;
+        if (error.response?.status === 400) {
+            errorMessage = `Bad request. Check if function group name is valid and package exists.`;
+        }
+        else if (error.response?.status === 409) {
+            errorMessage = `Function group ${params.function_group_name} already exists.`;
+        }
+        else if (error.response?.data && typeof error.response.data === 'string') {
+            try {
+                const parser = new fast_xml_parser_1.XMLParser({
+                    ignoreAttributes: false,
+                    attributeNamePrefix: '@_'
+                });
+                const errorData = parser.parse(error.response.data);
+                const errorMsg = errorData['exc:exception']?.message?.['#text'] || errorData['exc:exception']?.message;
+                if (errorMsg) {
+                    errorMessage = `SAP Error: ${errorMsg}`;
+                }
+            }
+            catch (parseError) {
+                // Ignore parse errors, use default message
+            }
+        }
+        throw new Error(errorMessage);
+    }
+}

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

@@ -0,0 +1,14 @@
+/**
+ * FunctionGroup delete operations
+ */
+import { AbapConnection } from '@mcp-abap-adt/connection';
+import { AxiosResponse } from 'axios';
+export interface DeleteFunctionGroupParams {
+    function_group_name: string;
+    transport_request?: string;
+}
+/**
+ * Delete ABAP function group
+ */
+export declare function deleteFunctionGroup(connection: AbapConnection, params: DeleteFunctionGroupParams): Promise<AxiosResponse>;
+//# sourceMappingURL=delete.d.ts.map

package/dist/core/functionGroup/delete.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"delete.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/delete.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAC1D,OAAO,EAAE,aAAa,EAAE,MAAM,OAAO,CAAC;AAGtC,MAAM,WAAW,yBAAyB;IACxC,mBAAmB,EAAE,MAAM,CAAC;IAC5B,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B;AAED;;GAEG;AACH,wBAAsB,mBAAmB,CACvC,UAAU,EAAE,cAAc,EAC1B,MAAM,EAAE,yBAAyB,GAChC,OAAO,CAAC,aAAa,CAAC,CAYxB"}

package/dist/core/functionGroup/delete.js

@@ -0,0 +1,21 @@
+"use strict";
+/**
+ * FunctionGroup delete operations
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.deleteFunctionGroup = deleteFunctionGroup;
+const delete_1 = require("../delete");
+/**
+ * Delete ABAP function group
+ */
+async function deleteFunctionGroup(connection, params) {
+    if (!params.function_group_name) {
+        throw new Error('function_group_name is required');
+    }
+    const deleteParams = {
+        object_name: params.function_group_name,
+        object_type: 'FUGR/F',
+        transport_request: params.transport_request
+    };
+    return await (0, delete_1.deleteObject)(connection, deleteParams);
+}

package/dist/core/functionGroup/index.d.ts

@@ -0,0 +1,15 @@
+/**
+ * FunctionGroup operations - exports
+ */
+export * from './types';
+export * from './create';
+export * from './read';
+export * from './activation';
+export * from './check';
+export * from './lock';
+export * from './unlock';
+export * from './update';
+export * from './delete';
+export * from './validation';
+export { FunctionGroupBuilder, type FunctionGroupBuilderConfig, type FunctionGroupBuilderLogger, type FunctionGroupBuilderState } from './FunctionGroupBuilder';
+//# sourceMappingURL=index.d.ts.map

package/dist/core/functionGroup/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/index.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,cAAc,SAAS,CAAC;AACxB,cAAc,UAAU,CAAC;AACzB,cAAc,QAAQ,CAAC;AACvB,cAAc,cAAc,CAAC;AAC7B,cAAc,SAAS,CAAC;AACxB,cAAc,QAAQ,CAAC;AACvB,cAAc,UAAU,CAAC;AACzB,cAAc,UAAU,CAAC;AACzB,cAAc,UAAU,CAAC;AACzB,cAAc,cAAc,CAAC;AAC7B,OAAO,EAAE,oBAAoB,EAAE,KAAK,0BAA0B,EAAE,KAAK,0BAA0B,EAAE,KAAK,yBAAyB,EAAE,MAAM,wBAAwB,CAAC"}

package/dist/core/functionGroup/index.js

@@ -0,0 +1,32 @@
+"use strict";
+/**
+ * FunctionGroup operations - exports
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __exportStar = (this && this.__exportStar) || function(m, exports) {
+    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.FunctionGroupBuilder = void 0;
+__exportStar(require("./types"), exports);
+__exportStar(require("./create"), exports);
+__exportStar(require("./read"), exports);
+__exportStar(require("./activation"), exports);
+__exportStar(require("./check"), exports);
+__exportStar(require("./lock"), exports);
+__exportStar(require("./unlock"), exports);
+__exportStar(require("./update"), exports);
+__exportStar(require("./delete"), exports);
+__exportStar(require("./validation"), exports);
+var FunctionGroupBuilder_1 = require("./FunctionGroupBuilder");
+Object.defineProperty(exports, "FunctionGroupBuilder", { enumerable: true, get: function () { return FunctionGroupBuilder_1.FunctionGroupBuilder; } });

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

@@ -0,0 +1,25 @@
+/**
+ * Lock Function Group operations
+ */
+import type { AbapConnection } from '@mcp-abap-adt/connection';
+import { AxiosResponse } from 'axios';
+/**
+ * Lock a function group for editing
+ *
+ * @param connection - ABAP connection
+ * @param functionGroupName - Name of the function group (e.g., 'Z_FUGR_TEST_0001')
+ * @param sessionId - Optional session ID for tracking
+ * @returns Lock handle string
+ */
+export declare function lockFunctionGroup(connection: AbapConnection, functionGroupName: string, sessionId?: string): Promise<string>;
+/**
+ * Unlock a function group
+ *
+ * @param connection - ABAP connection
+ * @param functionGroupName - Name of the function group
+ * @param lockHandle - Lock handle from lockFunctionGroup
+ * @param sessionId - Optional session ID for tracking
+ * @returns AxiosResponse from unlock request
+ */
+export declare function unlockFunctionGroup(connection: AbapConnection, functionGroupName: string, lockHandle: string, sessionId?: string): Promise<AxiosResponse>;
+//# sourceMappingURL=lock.d.ts.map

package/dist/core/functionGroup/lock.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"lock.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/lock.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAC/D,OAAO,EAAE,aAAa,EAAE,MAAM,OAAO,CAAC;AAItC;;;;;;;GAOG;AACH,wBAAsB,iBAAiB,CACrC,UAAU,EAAE,cAAc,EAC1B,iBAAiB,EAAE,MAAM,EACzB,SAAS,GAAE,MAAW,GACrB,OAAO,CAAC,MAAM,CAAC,CAwBjB;AAED;;;;;;;;GAQG;AACH,wBAAsB,mBAAmB,CACvC,UAAU,EAAE,cAAc,EAC1B,iBAAiB,EAAE,MAAM,EACzB,UAAU,EAAE,MAAM,EAClB,SAAS,GAAE,MAAW,GACrB,OAAO,CAAC,aAAa,CAAC,CAIxB"}

package/dist/core/functionGroup/lock.js

@@ -0,0 +1,50 @@
+"use strict";
+/**
+ * Lock Function Group operations
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.lockFunctionGroup = lockFunctionGroup;
+exports.unlockFunctionGroup = unlockFunctionGroup;
+const sessionUtils_1 = require("../../utils/sessionUtils");
+const fast_xml_parser_1 = require("fast-xml-parser");
+/**
+ * Lock a function group for editing
+ *
+ * @param connection - ABAP connection
+ * @param functionGroupName - Name of the function group (e.g., 'Z_FUGR_TEST_0001')
+ * @param sessionId - Optional session ID for tracking
+ * @returns Lock handle string
+ */
+async function lockFunctionGroup(connection, functionGroupName, sessionId = '') {
+    const url = `/sap/bc/adt/functions/groups/${functionGroupName.toLowerCase()}?_action=LOCK&accessMode=MODIFY`;
+    const headers = {
+        'Accept': 'application/vnd.sap.as+xml;charset=UTF-8;dataname=com.sap.adt.lock.result;q=0.8, application/vnd.sap.as+xml;charset=UTF-8;dataname=com.sap.adt.lock.result2;q=0.9',
+    };
+    const response = await (0, sessionUtils_1.makeAdtRequestWithSession)(connection, url, 'POST', sessionId, null, headers);
+    // Extract lock handle from response header
+    const lockHandle = response.headers['sap-adt-lm-handle'];
+    if (!lockHandle) {
+        // Try parsing from XML body if header not present
+        const parser = new fast_xml_parser_1.XMLParser({ ignoreAttributes: false, attributeNamePrefix: '' });
+        const result = parser.parse(response.data);
+        const xmlLockHandle = result?.['asx:abap']?.['asx:values']?.['DATA']?.['LOCK_HANDLE'];
+        if (!xmlLockHandle) {
+            throw new Error('Failed to acquire lock: no lock handle in response');
+        }
+        return xmlLockHandle;
+    }
+    return lockHandle;
+}
+/**
+ * Unlock a function group
+ *
+ * @param connection - ABAP connection
+ * @param functionGroupName - Name of the function group
+ * @param lockHandle - Lock handle from lockFunctionGroup
+ * @param sessionId - Optional session ID for tracking
+ * @returns AxiosResponse from unlock request
+ */
+async function unlockFunctionGroup(connection, functionGroupName, lockHandle, sessionId = '') {
+    const url = `/sap/bc/adt/functions/groups/${functionGroupName.toLowerCase()}?_action=UNLOCK&lockHandle=${lockHandle}`;
+    return (0, sessionUtils_1.makeAdtRequestWithSession)(connection, url, 'POST', sessionId, null);
+}

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

@@ -0,0 +1,17 @@
+/**
+ * FunctionGroup read operations
+ */
+import { AbapConnection } from '@mcp-abap-adt/connection';
+import { AxiosResponse } from 'axios';
+/**
+ * Get ABAP function group
+ */
+export declare function getFunctionGroup(connection: AbapConnection, functionGroupName: string): Promise<AxiosResponse>;
+/**
+ * Get transport request for ABAP function group
+ * @param connection - SAP connection
+ * @param functionGroupName - Function group name
+ * @returns Transport request information
+ */
+export declare function getFunctionGroupTransport(connection: AbapConnection, functionGroupName: string): Promise<AxiosResponse>;
+//# sourceMappingURL=read.d.ts.map

package/dist/core/functionGroup/read.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"read.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/read.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,cAAc,EAAc,MAAM,0BAA0B,CAAC;AACtE,OAAO,EAAE,aAAa,EAAE,MAAM,OAAO,CAAC;AAGtC;;GAEG;AACH,wBAAsB,gBAAgB,CAAC,UAAU,EAAE,cAAc,EAAE,iBAAiB,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,CAAC,CAWpH;AAED;;;;;GAKG;AACH,wBAAsB,yBAAyB,CAC7C,UAAU,EAAE,cAAc,EAC1B,iBAAiB,EAAE,MAAM,GACxB,OAAO,CAAC,aAAa,CAAC,CAaxB"}

package/dist/core/functionGroup/read.js

@@ -0,0 +1,42 @@
+"use strict";
+/**
+ * FunctionGroup read operations
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.getFunctionGroup = getFunctionGroup;
+exports.getFunctionGroupTransport = getFunctionGroupTransport;
+const connection_1 = require("@mcp-abap-adt/connection");
+const internalUtils_1 = require("../../utils/internalUtils");
+/**
+ * Get ABAP function group
+ */
+async function getFunctionGroup(connection, functionGroupName) {
+    const baseUrl = await connection.getBaseUrl();
+    const encodedName = (0, internalUtils_1.encodeSapObjectName)(functionGroupName);
+    const url = `${baseUrl}/sap/bc/adt/functions/groups/${encodedName}`;
+    return connection.makeAdtRequest({
+        url,
+        method: 'GET',
+        timeout: (0, connection_1.getTimeout)('default'),
+        headers: {}
+    });
+}
+/**
+ * Get transport request for ABAP function group
+ * @param connection - SAP connection
+ * @param functionGroupName - Function group name
+ * @returns Transport request information
+ */
+async function getFunctionGroupTransport(connection, functionGroupName) {
+    const baseUrl = await connection.getBaseUrl();
+    const encodedName = (0, internalUtils_1.encodeSapObjectName)(functionGroupName);
+    const url = `${baseUrl}/sap/bc/adt/functions/groups/${encodedName}/transport`;
+    return connection.makeAdtRequest({
+        url,
+        method: 'GET',
+        timeout: (0, connection_1.getTimeout)('default'),
+        headers: {
+            'Accept': 'application/vnd.sap.adt.transportorganizer.v1+xml'
+        }
+    });
+}

package/dist/core/functionGroup/types.d.ts

@@ -0,0 +1,11 @@
+/**
+ * FunctionGroup types
+ */
+export interface CreateFunctionGroupParams {
+    function_group_name: string;
+    description?: string;
+    package_name: string;
+    transport_request?: string;
+    activate?: boolean;
+}
+//# sourceMappingURL=types.d.ts.map

package/dist/core/functionGroup/types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/types.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,MAAM,WAAW,yBAAyB;IACxC,mBAAmB,EAAE,MAAM,CAAC;IAC5B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,EAAE,MAAM,CAAC;IACrB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB"}

package/dist/core/functionGroup/types.js

@@ -0,0 +1,5 @@
+"use strict";
+/**
+ * FunctionGroup types
+ */
+Object.defineProperty(exports, "__esModule", { value: true });

package/dist/core/functionGroup/unlock.d.ts

@@ -0,0 +1,5 @@
+/**
+ * FunctionGroup unlock operations
+ */
+export { unlockFunctionGroup } from './lock';
+//# sourceMappingURL=unlock.d.ts.map

package/dist/core/functionGroup/unlock.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"unlock.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/unlock.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,mBAAmB,EAAE,MAAM,QAAQ,CAAC"}

package/dist/core/functionGroup/unlock.js

@@ -0,0 +1,8 @@
+"use strict";
+/**
+ * FunctionGroup unlock operations
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.unlockFunctionGroup = void 0;
+var lock_1 = require("./lock");
+Object.defineProperty(exports, "unlockFunctionGroup", { enumerable: true, get: function () { return lock_1.unlockFunctionGroup; } });

package/dist/core/functionGroup/update.d.ts

@@ -0,0 +1,20 @@
+/**
+ * FunctionGroup update operations
+ *
+ * Note: Function groups are containers for function modules.
+ * They don't have source code to update directly, but metadata can be updated.
+ */
+import { AbapConnection } from '@mcp-abap-adt/connection';
+import { AxiosResponse } from 'axios';
+export interface UpdateFunctionGroupParams {
+    function_group_name: string;
+    description?: string;
+    transport_request?: string;
+    lock_handle?: string;
+}
+/**
+ * Update function group metadata (description)
+ * Full workflow: lock -> get current -> update -> unlock
+ */
+export declare function updateFunctionGroup(connection: AbapConnection, params: UpdateFunctionGroupParams): Promise<AxiosResponse>;
+//# sourceMappingURL=update.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/update.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,cAAc,EAAc,MAAM,0BAA0B,CAAC;AACtE,OAAO,EAAE,aAAa,EAAE,MAAM,OAAO,CAAC;AAOtC,MAAM,WAAW,yBAAyB;IACxC,mBAAmB,EAAE,MAAM,CAAC;IAC5B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAqDD;;;GAGG;AACH,wBAAsB,mBAAmB,CACvC,UAAU,EAAE,cAAc,EAC1B,MAAM,EAAE,yBAAyB,GAChC,OAAO,CAAC,aAAa,CAAC,CA+ExB"}

package/dist/core/functionGroup/update.js

@@ -0,0 +1,119 @@
+"use strict";
+/**
+ * FunctionGroup update operations
+ *
+ * Note: Function groups are containers for function modules.
+ * They don't have source code to update directly, but metadata can be updated.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.updateFunctionGroup = updateFunctionGroup;
+const connection_1 = require("@mcp-abap-adt/connection");
+const fast_xml_parser_1 = require("fast-xml-parser");
+const internalUtils_1 = require("../../utils/internalUtils");
+const read_1 = require("./read");
+const lock_1 = require("./lock");
+const unlock_1 = require("./unlock");
+/**
+ * Update function group metadata via PUT
+ */
+async function updateFunctionGroupMetadata(connection, functionGroupName, currentXml, newDescription, lockHandle, transportRequest) {
+    const baseUrl = await connection.getBaseUrl();
+    const encodedName = (0, internalUtils_1.encodeSapObjectName)(functionGroupName);
+    const url = `${baseUrl}/sap/bc/adt/functions/groups/${encodedName}${lockHandle ? `?lockHandle=${lockHandle}` : ''}${transportRequest ? `${lockHandle ? '&' : '?'}corrNr=${transportRequest}` : ''}`;
+    // Parse current XML to update description
+    const parser = new fast_xml_parser_1.XMLParser({
+        ignoreAttributes: false,
+        attributeNamePrefix: '@_',
+        parseAttributeValue: false
+    });
+    const parsedXml = parser.parse(currentXml);
+    const functionGroup = parsedXml['group:abapFunctionGroup'];
+    // Update description
+    if (functionGroup) {
+        functionGroup['@_adtcore:description'] = newDescription;
+    }
+    // Rebuild XML (simplified - use original XML with replaced description)
+    const updatedXml = currentXml.replace(/adtcore:description="[^"]*"/, `adtcore:description="${newDescription}"`);
+    const headers = {
+        'Content-Type': 'application/vnd.sap.adt.functions.groups.v3+xml; charset=utf-8',
+        'Accept': 'application/vnd.sap.adt.functions.groups.v3+xml'
+    };
+    return connection.makeAdtRequest({
+        url,
+        method: 'PUT',
+        timeout: (0, connection_1.getTimeout)('default'),
+        data: updatedXml,
+        headers
+    });
+}
+/**
+ * Update function group metadata (description)
+ * Full workflow: lock -> get current -> update -> unlock
+ */
+async function updateFunctionGroup(connection, params) {
+    if (!params.function_group_name) {
+        throw new Error('function_group_name is required');
+    }
+    if (!params.description) {
+        throw new Error('description is required for update');
+    }
+    let lockHandle = params.lock_handle;
+    let shouldUnlock = false;
+    try {
+        // Lock if not already locked
+        if (!lockHandle) {
+            lockHandle = await (0, lock_1.lockFunctionGroup)(connection, params.function_group_name);
+            shouldUnlock = true;
+        }
+        if (!lockHandle) {
+            throw new Error('Failed to acquire lock handle');
+        }
+        // Get current XML
+        const currentResponse = await (0, read_1.getFunctionGroup)(connection, params.function_group_name);
+        const currentXml = typeof currentResponse.data === 'string'
+            ? currentResponse.data
+            : JSON.stringify(currentResponse.data);
+        // Update metadata
+        const updateResponse = await updateFunctionGroupMetadata(connection, params.function_group_name, currentXml, params.description, lockHandle, params.transport_request);
+        // Unlock if we locked it
+        if (shouldUnlock && lockHandle) {
+            await (0, unlock_1.unlockFunctionGroup)(connection, params.function_group_name, lockHandle);
+        }
+        return updateResponse;
+    }
+    catch (error) {
+        // Unlock on error if we locked it
+        if (shouldUnlock && lockHandle) {
+            try {
+                await (0, unlock_1.unlockFunctionGroup)(connection, params.function_group_name, lockHandle);
+            }
+            catch (unlockError) {
+                // Ignore unlock errors
+            }
+        }
+        let errorMessage = `Failed to update function group: ${error}`;
+        if (error.response?.status === 400) {
+            errorMessage = `Bad request. Check parameters.`;
+        }
+        else if (error.response?.status === 404) {
+            errorMessage = `Function group ${params.function_group_name} not found.`;
+        }
+        else if (error.response?.data && typeof error.response.data === 'string') {
+            try {
+                const parser = new fast_xml_parser_1.XMLParser({
+                    ignoreAttributes: false,
+                    attributeNamePrefix: '@_'
+                });
+                const errorData = parser.parse(error.response.data);
+                const errorMsg = errorData['exc:exception']?.message?.['#text'] || errorData['exc:exception']?.message;
+                if (errorMsg) {
+                    errorMessage = `SAP Error: ${errorMsg}`;
+                }
+            }
+            catch (parseError) {
+                // Keep original error message
+            }
+        }
+        throw new Error(errorMessage);
+    }
+}

package/dist/core/functionGroup/validation.d.ts

@@ -0,0 +1,10 @@
+/**
+ * Function Group validation
+ */
+import { AbapConnection } from '@mcp-abap-adt/connection';
+import { ValidationResult } from '../shared/validation';
+/**
+ * Validate function group name
+ */
+export declare function validateFunctionGroupName(connection: AbapConnection, functionGroupName: string, description?: string): Promise<ValidationResult>;
+//# sourceMappingURL=validation.d.ts.map

package/dist/core/functionGroup/validation.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"validation.d.ts","sourceRoot":"","sources":["../../../src/core/functionGroup/validation.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAC1D,OAAO,EAAsB,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAE5E;;GAEG;AACH,wBAAsB,yBAAyB,CAC7C,UAAU,EAAE,cAAc,EAC1B,iBAAiB,EAAE,MAAM,EACzB,WAAW,CAAC,EAAE,MAAM,GACnB,OAAO,CAAC,gBAAgB,CAAC,CAQ3B"}

package/dist/core/functionGroup/validation.js

@@ -0,0 +1,17 @@
+"use strict";
+/**
+ * Function Group validation
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.validateFunctionGroupName = validateFunctionGroupName;
+const validation_1 = require("../shared/validation");
+/**
+ * Validate function group name
+ */
+async function validateFunctionGroupName(connection, functionGroupName, description) {
+    const params = {};
+    if (description) {
+        params.description = description;
+    }
+    return (0, validation_1.validateObjectName)(connection, 'FUGR/F', functionGroupName, params);
+}