|
| 1 | +# Pull Request |
| 2 | + |
| 3 | +## Description |
| 4 | + |
| 5 | +Brief description of the changes in this PR. |
| 6 | + |
| 7 | +## Type of Change |
| 8 | + |
| 9 | +- [ ] Bug fix (non-breaking change which fixes an issue) |
| 10 | +- [ ] New feature (non-breaking change which adds functionality) |
| 11 | +- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) |
| 12 | +- [ ] Documentation update |
| 13 | +- [ ] Code quality improvement |
| 14 | +- [ ] Performance improvement |
| 15 | +- [ ] Security improvement |
| 16 | + |
| 17 | +## Related Issues |
| 18 | + |
| 19 | +Fixes #(issue number) |
| 20 | +Relates to #(issue number) |
| 21 | + |
| 22 | +## Changes Made |
| 23 | + |
| 24 | +- [ ] Added/modified backend: [backend name] |
| 25 | +- [ ] Updated templates |
| 26 | +- [ ] Modified CLI commands |
| 27 | +- [ ] Updated documentation |
| 28 | +- [ ] Added/updated tests |
| 29 | +- [ ] Fixed code generation issues |
| 30 | + |
| 31 | +## Testing |
| 32 | + |
| 33 | +### Manual Testing |
| 34 | + |
| 35 | +- [ ] Tested code generation for affected backends |
| 36 | +- [ ] Verified generated code compiles without errors |
| 37 | +- [ ] Tested CLI commands work as expected |
| 38 | +- [ ] Verified generated servers follow MCP protocol |
| 39 | + |
| 40 | +### Automated Testing |
| 41 | + |
| 42 | +- [ ] All existing tests pass |
| 43 | +- [ ] Added tests for new functionality |
| 44 | +- [ ] Generated code passes syntax validation |
| 45 | +- [ ] Integration tests pass |
| 46 | + |
| 47 | +### Backend-Specific Testing (if applicable) |
| 48 | + |
| 49 | +- [ ] **Database**: Generated server connects and executes queries |
| 50 | +- [ ] **API**: Generated server makes HTTP requests correctly |
| 51 | +- [ ] **Filesystem**: Generated server handles file operations safely |
| 52 | +- [ ] **Webscraper**: Generated server respects robots.txt and rate limits |
| 53 | +- [ ] **Email**: Generated server sends/receives emails correctly |
| 54 | +- [ ] **Cloud Storage**: Generated server interacts with cloud APIs |
| 55 | + |
| 56 | +## Code Quality |
| 57 | + |
| 58 | +- [ ] Code follows project style guidelines |
| 59 | +- [ ] Code is properly documented (docstrings, comments) |
| 60 | +- [ ] No hard-coded values or secrets |
| 61 | +- [ ] Error handling is appropriate |
| 62 | +- [ ] Generated code is readable and maintainable |
| 63 | + |
| 64 | +## Breaking Changes |
| 65 | + |
| 66 | +If this PR introduces breaking changes, describe them here: |
| 67 | + |
| 68 | +- [ ] No breaking changes |
| 69 | +- [ ] Breaking changes described below: |
| 70 | + |
| 71 | +[Describe breaking changes and migration steps] |
| 72 | + |
| 73 | +## Screenshots/Output (if applicable) |
| 74 | + |
| 75 | +``` |
| 76 | +Include relevant command output, screenshots, or generated code examples |
| 77 | +``` |
| 78 | + |
| 79 | +## Checklist |
| 80 | + |
| 81 | +- [ ] I have read the [CONTRIBUTING.md](../CONTRIBUTING.md) guidelines |
| 82 | +- [ ] My code follows the project's style guidelines |
| 83 | +- [ ] I have performed a self-review of my code |
| 84 | +- [ ] I have commented my code, particularly in hard-to-understand areas |
| 85 | +- [ ] I have made corresponding changes to the documentation |
| 86 | +- [ ] My changes generate no new warnings |
| 87 | +- [ ] I have added tests that prove my fix is effective or that my feature works |
| 88 | +- [ ] New and existing unit tests pass locally with my changes |
| 89 | +- [ ] Any dependent changes have been merged and published in downstream modules |
| 90 | + |
| 91 | +## Generated Code Validation |
| 92 | + |
| 93 | +For changes affecting code generation, please verify: |
| 94 | + |
| 95 | +- [ ] Generated code compiles without syntax errors |
| 96 | +- [ ] Generated code follows MCP protocol specifications |
| 97 | +- [ ] Generated code includes proper error handling |
| 98 | +- [ ] Generated code has appropriate security measures |
| 99 | +- [ ] Generated tests pass |
| 100 | + |
| 101 | +## Documentation Updates |
| 102 | + |
| 103 | +- [ ] README.md updated (if needed) |
| 104 | +- [ ] CHANGELOG.md updated |
| 105 | +- [ ] API documentation updated (if needed) |
| 106 | +- [ ] Examples updated (if needed) |
| 107 | + |
| 108 | +## Deployment Considerations |
| 109 | + |
| 110 | +- [ ] No deployment considerations |
| 111 | +- [ ] Database migrations required |
| 112 | +- [ ] Configuration changes required |
| 113 | +- [ ] Dependencies updated |
| 114 | + |
| 115 | +## Additional Notes |
| 116 | + |
| 117 | +Add any additional notes, concerns, or discussion points for reviewers. |
0 commit comments