pull_request_template.md•2.7 kB
# Pull Request
## Summary
<!-- Provide a brief summary of the changes in this PR -->
## Type of Change
<!-- Mark the relevant option with an 'x' -->
- [ ] Bug fix (non-breaking change that fixes an issue)
- [ ] New feature (non-breaking change that adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] Documentation update
- [ ] Performance improvement
- [ ] Code refactoring
- [ ] Test improvements
- [ ] CI/CD changes
## Changes Made
<!-- Describe the changes made in this PR -->
## Testing
<!-- Describe the testing you have performed -->
- [ ] Unit tests pass (`npm test`)
- [ ] Test coverage maintained/improved (`npm run test:coverage`)
- [ ] Integration tests pass (if applicable)
- [ ] Manual testing completed
- [ ] MCP client integration tested
## MCP-Specific Checklist
<!-- Mark applicable items with 'x' -->
- [ ] New MCP tools properly registered in `src/index.ts`
- [ ] Tool schemas follow MCP specification
- [ ] Tool functions include proper error handling
- [ ] Tool responses follow consistent structure
- [ ] GraphQL client methods updated (if applicable)
- [ ] Safe scene management principles followed (if applicable)
## Code Quality
<!-- Ensure code quality standards are met -->
- [ ] ESLint passes (`npm run lint`)
- [ ] TypeScript compilation passes (`npm run build`)
- [ ] Code follows project conventions
- [ ] Functions are properly documented
- [ ] Error handling is comprehensive
- [ ] No console.log statements in production code
## Dependencies
<!-- If dependencies were added or changed -->
- [ ] No new dependencies added
- [ ] New dependencies are justified and documented
- [ ] Dependency versions are pinned appropriately
- [ ] Security audit passes (`npm audit`)
## Documentation
<!-- Documentation updates -->
- [ ] README updated (if needed)
- [ ] API documentation updated (if needed)
- [ ] Code comments added for complex logic
- [ ] CHANGELOG updated (if needed)
## Breaking Changes
<!-- If this PR introduces breaking changes, describe them -->
<!-- Include migration guide for users -->
## Screenshots/Examples
<!-- If applicable, add screenshots or examples of the changes -->
## Related Issues
<!-- Link any related issues -->
Fixes #
Closes #
Related to #
## Deployment Notes
<!-- Any special considerations for deployment -->
## Reviewer Notes
<!-- Any specific areas you'd like reviewers to focus on -->
---
## Checklist for Reviewers
- [ ] Code follows project standards
- [ ] Tests are comprehensive and pass
- [ ] Documentation is updated
- [ ] No security concerns
- [ ] Performance impact is acceptable
- [ ] Breaking changes are properly documented