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

package/dist/batch/buildBatchPayload.js

@@ -51,9 +51,9 @@ function buildBatchPayload(parts, boundary = createBatchBoundary()) {
             innerRequest,
         ].join('\r\n');
     })
-        .join('');
+        .join('\r\n');
     return {
         boundary,
-        body: `${multipartParts}--${boundary}--\r\n`,
+        body: `${multipartParts}\r\n--${boundary}--\r\n`,
     };
 }

package/dist/utils/acceptNegotiation.d.ts

@@ -7,6 +7,7 @@ export declare function clearAcceptCache(): void;
 export declare function setAcceptCorrectionEnabled(enabled?: boolean): void;
 export declare function getAcceptCorrectionEnabled(): boolean;
 export declare function extractSupportedAccept(error: unknown): string[];
+export declare function extractSupportedContentType(error: unknown): string[];
 export declare function wrapConnectionAcceptNegotiation(connection: IAbapConnection, logger?: ILogger): void;
 export declare function makeAdtRequestWithAcceptNegotiation<T = unknown, D = unknown>(connection: IAbapConnection, request: IAbapRequestOptions, options?: IAcceptNegotiationOptions): Promise<IAdtResponse<T, D>>;
 //# sourceMappingURL=acceptNegotiation.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"acceptNegotiation.d.ts","sourceRoot":"","sources":["../../src/utils/acceptNegotiation.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAEV,eAAe,EACf,mBAAmB,EACnB,YAAY,EACZ,OAAO,EACR,MAAM,0BAA0B,CAAC;AASlC,MAAM,WAAW,yBAAyB;IACxC,sBAAsB,CAAC,EAAE,OAAO,CAAC;IACjC,MAAM,CAAC,EAAE,OAAO,CAAC;CAClB;AAED,wBAAgB,gBAAgB,IAAI,IAAI,CAEvC;AAED,wBAAgB,0BAA0B,CAAC,OAAO,CAAC,EAAE,OAAO,GAAG,IAAI,CAElE;AAED,wBAAgB,0BAA0B,IAAI,OAAO,CAKpD;AAED,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,EAAE,CAoC/D;AAcD,wBAAgB,+BAA+B,CAC7C,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,GACf,IAAI,CAkBN;AAED,wBAAsB,mCAAmC,CACvD,CAAC,GAAG,OAAO,EACX,CAAC,GAAG,OAAO,EAEX,UAAU,EAAE,eAAe,EAC3B,OAAO,EAAE,mBAAmB,EAC5B,OAAO,CAAC,EAAE,yBAAyB,GAClC,OAAO,CAAC,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CA8C7B"}
+{"version":3,"file":"acceptNegotiation.d.ts","sourceRoot":"","sources":["../../src/utils/acceptNegotiation.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAEV,eAAe,EACf,mBAAmB,EACnB,YAAY,EACZ,OAAO,EACR,MAAM,0BAA0B,CAAC;AAUlC,MAAM,WAAW,yBAAyB;IACxC,sBAAsB,CAAC,EAAE,OAAO,CAAC;IACjC,MAAM,CAAC,EAAE,OAAO,CAAC;CAClB;AAED,wBAAgB,gBAAgB,IAAI,IAAI,CAGvC;AAED,wBAAgB,0BAA0B,CAAC,OAAO,CAAC,EAAE,OAAO,GAAG,IAAI,CAElE;AAED,wBAAgB,0BAA0B,IAAI,OAAO,CAKpD;AAED,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,EAAE,CAoC/D;AAED,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,EAAE,CAsCpE;AAcD,wBAAgB,+BAA+B,CAC7C,UAAU,EAAE,eAAe,EAC3B,MAAM,CAAC,EAAE,OAAO,GACf,IAAI,CAkBN;AAED,wBAAsB,mCAAmC,CACvD,CAAC,GAAG,OAAO,EACX,CAAC,GAAG,OAAO,EAEX,UAAU,EAAE,eAAe,EAC3B,OAAO,EAAE,mBAAmB,EAC5B,OAAO,CAAC,EAAE,yBAAyB,GAClC,OAAO,CAAC,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CA6E7B"}

package/dist/utils/acceptNegotiation.js

@@ -4,13 +4,16 @@ exports.clearAcceptCache = clearAcceptCache;
 exports.setAcceptCorrectionEnabled = setAcceptCorrectionEnabled;
 exports.getAcceptCorrectionEnabled = getAcceptCorrectionEnabled;
 exports.extractSupportedAccept = extractSupportedAccept;
+exports.extractSupportedContentType = extractSupportedContentType;
 exports.wrapConnectionAcceptNegotiation = wrapConnectionAcceptNegotiation;
 exports.makeAdtRequestWithAcceptNegotiation = makeAdtRequestWithAcceptNegotiation;
 const acceptCache = new Map();
+const contentTypeCache = new Map();
 const baseRequestMap = new WeakMap();
 let acceptCorrectionOverride;
 function clearAcceptCache() {
     acceptCache.clear();
+    contentTypeCache.clear();
 }
 function setAcceptCorrectionEnabled(enabled) {
     acceptCorrectionOverride = enabled;
@@ -19,7 +22,7 @@ function getAcceptCorrectionEnabled() {
     if (acceptCorrectionOverride !== undefined) {
         return acceptCorrectionOverride;
     }
-    return process.env.ADT_ACCEPT_CORRECTION === 'true';
+    return process.env.ADT_ACCEPT_CORRECTION !== 'false';
 }
 function extractSupportedAccept(error) {
     const types = new Set();
@@ -53,6 +56,39 @@ function extractSupportedAccept(error) {
     }
     return Array.from(types).filter(Boolean);
 }
+function extractSupportedContentType(error) {
+    const e = error;
+    if (e?.response?.status !== 415) {
+        return [];
+    }
+    const types = new Set();
+    const headers = (e?.response?.headers || {});
+    const headerCandidates = [
+        headers['content-type'],
+        headers['x-sap-adt-supported-content-type'],
+        headers['supported-content-type'],
+    ];
+    for (const value of headerCandidates) {
+        if (typeof value === 'string') {
+            value
+                .split(',')
+                .map((entry) => entry.trim())
+                .filter(Boolean)
+                .forEach((entry) => {
+                types.add(entry);
+            });
+        }
+    }
+    const data = e?.response?.data;
+    const text = typeof data === 'string' ? data : data ? JSON.stringify(data) : '';
+    if (text) {
+        const matches = text.match(/[a-zA-Z0-9.+-]+\/[a-zA-Z0-9.+-]+(?:;[^,\s]+)?/g) || [];
+        for (const match of matches) {
+            types.add(match);
+        }
+    }
+    return Array.from(types).filter(Boolean);
+}
 function buildCacheKey(request) {
     return `${request.method.toUpperCase()} ${request.url}`;
 }
@@ -78,6 +114,12 @@ async function makeAdtRequestWithAcceptNegotiation(connection, request, options)
     if (cachedAccept) {
         headers.Accept = cachedAccept;
     }
+    const cachedContentType = enableCorrection
+        ? contentTypeCache.get(cacheKey)
+        : undefined;
+    if (cachedContentType) {
+        headers['Content-Type'] = cachedContentType;
+    }
     const baseRequest = getBaseRequest(connection);
     try {
         return await baseRequest({
@@ -104,6 +146,23 @@ async function makeAdtRequestWithAcceptNegotiation(connection, request, options)
                 }
             }
         }
+        if (e.response?.status === 415) {
+            const supported = extractSupportedContentType(error);
+            if (supported.length > 0) {
+                logger?.warn?.(`Content-Type not supported for ${request.url}. Supported Content-Type: ${supported.join(', ')}`);
+            }
+            if (enableCorrection && supported.length > 0) {
+                const nextContentType = supported[0];
+                if (headers['Content-Type'] !== nextContentType) {
+                    contentTypeCache.set(cacheKey, nextContentType);
+                    logger?.warn?.(`Retrying ${request.url} with corrected Content-Type: ${nextContentType}`);
+                    return await baseRequest({
+                        ...request,
+                        headers: { ...headers, 'Content-Type': nextContentType },
+                    });
+                }
+            }
+        }
         throw error;
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mcp-abap-adt/adt-clients",
-  "version": "3.11.3",
+  "version": "3.12.0",
   "description": "ADT clients for SAP ABAP systems - AdtClient and AdtRuntimeClient",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -60,7 +60,8 @@
     "shared:teardown": "npx jest --testPathIgnorePatterns node_modules --testPathPatterns admin/shared-deps/teardown",
     "shared:check": "npx jest --testPathIgnorePatterns node_modules --testPathPatterns admin/shared-deps/check",
     "pretest": "npm run test:check:integration",
-    "prepublishOnly": "npm run build"
+    "prepublishOnly": "npm run build",
+    "prepare": "husky"
   },
   "engines": {
     "node": ">=18.0.0"
@@ -81,6 +82,7 @@
     "@types/jest": "^30.0.0",
     "@types/node": "^25.3.3",
     "dotenv": "^17.3.1",
+    "husky": "^9.1.7",
     "jest": "^30.0.5",
     "jest-util": "^30.2.0",
     "ts-jest": "^29.4.1",