sap-adt-mcp 0.8.41 → 0.8.43

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sap-adt-mcp",
-  "version": "0.8.41",
+  "version": "0.8.43",
   "mcpName": "io.github.yzonur/sap-adt-mcp",
   "description": "MCP server giving Claude live access to SAP systems via ADT (ABAP Development Tools) REST. Read source, search, run syntax checks, and edit ABAP objects from any MCP-compatible client.",
   "type": "module",

package/src/object-create.js

@@ -201,6 +201,48 @@ export function buildCreateRequest({
   }
 }
 
+// Most ADT create endpoints version their media type (…v2+xml, …v3+xml). The
+// shapes above target the newest version a modern stack accepts, but older
+// systems only know an earlier one and answer the newer media type with
+// 415 ExceptionUnsupportedMediaType. Given a content-type, return the chain to
+// try: the requested version first, then every lower version down to v1.
+// A content-type with no `.vN+xml` segment is returned unchanged (single item).
+export function mediaTypeFallbacks(contentType) {
+  const m = /^(.*\.)v(\d+)(\+xml)$/.exec(contentType);
+  if (!m) return [contentType];
+  const [, prefix, ver, suffix] = m;
+  const chain = [];
+  for (let v = Number(ver); v >= 1; v--) {
+    chain.push(`${prefix}v${v}${suffix}`);
+  }
+  return chain;
+}
+
+// POST a create request, retrying with successively lower media-type versions
+// when the server rejects the content-type with 415. Returns
+// { res, text, contentType } for the first attempt that is not a 415 — or the
+// last attempt if every version is rejected. The body is read once here so
+// callers must use the returned `text` rather than calling res.text() again.
+export async function postCreate(client, { path, contentType, body, query }) {
+  const types = mediaTypeFallbacks(contentType);
+  let res;
+  let text;
+  let used;
+  for (let i = 0; i < types.length; i++) {
+    used = types[i];
+    res = await client.request({
+      method: "POST",
+      path,
+      query,
+      headers: { "Content-Type": used },
+      body,
+    });
+    text = await res.text();
+    if (res.status !== 415 || i === types.length - 1) break;
+  }
+  return { res, text, contentType: used };
+}
+
 function xmlDecl() {
   return `<?xml version="1.0" encoding="UTF-8"?>`;
 }

package/src/object-uris.js

@@ -25,7 +25,10 @@ const TYPE_ALIASES = {
   fugr: "FUGR",
   table: "TABL",
   tabl: "TABL",
-  structure: "TABL",
+  // Structures live at their own ADT endpoint (/ddic/structures), NOT
+  // /ddic/tables — routing them to TABL sent reads/locks to the wrong URI.
+  structure: "STRU",
+  stru: "STRU",
   dataelement: "DTEL",
   dtel: "DTEL",
   domain: "DOMA",
@@ -105,6 +108,8 @@ export function objectUri({ type, name, group }) {
     }
     case "TABL":
       return `/sap/bc/adt/ddic/tables/${enc(n)}`;
+    case "STRU":
+      return `/sap/bc/adt/ddic/structures/${enc(n)}`;
     case "DTEL":
       return `/sap/bc/adt/ddic/dataelements/${enc(n)}`;
     case "DOMA":
@@ -160,7 +165,7 @@ export function sourceUri({ type, name, group, include }) {
   // via the object URI directly, returned as XML. CDS / DCLS / DDLX / BDEF do
   // use /source/main.
   if (t === "DTEL" || t === "DOMA" || t === "MSAG") return base;
-  if (t === "TABL") return `${base}/source/main`;
+  if (t === "TABL" || t === "STRU") return `${base}/source/main`;
 
   return `${base}/source/main`;
 }

package/src/tools/_shared.js

@@ -1,5 +1,5 @@
 export const OBJECT_TYPE_HINT =
-  "Object type — friendly alias (program, class, interface, function, functiongroup, include, table, dataelement, domain, cds) or TADIR code (PROG, CLAS, INTF, FUGR, FUGR/FF, INCL, TABL, DTEL, DOMA, DDLS).";
+  "Object type — friendly alias (program, class, interface, function, functiongroup, include, table, structure, dataelement, domain, cds) or TADIR code (PROG, CLAS, INTF, FUGR, FUGR/FF, INCL, TABL, DTEL, DOMA, DDLS).";
 
 export const SYSTEM_HINT = "System name. Omit for default.";
 

package/src/tools/lifecycle.js

@@ -1,7 +1,7 @@
 import { objectUri, normalizeType } from "../object-uris.js";
 import { escapeXml } from "../xml.js";
 import { acquireLock, releaseLock } from "../lock.js";
-import { buildCreateRequest } from "../object-create.js";
+import { buildCreateRequest, postCreate } from "../object-create.js";
 import { errorResult, jsonResult, textResult } from "../result.js";
 import { OBJECT_TYPE_HINT, SYSTEM_HINT } from "./_shared.js";
 
@@ -199,14 +199,12 @@ export function register({ getClient }) {
       }
       const query = {};
       if (args.transport) query.corrNr = args.transport;
-      const res = await client.request({
-        method: "POST",
+      const { res, text } = await postCreate(client, {
         path: req.path,
-        query,
-        headers: { "Content-Type": req.contentType },
+        contentType: req.contentType,
         body: req.body,
+        query,
       });
-      const text = await res.text();
       if (!res.ok) return errorResult(sys, res.status, text, res.headers.get("content-type"));
       const newUri = objectUri({
         type: args.type,

package/src/tools/rap.js

@@ -1,4 +1,4 @@
-import { buildCreateRequest } from "../object-create.js";
+import { buildCreateRequest, postCreate } from "../object-create.js";
 import { sourceUri } from "../object-uris.js";
 import { acquireLock, releaseLock } from "../lock.js";
 import { escapeXml } from "../xml.js";
@@ -159,15 +159,13 @@ function createInfoFor(artifact, spec) {
 
 async function createAndWrite(client, artifact, spec) {
   const info = createInfoFor(artifact, spec);
-  // 1) create the object
-  const createRes = await client.request({
-    method: "POST",
+  // 1) create the object (retrying with lower media-type versions on 415)
+  const { res: createRes, text: createText } = await postCreate(client, {
     path: info.path,
-    headers: { "Content-Type": info.contentType },
-    query: spec.transport ? { corrNr: spec.transport } : undefined,
+    contentType: info.contentType,
     body: info.body,
+    query: spec.transport ? { corrNr: spec.transport } : undefined,
   });
-  const createText = await createRes.text();
   if (!createRes.ok) {
     return { name: artifact.name, stage: "create", status: createRes.status, error: createText.slice(0, 400) };
   }