* Ruff and Pyright OK * lint * uv * add uv to readme * Replace pyright with TY * Lint is ok
6.6 KiB
u# Contributing to Aperi'Solve
Thank you for your interest in contributing! We welcome all contributions, whether it's bug fixes, new features, or documentation improvements.
Getting Started
Tip
New to the project? Adding a new analyzer is a great first contribution! Check out our Adding a New Analyzer Guide.
Quick Steps
- Fork the repository
- Create a feature branch (
git checkout -b feature/amazing-feature) - Make your changes
- Test your changes
- Commit with clear messages (
git commit -m 'Add amazing feature') - Push to your fork (
git push origin feature/amazing-feature) - Open a Pull Request describing your changes
What Can You Contribute?
🔧 Adding New Analyzers
Want to add support for a new steganography or forensics tool? We'd love that!
Note
Check out our detailed tutorial: Adding a New Analyzer
Quick checklist for new analyzers:
- Create analyzer file in
analyzers/ - Extend
SubprocessAnalyzerbase class - Register in
workers.py - Add tool to Docker setup if needed
- Test with sample images
- Update documentation
🐛 Bug Fixes
Found a bug? Please:
- Check if it's already reported in Issues
- If not, open a new issue with reproduction steps
- Feel free to submit a PR with the fix!
📚 Documentation
Help us improve:
- Fix typos or unclear explanations
- Add examples and use cases
- Improve installation instructions
- Translate documentation
🎨 UI/UX Improvements
Contributions to the web interface are welcome:
- Better error messages
- Improved result presentation
- Mobile responsiveness
- Accessibility improvements
Code Style & Quality
Important
Follow the project’s code style and run linters before submitting any code.
This project enforces:
- Ruff : Check + Format (line length 100)
- ty : Type checking
Tip
All tool configurations (Ruff, ty) are centralized in pyproject.toml. You can run each tool directly and it will automatically pick up the configuration.
Setup
Tip
Use a virtual environment and install development dependencies:
# Create & activate a virtual environment
uv venv
# Install project + development dependencies from pyproject.toml
uv sync --extra dev
Running Tools Manually
Option 1 - Bash script (no pre-commit file needed)
Run this script lint.sh at the project root folder.
🧹 Running Ruff lint checks...
🎨 Running Ruff format checks...
🔍 Running ty type checks...
✅ All checks passed!
Equivalent manual commands:
ruff check .
ruff format . --check
ty check aperisolve
Option 2 - Pre-Commit hook
Create a pre-commit hooks file .pre-commit-config.yaml at the project root folder.
repos:
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.13.0
hooks:
- id: ruff-check
- repo: local
hooks:
- id: ty-check
name: ty-check
entry: ty check
types: [python]
language: system
pass_filenames: true
Then run the following command so each staged files will be checked while commited.
pre-commit install
You can also do a manual pass on all files before commiting:
pre-commit run --all-files
Note
Continuous Integration (CI) runs all checks on every PR. Ensure your code passes before submitting.
Pull Request Guidelines
Use clear, descriptive titles:
- ✅
Add stegdetect analyzer - ✅
Fix binwalk extraction error handling - ✅
Update documentation for password-protected tools - ❌
Update code - ❌
Fix bug
Commit Messages
Write clear, concise commit messages:
# Good
git commit -m "Add zsteg analyzer for PNG steganography detection"
git commit -m "Fix foremost error detection when no files extracted"
# Not so good
git commit -m "updates"
git commit -m "fix"
Adding New Dependencies
Caution
Adding new dependencies requires careful consideration.
If your contribution needs new dependencies:
- Python packages: Add to
pyproject.toml([project.dependencies]or[project.optional-dependencies.dev]) - System tools: Add to
Dockerfile - Explain why in your PR description
- Keep dependencies minimal - avoid adding large libraries for small features
Useful commands
# development environment (hot reload and local volumes)
docker compose -f compose.dev.yml up --build
# Stop and remove containers, networks and volumes (results are stored in a volume)
docker compose down -v
# Enter web container shell
docker exec -it aperisolve-web bash
# Enter Postgres shell (from host)
docker exec -it postgres psql -U aperiuser -d aperisolve
# Backup all uploaded files
docker cp -r aperisolve-web:/app/aperisolve/results /path/to/backup/location
# Backup a single uploaded file
docker cp aperisolve-web:/app/aperisolve/results/filename.ext /path/to/backup/filename.ext
Warning
If switching between dev and production compose files, remove the
resultsdirectory or mounted volume to avoid conflicts:rm -rf aperisolve/results
Docker and Tool Installation
When adding a new analyzer that requires a new tool:
# In Dockerfile
RUN apt-get update && apt-get install -y \
your-new-tool \
&& rm -rf /var/lib/apt/lists/*
Or for tools requiring compilation:
# Builder stage example (jphide)
RUN git clone https://github.com/h3xx/jphs.git /tmp/jphs && \
cd /tmp/jphs && \
make && \
cp jphide jpseek /usr/local/bin/ && \
rm -rf /tmp/jphs
# Runtime stage - copy compiled binaries
COPY --from=builder /usr/local/bin/jphide /usr/local/bin/jphide
COPY --from=builder /usr/local/bin/jpseek /usr/local/bin/jpseek
Note
Test the Docker build locally before submitting!
Code Review Process
- Automated checks run first (linters, tests)
- Maintainer review - we'll provide feedback
- Address feedback - update your PR as needed
- Approval & merge - once everything looks good!
Tip
Don't worry if you need to make changes - it's a normal part of the process!
Need Help?
- 💬 Questions? Open a Discussion
- 🐛 Found a bug? Open an Issue
- 💡 Ideas? We'd love to hear them in Discussions!
Recognition
Note
All contributors will be recognized in our Contributors page and in release notes!
Thank you for contributing! 🎉
Your efforts help make this tool better for the entire security and CTF community!