Files
aperisolve/CONTRIBUTING.md
ZeeckaandGitHub 730ed50f70 Feature/ruff (#186)
* Ruff and Pyright OK

* lint

* uv

* add uv to readme

* Replace pyright with TY

* Lint is ok
2026-02-21 00:00:48 +01:00

6.6 KiB
Raw Permalink Blame History

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

  1. Fork the repository
  2. Create a feature branch (git checkout -b feature/amazing-feature)
  3. Make your changes
  4. Test your changes
  5. Commit with clear messages (git commit -m 'Add amazing feature')
  6. Push to your fork (git push origin feature/amazing-feature)
  7. 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 SubprocessAnalyzer base 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:

  1. Python packages: Add to pyproject.toml ([project.dependencies] or [project.optional-dependencies.dev])
  2. System tools: Add to Dockerfile
  3. Explain why in your PR description
  4. 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 results directory 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

  1. Automated checks run first (linters, tests)
  2. Maintainer review - we'll provide feedback
  3. Address feedback - update your PR as needed
  4. 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!