add warning when conda activation fails

Author: aktech

cmd/jhub-app-proxy/main.go

@@ -135,6 +135,12 @@ func run(cfg *config.Config) error {
 		return fmt.Errorf("failed to create process manager: %w", err)
 	}
 
+	// Add conda warning to log buffer if there was a conda activation failure
+	// This ensures the warning appears in the interim UI logs
+	if condaWarning := cmdBuilder.GetCondaWarning(); condaWarning != "" {
+		mgr.AddErrorLog(condaWarning)
+	}
+
 	// Create and start HTTP server
 	subprocessURL := fmt.Sprintf("http://127.0.0.1:%d", subprocessPort)
 	srv, err := server.New(server.Config{

pkg/command/builder.go

@@ -12,7 +12,8 @@ import (
 
 // Builder helps construct and manipulate commands for subprocess execution
 type Builder struct {
-	logger *logger.Logger
+	logger         *logger.Logger
+	condaWarning   string // Stores conda activation warning if any
 }
 
 // NewBuilder creates a new command builder
@@ -31,16 +32,29 @@ func (b *Builder) Build(command []string, condaEnv string) ([]string, error) {
 	// Apply conda activation if specified
 	if condaEnv != "" {
 		condaMgr := conda.NewManager(b.logger)
-		var err error
-		command, err = condaMgr.BuildActivationCommand(condaEnv, command)
+		activatedCommand, err := condaMgr.BuildActivationCommand(condaEnv, command)
 		if err != nil {
-			return nil, fmt.Errorf("failed to build conda activation: %w", err)
+			// Store warning message for later display in interim UI
+			b.condaWarning = fmt.Sprintf("WARNING: Conda environment activation failed: %s. Running command without conda activation.", err.Error())
+
+			// Log warning but continue with original command without conda activation
+			b.logger.Warn("conda environment activation failed, running command without conda activation",
+				"conda_env", condaEnv,
+				"error", err.Error())
+			// Return original command without conda activation
+			return command, nil
 		}
+		command = activatedCommand
 	}
 
 	return command, nil
 }
 
+// GetCondaWarning returns the conda activation warning message if any
+func (b *Builder) GetCondaWarning() string {
+	return b.condaWarning
+}
+
 // SubstitutePort replaces jhsingle-native-proxy style placeholders in command arguments
 // Handles: {port} → actual port, {-} → -, {--} → --, and strips surrounding quotes
 func SubstitutePort(command []string, allocatedPort int) []string {

pkg/ui/logs.js

@@ -139,7 +139,7 @@ async function checkAppStatus() {
         }
 
         if (data.version) {
-            versionText.textContent = 'jhub-app-proxy v' + data.version;
+            versionText.textContent = 'jhub-app-proxy ' + data.version;
         } else {
             versionText.textContent = 'jhub-app-proxy';
         }