@mcp-abap-adt/auth-broker 0.1.2 → 0.1.3

package/CHANGELOG.md

@@ -9,6 +9,30 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 Thank you to all contributors! See [CONTRIBUTORS.md](CONTRIBUTORS.md) for the complete list.
 
+## [0.1.3] - 2025-12-01
+
+### Added
+- **Configurable Log Levels** - Added log level control via environment variable `AUTH_LOG_LEVEL`
+  - `error` - only errors
+  - `warn` - errors and warnings
+  - `info` - errors, warnings, and info (default)
+  - `debug` - all messages
+  - Backward compatible: `DEBUG_AUTH_LOG=true` still works (sets level to debug)
+  - New `warn()` method in Logger interface for warning messages
+
+### Fixed
+- **Error Handling for Consumer** - Improved error handling to ensure consumer can distinguish different error types
+  - Service key missing error: throws `Error` with message containing "No authentication found" or "Service key file not found"
+  - Browser opening failed error: throws `Error` with message containing "Browser opening failed"
+  - Both errors are now properly thrown and can be caught by consumer in `catch` blocks
+  - Errors are distinct and can be programmatically handled differently
+
+### Changed
+- **Logger Implementation** - Enhanced logger with log level filtering
+  - All log methods now respect the configured log level
+  - `info()`, `debug()`, `error()`, and new `warn()` methods filter output based on `AUTH_LOG_LEVEL`
+  - Default log level is `info` (shows errors, warnings, and info messages)
+
 ## [0.1.2] - 2025-11-30
 
 ### Added

package/dist/browserAuth.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"browserAuth.d.ts","sourceRoot":"","sources":["../src/browserAuth.ts"],"names":[],"mappings":"AAAA;;GAEG;AAOH,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,MAAM,EAAiB,MAAM,UAAU,CAAC;AA4DjD;;;;;;GAMG;AACH,wBAAsB,gBAAgB,CACpC,UAAU,EAAE,UAAU,EACtB,OAAO,GAAE,MAAiB,EAC1B,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC;IAAE,WAAW,EAAE,MAAM,CAAC;IAAC,YAAY,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAqMzD"}
+{"version":3,"file":"browserAuth.d.ts","sourceRoot":"","sources":["../src/browserAuth.ts"],"names":[],"mappings":"AAAA;;GAEG;AAOH,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,MAAM,EAAiB,MAAM,UAAU,CAAC;AA4DjD;;;;;;GAMG;AACH,wBAAsB,gBAAgB,CACpC,UAAU,EAAE,UAAU,EACtB,OAAO,GAAE,MAAiB,EAC1B,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC;IAAE,WAAW,EAAE,MAAM,CAAC;IAAC,YAAY,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAyMzD"}

package/dist/browserAuth.js

@@ -284,7 +284,11 @@ async function startBrowserAuth(serviceKey, browser = 'system', logger) {
                     }
                 }
                 catch (error) {
-                    log.error(`❌ Failed to open browser: ${error.message}. Please open manually: ${authorizationUrl}`);
+                    // If browser cannot be opened, show URL and throw error for consumer to catch
+                    log.error(`❌ Failed to open browser: ${error?.message || String(error)}. Please open manually: ${authorizationUrl}`);
+                    log.browserUrl(authorizationUrl);
+                    // Throw error so consumer can distinguish this from "service key missing" error
+                    reject(new Error(`Browser opening failed for destination authentication. Please open manually: ${authorizationUrl}`));
                 }
             }
         });

package/dist/logger.d.ts

@@ -1,6 +1,15 @@
 /**
  * Logger interface and implementations for auth-broker package
  */
+/**
+ * Log levels
+ */
+export declare enum LogLevel {
+    ERROR = 0,
+    WARN = 1,
+    INFO = 2,
+    DEBUG = 3
+}
 /**
  * Logger interface - defines logging methods
  */
@@ -8,6 +17,7 @@ export interface Logger {
     info(message: string): void;
     debug(message: string): void;
     error(message: string): void;
+    warn(message: string): void;
     browserAuth(message: string): void;
     refresh(message: string): void;
     success(message: string): void;
@@ -26,4 +36,5 @@ export declare function success(message: string): void;
 export declare function browserUrl(url: string): void;
 export declare function browserOpening(): void;
 export declare function testSkip(message: string): void;
+export declare function warn(message: string): void;
 //# sourceMappingURL=logger.d.ts.map

package/dist/logger.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"logger.d.ts","sourceRoot":"","sources":["../src/logger.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH;;GAEG;AACH,MAAM,WAAW,MAAM;IACrB,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACnC,OAAO,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,OAAO,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,UAAU,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B,cAAc,IAAI,IAAI,CAAC;IACvB,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;CACjC;AA6GD,eAAO,MAAM,aAAa,EAAE,MAA4B,CAAC;AAGzD,eAAO,MAAM,UAAU,EAAE,MAAyB,CAAC;AAGnD,wBAAgB,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE1C;AAED,wBAAgB,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE3C;AAED,wBAAgB,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE3C;AAED,wBAAgB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAEjD;AAED,wBAAgB,OAAO,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE7C;AAED,wBAAgB,OAAO,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE7C;AAED,wBAAgB,UAAU,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAE5C;AAED,wBAAgB,cAAc,IAAI,IAAI,CAErC;AAED,wBAAgB,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE9C"}
+{"version":3,"file":"logger.d.ts","sourceRoot":"","sources":["../src/logger.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH;;GAEG;AACH,oBAAY,QAAQ;IAClB,KAAK,IAAI;IACT,IAAI,IAAI;IACR,IAAI,IAAI;IACR,KAAK,IAAI;CACV;AAED;;GAEG;AACH,MAAM,WAAW,MAAM;IACrB,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACnC,OAAO,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,OAAO,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,UAAU,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B,cAAc,IAAI,IAAI,CAAC;IACvB,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;CACjC;AAoJD,eAAO,MAAM,aAAa,EAAE,MAA4B,CAAC;AAGzD,eAAO,MAAM,UAAU,EAAE,MAAyB,CAAC;AAGnD,wBAAgB,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE1C;AAED,wBAAgB,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE3C;AAED,wBAAgB,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE3C;AAED,wBAAgB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAEjD;AAED,wBAAgB,OAAO,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE7C;AAED,wBAAgB,OAAO,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE7C;AAED,wBAAgB,UAAU,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAE5C;AAED,wBAAgB,cAAc,IAAI,IAAI,CAErC;AAED,wBAAgB,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE9C;AAED,wBAAgB,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE1C"}

package/dist/logger.js

@@ -3,7 +3,7 @@
  * Logger interface and implementations for auth-broker package
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.testLogger = exports.defaultLogger = void 0;
+exports.testLogger = exports.defaultLogger = exports.LogLevel = void 0;
 exports.info = info;
 exports.debug = debug;
 exports.error = error;
@@ -13,27 +13,70 @@ exports.success = success;
 exports.browserUrl = browserUrl;
 exports.browserOpening = browserOpening;
 exports.testSkip = testSkip;
+exports.warn = warn;
+/**
+ * Log levels
+ */
+var LogLevel;
+(function (LogLevel) {
+    LogLevel[LogLevel["ERROR"] = 0] = "ERROR";
+    LogLevel[LogLevel["WARN"] = 1] = "WARN";
+    LogLevel[LogLevel["INFO"] = 2] = "INFO";
+    LogLevel[LogLevel["DEBUG"] = 3] = "DEBUG";
+})(LogLevel || (exports.LogLevel = LogLevel = {}));
+/**
+ * Get log level from environment variable
+ * AUTH_LOG_LEVEL can be: error, warn, info, debug
+ * DEBUG_AUTH_LOG=true is also supported for backward compatibility (sets level to debug)
+ */
+function getLogLevel() {
+    const level = process.env.AUTH_LOG_LEVEL?.toLowerCase();
+    if (level === 'error')
+        return LogLevel.ERROR;
+    if (level === 'warn')
+        return LogLevel.WARN;
+    if (level === 'info')
+        return LogLevel.INFO;
+    if (level === 'debug')
+        return LogLevel.DEBUG;
+    // Backward compatibility
+    if (process.env.DEBUG_AUTH_LOG === 'true')
+        return LogLevel.DEBUG;
+    // Default: info level
+    return LogLevel.INFO;
+}
 /**
  * Default logger implementation
- * Controls output based on DEBUG_AUTH_LOG environment variable:
- * - If DEBUG_AUTH_LOG=true: shows debug messages
- * - Otherwise: only shows info messages (concise, one line)
+ * Controls output based on AUTH_LOG_LEVEL environment variable:
+ * - error: only errors
+ * - warn: errors and warnings
+ * - info: errors, warnings, and info (default)
+ * - debug: all messages
  */
 class DefaultLogger {
-    debugEnabled;
-    constructor(debugEnabled = process.env.DEBUG_AUTH_LOG === 'true') {
-        this.debugEnabled = debugEnabled;
+    logLevel;
+    constructor(logLevel) {
+        this.logLevel = logLevel ?? getLogLevel();
     }
     info(message) {
-        console.info(message);
+        if (this.logLevel >= LogLevel.INFO) {
+            console.info(message);
+        }
     }
     debug(message) {
-        if (this.debugEnabled) {
+        if (this.logLevel >= LogLevel.DEBUG) {
             console.debug(`[DEBUG] ${message}`);
         }
     }
     error(message) {
-        console.error(message);
+        if (this.logLevel >= LogLevel.ERROR) {
+            console.error(message);
+        }
+    }
+    warn(message) {
+        if (this.logLevel >= LogLevel.WARN) {
+            console.warn(`[WARN] ${message}`);
+        }
     }
     browserAuth(message) {
         this.info(`🌐 ${message}`);
@@ -58,23 +101,32 @@ class DefaultLogger {
 }
 /**
  * Test logger implementation
- * Always shows info messages, debug only if DEBUG_AUTH_LOG=true
+ * Uses same log levels as DefaultLogger
  */
 class TestLogger {
-    debugEnabled;
-    constructor(debugEnabled = process.env.DEBUG_AUTH_LOG === 'true') {
-        this.debugEnabled = debugEnabled;
+    logLevel;
+    constructor(logLevel) {
+        this.logLevel = logLevel ?? getLogLevel();
     }
     info(message) {
-        console.info(message);
+        if (this.logLevel >= LogLevel.INFO) {
+            console.info(message);
+        }
     }
     debug(message) {
-        if (this.debugEnabled) {
+        if (this.logLevel >= LogLevel.DEBUG) {
             console.info(`[DEBUG] ${message}`);
         }
     }
     error(message) {
-        console.error(message);
+        if (this.logLevel >= LogLevel.ERROR) {
+            console.error(message);
+        }
+    }
+    warn(message) {
+        if (this.logLevel >= LogLevel.WARN) {
+            console.warn(`[WARN] ${message}`);
+        }
     }
     browserAuth(message) {
         this.info(`🌐 ${message}`);
@@ -129,3 +181,6 @@ function browserOpening() {
 function testSkip(message) {
     exports.defaultLogger.testSkip(message);
 }
+function warn(message) {
+    exports.defaultLogger.warn(message);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mcp-abap-adt/auth-broker",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "JWT authentication broker for MCP ABAP ADT - manages tokens based on destination headers",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",