@mcp-abap-adt/adt-clients 3.10.2 → 3.11.3

package/README.md

@@ -1,5 +1,7 @@
 # @mcp-abap-adt/adt-clients
 
+[![Stand With Ukraine](https://raw.githubusercontent.com/vshymanskyy/StandWithUkraine/main/badges/StandWithUkraine.svg)](https://stand-with-ukraine.pp.ua)
+
 TypeScript clients for SAP ABAP Development Tools (ADT).
 
 ## Features

package/dist/core/class/AdtClassLegacy.d.ts

@@ -1,11 +1,27 @@
 /**
  * AdtClassLegacy - Class handler for legacy SAP systems (BASIS < 7.50)
  *
- * Overrides delete() to use direct DELETE instead of /sap/bc/adt/deletion/ API.
+ * On legacy systems, the x-sap-adt-sessiontype: stateful header causes locks
+ * to be stored in ABAP session memory instead of the global enqueue server.
+ * This means lock + update + unlock MUST happen within the same stateful
+ * HTTP session — switching to stateless between lock and update invalidates
+ * the lock handle (GitHub #11).
+ *
+ * Overrides:
+ * - update() — keeps lock→check→update→unlock in one stateful session
+ * - delete() — uses direct DELETE instead of /sap/bc/adt/deletion/ API
  */
+import type { IAdtOperationOptions } from '@mcp-abap-adt/interfaces';
 import { AdtClass } from './AdtClass';
 import type { IClassConfig, IClassState } from './types';
 export declare class AdtClassLegacy extends AdtClass {
+    /**
+     * Update class — legacy override.
+     *
+     * Keeps lock→check→update→unlock in a single stateful session so the
+     * lock handle remains valid (legacy stores locks in ABAP session memory).
+     */
+    update(config: Partial<IClassConfig>, options?: IAdtOperationOptions): Promise<IClassState>;
     delete(config: Partial<IClassConfig>): Promise<IClassState>;
 }
 //# sourceMappingURL=AdtClassLegacy.d.ts.map

package/dist/core/class/AdtClassLegacy.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"AdtClassLegacy.d.ts","sourceRoot":"","sources":["../../../src/core/class/AdtClassLegacy.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAQH,OAAO,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAEtC,OAAO,KAAK,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAGzD,qBAAa,cAAe,SAAQ,QAAQ;IAC3B,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;CA+C3E"}
+{"version":3,"file":"AdtClassLegacy.d.ts","sourceRoot":"","sources":["../../../src/core/class/AdtClassLegacy.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,KAAK,EAAa,oBAAoB,EAAE,MAAM,0BAA0B,CAAC;AAMhF,OAAO,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAItC,OAAO,KAAK,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAIzD,qBAAa,cAAe,SAAQ,QAAQ;IAC1C;;;;;OAKG;IACY,MAAM,CACnB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,WAAW,CAAC;IA6HR,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;CA+C3E"}

package/dist/core/class/AdtClassLegacy.js

@@ -2,16 +2,115 @@
 /**
  * AdtClassLegacy - Class handler for legacy SAP systems (BASIS < 7.50)
  *
- * Overrides delete() to use direct DELETE instead of /sap/bc/adt/deletion/ API.
+ * On legacy systems, the x-sap-adt-sessiontype: stateful header causes locks
+ * to be stored in ABAP session memory instead of the global enqueue server.
+ * This means lock + update + unlock MUST happen within the same stateful
+ * HTTP session — switching to stateless between lock and update invalidates
+ * the lock handle (GitHub #11).
+ *
+ * Overrides:
+ * - update() — keeps lock→check→update→unlock in one stateful session
+ * - delete() — uses direct DELETE instead of /sap/bc/adt/deletion/ API
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.AdtClassLegacy = void 0;
 const internalUtils_1 = require("../../utils/internalUtils");
 const deleteLegacy_1 = require("../shared/deleteLegacy");
 const AdtClass_1 = require("./AdtClass");
+const activation_1 = require("./activation");
+const check_1 = require("./check");
 const lock_1 = require("./lock");
 const unlock_1 = require("./unlock");
+const update_1 = require("./update");
 class AdtClassLegacy extends AdtClass_1.AdtClass {
+    /**
+     * Update class — legacy override.
+     *
+     * Keeps lock→check→update→unlock in a single stateful session so the
+     * lock handle remains valid (legacy stores locks in ABAP session memory).
+     */
+    async update(config, options) {
+        if (!config.className) {
+            throw new Error('Class name is required');
+        }
+        // Low-level mode: caller owns the session
+        if (options?.lockHandle) {
+            return super.update(config, options);
+        }
+        let lockHandle;
+        const state = { errors: [] };
+        try {
+            // Enter stateful session for the entire lock→update→unlock chain
+            this.connection.setSessionType('stateful');
+            // 1. Lock
+            this.logger?.info?.('Legacy update step 1: Locking class');
+            lockHandle = await (0, lock_1.lockClass)(this.connection, config.className);
+            state.lockHandle = lockHandle;
+            this.logger?.info?.('Class locked, handle:', lockHandle);
+            // 2. Check inactive with source code
+            const codeToUpdate = options?.sourceCode || config.sourceCode;
+            if (codeToUpdate) {
+                this.logger?.info?.('Legacy update step 2: Checking inactive version');
+                state.checkResult = await (0, check_1.checkClass)(this.connection, config.className, 'inactive', codeToUpdate, this.contentTypes?.sourceArtifactContentType());
+                this.logger?.info?.('Check passed');
+            }
+            // 3. Update
+            if (codeToUpdate && lockHandle) {
+                this.logger?.info?.('Legacy update step 3: Updating class');
+                state.updateResult = await (0, update_1.updateClass)(this.connection, config.className, codeToUpdate, lockHandle, config.transportRequest, this.contentTypes?.sourceArtifactContentType());
+                this.logger?.info?.('Class updated');
+            }
+            // 4. Unlock (still within the same stateful session)
+            if (lockHandle) {
+                this.logger?.info?.('Legacy update step 4: Unlocking class');
+                state.unlockResult = await (0, unlock_1.unlockClass)(this.connection, config.className, lockHandle);
+                lockHandle = undefined;
+                this.logger?.info?.('Class unlocked');
+            }
+        }
+        catch (error) {
+            // Cleanup: try to unlock if still locked (within same session)
+            if (lockHandle) {
+                try {
+                    this.logger?.warn?.('Unlocking class during error cleanup');
+                    await (0, unlock_1.unlockClass)(this.connection, config.className, lockHandle);
+                }
+                catch (unlockError) {
+                    this.logger?.warn?.('Failed to unlock during cleanup:', (0, internalUtils_1.safeErrorMessage)(unlockError));
+                }
+            }
+            throw error;
+        }
+        finally {
+            // Always return to stateless after the chain
+            this.connection.setSessionType('stateless');
+        }
+        // Post-lock operations (stateless is fine)
+        // 5. Final check
+        this.logger?.info?.('Legacy update step 5: Final check');
+        state.checkResult = await (0, check_1.checkClass)(this.connection, config.className, 'inactive');
+        this.logger?.info?.('Final check passed');
+        // 6. Activate (if requested)
+        if (options?.activateOnUpdate) {
+            this.logger?.info?.('Legacy update step 6: Activating class');
+            const activateResult = await (0, activation_1.activateClass)(this.connection, config.className);
+            state.activateResult = activateResult;
+            this.logger?.info?.('Class activated, status:', activateResult.status);
+            // Read with long polling to ensure object is ready after activation
+            this.logger?.info?.('Read (wait for object ready after activation)');
+            try {
+                const readState = await this.read({ className: config.className }, 'active', { withLongPolling: true });
+                if (readState) {
+                    state.readResult = readState.readResult;
+                }
+                this.logger?.info?.('Object is ready after activation');
+            }
+            catch (readError) {
+                this.logger?.warn?.('Read with long polling failed after activation:', (0, internalUtils_1.safeErrorMessage)(readError));
+            }
+        }
+        return state;
+    }
     async delete(config) {
         if (!config.className) {
             throw new Error('Class name is required');

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

@@ -13,5 +13,5 @@ import type { IAdtResponse as AxiosResponse, IAbapConnection, ILogger } from '@m
  *
  * Note: When xmlContent is provided, it should be the same XML that will be sent in PUT request.
  */
-export declare function checkDomainSyntax(connection: IAbapConnection, domainName: string, version: 'active' | 'inactive', xmlContent?: string, logger?: ILogger): Promise<AxiosResponse>;
+export declare function checkDomainSyntax(connection: IAbapConnection, domainName: string, version: 'active' | 'inactive', xmlContent?: string, _logger?: ILogger): Promise<AxiosResponse>;
 //# sourceMappingURL=check.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"check.d.ts","sourceRoot":"","sources":["../../../src/core/domain/check.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAC7B,eAAe,EACf,OAAO,EACR,MAAM,0BAA0B,CAAC;AASlC;;;;;;;;;;GAUG;AACH,wBAAsB,iBAAiB,CACrC,UAAU,EAAE,eAAe,EAC3B,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,QAAQ,GAAG,UAAU,EAC9B,UAAU,CAAC,EAAE,MAAM,EACnB,MAAM,CAAC,EAAE,OAAO,GACf,OAAO,CAAC,aAAa,CAAC,CAuDxB"}
+{"version":3,"file":"check.d.ts","sourceRoot":"","sources":["../../../src/core/domain/check.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAC7B,eAAe,EACf,OAAO,EACR,MAAM,0BAA0B,CAAC;AASlC;;;;;;;;;;GAUG;AACH,wBAAsB,iBAAiB,CACrC,UAAU,EAAE,eAAe,EAC3B,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,QAAQ,GAAG,UAAU,EAC9B,UAAU,CAAC,EAAE,MAAM,EACnB,OAAO,CAAC,EAAE,OAAO,GAChB,OAAO,CAAC,aAAa,CAAC,CAuDxB"}

package/dist/core/domain/check.js

@@ -19,7 +19,7 @@ const timeouts_1 = require("../../utils/timeouts");
  *
  * Note: When xmlContent is provided, it should be the same XML that will be sent in PUT request.
  */
-async function checkDomainSyntax(connection, domainName, version, xmlContent, logger) {
+async function checkDomainSyntax(connection, domainName, version, xmlContent, _logger) {
     let response;
     if (xmlContent) {
         // Check with XML content (for unsaved changes or new content validation)

package/dist/core/package/AdtPackage.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"AdtPackage.d.ts","sourceRoot":"","sources":["../../../src/core/package/AdtPackage.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAEH,OAAO,KAAK,EAEV,eAAe,EACf,UAAU,EACV,oBAAoB,EACpB,OAAO,EACR,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AAEjE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAMpD,OAAO,KAAK,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,SAAS,CAAC;AAK7D,qBAAa,UAAW,YAAW,UAAU,CAAC,cAAc,EAAE,aAAa,CAAC;IAC1E,SAAS,CAAC,QAAQ,CAAC,UAAU,EAAE,eAAe,CAAC;IAC/C,SAAS,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,OAAO,CAAC;IACpC,SAAS,CAAC,QAAQ,CAAC,aAAa,EAAE,iBAAiB,CAAC;IACpD,SAAgB,UAAU,EAAE,MAAM,CAAa;gBAG7C,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,iBAAiB;IAOnC;;OAEG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,aAAa,CAAC;IA0BvE;;;OAGG;IACG,MAAM,CACV,MAAM,EAAE,cAAc,EACtB,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,aAAa,CAAC;IAiGzB;;OAEG;IACG,IAAI,CACR,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,OAAO,CAAC,EAAE,QAAQ,GAAG,UAAU,EAC/B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,aAAa,GAAG,SAAS,CAAC;IA0BrC;;;OAGG;IACG,YAAY,CAChB,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,aAAa,CAAC;IAoCzB;;OAEG;IACG,aAAa,CACjB,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,OAAO,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,OAAO,CAAA;KAAE,GACtC,OAAO,CAAC,aAAa,CAAC;IAkCzB;;;;;OAKG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,aAAa,CAAC;IAwKzB;;OAEG;IACG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,aAAa,CAAC;IA6BrE;;;OAGG;IACG,QAAQ,CAAC,OAAO,EAAE,OAAO,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,aAAa,CAAC;IAMxE;;OAEG;IACG,KAAK,CACT,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,aAAa,CAAC;IAkBzB;;OAEG;IACG,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAW5D;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,aAAa,CAAC;CAiB1B"}
+{"version":3,"file":"AdtPackage.d.ts","sourceRoot":"","sources":["../../../src/core/package/AdtPackage.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAEH,OAAO,KAAK,EAEV,eAAe,EACf,UAAU,EACV,oBAAoB,EACpB,OAAO,EACR,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AAEjE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAMpD,OAAO,KAAK,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,SAAS,CAAC;AAK7D,qBAAa,UAAW,YAAW,UAAU,CAAC,cAAc,EAAE,aAAa,CAAC;IAC1E,SAAS,CAAC,QAAQ,CAAC,UAAU,EAAE,eAAe,CAAC;IAC/C,SAAS,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,OAAO,CAAC;IACpC,SAAS,CAAC,QAAQ,CAAC,aAAa,EAAE,iBAAiB,CAAC;IACpD,SAAgB,UAAU,EAAE,MAAM,CAAa;gBAG7C,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,iBAAiB;IAOnC;;OAEG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,aAAa,CAAC;IA2BvE;;;OAGG;IACG,MAAM,CACV,MAAM,EAAE,cAAc,EACtB,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,aAAa,CAAC;IA6FzB;;OAEG;IACG,IAAI,CACR,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,OAAO,CAAC,EAAE,QAAQ,GAAG,UAAU,EAC/B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,aAAa,GAAG,SAAS,CAAC;IA0BrC;;;OAGG;IACG,YAAY,CAChB,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,aAAa,CAAC;IAoCzB;;OAEG;IACG,aAAa,CACjB,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,OAAO,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,OAAO,CAAA;KAAE,GACtC,OAAO,CAAC,aAAa,CAAC;IAkCzB;;;;;OAKG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,aAAa,CAAC;IAsLzB;;OAEG;IACG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,aAAa,CAAC;IA6BrE;;;OAGG;IACG,QAAQ,CAAC,OAAO,EAAE,OAAO,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,aAAa,CAAC;IAMxE;;OAEG;IACG,KAAK,CACT,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,aAAa,CAAC;IAkBzB;;OAEG;IACG,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAW5D;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,cAAc,CAAC,EAC/B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,aAAa,CAAC;CAiB1B"}

package/dist/core/package/AdtPackage.js

@@ -62,6 +62,7 @@ class AdtPackage {
             transport_request: config.transportRequest,
             application_component: config.applicationComponent,
             responsible: config.responsible,
+            record_changes: config.recordChanges ?? false,
         });
         return {
             validationResponse: response,
@@ -99,6 +100,7 @@ class AdtPackage {
                 transport_request: config.transportRequest,
                 application_component: config.applicationComponent,
                 responsible: config.responsible,
+                record_changes: config.recordChanges ?? false,
             });
             this.logger?.info?.('Validation passed');
             // 2. Create (no stateful needed)
@@ -113,8 +115,8 @@ class AdtPackage {
                 transport_request: config.transportRequest,
                 application_component: config.applicationComponent,
                 responsible: config.responsible ?? this.systemContext.responsible,
-                masterSystem: this.systemContext.masterSystem,
-                record_changes: config.recordChanges,
+                master_system: this.systemContext.masterSystem,
+                record_changes: config.recordChanges ?? false,
             });
             this.logger?.info?.('Package created');
             // 2.5. Read with long polling (wait for object to be ready)
@@ -130,11 +132,7 @@ class AdtPackage {
                 // Continue anyway - check might still work
             }
             objectCreated = true;
-            // 3. Check after create (no stateful needed)
-            this.logger?.info?.('Step 3: Checking created package');
-            await (0, check_1.checkPackage)(this.connection, config.packageName, 'inactive');
-            this.logger?.info?.('Check after create passed');
-            // Packages are containers — no source code, no activation, no need to read back
+            // Packages are containers — no source code, no activation, no syntax check needed
             return { errors: [] };
         }
         catch (error) {
@@ -273,7 +271,7 @@ class AdtPackage {
                 description: config.description,
                 package_type: config.packageType,
                 responsible: config.responsible,
-                record_changes: config.recordChanges,
+                record_changes: config.recordChanges ?? false,
             }, options.lockHandle);
             this.logger?.info?.('Package updated (low-level)');
             return {
@@ -281,14 +279,28 @@ class AdtPackage {
                 errors: [],
             };
         }
+        // TODO: Package update via RFC (SADT_REST_RFC_ENDPOINT) fails with HTTP 400
+        // "Package is already locked" on PUT even though LOCK/UNLOCK succeed.
+        // Root cause: PAK framework locks are session-scoped. Each call to
+        // SADT_REST_RFC_ENDPOINT runs in a new internal ABAP context, so the
+        // PUT cannot access the PAK lock created by the LOCK call.
+        // DDIC objects are unaffected because their locks live in the DDIC buffer
+        // (accessible by lockHandle from any context).
+        // This is non-critical for release — HTTP is the primary transport for
+        // modern on-premise systems. RFC is used for legacy (BASIS < 7.50) where
+        // package CRUD is not supported anyway.
+        // Reference: docs/development/RFC_TESTING.md
         let lockHandle;
+        let lockCorrNr;
         try {
-            // 1. Lock (update always starts with lock, stateful ONLY before lock)
+            // 1. Lock — stateful mode stays active until after unlock
             this.logger?.info?.('Step 1: Locking package');
             this.connection.setSessionType('stateful');
-            lockHandle = await (0, lock_1.lockPackage)(this.connection, config.packageName);
+            const lockResult = await (0, lock_1.lockPackage)(this.connection, config.packageName);
+            lockHandle = lockResult.lockHandle;
+            lockCorrNr = lockResult.corrNr;
             this.connection.setSessionType('stateless');
-            this.logger?.info?.('Package locked, handle:', lockHandle);
+            this.logger?.info?.(`Package locked, handle: ${lockHandle}, corrNr: ${lockCorrNr}`);
             // 2. Check inactive with XML for update (if provided)
             const xmlToCheck = options?.xmlContent;
             if (xmlToCheck) {
@@ -308,9 +320,10 @@ class AdtPackage {
                     package_type: config.packageType,
                     software_component: config.softwareComponent,
                     transport_layer: config.transportLayer,
-                    transport_request: config.transportRequest,
+                    transport_request: config.transportRequest || lockCorrNr,
                     application_component: config.applicationComponent,
                     responsible: config.responsible ?? this.systemContext.responsible,
+                    master_system: config.masterSystem ?? this.systemContext.masterSystem,
                     record_changes: config.recordChanges,
                 }, lockHandle);
                 this.logger?.info?.('Package updated');
@@ -327,7 +340,7 @@ class AdtPackage {
                     // Continue anyway - unlock might still work
                 }
             }
-            // 4. Unlock (obligatory stateless after unlock)
+            // 4. Unlock — set stateful before unlock, stateless after (standard pattern)
             if (lockHandle) {
                 this.logger?.info?.('Step 4: Unlocking package');
                 this.connection.setSessionType('stateful');
@@ -336,11 +349,7 @@ class AdtPackage {
                 lockHandle = undefined;
                 this.logger?.info?.('Package unlocked');
             }
-            // 5. Final check (no stateful needed)
-            this.logger?.info?.('Step 5: Final check');
-            await (0, check_1.checkPackage)(this.connection, config.packageName, 'inactive');
-            this.logger?.info?.('Final check passed');
-            // Note: Packages don't have activate operation
+            // Note: Packages have no source code — no check or activate needed
             // Read and return result (no stateful needed)
             const readResponse = await (0, read_1.getPackage)(this.connection, config.packageName);
             return {
@@ -440,9 +449,9 @@ class AdtPackage {
             throw new Error('Package name is required');
         }
         this.connection.setSessionType('stateful');
-        const lockHandle = await (0, lock_1.lockPackage)(this.connection, config.packageName);
+        const lockResult = await (0, lock_1.lockPackage)(this.connection, config.packageName);
         this.connection.setSessionType('stateless');
-        return lockHandle;
+        return lockResult.lockHandle;
     }
     /**
      * Unlock package

package/dist/core/package/create.js

@@ -24,7 +24,7 @@ async function createPackage(connection, params) {
     // Description is limited to 60 characters in SAP ADT
     const description = escapeXml((0, internalUtils_1.limitDescription)(params.description || params.package_name));
     const packageType = params.package_type || 'development';
-    const masterSystem = params.masterSystem;
+    const masterSystem = params.master_system;
     const responsibleUser = params.responsible || '';
     // Software component is required for package creation
     if (!params.software_component) {

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

@@ -1,18 +1,14 @@
 /**
  * Package delete operations
  */
-import type { IAdtResponse as AxiosResponse, IAbapConnection } from '@mcp-abap-adt/interfaces';
-export interface DeletePackageParams {
-    package_name: string;
-    transport_request?: string;
-}
+import type { IAdtResponse as AxiosResponse, IAbapConnection, IDeletePackageParams } from '@mcp-abap-adt/interfaces';
 /**
  * Check if package can be deleted (deletion check)
  * Returns response with isDeletable flag
  *
  * NOTE: Uses stateful session headers automatically if connection has stateful mode enabled
  */
-export declare function checkPackageDeletion(connection: IAbapConnection, params: DeletePackageParams): Promise<AxiosResponse>;
+export declare function checkPackageDeletion(connection: IAbapConnection, params: IDeletePackageParams): Promise<AxiosResponse>;
 /**
  * Parse deletion check response to get isDeletable flag
  */
@@ -24,5 +20,5 @@ export declare function parsePackageDeletionCheck(response: AxiosResponse): {
  * Delete ABAP package using ADT deletion API
  * For packages, empty transportNumber tag may be required
  */
-export declare function deletePackage(connection: IAbapConnection, params: DeletePackageParams): Promise<AxiosResponse>;
+export declare function deletePackage(connection: IAbapConnection, params: IDeletePackageParams): Promise<AxiosResponse>;
 //# sourceMappingURL=delete.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"delete.d.ts","sourceRoot":"","sources":["../../../src/core/package/delete.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAE7B,eAAe,EAChB,MAAM,0BAA0B,CAAC;AAWlC,MAAM,WAAW,mBAAmB;IAClC,YAAY,EAAE,MAAM,CAAC;IACrB,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B;AAED;;;;;GAKG;AACH,wBAAsB,oBAAoB,CACxC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,mBAAmB,GAC1B,OAAO,CAAC,aAAa,CAAC,CA4BxB;AAED;;GAEG;AACH,wBAAgB,yBAAyB,CAAC,QAAQ,EAAE,aAAa,GAAG;IAClE,WAAW,EAAE,OAAO,CAAC;IACrB,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB,CA+BA;AAED;;;GAGG;AACH,wBAAsB,aAAa,CACjC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,mBAAmB,GAC1B,OAAO,CAAC,aAAa,CAAC,CAwFxB"}
+{"version":3,"file":"delete.d.ts","sourceRoot":"","sources":["../../../src/core/package/delete.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAE7B,eAAe,EACf,oBAAoB,EACrB,MAAM,0BAA0B,CAAC;AAWlC;;;;;GAKG;AACH,wBAAsB,oBAAoB,CACxC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,oBAAoB,GAC3B,OAAO,CAAC,aAAa,CAAC,CA4BxB;AAED;;GAEG;AACH,wBAAgB,yBAAyB,CAAC,QAAQ,EAAE,aAAa,GAAG;IAClE,WAAW,EAAE,OAAO,CAAC;IACrB,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB,CA+BA;AAED;;;GAGG;AACH,wBAAsB,aAAa,CACjC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,oBAAoB,GAC3B,OAAO,CAAC,aAAa,CAAC,CAwFxB"}

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

@@ -9,5 +9,9 @@ import type { IAbapConnection } from '@mcp-abap-adt/interfaces';
  * NOTE: Caller must enable stateful session mode via connection.setSessionType("stateful")
  * before calling this function
  */
-export declare function lockPackage(connection: IAbapConnection, packageName: string): Promise<string>;
+export interface IPackageLockResult {
+    lockHandle: string;
+    corrNr?: string;
+}
+export declare function lockPackage(connection: IAbapConnection, packageName: string): Promise<IPackageLockResult>;
 //# sourceMappingURL=lock.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"lock.d.ts","sourceRoot":"","sources":["../../../src/core/package/lock.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAC;AAMhE;;;;;;GAMG;AACH,wBAAsB,WAAW,CAC/B,UAAU,EAAE,eAAe,EAC3B,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC,MAAM,CAAC,CA8BjB"}
+{"version":3,"file":"lock.d.ts","sourceRoot":"","sources":["../../../src/core/package/lock.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAC;AAMhE;;;;;;GAMG;AACH,MAAM,WAAW,kBAAkB;IACjC,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,wBAAsB,WAAW,CAC/B,UAAU,EAAE,eAAe,EAC3B,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC,kBAAkB,CAAC,CAgC7B"}

package/dist/core/package/lock.js

@@ -8,15 +8,8 @@ const fast_xml_parser_1 = require("fast-xml-parser");
 const contentTypes_1 = require("../../constants/contentTypes");
 const internalUtils_1 = require("../../utils/internalUtils");
 const timeouts_1 = require("../../utils/timeouts");
-/**
- * Lock package for modification
- * Returns lock handle that must be used in subsequent requests
- *
- * NOTE: Caller must enable stateful session mode via connection.setSessionType("stateful")
- * before calling this function
- */
 async function lockPackage(connection, packageName) {
-    const url = `/sap/bc/adt/packages/${(0, internalUtils_1.encodeSapObjectName)(packageName)}?_action=LOCK&accessMode=MODIFY`;
+    const url = `/sap/bc/adt/packages/${(0, internalUtils_1.encodeSapObjectName)(packageName.toLowerCase())}?_action=LOCK&accessMode=MODIFY`;
     const headers = {
         Accept: contentTypes_1.ACCEPT_LOCK,
     };
@@ -33,9 +26,11 @@ async function lockPackage(connection, packageName) {
         attributeNamePrefix: '',
     });
     const result = parser.parse(response.data);
-    const lockHandle = result?.['asx:abap']?.['asx:values']?.DATA?.LOCK_HANDLE;
+    const data = result?.['asx:abap']?.['asx:values']?.DATA;
+    const lockHandle = data?.LOCK_HANDLE;
     if (!lockHandle) {
         throw new Error('Failed to obtain lock handle from SAP. Package may be locked by another user.');
     }
-    return lockHandle;
+    const corrNr = data?.CORR_NUMBER || undefined;
+    return { lockHandle, corrNr };
 }

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

@@ -2,19 +2,7 @@
  * Package module type definitions
  */
 import type { IAdtObjectState } from '@mcp-abap-adt/interfaces';
-export interface ICreatePackageParams {
-    package_name: string;
-    description?: string;
-    super_package: string;
-    package_type?: string;
-    software_component?: string;
-    transport_layer?: string;
-    transport_request?: string;
-    application_component?: string;
-    responsible?: string;
-    masterSystem?: string;
-    record_changes?: boolean;
-}
+export type { ICreatePackageParams, IDeletePackageParams, IReadPackageParams, IUpdatePackageParams, } from '@mcp-abap-adt/interfaces';
 export interface IPackageConfig {
     packageName: string;
     superPackage?: string;
@@ -26,6 +14,7 @@ export interface IPackageConfig {
     transportRequest?: string;
     applicationComponent?: string;
     responsible?: string;
+    masterSystem?: string;
     recordChanges?: boolean;
     onLock?: (lockHandle: string) => void;
 }

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

@@ -1 +1 @@
-{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../src/core/package/types.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAC;AAGhE,MAAM,WAAW,oBAAoB;IACnC,YAAY,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IACtB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,qBAAqB,CAAC,EAAE,MAAM,CAAC;IAC/B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAKD,MAAM,WAAW,cAAc;IAC7B,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,MAAM,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;CACvC;AAED,MAAM,WAAW,aAAc,SAAQ,eAAe;CAAG"}
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../src/core/package/types.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAC;AAEhE,YAAY,EACV,oBAAoB,EACpB,oBAAoB,EACpB,kBAAkB,EAClB,oBAAoB,GACrB,MAAM,0BAA0B,CAAC;AAKlC,MAAM,WAAW,cAAc;IAC7B,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,MAAM,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;CACvC;AAED,MAAM,WAAW,aAAc,SAAQ,eAAe;CAAG"}

package/dist/core/package/unlock.js

@@ -14,7 +14,7 @@ const timeouts_1 = require("../../utils/timeouts");
  * after calling this function
  */
 async function unlockPackage(connection, packageName, lockHandle) {
-    const url = `/sap/bc/adt/packages/${(0, internalUtils_1.encodeSapObjectName)(packageName)}?_action=UNLOCK&lockHandle=${encodeURIComponent(lockHandle)}`;
+    const url = `/sap/bc/adt/packages/${(0, internalUtils_1.encodeSapObjectName)(packageName.toLowerCase())}?_action=UNLOCK&lockHandle=${encodeURIComponent(lockHandle)}`;
     return await connection.makeAdtRequest({
         url,
         method: 'POST',

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

@@ -5,17 +5,13 @@
  * This preserves all SAP-managed fields (abapLanguageVersion, etc.)
  * that would be lost if XML were built from scratch.
  */
-import type { IAdtResponse as AxiosResponse, IAbapConnection } from '@mcp-abap-adt/interfaces';
-import type { ICreatePackageParams } from './types';
-export interface UpdatePackageParams extends ICreatePackageParams {
-    package_name: string;
-}
+import type { IAdtResponse as AxiosResponse, IAbapConnection, IUpdatePackageParams } from '@mcp-abap-adt/interfaces';
 /**
  * Update package with new data (read-modify-write pattern)
  *
  * NOTE: Requires stateful session mode enabled via connection.setSessionType("stateful")
  */
-export declare function updatePackage(connection: IAbapConnection, params: UpdatePackageParams, lockHandle: string): Promise<AxiosResponse>;
+export declare function updatePackage(connection: IAbapConnection, params: IUpdatePackageParams, lockHandle: string): Promise<AxiosResponse>;
 /**
  * Update only package description (safe update - only modifiable field)
  *

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

@@ -1 +1 @@
-{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/core/package/update.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAC7B,eAAe,EAChB,MAAM,0BAA0B,CAAC;AAalC,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,SAAS,CAAC;AAwDpD,MAAM,WAAW,mBAAoB,SAAQ,oBAAoB;IAC/D,YAAY,EAAE,MAAM,CAAC;CACtB;AAED;;;;GAIG;AACH,wBAAsB,aAAa,CACjC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,mBAAmB,EAC3B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,aAAa,CAAC,CAqCxB;AAED;;;;GAIG;AACH,wBAAsB,wBAAwB,CAC5C,UAAU,EAAE,eAAe,EAC3B,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,MAAM,EACnB,UAAU,EAAE,MAAM,EAClB,YAAY,CAAC,EAAE,MAAM,GACpB,OAAO,CAAC,aAAa,CAAC,CAiBxB"}
+{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../../../src/core/package/update.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAC7B,eAAe,EACf,oBAAoB,EACrB,MAAM,0BAA0B,CAAC;AAyElC;;;;GAIG;AACH,wBAAsB,aAAa,CACjC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,oBAAoB,EAC5B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,aAAa,CAAC,CAuCxB;AAED;;;;GAIG;AACH,wBAAsB,wBAAwB,CAC5C,UAAU,EAAE,eAAe,EAC3B,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,MAAM,EACnB,UAAU,EAAE,MAAM,EAClB,YAAY,CAAC,EAAE,MAAM,GACpB,OAAO,CAAC,aAAa,CAAC,CAkBxB"}

package/dist/core/package/update.js

@@ -26,6 +26,8 @@ function patchPackageXml(currentXml, args) {
     }
     // Responsible
     xml = (0, xmlPatch_1.patchIf)(xml, args.responsible, (x, val) => (0, xmlPatch_1.patchXmlAttribute)(x, 'adtcore:responsible', val));
+    // Master system
+    xml = (0, xmlPatch_1.patchIf)(xml, args.master_system, (x, val) => (0, xmlPatch_1.patchXmlAttribute)(x, 'adtcore:masterSystem', val));
     // Package type (pak:packageType attribute on pak:attributes element)
     xml = (0, xmlPatch_1.patchIf)(xml, args.package_type, (x, val) => (0, xmlPatch_1.patchXmlElementAttribute)(x, 'pak:attributes', 'pak:packageType', val));
     // Record changes
@@ -49,7 +51,7 @@ async function updatePackage(connection, params, lockHandle) {
     if (!params.package_name) {
         throw new Error('package_name is required');
     }
-    const packageNameEncoded = (0, internalUtils_1.encodeSapObjectName)(params.package_name);
+    const packageNameEncoded = (0, internalUtils_1.encodeSapObjectName)(params.package_name.toLowerCase());
     // 1. GET current XML
     const currentResponse = await connection.makeAdtRequest({
         url: `/sap/bc/adt/packages/${packageNameEncoded}`,
@@ -93,5 +95,6 @@ async function updatePackageDescription(connection, packageName, description, lo
         package_name: packageName,
         description: (0, internalUtils_1.limitDescription)(description),
         super_package: superPackage || '',
+        record_changes: false,
     }, lockHandle);
 }

package/dist/core/unitTest/AdtUnitTestLegacy.d.ts

@@ -8,11 +8,10 @@
  * Key difference: Legacy systems return results synchronously (aunit:runResult)
  * from the POST to /testruns — no run ID, no async polling needed.
  */
-import type { IAdtResponse as AxiosResponse, IAbapConnection, IAdtOperationOptions, ILogger } from '@mcp-abap-adt/interfaces';
+import type { IAdtResponse as AxiosResponse, IAdtOperationOptions } from '@mcp-abap-adt/interfaces';
 import { AdtUnitTest } from './AdtUnitTest';
 import type { IClassUnitTestDefinition, IClassUnitTestRunOptions, IUnitTestConfig, IUnitTestState } from './types';
 export declare class AdtUnitTestLegacy extends AdtUnitTest {
-    constructor(connection: IAbapConnection, logger?: ILogger);
     /**
      * Create unit test run using legacy endpoint.
      * Legacy returns results synchronously — no run ID or polling needed.

package/dist/core/unitTest/AdtUnitTestLegacy.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"AdtUnitTestLegacy.d.ts","sourceRoot":"","sources":["../../../src/core/unitTest/AdtUnitTestLegacy.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAC7B,eAAe,EACf,oBAAoB,EACpB,OAAO,EACR,MAAM,0BAA0B,CAAC;AAElC,OAAO,EAAE,WAAW,EAAE,MAAM,eAAe,CAAC;AAE5C,OAAO,KAAK,EACV,wBAAwB,EACxB,wBAAwB,EACxB,eAAe,EACf,cAAc,EACf,MAAM,SAAS,CAAC;AAKjB,qBAAa,iBAAkB,SAAQ,WAAW;gBACpC,UAAU,EAAE,eAAe,EAAE,MAAM,CAAC,EAAE,OAAO;IAIzD;;;OAGG;IACY,MAAM,CACnB,MAAM,EAAE,eAAe,EACvB,QAAQ,CAAC,EAAE,oBAAoB,GAC9B,OAAO,CAAC,cAAc,CAAC;IAkC1B;;OAEG;IACY,GAAG,CAChB,KAAK,EAAE,wBAAwB,EAAE,EACjC,OAAO,CAAC,EAAE,wBAAwB,GACjC,OAAO,CAAC,MAAM,CAAC;IAKlB;;;OAGG;IACY,SAAS,CACtB,MAAM,EAAE,MAAM,EACd,gBAAgB,GAAE,OAAc,GAC/B,OAAO,CAAC,aAAa,CAAC;IASzB;;;OAGG;IACY,SAAS,CACtB,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE;QAAE,kBAAkB,CAAC,EAAE,OAAO,CAAC;QAAC,MAAM,CAAC,EAAE,UAAU,GAAG,OAAO,CAAA;KAAE,GACzE,OAAO,CAAC,aAAa,CAAC;IASzB;;;OAGG;IACY,IAAI,CACjB,OAAO,EAAE,OAAO,CAAC,eAAe,CAAC,EACjC,QAAQ,GAAE,QAAQ,GAAG,UAAqB,GACzC,OAAO,CAAC,cAAc,GAAG,SAAS,CAAC;CAWvC"}
+{"version":3,"file":"AdtUnitTestLegacy.d.ts","sourceRoot":"","sources":["../../../src/core/unitTest/AdtUnitTestLegacy.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAC7B,oBAAoB,EACrB,MAAM,0BAA0B,CAAC;AAElC,OAAO,EAAE,WAAW,EAAE,MAAM,eAAe,CAAC;AAE5C,OAAO,KAAK,EACV,wBAAwB,EACxB,wBAAwB,EACxB,eAAe,EACf,cAAc,EACf,MAAM,SAAS,CAAC;AAKjB,qBAAa,iBAAkB,SAAQ,WAAW;IAChD;;;OAGG;IACY,MAAM,CACnB,MAAM,EAAE,eAAe,EACvB,QAAQ,CAAC,EAAE,oBAAoB,GAC9B,OAAO,CAAC,cAAc,CAAC;IAkC1B;;OAEG;IACY,GAAG,CAChB,KAAK,EAAE,wBAAwB,EAAE,EACjC,OAAO,CAAC,EAAE,wBAAwB,GACjC,OAAO,CAAC,MAAM,CAAC;IAKlB;;;OAGG;IACY,SAAS,CACtB,MAAM,EAAE,MAAM,EACd,gBAAgB,GAAE,OAAc,GAC/B,OAAO,CAAC,aAAa,CAAC;IASzB;;;OAGG;IACY,SAAS,CACtB,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE;QAAE,kBAAkB,CAAC,EAAE,OAAO,CAAC;QAAC,MAAM,CAAC,EAAE,UAAU,GAAG,OAAO,CAAA;KAAE,GACzE,OAAO,CAAC,aAAa,CAAC;IASzB;;;OAGG;IACY,IAAI,CACjB,OAAO,EAAE,OAAO,CAAC,eAAe,CAAC,EACjC,QAAQ,GAAE,QAAQ,GAAG,UAAqB,GACzC,OAAO,CAAC,cAAc,GAAG,SAAS,CAAC;CAWvC"}

package/dist/core/unitTest/AdtUnitTestLegacy.js

@@ -17,9 +17,6 @@ const runLegacy_1 = require("./runLegacy");
 /** Synthetic run ID for legacy synchronous results */
 const LEGACY_SYNC_RUN_ID = 'legacy-sync';
 class AdtUnitTestLegacy extends AdtUnitTest_1.AdtUnitTest {
-    constructor(connection, logger) {
-        super(connection, logger);
-    }
     /**
      * Create unit test run using legacy endpoint.
      * Legacy returns results synchronously — no run ID or polling needed.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mcp-abap-adt/adt-clients",
-  "version": "3.10.2",
+  "version": "3.11.3",
   "description": "ADT clients for SAP ABAP systems - AdtClient and AdtRuntimeClient",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -66,7 +66,7 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "@mcp-abap-adt/interfaces": "^3.1.0",
+    "@mcp-abap-adt/interfaces": "^5.0.0",
     "@mcp-abap-adt/logger": "^0.1.3",
     "axios": "^1.13.6",
     "fast-xml-parser": "^5.4.1",
@@ -77,7 +77,7 @@
   },
   "devDependencies": {
     "@biomejs/biome": "^2.4.4",
-    "@mcp-abap-adt/connection": "^1.5.1",
+    "@mcp-abap-adt/connection": "^1.5.2",
     "@types/jest": "^30.0.0",
     "@types/node": "^25.3.3",
     "dotenv": "^17.3.1",