Add real-time progress display for npm install in publish.py

Author: Taniya Mathur

publish.py

@@ -116,11 +116,64 @@ def log_error_details(self, component, error_output):
                 f"[red]❌ {component} build failed (use --verbose for details)[/red]"
             )
 
-    def run_subprocess_with_logging(self, cmd, component_name, cwd=None):
+    def run_subprocess_with_logging(self, cmd, component_name, cwd=None, realtime=False):
         """Run subprocess with standardized logging"""
-        result = subprocess.run(cmd, capture_output=True, text=True, cwd=cwd)
-        if result.returncode != 0:
-            error_msg = f"""Command failed: {" ".join(cmd)}
+        if realtime:
+            # Real-time output for long-running processes like npm install
+            self.console.print(f"[cyan]Running: {' '.join(cmd)}[/cyan]")
+            
+            try:
+                process = subprocess.Popen(
+                    cmd,
+                    stdout=subprocess.PIPE,
+                    stderr=subprocess.STDOUT,
+                    text=True,
+                    cwd=cwd,
+                    bufsize=1,
+                    universal_newlines=True
+                )
+                
+                output_lines = []
+                while True:
+                    output = process.stdout.readline()
+                    if output == '' and process.poll() is not None:
+                        break
+                    if output:
+                        line = output.strip()
+                        output_lines.append(line)
+                        # Show progress for npm commands
+                        if 'npm' in ' '.join(cmd):
+                            if any(keyword in line.lower() for keyword in ['downloading', 'installing', 'added', 'updated', 'audited']):
+                                self.console.print(f"[dim]  {line}[/dim]")
+                            elif 'warn' in line.lower():
+                                self.console.print(f"[yellow]  {line}[/yellow]")
+                            elif 'error' in line.lower():
+                                self.console.print(f"[red]  {line}[/red]")
+                
+                return_code = process.poll()
+                
+                if return_code != 0:
+                    error_msg = f"""Command failed: {" ".join(cmd)}
+Working directory: {cwd or os.getcwd()}
+Return code: {return_code}
+
+OUTPUT:
+{chr(10).join(output_lines)}"""
+                    print(error_msg)
+                    self.log_error_details(component_name, error_msg)
+                    return False, error_msg
+                
+                return True, None  # Success, no result object needed for real-time
+                
+            except Exception as e:
+                error_msg = f"Failed to execute command: {' '.join(cmd)}\nError: {str(e)}"
+                self.log_error_details(component_name, error_msg)
+                return False, error_msg
+        else:
+            # Original behavior - capture all output
+            result = subprocess.run(cmd, capture_output=True, text=True, cwd=cwd)
+            if result.returncode != 0:
+                error_msg = f"""Command failed: {" ".join(cmd)}
 Working directory: {cwd or os.getcwd()}
 Return code: {result.returncode}
 
@@ -129,10 +182,10 @@ def run_subprocess_with_logging(self, cmd, component_name, cwd=None):
 
 STDERR:
 {result.stderr}"""
-            print(error_msg)
-            self.log_error_details(component_name, error_msg)
-            return False, error_msg
-        return True, result
+                print(error_msg)
+                self.log_error_details(component_name, error_msg)
+                return False, error_msg
+            return True, result
 
     def print_error_summary(self):
         """Print summary of all build errors"""
@@ -1123,18 +1176,18 @@ def validate_ui_build(self):
                 return
 
             # Run npm install first
-            self.log_verbose("Running npm install for UI dependencies...")
+            self.console.print("[cyan]📦 Installing UI dependencies (this may take a while)...[/cyan]")
             success, result = self.run_subprocess_with_logging(
-                ["npm", "install"], "UI npm install", ui_dir
+                ["npm", "install"], "UI npm install", ui_dir, realtime=True
             )
 
             if not success:
                 raise Exception("npm install failed")
 
             # Run npm run build to validate ESLint/Prettier
-            self.log_verbose("Running npm run build for UI validation...")
+            self.console.print("[cyan]🔨 Building UI (validating ESLint/Prettier)...[/cyan]")
             success, result = self.run_subprocess_with_logging(
-                ["npm", "run", "build"], "UI build validation", ui_dir
+                ["npm", "run", "build"], "UI build validation", ui_dir, realtime=True
             )
 
             if not success: