feat(server-lib): extract torrust-server-lib to standalone repository#1922
Conversation
There was a problem hiding this comment.
Pull request overview
This PR completes the extraction of torrust-server-lib from the torrust-tracker Cargo workspace into a standalone repository (torrust/torrust-server-lib) and migrates all in-workspace consumers to the crates.io release (torrust-server-lib = "0.1.0"), removing the in-repo packages/server-lib/ crate.
Changes:
- Migrates all workspace consumers (and the root workspace dependency registration) from a path dependency on
packages/server-libtotorrust-server-lib = "0.1.0". - Removes the
packages/server-lib/crate from this repository (source, Cargo.toml, README, LICENSE). - Updates package inventory/docs to reflect
server-libas an extracted package and updates EPIC #1669 tracking.
Reviewed changes
Copilot reviewed 18 out of 19 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| Cargo.toml | Switches workspace dep registration for torrust-server-lib from path to crates.io 0.1.0. |
| Cargo.lock | Updates lockfile entry to registry-sourced torrust-server-lib 0.1.0. |
| AGENTS.md | Moves server-lib to extracted packages list (but currently misses clock in that table). |
| packages/AGENTS.md | Updates architecture + package catalog to classify server-lib as extracted. |
| docs/packages.md | Removes server-lib from in-workspace list and adds it to extracted packages table. |
| docs/issues/open/1909-1669-si-28-extract-server-lib-to-standalone-repo.md | Marks the extraction work as completed and updates checklists/status (with some internal inconsistencies to resolve). |
| docs/issues/open/1669-overhaul-packages/EPIC.md | Updates EPIC inventory and extracted packages tracking to include torrust-server-lib. |
| packages/axum-health-check-api-server/Cargo.toml | Migrates torrust-server-lib dependency from path to crates.io 0.1.0. |
| packages/axum-http-server/Cargo.toml | Migrates torrust-server-lib dependency from path to crates.io 0.1.0. |
| packages/axum-rest-api-server/Cargo.toml | Migrates torrust-server-lib dependency from path to crates.io 0.1.0. |
| packages/axum-server/Cargo.toml | Migrates torrust-server-lib dependency from path to crates.io 0.1.0. |
| packages/udp-server/Cargo.toml | Migrates torrust-server-lib dependency from path to crates.io 0.1.0. |
| packages/server-lib/src/signals.rs | Removed as part of deleting the extracted crate from this repo. |
| packages/server-lib/src/registar.rs | Removed as part of deleting the extracted crate from this repo. |
| packages/server-lib/src/logging.rs | Removed as part of deleting the extracted crate from this repo. |
| packages/server-lib/src/lib.rs | Removed as part of deleting the extracted crate from this repo. |
| packages/server-lib/README.md | Removed as part of deleting the extracted crate from this repo. |
| packages/server-lib/LICENSE | Removed as part of deleting the extracted crate from this repo. |
| packages/server-lib/Cargo.toml | Removed as part of deleting the extracted crate from this repo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #1922 +/- ##
===========================================
- Coverage 79.42% 79.41% -0.01%
===========================================
Files 327 324 -3
Lines 23024 22963 -61
Branches 23024 22963 -61
===========================================
- Hits 18287 18237 -50
+ Misses 4476 4462 -14
- Partials 261 264 +3 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
- Create standalone repo torrust/torrust-server-lib - Publish crate as v0.1.0 on crates.io - Set up CI workflow with linting and testing - Update all 6 workspace consumers to crates.io dep - Remove packages/server-lib/ from workspace - Update AGENTS.md, packages/AGENTS.md, docs/packages.md Closes torrust#1909
61d579e to
0886da4
Compare
|
ACK 22eb810 |
Summary
Extract
torrust-server-libto its own standalone repository at torrust/torrust-server-lib.Changes
Cargo.tomltorrust-server-libv0.1.0 on crates.iotorrust-server-lib = "0.1.0":Cargo.tomlworkspace dep registrationaxum-health-check-api-server,axum-http-server,axum-rest-api-serveraxum-server,udp-serverpackages/server-lib/removed from workspaceAGENTS.md,packages/AGENTS.md,docs/packages.mdupdated to moveserver-libto extracted sectionVerification
cargo build --workspacepassescargo test --doc --workspacepassescargo test --tests --workspace --all-targets --all-featurespasseslinter allpassescargo macheteclean (pre-existing unrelated warning only)torrust-server-libremainpackages/server-lib/directory removedCloses #1909