@mcp-abap-adt/connection 0.1.15 → 0.2.0

package/dist/connection/AbstractAbapConnection.js

@@ -48,16 +48,12 @@ class AbstractAbapConnection {
     cookieStore = new Map();
     baseUrl;
     sessionId = null;
-    sessionStorage = null;
     sessionMode = "stateless";
-    constructor(config, logger, sessionStorage, sessionId) {
+    constructor(config, logger, sessionId) {
         this.config = config;
         this.logger = logger;
-        this.sessionStorage = sessionStorage || null;
-        // Always generate sessionId (used for sap-adt-connection-id header in all session types)
+        // Generate sessionId (used for sap-adt-connection-id header)
         this.sessionId = sessionId || (0, crypto_1.randomUUID)();
-        // Session mode depends only on storage availability (sessionId exists for both modes)
-        this.sessionMode = sessionStorage ? "stateful" : "stateless";
         // Initialize baseUrl from config (required, will throw if invalid)
         try {
             const urlObj = new URL(config.url);
@@ -66,7 +62,7 @@ class AbstractAbapConnection {
         catch (error) {
             throw new Error(`Invalid URL in configuration: ${error instanceof Error ? error.message : error}`);
         }
-        this.logger.debug(`AbstractAbapConnection - Session ID: ${this.sessionId.substring(0, 8)}..., mode: ${this.sessionMode}`);
+        this.logger?.debug(`AbstractAbapConnection - Session ID: ${this.sessionId.substring(0, 8)}...`);
     }
     /**
      * Set session type (stateful or stateless)
@@ -76,15 +72,13 @@ class AbstractAbapConnection {
      */
     setSessionType(type) {
         this.sessionMode = type;
-        this.logger.debug(`Session type set to: ${type}`, {
-            sessionId: this.sessionId?.substring(0, 8),
-            hasStorage: !!this.sessionStorage
+        this.logger?.debug(`Session type set to: ${type}`, {
+            sessionId: this.sessionId?.substring(0, 8)
         });
     }
     /**
      * Enable stateful session mode (tells SAP to maintain stateful session)
      * This controls whether x-sap-adt-sessiontype: stateful header is used
-     * Storage is controlled separately via setSessionStorage()
      * @deprecated Use setSessionType("stateful") instead
      */
     enableStatefulSession() {
@@ -92,20 +86,14 @@ class AbstractAbapConnection {
     }
     /**
      * Disable stateful session mode (switch to stateless)
-     * Optionally saves current state before switching
      * @deprecated Use setSessionType("stateless") instead
      */
-    async disableStatefulSession(saveBeforeDisable = false) {
+    disableStatefulSession() {
         if (this.sessionMode === "stateless") {
             return;
         }
-        if (saveBeforeDisable && this.sessionId && this.sessionStorage) {
-            await this.saveSessionState();
-        }
         this.sessionMode = "stateless";
-        this.logger.debug("Stateful session mode disabled", {
-            savedBeforeDisable: saveBeforeDisable
-        });
+        this.logger?.debug("Stateful session mode disabled");
     }
     /**
      * Get current session mode
@@ -114,18 +102,12 @@ class AbstractAbapConnection {
         return this.sessionMode;
     }
     /**
-     * Set session ID for stateful operations
-     * When session ID is set, session state (cookies, CSRF token) will be persisted
-     * @deprecated Use enableStatefulSession() instead
+     * Set session ID
+     * @deprecated Session ID is auto-generated, use setSessionType() to control session mode
      */
     setSessionId(sessionId) {
-        if (this.sessionStorage) {
-            this.sessionId = sessionId;
-            this.sessionMode = "stateful";
-        }
-        else {
-            this.logger.warn("Cannot set session ID without session storage. Use enableStatefulSession() instead.");
-        }
+        this.sessionId = sessionId;
+        this.logger?.debug(`Session ID set to: ${sessionId.substring(0, 8)}...`);
     }
     /**
      * Get current session ID
@@ -133,132 +115,6 @@ class AbstractAbapConnection {
     getSessionId() {
         return this.sessionId;
     }
-    /**
-     * Set session storage (can be changed at runtime)
-     * This controls whether session state (cookies, CSRF token) is persisted to disk/storage
-     */
-    async setSessionStorage(storage) {
-        this.sessionStorage = storage;
-        // Load existing session state if storage is provided
-        if (storage && this.sessionId) {
-            await this.loadSessionState();
-        }
-        this.logger.debug("Session storage configured", {
-            hasStorage: !!storage,
-            sessionMode: this.sessionMode
-        });
-    }
-    /**
-     * Get current session storage
-     */
-    getSessionStorage() {
-        return this.sessionStorage;
-    }
-    /**
-     * Load session state from storage
-     */
-    async loadSessionState() {
-        if (!this.sessionId || !this.sessionStorage) {
-            return;
-        }
-        try {
-            const state = await this.sessionStorage.load(this.sessionId);
-            if (state) {
-                this.csrfToken = state.csrfToken;
-                this.cookies = state.cookies;
-                this.cookieStore = new Map(Object.entries(state.cookieStore));
-                this.logger.debug("Session state loaded", {
-                    sessionId: this.sessionId,
-                    hasCookies: !!this.cookies,
-                    hasCsrfToken: !!this.csrfToken
-                });
-            }
-        }
-        catch (error) {
-            this.logger.warn("Failed to load session state", {
-                sessionId: this.sessionId,
-                error: error instanceof Error ? error.message : String(error)
-            });
-        }
-    }
-    /**
-     * Save session state to storage
-     * Only saves if in stateful mode
-     */
-    async saveSessionState() {
-        if (this.sessionMode !== "stateful" || !this.sessionId || !this.sessionStorage) {
-            return;
-        }
-        try {
-            const state = {
-                cookies: this.cookies,
-                csrfToken: this.csrfToken,
-                cookieStore: Object.fromEntries(this.cookieStore)
-            };
-            await this.sessionStorage.save(this.sessionId, state);
-            this.logger.debug("Session state saved", {
-                sessionId: this.sessionId,
-                hasCookies: !!this.cookies,
-                hasCsrfToken: !!this.csrfToken
-            });
-        }
-        catch (error) {
-            this.logger.warn("Failed to save session state", {
-                sessionId: this.sessionId,
-                error: error instanceof Error ? error.message : String(error)
-            });
-        }
-    }
-    /**
-     * Get current session state
-     * Returns cookies, CSRF token, and cookie store for manual persistence
-     * @returns Current session state or null if no session data
-     */
-    getSessionState() {
-        if (!this.cookies && !this.csrfToken) {
-            return null;
-        }
-        return {
-            cookies: this.cookies,
-            csrfToken: this.csrfToken,
-            cookieStore: Object.fromEntries(this.cookieStore)
-        };
-    }
-    /**
-     * Set session state manually
-     * Allows user to restore session from custom storage (e.g., database, Redis)
-     * @param state - Session state with cookies, CSRF token, and cookie store
-     */
-    setSessionState(state) {
-        this.cookies = state.cookies || null;
-        this.csrfToken = state.csrfToken || null;
-        this.cookieStore = new Map(Object.entries(state.cookieStore || {}));
-        this.logger.debug("Session state set manually", {
-            hasCookies: !!this.cookies,
-            hasCsrfToken: !!this.csrfToken,
-            cookieCount: this.cookieStore.size
-        });
-    }
-    /**
-     * Clear session state from storage
-     */
-    async clearSessionState() {
-        if (!this.sessionId || !this.sessionStorage) {
-            return;
-        }
-        try {
-            await this.sessionStorage.delete(this.sessionId);
-            this.logger.debug("Session state cleared", {
-                sessionId: this.sessionId
-            });
-        }
-        catch (error) {
-            this.logger.warn("Failed to clear session state", {
-                sessionId: this.sessionId,
-                error: error instanceof Error ? error.message : String(error)
-            });
-        }
-    }
     getConfig() {
         return this.config;
     }
@@ -302,7 +158,7 @@ class AbstractAbapConnection {
                 catch (error) {
                     // If CSRF token can't be fetched upfront, continue anyway
                     // The retry logic will handle CSRF token errors automatically
-                    this.logger.debug(`[DEBUG] BaseAbapConnection - Could not fetch CSRF token upfront, will retry on error: ${error instanceof Error ? error.message : String(error)}`);
+                    this.logger?.debug(`[DEBUG] BaseAbapConnection - Could not fetch CSRF token upfront, will retry on error: ${error instanceof Error ? error.message : String(error)}`);
                 }
             }
         }
@@ -333,10 +189,10 @@ class AbstractAbapConnection {
         // Add cookies LAST (MUST NOT be overridden by custom headers)
         if (this.cookies) {
             requestHeaders["Cookie"] = this.cookies;
-            this.logger.debug(`[DEBUG] BaseAbapConnection - Adding cookies to request (first 100 chars): ${this.cookies.substring(0, 100)}...`);
+            this.logger?.debug(`[DEBUG] BaseAbapConnection - Adding cookies to request (first 100 chars): ${this.cookies.substring(0, 100)}...`);
         }
         else {
-            this.logger.debug(`[DEBUG] BaseAbapConnection - NO COOKIES available for this request to ${requestUrl}`);
+            this.logger?.debug(`[DEBUG] BaseAbapConnection - NO COOKIES available for this request to ${requestUrl}`);
         }
         if ((normalizedMethod === "POST" || normalizedMethod === "PUT") && data) {
             if (typeof data === "string" && !requestHeaders["Content-Type"]) {
@@ -359,7 +215,7 @@ class AbstractAbapConnection {
         if (data !== undefined) {
             requestConfig.data = data;
         }
-        this.logger.debug(`Executing ${normalizedMethod} request to: ${requestUrl}`, {
+        this.logger?.debug(`Executing ${normalizedMethod} request to: ${requestUrl}`, {
             type: "REQUEST_INFO",
             url: requestUrl,
             method: normalizedMethod
@@ -367,9 +223,7 @@ class AbstractAbapConnection {
         try {
             const response = await this.getAxiosInstance()(requestConfig);
             this.updateCookiesFromResponse(response.headers);
-            // Save session state after successful request (if session storage is configured)
-            await this.saveSessionState();
-            this.logger.debug(`Request succeeded with status ${response.status}`, {
+            this.logger?.debug(`Request succeeded with status ${response.status}`, {
                 type: "REQUEST_SUCCESS",
                 status: response.status,
                 url: requestUrl,
@@ -393,18 +247,16 @@ class AbstractAbapConnection {
                         : JSON.stringify(error.response.data).slice(0, 200);
                 this.updateCookiesFromResponse(error.response.headers);
             }
-            // Save session state even on error (cookies might have been updated)
-            await this.saveSessionState();
             // Log 404 as debug (common for existence checks), other errors as error
             if (errorDetails.status === 404) {
-                this.logger.debug(errorDetails.message, errorDetails);
+                this.logger?.debug(errorDetails.message, errorDetails);
             }
             else {
-                this.logger.error(errorDetails.message, errorDetails);
+                this.logger?.error(errorDetails.message, errorDetails);
             }
             // Retry logic for CSRF token errors (403 with CSRF message)
             if (this.shouldRetryCsrf(error)) {
-                this.logger.debug("CSRF token validation failed, fetching new token and retrying request", {
+                this.logger?.debug("CSRF token validation failed, fetching new token and retrying request", {
                     url: requestUrl,
                     method: normalizedMethod
                 });
@@ -417,8 +269,6 @@ class AbstractAbapConnection {
                 }
                 const retryResponse = await this.getAxiosInstance()(requestConfig);
                 this.updateCookiesFromResponse(retryResponse.headers);
-                // Save session state after retry
-                await this.saveSessionState();
                 return retryResponse;
             }
             // Retry logic for 401 errors on GET requests (authentication issue - need cookies)
@@ -430,29 +280,27 @@ class AbstractAbapConnection {
             ) {
                 // If we already have cookies from error response, retry immediately
                 if (this.cookies) {
-                    this.logger.debug(`[DEBUG] BaseAbapConnection - 401 on GET request, retrying with cookies from error response`);
+                    this.logger?.debug(`[DEBUG] BaseAbapConnection - 401 on GET request, retrying with cookies from error response`);
                     requestHeaders["Cookie"] = this.cookies;
                     const retryResponse = await this.getAxiosInstance()(requestConfig);
                     this.updateCookiesFromResponse(retryResponse.headers);
-                    await this.saveSessionState();
                     return retryResponse;
                 }
                 // If no cookies, try to get them via CSRF token fetch
-                this.logger.debug(`[DEBUG] BaseAbapConnection - 401 on GET request, attempting to get cookies via CSRF token fetch`);
+                this.logger?.debug(`[DEBUG] BaseAbapConnection - 401 on GET request, attempting to get cookies via CSRF token fetch`);
                 try {
                     // Try to get CSRF token (this will also get cookies)
                     this.csrfToken = await this.fetchCsrfToken(requestUrl, 3, 1000);
                     if (this.cookies) {
                         requestHeaders["Cookie"] = this.cookies;
-                        this.logger.debug(`[DEBUG] BaseAbapConnection - Retrying GET request with cookies from CSRF fetch`);
+                        this.logger?.debug(`[DEBUG] BaseAbapConnection - Retrying GET request with cookies from CSRF fetch`);
                         const retryResponse = await this.getAxiosInstance()(requestConfig);
                         this.updateCookiesFromResponse(retryResponse.headers);
-                        await this.saveSessionState();
                         return retryResponse;
                     }
                 }
                 catch (csrfError) {
-                    this.logger.debug(`[DEBUG] BaseAbapConnection - Failed to get CSRF token for 401 retry: ${csrfError instanceof Error ? csrfError.message : String(csrfError)}`);
+                    this.logger?.debug(`[DEBUG] BaseAbapConnection - Failed to get CSRF token for 401 retry: ${csrfError instanceof Error ? csrfError.message : String(csrfError)}`);
                     // Fall through to throw original error
                 }
             }
@@ -476,11 +324,11 @@ class AbstractAbapConnection {
             csrfUrl = `${base}${csrfConfig_js_1.CSRF_CONFIG.ENDPOINT}`;
         }
         // If URL already contains the endpoint, use it as is
-        this.logger.debug(`Fetching CSRF token from: ${csrfUrl}`);
+        this.logger?.debug(`Fetching CSRF token from: ${csrfUrl}`);
         for (let attempt = 0; attempt <= retryCount; attempt++) {
             try {
                 if (attempt > 0) {
-                    this.logger.debug(`Retry attempt ${attempt}/${retryCount} for CSRF token`);
+                    this.logger?.debug(`Retry attempt ${attempt}/${retryCount} for CSRF token`);
                 }
                 const authHeaders = await this.getAuthHeaders();
                 const headers = {
@@ -491,13 +339,13 @@ class AbstractAbapConnection {
                 // Even on first attempt, if we have cookies from previous session or error response, use them
                 if (this.cookies) {
                     headers["Cookie"] = this.cookies;
-                    this.logger.debug(`[DEBUG] BaseAbapConnection - Adding cookies to CSRF token request (attempt ${attempt + 1}, first 100 chars): ${this.cookies.substring(0, 100)}...`);
+                    this.logger?.debug(`[DEBUG] BaseAbapConnection - Adding cookies to CSRF token request (attempt ${attempt + 1}, first 100 chars): ${this.cookies.substring(0, 100)}...`);
                 }
                 else {
-                    this.logger.debug(`[DEBUG] BaseAbapConnection - No cookies available for CSRF token request (will get fresh cookies from response)`);
+                    this.logger?.debug(`[DEBUG] BaseAbapConnection - No cookies available for CSRF token request (will get fresh cookies from response)`);
                 }
                 // Log request details for debugging (only if debug logging is enabled)
-                this.logger.debug(`[DEBUG] CSRF Token Request: url=${csrfUrl}, method=GET, hasAuth=${!!authHeaders.Authorization}, hasClient=${!!authHeaders["X-SAP-Client"]}, hasCookies=${!!headers["Cookie"]}, attempt=${attempt + 1}`);
+                this.logger?.debug(`[DEBUG] CSRF Token Request: url=${csrfUrl}, method=GET, hasAuth=${!!authHeaders.Authorization}, hasClient=${!!authHeaders["X-SAP-Client"]}, hasCookies=${!!headers["Cookie"]}, attempt=${attempt + 1}`);
                 const response = await this.getAxiosInstance()({
                     method: "GET",
                     url: csrfUrl,
@@ -507,7 +355,7 @@ class AbstractAbapConnection {
                 this.updateCookiesFromResponse(response.headers);
                 const token = response.headers["x-csrf-token"];
                 if (!token) {
-                    this.logger.error("No CSRF token in response headers", {
+                    this.logger?.error("No CSRF token in response headers", {
                         headers: response.headers,
                         status: response.status
                     });
@@ -520,15 +368,13 @@ class AbstractAbapConnection {
                 if (response.headers["set-cookie"]) {
                     this.updateCookiesFromResponse(response.headers);
                     if (this.cookies) {
-                        this.logger.debug(`[DEBUG] BaseAbapConnection - Cookies received from CSRF response (first 100 chars): ${this.cookies.substring(0, 100)}...`);
-                        this.logger.debug("Cookies extracted from response", {
+                        this.logger?.debug(`[DEBUG] BaseAbapConnection - Cookies received from CSRF response (first 100 chars): ${this.cookies.substring(0, 100)}...`);
+                        this.logger?.debug("Cookies extracted from response", {
                             cookieLength: this.cookies.length
                         });
                     }
                 }
-                // Save session state after CSRF token fetch (cookies and token are now available)
-                await this.saveSessionState();
-                this.logger.debug("CSRF token successfully obtained");
+                this.logger?.debug("CSRF token successfully obtained");
                 return token;
             }
             catch (error) {
@@ -538,20 +384,20 @@ class AbstractAbapConnection {
                     if (error.response?.headers) {
                         this.updateCookiesFromResponse(error.response.headers);
                         if (this.cookies) {
-                            this.logger.debug("Cookies extracted from error response", {
+                            this.logger?.debug("Cookies extracted from error response", {
                                 status: error.response.status,
                                 cookieLength: this.cookies.length
                             });
                         }
                     }
-                    this.logger.error(`CSRF token error: ${error.message}`, {
+                    this.logger?.error(`CSRF token error: ${error.message}`, {
                         url: csrfUrl,
                         status: error.response?.status,
                         attempt: attempt + 1,
                         maxAttempts: retryCount + 1
                     });
                     if (error.response?.status === 405 && error.response?.headers["x-csrf-token"]) {
-                        this.logger.debug("CSRF: SAP returned 405 (Method Not Allowed) — not critical, token found in header");
+                        this.logger?.debug("CSRF: SAP returned 405 (Method Not Allowed) — not critical, token found in header");
                         const token = error.response.headers["x-csrf-token"];
                         if (token) {
                             this.updateCookiesFromResponse(error.response.headers);
@@ -559,13 +405,13 @@ class AbstractAbapConnection {
                         }
                     }
                     if (error.response?.headers["x-csrf-token"]) {
-                        this.logger.debug(`Got CSRF token despite error (status: ${error.response?.status})`);
+                        this.logger?.debug(`Got CSRF token despite error (status: ${error.response?.status})`);
                         const token = error.response.headers["x-csrf-token"];
                         this.updateCookiesFromResponse(error.response.headers);
                         return token;
                     }
                     if (error.response) {
-                        this.logger.error("CSRF error details", {
+                        this.logger?.error("CSRF error details", {
                             status: error.response.status,
                             statusText: error.response.statusText,
                             headers: Object.keys(error.response.headers),
@@ -575,13 +421,13 @@ class AbstractAbapConnection {
                         });
                     }
                     else if (error.request) {
-                        this.logger.error("CSRF request error - no response received", {
+                        this.logger?.error("CSRF request error - no response received", {
                             request: error.request.path
                         });
                     }
                 }
                 else {
-                    this.logger.error("CSRF non-axios error", {
+                    this.logger?.error("CSRF non-axios error", {
                         error: error instanceof Error ? error.message : String(error)
                     });
                 }
@@ -655,14 +501,14 @@ class AbstractAbapConnection {
             return;
         }
         this.cookies = combined;
-        this.logger.debug(`[DEBUG] BaseAbapConnection - Updated cookies from response (first 100 chars): ${this.cookies.substring(0, 100)}...`);
+        this.logger?.debug(`[DEBUG] BaseAbapConnection - Updated cookies from response (first 100 chars): ${this.cookies.substring(0, 100)}...`);
     }
     getAxiosInstance() {
         if (!this.axiosInstance) {
             const rejectUnauthorized = process.env.NODE_TLS_REJECT_UNAUTHORIZED === "1" ||
                 (process.env.TLS_REJECT_UNAUTHORIZED === "1" &&
                     process.env.NODE_TLS_REJECT_UNAUTHORIZED !== "0");
-            this.logger.debug(`TLS configuration: rejectUnauthorized=${rejectUnauthorized}`);
+            this.logger?.debug(`TLS configuration: rejectUnauthorized=${rejectUnauthorized}`);
             this.axiosInstance = axios_1.default.create({
                 httpsAgent: new https_1.Agent({
                     rejectUnauthorized
@@ -675,20 +521,20 @@ class AbstractAbapConnection {
         // If we already have a CSRF token, reuse it to keep the same SAP session
         // SAP ties the lock handle to the HTTP session (SAP_SESSIONID cookie)
         if (this.csrfToken) {
-            this.logger.debug(`[DEBUG] BaseAbapConnection - Reusing existing CSRF token to maintain session`);
+            this.logger?.debug(`[DEBUG] BaseAbapConnection - Reusing existing CSRF token to maintain session`);
             return;
         }
         try {
-            this.logger.debug(`[DEBUG] BaseAbapConnection - Fetching NEW CSRF token (will create new SAP session)`);
+            this.logger?.debug(`[DEBUG] BaseAbapConnection - Fetching NEW CSRF token (will create new SAP session)`);
             this.csrfToken = await this.fetchCsrfToken(requestUrl);
         }
         catch (error) {
-            // fetchCsrfToken already handles auth errors and auto-refresh
+            // fetchCsrfToken handles auth errors
             // Just re-throw the error with minimal logging to avoid duplicate error messages
             const errorMsg = error instanceof Error ? error.message : csrfConfig_js_1.CSRF_ERROR_MESSAGES.REQUIRED_FOR_MUTATION;
             // Only log at DEBUG level to avoid duplicate error messages
             // (fetchCsrfToken already logged the error at ERROR level if auth failed)
-            this.logger.debug(`[DEBUG] BaseAbapConnection - ensureFreshCsrfToken failed: ${errorMsg}`);
+            this.logger?.debug(`[DEBUG] BaseAbapConnection - ensureFreshCsrfToken failed: ${errorMsg}`);
             throw error;
         }
     }

package/dist/connection/BaseAbapConnection.d.ts

@@ -1,11 +1,11 @@
 import { SapConfig } from "../config/sapConfig.js";
 import { AbstractAbapConnection } from "./AbstractAbapConnection.js";
-import { ILogger, ISessionStorage } from "../logger.js";
+import { ILogger } from "../logger.js";
 /**
  * Basic Authentication connection for on-premise SAP systems
  */
 export declare class BaseAbapConnection extends AbstractAbapConnection {
-    constructor(config: SapConfig, logger: ILogger, sessionStorage?: ISessionStorage, sessionId?: string);
+    constructor(config: SapConfig, logger?: ILogger | null, sessionId?: string);
     /**
      * Connect to SAP system with Basic Auth
      * Fetches CSRF token which also establishes session cookies

package/dist/connection/BaseAbapConnection.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"BaseAbapConnection.d.ts","sourceRoot":"","sources":["../../src/connection/BaseAbapConnection.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAC;AACnD,OAAO,EAAE,sBAAsB,EAAE,MAAM,6BAA6B,CAAC;AACrE,OAAO,EAAE,OAAO,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAGxD;;GAEG;AACH,qBAAa,kBAAmB,SAAQ,sBAAsB;gBAE1D,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,OAAO,EACf,cAAc,CAAC,EAAE,eAAe,EAChC,SAAS,CAAC,EAAE,MAAM;IAMpB;;;OAGG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAoC9B,SAAS,CAAC,wBAAwB,IAAI,MAAM;IAQ5C,OAAO,CAAC,MAAM,CAAC,cAAc;CAa9B"}
+{"version":3,"file":"BaseAbapConnection.d.ts","sourceRoot":"","sources":["../../src/connection/BaseAbapConnection.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAC;AACnD,OAAO,EAAE,sBAAsB,EAAE,MAAM,6BAA6B,CAAC;AACrE,OAAO,EAAE,OAAO,EAAE,MAAM,cAAc,CAAC;AAGvC;;GAEG;AACH,qBAAa,kBAAmB,SAAQ,sBAAsB;gBAE1D,MAAM,EAAE,SAAS,EACjB,MAAM,CAAC,EAAE,OAAO,GAAG,IAAI,EACvB,SAAS,CAAC,EAAE,MAAM;IAMpB;;;OAGG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAiC9B,SAAS,CAAC,wBAAwB,IAAI,MAAM;IAQ5C,OAAO,CAAC,MAAM,CAAC,cAAc;CAa9B"}

package/dist/connection/BaseAbapConnection.js

@@ -7,9 +7,9 @@ const axios_1 = require("axios");
  * Basic Authentication connection for on-premise SAP systems
  */
 class BaseAbapConnection extends AbstractAbapConnection_js_1.AbstractAbapConnection {
-    constructor(config, logger, sessionStorage, sessionId) {
+    constructor(config, logger, sessionId) {
         BaseAbapConnection.validateConfig(config);
-        super(config, logger, sessionStorage, sessionId);
+        super(config, logger || null, sessionId);
     }
     /**
      * Connect to SAP system with Basic Auth
@@ -18,14 +18,12 @@ class BaseAbapConnection extends AbstractAbapConnection_js_1.AbstractAbapConnect
     async connect() {
         const baseUrl = await this.getBaseUrl();
         const discoveryUrl = `${baseUrl}/sap/bc/adt/discovery`;
-        this.logger.debug(`[DEBUG] BaseAbapConnection - Connecting to SAP system: ${discoveryUrl}`);
+        this.logger?.debug(`[DEBUG] BaseAbapConnection - Connecting to SAP system: ${discoveryUrl}`);
         try {
             // Try to get CSRF token (this will also get cookies)
             const token = await this.fetchCsrfToken(discoveryUrl, 3, 1000);
             this.setCsrfToken(token);
-            // Save session state after successful connection
-            await this.saveSessionState();
-            this.logger.debug("Successfully connected to SAP system", {
+            this.logger?.debug("Successfully connected to SAP system", {
                 hasCsrfToken: !!this.getCsrfToken(),
                 hasCookies: !!this.getCookies(),
                 cookieLength: this.getCookies()?.length || 0
@@ -35,13 +33,12 @@ class BaseAbapConnection extends AbstractAbapConnection_js_1.AbstractAbapConnect
             // For Basic auth, log warning but don't fail
             // The retry logic in makeAdtRequest will handle transient errors automatically
             const errorMsg = error instanceof Error ? error.message : String(error);
-            this.logger.warn(`[WARN] BaseAbapConnection - Could not connect to SAP system upfront: ${errorMsg}. Will retry on first request.`);
+            this.logger?.warn(`[WARN] BaseAbapConnection - Could not connect to SAP system upfront: ${errorMsg}. Will retry on first request.`);
             // Still try to extract cookies from error response if available
             if (error instanceof axios_1.AxiosError && error.response?.headers) {
                 // updateCookiesFromResponse is private, but cookies are extracted in fetchCsrfToken
                 if (this.getCookies()) {
-                    this.logger.debug(`[DEBUG] BaseAbapConnection - Cookies extracted from error response during connect (first 100 chars): ${this.getCookies().substring(0, 100)}...`);
-                    await this.saveSessionState();
+                    this.logger?.debug(`[DEBUG] BaseAbapConnection - Cookies extracted from error response during connect (first 100 chars): ${this.getCookies().substring(0, 100)}...`);
                 }
             }
         }

package/dist/connection/JwtAbapConnection.d.ts

@@ -1,36 +1,28 @@
 import { SapConfig } from "../config/sapConfig.js";
 import { AbstractAbapConnection } from "./AbstractAbapConnection.js";
 import { AbapRequestOptions } from "./AbapConnection.js";
-import { ILogger, ISessionStorage } from "../logger.js";
+import { ILogger } from "../logger.js";
 import { AxiosResponse } from "axios";
 /**
  * JWT Authentication connection for SAP BTP Cloud systems
- * Supports automatic token refresh using OAuth2 refresh tokens
+ * Note: Token refresh functionality is not supported in this package.
+ * Use @mcp-abap-adt/auth-broker for token refresh functionality.
  */
 export declare class JwtAbapConnection extends AbstractAbapConnection {
-    private tokenRefreshInProgress;
-    constructor(config: SapConfig, logger: ILogger, sessionStorage?: ISessionStorage, sessionId?: string);
+    constructor(config: SapConfig, logger?: ILogger | null, sessionId?: string);
     protected buildAuthorizationHeader(): string;
-    /**
-     * Refresh JWT token using refresh token
-     * @returns Promise that resolves when token is refreshed
-     */
-    refreshToken(): Promise<void>;
-    /**
-     * Check if token refresh is possible
-     */
-    canRefreshToken(): boolean;
     /**
      * Override connect to handle JWT token refresh on errors
      */
     connect(): Promise<void>;
     /**
-     * Override makeAdtRequest to handle JWT token refresh on 401/403
+     * Override makeAdtRequest to handle JWT auth errors
+     * Note: Token refresh is not supported in connection package - use auth-broker instead
      */
     makeAdtRequest(options: AbapRequestOptions): Promise<AxiosResponse>;
     /**
-     * Override fetchCsrfToken to handle JWT token refresh on 401/403 errors
-     * This ensures that token refresh works even when CSRF token is fetched during makeAdtRequest
+     * Override fetchCsrfToken to handle JWT auth errors
+     * Note: Token refresh is not supported in connection package - use auth-broker instead
      */
     protected fetchCsrfToken(url: string, retryCount?: number, retryDelay?: number): Promise<string>;
     private static validateConfig;

package/dist/connection/JwtAbapConnection.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"JwtAbapConnection.d.ts","sourceRoot":"","sources":["../../src/connection/JwtAbapConnection.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAC;AACnD,OAAO,EAAE,sBAAsB,EAAE,MAAM,6BAA6B,CAAC;AACrE,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,EAAE,OAAO,EAAE,eAAe,EAAgB,MAAM,cAAc,CAAC;AAEtE,OAAO,EAAc,aAAa,EAAE,MAAM,OAAO,CAAC;AAElD;;;GAGG;AACH,qBAAa,iBAAkB,SAAQ,sBAAsB;IAC3D,OAAO,CAAC,sBAAsB,CAAkB;gBAG9C,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,OAAO,EACf,cAAc,CAAC,EAAE,eAAe,EAChC,SAAS,CAAC,EAAE,MAAM;IAMpB,SAAS,CAAC,wBAAwB,IAAI,MAAM;IAS5C;;;OAGG;IACG,YAAY,IAAI,OAAO,CAAC,IAAI,CAAC;IA6EnC;;OAEG;IACH,eAAe,IAAI,OAAO;IAU1B;;OAEG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IA4G9B;;OAEG;IACG,cAAc,CAAC,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAAC,aAAa,CAAC;IAuGzE;;;OAGG;cACa,cAAc,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,SAAI,EAAE,UAAU,SAAO,GAAG,OAAO,CAAC,MAAM,CAAC;IA4C/F,OAAO,CAAC,MAAM,CAAC,cAAc;CAS9B"}
+{"version":3,"file":"JwtAbapConnection.d.ts","sourceRoot":"","sources":["../../src/connection/JwtAbapConnection.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAC;AACnD,OAAO,EAAE,sBAAsB,EAAE,MAAM,6BAA6B,CAAC;AACrE,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,EAAE,OAAO,EAAE,MAAM,cAAc,CAAC;AACvC,OAAO,EAAc,aAAa,EAAE,MAAM,OAAO,CAAC;AAElD;;;;GAIG;AACH,qBAAa,iBAAkB,SAAQ,sBAAsB;gBAGzD,MAAM,EAAE,SAAS,EACjB,MAAM,CAAC,EAAE,OAAO,GAAG,IAAI,EACvB,SAAS,CAAC,EAAE,MAAM;IAMpB,SAAS,CAAC,wBAAwB,IAAI,MAAM;IAW5C;;OAEG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IA2C9B;;;OAGG;IACG,cAAc,CAAC,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAAC,aAAa,CAAC;IAkCzE;;;OAGG;cACa,cAAc,CAAC,GAAG,EAAE,MAAM,EAAE,UAAU,SAAI,EAAE,UAAU,SAAO,GAAG,OAAO,CAAC,MAAM,CAAC;IA8B/F,OAAO,CAAC,MAAM,CAAC,cAAc;CAS9B"}