|
| 1 | +# Contributing to Solr MCP Server |
| 2 | + |
| 3 | +Thank you for your interest in contributing to the Solr MCP Server! This document provides guidelines for contributing to the project. |
| 4 | + |
| 5 | +## Developer documentation |
| 6 | + |
| 7 | +To avoid duplication, the environment setup, build/run/test workflows, and detailed developer guides live in the dev-docs folder: |
| 8 | + |
| 9 | +- Development Guide (build, run, test, IDE, CI): dev-docs/DEVELOPMENT.md |
| 10 | +- Architecture: dev-docs/ARCHITECTURE.md |
| 11 | +- Deployment (Docker, HTTP vs STDIO): dev-docs/DEPLOYMENT.md |
| 12 | +- Troubleshooting: dev-docs/TROUBLESHOOTING.md |
| 13 | + |
| 14 | +If you're ready to contribute code, see Submitting Changes below. |
| 15 | + |
| 16 | +## Code Style and Quality |
| 17 | + |
| 18 | +We use Spotless for code formatting and style enforcement. CI enforces `spotlessCheck` on pull requests. |
| 19 | + |
| 20 | +- Commands and details: dev-docs/DEVELOPMENT.md#common-gradle-tasks |
| 21 | +- Build system overview: dev-docs/DEVELOPMENT.md#build-system |
| 22 | + |
| 23 | +### Coding Standards |
| 24 | + |
| 25 | +- Follow standard Java conventions |
| 26 | +- Write meaningful commit messages |
| 27 | +- Add JavaDoc for public APIs |
| 28 | +- Include unit tests for new features |
| 29 | +- Keep methods focused and concise |
| 30 | + |
| 31 | +## Testing |
| 32 | + |
| 33 | +To keep this document concise, please see the Development Guide for all testing workflows and tips: |
| 34 | + |
| 35 | +- Testing overview: dev-docs/DEVELOPMENT.md#testing |
| 36 | +- Unit tests: dev-docs/DEVELOPMENT.md#unit-tests |
| 37 | +- Integration tests: dev-docs/DEVELOPMENT.md#integration-tests |
| 38 | +- Docker image tests: dev-docs/DEVELOPMENT.md#docker-integration-tests |
| 39 | +- Coverage reports: dev-docs/DEVELOPMENT.md#testing |
| 40 | + |
| 41 | +## Submitting Changes |
| 42 | + |
| 43 | +### Pull Request Process |
| 44 | + |
| 45 | +1. **Create a feature branch** |
| 46 | + ```bash |
| 47 | + git checkout -b feature/your-feature-name |
| 48 | + ``` |
| 49 | + |
| 50 | +2. **Make your changes** |
| 51 | + - Write clean, well-documented code |
| 52 | + - Add tests for new functionality |
| 53 | + - Update documentation as needed |
| 54 | + |
| 55 | +3. **Format your code** |
| 56 | + ```bash |
| 57 | + ./gradlew spotlessApply |
| 58 | + ``` |
| 59 | + |
| 60 | +4. **Run tests** |
| 61 | + ```bash |
| 62 | + ./gradlew build |
| 63 | + ``` |
| 64 | + |
| 65 | +5. **Commit your changes** |
| 66 | + ```bash |
| 67 | + git add . |
| 68 | + git commit -m "feat: add your feature description" |
| 69 | + ``` |
| 70 | + |
| 71 | +6. **Push to your fork** |
| 72 | + ```bash |
| 73 | + git push origin feature/your-feature-name |
| 74 | + ``` |
| 75 | + |
| 76 | +7. **Create a Pull Request** |
| 77 | + - Provide a clear description of the changes |
| 78 | + - Reference any related issues |
| 79 | + - Ensure CI checks pass |
| 80 | + |
| 81 | +### Commit Message Guidelines |
| 82 | + |
| 83 | +We follow the [Conventional Commits](https://www.conventionalcommits.org/) specification: |
| 84 | + |
| 85 | +``` |
| 86 | +<type>(<scope>): <description> |
| 87 | +
|
| 88 | +[optional body] |
| 89 | +
|
| 90 | +[optional footer] |
| 91 | +``` |
| 92 | + |
| 93 | +**Types:** |
| 94 | +- `feat`: New feature |
| 95 | +- `fix`: Bug fix |
| 96 | +- `docs`: Documentation changes |
| 97 | +- `style`: Code style changes (formatting, etc.) |
| 98 | +- `refactor`: Code refactoring |
| 99 | +- `test`: Adding or updating tests |
| 100 | +- `chore`: Maintenance tasks |
| 101 | + |
| 102 | +**Examples:** |
| 103 | +``` |
| 104 | +feat(search): add fuzzy search support |
| 105 | +fix(indexing): handle null values in CSV parser |
| 106 | +docs: update installation instructions |
| 107 | +test: add integration tests for collection service |
| 108 | +``` |
| 109 | + |
| 110 | +## Development workflow references |
| 111 | + |
| 112 | +For implementation details and examples, see the Development Guide: |
| 113 | + |
| 114 | +- Adding new MCP tools: dev-docs/DEVELOPMENT.md#adding-a-new-mcp-tool |
| 115 | +- Adding a new document format: dev-docs/DEVELOPMENT.md#adding-a-new-document-format |
| 116 | +- Project structure and architecture: dev-docs/ARCHITECTURE.md |
| 117 | +- Dependencies and version catalogs: dev-docs/DEVELOPMENT.md#build-system |
| 118 | +- Documentation practices: dev-docs/DEVELOPMENT.md#modifying-configuration |
| 119 | + |
| 120 | +## Questions or Need Help? |
| 121 | + |
| 122 | +- Open an issue for bugs or feature requests |
| 123 | +- Start a discussion for questions or ideas |
| 124 | +- Check existing issues and discussions first |
| 125 | + |
| 126 | +## Code of Conduct |
| 127 | + |
| 128 | +Be respectful, inclusive, and professional. We're all here to build something great together. |
| 129 | + |
| 130 | +## License |
| 131 | + |
| 132 | +By contributing, you agree that your contributions will be licensed under the Apache License 2.0. |
0 commit comments