mirror of
https://github.com/sbroenne/mcp-server-excel.git
synced 2026-09-19 07:53:08 +08:00
Stefan Broenner
11de41ac7f
* chore: refresh coding agent setup Remove the unused Squad configuration and align Copilot instructions with the current generated architecture. Add deterministic Windows cloud-agent dependency setup. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: b6c52095-fe7c-4482-8789-f490af2ffad3 * docs: update generated surface checklist Replace obsolete hand-written MCP wiring steps in the pull request template with the current generator and parity checks. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: b6c52095-fe7c-4482-8789-f490af2ffad3 --------- Copilot-Session: b6c52095-fe7c-4482-8789-f490af2ffad3
78 lines
3.0 KiB
Markdown
78 lines
3.0 KiB
Markdown
## Summary
|
|
Brief description of what this PR does.
|
|
|
|
## Type of Change
|
|
- [ ] 🐛 Bug fix (non-breaking change which fixes an issue)
|
|
- [ ] ✨ New feature (non-breaking change which adds functionality)
|
|
- [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
|
|
- [ ] 📚 Documentation update
|
|
- [ ] 🔧 Maintenance (dependency updates, code cleanup, etc.)
|
|
|
|
## Related Issues
|
|
Closes #[issue number]
|
|
Relates to #[issue number]
|
|
|
|
## Changeset
|
|
- [ ] Added a changeset (`npx changeset`) describing this change for end users — see `.changeset/README.md`
|
|
- [ ] Not applicable (docs/tests/CI/dependency-only) — added `skip-changelog` label instead
|
|
|
|
## Changes Made
|
|
- Change 1
|
|
- Change 2
|
|
- Change 3
|
|
|
|
## Testing Performed
|
|
- [ ] If Core/CLI/MCP runtime paths changed: ran `& .\scripts\Test-E2E.ps1` locally and confirmed it completed with no failures or unresolved issues
|
|
- [ ] Excel E2E not applicable because no Core/CLI/MCP runtime path changed
|
|
- [ ] Ran the relevant feature-specific integration tests and recorded the exact command and result below
|
|
- [ ] Tested manually with various Excel files
|
|
- [ ] Verified Excel process cleanup (no excel.exe remains after 5 seconds)
|
|
- [ ] Tested error conditions (missing files, invalid arguments, etc.)
|
|
- [ ] All existing commands still work
|
|
- [ ] VBA script execution tested (if applicable)
|
|
- [ ] XLSM file format validation tested (if applicable)
|
|
- [ ] VBA trust setup tested (if applicable)
|
|
- [ ] Build produces zero warnings
|
|
|
|
## Test Commands
|
|
```powershell
|
|
# Commands used for testing
|
|
ExcelMcp command1 "test.xlsx"
|
|
ExcelMcp command2 "test.xlsx" "param"
|
|
```
|
|
|
|
## Screenshots (if applicable)
|
|
[Add screenshots showing the new functionality]
|
|
|
|
## Core Commands Coverage Checklist ⚠️
|
|
|
|
**Does this PR add or modify Core Commands methods?** [ ] Yes [ ] No
|
|
|
|
If YES, verify all steps completed:
|
|
|
|
- [ ] Updated the annotated Core Commands interface and implementation
|
|
- [ ] Built Release so source generators refreshed Service, CLI, and MCP surfaces
|
|
- [ ] Ran `scripts\audit-core-coverage.ps1 -CheckNaming -FailOnGaps`
|
|
- [ ] Ran `scripts\check-mcp-core-implementations.ps1`
|
|
- [ ] Verified CLI and MCP names, parameters, defaults, validation, and results match
|
|
- [ ] Updated focused integration tests for the affected entry points
|
|
- [ ] Updated canonical guidance in `skills/shared` and user documentation when behavior changed
|
|
|
|
**Coverage Impact**: +___ methods, ___% → ___% coverage
|
|
|
|
## Checklist
|
|
- [ ] Code follows project style guidelines
|
|
- [ ] Self-review of code completed
|
|
- [ ] Code builds with zero warnings
|
|
- [ ] Appropriate error handling added
|
|
- [ ] Updated help text (if adding new commands)
|
|
- [ ] Updated README.md (if needed)
|
|
- [ ] Follows Excel COM best practices from copilot-instructions.md
|
|
- [ ] Uses batch API with proper disposal (`using var batch` or `await using var batch`)
|
|
- [ ] Properly handles 1-based Excel indexing
|
|
- [ ] Escapes user input with `.EscapeMarkup()`
|
|
- [ ] Returns consistent exit codes (0 = success, 1+ = error)
|
|
|
|
## Additional Notes
|
|
Any additional information that reviewers should know.
|