flip/flipside-mcp-extension
1
0
Fork 0
mirror of https://github.com/FlipsideCrypto/flipside-mcp-extension.git synced 2026-02-06 11:16:55 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity
main
flipside-mcp-extension/.github/pull_request_template.md
Erik Brakke 63f6080f01
Add comprehensive unit tests for MCP proxy server (#7) 
* 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>
2025-07-02 11:02:58 -06:00

1.0 KiB
Raw Permalink Blame History

Pull Request

Summary

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