Merge #1883: SI-16 #1881: Migrate contrib/bencode to torrust/torrust-bittorrent as torrust-bencode

a817386 fix: address Copilot review comments on PR #1883 (Jose Celano)
11b7172 chore(container): remove contrib/bencode from Containerfile (T8) (Jose Celano)
7863546 docs(packages): update EPIC #1669 and ISSUE spec after SI-16 completion (T13) (Jose Celano)
a55c27f chore(packages): remove contrib/bencode and update references (T8-T10) (Jose Celano)
880333e chore: update Cargo.lock after switching to torrust-bencode 3.0.0 (Jose Celano)
a0e0e8e refactor(http-protocol): replace path dep with published torrust-bencode 3.0.0 (Jose Celano)
aaaadc8 docs(packages): open SI-16 #1881 migrate contrib/bencode to torrust-bittorrent (Jose Celano)

Pull request description:

  ## Summary

  Closes #1881 (subissue of EPIC #1669 — Overhaul: Packages)

  Migrates the `contrib/bencode` crate (`torrust-tracker-contrib-bencode`) out of this workspace into `torrust/torrust-bittorrent` as the published `torrust-bencode 3.0.0` crate, and removes all traces of it from the tracker workspace.

  ## Changes

  ### `torrust/torrust-bittorrent` (already merged — separate repo)

  - Renamed crate to `torrust-bencode`, updated metadata and repository URL
  - Applied clippy fixes (`ok_or_else`, `map_or`) in `access/convert.rs` and `reference/decode.rs`
  - Published `torrust-bencode 3.0.0` to crates.io

  ### This PR (`torrust/torrust-tracker`)

  - **`packages/http-protocol`**: replace path dep `torrust-tracker-contrib-bencode` with published `torrust-bencode = "3.0.0"`; update imports in `announce.rs` and `scrape.rs`
  - **`Cargo.lock`**: remove `torrust-tracker-contrib-bencode`, add `torrust-bencode 3.0.0`
  - **`contrib/bencode/`**: delete directory (20 files)
  - **`AGENTS.md`**: remove bencode from `contrib/` description and Community contributions table
  - **`docs/packages.md`**: update `contrib/` tree noting migration
  - **`docs/release_process.md`**: remove old crate from publish checklist
  - **`src/console/ci/qbittorrent_e2e/bencode.rs`**: update crate name in doc comment
  - **`.github/skills/release-new-version/SKILL.md`**: remove `contrib/bencode` version bump step
  - **`docs/issues/open/1669-overhaul-packages/EPIC.md`**: mark SI-16 done, update tables
  - **`docs/issues/open/1881-.../ISSUE.md`**: mark all tasks done (T1–T11, T13)

  ## Pending (not in this PR)

  - **T12**: Yank `torrust-tracker-contrib-bencode` on crates.io — deferred until no downstream consumers confirmed

  ## Verification

  - `cargo build -p torrust-tracker-http-tracker-protocol` ✅
  - `cargo build --workspace` ✅ (verified locally)
  - CI will run full suite on this PR

ACKs for top commit:
  josecelano:
    ACK a817386

Tree-SHA512: e7f3b33c8a2f2ac06a0a9089d3cb6f9b479a5420ddff7dca4ab5f0f7e17b80c9d2a855c7c2c728b0972018ab250ae9db6005d3eae1438bacff9653df4325aab4

Author: josecelano

.github/skills/dev/git-workflow/release-new-version/SKILL.md

@@ -67,7 +67,6 @@ Edit `version` in:
 - `Cargo.toml` (workspace)
 - All packages under `packages/` that publish crates
 - `console/tracker-client/Cargo.toml`
-- `contrib/bencode/Cargo.toml`
 
 Also update any internal path dependency `version` constraints.
 
@@ -100,7 +99,6 @@ Check the
 [deployment workflow](https://github.com/torrust/torrust-tracker/actions/workflows/deployment.yaml)
 ran successfully and the following crates were published:
 
-- `torrust-tracker-contrib-bencode`
 - `torrust-located-error`
 - `torrust-tracker-primitives`
 - `torrust-clock`

.github/workflows/deployment.yaml

@@ -73,7 +73,6 @@ jobs:
           cargo publish -p torrust-tracker-client
           cargo publish -p torrust-clock
           cargo publish -p torrust-tracker-configuration
-          cargo publish -p torrust-tracker-contrib-bencode
           cargo publish -p torrust-tracker-events
           cargo publish -p torrust-metrics
           cargo publish -p torrust-tracker-primitives

AGENTS.md

@@ -41,7 +41,7 @@ native IPv4/IPv6 support, private/whitelisted mode, and a management REST API.
 - `src/console/` — Console entry points
 - `packages/` — Cargo workspace packages (all domain logic lives here; see package catalog below)
 - `console/` — Console tools (e.g., `tracker-client`)
-- `contrib/` — Community-contributed utilities (`bencode`) and developer tooling
+- `contrib/` — Developer tooling
 - `contrib/dev-tools/` — Developer tools: git hooks (`pre-commit.sh`, `pre-push.sh`, `install-git-hooks.sh`),
   container scripts, and init scripts
 - `tests/` — Integration tests (`integration.rs`, `servers/`)
@@ -94,9 +94,9 @@ All packages live under `packages/`. The workspace version is `3.0.0-develop`.
 
 **Community contributions** (under `contrib/`):
 
-| Crate     | Description                     |
-| --------- | ------------------------------- |
-| `bencode` | Bencode encode/decode utilities |
+| Crate | Description                                                                                            |
+| ----- | ------------------------------------------------------------------------------------------------------ |
+| —     | None (bencode migrated to [torrust/torrust-bittorrent](https://github.com/torrust/torrust-bittorrent)) |
 
 ## 🏷️ Package Naming Conventions
 

Cargo.lock

@@ -51,14 +51,12 @@ WORKDIR /build/src
 # or moved crate must have its line updated or removed accordingly.
 COPY Cargo.toml Cargo.lock ./
 COPY console/tracker-client/Cargo.toml console/tracker-client/
-COPY contrib/bencode/Cargo.toml contrib/bencode/
 # The following packages are excluded from cargo nextest archive (see Cook and
 # Build stages below) because they are not part of the production tracker service
 # and do not need to be tested inside the container image:
 #   - workspace-coupling (analysis/coupling tool, no production value)
 #   - torrust-tracker-torrent-repository-benchmarking (benchmarking only)
 #   - torrust-tracker-client (CLI dev tools: tracker_client, tracker_checker, etc.)
-#   - torrust-tracker-contrib-bencode (contrib utility; its own tests/bins are not needed in the container)
 #   - torrust-tracker-e2e-tools (E2E runners + profiling tool, GHA host-only)
 #   - torrust-tracker-persistence-benchmark (persistence layer dev benchmarking tool)
 # Their Cargo.toml manifests and stub source files must still be present here
@@ -114,8 +112,6 @@ RUN mkdir -p \
       src/bin \
       packages/e2e-tools/src/bin \
       packages/persistence-benchmark/src/bin \
-      contrib/bencode/src \
-      contrib/bencode/benches \
       contrib/dev-tools/analysis/workspace-coupling/src \
       console/tracker-client/src/bin \
       packages/axum-health-check-api-server/src \
@@ -155,8 +151,6 @@ RUN mkdir -p \
       packages/e2e-tools/src/bin/profiling.rs \
       packages/e2e-tools/src/bin/qbittorrent_e2e_runner.rs \
       packages/persistence-benchmark/src/bin/persistence_benchmark_runner.rs \
-      contrib/bencode/src/lib.rs \
-      contrib/bencode/benches/bencode_benchmark.rs \
       contrib/dev-tools/analysis/workspace-coupling/src/main.rs \
       console/tracker-client/src/lib.rs \
       console/tracker-client/src/bin/http_tracker_client.rs \