Contributing to aMQTT¤
:+1::tada: First off, thanks for taking the time to contribute! :tada::+1:
The following is a set of guidelines for contributing to aMQTT on GitHub. These are mostly guidelines, not rules. Use your best judgment, and feel free to propose changes to this document in a pull request.
Development Setup¤
Requirements¤
- Install uv
- Use
uv python install <version>to install 3.10, 3.11, 3.12 or 3.13 - Fork repository (if you intend to open a pull request)
- Clone repo
git clone git@github.com:<username>/amqtt.git - Add repo to receive latest updates
git add upstream git@github.com:Yakio/amqtt.git
Installation¤
Create and start virtual environment with UV
uv venv .venv --python 3.13.0
source .venv/bin/activate
Install the package with development (and doc) dependencies:
uv pip install -e . --group dev --group doc
Add git pre-commit checks (which parallel the CI checks):
pre-commit install
The pre-commit configuration includes Gitleaks secret scanning to block accidental commits of credentials, tokens, private keys, and similar sensitive data. Run a manual repository scan with:
pre-commit run gitleaks-full --hook-stage manual
Run¤
Run CLI commands:
uv run amqtt
uv run amqtt_pub
uv run amqtt_sub
Run the test case suite:
pytest
Run the full suite including interoperability tests that require Go, Java, or Node.js client dependencies:
pytest --extended
Run only the extended interoperability tests:
pytest --extended -m extended
Run the type checker and linters manually:
pre-commit run --all-files
Testing¤
When adding a new feature, please add corollary tests. Pull requests must maintain or increase the reported test coverage percentage; changes that reduce coverage need additional tests before they can be merged.
Run uv run --frozen pytest locally to see the coverage summary before opening a pull request.
If you encounter a bug when using aMQTT which you then resolve, please reproduce the issue in a test as well.
Sample tests¤
Sample scripts in samples/ are excluded from coverage scoring, but they still need smoke tests. When adding or updating
a Python sample, add or update a test in tests/test_samples.py and mark the test with the sample script it covers:
@pytest.mark.sample("client_publish.py")
async def test_client_publish():
...
The test_all_sample_files_are_accounted_for guard checks every samples/*.py file against these markers. If a sample is
intentionally not testable, add it to IGNORED_SAMPLE_FILES in tests/test_samples.py with a short explanation in the
pull request.
Local fuzzing¤
The MQTT packet parser has local fuzz coverage using Hypothesis. These tests generate malformed and boundary-case byte streams and assert that packet decode paths either parse successfully or fail with expected parser exceptions.
Run the local fuzz tests directly with:
uv run --frozen pytest tests/mqtt/test_fuzz_packet.py
Run them with the related MQTT parser tests before changing packet decode logic:
uv run --frozen pytest tests/mqtt tests/test_codecs.py
If Hypothesis finds a failure, keep the minimized example in the test output and add or adjust a regression test before changing the parser behavior.
Dependencies¤
Dependencies are managed with uv, based on the pyproject.toml file and version locked with uv.lock. To support
OpenSSF Scorecard, the uv.lock file needs to be exported into the hash-based requirements.txt file. This file is
only used for OpenSSF Scorecard dependency analysis; the project itself uses uv.lock.
If dependencies are added or updated, CI will require that requirements.txt aligns with uv.lock:
uv export --frozen --format requirements.txt --no-default-groups --no-emit-project --no-header --output-file requirements.txt
Go MQTT Interoperability Tests¤
Some interoperability tests use the Eclipse Paho Go MQTT client. To run them locally, install Go with go on your PATH, then download the Go module dependencies:
cd tests/support/go-mqtt-client
go mod download
cd -
Then run the Go interoperability tests:
uv run pytest tests/test_go_mqtt.py -v
If Go or the downloaded module dependencies are unavailable, these tests are skipped during local runs. In CI, missing Go dependencies are treated as failures.
Java MQTT Interoperability Tests¤
Some interoperability tests use the Eclipse Paho Java MQTT client. To run them locally, install a JDK with java and javac on your PATH, then download the Paho client jar and expose it with PAHO_MQTT_JAR:
curl -L -o /tmp/org.eclipse.paho.client.mqttv3-1.2.5.jar \
https://repo1.maven.org/maven2/org/eclipse/paho/org.eclipse.paho.client.mqttv3/1.2.5/org.eclipse.paho.client.mqttv3-1.2.5.jar
export PAHO_MQTT_JAR=/tmp/org.eclipse.paho.client.mqttv3-1.2.5.jar
Then run the Java interoperability tests:
uv run pytest tests/test_java_mqtt.py -v
If PAHO_MQTT_JAR is unset, pytest also checks the local Maven and Gradle caches for org.eclipse.paho.client.mqttv3. If no jar is available, these tests are skipped.