arc-1 0.9.13 → 0.9.15

package/dist/handlers/transport.js

@@ -0,0 +1,281 @@
+/**
+ * SAPTransport handler — CTS transport management (create, release, list, history, targets,
+ * layers).
+ */
+import { AdtApiError } from '../adt/errors.js';
+import { createTransport, createTransportWithTarget, deleteTransport, getObjectTransports, getTransport, getTransportInfo, listTransportLayers, listTransports, listTransportTargets, reassignTransport, releaseTransport, releaseTransportRecursive, supportsExplicitTransportTarget, } from '../adt/transport.js';
+import { objectUrlForType } from './object-types.js';
+import { errorResult, textResult } from './shared.js';
+// ─── SAPTransport Handler ────────────────────────────────────────────
+export async function handleSAPTransport(client, args) {
+    const action = String(args.action ?? '');
+    switch (action) {
+        case 'list': {
+            const user = args.user || client.username;
+            const status = args.status ?? 'D';
+            const transports = await listTransports(client.http, client.safety, user, status === '*' ? undefined : status);
+            return textResult(JSON.stringify(transports, null, 2));
+        }
+        case 'get': {
+            const id = String(args.id ?? '');
+            if (!id)
+                return errorResult('Transport ID is required for "get" action.');
+            const transport = await getTransport(client.http, client.safety, id);
+            if (!transport)
+                return textResult(`Transport ${id} not found.`);
+            return textResult(JSON.stringify(transport, null, 2));
+        }
+        case 'create': {
+            const description = String(args.description ?? '');
+            if (!description)
+                return errorResult('Description is required for "create" action.');
+            // Distinguish "target omitted" from "target present but empty": an explicit but
+            // blank target is a caller mistake, not a request to use the package/layer path.
+            const targetProvided = args.target !== undefined && args.target !== null;
+            const explicitTarget = targetProvided ? String(args.target).trim() : undefined;
+            if (targetProvided && !explicitTarget) {
+                return errorResult('"target" was provided but is empty. Pass a transport target — a system (C11), ' +
+                    'system.client (C11.021), or target group (/GROUP/) — or omit target to let SAP resolve it from the package.');
+            }
+            // Shared guidance when this release's ADT stack can't set an explicit target.
+            const targetUnsupportedMsg = "This system's ADT stack does not support setting an explicit transport target — the tm:root/newrequest " +
+                'action is not implemented (an ADT-framework limitation on SAP_BASIS 7.50, verified on SP02 and SP32, ' +
+                'independent of STMS/CTC config). It works on newer ABAP Platform / S/4HANA releases. Workaround here: ' +
+                'create the request without "target", then set the Transportziel manually in SE09/SE10 (SAP GUI), which ' +
+                'works when CTC and the target group are configured.';
+            let id;
+            if (explicitTarget) {
+                // Discovery-gate first — the same capability SAP's own Eclipse client checks: the TM
+                // resource that sets an explicit target is advertised in ADT discovery only on releases
+                // that implement it (the transportorganizer Accept type on cts/transportrequests). When
+                // discovery is loaded and the capability is absent (NW 7.50/7.51), fail fast with guidance
+                // rather than POST a request the backend rejects with "user action is not supported".
+                if (supportsExplicitTransportTarget(client.http) === false) {
+                    return errorResult(targetUnsupportedMsg);
+                }
+                // Explicit transport target (Transportziel / TR_TARGET): a system (C11),
+                // system.client (C11.021), or target group (/TRG/). Routed via the tm:root/
+                // newrequest endpoint — the only ADT path that sets the target directly. The
+                // group and system.client forms require extended transport control (CTC).
+                try {
+                    id = await createTransportWithTarget(client.http, client.safety, description, explicitTarget, client.username);
+                }
+                catch (err) {
+                    if (err instanceof AdtApiError && (err.statusCode === 400 || err.statusCode === 404)) {
+                        // SAP validates the target server-side: a4h (7.58) returns 400 "Target 'X'
+                        // does not exist"; other releases may use 404.
+                        if (/does not exist/i.test(err.message)) {
+                            return errorResult(`Transport target "${explicitTarget}" does not exist on this system. Valid targets are a ` +
+                                'system (e.g. C11), system.client (C11.021), or a target group (/GROUP/) — the group and ' +
+                                'system.client forms require extended transport control (CTC) to be active. Use ' +
+                                'SAPTransport(action="targets") to list the targets this system actually offers.');
+                        }
+                        // Fallback for when discovery was not loaded: NW 7.50/7.51 reject tm:root/newrequest
+                        // with "user action is not supported" (the gate above pre-empts this when discovery is known).
+                        if (/user action/i.test(err.message) && /not supported/i.test(err.message)) {
+                            return errorResult(targetUnsupportedMsg);
+                        }
+                    }
+                    throw err;
+                }
+            }
+            else {
+                const targetPackage = args.package ? String(args.package) : undefined;
+                const transportLayer = args.transportLayer ? String(args.transportLayer) : undefined;
+                id = await createTransport(client.http, client.safety, description, targetPackage, undefined, transportLayer);
+            }
+            if (!id)
+                return errorResult('Transport creation succeeded but no transport ID was returned. Check the SAP system manually.');
+            // Read the new request back (best-effort) to report its actual transport target.
+            // An empty target means the request is local ("Local Change Requests") — the #1
+            // source of "why does it always create a local transport?" confusion — so we
+            // surface it explicitly instead of just echoing the ID.
+            const created = await getTransport(client.http, client.safety, id).catch(() => null);
+            const target = created?.target?.trim() ?? '';
+            const targetDesc = created?.targetDesc?.trim() ?? '';
+            if (!created)
+                return textResult(`Created transport request: ${id}`);
+            if (target) {
+                return textResult(`Created transport request: ${id}\nTransport target: ${target}${targetDesc ? ` (${targetDesc})` : ''}`);
+            }
+            return textResult(`Created transport request: ${id}\n` +
+                `Transport target: <none>${targetDesc ? ` — "${targetDesc}"` : ''}. This is a LOCAL request — it cannot be transported onward.\n\n` +
+                'To create a request that targets another system, either:\n' +
+                '  • set an explicit target — pass target=<system | system.client | /group/> (e.g. target="/TRG/" or "C11"); ' +
+                'the group and system.client forms require extended transport control (CTC) to be active; or\n' +
+                '  • let SAP resolve it from the package transport layer + STMS consolidation route (pass transportLayer=<layer> to override the layer).\n' +
+                'Both require the SAP system to actually have transport routes/targets configured (a Basis task). ' +
+                'On a standalone system with no routes, every request is local — expected, and no ADT/Eclipse/SE10 client can change it.');
+        }
+        case 'layers': {
+            // Discovery: list valid values for create's `transportLayer`. Lets a client pick a
+            // real layer instead of guessing. Read-only (no allowTransportWrites required).
+            let layers;
+            try {
+                layers = await listTransportLayers(client.http, client.safety);
+            }
+            catch (err) {
+                // The package transport-layer value help is 7.52+; NW 7.50/7.51 return 404
+                // "No suitable resource found" (verified live on npl 7.50). Surface that clearly
+                // instead of a raw 404, so the caller knows discovery is unavailable on this release.
+                if (err instanceof AdtApiError && err.isNotFound) {
+                    return errorResult('Transport-layer discovery is not available on this SAP release — the value help ' +
+                        '(/sap/bc/adt/packages/valuehelps/transportlayers) returned 404 (typically NW < 7.52). ' +
+                        'Create requests without transportLayer; the route/target is governed by the package + STMS on these releases.');
+                }
+                throw err;
+            }
+            const routed = layers.filter((l) => l.target);
+            const summary = layers.length
+                ? routed.length
+                    ? `${layers.length} transport layer(s); ${routed.length} carry a target. Pass one as transportLayer= on create.`
+                    : `${layers.length} transport layer(s), but none expose a consolidation target — created requests will be local on this system.`
+                : 'No transport layers are defined on this system — every request will be local.';
+            return textResult(JSON.stringify({ transportLayers: layers, summary }, null, 2));
+        }
+        case 'targets': {
+            // Discovery: list valid values for create's `target` (Transportziel / TR_TARGET) via the
+            // official ADT target value help. Read-only (no allowTransportWrites required).
+            // Gate on the same capability as create's target path: NW 7.50 returns an empty list
+            // (HTTP 200) from the value help rather than 404, so the discovery accept type — not the
+            // HTTP status — is the reliable "is target discovery meaningful here?" signal.
+            if (supportsExplicitTransportTarget(client.http) === false) {
+                return errorResult("Transport-target discovery is not available on this SAP release — its ADT stack doesn't support " +
+                    'explicit transport targets (verified on NW 7.50). Set the target in SE09/SE10 instead.');
+            }
+            let targets;
+            try {
+                targets = await listTransportTargets(client.http, client.safety);
+            }
+            catch (err) {
+                // Fallback when discovery wasn't loaded: some releases 404 the value help.
+                if (err instanceof AdtApiError && err.isNotFound) {
+                    return errorResult('Transport-target discovery is not available on this SAP release — the value help ' +
+                        '(/sap/bc/adt/cts/transportrequests/valuehelp/target) returned 404. Set the target in SE09/SE10 instead.');
+                }
+                throw err;
+            }
+            const summary = targets.length
+                ? `${targets.length} valid transport target(s). Pass one as target= on create.`
+                : 'No transport targets are configured on this system (so created requests are local).';
+            return textResult(JSON.stringify({ transportTargets: targets, summary }, null, 2));
+        }
+        case 'release': {
+            const id = String(args.id ?? '');
+            if (!id)
+                return errorResult('Transport ID is required for "release" action.');
+            await releaseTransport(client.http, client.safety, id);
+            return textResult(`Released transport request: ${id}`);
+        }
+        case 'delete': {
+            const id = String(args.id ?? '');
+            if (!id)
+                return errorResult('Transport ID is required for "delete" action.');
+            const recursive = Boolean(args.recursive ?? false);
+            const removeLockedObjects = Boolean(args.removeLockedObjects ?? false);
+            try {
+                await deleteTransport(client.http, client.safety, id, recursive, removeLockedObjects);
+            }
+            catch (e) {
+                // ADT refuses to delete a request/task that still holds locked objects (e.g. a deleted
+                // object's lingering record). Point the caller at removeLockedObjects instead of a raw [?/009].
+                if (!removeLockedObjects && e instanceof Error && /locked objects/i.test(e.message)) {
+                    return errorResult(`${e.message}\n\nThe request still holds locked object(s). ` +
+                        `Retry with removeLockedObjects=true to strip them first:\n` +
+                        `  SAPTransport(action="delete", id="${id}", removeLockedObjects=true)`);
+                }
+                throw e;
+            }
+            const extras = [recursive ? 'recursive' : '', removeLockedObjects ? 'removed locked objects' : '']
+                .filter(Boolean)
+                .join(', ');
+            return textResult(`Deleted transport request: ${id}${extras ? ` (${extras})` : ''}`);
+        }
+        case 'reassign': {
+            const id = String(args.id ?? '');
+            if (!id)
+                return errorResult('Transport ID is required for "reassign" action.');
+            const owner = String(args.owner ?? '');
+            if (!owner)
+                return errorResult('Owner is required for "reassign" action.');
+            const recursive = Boolean(args.recursive ?? false);
+            await reassignTransport(client.http, client.safety, id, owner, recursive);
+            return textResult(`Reassigned transport ${id} to ${owner}${recursive ? ' (recursive)' : ''}`);
+        }
+        case 'release_recursive': {
+            const id = String(args.id ?? '');
+            if (!id)
+                return errorResult('Transport ID is required for "release_recursive" action.');
+            const result = await releaseTransportRecursive(client.http, client.safety, id);
+            return textResult(JSON.stringify(result, null, 2));
+        }
+        case 'check': {
+            // Check transport requirements for an object/package combination.
+            // Does NOT require allowTransportWrites — this is a read-only check.
+            const objectType = String(args.type ?? '');
+            const objectName = String(args.name ?? '');
+            const pkg = String(args.package ?? '');
+            if (!objectType || !objectName)
+                return errorResult('"type" and "name" are required for "check" action.');
+            if (!pkg)
+                return errorResult('"package" is required for "check" action.');
+            const objectUrl = objectUrlForType(objectType, objectName);
+            const info = await getTransportInfo(client.http, client.safety, objectUrl, pkg, 'I');
+            const summary = info.isLocal
+                ? `Package "${pkg}" is local — no transport required.`
+                : info.recording
+                    ? `Package "${pkg}" requires a transport for object creation.`
+                    : `Package "${pkg}" does not require transport recording.`;
+            return textResult(JSON.stringify({
+                package: pkg,
+                transportRequired: !info.isLocal && info.recording,
+                isLocal: info.isLocal,
+                deliveryUnit: info.deliveryUnit,
+                existingTransports: info.existingTransports,
+                ...(info.lockedTransport ? { lockedTransport: info.lockedTransport } : {}),
+                summary,
+            }, null, 2));
+        }
+        case 'history': {
+            const objectType = String(args.type ?? '');
+            const objectName = String(args.name ?? '');
+            if (!objectType || !objectName) {
+                return errorResult('"type" and "name" are required for "history" action.');
+            }
+            const objectUrl = objectUrlForType(objectType, objectName);
+            const primary = await getObjectTransports(client.http, client.safety, objectUrl);
+            let candidateTransports = primary.candidateTransports;
+            // Fallback: if per-object transport lookup is empty, derive the package via
+            // the object metadata endpoint and ask transportchecks for candidate transports.
+            if (primary.relatedTransports.length === 0 && candidateTransports.length === 0) {
+                try {
+                    const pkg = await client.resolveObjectPackage(objectUrl);
+                    if (pkg && pkg !== '$TMP') {
+                        const info = await getTransportInfo(client.http, client.safety, objectUrl, pkg, '');
+                        candidateTransports = info.existingTransports;
+                    }
+                }
+                catch {
+                    // best-effort-fallback
+                }
+            }
+            const lockOwner = primary.relatedTransports[0]?.owner;
+            const summary = primary.lockedTransport
+                ? `Object ${objectName} is locked in transport ${primary.lockedTransport}${lockOwner ? ` by ${lockOwner}` : ''}.`
+                : candidateTransports.length > 0
+                    ? `Object ${objectName} has no active lock; ${candidateTransports.length} transport(s) available for assignment.`
+                    : `Object ${objectName} has no related or candidate transports (likely $TMP / local object).`;
+            const history = {
+                object: { type: objectType, name: objectName, uri: objectUrl },
+                ...(primary.lockedTransport ? { lockedTransport: primary.lockedTransport } : {}),
+                relatedTransports: primary.relatedTransports,
+                candidateTransports,
+                summary,
+            };
+            return textResult(JSON.stringify(history, null, 2));
+        }
+        default:
+            return errorResult(`Unknown SAPTransport action: ${action}. Supported: list, get, create, release, delete, reassign, release_recursive, check, history`);
+    }
+}
+//# sourceMappingURL=transport.js.map

package/dist/handlers/transport.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"transport.js","sourceRoot":"","sources":["../../src/handlers/transport.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAGH,OAAO,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAC;AAC/C,OAAO,EACL,eAAe,EACf,yBAAyB,EACzB,eAAe,EACf,mBAAmB,EACnB,YAAY,EACZ,gBAAgB,EAChB,mBAAmB,EACnB,cAAc,EACd,oBAAoB,EACpB,iBAAiB,EACjB,gBAAgB,EAChB,yBAAyB,EACzB,+BAA+B,GAChC,MAAM,qBAAqB,CAAC;AAE7B,OAAO,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAC;AACrD,OAAO,EAAE,WAAW,EAAmB,UAAU,EAAE,MAAM,aAAa,CAAC;AAEvE,wEAAwE;AAExE,MAAM,CAAC,KAAK,UAAU,kBAAkB,CAAC,MAAiB,EAAE,IAA6B;IACvF,MAAM,MAAM,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,IAAI,EAAE,CAAC,CAAC;IAEzC,QAAQ,MAAM,EAAE,CAAC;QACf,KAAK,MAAM,CAAC,CAAC,CAAC;YACZ,MAAM,IAAI,GAAI,IAAI,CAAC,IAA2B,IAAI,MAAM,CAAC,QAAQ,CAAC;YAClE,MAAM,MAAM,GAAI,IAAI,CAAC,MAA6B,IAAI,GAAG,CAAC;YAC1D,MAAM,UAAU,GAAG,MAAM,cAAc,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,IAAI,EAAE,MAAM,KAAK,GAAG,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;YAC/G,OAAO,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,UAAU,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QACzD,CAAC;QACD,KAAK,KAAK,CAAC,CAAC,CAAC;YACX,MAAM,EAAE,GAAG,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;YACjC,IAAI,CAAC,EAAE;gBAAE,OAAO,WAAW,CAAC,4CAA4C,CAAC,CAAC;YAC1E,MAAM,SAAS,GAAG,MAAM,YAAY,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;YACrE,IAAI,CAAC,SAAS;gBAAE,OAAO,UAAU,CAAC,aAAa,EAAE,aAAa,CAAC,CAAC;YAChE,OAAO,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QACxD,CAAC;QACD,KAAK,QAAQ,CAAC,CAAC,CAAC;YACd,MAAM,WAAW,GAAG,MAAM,CAAC,IAAI,CAAC,WAAW,IAAI,EAAE,CAAC,CAAC;YACnD,IAAI,CAAC,WAAW;gBAAE,OAAO,WAAW,CAAC,8CAA8C,CAAC,CAAC;YACrF,gFAAgF;YAChF,iFAAiF;YACjF,MAAM,cAAc,GAAG,IAAI,CAAC,MAAM,KAAK,SAAS,IAAI,IAAI,CAAC,MAAM,KAAK,IAAI,CAAC;YACzE,MAAM,cAAc,GAAG,cAAc,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC;YAC/E,IAAI,cAAc,IAAI,CAAC,cAAc,EAAE,CAAC;gBACtC,OAAO,WAAW,CAChB,gFAAgF;oBAC9E,6GAA6G,CAChH,CAAC;YACJ,CAAC;YAED,8EAA8E;YAC9E,MAAM,oBAAoB,GACxB,yGAAyG;gBACzG,uGAAuG;gBACvG,wGAAwG;gBACxG,yGAAyG;gBACzG,qDAAqD,CAAC;YAExD,IAAI,EAAU,CAAC;YACf,IAAI,cAAc,EAAE,CAAC;gBACnB,qFAAqF;gBACrF,wFAAwF;gBACxF,wFAAwF;gBACxF,2FAA2F;gBAC3F,sFAAsF;gBACtF,IAAI,+BAA+B,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,KAAK,EAAE,CAAC;oBAC3D,OAAO,WAAW,CAAC,oBAAoB,CAAC,CAAC;gBAC3C,CAAC;gBACD,yEAAyE;gBACzE,4EAA4E;gBAC5E,6EAA6E;gBAC7E,0EAA0E;gBAC1E,IAAI,CAAC;oBACH,EAAE,GAAG,MAAM,yBAAyB,CAClC,MAAM,CAAC,IAAI,EACX,MAAM,CAAC,MAAM,EACb,WAAW,EACX,cAAc,EACd,MAAM,CAAC,QAAQ,CAChB,CAAC;gBACJ,CAAC;gBAAC,OAAO,GAAG,EAAE,CAAC;oBACb,IAAI,GAAG,YAAY,WAAW,IAAI,CAAC,GAAG,CAAC,UAAU,KAAK,GAAG,IAAI,GAAG,CAAC,UAAU,KAAK,GAAG,CAAC,EAAE,CAAC;wBACrF,2EAA2E;wBAC3E,+CAA+C;wBAC/C,IAAI,iBAAiB,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;4BACxC,OAAO,WAAW,CAChB,qBAAqB,cAAc,uDAAuD;gCACxF,0FAA0F;gCAC1F,iFAAiF;gCACjF,iFAAiF,CACpF,CAAC;wBACJ,CAAC;wBACD,qFAAqF;wBACrF,+FAA+F;wBAC/F,IAAI,cAAc,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,gBAAgB,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;4BAC3E,OAAO,WAAW,CAAC,oBAAoB,CAAC,CAAC;wBAC3C,CAAC;oBACH,CAAC;oBACD,MAAM,GAAG,CAAC;gBACZ,CAAC;YACH,CAAC;iBAAM,CAAC;gBACN,MAAM,aAAa,GAAG,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;gBACtE,MAAM,cAAc,GAAG,IAAI,CAAC,cAAc,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;gBACrF,EAAE,GAAG,MAAM,eAAe,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,WAAW,EAAE,aAAa,EAAE,SAAS,EAAE,cAAc,CAAC,CAAC;YAChH,CAAC;YAED,IAAI,CAAC,EAAE;gBACL,OAAO,WAAW,CAChB,+FAA+F,CAChG,CAAC;YAEJ,iFAAiF;YACjF,gFAAgF;YAChF,6EAA6E;YAC7E,wDAAwD;YACxD,MAAM,OAAO,GAAG,MAAM,YAAY,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;YACrF,MAAM,MAAM,GAAG,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC;YAC7C,MAAM,UAAU,GAAG,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC;YAErD,IAAI,CAAC,OAAO;gBAAE,OAAO,UAAU,CAAC,8BAA8B,EAAE,EAAE,CAAC,CAAC;YACpE,IAAI,MAAM,EAAE,CAAC;gBACX,OAAO,UAAU,CACf,8BAA8B,EAAE,uBAAuB,MAAM,GAAG,UAAU,CAAC,CAAC,CAAC,KAAK,UAAU,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACvG,CAAC;YACJ,CAAC;YACD,OAAO,UAAU,CACf,8BAA8B,EAAE,IAAI;gBAClC,2BAA2B,UAAU,CAAC,CAAC,CAAC,OAAO,UAAU,GAAG,CAAC,CAAC,CAAC,EAAE,kEAAkE;gBACnI,4DAA4D;gBAC5D,8GAA8G;gBAC9G,+FAA+F;gBAC/F,2IAA2I;gBAC3I,mGAAmG;gBACnG,yHAAyH,CAC5H,CAAC;QACJ,CAAC;QACD,KAAK,QAAQ,CAAC,CAAC,CAAC;YACd,mFAAmF;YACnF,gFAAgF;YAChF,IAAI,MAAuD,CAAC;YAC5D,IAAI,CAAC;gBACH,MAAM,GAAG,MAAM,mBAAmB,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,CAAC,CAAC;YACjE,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,2EAA2E;gBAC3E,iFAAiF;gBACjF,sFAAsF;gBACtF,IAAI,GAAG,YAAY,WAAW,IAAI,GAAG,CAAC,UAAU,EAAE,CAAC;oBACjD,OAAO,WAAW,CAChB,kFAAkF;wBAChF,wFAAwF;wBACxF,+GAA+G,CAClH,CAAC;gBACJ,CAAC;gBACD,MAAM,GAAG,CAAC;YACZ,CAAC;YACD,MAAM,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;YAC9C,MAAM,OAAO,GAAG,MAAM,CAAC,MAAM;gBAC3B,CAAC,CAAC,MAAM,CAAC,MAAM;oBACb,CAAC,CAAC,GAAG,MAAM,CAAC,MAAM,wBAAwB,MAAM,CAAC,MAAM,yDAAyD;oBAChH,CAAC,CAAC,GAAG,MAAM,CAAC,MAAM,8GAA8G;gBAClI,CAAC,CAAC,+EAA+E,CAAC;YACpF,OAAO,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,EAAE,eAAe,EAAE,MAAM,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QACnF,CAAC;QACD,KAAK,SAAS,CAAC,CAAC,CAAC;YACf,yFAAyF;YACzF,gFAAgF;YAChF,qFAAqF;YACrF,yFAAyF;YACzF,+EAA+E;YAC/E,IAAI,+BAA+B,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,KAAK,EAAE,CAAC;gBAC3D,OAAO,WAAW,CAChB,kGAAkG;oBAChG,wFAAwF,CAC3F,CAAC;YACJ,CAAC;YACD,IAAI,OAAyD,CAAC;YAC9D,IAAI,CAAC;gBACH,OAAO,GAAG,MAAM,oBAAoB,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,CAAC,CAAC;YACnE,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,2EAA2E;gBAC3E,IAAI,GAAG,YAAY,WAAW,IAAI,GAAG,CAAC,UAAU,EAAE,CAAC;oBACjD,OAAO,WAAW,CAChB,mFAAmF;wBACjF,yGAAyG,CAC5G,CAAC;gBACJ,CAAC;gBACD,MAAM,GAAG,CAAC;YACZ,CAAC;YACD,MAAM,OAAO,GAAG,OAAO,CAAC,MAAM;gBAC5B,CAAC,CAAC,GAAG,OAAO,CAAC,MAAM,4DAA4D;gBAC/E,CAAC,CAAC,qFAAqF,CAAC;YAC1F,OAAO,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,EAAE,gBAAgB,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QACrF,CAAC;QACD,KAAK,SAAS,CAAC,CAAC,CAAC;YACf,MAAM,EAAE,GAAG,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;YACjC,IAAI,CAAC,EAAE;gBAAE,OAAO,WAAW,CAAC,gDAAgD,CAAC,CAAC;YAC9E,MAAM,gBAAgB,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;YACvD,OAAO,UAAU,CAAC,+BAA+B,EAAE,EAAE,CAAC,CAAC;QACzD,CAAC;QACD,KAAK,QAAQ,CAAC,CAAC,CAAC;YACd,MAAM,EAAE,GAAG,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;YACjC,IAAI,CAAC,EAAE;gBAAE,OAAO,WAAW,CAAC,+CAA+C,CAAC,CAAC;YAC7E,MAAM,SAAS,GAAG,OAAO,CAAC,IAAI,CAAC,SAAS,IAAI,KAAK,CAAC,CAAC;YACnD,MAAM,mBAAmB,GAAG,OAAO,CAAC,IAAI,CAAC,mBAAmB,IAAI,KAAK,CAAC,CAAC;YACvE,IAAI,CAAC;gBACH,MAAM,eAAe,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,EAAE,EAAE,SAAS,EAAE,mBAAmB,CAAC,CAAC;YACxF,CAAC;YAAC,OAAO,CAAC,EAAE,CAAC;gBACX,uFAAuF;gBACvF,gGAAgG;gBAChG,IAAI,CAAC,mBAAmB,IAAI,CAAC,YAAY,KAAK,IAAI,iBAAiB,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC;oBACpF,OAAO,WAAW,CAChB,GAAG,CAAC,CAAC,OAAO,gDAAgD;wBAC1D,4DAA4D;wBAC5D,uCAAuC,EAAE,8BAA8B,CAC1E,CAAC;gBACJ,CAAC;gBACD,MAAM,CAAC,CAAC;YACV,CAAC;YACD,MAAM,MAAM,GAAG,CAAC,SAAS,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,EAAE,EAAE,mBAAmB,CAAC,CAAC,CAAC,wBAAwB,CAAC,CAAC,CAAC,EAAE,CAAC;iBAC/F,MAAM,CAAC,OAAO,CAAC;iBACf,IAAI,CAAC,IAAI,CAAC,CAAC;YACd,OAAO,UAAU,CAAC,8BAA8B,EAAE,GAAG,MAAM,CAAC,CAAC,CAAC,KAAK,MAAM,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QACvF,CAAC;QACD,KAAK,UAAU,CAAC,CAAC,CAAC;YAChB,MAAM,EAAE,GAAG,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;YACjC,IAAI,CAAC,EAAE;gBAAE,OAAO,WAAW,CAAC,iDAAiD,CAAC,CAAC;YAC/E,MAAM,KAAK,GAAG,MAAM,CAAC,IAAI,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC;YACvC,IAAI,CAAC,KAAK;gBAAE,OAAO,WAAW,CAAC,0CAA0C,CAAC,CAAC;YAC3E,MAAM,SAAS,GAAG,OAAO,CAAC,IAAI,CAAC,SAAS,IAAI,KAAK,CAAC,CAAC;YACnD,MAAM,iBAAiB,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,EAAE,EAAE,KAAK,EAAE,SAAS,CAAC,CAAC;YAC1E,OAAO,UAAU,CAAC,wBAAwB,EAAE,OAAO,KAAK,GAAG,SAAS,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QAChG,CAAC;QACD,KAAK,mBAAmB,CAAC,CAAC,CAAC;YACzB,MAAM,EAAE,GAAG,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;YACjC,IAAI,CAAC,EAAE;gBAAE,OAAO,WAAW,CAAC,0DAA0D,CAAC,CAAC;YACxF,MAAM,MAAM,GAAG,MAAM,yBAAyB,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;YAC/E,OAAO,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QACrD,CAAC;QACD,KAAK,OAAO,CAAC,CAAC,CAAC;YACb,kEAAkE;YAClE,qEAAqE;YACrE,MAAM,UAAU,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC;YAC3C,MAAM,UAAU,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC;YAC3C,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC;YACvC,IAAI,CAAC,UAAU,IAAI,CAAC,UAAU;gBAAE,OAAO,WAAW,CAAC,oDAAoD,CAAC,CAAC;YACzG,IAAI,CAAC,GAAG;gBAAE,OAAO,WAAW,CAAC,2CAA2C,CAAC,CAAC;YAE1E,MAAM,SAAS,GAAG,gBAAgB,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;YAC3D,MAAM,IAAI,GAAG,MAAM,gBAAgB,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC;YAErF,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO;gBAC1B,CAAC,CAAC,YAAY,GAAG,qCAAqC;gBACtD,CAAC,CAAC,IAAI,CAAC,SAAS;oBACd,CAAC,CAAC,YAAY,GAAG,6CAA6C;oBAC9D,CAAC,CAAC,YAAY,GAAG,yCAAyC,CAAC;YAE/D,OAAO,UAAU,CACf,IAAI,CAAC,SAAS,CACZ;gBACE,OAAO,EAAE,GAAG;gBACZ,iBAAiB,EAAE,CAAC,IAAI,CAAC,OAAO,IAAI,IAAI,CAAC,SAAS;gBAClD,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,YAAY,EAAE,IAAI,CAAC,YAAY;gBAC/B,kBAAkB,EAAE,IAAI,CAAC,kBAAkB;gBAC3C,GAAG,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,CAAC,EAAE,eAAe,EAAE,IAAI,CAAC,eAAe,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;gBAC1E,OAAO;aACR,EACD,IAAI,EACJ,CAAC,CACF,CACF,CAAC;QACJ,CAAC;QACD,KAAK,SAAS,CAAC,CAAC,CAAC;YACf,MAAM,UAAU,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC;YAC3C,MAAM,UAAU,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC;YAC3C,IAAI,CAAC,UAAU,IAAI,CAAC,UAAU,EAAE,CAAC;gBAC/B,OAAO,WAAW,CAAC,sDAAsD,CAAC,CAAC;YAC7E,CAAC;YAED,MAAM,SAAS,GAAG,gBAAgB,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;YAC3D,MAAM,OAAO,GAAG,MAAM,mBAAmB,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;YACjF,IAAI,mBAAmB,GAAG,OAAO,CAAC,mBAAmB,CAAC;YAEtD,4EAA4E;YAC5E,iFAAiF;YACjF,IAAI,OAAO,CAAC,iBAAiB,CAAC,MAAM,KAAK,CAAC,IAAI,mBAAmB,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBAC/E,IAAI,CAAC;oBACH,MAAM,GAAG,GAAG,MAAM,MAAM,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAC;oBACzD,IAAI,GAAG,IAAI,GAAG,KAAK,MAAM,EAAE,CAAC;wBAC1B,MAAM,IAAI,GAAG,MAAM,gBAAgB,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC;wBACpF,mBAAmB,GAAG,IAAI,CAAC,kBAAkB,CAAC;oBAChD,CAAC;gBACH,CAAC;gBAAC,MAAM,CAAC;oBACP,uBAAuB;gBACzB,CAAC;YACH,CAAC;YAED,MAAM,SAAS,GAAG,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC;YACtD,MAAM,OAAO,GAAG,OAAO,CAAC,eAAe;gBACrC,CAAC,CAAC,UAAU,UAAU,2BAA2B,OAAO,CAAC,eAAe,GAAG,SAAS,CAAC,CAAC,CAAC,OAAO,SAAS,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG;gBACjH,CAAC,CAAC,mBAAmB,CAAC,MAAM,GAAG,CAAC;oBAC9B,CAAC,CAAC,UAAU,UAAU,wBAAwB,mBAAmB,CAAC,MAAM,yCAAyC;oBACjH,CAAC,CAAC,UAAU,UAAU,uEAAuE,CAAC;YAElG,MAAM,OAAO,GAA2B;gBACtC,MAAM,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,IAAI,EAAE,UAAU,EAAE,GAAG,EAAE,SAAS,EAAE;gBAC9D,GAAG,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,CAAC,EAAE,eAAe,EAAE,OAAO,CAAC,eAAe,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;gBAChF,iBAAiB,EAAE,OAAO,CAAC,iBAAiB;gBAC5C,mBAAmB;gBACnB,OAAO;aACR,CAAC;YAEF,OAAO,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;QACtD,CAAC;QACD;YACE,OAAO,WAAW,CAChB,gCAAgC,MAAM,8FAA8F,CACrI,CAAC;IACN,CAAC;AACH,CAAC"}

package/dist/handlers/write/class-surgery.d.ts

@@ -0,0 +1,18 @@
+/**
+ * SAPWrite actions — class-section surgery (issue #303).
+ *
+ * These actions share a common shape: fetch objectstructure → optional diff/refuse → splice into
+ * /source/main (or /includes/<inc> when include= is set) → PUT under lock → no auto-activate.
+ * Pre-write lint runs on the SPLICED FULL source (not the partial input fragment) because a raw
+ * DEFINITION block alone fails abaplint with "Expected CLASSIMPLEMENTATION" (verified live on a4h);
+ * lint is skipped for include= writes (same precedent as the `update include=` path).
+ */
+import { type ToolResult } from '../shared.js';
+import type { SapWriteContext } from './context.js';
+export declare function writeActionEditMethod(ctx: SapWriteContext): Promise<ToolResult>;
+export declare function writeActionEditClassDefinition(ctx: SapWriteContext): Promise<ToolResult>;
+export declare function writeActionEditMethodSignature(ctx: SapWriteContext): Promise<ToolResult>;
+export declare function writeActionAddMethod(ctx: SapWriteContext): Promise<ToolResult>;
+export declare function writeActionDeleteMethod(ctx: SapWriteContext): Promise<ToolResult>;
+export declare function writeActionChangeMethodVisibility(ctx: SapWriteContext): Promise<ToolResult>;
+//# sourceMappingURL=class-surgery.d.ts.map

package/dist/handlers/write/class-surgery.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"class-surgery.d.ts","sourceRoot":"","sources":["../../../src/handlers/write/class-surgery.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAwBH,OAAO,EAAe,KAAK,UAAU,EAAc,MAAM,cAAc,CAAC;AAExE,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAEpD,wBAAsB,qBAAqB,CAAC,GAAG,EAAE,eAAe,GAAG,OAAO,CAAC,UAAU,CAAC,CAoIrF;AAED,wBAAsB,8BAA8B,CAAC,GAAG,EAAE,eAAe,GAAG,OAAO,CAAC,UAAU,CAAC,CAuG9F;AAED,wBAAsB,8BAA8B,CAAC,GAAG,EAAE,eAAe,GAAG,OAAO,CAAC,UAAU,CAAC,CAkE9F;AAED,wBAAsB,oBAAoB,CAAC,GAAG,EAAE,eAAe,GAAG,OAAO,CAAC,UAAU,CAAC,CAiGpF;AAED,wBAAsB,uBAAuB,CAAC,GAAG,EAAE,eAAe,GAAG,OAAO,CAAC,UAAU,CAAC,CA6DvF;AAED,wBAAsB,iCAAiC,CAAC,GAAG,EAAE,eAAe,GAAG,OAAO,CAAC,UAAU,CAAC,CAsFjG"}