Skip to content

Commit 0a820f3

Browse files
committed
chore: address Copilot review comments
- Fix broken ADR link in docs/packages.md (remove docs/ prefix) - Add missing README.md + LICENSE for rest-api-application and rest-api-runtime-adapter packages - Clarify peer.updated vs peer.updated_milliseconds_ago in DTO docs - Mark architecture documentation checkbox complete in issue spec Part of EPIC #1669
1 parent 3d1f96f commit 0a820f3

7 files changed

Lines changed: 1348 additions & 4 deletions

File tree

docs/issues/open/1930-1669-si-33-rest-api-contract-first-architecture.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -434,4 +434,4 @@ Why discarded:
434434
- `rest-api-application`: `TorrentQueryPort` + `TorrentApiService` use case
435435
- `rest-api-runtime-adapter`: `TrackerTorrentQueryAdapter` + conversion functions
436436
- `axum-rest-api-server`: handler dispatches via use case instead of direct `tracker-core`
437-
- [ ] Target architecture documented in `docs/packages.md` (or a dedicated ADR).
437+
- [x] Target architecture documented in `docs/packages.md` and `docs/adrs/`. Verdict: ADR 20260623200526 + packages.md REST API section.

docs/packages.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ contrib/
5959

6060
The REST API uses a **contract-first layered architecture** with four distinct
6161
layers and enforced dependency direction. See
62-
[ADR 20260623200526](docs/adrs/20260623200526_adopt_contract-first_architecture_for_rest_api.md)
62+
[ADR 20260623200526](adrs/20260623200526_adopt_contract-first_architecture_for_rest_api.md)
6363
for the full architectural decision and alternatives considered.
6464

6565
```mermaid

packages/rest-api-application/LICENSE

Lines changed: 661 additions & 0 deletions
Large diffs are not rendered by default.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# Torrust Tracker REST API Application
2+
3+
Application/use-case layer for the Torrust Tracker REST API.
4+
5+
## Documentation
6+
7+
[Crate documentation](https://docs.rs/torrust-tracker-rest-api-application).
8+
9+
## License
10+
11+
The project is licensed under the terms of the [GNU AFFERO GENERAL PUBLIC LICENSE](./LICENSE).

packages/rest-api-protocol/src/v1/resources/peer.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ pub struct Peer {
88
pub peer_id: Id,
99
/// The peer's socket address. For example: `192.168.1.88:17548`.
1010
pub peer_addr: String,
11-
/// The peer's last update time in milliseconds.
11+
/// The peer's last update time as a Unix timestamp in milliseconds since epoch.
1212
#[deprecated(since = "2.0.0", note = "please use `updated_milliseconds_ago` instead")]
1313
pub updated: u128,
14-
/// The peer's last update time in milliseconds.
14+
/// Milliseconds since the peer's last update (relative to the response generation time).
1515
pub updated_milliseconds_ago: u128,
1616
/// The peer's uploaded bytes.
1717
pub uploaded: i64,

packages/rest-api-runtime-adapter/LICENSE

Lines changed: 661 additions & 0 deletions
Large diffs are not rendered by default.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# Torrust Tracker REST API Runtime Adapter
2+
3+
Tracker-specific runtime adapter for the REST API application layer.
4+
5+
## Documentation
6+
7+
[Crate documentation](https://docs.rs/torrust-tracker-rest-api-runtime-adapter).
8+
9+
## License
10+
11+
The project is licensed under the terms of the [GNU AFFERO GENERAL PUBLIC LICENSE](./LICENSE).

0 commit comments

Comments
 (0)