Cleanup and improvements

Author: olesho

docker/Dockerfile

@@ -51,12 +51,8 @@ RUN git remote add upstream https://github.com/BrowserOperator/browser-operator-
 RUN git fetch upstream
 RUN git checkout upstream/main
 
-# Copy our local modifications into the container
-COPY front_end/panels/ai_chat/core/EnvironmentConfig.ts /workspace/devtools/devtools-frontend/front_end/panels/ai_chat/core/EnvironmentConfig.ts
-COPY front_end/panels/ai_chat/ui/SettingsDialog.ts /workspace/devtools/devtools-frontend/front_end/panels/ai_chat/ui/SettingsDialog.ts  
-COPY front_end/entrypoints/devtools_app/devtools_app.ts /workspace/devtools/devtools-frontend/front_end/entrypoints/devtools_app/devtools_app.ts
-
-# Build Browser Operator version with our modifications
+# Build Browser Operator version (using upstream code to avoid TypeScript compilation issues)
+# Runtime API key injection will be handled by the entrypoint script
 RUN npm run build
 
 # Production stage

front_end/panels/ai_chat/LLM/GroqProvider.ts

@@ -54,8 +54,8 @@ export class GroqProvider extends LLMBaseProvider {
    */
   private getApiKey(): string {
     // Constructor parameter (highest priority for backward compatibility)
-    if (this.getApiKey() && this.getApiKey().trim() !== '') {
-      return this.getApiKey().trim();
+    if (this.apiKey && this.apiKey.trim() !== '') {
+      return this.apiKey.trim();
     }
 
     // Use environment config which handles localStorage -> build-time -> empty fallback

front_end/panels/ai_chat/LLM/OpenRouterProvider.ts

@@ -651,8 +651,7 @@ export class OpenRouterProvider extends LLMBaseProvider {
 
     logger.debug('API key check:');
     logger.debug('- API key exists:', !!apiKey);
-    logger.debug('- API key length:', apiKey?.length || 0);
-    logger.debug('- API key prefix:', apiKey?.substring(0, 8) + '...' || 'none');
+    logger.debug('- API key presence:', apiKey ? '<redacted>' : 'none');
     logger.debug('- API key source:', source);
     logger.debug('- Build config available:', buildInfo.hasBuildConfig);
     logger.debug('- Build time:', buildInfo.buildTime);

front_end/panels/ai_chat/core/EnvironmentConfig.ts

@@ -261,7 +261,8 @@ export class EnvironmentConfig {
       return 'runtime';
     }
 
-    if (BUILD_CONFIG?.apiKeys?.[provider]) {
+    if (typeof BUILD_CONFIG?.apiKeys?.[provider] === 'string' && 
+        BUILD_CONFIG.apiKeys[provider].trim() !== '') {
       return 'build-time';
     }