@mcp-abap-adt/adt-backup 0.1.2 → 1.0.0

package/README.md

@@ -84,8 +84,10 @@ See `docs/roadmap.yaml` for per-object backup/restore status and the plan for re
 | `behaviorDefinition` | implemented | implemented | source |
 | `behaviorImplementation` | implemented | implemented | source |
 | `enhancement` | implemented | implemented | source |
-| `unitTest` | not-implemented | not-implemented | n/a |
-| `cdsUnitTest` | not-implemented | not-implemented | n/a |
+| `unitTest` | implemented | implemented | as class |
+| `cdsUnitTest` | implemented | implemented | as class |
+
+> **Note**: Unit tests are stored as classes in backups. When restoring, they are created as test classes in the system.
 
 ## Smoke Checklist
 

package/dist/lib/auth/createTokenProvider.d.ts

@@ -1,4 +1,4 @@
 import type { IAuthorizationConfig, ITokenProvider } from '@mcp-abap-adt/interfaces';
 import type { createLogger } from '../cli/createLogger';
-export declare function createTokenProvider(authConfig?: IAuthorizationConfig | null, logger?: ReturnType<typeof createLogger>): ITokenProvider;
+export declare function createTokenProvider(authConfig?: IAuthorizationConfig | null, browserAuthPort?: number, logger?: ReturnType<typeof createLogger>): ITokenProvider;
 //# sourceMappingURL=createTokenProvider.d.ts.map

package/dist/lib/auth/createTokenProvider.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"createTokenProvider.d.ts","sourceRoot":"","sources":["../../../src/lib/auth/createTokenProvider.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,oBAAoB,EACpB,cAAc,EACf,MAAM,0BAA0B,CAAC;AAElC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAC;AAGxD,wBAAgB,mBAAmB,CACjC,UAAU,CAAC,EAAE,oBAAoB,GAAG,IAAI,EACxC,MAAM,CAAC,EAAE,UAAU,CAAC,OAAO,YAAY,CAAC,GACvC,cAAc,CAkBhB"}
+{"version":3,"file":"createTokenProvider.d.ts","sourceRoot":"","sources":["../../../src/lib/auth/createTokenProvider.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,oBAAoB,EACpB,cAAc,EACf,MAAM,0BAA0B,CAAC;AAElC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAC;AAGxD,wBAAgB,mBAAmB,CACjC,UAAU,CAAC,EAAE,oBAAoB,GAAG,IAAI,EACxC,eAAe,CAAC,EAAE,MAAM,EACxB,MAAM,CAAC,EAAE,UAAU,CAAC,OAAO,YAAY,CAAC,GACvC,cAAc,CAmBhB"}

package/dist/lib/auth/createTokenProvider.js

@@ -4,7 +4,7 @@ exports.createTokenProvider = createTokenProvider;
 const auth_providers_1 = require("@mcp-abap-adt/auth-providers");
 const NoopTokenProvider_1 = require("../auth/NoopTokenProvider");
 const shouldEnableProviderLogger_1 = require("../cli/shouldEnableProviderLogger");
-function createTokenProvider(authConfig, logger) {
+function createTokenProvider(authConfig, browserAuthPort, logger) {
     if (!authConfig ||
         !authConfig.uaaUrl ||
         !authConfig.uaaClientId ||
@@ -17,6 +17,7 @@ function createTokenProvider(authConfig, logger) {
         clientSecret: authConfig.uaaClientSecret,
         refreshToken: authConfig.refreshToken,
         browser: 'system',
+        redirectPort: browserAuthPort || 10001,
         logger: (0, shouldEnableProviderLogger_1.shouldEnableProviderLogger)() ? logger : undefined,
     });
 }

package/dist/lib/auth/getSapConfigFromBroker.d.ts

@@ -5,6 +5,7 @@ export declare function getSapConfigFromBroker(options: {
     destination?: string;
     envPath?: string;
     authRoot?: string;
+    browserAuthPort?: number;
     logger: ReturnType<typeof createLogger>;
 }): Promise<{
     config: SapConfig;

package/dist/lib/auth/getSapConfigFromBroker.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"getSapConfigFromBroker.d.ts","sourceRoot":"","sources":["../../../src/lib/auth/getSapConfigFromBroker.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,0BAA0B,CAAC;AAC1D,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAC;AAChE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAC;AAKxD,wBAAsB,sBAAsB,CAAC,OAAO,EAAE;IACpD,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,UAAU,CAAC,OAAO,YAAY,CAAC,CAAC;CACzC,GAAG,OAAO,CAAC;IAAE,MAAM,EAAE,SAAS,CAAC;IAAC,cAAc,CAAC,EAAE,eAAe,CAAA;CAAE,CAAC,CA4BnE"}
+{"version":3,"file":"getSapConfigFromBroker.d.ts","sourceRoot":"","sources":["../../../src/lib/auth/getSapConfigFromBroker.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,0BAA0B,CAAC;AAC1D,OAAO,KAAK,EAGV,eAAe,EAChB,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAC;AAMxD,wBAAsB,sBAAsB,CAAC,OAAO,EAAE;IACpD,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,MAAM,EAAE,UAAU,CAAC,OAAO,YAAY,CAAC,CAAC;CACzC,GAAG,OAAO,CAAC;IAAE,MAAM,EAAE,SAAS,CAAC;IAAC,cAAc,CAAC,EAAE,eAAe,CAAA;CAAE,CAAC,CA0HnE"}

package/dist/lib/auth/getSapConfigFromBroker.js

@@ -39,6 +39,7 @@ const os = __importStar(require("node:os"));
 const path = __importStar(require("node:path"));
 const auth_broker_1 = require("@mcp-abap-adt/auth-broker");
 const auth_stores_1 = require("@mcp-abap-adt/auth-stores");
+const logVerbose_1 = require("../cli/logVerbose");
 const shouldEnableBrokerLogger_1 = require("../cli/shouldEnableBrokerLogger");
 const shouldEnableStoreLogger_1 = require("../cli/shouldEnableStoreLogger");
 const createTokenProvider_1 = require("./createTokenProvider");
@@ -46,28 +47,105 @@ async function getSapConfigFromBroker(options) {
     const { logger } = options;
     const brokerLogger = (0, shouldEnableBrokerLogger_1.shouldEnableBrokerLogger)() ? logger : undefined;
     const storeLogger = (0, shouldEnableStoreLogger_1.shouldEnableStoreLogger)() ? logger : undefined;
-    const destination = options.destination || 'env';
-    if (options.envPath) {
-        const sessionStore = new auth_stores_1.EnvFileSessionStore(options.envPath, storeLogger);
-        return getConfigWithStores({
-            destination,
-            sessionStore,
-            serviceKeyStore: undefined,
-            logger,
-            brokerLogger,
-        });
+    // 1. If --env-path is provided, we load it into process.env first
+    if (options.envPath && fs.existsSync(options.envPath)) {
+        const content = fs.readFileSync(options.envPath, 'utf8');
+        const envVars = parseEnvContent(content);
+        for (const [key, value] of Object.entries(envVars)) {
+            process.env[key] = value;
+        }
     }
+    const destination = options.destination || 'env';
     const roots = resolveAuthRoots(options.authRoot);
     const { sessionDir, serviceKeyDir } = resolveStoreDirs(roots, destination);
-    const sessionStore = new auth_stores_1.AbapSessionStore(sessionDir, storeLogger);
+    // 2. Initialize stores
+    const sessionStore = options.envPath
+        ? new auth_stores_1.EnvFileSessionStore(options.envPath, storeLogger)
+        : new auth_stores_1.AbapSessionStore(sessionDir, storeLogger);
     const serviceKeyStore = new auth_stores_1.AbapServiceKeyStore(serviceKeyDir, storeLogger);
-    return getConfigWithStores({
-        destination,
+    // 3. Get authorization config from store
+    const sessionAuthConfig = (await sessionStore.getAuthorizationConfig(destination));
+    const serviceKeyAuthConfig = (await serviceKeyStore.getAuthorizationConfig(destination));
+    const authConfig = sessionAuthConfig || serviceKeyAuthConfig;
+    const broker = new auth_broker_1.AuthBroker({
         sessionStore,
         serviceKeyStore,
-        logger,
-        brokerLogger,
-    });
+        tokenProvider: (0, createTokenProvider_1.createTokenProvider)(authConfig, options.browserAuthPort, logger),
+    }, undefined, brokerLogger);
+    // 4. Try to get connection.
+    let connection = (await broker.getConnectionConfig(destination));
+    // Perform authentication if session is missing but auth config is available
+    if (!connection && authConfig && destination !== 'env') {
+        (0, logVerbose_1.logVerbose)(1, `Initiating authentication for ${destination}...`);
+        await broker.getToken(destination);
+        connection = (await broker.getConnectionConfig(destination));
+    }
+    // Fallback to process.env if not found in stores
+    if (!connection &&
+        (destination === 'env' || destination === 'SAP' || options.envPath)) {
+        const url = process.env.SAP_URL || process.env.SAP_SERVICEURL;
+        if (url) {
+            connection = {
+                serviceUrl: url,
+                authorizationToken: process.env.SAP_JWT_TOKEN || process.env.SAP_TOKEN,
+                username: process.env.SAP_USERNAME || process.env.SAP_USER,
+                password: process.env.SAP_PASSWORD || process.env.SAP_PASS,
+                authType: (process.env.SAP_AUTH_TYPE ||
+                    (process.env.SAP_JWT_TOKEN ? 'jwt' : 'basic')),
+            };
+        }
+    }
+    if (!connection) {
+        throw new Error(`Missing connection config for destination ${destination}. If using service keys, ensure the JSON file exists in ${serviceKeyDir}`);
+    }
+    // If we have authConfig but no token in session, try to refresh/get it
+    if (!connection.authorizationToken &&
+        !connection.username &&
+        authConfig &&
+        destination !== 'env') {
+        await broker.getToken(destination);
+        connection = (await broker.getConnectionConfig(destination));
+    }
+    const authType = connection.authType ||
+        (connection.authorizationToken
+            ? 'jwt'
+            : connection.username && connection.password
+                ? 'basic'
+                : 'jwt');
+    const serviceUrl = connection.serviceUrl;
+    if (!serviceUrl) {
+        throw new Error(`Missing service URL for destination ${destination}`);
+    }
+    const config = {
+        url: serviceUrl,
+        authType: authType,
+    };
+    if (authType === 'jwt') {
+        config.jwtToken = connection.authorizationToken;
+    }
+    else {
+        config.username = connection.username;
+        config.password = connection.password;
+    }
+    const tokenRefresher = authType === 'jwt' ? broker.createTokenRefresher(destination) : undefined;
+    return { config, tokenRefresher };
+}
+function parseEnvContent(content) {
+    const envVars = {};
+    for (const line of content.split(/\r?\n/)) {
+        const trimmed = line.trim();
+        if (!trimmed || trimmed.startsWith('#'))
+            continue;
+        const eqIndex = trimmed.indexOf('=');
+        if (eqIndex === -1)
+            continue;
+        const key = trimmed.substring(0, eqIndex).trim();
+        let value = trimmed.substring(eqIndex + 1).trim();
+        value = value.replace(/^["']+|["']+$/g, '').trim();
+        if (key)
+            envVars[key] = value;
+    }
+    return envVars;
 }
 function resolveAuthRoots(authRoot) {
     if (authRoot) {
@@ -89,18 +167,6 @@ function resolveAuthRoots(authRoot) {
 function resolveStoreDirs(roots, destination) {
     const candidates = roots.map((root) => {
         const normalized = path.resolve(root);
-        if (normalized.endsWith(`${path.sep}sessions`)) {
-            return {
-                sessionDir: normalized,
-                serviceKeyDir: path.join(path.dirname(normalized), 'service-keys'),
-            };
-        }
-        if (normalized.endsWith(`${path.sep}service-keys`)) {
-            return {
-                sessionDir: path.join(path.dirname(normalized), 'sessions'),
-                serviceKeyDir: normalized,
-            };
-        }
         return {
             sessionDir: path.join(normalized, 'sessions'),
             serviceKeyDir: path.join(normalized, 'service-keys'),
@@ -115,70 +181,3 @@ function resolveStoreDirs(roots, destination) {
     }
     return candidates[0];
 }
-async function getConfigWithStores(options) {
-    const { destination, sessionStore, serviceKeyStore, logger, brokerLogger } = options;
-    const sessionAuthConfig = destination === 'env'
-        ? null
-        : await sessionStore.getAuthorizationConfig(destination);
-    const serviceKeyAuthConfig = destination === 'env' || !serviceKeyStore?.getAuthorizationConfig
-        ? null
-        : await serviceKeyStore.getAuthorizationConfig(destination);
-    const authConfig = sessionAuthConfig || serviceKeyAuthConfig;
-    const broker = new auth_broker_1.AuthBroker({
-        sessionStore,
-        serviceKeyStore,
-        tokenProvider: (0, createTokenProvider_1.createTokenProvider)(authConfig, logger),
-    }, undefined, brokerLogger);
-    const session = await sessionStore.loadSession(destination);
-    let connection = await broker.getConnectionConfig(destination);
-    if (!connection && authConfig && destination !== 'env') {
-        await broker.getToken(destination);
-        connection = await broker.getConnectionConfig(destination);
-    }
-    if (!connection) {
-        throw new Error(`Missing connection config for destination ${destination}`);
-    }
-    if (!connection.authorizationToken &&
-        !connection.username &&
-        !connection.password &&
-        authConfig &&
-        destination !== 'env') {
-        await broker.getToken(destination);
-        connection = await broker.getConnectionConfig(destination);
-        if (!connection) {
-            throw new Error(`Missing connection config for destination ${destination}`);
-        }
-    }
-    const resolvedAuthType = connection.authorizationToken
-        ? 'jwt'
-        : connection.username && connection.password
-            ? 'basic'
-            : authConfig && destination !== 'env'
-                ? 'jwt'
-                : connection.authType || 'basic';
-    const serviceUrl = connection.serviceUrl || session?.serviceUrl;
-    if (!serviceUrl) {
-        throw new Error(`Missing service URL for destination ${destination}`);
-    }
-    const config = {
-        url: serviceUrl,
-        authType: resolvedAuthType,
-    };
-    if (resolvedAuthType === 'jwt') {
-        if (!connection.authorizationToken) {
-            throw new Error(`Missing JWT token for destination ${destination}`);
-        }
-        config.jwtToken = connection.authorizationToken;
-    }
-    else {
-        if (!connection.username || !connection.password) {
-            throw new Error(`Missing username/password for destination ${destination}`);
-        }
-        config.username = connection.username;
-        config.password = connection.password;
-    }
-    const tokenRefresher = resolvedAuthType === 'jwt'
-        ? broker.createTokenRefresher(destination)
-        : undefined;
-    return { config, tokenRefresher };
-}

package/dist/lib/backup/backupObject.js

@@ -90,7 +90,7 @@ async function backupObject(client, spec) {
                 type: spec.type,
                 name: spec.name,
                 config: (0, applyConfigName_1.applyConfigName)(spec.type, spec.name, undefined, (0, toBackupConfig_1.toBackupConfig)(config)),
-                source,
+                source: source ?? undefined,
             };
         }
         case 'serviceBinding': {
@@ -120,7 +120,7 @@ async function backupObject(client, spec) {
                 type: spec.type,
                 name: spec.name,
                 config,
-                source,
+                source: source ?? undefined,
             };
         }
         case 'behaviorImplementation':
@@ -140,7 +140,7 @@ async function backupObject(client, spec) {
                 type: spec.type,
                 name: spec.name,
                 config,
-                source,
+                source: source ?? undefined,
             };
         }
         case 'functionModule': {
@@ -158,7 +158,7 @@ async function backupObject(client, spec) {
                 name: spec.name,
                 functionGroupName: spec.functionGroupName,
                 config,
-                source,
+                source: source ?? undefined,
             };
         }
         default: {
@@ -173,7 +173,7 @@ async function backupObject(client, spec) {
                 type: spec.type,
                 name: spec.name,
                 config,
-                source,
+                source: source ?? undefined,
             };
         }
     }

package/dist/lib/backup/readMetadataXmlForType.d.ts

@@ -1,4 +1,4 @@
 import type { AdtClient } from '@mcp-abap-adt/adt-clients';
 import type { SupportedType } from '../types';
-export declare function readMetadataXmlForType(client: AdtClient, type: SupportedType, name: string, functionGroupName?: string): Promise<string | undefined>;
+export declare function readMetadataXmlForType(client: AdtClient, type: SupportedType, name: string, _functionGroupName?: string): Promise<string | null | undefined>;
 //# sourceMappingURL=readMetadataXmlForType.d.ts.map

package/dist/lib/backup/readMetadataXmlForType.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"readMetadataXmlForType.d.ts","sourceRoot":"","sources":["../../../src/lib/backup/readMetadataXmlForType.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAC3D,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAG9C,wBAAsB,sBAAsB,CAC1C,MAAM,EAAE,SAAS,EACjB,IAAI,EAAE,aAAa,EACnB,IAAI,EAAE,MAAM,EACZ,iBAAiB,CAAC,EAAE,MAAM,GACzB,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CAqG7B"}
+{"version":3,"file":"readMetadataXmlForType.d.ts","sourceRoot":"","sources":["../../../src/lib/backup/readMetadataXmlForType.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAC3D,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAG9C,wBAAsB,sBAAsB,CAC1C,MAAM,EAAE,SAAS,EACjB,IAAI,EAAE,aAAa,EACnB,IAAI,EAAE,MAAM,EACZ,kBAAkB,CAAC,EAAE,MAAM,GAC1B,OAAO,CAAC,MAAM,GAAG,IAAI,GAAG,SAAS,CAAC,CAsHpC"}

package/dist/lib/backup/readMetadataXmlForType.js

@@ -2,105 +2,120 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.readMetadataXmlForType = readMetadataXmlForType;
 const responseToText_1 = require("../utils/responseToText");
-async function readMetadataXmlForType(client, type, name, functionGroupName) {
-    switch (type) {
-        case 'class': {
-            const state = await client.getClass().readMetadata({ className: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'interface': {
-            const state = await client
-                .getInterface()
-                .readMetadata({ interfaceName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'program': {
-            const state = await client
-                .getProgram()
-                .readMetadata({ programName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'structure': {
-            const state = await client
-                .getStructure()
-                .readMetadata({ structureName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'table': {
-            const state = await client.getTable().readMetadata({ tableName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'tableType': {
-            const state = await client
-                .getTableType()
-                .readMetadata({ tableTypeName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'view': {
-            const state = await client.getView().readMetadata({ viewName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'domain': {
-            const state = await client.getDomain().readMetadata({ domainName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'dataElement': {
-            const state = await client
-                .getDataElement()
-                .readMetadata({ dataElementName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'functionGroup': {
-            const state = await client
-                .getFunctionGroup()
-                .readMetadata({ functionGroupName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'functionModule': {
-            if (!functionGroupName) {
-                return undefined;
+async function readMetadataXmlForType(client, type, name, _functionGroupName) {
+    try {
+        switch (type) {
+            case 'package': {
+                const state = await client
+                    .getPackage()
+                    .readMetadata({ packageName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
             }
-            const state = await client.getFunctionModule().readMetadata({
-                functionGroupName,
-                functionModuleName: name,
-            });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'package': {
-            const state = await client
-                .getPackage()
-                .readMetadata({ packageName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'serviceDefinition': {
-            const state = await client
-                .getServiceDefinition()
-                .readMetadata({ serviceDefinitionName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'serviceBinding': {
-            const state = await client
-                .getServiceBinding()
-                .readMetadata({ bindingName: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'metadataExtension': {
-            const state = await client.getMetadataExtension().readMetadata({ name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
-        }
-        case 'behaviorDefinition': {
-            const state = await client
-                .getBehaviorDefinition()
-                .read({ name }, 'active');
-            return (0, responseToText_1.responseToText)(state?.readResult);
-        }
-        case 'behaviorImplementation': {
-            const state = await client
-                .getBehaviorImplementation()
-                .readMetadata({ className: name });
-            return (0, responseToText_1.responseToText)(state.metadataResult);
+            case 'class': {
+                const state = await client.getClass().readMetadata({ className: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'interface': {
+                const state = await client
+                    .getInterface()
+                    .readMetadata({ interfaceName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'domain': {
+                const state = await client
+                    .getDomain()
+                    .readMetadata({ domainName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'dataElement': {
+                const state = await client
+                    .getDataElement()
+                    .readMetadata({ dataElementName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'table': {
+                const state = await client.getTable().readMetadata({ tableName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'tableType': {
+                const state = await client
+                    .getTableType()
+                    .readMetadata({ tableTypeName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'structure': {
+                const state = await client
+                    .getStructure()
+                    .readMetadata({ structureName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'view': {
+                const state = await client.getView().readMetadata({ viewName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'behaviorDefinition': {
+                const state = await client
+                    .getBehaviorDefinition()
+                    .readMetadata({ name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'behaviorImplementation': {
+                const state = await client
+                    .getBehaviorImplementation()
+                    .readMetadata({ className: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'serviceDefinition': {
+                const state = await client
+                    .getServiceDefinition()
+                    .readMetadata({ serviceDefinitionName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'serviceBinding': {
+                const state = await client
+                    .getServiceBinding()
+                    .readMetadata({ bindingName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'metadataExtension': {
+                const state = await client
+                    .getMetadataExtension()
+                    .readMetadata({ name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'functionGroup': {
+                const state = await client
+                    .getFunctionGroup()
+                    .readMetadata({ functionGroupName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'enhancement': {
+                const state = await client
+                    .getEnhancement()
+                    .readMetadata({ enhancementName: name, enhancementType: 'enhoxh' });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            case 'accessControl': {
+                const state = await client
+                    .getAccessControl()
+                    .readMetadata({ accessControlName: name });
+                return (0, responseToText_1.responseToText)(state.metadataResult);
+            }
+            default:
+                return undefined;
         }
-        default:
-            return undefined;
+    }
+    catch (error) {
+        const status = error.status || error.response?.status;
+        const message = error.message || String(error);
+        const isNotFound = status === 404 ||
+            status === 410 ||
+            /not found/i.test(message) ||
+            /does not exist/i.test(message) ||
+            /ExceptionResourceNotFound/i.test(message);
+        if (isNotFound) {
+            return null;
+        }
+        throw error;
     }
 }

package/dist/lib/backup/readSourceText.d.ts

@@ -1,4 +1,4 @@
 import type { AdtClient } from '@mcp-abap-adt/adt-clients';
 import type { ObjectSpec } from '../types';
-export declare function readSourceText(client: AdtClient, spec: ObjectSpec): Promise<string | undefined>;
+export declare function readSourceText(client: AdtClient, spec: ObjectSpec, version?: 'active' | 'inactive'): Promise<string | null | undefined>;
 //# sourceMappingURL=readSourceText.d.ts.map

package/dist/lib/backup/readSourceText.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"readSourceText.d.ts","sourceRoot":"","sources":["../../../src/lib/backup/readSourceText.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAC3D,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAG3C,wBAAsB,cAAc,CAClC,MAAM,EAAE,SAAS,EACjB,IAAI,EAAE,UAAU,GACf,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CAuG7B"}
+{"version":3,"file":"readSourceText.d.ts","sourceRoot":"","sources":["../../../src/lib/backup/readSourceText.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAC3D,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAG3C,wBAAsB,cAAc,CAClC,MAAM,EAAE,SAAS,EACjB,IAAI,EAAE,UAAU,EAChB,OAAO,GAAE,QAAQ,GAAG,UAAqB,GACxC,OAAO,CAAC,MAAM,GAAG,IAAI,GAAG,SAAS,CAAC,CAwGpC"}