mirror of
https://github.com/FlipsideCrypto/flipside-mcp-extension.git
synced 2026-02-06 11:16:55 +00:00
63f6080f01
* Add comprehensive unit tests for MCP proxy server - Add main_test.go with full test coverage for proxy functionality - Test client creation, authentication, URL conversion, and error handling - Include mock MCP server for integration testing - Update Makefile to run unit tests via 'make test' - Add build target for local development - Ensure proper proxy call forwarding and authentication verification 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Add comprehensive CI/CD pipeline with GitHub Actions - Add test.yml workflow for basic unit testing on PRs and pushes - Add ci.yml workflow with comprehensive testing, linting, and security scanning - Include cross-platform build verification (Linux, macOS, Windows) - Add golangci-lint configuration for consistent code quality - Add PR template for structured pull request reviews - Enable test coverage reporting with race detection - Include security scanning with gosec and govulncheck - Integrate Makefile testing to verify build system works 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Update README with testing and CI/CD documentation - Add comprehensive testing section with coverage details - Document CI/CD pipeline and quality gates - Include local testing commands for contributors - Explain how to ensure PRs pass all checks 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Fix linting and formatting issues for CI compliance - Fix errcheck violations: handle ParseBool and JSON encoding errors - Fix gofmt issues: remove trailing whitespace and ensure newlines - Update GitHub Actions to use upload-artifact@v4 (v3 deprecated) - Ensure all error return values are properly checked - Add proper error handling in mock server responses 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * ignore some ci checks for now --------- Co-authored-by: Claude <noreply@anthropic.com>
33 lines
1.0 KiB
Markdown
33 lines
1.0 KiB
Markdown
# Pull Request
|
|
|
|
## Summary
|
|
<!-- Provide a brief description of the changes -->
|
|
|
|
## 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
|
|
- [ ] Code refactoring
|
|
- [ ] Test improvements
|
|
|
|
## Testing
|
|
- [ ] Unit tests pass (`go test -v`)
|
|
- [ ] Integration tests pass (`make test`)
|
|
- [ ] Manual testing completed
|
|
- [ ] Cross-platform builds work (`make build-all`)
|
|
|
|
## Security
|
|
- [ ] No sensitive information (API keys, secrets) exposed
|
|
- [ ] Security scan passes (gosec, govulncheck)
|
|
- [ ] Authentication and authorization properly handled
|
|
|
|
## Checklist
|
|
- [ ] Code follows the project's style guidelines
|
|
- [ ] Self-review of code completed
|
|
- [ ] Code is well-documented
|
|
- [ ] Tests added/updated for new functionality
|
|
- [ ] All CI checks pass
|
|
|
|
## Additional Notes
|
|
<!-- Any additional information that reviewers should know --> |