Merge branch 'feature/publish-script-updates' into 'develop'

Update publish scripts for UI validation improvements

See merge request genaiic-reusable-assets/engagement-artifacts/genaiic-idp-accelerator!328

Author: rstrahan

docs/deployment.md

@@ -36,6 +36,7 @@ You need to have the following packages installed on your computer:
 4. python 3.11 or later
 5. A local Docker daemon
 6. Python packages for publish.py: `pip install boto3 rich typer PyYAML botocore setuptools`
+7. **Node.js 18+** and **npm** (required for UI validation in publish script)
 
 For guidance on setting up a development environment, see:
 - [Development Environment Setup Guide on Linux](./setup-development-env-linux.md) 

publish.py

@@ -1110,6 +1110,43 @@ def compute_ui_hash(self):
         ui_dir = "src/ui"
         return self.get_directory_checksum(ui_dir)
 
+    def validate_ui_build(self):
+        """Validate UI build to catch ESLint/Prettier errors before packaging"""
+        try:
+            self.console.print("[bold cyan]🔍 VALIDATING UI build[/bold cyan]")
+            ui_dir = "src/ui"
+
+            if not os.path.exists(ui_dir):
+                self.console.print(
+                    "[yellow]No UI directory found, skipping UI validation[/yellow]"
+                )
+                return
+
+            # Run npm install first
+            self.log_verbose("Running npm install for UI dependencies...")
+            success, result = self.run_subprocess_with_logging(
+                ["npm", "install"], "UI npm install", ui_dir
+            )
+
+            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...")
+            success, result = self.run_subprocess_with_logging(
+                ["npm", "run", "build"], "UI build validation", ui_dir
+            )
+
+            if not success:
+                raise Exception("UI build validation failed")
+
+            self.console.print("[green]✅ UI build validation passed[/green]")
+
+        except Exception as e:
+            self.console.print("[red]❌ UI build validation failed:[/red]")
+            self.console.print(str(e), style="red", markup=False)
+            sys.exit(1)
+
     def package_ui(self):
         """Package UI source code"""
         ui_hash = self.compute_ui_hash()
@@ -1222,6 +1259,10 @@ def build_main_template(self, webui_zipfile, components_needing_rebuild):
             # Main template needs rebuilding, if any component needs rebuilding
             if components_needing_rebuild:
                 self.console.print("[yellow]Main template needs rebuilding[/yellow]")
+
+                # Validate UI build before rebuilding
+                self.validate_ui_build()
+
                 # Validate Python syntax in src directory before building
                 if not self._validate_python_syntax("src"):
                     raise Exception("Python syntax validation failed")

publish.sh

@@ -86,6 +86,37 @@ check_python_version() {
     exit 1
 }
 
+# Check if Node.js and npm are available for UI validation
+check_nodejs_dependencies() {
+    print_info "Checking Node.js dependencies for UI validation..."
+    
+    # Check Node.js
+    if ! command -v node >/dev/null 2>&1; then
+        print_error "Node.js not found but is required for UI build validation"
+        print_info "Install Node.js 18+ from: https://nodejs.org/"
+        exit 1
+    fi
+    
+    # Check npm
+    if ! command -v npm >/dev/null 2>&1; then
+        print_error "npm not found but is required for UI build validation"
+        print_info "npm is typically installed with Node.js"
+        exit 1
+    fi
+    
+    # Check Node.js version (require 18+)
+    node_version=$(node --version 2>/dev/null | sed 's/v//')
+    node_major=$(echo "$node_version" | cut -d'.' -f1)
+    
+    if [[ "$node_major" -lt 18 ]]; then
+        print_error "Node.js $node_version found, but 18+ is required for UI validation"
+        print_info "Please upgrade Node.js to version 18 or later"
+        exit 1
+    else
+        print_success "Found Node.js $node_version and npm $(npm --version)"
+    fi
+}
+
 # Check if required packages are installed and install them if missing
 check_and_install_packages() {
     print_info "Checking required Python packages..."
@@ -169,6 +200,9 @@ main() {
     # Check Python version
     check_python_version
 
+    # Check Node.js dependencies for UI validation
+    check_nodejs_dependencies
+    
     # Check and install required packages
     check_and_install_packages