Fix GitHub workflow configuration mismatches and add prevention guidelines

.github/copilot-instructions.md

@@ -1299,6 +1299,242 @@ When extending excelcli with Copilot:
 - Practical GitHub Copilot integration examples essential for adoption
 - Resource-based architecture must be explained vs granular approach
 
+## 🔧 **CRITICAL: GitHub Workflows Configuration Management**
+
+### **Keep Workflows in Sync with Project Configuration**
+
+**ALWAYS update GitHub workflows when making configuration changes.** This prevents build and deployment failures.
+
+#### **Configuration Points That Require Workflow Updates**
+
+When making ANY of these changes, you MUST update all relevant workflows:
+
+1. **.NET SDK Version Changes**
+   ```yaml
+   # If you change global.json or .csproj target frameworks:
+   # UPDATE ALL workflows that use actions/setup-dotnet@v4
+
+   - name: Setup .NET
+     uses: actions/setup-dotnet@v4
+     with:
+       dotnet-version: 10.0.x  # ⚠️ MUST match global.json and project files
+   ```
+
+   **Files to check:**
+   - `.github/workflows/build-cli.yml`
+   - `.github/workflows/build-mcp-server.yml`
+   - `.github/workflows/release-cli.yml`
+   - `.github/workflows/release-mcp-server.yml`
+   - `.github/workflows/codeql.yml`
+   - `.github/workflows/publish-nuget.yml`
+
+2. **Assembly/Package Name Changes**
+   ```yaml
+   # If you change AssemblyName or PackageId in .csproj:
+   # UPDATE ALL workflow references to executables and packages
+
+   # Example: Build verification
+   if (Test-Path "src/ExcelMcp.McpServer/bin/Release/net10.0/Sbroenne.ExcelMcp.McpServer.exe")
+
+   # Example: NuGet package operations
+   $packagePath = "nupkg/Sbroenne.ExcelMcp.McpServer.$version.nupkg"
+   dotnet tool install --global Sbroenne.ExcelMcp.McpServer
+   ```
+
+   **Files to check:**
+   - `.github/workflows/build-mcp-server.yml` - Executable name checks
+   - `.github/workflows/publish-nuget.yml` - Package names
+   - `.github/workflows/release-mcp-server.yml` - Installation instructions
+   - `.github/workflows/release-cli.yml` - DLL references
+
+3. **Runtime Requirements Documentation**
+   ```powershell
+   # If you change target framework (net8.0 → net10.0):
+   # UPDATE ALL release notes that mention runtime requirements
+
+   $releaseNotes += "- .NET 10.0 runtime`n"  # ⚠️ MUST match project target
+   ```
+
+   **Files to check:**
+   - `.github/workflows/release-cli.yml` - Quick start and release notes
+   - `.github/workflows/release-mcp-server.yml` - Installation requirements
+
+4. **Project Structure Changes**
+   ```yaml
+   # If you rename projects or move directories:
+   # UPDATE path filters and build commands
+
+   paths:
+     - 'src/ExcelMcp.CLI/**'  # ⚠️ MUST match actual directory structure
+
+   run: dotnet build src/ExcelMcp.CLI/ExcelMcp.CLI.csproj  # ⚠️ MUST be valid path
+   ```
+
+### **Workflow Validation Checklist**
+
+Before committing configuration changes, run this validation:
+
+```powershell
+# 1. Check .NET version consistency
+$globalJsonVersion = (Get-Content global.json | ConvertFrom-Json).sdk.version
+$workflowVersions = Select-String -Path .github/workflows/*.yml -Pattern "dotnet-version:" -Context 0,0
+Write-Output "global.json: $globalJsonVersion"
+Write-Output "Workflows:"
+$workflowVersions
+
+# 2. Check assembly names match
+$assemblyNames = Select-String -Path src/**/*.csproj -Pattern "<AssemblyName>(.*)</AssemblyName>"
+$workflowExeRefs = Select-String -Path .github/workflows/*.yml -Pattern "\.exe" -Context 1,0
+Write-Output "Assembly Names in .csproj:"
+$assemblyNames
+Write-Output "Executable references in workflows:"
+$workflowExeRefs
+
+# 3. Check package IDs match
+$packageIds = Select-String -Path src/**/*.csproj -Pattern "<PackageId>(.*)</PackageId>"
+$workflowPkgRefs = Select-String -Path .github/workflows/*.yml -Pattern "\.nupkg|tool install" -Context 1,0
+Write-Output "Package IDs in .csproj:"
+$packageIds
+Write-Output "Package references in workflows:"
+$workflowPkgRefs
+```
+
+### **Automated Workflow Validation (Future Enhancement)**
+
+Create `.github/scripts/validate-workflows.ps1`:
+
+```powershell
+#!/usr/bin/env pwsh
+# Validates workflow configurations match project files
+
+param(
+    [switch]$Fix  # Auto-fix issues if possible
+)
+
+$errors = @()
+
+# Check .NET versions
+$globalJson = Get-Content global.json | ConvertFrom-Json
+$expectedVersion = $globalJson.sdk.version -replace '^\d+\.(\d+)\..*', '$1.0.x'
+
+$workflows = Get-ChildItem .github/workflows/*.yml
+foreach ($workflow in $workflows) {
+    $content = Get-Content $workflow.FullName -Raw
+    if ($content -match 'dotnet-version:\s*(\d+\.\d+\.x)') {
+        $workflowVersion = $Matches[1]
+        if ($workflowVersion -ne $expectedVersion) {
+            $errors += "❌ $($workflow.Name): Uses .NET $workflowVersion but should be $expectedVersion"
+        }
+    }
+}
+
+# Check assembly names
+$projects = Get-ChildItem src/**/*.csproj
+foreach ($project in $projects) {
+    [xml]$csproj = Get-Content $project.FullName
+    $assemblyName = $csproj.Project.PropertyGroup.AssemblyName
+
+    if ($assemblyName) {
+        # Check if workflows reference this assembly
+        $exeName = "$assemblyName.exe"
+        $workflowRefs = Select-String -Path .github/workflows/*.yml -Pattern $exeName -Quiet
+
+        if (-not $workflowRefs -and $project.Name -match "McpServer") {
+            $errors += "⚠️ Assembly $assemblyName not found in workflows"
+        }
+    }
+}
+
+# Report results
+if ($errors.Count -eq 0) {
+    Write-Output "✅ All workflow configurations are valid!"
+    exit 0
+} else {
+    Write-Output "❌ Found $($errors.Count) workflow configuration issues:"
+    $errors | ForEach-Object { Write-Output "  $_" }
+    exit 1
+}
+```
+
+### **When to Run Validation**
+
+Run workflow validation:
+- ✅ Before creating PR with configuration changes
+- ✅ After upgrading .NET SDK version
+- ✅ After renaming projects or assemblies
+- ✅ After changing package IDs or branding
+- ✅ As part of pre-commit hooks (recommended)
+
+### **Common Workflow Configuration Mistakes to Prevent**
+
+❌ **Don't:**
+- Change .NET version in code without updating workflows
+- Rename assemblies without updating executable checks
+- Change package IDs without updating install commands
+- Update target frameworks without updating runtime requirements
+- Assume workflows will "just work" after configuration changes
+
+✅ **Always:**
+- Update ALL affected workflows when changing configuration
+- Validate executable names match AssemblyName properties
+- Verify package IDs match PackageId properties
+- Keep runtime requirement docs in sync with target frameworks
+- Test workflows locally with `act` or similar tools before pushing
+
+### **Workflow Update Template**
+
+When making configuration changes, use this checklist:
+
+```markdown
+## Configuration Change: [Brief Description]
+
+### Changes Made:
+- [ ] Updated global.json/.csproj files
+- [ ] Updated all workflow .NET versions
+- [ ] Updated executable name references
+- [ ] Updated package ID references
+- [ ] Updated runtime requirement documentation
+- [ ] Tested workflow locally (if possible)
+- [ ] Verified all path filters still match
+- [ ] Updated this checklist in PR description
+
+### Workflows Reviewed:
+- [ ] build-cli.yml
+- [ ] build-mcp-server.yml
+- [ ] release-cli.yml
+- [ ] release-mcp-server.yml
+- [ ] codeql.yml
+- [ ] publish-nuget.yml
+- [ ] dependency-review.yml (if applicable)
+```
+
+### **Integration with CI/CD**
+
+Add workflow validation to CI pipeline:
+
+```yaml
+# .github/workflows/validate-config.yml
+name: Validate Configuration
+
+on:
+  pull_request:
+    paths:
+      - 'src/**/*.csproj'
+      - 'global.json'
+      - 'Directory.*.props'
+      - '.github/workflows/**'
+
+jobs:
+  validate:
+    runs-on: windows-latest
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Validate Workflows
+      run: .github/scripts/validate-workflows.ps1
+      shell: pwsh
+```
+
 ## �🚨 **CRITICAL: Development Workflow Requirements**
 
 ### **All Changes Must Use Pull Requests**

.github/workflows/build-mcp-server.yml

@@ -46,9 +46,9 @@ jobs:
     - name: Verify MCP Server build
       run: |
         # Check MCP Server executable
-        if (Test-Path "src/ExcelMcp.McpServer/bin/Release/net10.0/ExcelMcp.McpServer.exe") {
-          Write-Output "✅ ExcelMcp.McpServer.exe built successfully"
-          $mcpVersion = (Get-Item "src/ExcelMcp.McpServer/bin/Release/net10.0/ExcelMcp.McpServer.exe").VersionInfo.FileVersion
+        if (Test-Path "src/ExcelMcp.McpServer/bin/Release/net10.0/Sbroenne.ExcelMcp.McpServer.exe") {
+          Write-Output "✅ Sbroenne.ExcelMcp.McpServer.exe built successfully"
+          $mcpVersion = (Get-Item "src/ExcelMcp.McpServer/bin/Release/net10.0/Sbroenne.ExcelMcp.McpServer.exe").VersionInfo.FileVersion
           Write-Output "📦 MCP Server Version: $mcpVersion"
 
           # Check for MCP server.json configuration
@@ -58,7 +58,7 @@ jobs:
             Write-Warning "⚠️ MCP server.json configuration not found"
           }
         } else {
-          Write-Error "❌ ExcelMcp.McpServer.exe not found"
+          Write-Error "❌ Sbroenne.ExcelMcp.McpServer.exe not found"
           exit 1
         }
 

.github/workflows/codeql.yml

@@ -56,7 +56,7 @@ jobs:
     - name: Setup .NET
       uses: actions/setup-dotnet@v4
       with:
-        dotnet-version: 8.0.x
+        dotnet-version: 10.0.x
 
     # Initializes the CodeQL tools for scanning
     - name: Initialize CodeQL

.github/workflows/publish-nuget.yml

@@ -17,7 +17,7 @@ jobs:
     - name: Setup .NET
       uses: actions/setup-dotnet@v4
       with:
-        dotnet-version: 8.0.x
+        dotnet-version: 10.0.x
 
     - name: Extract version from tag
       id: version
@@ -58,13 +58,13 @@ jobs:
           --output ./nupkg `
           /p:PackageVersion=$version
 
-        Write-Output "📦 Created NuGet package: ExcelMcp.McpServer.$version.nupkg"
+        Write-Output "📦 Created NuGet package: Sbroenne.ExcelMcp.McpServer.$version.nupkg"
       shell: pwsh
 
     - name: Verify package contents
       run: |
         $version = "${{ steps.version.outputs.version }}"
-        $packagePath = "nupkg/ExcelMcp.McpServer.$version.nupkg"
+        $packagePath = "nupkg/Sbroenne.ExcelMcp.McpServer.$version.nupkg"
 
         if (Test-Path $packagePath) {
           Write-Output "✅ Package created successfully"
@@ -78,14 +78,14 @@ jobs:
     - name: Publish to NuGet.org
       run: |
         $version = "${{ steps.version.outputs.version }}"
-        $packagePath = "nupkg/ExcelMcp.McpServer.$version.nupkg"
+        $packagePath = "nupkg/Sbroenne.ExcelMcp.McpServer.$version.nupkg"
 
         dotnet nuget push $packagePath `
           --source https://api.nuget.org/v3/index.json `
           --skip-duplicate
 
-        Write-Output "🚀 Published ExcelMcp.McpServer.$version to NuGet.org"
-        Write-Output "🔗 Package will be available at: https://www.nuget.org/packages/ExcelMcp.McpServer/$version"
+        Write-Output "🚀 Published Sbroenne.ExcelMcp.McpServer.$version to NuGet.org"
+        Write-Output "🔗 Package will be available at: https://www.nuget.org/packages/Sbroenne.ExcelMcp.McpServer/$version"
       shell: pwsh
 
     - name: Upload package artifact

.github/workflows/release-cli.yml

@@ -19,7 +19,7 @@ jobs:
     - name: Setup .NET
       uses: actions/setup-dotnet@v4
       with:
-        dotnet-version: 8.0.x
+        dotnet-version: 10.0.x
 
     - name: Update CLI Version
       run: |
@@ -100,7 +100,7 @@ jobs:
         $quickStartContent += "- **GitHub**: https://github.com/sbroenne/mcp-server-excel`n`n"
         $quickStartContent += "## Requirements`n`n"
         $quickStartContent += "- Windows OS with Microsoft Excel installed`n"
-        $quickStartContent += "- .NET 8.0 runtime`n"
+        $quickStartContent += "- .NET 10.0 runtime`n"
         $quickStartContent += "- Excel 2016+ (for COM interop)`n`n"
         $quickStartContent += "## Features`n`n"
         $quickStartContent += "- 40+ Excel automation commands`n"
@@ -162,7 +162,7 @@ jobs:
         $releaseNotes += "``````n`n"
         $releaseNotes += "### Requirements`n"
         $releaseNotes += "- Windows OS with Microsoft Excel installed`n"
-        $releaseNotes += "- .NET 8.0 runtime`n"
+        $releaseNotes += "- .NET 10.0 runtime`n"
         $releaseNotes += "- Excel 2016+ (for COM interop)`n`n"
         $releaseNotes += "### Documentation`n"
         $releaseNotes += "- Complete Command Reference: COMMANDS.md in package`n"

.github/workflows/release-mcp-server.yml

@@ -94,7 +94,7 @@ jobs:
         $readmeContent += "- Excel Development Focus - Power Query, VBA, worksheets`n`n"
         $readmeContent += "## Requirements`n`n"
         $readmeContent += "- Windows OS with Microsoft Excel installed`n"
-        $readmeContent += "- .NET 8.0 runtime`n"
+        $readmeContent += "- .NET 10.0 runtime`n"
         $readmeContent += "- Excel 2016+ (for COM interop)`n`n"
         $readmeContent += "## License`n`n"
         $readmeContent += "MIT License - see LICENSE file for details.`n"
@@ -134,7 +134,7 @@ jobs:
         $releaseNotes += "### Installation`n`n"
         $releaseNotes += "**Option 1: .NET Tool (Recommended)**`n"
         $releaseNotes += "``````powershell`n"
-        $releaseNotes += "dotnet tool install --global ExcelMcp.McpServer --version $version`n"
+        $releaseNotes += "dotnet tool install --global Sbroenne.ExcelMcp.McpServer --version $version`n"
         $releaseNotes += "mcp-excel`n"
         $releaseNotes += "``````n`n"
         $releaseNotes += "**Option 2: Download Binary**`n"
@@ -149,7 +149,7 @@ jobs:
         $releaseNotes += "- excel_vba - VBA script management (list, export, import, update, run, delete)`n`n"
         $releaseNotes += "### Requirements`n"
         $releaseNotes += "- Windows OS with Microsoft Excel installed`n"
-        $releaseNotes += "- .NET 8.0 runtime`n"
+        $releaseNotes += "- .NET 10.0 runtime`n"
         $releaseNotes += "- Excel 2016+ (for COM interop)`n`n"
         $releaseNotes += "### Documentation`n"
         $releaseNotes += "- Configuration Guide: See README.md in package`n"