@mcp-abap-adt/adt-clients 7.5.0 → 7.6.0

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

@@ -30,6 +30,9 @@ export declare class AdtClass implements IAdtObject<IClassConfig, IClassState> {
     protected readonly contentTypes?: IAdtContentTypes;
     private readonly lockTracker;
     readonly objectType: string;
+    private readonly capCtx;
+    private readonly lockCap;
+    private readonly versionsCap;
     constructor(connection: IAbapConnection, logger?: ILogger, systemContext?: IAdtSystemContext, contentTypes?: IAdtContentTypes, lockRegistry?: LockRegistry);
     /**
      * Validate class configuration before creation

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

@@ -1 +1 @@
-{"version":3,"file":"AdtClass.d.ts","sourceRoot":"","sources":["../../../src/core/class/AdtClass.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,EAGL,KAAK,YAAY,IAAI,aAAa,EAElC,KAAK,eAAe,EACpB,KAAK,UAAU,EACf,KAAK,oBAAoB,EACzB,KAAK,OAAO,EACZ,KAAK,cAAc,EACpB,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AAEjE,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAC/D,OAAO,EAEL,KAAK,YAAY,EAElB,MAAM,wBAAwB,CAAC;AAChC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAWpD,OAAO,KAAK,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAIzD,OAAO,EACL,KAAK,gBAAgB,EAGtB,MAAM,YAAY,CAAC;AAEpB,qBAAa,QAAS,YAAW,UAAU,CAAC,YAAY,EAAE,WAAW,CAAC;IACpE,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,SAAS,CAAC,QAAQ,CAAC,YAAY,CAAC,EAAE,gBAAgB,CAAC;IACnD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAc;IAC1C,SAAgB,UAAU,EAAE,MAAM,CAAW;gBAG3C,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,iBAAiB,EACjC,YAAY,CAAC,EAAE,gBAAgB,EAC/B,YAAY,CAAC,EAAE,YAAY;IAc7B;;OAEG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IAgDnE;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,YAAY,EACpB,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,WAAW,CAAC;IAoEvB;;OAEG;IACG,IAAI,CACR,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE,QAAQ,GAAG,UAAU,EAC/B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,WAAW,GAAG,SAAS,CAAC;IAqDnC;;OAEG;IACG,YAAY,CAChB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,WAAW,CAAC;IAmCvB;;;;OAIG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,WAAW,CAAC;IA+LvB;;OAEG;IACG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IA8DjE;;;OAGG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IA8CnE;;OAEG;IACG,KAAK,CACT,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,WAAW,CAAC;IA6EvB;;OAEG;IACG,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAe1D;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,WAAW,CAAC;IAkBvB;;;OAGG;IACG,eAAe,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAUrE;;;OAGG;IACG,iBAAiB,CACrB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,aAAa,CAAC;IAczB;;OAEG;IACG,cAAc,CAClB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG;QAAE,aAAa,EAAE,MAAM,CAAA;KAAE,EACzD,OAAO,GAAE,QAAQ,GAAG,UAAuB,GAC1C,OAAO,CAAC,aAAa,CAAC;IAgBzB;;;OAGG;IACG,iBAAiB,CACrB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG;QAAE,aAAa,EAAE,MAAM,CAAA;KAAE,GACxD,OAAO,CAAC,aAAa,CAAC;IA2DzB;;OAEG;IACG,mBAAmB,CACvB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG;QAAE,aAAa,EAAE,MAAM,CAAA;KAAE,GACxD,OAAO,CAAC,aAAa,CAAC;IAczB;;OAEG;IACG,aAAa,CACjB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,OAAO,CAAA;KAAE,GACtC,OAAO,CAAC,WAAW,CAAC;IAoCvB,WAAW,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC;IAKrE,gBAAgB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAIrD,sEAAsE;IACtE,SAAS,CAAC,kBAAkB,CAC1B,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE,gBAAgB,GAC5B,OAAO,CAAC,cAAc,EAAE,CAAC;CAG7B"}
+{"version":3,"file":"AdtClass.d.ts","sourceRoot":"","sources":["../../../src/core/class/AdtClass.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,EAGL,KAAK,YAAY,IAAI,aAAa,EAElC,KAAK,eAAe,EACpB,KAAK,UAAU,EACf,KAAK,oBAAoB,EACzB,KAAK,OAAO,EACZ,KAAK,cAAc,EACpB,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AAOjE,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAC/D,OAAO,EAEL,KAAK,YAAY,EAElB,MAAM,wBAAwB,CAAC;AAChC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAWpD,OAAO,KAAK,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAIzD,OAAO,EACL,KAAK,gBAAgB,EAGtB,MAAM,YAAY,CAAC;AAEpB,qBAAa,QAAS,YAAW,UAAU,CAAC,YAAY,EAAE,WAAW,CAAC;IACpE,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,SAAS,CAAC,QAAQ,CAAC,YAAY,CAAC,EAAE,gBAAgB,CAAC;IACnD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAc;IAC1C,SAAgB,UAAU,EAAE,MAAM,CAAW;IAK7C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAGpB;IAEH,OAAO,CAAC,QAAQ,CAAC,OAAO,CAetB;IAEF,OAAO,CAAC,QAAQ,CAAC,WAAW,CAW1B;gBAGA,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,iBAAiB,EACjC,YAAY,CAAC,EAAE,gBAAgB,EAC/B,YAAY,CAAC,EAAE,YAAY;IAc7B;;OAEG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IAgDnE;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,YAAY,EACpB,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,WAAW,CAAC;IAoEvB;;OAEG;IACG,IAAI,CACR,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE,QAAQ,GAAG,UAAU,EAC/B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,WAAW,GAAG,SAAS,CAAC;IAqDnC;;OAEG;IACG,YAAY,CAChB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,WAAW,CAAC;IAmCvB;;;;OAIG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,WAAW,CAAC;IA+LvB;;OAEG;IACG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IA8DjE;;;OAGG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IA8CnE;;OAEG;IACG,KAAK,CACT,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,WAAW,CAAC;IA6EvB;;OAEG;IACG,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAM1D;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,WAAW,CAAC;IAMvB;;;OAGG;IACG,eAAe,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAUrE;;;OAGG;IACG,iBAAiB,CACrB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,aAAa,CAAC;IAczB;;OAEG;IACG,cAAc,CAClB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG;QAAE,aAAa,EAAE,MAAM,CAAA;KAAE,EACzD,OAAO,GAAE,QAAQ,GAAG,UAAuB,GAC1C,OAAO,CAAC,aAAa,CAAC;IAgBzB;;;OAGG;IACG,iBAAiB,CACrB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG;QAAE,aAAa,EAAE,MAAM,CAAA;KAAE,GACxD,OAAO,CAAC,aAAa,CAAC;IA2DzB;;OAEG;IACG,mBAAmB,CACvB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG;QAAE,aAAa,EAAE,MAAM,CAAA;KAAE,GACxD,OAAO,CAAC,aAAa,CAAC;IAczB;;OAEG;IACG,aAAa,CACjB,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,EAC7B,OAAO,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,OAAO,CAAA;KAAE,GACtC,OAAO,CAAC,WAAW,CAAC;IAoCvB,WAAW,CAAC,MAAM,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC;IAIrE,gBAAgB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAIrD,sEAAsE;IACtE,SAAS,CAAC,kBAAkB,CAC1B,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE,gBAAgB,GAC5B,OAAO,CAAC,cAAc,EAAE,CAAC;CAG7B"}

package/dist/core/class/AdtClass.js

@@ -54,6 +54,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.AdtClass = void 0;
 const interfaces_1 = require("@mcp-abap-adt/interfaces");
 const internalUtils_1 = require("../../utils/internalUtils");
+const capabilities_1 = require("../shared/capabilities");
 const LockRegistry_1 = require("../shared/LockRegistry");
 const activation_1 = require("./activation");
 const check_1 = require("./check");
@@ -73,6 +74,36 @@ class AdtClass {
     contentTypes;
     lockTracker;
     objectType = 'Class';
+    // LAZY thunk (not a getter that snapshots): captures `this` but reads
+    // this.connection/this.logger only when invoked, after the constructor has
+    // run — so building the capabilities below as class fields is safe.
+    capCtx = () => ({
+        connection: this.connection,
+        logger: this.logger,
+    });
+    lockCap = new capabilities_1.LockCapability(this.capCtx, {
+        nameOf: (c) => {
+            if (!c.className)
+                throw new Error('Class name is required');
+            return c.className;
+        },
+        acquire: async (ctx, name) => ({
+            lockHandle: await (0, lock_1.lockClass)(ctx.connection, name),
+        }),
+        release: async (ctx, name, handle) => {
+            const result = await (0, unlock_1.unlockClass)(ctx.connection, name, handle);
+            return { unlockResult: result, errors: [] };
+        },
+    });
+    versionsCap = new capabilities_1.VersionsCapability(this.capCtx, {
+        nameOf: (c) => {
+            if (!c.className)
+                throw new Error('className is required');
+            return c.className;
+        },
+        list: (ctx, name) => (0, versions_1.getClassIncludeVersions)(ctx.connection, name, 'main'),
+        source: (ctx, uri) => (0, versions_1.getClassVersionSource)(ctx.connection, uri),
+    });
     constructor(connection, logger, systemContext, contentTypes, lockRegistry) {
         this.connection = connection;
         this.logger = logger;
@@ -537,33 +568,17 @@ class AdtClass {
      * Lock class
      */
     async lock(config) {
-        if (!config.className) {
-            throw new Error('Class name is required');
-        }
-        // Stay stateful while the lock is held — the caller must release it via
-        // unlock(), which restores stateless. On older BASIS (#106) the lock handle
-        // is only valid inside stateful requests, so a stateless write between lock
-        // and unlock fails with 423.
-        this.connection.setSessionType('stateful');
-        const lockHandle = await (0, lock_1.lockClass)(this.connection, config.className);
-        this.lockTracker.track(config.className, lockHandle);
-        return lockHandle;
+        const handle = await this.lockCap.lock(config);
+        this.lockTracker.track(config.className, handle);
+        return handle;
     }
     /**
      * Unlock class
      */
     async unlock(config, lockHandle) {
-        if (!config.className) {
-            throw new Error('Class name is required');
-        }
-        this.connection.setSessionType('stateful');
-        const result = await (0, unlock_1.unlockClass)(this.connection, config.className, lockHandle);
-        this.connection.setSessionType('stateless');
+        const state = await this.lockCap.unlock(config, lockHandle);
         this.lockTracker.untrack(config.className);
-        return {
-            unlockResult: result,
-            errors: [],
-        };
+        return state;
     }
     /**
      * Lock test classes (local classes) for modification
@@ -698,12 +713,10 @@ class AdtClass {
         }
     }
     getVersions(config) {
-        if (!config.className)
-            throw new Error('className is required');
-        return (0, versions_1.getClassIncludeVersions)(this.connection, config.className, 'main');
+        return this.versionsCap.getVersions(config);
     }
     getVersionSource(contentUri) {
-        return (0, versions_1.getClassVersionSource)(this.connection, contentUri);
+        return this.versionsCap.getVersionSource(contentUri);
     }
     /** Shared by local-include subclasses to target their own include. */
     getIncludeVersions(className, includeType) {

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

@@ -28,6 +28,8 @@ export declare class AdtDomain implements IAdtObject<IDomainConfig, IDomainState
     private readonly systemContext;
     private readonly lockTracker;
     readonly objectType: string;
+    private readonly capCtx;
+    private readonly lockCap;
     constructor(connection: IAbapConnection, logger?: ILogger, systemContext?: IAdtSystemContext, lockRegistry?: LockRegistry);
     /**
      * Validate domain configuration before creation

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

@@ -1 +1 @@
-{"version":3,"file":"AdtDomain.d.ts","sourceRoot":"","sources":["../../../src/core/domain/AdtDomain.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAEH,OAAO,KAAK,EAEV,eAAe,EACf,UAAU,EACV,oBAAoB,EACpB,OAAO,EACP,cAAc,EACf,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AAEjE,OAAO,EAEL,KAAK,YAAY,EAElB,MAAM,wBAAwB,CAAC;AAChC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAQpD,OAAO,KAAK,EAAE,aAAa,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAI3D,qBAAa,SAAU,YAAW,UAAU,CAAC,aAAa,EAAE,YAAY,CAAC;IACvE,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAkB;IAC7C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAU;IAClC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAoB;IAClD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAc;IAC1C,SAAgB,UAAU,EAAE,MAAM,CAAY;gBAG5C,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,iBAAiB,EACjC,YAAY,CAAC,EAAE,YAAY;IAa7B;;OAEG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,GAAG,OAAO,CAAC,YAAY,CAAC;IAkBrE;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,aAAa,EACrB,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,YAAY,CAAC;IAmExB;;OAEG;IACG,IAAI,CACR,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,QAAQ,CAAC,EAAE,QAAQ,GAAG,UAAU,EAChC,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,YAAY,GAAG,SAAS,CAAC;IAyBpC;;;OAGG;IACG,YAAY,CAChB,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,YAAY,CAAC;IA4CxB;;;OAGG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,YAAY,CAAC;IA8NxB;;OAEG;IACG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,GAAG,OAAO,CAAC,YAAY,CAAC;IAmCnE;;;OAGG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,GAAG,OAAO,CAAC,YAAY,CAAC;IAsBrE;;OAEG;IACG,KAAK,CACT,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,YAAY,CAAC;IAuBxB;;OAEG;IACG,aAAa,CACjB,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,OAAO,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,OAAO,CAAA;KAAE,GACtC,OAAO,CAAC,YAAY,CAAC;IAoCxB;;OAEG;IACG,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAW3D;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,YAAY,CAAC;IAmBlB,WAAW,CACf,OAAO,EAAE,OAAO,CAAC,aAAa,CAAC,GAC9B,OAAO,CAAC,cAAc,EAAE,CAAC;IAItB,gBAAgB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;CAG7D"}
+{"version":3,"file":"AdtDomain.d.ts","sourceRoot":"","sources":["../../../src/core/domain/AdtDomain.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAEH,OAAO,KAAK,EAEV,eAAe,EACf,UAAU,EACV,oBAAoB,EACpB,OAAO,EACP,cAAc,EACf,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AAMjE,OAAO,EAEL,KAAK,YAAY,EAElB,MAAM,wBAAwB,CAAC;AAChC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAQpD,OAAO,KAAK,EAAE,aAAa,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAI3D,qBAAa,SAAU,YAAW,UAAU,CAAC,aAAa,EAAE,YAAY,CAAC;IACvE,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAkB;IAC7C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAU;IAClC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAoB;IAClD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAc;IAC1C,SAAgB,UAAU,EAAE,MAAM,CAAY;IAK9C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAGpB;IAEH,OAAO,CAAC,QAAQ,CAAC,OAAO,CAetB;gBAGA,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,iBAAiB,EACjC,YAAY,CAAC,EAAE,YAAY;IAa7B;;OAEG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,GAAG,OAAO,CAAC,YAAY,CAAC;IAkBrE;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,aAAa,EACrB,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,YAAY,CAAC;IAmExB;;OAEG;IACG,IAAI,CACR,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,QAAQ,CAAC,EAAE,QAAQ,GAAG,UAAU,EAChC,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,YAAY,GAAG,SAAS,CAAC;IAyBpC;;;OAGG;IACG,YAAY,CAChB,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,YAAY,CAAC;IA4CxB;;;OAGG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,YAAY,CAAC;IA8NxB;;OAEG;IACG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,GAAG,OAAO,CAAC,YAAY,CAAC;IAmCnE;;;OAGG;IACG,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,GAAG,OAAO,CAAC,YAAY,CAAC;IAsBrE;;OAEG;IACG,KAAK,CACT,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,YAAY,CAAC;IAuBxB;;OAEG;IACG,aAAa,CACjB,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,OAAO,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,OAAO,CAAA;KAAE,GACtC,OAAO,CAAC,YAAY,CAAC;IAoCxB;;OAEG;IACG,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAM3D;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,aAAa,CAAC,EAC9B,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,YAAY,CAAC;IAMlB,WAAW,CACf,OAAO,EAAE,OAAO,CAAC,aAAa,CAAC,GAC9B,OAAO,CAAC,cAAc,EAAE,CAAC;IAItB,gBAAgB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;CAG7D"}

package/dist/core/domain/AdtDomain.js

@@ -21,6 +21,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.AdtDomain = void 0;
 const internalUtils_1 = require("../../utils/internalUtils");
+const capabilities_1 = require("../shared/capabilities");
 const LockRegistry_1 = require("../shared/LockRegistry");
 const versions_1 = require("../shared/versions");
 const activation_1 = require("./activation");
@@ -38,6 +39,27 @@ class AdtDomain {
     systemContext;
     lockTracker;
     objectType = 'Domain';
+    // LAZY thunk (not a getter that snapshots): captures `this` but reads
+    // this.connection/this.logger only when invoked, after the constructor has
+    // run — so building the capability below as a class field is safe.
+    capCtx = () => ({
+        connection: this.connection,
+        logger: this.logger,
+    });
+    lockCap = new capabilities_1.LockCapability(this.capCtx, {
+        nameOf: (c) => {
+            if (!c.domainName)
+                throw new Error('Domain name is required');
+            return c.domainName;
+        },
+        acquire: async (ctx, name) => ({
+            lockHandle: await (0, lock_1.lockDomain)(ctx.connection, name),
+        }),
+        release: async (ctx, name, handle) => {
+            const result = await (0, unlock_1.unlockDomain)(ctx.connection, name, handle);
+            return { unlockResult: result, errors: [] };
+        },
+    });
     constructor(connection, logger, systemContext, lockRegistry) {
         this.connection = connection;
         this.logger = logger;
@@ -465,29 +487,17 @@ class AdtDomain {
      * Lock domain for modification
      */
     async lock(config) {
-        if (!config.domainName) {
-            throw new Error('Domain name is required');
-        }
-        this.connection.setSessionType('stateful');
-        const lockHandle = await (0, lock_1.lockDomain)(this.connection, config.domainName);
-        this.lockTracker.track(config.domainName, lockHandle);
-        return lockHandle;
+        const handle = await this.lockCap.lock(config);
+        this.lockTracker.track(config.domainName, handle);
+        return handle;
     }
     /**
      * Unlock domain
      */
     async unlock(config, lockHandle) {
-        if (!config.domainName) {
-            throw new Error('Domain name is required');
-        }
-        this.connection.setSessionType('stateful');
-        const result = await (0, unlock_1.unlockDomain)(this.connection, config.domainName, lockHandle);
-        this.connection.setSessionType('stateless');
+        const state = await this.lockCap.unlock(config, lockHandle);
         this.lockTracker.untrack(config.domainName);
-        return {
-            unlockResult: result,
-            errors: [],
-        };
+        return state;
     }
     async getVersions(_config) {
         (0, versions_1.throwUnsupportedVersions)('domain');

package/dist/core/serviceDefinition/AdtServiceDefinition.d.ts

@@ -29,6 +29,9 @@ export declare class AdtServiceDefinition implements IAdtObject<IServiceDefiniti
     private readonly systemContext;
     private readonly lockTracker;
     readonly objectType: string;
+    private readonly capCtx;
+    private readonly lockCap;
+    private readonly versionsCap;
     constructor(connection: IAbapConnection, logger?: ILogger, systemContext?: IAdtSystemContext, lockRegistry?: LockRegistry);
     /**
      * Validate service definition configuration before creation

package/dist/core/serviceDefinition/AdtServiceDefinition.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"AdtServiceDefinition.d.ts","sourceRoot":"","sources":["../../../src/core/serviceDefinition/AdtServiceDefinition.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;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,EAEL,KAAK,YAAY,EAElB,MAAM,wBAAwB,CAAC;AAChC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAWpD,OAAO,KAAK,EACV,wBAAwB,EACxB,uBAAuB,EACxB,MAAM,SAAS,CAAC;AASjB,qBAAa,oBACX,YAAW,UAAU,CAAC,wBAAwB,EAAE,uBAAuB,CAAC;IAExE,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAkB;IAC7C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAU;IAClC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAoB;IAClD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAc;IAC1C,SAAgB,UAAU,EAAE,MAAM,CAAuB;gBAGvD,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,iBAAiB,EACjC,YAAY,CAAC,EAAE,YAAY;IAa7B;;OAEG;IACG,QAAQ,CACZ,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,GACxC,OAAO,CAAC,uBAAuB,CAAC;IA8BnC;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,wBAAwB,EAChC,QAAQ,CAAC,EAAE,oBAAoB,GAC9B,OAAO,CAAC,uBAAuB,CAAC;IAqCnC;;OAEG;IACG,IAAI,CACR,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,OAAO,CAAC,EAAE,QAAQ,GAAG,UAAU,EAC/B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,uBAAuB,GAAG,SAAS,CAAC;IA2B/C;;OAEG;IACG,YAAY,CAChB,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,uBAAuB,CAAC;IAkCnC;;OAEG;IACG,aAAa,CACjB,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,OAAO,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,OAAO,CAAA;KAAE,GACtC,OAAO,CAAC,uBAAuB,CAAC;IAoCnC;;;;OAIG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,uBAAuB,CAAC;IAuNnC;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,GACxC,OAAO,CAAC,uBAAuB,CAAC;IAmCnC;;;OAGG;IACG,QAAQ,CACZ,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,GACxC,OAAO,CAAC,uBAAuB,CAAC;IAqBnC;;OAEG;IACG,KAAK,CACT,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,uBAAuB,CAAC;IAkBnC;;OAEG;IACG,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IActE;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,uBAAuB,CAAC;IAmBnC,WAAW,CAAC,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC;IAIrD,gBAAgB,CAAC,UAAU,EAAE,MAAM;CAGpC"}
+{"version":3,"file":"AdtServiceDefinition.d.ts","sourceRoot":"","sources":["../../../src/core/serviceDefinition/AdtServiceDefinition.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;AAEH,OAAO,KAAK,EAEV,eAAe,EACf,UAAU,EACV,oBAAoB,EACpB,OAAO,EACR,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AAOjE,OAAO,EAEL,KAAK,YAAY,EAElB,MAAM,wBAAwB,CAAC;AAChC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAWpD,OAAO,KAAK,EACV,wBAAwB,EACxB,uBAAuB,EACxB,MAAM,SAAS,CAAC;AASjB,qBAAa,oBACX,YAAW,UAAU,CAAC,wBAAwB,EAAE,uBAAuB,CAAC;IAExE,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAkB;IAC7C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAU;IAClC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAoB;IAClD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAc;IAC1C,SAAgB,UAAU,EAAE,MAAM,CAAuB;IAKzD,OAAO,CAAC,QAAQ,CAAC,MAAM,CAGpB;IAEH,OAAO,CAAC,QAAQ,CAAC,OAAO,CAoBrB;IAEH,OAAO,CAAC,QAAQ,CAAC,WAAW,CAgBvB;gBAGH,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,iBAAiB,EACjC,YAAY,CAAC,EAAE,YAAY;IAa7B;;OAEG;IACG,QAAQ,CACZ,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,GACxC,OAAO,CAAC,uBAAuB,CAAC;IA8BnC;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,wBAAwB,EAChC,QAAQ,CAAC,EAAE,oBAAoB,GAC9B,OAAO,CAAC,uBAAuB,CAAC;IAqCnC;;OAEG;IACG,IAAI,CACR,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,OAAO,CAAC,EAAE,QAAQ,GAAG,UAAU,EAC/B,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,uBAAuB,GAAG,SAAS,CAAC;IA2B/C;;OAEG;IACG,YAAY,CAChB,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,OAAO,CAAC,EAAE,YAAY,GACrB,OAAO,CAAC,uBAAuB,CAAC;IAkCnC;;OAEG;IACG,aAAa,CACjB,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,OAAO,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,OAAO,CAAA;KAAE,GACtC,OAAO,CAAC,uBAAuB,CAAC;IAoCnC;;;;OAIG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,OAAO,CAAC,EAAE,oBAAoB,GAC7B,OAAO,CAAC,uBAAuB,CAAC;IAuNnC;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,GACxC,OAAO,CAAC,uBAAuB,CAAC;IAmCnC;;;OAGG;IACG,QAAQ,CACZ,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,GACxC,OAAO,CAAC,uBAAuB,CAAC;IAqBnC;;OAEG;IACG,KAAK,CACT,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,uBAAuB,CAAC;IAkBnC;;OAEG;IACG,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAMtE;;OAEG;IACG,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC,EACzC,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,uBAAuB,CAAC;IAMnC,WAAW,CAAC,MAAM,EAAE,OAAO,CAAC,wBAAwB,CAAC;IAIrD,gBAAgB,CAAC,UAAU,EAAE,MAAM;CAGpC"}

package/dist/core/serviceDefinition/AdtServiceDefinition.js

@@ -22,6 +22,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.AdtServiceDefinition = void 0;
 const internalUtils_1 = require("../../utils/internalUtils");
+const capabilities_1 = require("../shared/capabilities");
 const LockRegistry_1 = require("../shared/LockRegistry");
 const activation_1 = require("./activation");
 const check_1 = require("./check");
@@ -39,6 +40,41 @@ class AdtServiceDefinition {
     systemContext;
     lockTracker;
     objectType = 'ServiceDefinition';
+    // LAZY thunk (not a getter that snapshots): captures `this` but reads
+    // this.connection/this.logger only when invoked, after the constructor has
+    // run — so building the capabilities below as class fields is safe.
+    capCtx = () => ({
+        connection: this.connection,
+        logger: this.logger,
+    });
+    lockCap = new capabilities_1.LockCapability(this.capCtx, {
+        nameOf: (c) => {
+            if (!c.serviceDefinitionName)
+                throw new Error('Service definition name is required');
+            return c.serviceDefinitionName;
+        },
+        acquire: async (ctx, name) => ({
+            lockHandle: await (0, lock_1.lockServiceDefinition)(ctx.connection, name),
+        }),
+        release: async (ctx, name, handle) => {
+            const result = await (0, unlock_1.unlockServiceDefinition)(ctx.connection, name, handle);
+            return { unlockResult: result, errors: [] };
+        },
+    });
+    versionsCap = new capabilities_1.VersionsCapability(this.capCtx, {
+        nameOf: (c) => {
+            if (!c.serviceDefinitionName)
+                throw new Error('serviceDefinitionName is required');
+            return c.serviceDefinitionName;
+        },
+        // NOTE: getServiceDefinitionVersions takes a config, not a bare name —
+        // the strategy re-wraps. (getClassIncludeVersions, by contrast, takes a
+        // name; the two low-level shapes differ and the strategy absorbs that.)
+        list: (ctx, name) => (0, versions_1.getServiceDefinitionVersions)(ctx.connection, {
+            serviceDefinitionName: name,
+        }),
+        source: (ctx, uri) => (0, versions_1.getServiceDefinitionVersionSource)(ctx.connection, uri),
+    });
     constructor(connection, logger, systemContext, lockRegistry) {
         this.connection = connection;
         this.logger = logger;
@@ -415,11 +451,7 @@ class AdtServiceDefinition {
      * Lock service definition for modification
      */
     async lock(config) {
-        if (!config.serviceDefinitionName) {
-            throw new Error('Service definition name is required');
-        }
-        this.connection.setSessionType('stateful');
-        const lockHandle = await (0, lock_1.lockServiceDefinition)(this.connection, config.serviceDefinitionName);
+        const lockHandle = await this.lockCap.lock(config);
         this.lockTracker.track(config.serviceDefinitionName, lockHandle);
         return lockHandle;
     }
@@ -427,23 +459,15 @@ class AdtServiceDefinition {
      * Unlock service definition
      */
     async unlock(config, lockHandle) {
-        if (!config.serviceDefinitionName) {
-            throw new Error('Service definition name is required');
-        }
-        this.connection.setSessionType('stateful');
-        const result = await (0, unlock_1.unlockServiceDefinition)(this.connection, config.serviceDefinitionName, lockHandle);
-        this.connection.setSessionType('stateless');
+        const state = await this.lockCap.unlock(config, lockHandle);
         this.lockTracker.untrack(config.serviceDefinitionName);
-        return {
-            unlockResult: result,
-            errors: [],
-        };
+        return state;
     }
     getVersions(config) {
-        return (0, versions_1.getServiceDefinitionVersions)(this.connection, config);
+        return this.versionsCap.getVersions(config);
     }
     getVersionSource(contentUri) {
-        return (0, versions_1.getServiceDefinitionVersionSource)(this.connection, contentUri);
+        return this.versionsCap.getVersionSource(contentUri);
     }
 }
 exports.AdtServiceDefinition = AdtServiceDefinition;

package/dist/core/shared/capabilities/LockCapability.d.ts

@@ -0,0 +1,37 @@
+import type { IAdtLockable } from '@mcp-abap-adt/interfaces';
+import type { ICapabilityContext, ILockStrategy } from './types';
+/**
+ * Shared lock/unlock for handlers whose lock differs only by endpoint and
+ * name field. `lock` leaves the session stateful (the caller must unlock).
+ * `unlock` runs the release inside a stateful request — older BASIS (#106)
+ * only accepts UNLOCK while stateful — then restores stateless. The handler's
+ * state shape (including the ADT `unlockResult`) is built by `strategy.release`
+ * and returned unchanged. See the spec's IAdtLockable obligations — idempotent
+ * unlock is a TARGET, not implemented here, and is left to the per-endpoint
+ * probe + adaptation rule of the full-migration plan.
+ *
+ * DELIBERATELY byte-identical to the current handlers, including the failure
+ * path: if `acquire`/`release` throws, the session is left as-is (stateful) and
+ * the error propagates — exactly as today. Failure/abandonment handling is
+ * DISTRIBUTED and this capability owns none of it:
+ *   - the consumer largely owns lock/unlock atomicity (it decides when to lock
+ *     and unlock);
+ *   - adt-clients' `LockRegistry.unlockAll()` is a disposal safety net that
+ *     raw-releases abandoned locks — deliberately WITHOUT toggling the session,
+ *     because `unlockAll()` manages the session once for the whole batch;
+ *   - the operation chain's create/update catch blocks also call
+ *     setSessionType('stateless').
+ * Adding a try/finally here would change behaviour and relocate responsibility
+ * across those layers, so the atom-level "restore stateless on failure"
+ * contract is deferred to the behavioural-conformance work (same bucket as
+ * activate/check error unification), where all three layers are reconciled
+ * rather than double-cleaned.
+ */
+export declare class LockCapability<TConfig, TReadResult> implements IAdtLockable<TConfig, TReadResult> {
+    private readonly getCtx;
+    private readonly strategy;
+    constructor(getCtx: () => ICapabilityContext, strategy: ILockStrategy<TConfig, TReadResult>);
+    lock(config: Partial<TConfig>): Promise<string>;
+    unlock(config: Partial<TConfig>, lockHandle: string): Promise<TReadResult>;
+}
+//# sourceMappingURL=LockCapability.d.ts.map

package/dist/core/shared/capabilities/LockCapability.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"LockCapability.d.ts","sourceRoot":"","sources":["../../../../src/core/shared/capabilities/LockCapability.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAC7D,OAAO,KAAK,EAAE,kBAAkB,EAAE,aAAa,EAAE,MAAM,SAAS,CAAC;AAEjE;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AACH,qBAAa,cAAc,CAAC,OAAO,EAAE,WAAW,CAC9C,YAAW,YAAY,CAAC,OAAO,EAAE,WAAW,CAAC;IAK3C,OAAO,CAAC,QAAQ,CAAC,MAAM;IACvB,OAAO,CAAC,QAAQ,CAAC,QAAQ;gBADR,MAAM,EAAE,MAAM,kBAAkB,EAChC,QAAQ,EAAE,aAAa,CAAC,OAAO,EAAE,WAAW,CAAC;IAG1D,IAAI,CAAC,MAAM,EAAE,OAAO,CAAC,OAAO,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAS/C,MAAM,CACV,MAAM,EAAE,OAAO,CAAC,OAAO,CAAC,EACxB,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,WAAW,CAAC;CASxB"}

package/dist/core/shared/capabilities/LockCapability.js

@@ -0,0 +1,59 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.LockCapability = void 0;
+/**
+ * Shared lock/unlock for handlers whose lock differs only by endpoint and
+ * name field. `lock` leaves the session stateful (the caller must unlock).
+ * `unlock` runs the release inside a stateful request — older BASIS (#106)
+ * only accepts UNLOCK while stateful — then restores stateless. The handler's
+ * state shape (including the ADT `unlockResult`) is built by `strategy.release`
+ * and returned unchanged. See the spec's IAdtLockable obligations — idempotent
+ * unlock is a TARGET, not implemented here, and is left to the per-endpoint
+ * probe + adaptation rule of the full-migration plan.
+ *
+ * DELIBERATELY byte-identical to the current handlers, including the failure
+ * path: if `acquire`/`release` throws, the session is left as-is (stateful) and
+ * the error propagates — exactly as today. Failure/abandonment handling is
+ * DISTRIBUTED and this capability owns none of it:
+ *   - the consumer largely owns lock/unlock atomicity (it decides when to lock
+ *     and unlock);
+ *   - adt-clients' `LockRegistry.unlockAll()` is a disposal safety net that
+ *     raw-releases abandoned locks — deliberately WITHOUT toggling the session,
+ *     because `unlockAll()` manages the session once for the whole batch;
+ *   - the operation chain's create/update catch blocks also call
+ *     setSessionType('stateless').
+ * Adding a try/finally here would change behaviour and relocate responsibility
+ * across those layers, so the atom-level "restore stateless on failure"
+ * contract is deferred to the behavioural-conformance work (same bucket as
+ * activate/check error unification), where all three layers are reconciled
+ * rather than double-cleaned.
+ */
+class LockCapability {
+    getCtx;
+    strategy;
+    constructor(
+    // LAZY: read at method-call time, so the handler can build this capability
+    // as a class field before its constructor assigns this.connection.
+    getCtx, strategy) {
+        this.getCtx = getCtx;
+        this.strategy = strategy;
+    }
+    async lock(config) {
+        const ctx = this.getCtx();
+        const name = this.strategy.nameOf(config);
+        // Stay stateful while the lock is held; the caller releases via unlock().
+        ctx.connection.setSessionType('stateful');
+        const { lockHandle } = await this.strategy.acquire(ctx, name);
+        return lockHandle;
+    }
+    async unlock(config, lockHandle) {
+        const ctx = this.getCtx();
+        const name = this.strategy.nameOf(config);
+        // UNLOCK must run stateful (older BASIS #106); restore stateless after.
+        ctx.connection.setSessionType('stateful');
+        const state = await this.strategy.release(ctx, name, lockHandle);
+        ctx.connection.setSessionType('stateless');
+        return state;
+    }
+}
+exports.LockCapability = LockCapability;

package/dist/core/shared/capabilities/VersionsCapability.d.ts

@@ -0,0 +1,15 @@
+import type { IAdtVersionable, IObjectVersion } from '@mcp-abap-adt/interfaces';
+import type { ICapabilityContext, IVersionsStrategy } from './types';
+/**
+ * Shared version history for source-backed objects. Types without a
+ * /source/main resource do NOT compose this capability and do not implement
+ * IAdtVersionable — absence is expressed structurally, not by throwing.
+ */
+export declare class VersionsCapability<TConfig> implements IAdtVersionable<TConfig> {
+    private readonly getCtx;
+    private readonly strategy;
+    constructor(getCtx: () => ICapabilityContext, strategy: IVersionsStrategy<TConfig>);
+    getVersions(config: Partial<TConfig>): Promise<IObjectVersion[]>;
+    getVersionSource(contentUri: string): Promise<string>;
+}
+//# sourceMappingURL=VersionsCapability.d.ts.map

package/dist/core/shared/capabilities/VersionsCapability.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"VersionsCapability.d.ts","sourceRoot":"","sources":["../../../../src/core/shared/capabilities/VersionsCapability.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAChF,OAAO,KAAK,EAAE,kBAAkB,EAAE,iBAAiB,EAAE,MAAM,SAAS,CAAC;AAErE;;;;GAIG;AACH,qBAAa,kBAAkB,CAAC,OAAO,CAAE,YAAW,eAAe,CAAC,OAAO,CAAC;IAGxE,OAAO,CAAC,QAAQ,CAAC,MAAM;IACvB,OAAO,CAAC,QAAQ,CAAC,QAAQ;gBADR,MAAM,EAAE,MAAM,kBAAkB,EAChC,QAAQ,EAAE,iBAAiB,CAAC,OAAO,CAAC;IAGvD,WAAW,CAAC,MAAM,EAAE,OAAO,CAAC,OAAO,CAAC,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC;IAKhE,gBAAgB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;CAGtD"}

package/dist/core/shared/capabilities/VersionsCapability.js

@@ -0,0 +1,26 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.VersionsCapability = void 0;
+/**
+ * Shared version history for source-backed objects. Types without a
+ * /source/main resource do NOT compose this capability and do not implement
+ * IAdtVersionable — absence is expressed structurally, not by throwing.
+ */
+class VersionsCapability {
+    getCtx;
+    strategy;
+    constructor(
+    // LAZY: see LockCapability — read at call time so it can be a class field.
+    getCtx, strategy) {
+        this.getCtx = getCtx;
+        this.strategy = strategy;
+    }
+    getVersions(config) {
+        const name = this.strategy.nameOf(config);
+        return this.strategy.list(this.getCtx(), name);
+    }
+    getVersionSource(contentUri) {
+        return this.strategy.source(this.getCtx(), contentUri);
+    }
+}
+exports.VersionsCapability = VersionsCapability;

package/dist/core/shared/capabilities/index.d.ts

@@ -0,0 +1,4 @@
+export { LockCapability } from './LockCapability';
+export * from './types';
+export { VersionsCapability } from './VersionsCapability';
+//# sourceMappingURL=index.d.ts.map

package/dist/core/shared/capabilities/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/core/shared/capabilities/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAClD,cAAc,SAAS,CAAC;AACxB,OAAO,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAC"}

package/dist/core/shared/capabilities/index.js

@@ -0,0 +1,22 @@
+"use strict";
+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.VersionsCapability = exports.LockCapability = void 0;
+var LockCapability_1 = require("./LockCapability");
+Object.defineProperty(exports, "LockCapability", { enumerable: true, get: function () { return LockCapability_1.LockCapability; } });
+__exportStar(require("./types"), exports);
+var VersionsCapability_1 = require("./VersionsCapability");
+Object.defineProperty(exports, "VersionsCapability", { enumerable: true, get: function () { return VersionsCapability_1.VersionsCapability; } });

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

@@ -0,0 +1,41 @@
+import type { IAbapConnection, ILogger, IObjectVersion } from '@mcp-abap-adt/interfaces';
+/** The connection + logger every capability implementation needs. */
+export interface ICapabilityContext {
+    readonly connection: IAbapConnection;
+    readonly logger?: ILogger;
+}
+/**
+ * Normalized lock result. The per-type helpers return different shapes
+ * (bare string vs { lockHandle, corrNr }); the capability normalizes up to
+ * this superset. See the spec's "lock normalization contract".
+ */
+export interface INormalizedLock {
+    lockHandle: string;
+    corrNr?: string;
+}
+/**
+ * Per-handler strategy for LockCapability. The handler supplies its own
+ * endpoint knowledge — there is no centralized lifecycle-URI resolver
+ * (buildObjectUri is for group operations only).
+ *
+ * `release` returns the handler's OWN state shape (e.g. `{ unlockResult,
+ * errors: [] }`), not void — the current handlers put the ADT unlock response
+ * into `state.unlockResult`, and dropping it would change observable behaviour.
+ * The capability owns only the session toggling around it.
+ */
+export interface ILockStrategy<TConfig, TReadResult> {
+    /** Extract the object name from config, or throw if missing. */
+    nameOf(config: Partial<TConfig>): string;
+    /** POST _action=LOCK, return the normalized handle. */
+    acquire(ctx: ICapabilityContext, name: string): Promise<INormalizedLock>;
+    /** POST _action=UNLOCK with the handle; build and return the handler state. */
+    release(ctx: ICapabilityContext, name: string, lockHandle: string): Promise<TReadResult>;
+}
+/** Per-handler strategy for VersionsCapability. */
+export interface IVersionsStrategy<TConfig> {
+    nameOf(config: Partial<TConfig>): string;
+    list(ctx: ICapabilityContext, name: string): Promise<IObjectVersion[]>;
+    /** GET the content URI, return the source text. */
+    source(ctx: ICapabilityContext, contentUri: string): Promise<string>;
+}
+//# sourceMappingURL=types.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../../src/core/shared/capabilities/types.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,eAAe,EACf,OAAO,EACP,cAAc,EACf,MAAM,0BAA0B,CAAC;AAElC,qEAAqE;AACrE,MAAM,WAAW,kBAAkB;IACjC,QAAQ,CAAC,UAAU,EAAE,eAAe,CAAC;IACrC,QAAQ,CAAC,MAAM,CAAC,EAAE,OAAO,CAAC;CAC3B;AAED;;;;GAIG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED;;;;;;;;;GASG;AACH,MAAM,WAAW,aAAa,CAAC,OAAO,EAAE,WAAW;IACjD,gEAAgE;IAChE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,OAAO,CAAC,GAAG,MAAM,CAAC;IACzC,uDAAuD;IACvD,OAAO,CAAC,GAAG,EAAE,kBAAkB,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,CAAC,CAAC;IACzE,+EAA+E;IAC/E,OAAO,CACL,GAAG,EAAE,kBAAkB,EACvB,IAAI,EAAE,MAAM,EACZ,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,WAAW,CAAC,CAAC;CACzB;AAED,mDAAmD;AACnD,MAAM,WAAW,iBAAiB,CAAC,OAAO;IACxC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,OAAO,CAAC,GAAG,MAAM,CAAC;IACzC,IAAI,CAAC,GAAG,EAAE,kBAAkB,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC,CAAC;IACvE,mDAAmD;IACnD,MAAM,CAAC,GAAG,EAAE,kBAAkB,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC;CACtE"}

package/dist/core/shared/capabilities/types.js

@@ -0,0 +1,2 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });

package/dist/utils/activationUtils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"activationUtils.d.ts","sourceRoot":"","sources":["../../src/utils/activationUtils.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAC7B,eAAe,EAChB,MAAM,0BAA0B,CAAC;AAKlC;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAC5B,IAAI,EAAE,MAAM,EACZ,IAAI,CAAC,EAAE,MAAM,EACb,UAAU,CAAC,EAAE,MAAM,GAClB,MAAM,CAyGR;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,uBAAuB,CAC3C,UAAU,EAAE,eAAe,EAC3B,SAAS,EAAE,MAAM,EACjB,UAAU,EAAE,MAAM,EAClB,QAAQ,GAAE,OAAc,GACvB,OAAO,CAAC,aAAa,CAAC,CAoBxB"}
+{"version":3,"file":"activationUtils.d.ts","sourceRoot":"","sources":["../../src/utils/activationUtils.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EACV,YAAY,IAAI,aAAa,EAC7B,eAAe,EAChB,MAAM,0BAA0B,CAAC;AAmGlC;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAC5B,IAAI,EAAE,MAAM,EACZ,IAAI,CAAC,EAAE,MAAM,EACb,UAAU,CAAC,EAAE,MAAM,GAClB,MAAM,CAyGR;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,uBAAuB,CAC3C,UAAU,EAAE,eAAe,EAC3B,SAAS,EAAE,MAAM,EACjB,UAAU,EAAE,MAAM,EAClB,QAAQ,GAAE,OAAc,GACvB,OAAO,CAAC,aAAa,CAAC,CA8BxB"}

package/dist/utils/activationUtils.js

@@ -9,9 +9,82 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.buildObjectUri = buildObjectUri;
 exports.activateObjectInSession = activateObjectInSession;
+const fast_xml_parser_1 = require("fast-xml-parser");
 const contentTypes_1 = require("../constants/contentTypes");
 const internalUtils_1 = require("./internalUtils");
 const timeouts_1 = require("./timeouts");
+/**
+ * Extract a human-readable text from a single ADT `<msg>` node.
+ */
+function extractActivationMsgText(msg) {
+    const shortText = msg?.shortText;
+    if (shortText && typeof shortText === 'object' && 'txt' in shortText) {
+        const txt = shortText.txt;
+        if (typeof txt === 'string' && txt.trim()) {
+            return txt.trim();
+        }
+    }
+    if (typeof shortText === 'string' && shortText.trim()) {
+        return shortText.trim();
+    }
+    if (typeof msg?.objDescr === 'string' && msg.objDescr.trim()) {
+        return msg.objDescr.trim();
+    }
+    return 'activation error';
+}
+/**
+ * Inspect an ADT activation response body for an **explicit failure signal**.
+ *
+ * ADT's `/sap/bc/adt/activation` endpoint returns HTTP 200 even when activation
+ * fails (object locked by another session, syntax errors), carrying a
+ * `<chkl:messages>` body with `chkl:properties activationExecuted="false"` and/or
+ * `<msg type="E">` entries. Treating "no HTTP error" as success masks these
+ * failures (issue #78).
+ *
+ * Conservative by design: returns a failure detail string ONLY on a positive
+ * error signal. Empty, unparseable, or unrecognized bodies return `null`
+ * (success) so the many object types whose success-body shape differs are never
+ * regressed into false failures.
+ *
+ * @returns failure detail text, or `null` when no failure signal is present
+ */
+function detectActivationFailure(responseData) {
+    if (typeof responseData !== 'string' || responseData.trim() === '') {
+        return null;
+    }
+    let parsed;
+    try {
+        const parser = new fast_xml_parser_1.XMLParser({
+            ignoreAttributes: false,
+            attributeNamePrefix: '',
+        });
+        parsed = parser.parse(responseData);
+    }
+    catch {
+        return null;
+    }
+    const messages = parsed?.['chkl:messages'];
+    if (!messages) {
+        return null;
+    }
+    const activationExecuted = messages['chkl:properties']?.activationExecuted;
+    const executedFalse = activationExecuted === 'false' || activationExecuted === false;
+    const rawMsg = messages.msg;
+    const msgList = Array.isArray(rawMsg) ? rawMsg : rawMsg ? [rawMsg] : [];
+    const errorTexts = msgList
+        .filter((m) => {
+        const severity = m?.type ??
+            m?.severity;
+        return typeof severity === 'string' && severity.toUpperCase() === 'E';
+    })
+        .map((m) => extractActivationMsgText(m));
+    if (executedFalse || errorTexts.length > 0) {
+        return errorTexts.length > 0
+            ? errorTexts.join('; ')
+            : 'activationExecuted=false';
+    }
+    return null;
+}
 /**
  * Build object URI from name and type
  * Used by both individual and group activation
@@ -127,11 +200,19 @@ async function activateObjectInSession(connection, objectUri, objectName, preaud
         'Content-Type': contentTypes_1.CT_ACTIVATION,
         Accept: 'application/xml',
     };
-    return await connection.makeAdtRequest({
+    const response = await connection.makeAdtRequest({
         url,
         method: 'POST',
         timeout: (0, timeouts_1.getTimeout)('default'),
         data: activationXml,
         headers,
     });
+    // ADT returns HTTP 200 even on failed activation (locked object, syntax
+    // errors). Surface an explicit failure signal as a thrown error instead of
+    // letting callers report a false success (issue #78).
+    const failure = detectActivationFailure(response.data);
+    if (failure) {
+        throw new Error(`Activation of ${objectName} failed: ${failure}`);
+    }
+    return response;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mcp-abap-adt/adt-clients",
-  "version": "7.5.0",
+  "version": "7.6.0",
   "description": "ADT clients for SAP ABAP systems - AdtClient and AdtRuntimeClient",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -113,7 +113,7 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "@mcp-abap-adt/interfaces": "^11.0.0",
+    "@mcp-abap-adt/interfaces": "^11.2.0",
     "@mcp-abap-adt/logger": "^0.1.3",
     "axios": "^1.18.1",
     "fast-xml-parser": "^5.9.3"