chore: remove debug logging from api-key-storage

The Uint8Array fix resolved the CI issue. Remove temporary debug
logging that was added to diagnose the problem.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: msujaws

src/lib/storage/api-key-storage.test.ts

@@ -1,53 +1,24 @@
-import { describe, it, expect, beforeEach, beforeAll } from 'vitest'
+import { describe, it, expect, beforeEach } from 'vitest'
 import { ApiKeyStorage, type Storage } from './api-key-storage'
 
-// Debug logging for CI - TODO: remove after debugging
-const DEBUG = true
-const log = (...args: unknown[]) => {
-  // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- DEBUG may change
-  if (DEBUG) {
-    console.log('[ApiKeyStorage Test]', ...args)
-  }
-}
-
 describe('ApiKeyStorage', () => {
   let storage: ApiKeyStorage
   let mockStorageData: Record<string, string>
   let mockStorage: Storage
   const testKeyMaterial = 'test-key-material-for-encryption'
 
-  beforeAll(() => {
-    // Log environment info - eslint disabled for debug logging
-    /* eslint-disable @typescript-eslint/no-unnecessary-condition */
-    log('=== Environment Info ===')
-    log('crypto available:', typeof crypto !== 'undefined')
-    log('crypto.subtle available:', typeof crypto?.subtle !== 'undefined')
-    log('globalThis.crypto available:', typeof globalThis.crypto !== 'undefined')
-    log('globalThis.crypto.subtle available:', typeof globalThis.crypto?.subtle !== 'undefined')
-    if (typeof crypto !== 'undefined' && crypto.subtle) {
-      log('crypto.subtle methods:', Object.keys(crypto.subtle))
-    }
-    log('========================')
-    /* eslint-enable @typescript-eslint/no-unnecessary-condition */
-  })
-
   beforeEach(() => {
     // Create in-memory storage mock (no global stubbing needed)
     mockStorageData = {}
     mockStorage = {
       getItem: (key: string) => {
-        const value = mockStorageData[key]
         // eslint-disable-next-line unicorn/no-null -- localStorage API returns null
-        const result = value ?? null
-        log(`mockStorage.getItem('${key}'):`, value ? `${value.slice(0, 50)}...` : 'null')
-        return result
+        return mockStorageData[key] ?? null
       },
       setItem: (key: string, value: string) => {
-        log(`mockStorage.setItem('${key}'):`, value.slice(0, 50) + '...')
         mockStorageData[key] = value
       },
       removeItem: (key: string) => {
-        log(`mockStorage.removeItem('${key}')`)
         // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
         delete mockStorageData[key]
       },
@@ -58,7 +29,6 @@ describe('ApiKeyStorage', () => {
       keyMaterial: testKeyMaterial,
       storage: mockStorage,
     })
-    log('Created new ApiKeyStorage instance with keyMaterial:', testKeyMaterial)
   })
 
   describe('saveApiKey', () => {
@@ -96,30 +66,10 @@ describe('ApiKeyStorage', () => {
     it('should retrieve and decrypt saved API key', async () => {
       const apiKey = 'test-api-key-456'
 
-      log('--- Test: should retrieve and decrypt saved API key ---')
-      log('Input apiKey:', apiKey)
-
-      try {
-        log('Calling saveApiKey...')
-        await storage.saveApiKey(apiKey)
-        log('saveApiKey completed, mockStorageData:', JSON.stringify(mockStorageData))
-      } catch (error) {
-        log('saveApiKey FAILED with error:', error)
-        throw error
-      }
+      await storage.saveApiKey(apiKey)
+      const retrieved = await storage.getApiKey()
 
-      try {
-        log('Calling getApiKey...')
-        const retrieved = await storage.getApiKey()
-        log('getApiKey returned:', retrieved)
-        log('Expected:', apiKey)
-        log('Match:', retrieved === apiKey)
-
-        expect(retrieved).toBe(apiKey)
-      } catch (error) {
-        log('getApiKey FAILED with error:', error)
-        throw error
-      }
+      expect(retrieved).toBe(apiKey)
     })
 
     it('should return undefined when no API key is stored', async () => {
@@ -195,34 +145,10 @@ describe('ApiKeyStorage', () => {
     it('should successfully decrypt after multiple save/retrieve cycles', async () => {
       const keys = ['key1', 'key2', 'key3']
 
-      log('--- Test: should successfully decrypt after multiple save/retrieve cycles ---')
-
       for (const key of keys) {
-        log(`\nCycle for key: '${key}'`)
-        try {
-          log('Calling saveApiKey...')
-          await storage.saveApiKey(key)
-          log(
-            'saveApiKey completed, stored value:',
-            mockStorageData['bugzilla_api_key']?.slice(0, 50),
-          )
-        } catch (error) {
-          log('saveApiKey FAILED:', error)
-          throw error
-        }
-
-        try {
-          log('Calling getApiKey...')
-          const retrieved = await storage.getApiKey()
-          log('getApiKey returned:', retrieved)
-          log('Expected:', key)
-          log('Match:', retrieved === key)
-
-          expect(retrieved).toBe(key)
-        } catch (error) {
-          log('getApiKey FAILED:', error)
-          throw error
-        }
+        await storage.saveApiKey(key)
+        const retrieved = await storage.getApiKey()
+        expect(retrieved).toBe(key)
       }
     })
   })

src/lib/storage/api-key-storage.ts

@@ -43,16 +43,6 @@ export class ApiKeyStorage {
   private async deriveCryptoKey(): Promise<CryptoKey> {
     // Use user agent as key material (stable across sessions), or override for testing
     const keySource = this.keyMaterialOverride ?? navigator.userAgent
-    // Debug logging - TODO: remove after debugging CI issue
-    /* eslint-disable @typescript-eslint/no-unnecessary-condition */
-    console.log(
-      '[ApiKeyStorage.deriveCryptoKey] keySource length:',
-      keySource?.length,
-      'using override:',
-      !!this.keyMaterialOverride,
-    )
-    console.log('[ApiKeyStorage.deriveCryptoKey] crypto.subtle available:', !!crypto?.subtle)
-    /* eslint-enable @typescript-eslint/no-unnecessary-condition */
     const keyMaterial = this.encoder.encode(keySource + 'bugzilla-kanban-salt')
 
     // Import as raw key material
@@ -63,10 +53,9 @@ export class ApiKeyStorage {
       false,
       ['deriveBits', 'deriveKey'],
     )
-    console.log('[ApiKeyStorage.deriveCryptoKey] importKey succeeded')
 
     // Derive actual encryption key
-    const derivedKey = await crypto.subtle.deriveKey(
+    return crypto.subtle.deriveKey(
       {
         name: 'PBKDF2',
         salt: this.encoder.encode('bugzilla-kanban'),
@@ -78,8 +67,6 @@ export class ApiKeyStorage {
       false,
       ['encrypt', 'decrypt'],
     )
-    console.log('[ApiKeyStorage.deriveCryptoKey] deriveKey succeeded')
-    return derivedKey
   }
 
   /**
@@ -153,16 +140,8 @@ export class ApiKeyStorage {
    * Save API key to localStorage (encrypted)
    */
   async saveApiKey(apiKey: string): Promise<void> {
-    console.log('[ApiKeyStorage.saveApiKey] saving key of length:', apiKey.length)
     const encrypted = await this.encrypt(apiKey)
-    console.log(
-      '[ApiKeyStorage.saveApiKey] encrypted, iv length:',
-      encrypted.iv.length,
-      'data length:',
-      encrypted.encryptedData.length,
-    )
     this.storage.setItem(STORAGE_KEY, JSON.stringify(encrypted))
-    console.log('[ApiKeyStorage.saveApiKey] saved to storage')
   }
 
   /**
@@ -171,27 +150,15 @@ export class ApiKeyStorage {
   async getApiKey(): Promise<string | undefined> {
     const stored = this.storage.getItem(STORAGE_KEY)
 
-    // Debug logging for CI
-    console.log('[ApiKeyStorage.getApiKey] stored value:', stored ? 'exists' : 'null')
-
     if (!stored) {
       return undefined
     }
 
     try {
       const encrypted = JSON.parse(stored) as EncryptedData
-      /* eslint-disable @typescript-eslint/no-unnecessary-condition */
-      console.log(
-        '[ApiKeyStorage.getApiKey] parsed encrypted data, iv length:',
-        encrypted.iv?.length,
-      )
-      const result = await this.decrypt(encrypted)
-      console.log('[ApiKeyStorage.getApiKey] decrypt succeeded, result length:', result?.length)
-      /* eslint-enable @typescript-eslint/no-unnecessary-condition */
-      return result
-    } catch (error) {
+      return await this.decrypt(encrypted)
+    } catch {
       // Invalid or corrupted data
-      console.error('[ApiKeyStorage.getApiKey] decrypt FAILED:', error)
       return undefined
     }
   }