arc-1 0.6.7 → 0.6.8

package/dist/adt/refactoring.js

@@ -0,0 +1,117 @@
+/**
+ * ADT Refactoring API — Change Package Assignment.
+ *
+ * Moves ABAP objects between packages via the ADT refactoring endpoint.
+ * Two-step flow: preview (validate + discover transport) then execute.
+ *
+ * Endpoint: POST /sap/bc/adt/refactorings
+ *   - Preview: ?step=preview&rel=http://www.sap.com/adt/relations/refactoring/changepackage
+ *   - Execute: ?step=execute
+ *
+ * Preview uses "wrapped" XML (outer changePackageRefactoring element).
+ * Execute uses "unwrapped" XML (just genericRefactoring element).
+ */
+import { checkOperation, OperationType } from './safety.js';
+const NS_CHANGEPACKAGE = 'http://www.sap.com/adt/refactoring/changepackagerefactoring';
+const NS_GENERIC = 'http://www.sap.com/adt/refactoring/genericrefactoring';
+const NS_ADTCORE = 'http://www.sap.com/adt/core';
+/** Escape XML special characters */
+function escapeXml(s) {
+    return s.replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;').replace(/"/g, '&quot;');
+}
+/** Build the inner genericRefactoring XML fragment (shared by preview and execute) */
+function buildGenericRefactoringInner(params) {
+    const desc = escapeXml(params.description ?? 'ABAP Object');
+    const transport = params.transport ?? '';
+    return [
+        `<generic:title>Change Package</generic:title>`,
+        `<generic:adtObjectUri>${escapeXml(params.objectUri)}</generic:adtObjectUri>`,
+        `<generic:affectedObjects>`,
+        `<generic:affectedObject`,
+        ` adtcore:description="${desc}"`,
+        ` adtcore:name="${escapeXml(params.objectName)}"`,
+        ` adtcore:packageName="${escapeXml(params.oldPackage)}"`,
+        ` adtcore:type="${escapeXml(params.objectType)}"`,
+        ` adtcore:uri="${escapeXml(params.objectUri)}">`,
+        `<generic:userContent/>`,
+        `<generic:changePackageDelta>`,
+        `<generic:newPackage>${escapeXml(params.newPackage)}</generic:newPackage>`,
+        `</generic:changePackageDelta>`,
+        `</generic:affectedObject>`,
+        `</generic:affectedObjects>`,
+        `<generic:transport>${escapeXml(transport)}</generic:transport>`,
+        `<generic:ignoreSyntaxErrorsAllowed>false</generic:ignoreSyntaxErrorsAllowed>`,
+        `<generic:ignoreSyntaxErrors>false</generic:ignoreSyntaxErrors>`,
+        `<generic:userContent/>`,
+    ].join('');
+}
+/**
+ * Build "wrapped" preview XML.
+ * Root: <changepackage:changePackageRefactoring> with inner <generic:genericRefactoring>.
+ */
+export function buildPreviewXml(params) {
+    return [
+        `<?xml version="1.0" encoding="UTF-8"?>`,
+        `<changepackage:changePackageRefactoring`,
+        ` xmlns:adtcore="${NS_ADTCORE}"`,
+        ` xmlns:generic="${NS_GENERIC}"`,
+        ` xmlns:changepackage="${NS_CHANGEPACKAGE}">`,
+        `<changepackage:oldPackage>${escapeXml(params.oldPackage)}</changepackage:oldPackage>`,
+        `<changepackage:newPackage>${escapeXml(params.newPackage)}</changepackage:newPackage>`,
+        `<generic:genericRefactoring>`,
+        buildGenericRefactoringInner(params),
+        `</generic:genericRefactoring>`,
+        `<changepackage:userContent/>`,
+        `</changepackage:changePackageRefactoring>`,
+    ].join('');
+}
+/**
+ * Build "unwrapped" execute XML.
+ * Root: <generic:genericRefactoring> (no changePackageRefactoring wrapper).
+ */
+export function buildExecuteXml(params) {
+    return [
+        `<?xml version="1.0" encoding="UTF-8"?>`,
+        `<generic:genericRefactoring`,
+        ` xmlns:generic="${NS_GENERIC}"`,
+        ` xmlns:adtcore="${NS_ADTCORE}">`,
+        buildGenericRefactoringInner(params),
+        `</generic:genericRefactoring>`,
+    ].join('');
+}
+/** Extract transport value from preview response XML */
+export function parsePreviewResponse(xml) {
+    const match = xml.match(/<generic:transport>([^<]*)<\/generic:transport>/);
+    if (!match) {
+        // Try without namespace prefix (SAP may strip them)
+        const fallback = xml.match(/<transport>([^<]*)<\/transport>/);
+        const value = fallback?.[1]?.trim();
+        return { transport: value || undefined };
+    }
+    const value = match[1]?.trim();
+    return { transport: value || undefined };
+}
+const PREVIEW_URL = '/sap/bc/adt/refactorings?step=preview&rel=http://www.sap.com/adt/relations/refactoring/changepackage';
+const EXECUTE_URL = '/sap/bc/adt/refactorings?step=execute';
+/**
+ * Move an ABAP object to a different package via the ADT refactoring API.
+ *
+ * Two-step flow:
+ * 1. Preview: validates the operation, returns server-assigned transport (if any)
+ * 2. Execute: performs the actual TADIR change
+ */
+export async function changePackage(http, safety, params) {
+    checkOperation(safety, OperationType.Update, 'ChangePackage');
+    // Step 1: Preview
+    const previewXml = buildPreviewXml(params);
+    const previewResp = await http.post(PREVIEW_URL, previewXml, 'application/*', { Accept: 'application/*' });
+    const preview = parsePreviewResponse(previewResp.body);
+    // Merge transport: explicit param > preview response > empty
+    const effectiveTransport = params.transport || preview.transport || '';
+    const executeParams = { ...params, transport: effectiveTransport };
+    // Step 2: Execute
+    const executeXml = buildExecuteXml(executeParams);
+    await http.post(EXECUTE_URL, executeXml, 'application/*', { Accept: 'application/*' });
+    return { transport: effectiveTransport || undefined };
+}
+//# sourceMappingURL=refactoring.js.map

package/dist/adt/refactoring.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"refactoring.js","sourceRoot":"","sources":["../../src/adt/refactoring.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAGH,OAAO,EAAE,cAAc,EAAE,aAAa,EAAqB,MAAM,aAAa,CAAC;AA0B/E,MAAM,gBAAgB,GAAG,6DAA6D,CAAC;AACvF,MAAM,UAAU,GAAG,uDAAuD,CAAC;AAC3E,MAAM,UAAU,GAAG,6BAA6B,CAAC;AAEjD,oCAAoC;AACpC,SAAS,SAAS,CAAC,CAAS;IAC1B,OAAO,CAAC,CAAC,OAAO,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC,OAAO,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC,OAAO,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC,OAAO,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC;AACtG,CAAC;AAED,sFAAsF;AACtF,SAAS,4BAA4B,CAAC,MAA2B;IAC/D,MAAM,IAAI,GAAG,SAAS,CAAC,MAAM,CAAC,WAAW,IAAI,aAAa,CAAC,CAAC;IAC5D,MAAM,SAAS,GAAG,MAAM,CAAC,SAAS,IAAI,EAAE,CAAC;IACzC,OAAO;QACL,+CAA+C;QAC/C,yBAAyB,SAAS,CAAC,MAAM,CAAC,SAAS,CAAC,yBAAyB;QAC7E,2BAA2B;QAC3B,yBAAyB;QACzB,yBAAyB,IAAI,GAAG;QAChC,kBAAkB,SAAS,CAAC,MAAM,CAAC,UAAU,CAAC,GAAG;QACjD,yBAAyB,SAAS,CAAC,MAAM,CAAC,UAAU,CAAC,GAAG;QACxD,kBAAkB,SAAS,CAAC,MAAM,CAAC,UAAU,CAAC,GAAG;QACjD,iBAAiB,SAAS,CAAC,MAAM,CAAC,SAAS,CAAC,IAAI;QAChD,wBAAwB;QACxB,8BAA8B;QAC9B,uBAAuB,SAAS,CAAC,MAAM,CAAC,UAAU,CAAC,uBAAuB;QAC1E,+BAA+B;QAC/B,2BAA2B;QAC3B,4BAA4B;QAC5B,sBAAsB,SAAS,CAAC,SAAS,CAAC,sBAAsB;QAChE,8EAA8E;QAC9E,gEAAgE;QAChE,wBAAwB;KACzB,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;AACb,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,eAAe,CAAC,MAA2B;IACzD,OAAO;QACL,wCAAwC;QACxC,yCAAyC;QACzC,mBAAmB,UAAU,GAAG;QAChC,mBAAmB,UAAU,GAAG;QAChC,yBAAyB,gBAAgB,IAAI;QAC7C,6BAA6B,SAAS,CAAC,MAAM,CAAC,UAAU,CAAC,6BAA6B;QACtF,6BAA6B,SAAS,CAAC,MAAM,CAAC,UAAU,CAAC,6BAA6B;QACtF,8BAA8B;QAC9B,4BAA4B,CAAC,MAAM,CAAC;QACpC,+BAA+B;QAC/B,8BAA8B;QAC9B,2CAA2C;KAC5C,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;AACb,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,eAAe,CAAC,MAA2B;IACzD,OAAO;QACL,wCAAwC;QACxC,6BAA6B;QAC7B,mBAAmB,UAAU,GAAG;QAChC,mBAAmB,UAAU,IAAI;QACjC,4BAA4B,CAAC,MAAM,CAAC;QACpC,+BAA+B;KAChC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;AACb,CAAC;AAED,wDAAwD;AACxD,MAAM,UAAU,oBAAoB,CAAC,GAAW;IAC9C,MAAM,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC,iDAAiD,CAAC,CAAC;IAC3E,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,oDAAoD;QACpD,MAAM,QAAQ,GAAG,GAAG,CAAC,KAAK,CAAC,iCAAiC,CAAC,CAAC;QAC9D,MAAM,KAAK,GAAG,QAAQ,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC;QACpC,OAAO,EAAE,SAAS,EAAE,KAAK,IAAI,SAAS,EAAE,CAAC;IAC3C,CAAC;IACD,MAAM,KAAK,GAAG,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC;IAC/B,OAAO,EAAE,SAAS,EAAE,KAAK,IAAI,SAAS,EAAE,CAAC;AAC3C,CAAC;AAED,MAAM,WAAW,GACf,sGAAsG,CAAC;AACzG,MAAM,WAAW,GAAG,uCAAuC,CAAC;AAE5D;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CACjC,IAAmB,EACnB,MAAoB,EACpB,MAA2B;IAE3B,cAAc,CAAC,MAAM,EAAE,aAAa,CAAC,MAAM,EAAE,eAAe,CAAC,CAAC;IAE9D,kBAAkB;IAClB,MAAM,UAAU,GAAG,eAAe,CAAC,MAAM,CAAC,CAAC;IAC3C,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,UAAU,EAAE,eAAe,EAAE,EAAE,MAAM,EAAE,eAAe,EAAE,CAAC,CAAC;IAC3G,MAAM,OAAO,GAAG,oBAAoB,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC;IAEvD,6DAA6D;IAC7D,MAAM,kBAAkB,GAAG,MAAM,CAAC,SAAS,IAAI,OAAO,CAAC,SAAS,IAAI,EAAE,CAAC;IACvE,MAAM,aAAa,GAAG,EAAE,GAAG,MAAM,EAAE,SAAS,EAAE,kBAAkB,EAAE,CAAC;IAEnE,kBAAkB;IAClB,MAAM,UAAU,GAAG,eAAe,CAAC,aAAa,CAAC,CAAC;IAClD,MAAM,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,UAAU,EAAE,eAAe,EAAE,EAAE,MAAM,EAAE,eAAe,EAAE,CAAC,CAAC;IAEvF,OAAO,EAAE,SAAS,EAAE,kBAAkB,IAAI,SAAS,EAAE,CAAC;AACxD,CAAC"}

package/dist/handlers/intent.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"intent.d.ts","sourceRoot":"","sources":["../../src/handlers/intent.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gDAAgD,CAAC;AAC/E,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,2CAA2C,CAAC;AACxE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AA+ElD,OAAO,KAAK,EAAkB,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AAGxE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AAc9D,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAKvD,2BAA2B;AAC3B,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC/C,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAED;;;;;;;;;GASG;AACH,eAAO,MAAM,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAY9C,CAAC;AAEF;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,QAAQ,EAAE,aAAa,EAAE,MAAM,GAAG,OAAO,CASnF;AAeD;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,OAAO,CAAA;CAAE,CAe1F;AAED;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CASzD;AA2KD;;;;;;;GAOG;AACH,wBAAsB,cAAc,CAClC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,YAAY,EACpB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,QAAQ,CAAC,EAAE,QAAQ,EACnB,OAAO,CAAC,EAAE,MAAM,EAChB,YAAY,CAAC,EAAE,YAAY,EAC3B,eAAe,CAAC,EAAE,OAAO,GACxB,OAAO,CAAC,UAAU,CAAC,CAsLrB;AAizBD;;;GAGG;AACH,wBAAgB,uBAAuB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAyB1E;AAED,wBAAgB,cAAc,CAC5B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,EACZ,GAAG,EAAE,MAAM,EACX,WAAW,EAAE,MAAM,EACnB,UAAU,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GACnC,MAAM,CA6NR;AAuCD,2EAA2E;AAC3E,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAIxD;AAqnDD,0CAA0C;AAC1C,wBAAgB,mBAAmB,IAAI,IAAI,CAG1C;AAED,gEAAgE;AAChE,wBAAgB,iBAAiB,CAAC,QAAQ,EAAE,gBAAgB,GAAG,SAAS,GAAG,IAAI,CAE9E;AAED,2DAA2D;AAC3D,wBAAgB,iBAAiB,IAAI,gBAAgB,GAAG,SAAS,CAEhE;AAED,iDAAiD;AACjD,wBAAgB,kBAAkB,CAAC,GAAG,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,CAAC,GAAG,IAAI,CAEnE;AAED,iDAAiD;AACjD,wBAAgB,kBAAkB,IAAI,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,CAAC,CAE1D"}
+{"version":3,"file":"intent.d.ts","sourceRoot":"","sources":["../../src/handlers/intent.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gDAAgD,CAAC;AAC/E,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,2CAA2C,CAAC;AACxE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAkFlD,OAAO,KAAK,EAAkB,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AAGxE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AAc9D,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAKvD,2BAA2B;AAC3B,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC/C,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAED;;;;;;;;;GASG;AACH,eAAO,MAAM,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAY9C,CAAC;AAEF;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,QAAQ,EAAE,aAAa,EAAE,MAAM,GAAG,OAAO,CASnF;AAeD;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,OAAO,CAAA;CAAE,CAe1F;AAED;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CASzD;AA2KD;;;;;;;GAOG;AACH,wBAAsB,cAAc,CAClC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,YAAY,EACpB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,QAAQ,CAAC,EAAE,QAAQ,EACnB,OAAO,CAAC,EAAE,MAAM,EAChB,YAAY,CAAC,EAAE,YAAY,EAC3B,eAAe,CAAC,EAAE,OAAO,GACxB,OAAO,CAAC,UAAU,CAAC,CAsLrB;AAo0BD;;;GAGG;AACH,wBAAgB,uBAAuB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAyB1E;AAED,wBAAgB,cAAc,CAC5B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,EACZ,GAAG,EAAE,MAAM,EACX,WAAW,EAAE,MAAM,EACnB,UAAU,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GACnC,MAAM,CA6NR;AAwCD,2EAA2E;AAC3E,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAIxD;AA+wDD,0CAA0C;AAC1C,wBAAgB,mBAAmB,IAAI,IAAI,CAG1C;AAED,gEAAgE;AAChE,wBAAgB,iBAAiB,CAAC,QAAQ,EAAE,gBAAgB,GAAG,SAAS,GAAG,IAAI,CAE9E;AAED,2DAA2D;AAC3D,wBAAgB,iBAAiB,IAAI,gBAAgB,GAAG,SAAS,CAEhE;AAED,iDAAiD;AACjD,wBAAgB,kBAAkB,CAAC,GAAG,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,CAAC,GAAG,IAAI,CAEnE;AAED,iDAAiD;AACjD,wBAAgB,kBAAkB,IAAI,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,CAAC,CAE1D"}

package/dist/handlers/intent.js

@@ -11,12 +11,13 @@
  */
 import { findDefinition, findReferences, findWhereUsed, getCompletion, } from '../adt/codeintel.js';
 import { createObject, deleteObject, lockObject, safeUpdateObject, safeUpdateSource, unlockObject, updateObject, } from '../adt/crud.js';
-import { buildDataElementXml, buildDomainXml, buildMessageClassXml, buildPackageXml, buildServiceBindingXml, } from '../adt/ddic-xml.js';
+import { buildDataElementXml, buildDomainXml, buildMessageClassXml, buildPackageXml, buildServiceBindingXml, decodeKtdText, rewriteKtdText, } from '../adt/ddic-xml.js';
 import { activate, activateBatch, applyFixProposal, getFixProposals, publishServiceBinding, runAtcCheck, runUnitTests, syntaxCheck, unpublishServiceBinding, } from '../adt/devtools.js';
 import { getDump, getTraceDbAccesses, getTraceHitlist, getTraceStatements, listDumps, listTraces, } from '../adt/diagnostics.js';
 import { AdtApiError, AdtNetworkError, AdtSafetyError, classifySapDomainError, isNotFoundError, } from '../adt/errors.js';
 import { classifyTextSearchError, mapSapReleaseToAbaplintVersion, probeFeatures } from '../adt/features.js';
 import { addTileToGroup, createCatalog, createGroup, createTile, deleteCatalog, listCatalogs, listGroups, listTiles, } from '../adt/flp.js';
+import { changePackage } from '../adt/refactoring.js';
 import { checkOperation, checkPackage, isOperationAllowed, OperationType } from '../adt/safety.js';
 import { createTransport, deleteTransport, getTransport, getTransportInfo, listTransports, reassignTransport, releaseTransport, releaseTransportRecursive, } from '../adt/transport.js';
 import { getAppInfo } from '../adt/ui5-repository.js';
@@ -591,6 +592,21 @@ async function handleSAPRead(client, args, cachingLayer) {
             const { source, cacheHit } = await cachedGet('SRVB', name, () => client.getSrvb(name));
             return cachedTextResult(source, cacheHit);
         }
+        case 'SKTD': {
+            try {
+                // ADT returns a <sktd:docu> XML envelope with the Markdown body base64-encoded
+                // inside <sktd:text>. Cache the raw envelope (update flow re-uses it) and
+                // return the decoded Markdown to the LLM.
+                const { source, cacheHit } = await cachedGet('SKTD', name, () => client.getKtd(name));
+                return cachedTextResult(decodeKtdText(source), cacheHit);
+            }
+            catch (err) {
+                if (isNotFoundError(err)) {
+                    return textResult(`No Knowledge Transfer Document (SKTD) found for "${name}". KTD docs are optional Markdown documentation attached to ABAP objects — either one was never created for "${name}", or the name is wrong.`);
+                }
+                throw err;
+            }
+        }
         case 'TABL': {
             const { source, cacheHit } = await cachedGet('TABL', name, () => client.getTable(name));
             return cachedTextResult(source, cacheHit);
@@ -747,7 +763,7 @@ async function handleSAPRead(client, args, cachingLayer) {
             }
         }
         default:
-            return errorResult(`Unknown SAPRead type: "${type}". Supported types: PROG, CLAS, INTF, FUNC, FUGR, INCL, DDLS, DCLS, DDLX, BDEF, SRVD, SRVB, TABL, VIEW, STRU, DOMA, DTEL, TRAN, TABLE_CONTENTS, DEVC, SOBJ, SYSTEM, COMPONENTS, MESSAGES, TEXT_ELEMENTS, VARIANTS, BSP, BSP_DEPLOY, API_STATE, INACTIVE_OBJECTS. ` +
+            return errorResult(`Unknown SAPRead type: "${type}". Supported types: PROG, CLAS, INTF, FUNC, FUGR, INCL, DDLS, DCLS, DDLX, BDEF, SRVD, SRVB, SKTD, TABL, VIEW, STRU, DOMA, DTEL, TRAN, TABLE_CONTENTS, DEVC, SOBJ, SYSTEM, COMPONENTS, MESSAGES, TEXT_ELEMENTS, VARIANTS, BSP, BSP_DEPLOY, API_STATE, INACTIVE_OBJECTS. ` +
                 'Tip: Type aliases are auto-normalized (e.g., DDLS/DF → DDLS, DCLS/DL → DCLS, CLAS/OC → CLAS, PROG/P → PROG). ' +
                 'Do not pass a URI — use the "type" and "name" parameters instead.');
     }
@@ -904,12 +920,13 @@ const DATAELEMENT_V2_CONTENT_TYPE = 'application/vnd.sap.adt.dataelements.v2+xml
 const SERVICEBINDING_V2_CONTENT_TYPE = 'application/vnd.sap.adt.businessservices.servicebinding.v2+xml; charset=utf-8';
 const BDEF_CONTENT_TYPE = 'application/vnd.sap.adt.blues.v1+xml';
 const MESSAGECLASS_CONTENT_TYPE = 'application/vnd.sap.adt.mc.messageclass+xml';
+const SKTD_V2_CONTENT_TYPE = 'application/vnd.sap.adt.sktdv2+xml';
 function isMetadataWriteType(type) {
     return type === 'DOMA' || type === 'DTEL' || type === 'MSAG' || type === 'SRVB';
 }
 /** Types that require a specific vendor content type for creation (not application/*) */
 function needsVendorContentType(type) {
-    return type === 'DOMA' || type === 'DTEL' || type === 'BDEF' || type === 'MSAG';
+    return type === 'DOMA' || type === 'DTEL' || type === 'BDEF' || type === 'MSAG' || type === 'SKTD';
 }
 /** Content type used for create POST */
 function createContentTypeForType(type) {
@@ -946,6 +963,8 @@ function vendorContentTypeForType(type) {
             return BDEF_CONTENT_TYPE;
         case 'MSAG':
             return MESSAGECLASS_CONTENT_TYPE;
+        case 'SKTD':
+            return SKTD_V2_CONTENT_TYPE;
         default:
             // Wildcard lets the SAP server resolve the correct handler.
             // Sending 'application/xml' causes 415 on DDL-based endpoints
@@ -1435,6 +1454,7 @@ const SLASH_TYPE_MAP = {
     'DEVC/K': 'DEVC',
     'TRAN/O': 'TRAN',
     'VIEW/V': 'VIEW',
+    'SKTD/TYP': 'SKTD',
 };
 /** Normalize ADT type codes and aliases to ARC-1 canonical short types. */
 export function normalizeObjectType(type) {
@@ -1545,13 +1565,17 @@ function objectBasePath(type) {
             return '/sap/bc/adt/packages/';
         case 'TRAN':
             return '/sap/bc/adt/vit/wb/object_type/trant/object_name/';
+        case 'SKTD':
+            return '/sap/bc/adt/documentation/ktd/documents/';
         default:
             return '/sap/bc/adt/programs/programs/';
     }
 }
 /** Map object type + name to the ADT object URL used by CRUD/DevTools/etc. Name is URI-encoded. */
 function objectUrlForType(type, name) {
-    return `${objectBasePath(type)}${encodeURIComponent(name)}`;
+    // KTD endpoints require lowercase object names in the URL path (confirmed via Eclipse ADT trace).
+    const effectiveName = type === 'SKTD' ? name.toLowerCase() : name;
+    return `${objectBasePath(type)}${encodeURIComponent(effectiveName)}`;
 }
 /** Infer SAP object type from naming conventions. Returns empty string if type cannot be determined. */
 function inferObjectType(name) {
@@ -1569,7 +1593,8 @@ function inferObjectType(name) {
  * Used for API release state where the full URI is encoded as a single path segment by the caller.
  */
 function objectUrlForTypeRaw(type, name) {
-    return `${objectBasePath(type)}${name}`;
+    const effectiveName = type === 'SKTD' ? name.toLowerCase() : name;
+    return `${objectBasePath(type)}${effectiveName}`;
 }
 /** Get the source URL for an object (appends /source/main) */
 function sourceUrlForType(type, name) {
@@ -1602,6 +1627,19 @@ async function handleSAPWrite(client, args, config, cachingLayer) {
     switch (action) {
         case 'update': {
             const existingPackage = await enforcePackageForExistingObject();
+            if (type === 'SKTD') {
+                // KTD update requires the full <sktd:docu> XML envelope with the Markdown
+                // body base64-encoded inside <sktd:text>, PUT with
+                // `application/vnd.sap.adt.sktdv2+xml`. PUTting raw text/plain silently
+                // no-ops (or 415s on strict systems). Fetch the current envelope,
+                // replace only the <sktd:text> body, and PUT it back — preserves
+                // responsible/masterLanguage/packageRef/refObject metadata.
+                const currentEnvelope = await client.getKtd(name);
+                const body = rewriteKtdText(currentEnvelope, source);
+                await safeUpdateObject(client.http, client.safety, objectUrl, body, SKTD_V2_CONTENT_TYPE, transport);
+                cachingLayer?.invalidate(type, name);
+                return textResult(`Successfully updated ${type} ${name}.`);
+            }
             if (isMetadataWriteType(type)) {
                 // Metadata updates are full-XML-replace — we must fetch existing metadata
                 // and merge with provided fields so omitted fields keep their current values.
@@ -1675,6 +1713,45 @@ async function handleSAPWrite(client, args, config, cachingLayer) {
             if (!affResult.valid) {
                 return errorResult(`AFF metadata validation failed for ${type} ${name}:\n- ${(affResult.errors ?? []).join('\n- ')}\n\nFix the metadata and retry.`);
             }
+            if (type === 'SKTD') {
+                // A KTD is not a standalone object — it documents a parent object (e.g., a DDLS view or a CLAS).
+                // The create POST goes to the collection URL with a sktd:docu XML body that references the parent.
+                const refType = String(args.refObjectType ?? '');
+                if (!refType) {
+                    return errorResult('"refObjectType" is required for SKTD create — the ADT type+subtype of the parent object being documented (e.g., "DDLS/DF", "CLAS/OC", "PROG/P", "INTF/OI", "BDEF/BDO", "SRVD/SRV").');
+                }
+                const refName = String(args.refObjectName ?? name);
+                // SAP rule: a KTD's own name must equal the parent object's name (one KTD per object).
+                // Creating a KTD named differently from its parent fails server-side with a cryptic
+                // "Check of condition failed" — fail fast with a clear message instead.
+                if (refName.toUpperCase() !== name.toUpperCase()) {
+                    return errorResult(`SKTD name "${name}" must match refObjectName "${refName}" — a Knowledge Transfer Document inherits the name of the ABAP object it documents (one KTD per object). To document "${refName}", call SAPWrite(action="create", type="SKTD", name="${refName}", refObjectType="${refType}", ...).`);
+                }
+                const refDescription = String(args.refObjectDescription ?? '');
+                // Build the parent URI. ADT URIs use lowercase names by convention (matches the Eclipse trace).
+                const refParentType = refType.split('/')[0] ?? '';
+                const refUri = `${objectBasePath(refParentType)}${encodeURIComponent(refName.toLowerCase())}`;
+                const ktdBody = `<?xml version="1.0" encoding="UTF-8"?>
+<sktd:docu xmlns:sktd="http://www.sap.com/wbobj/texts/sktd" xmlns:adtcore="http://www.sap.com/adt/core" adtcore:language="EN" adtcore:name="${escapeXml(name)}" adtcore:type="SKTD/TYP" adtcore:masterLanguage="EN">
+  <adtcore:packageRef adtcore:name="${escapeXml(pkg)}"/>
+  <sktd:refObject adtcore:description="${escapeXml(refDescription)}" adtcore:name="${escapeXml(refName)}" adtcore:type="${escapeXml(refType)}" adtcore:uri="${escapeXml(refUri)}"/>
+</sktd:docu>`;
+                const ktdCreateUrl = '/sap/bc/adt/documentation/ktd/documents';
+                const ktdResult = await createObject(client.http, client.safety, ktdCreateUrl, ktdBody, SKTD_V2_CONTENT_TYPE, effectiveTransport);
+                // If initial Markdown was provided, follow up with an update PUT to write it.
+                // Same envelope contract as the update path: fetch-then-rewrite ensures we
+                // PUT back exactly the shape SAP gave us (with all the server-assigned
+                // metadata), only swapping <sktd:text>.
+                if (source) {
+                    const currentEnvelope = await client.getKtd(name);
+                    const body = rewriteKtdText(currentEnvelope, source);
+                    await safeUpdateObject(client.http, client.safety, objectUrl, body, SKTD_V2_CONTENT_TYPE, effectiveTransport);
+                    cachingLayer?.invalidate(type, name);
+                    return textResult(`Created SKTD ${name} in package ${pkg} and wrote Markdown content.\nNext step: SAPActivate(type="SKTD", name="${name}").\n${ktdResult}`);
+                }
+                cachingLayer?.invalidate(type, name);
+                return textResult(`Created SKTD ${name} in package ${pkg} (no Markdown content written — pass "source" to write the body).\nNext step: SAPActivate(type="SKTD", name="${name}").\n${ktdResult}`);
+            }
             // Build type-specific creation XML body.
             // SAP ADT requires the root element to match the object type —
             // a generic objectReferences body returns 400 "System expected the element ...".
@@ -1785,7 +1862,7 @@ async function handleSAPWrite(client, args, config, cachingLayer) {
         }
         case 'delete': {
             await enforcePackageForExistingObject();
-            // Lock, delete, unlock pattern — auto-propagate lock corrNr if no explicit transport
+            // Lock, delete, unlock pattern (works for all types including SKTD) — auto-propagate lock corrNr if no explicit transport
             await client.http.withStatefulSession(async (session) => {
                 const lock = await lockObject(session, client.safety, objectUrl);
                 const effectiveTransport = transport ?? (lock.corrNr || undefined);
@@ -2713,6 +2790,72 @@ async function handleSAPManage(client, config, args, cachingLayer, isPerUserClie
             });
             return textResult(`Deleted package ${name}.`);
         }
+        case 'change_package': {
+            const objectName = String(args.objectName ?? '').trim();
+            const objectType = String(args.objectType ?? '').trim();
+            const oldPackage = String(args.oldPackage ?? '').trim();
+            const newPackage = String(args.newPackage ?? '').trim();
+            const transport = String(args.transport ?? '').trim();
+            let objectUri = String(args.objectUri ?? '').trim();
+            if (!objectName)
+                return errorResult('"objectName" is required for change_package action.');
+            if (!objectType)
+                return errorResult('"objectType" is required for change_package action.');
+            if (!oldPackage)
+                return errorResult('"oldPackage" is required for change_package action.');
+            if (!newPackage)
+                return errorResult('"newPackage" is required for change_package action.');
+            checkOperation(client.safety, OperationType.Update, 'ChangePackage');
+            checkPackage(client.safety, oldPackage);
+            checkPackage(client.safety, newPackage);
+            // Resolve object URI via search if not provided
+            if (!objectUri) {
+                const searchResp = await client.http.get(`/sap/bc/adt/repository/informationsystem/search?operation=quickSearch&query=${encodeURIComponent(objectName)}&maxResults=10`);
+                const uriMatch = searchResp.body.match(new RegExp(`adtcore:uri="([^"]*)"[^>]*adtcore:type="${objectType.replace('/', '\\/')}"`, 'i'));
+                if (!uriMatch?.[1]) {
+                    return errorResult(`Could not find object "${objectName}" with type "${objectType}" via ADT search. ` +
+                        `Verify the object exists and the type is correct (e.g., CLAS/OC, DDLS/DF, PROG/P).`);
+                }
+                objectUri = uriMatch[1];
+            }
+            // Transport pre-flight for non-local target packages
+            let effectiveTransport = transport || undefined;
+            if (!effectiveTransport && newPackage.toUpperCase() !== '$TMP') {
+                try {
+                    const transportInfo = await getTransportInfo(client.http, client.safety, objectUri, newPackage, 'I');
+                    if (transportInfo.lockedTransport) {
+                        effectiveTransport = transportInfo.lockedTransport;
+                    }
+                    else if (!transportInfo.isLocal && transportInfo.recording) {
+                        const existingList = transportInfo.existingTransports.length > 0
+                            ? `\n\nExisting transports for this package:\n${transportInfo.existingTransports
+                                .slice(0, 10)
+                                .map((t) => `  - ${t.id}: ${t.description} (${t.owner})`)
+                                .join('\n')}`
+                            : '';
+                        return errorResult(`Package "${newPackage}" requires a transport number for change_package, but none was provided.\n\n` +
+                            `To fix this, either:\n` +
+                            `1. Use SAPTransport(action="list") to find an existing modifiable transport\n` +
+                            `2. Use SAPTransport(action="create", description="...") to create a new one\n` +
+                            `3. Then retry SAPManage(action="change_package", ..., transport="<transport_id>")` +
+                            existingList);
+                    }
+                }
+                catch {
+                    // Graceful fallback: let SAP enforce transport requirements if the pre-check fails.
+                }
+            }
+            const result = await changePackage(client.http, client.safety, {
+                objectUri,
+                objectType,
+                objectName,
+                oldPackage,
+                newPackage,
+                transport: effectiveTransport,
+            });
+            const transportNote = result.transport ? ` (transport: ${result.transport})` : '';
+            return textResult(`Moved ${objectName} from package ${oldPackage} to ${newPackage}${transportNote}.`);
+        }
         case 'flp_list_catalogs': {
             const catalogs = await listCatalogs(client.http, client.safety);
             const customCount = catalogs.filter((c) => /^(Z|Y)/i.test(c.domainId)).length;