Cleanup #3
Merged
Cleanup #3
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… and automation - Implemented ExcelDiagnostics class for comprehensive error reporting, including detailed context for Excel operations. - Added ExcelHelper class to manage Excel COM automation with improved resource management and security checks. - Created project file for ExcelMcp.CLI with necessary dependencies and metadata. - Developed main program logic for command-line interface, including argument validation and command handling for various Excel operations. - Included detailed help and usage instructions for users, along with examples and requirements.
- Implemented integration tests for ScriptCommands and SheetCommands to validate VBA script operations and worksheet manipulations. - Created unit tests for argument validation and Excel file checks in the ExcelMcp CLI. - Added project files for test projects with necessary dependencies. - Developed integration tests for ExcelMcp Server tools to ensure proper functionality of resource-based commands. - Included cleanup logic in tests to manage temporary files and directories.
…ble name, and implement .NET best practices (#1) * Initial plan * Apply Sbroenne prefix to all project namespaces and solution file - Rename ExcelMcp.sln to Sbroenne.ExcelMcp.sln - Update all CLI namespaces from ExcelMcp.* to Sbroenne.ExcelMcp.CLI.* - Add AssemblyName and RootNamespace properties to all projects - Fix using statements in CLI to reference correct namespaces - Update Program.cs to use CLI Commands instead of Core Commands Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Fix old repository and product naming references - Update Directory.Build.props: Product name from ExcelCLI to Sbroenne.ExcelMcp - Update Directory.Build.props: Repository URLs from ExcelCLI to mcp-server-excel - Update Program.cs: Repository links and branding from ExcelCLI to ExcelMcp.CLI - Update documentation: Replace ExcelCLI with Sbroenne.ExcelMcp/ExcelMcp.CLI - Update workflows: Fix executable name and help text validation - Update CONTRIBUTING, SECURITY, CLI, AUTHOR, COPILOT docs Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Complete project cleanup and NuGet best practices - Remove legacy ExcelMcp.Tests folder (not in solution) - Update TEST_GUIDE.md with correct test structure - Add PackageReadmeFile to CLI project (includes CLI.md) - Fix .NET version reference from 8.0 to 10.0 in Program.cs - Fix release workflow paths: net8.0 → net10.0 - Fix release workflow DLL names with Sbroenne prefix - Ensure all NuGet metadata follows best practices Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Add comprehensive naming review summary documentation - Document all naming convention changes - Provide migration notes for developers - List all best practices compliance items - Include before/after examples - Summary statistics of changes made Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Simplify CLI executable name to 'excelcli.exe' - Change AssemblyName from Sbroenne.ExcelMcp.CLI to excelcli - Update build-cli.yml workflow for new executable name - Update release-cli.yml with excelcli.exe references - Update all documentation (INSTALLATION, CLI, DEVELOPMENT, etc.) - Update Program.cs usage examples to use 'excelcli' command - Update NAMING_REVIEW_SUMMARY.md with new executable name Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Apply .NET best practices improvements - Add XML documentation generation to Core library - Add version information to Core library (2.0.0) - Enable deterministic builds for reproducible binaries - Enable package validation for NuGet packages (CLI and MCP Server) - Create comprehensive DOTNET_BEST_PRACTICES_REVIEW.md document All changes maintain backward compatibility and enhance professional quality. Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Update all documentation for simplified CLI executable name - Update CLI.md: Change zip file name and path references to excelcli - Update COMMANDS.md: All 80+ command examples now use 'excelcli' command - Update excel-powerquery-vba-copilot-instructions.md: All command examples updated - Update .github/copilot-instructions.md: All command examples and workflows updated All documentation now consistently references 'excelcli' as the executable name (156 command references updated). Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Refocus excel-powerquery-vba-copilot-instructions.md on MCP server - Changed from CLI command examples to MCP server conversational workflows - Updated title and introduction to focus on AI-assisted development - Replaced CLI commands with 6 resource-based MCP tools (excel_file, excel_powerquery, excel_worksheet, excel_parameter, excel_cell, excel_vba) - Added conversational workflow examples (Power Query refactoring, VBA enhancement, etc.) - Updated use cases to emphasize AI development vs scripted automation - Added MCP server installation and configuration instructions - Added example prompts for GitHub Copilot integration - Clarified distinction between MCP server (development) and CLI (automation) The file now properly guides users on using the MCP server for AI-powered Excel development workflows. Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Remove intermediary documentation files - Remove NAMING_REVIEW_SUMMARY.md (documented changes during PR, no longer needed) - Remove DOTNET_BEST_PRACTICES_REVIEW.md (review completed, current state documented elsewhere) - Remove .github/BRANCH_PROTECTION.md (setup guide, not needed for current state) - Remove .github/CODEQL_SETUP_FIX.md (troubleshooting doc, issue resolved) - Remove .github/SECURITY_SETUP.md (setup guide, security already configured) All removed files documented the transition process but aren't needed to document the current project state. Essential documentation remains: - README.md (project overview and quick start) - SECURITY.md (security policy) - docs/CONTRIBUTING.md (contribution guidelines) - docs/CLI.md, docs/COMMANDS.md (usage documentation) - .github/copilot-instructions.md (AI assistant integration) Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com>
…lines (#2) * Initial plan * Fix GitHub workflow configuration issues - .NET version and package naming Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> * Add GitHub workflows configuration management guidelines to prevent sync issues Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sbroenne <3026464+sbroenne@users.noreply.github.com>
Dependency ReviewThe following issues were found:
License Issuestests/ExcelMcp.CLI.Tests/ExcelMcp.CLI.Tests.csproj
tests/ExcelMcp.McpServer.Tests/ExcelMcp.McpServer.Tests.csproj
OpenSSF ScorecardScorecard details
Scanned Files
|
added 2 commits
October 19, 2025 13:47
…ove test output messages, and adjust dependency review settings
…port and documentation clarity
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Initial plan
Fix GitHub workflow configuration issues - .NET version and package naming
Co-authored-by: sbroenne 3026464+sbroenne@users.noreply.github.com
Co-authored-by: sbroenne 3026464+sbroenne@users.noreply.github.com
Co-authored-by: copilot-swe-agent[bot] 198982749+Copilot@users.noreply.github.com
Co-authored-by: sbroenne 3026464+sbroenne@users.noreply.github.com