fix: make AZURE_DEVOPS_AUTH_METHOD parameter case-insensitive

Author: Tiberriver256

README.md

@@ -120,7 +120,7 @@ Key environment variables include:
 
 | Variable                       | Description                                                     | Required           | Default            |
 | ------------------------------ | --------------------------------------------------------------- | ------------------ | ------------------ |
-| `AZURE_DEVOPS_AUTH_METHOD`     | Authentication method (`pat`, `azure-identity`, or `azure-cli`) | No                 | `azure-identity`   |
+| `AZURE_DEVOPS_AUTH_METHOD`     | Authentication method (`pat`, `azure-identity`, or `azure-cli`) - case-insensitive | No                 | `azure-identity`   |
 | `AZURE_DEVOPS_ORG`             | Azure DevOps organization name                                  | No                 | Extracted from URL |
 | `AZURE_DEVOPS_ORG_URL`         | Full URL to your Azure DevOps organization                      | Yes                | -                  |
 | `AZURE_DEVOPS_PAT`             | Personal Access Token (for PAT auth)                            | Only with PAT auth | -                  |

docs/authentication.md

@@ -135,7 +135,7 @@ Azure CLI authentication uses the `AzureCliCredential` class from the `@azure/id
 
 | Environment Variable           | Description                                                     | Required                     | Default          |
 | ------------------------------ | --------------------------------------------------------------- | ---------------------------- | ---------------- |
-| `AZURE_DEVOPS_AUTH_METHOD`     | Authentication method (`pat`, `azure-identity`, or `azure-cli`) | No                           | `azure-identity` |
+| `AZURE_DEVOPS_AUTH_METHOD`     | Authentication method (`pat`, `azure-identity`, or `azure-cli`) - case-insensitive | No                           | `azure-identity` |
 | `AZURE_DEVOPS_ORG_URL`         | Full URL to your Azure DevOps organization                      | Yes                          | -                |
 | `AZURE_DEVOPS_PAT`             | Personal Access Token (for PAT auth)                            | Only with PAT auth           | -                |
 | `AZURE_DEVOPS_DEFAULT_PROJECT` | Default project if none specified                               | No                           | -                |

src/index.spec.unit.ts

@@ -0,0 +1,100 @@
+import { normalizeAuthMethod } from './index';
+import { AuthenticationMethod } from './shared/auth/auth-factory';
+
+describe('index', () => {
+  describe('normalizeAuthMethod', () => {
+    it('should return AzureIdentity when authMethodStr is undefined', () => {
+      // Arrange
+      const authMethodStr = undefined;
+      
+      // Act
+      const result = normalizeAuthMethod(authMethodStr);
+      
+      // Assert
+      expect(result).toBe(AuthenticationMethod.AzureIdentity);
+    });
+
+    it('should return AzureIdentity when authMethodStr is empty', () => {
+      // Arrange
+      const authMethodStr = '';
+      
+      // Act
+      const result = normalizeAuthMethod(authMethodStr);
+      
+      // Assert
+      expect(result).toBe(AuthenticationMethod.AzureIdentity);
+    });
+
+    it('should handle PersonalAccessToken case-insensitively', () => {
+      // Arrange
+      const variations = [
+        'pat',
+        'PAT',
+        'Pat',
+        'pAt',
+        'paT',
+      ];
+      
+      // Act & Assert
+      variations.forEach(variant => {
+        expect(normalizeAuthMethod(variant)).toBe(AuthenticationMethod.PersonalAccessToken);
+      });
+    });
+
+    it('should handle AzureIdentity case-insensitively', () => {
+      // Arrange
+      const variations = [
+        'azure-identity',
+        'AZURE-IDENTITY',
+        'Azure-Identity',
+        'azure-Identity',
+        'Azure-identity',
+      ];
+      
+      // Act & Assert
+      variations.forEach(variant => {
+        expect(normalizeAuthMethod(variant)).toBe(AuthenticationMethod.AzureIdentity);
+      });
+    });
+
+    it('should handle AzureCli case-insensitively', () => {
+      // Arrange
+      const variations = [
+        'azure-cli',
+        'AZURE-CLI',
+        'Azure-Cli',
+        'azure-Cli',
+        'Azure-cli',
+      ];
+      
+      // Act & Assert
+      variations.forEach(variant => {
+        expect(normalizeAuthMethod(variant)).toBe(AuthenticationMethod.AzureCli);
+      });
+    });
+
+    it('should return AzureIdentity for unrecognized values', () => {
+      // Arrange
+      const unrecognized = [
+        'unknown',
+        'azureCli', // no hyphen
+        'azureIdentity', // no hyphen
+        'personal-access-token', // not matching enum value
+        'cli',
+        'identity',
+      ];
+      
+      // Act & Assert (mute stderr for warning messages)
+      const originalStderrWrite = process.stderr.write;
+      process.stderr.write = jest.fn();
+      
+      try {
+        unrecognized.forEach(value => {
+          expect(normalizeAuthMethod(value)).toBe(AuthenticationMethod.AzureIdentity);
+        });
+      } finally {
+        process.stderr.write = originalStderrWrite;
+      }
+    });
+  });
+}); 

src/index.ts

@@ -9,6 +9,35 @@ import dotenv from 'dotenv';
 import { AzureDevOpsConfig } from './shared/types';
 import { AuthenticationMethod } from './shared/auth/auth-factory';
 
+/**
+ * Normalize auth method string to a valid AuthenticationMethod enum value
+ * in a case-insensitive manner
+ * 
+ * @param authMethodStr The auth method string from environment variable
+ * @returns A valid AuthenticationMethod value
+ */
+export function normalizeAuthMethod(authMethodStr?: string): AuthenticationMethod {
+  if (!authMethodStr) {
+    return AuthenticationMethod.AzureIdentity; // Default
+  }
+
+  // Convert to lowercase for case-insensitive comparison
+  const normalizedMethod = authMethodStr.toLowerCase();
+
+  // Check against known enum values (as lowercase strings)
+  if (normalizedMethod === AuthenticationMethod.PersonalAccessToken.toLowerCase()) {
+    return AuthenticationMethod.PersonalAccessToken;
+  } else if (normalizedMethod === AuthenticationMethod.AzureIdentity.toLowerCase()) {
+    return AuthenticationMethod.AzureIdentity;
+  } else if (normalizedMethod === AuthenticationMethod.AzureCli.toLowerCase()) {
+    return AuthenticationMethod.AzureCli;
+  }
+
+  // If not recognized, log a warning and use the default
+  process.stderr.write(`WARNING: Unrecognized auth method '${authMethodStr}'. Using default (${AuthenticationMethod.AzureIdentity}).\n`);
+  return AuthenticationMethod.AzureIdentity;
+}
+
 // Load environment variables
 dotenv.config();
 
@@ -25,8 +54,7 @@ function getConfig(): AzureDevOpsConfig {
 
   return {
     organizationUrl: process.env.AZURE_DEVOPS_ORG_URL || '',
-    authMethod: (process.env.AZURE_DEVOPS_AUTH_METHOD ||
-      AuthenticationMethod.AzureIdentity) as AuthenticationMethod,
+    authMethod: normalizeAuthMethod(process.env.AZURE_DEVOPS_AUTH_METHOD),
     personalAccessToken: process.env.AZURE_DEVOPS_PAT,
     defaultProject: process.env.AZURE_DEVOPS_DEFAULT_PROJECT,
     apiVersion: process.env.AZURE_DEVOPS_API_VERSION,