Skip to content

Commit 1293cc3

Browse files
committed
chore(issues): address Copilot PR review comments on #1886
- Fix broken EPIC link (../open/ -> sibling) in #1885 spec - Align CI acceptance criterion with deferred note in #1885 spec - Rename Quick list section 4 heading (stale 'no GitHub issue' claim) - Rename 'Draft issues' section to 'Subissue Specs Index' - Fix stale clock spec path in EPIC details table (open/ -> closed/) - Fix semantics-links typo -> semantic-links in closed #1879 spec
1 parent e159b61 commit 1293cc3

3 files changed

Lines changed: 6 additions & 6 deletions

File tree

docs/issues/closed/1879-1669-17-extract-torrust-clock-to-standalone-repo.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ spec-path: docs/issues/closed/1879-1669-17-extract-torrust-clock-to-standalone-r
88
branch: 1879-1669-extract-torrust-clock-to-standalone-repo
99
related-pr: 1880
1010
last-updated-utc: 2026-06-05 00:00
11-
semantics-links:
11+
semantic-links:
1212
skill-links:
1313
- create-issue
1414
related-artifacts:

docs/issues/open/1669-overhaul-packages/EPIC.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -545,7 +545,7 @@ Status: TODO unless noted.
545545
- [ ] [#1884](https://github.com/torrust/torrust-tracker/issues/1884) SI-19: Move `bittorrent-peer-id` to `torrust/torrust-bittorrent` as `torrust-peer-id` _(Rule E; no workspace deps; first `bittorrent-*` extraction)_
546546
- [ ] [#1885](https://github.com/torrust/torrust-tracker/issues/1885) SI-20: Extract `torrust-net-primitives` to standalone repository _(Rule E; no workspace deps; no prerequisites)_
547547

548-
#### 4. Draft Specs (No Subissue Number, No GitHub Issue)
548+
#### 4. Other Tracked Items (Drafts and Promoted Issues)
549549

550550
- [ ] Establish baseline: dependency graph + README audit _(analysis; no blockers; informs all other subissues)_
551551
- [ ] Update all package READMEs _(documentation; after completed rename work; before extractions)_
@@ -575,7 +575,7 @@ Details:
575575
| README refresh | #TBD — Update all package READMEs | [docs/issues/drafts/1669-update-all-package-readmes.md](../../drafts/1669-update-all-package-readmes.md) | TODO | Documentation; requires completed rename work; before extraction work |
576576
| Bencode migration | [#1881](https://github.com/torrust/torrust-tracker/issues/1881) SI-16: Migrate `contrib/bencode` to `torrust/torrust-bittorrent` as `torrust-bencode` | [docs/issues/open/1881-1669-16-migrate-contrib-bencode-to-torrust-bittorrent/ISSUE.md](../../open/1881-1669-16-migrate-contrib-bencode-to-torrust-bittorrent/ISSUE.md) | DONE | Rule E; torrust-bencode 3.0.0 published; contrib/bencode removed from tracker workspace |
577577
| Peer-ID move | [#1884](https://github.com/torrust/torrust-tracker/issues/1884) — Move `bittorrent-peer-id` to `torrust/torrust-bittorrent` as `torrust-peer-id` | [docs/issues/open/1884-1669-19-move-bittorrent-peer-id-to-torrust-bittorrent.md](../../open/1884-1669-19-move-bittorrent-peer-id-to-torrust-bittorrent.md) | TODO | Rule E; no workspace deps in crate; 3 consumers to migrate; first `bittorrent-*` extraction sequence item |
578-
| Clock extraction | [#1879](https://github.com/torrust/torrust-tracker/issues/1879) — Extract `torrust-clock` to standalone repository | [docs/issues/open/1879-1669-17-extract-torrust-clock-to-standalone-repo.md](../../open/1879-1669-17-extract-torrust-clock-to-standalone-repo.md) | DONE | Rule E; torrust-clock v3.0.0 published; 13 consumers migrated; packages/clock removed |
578+
| Clock extraction | [#1879](https://github.com/torrust/torrust-tracker/issues/1879) — Extract `torrust-clock` to standalone repository | [docs/issues/closed/1879-1669-17-extract-torrust-clock-to-standalone-repo.md](../../closed/1879-1669-17-extract-torrust-clock-to-standalone-repo.md) | DONE | Rule E; torrust-clock v3.0.0 published; 13 consumers migrated; packages/clock removed |
579579
| Metrics extraction | [#1882](https://github.com/torrust/torrust-tracker/issues/1882) — Extract `torrust-metrics` to standalone repository | [docs/issues/open/1882-1669-18-extract-torrust-metrics-to-standalone-repo.md](../../open/1882-1669-18-extract-torrust-metrics-to-standalone-repo.md) | TODO | Rule E; requires completed metrics rename; 7 workspace consumers to migrate |
580580
| Net-primitives extraction | [#1885](https://github.com/torrust/torrust-tracker/issues/1885) — Extract `torrust-net-primitives` to standalone repository | [docs/issues/open/1885-1669-20-extract-torrust-net-primitives-to-standalone-repo.md](../../open/1885-1669-20-extract-torrust-net-primitives-to-standalone-repo.md) | TODO | Rule E; no workspace deps; no prerequisites; 10 consumers to migrate; new repo torrust/torrust-net-primitives |
581581
| Tracker client extraction | #TBD — Extract `torrust-tracker-client` to standalone repository | [docs/issues/drafts/1669-extract-torrust-tracker-client-to-standalone-repo.md](../../drafts/1669-extract-torrust-tracker-client-to-standalone-repo.md) | TODO | Rule E; blocked by `torrust-tracker-udp-tracker-protocol` publication (external to this EPIC) |
@@ -593,7 +593,7 @@ Details:
593593
Proposal note:
594594
After SI-14, there is a proposal to evaluate a dedicated repository for protocol crates so protocol packages can evolve with BEP/spec changes while tracker app packages evolve with domain/product changes. This is proposal-only for now (not committed scope) and is tracked in [#1835](https://github.com/torrust/torrust-tracker/issues/1835) and [docs/issues/open/1835-1669-14-decouple-http-protocol-from-tracker-primitives.md](../../open/1835-1669-14-decouple-http-protocol-from-tracker-primitives.md).
595595

596-
### Draft issues
596+
### Subissue Specs Index
597597

598598
- [docs/issues/drafts/1669-01-establish-baseline-analysis.md](../../drafts/1669-01-establish-baseline-analysis.md)
599599
- [docs/issues/drafts/1669-update-all-package-readmes.md](../../drafts/1669-update-all-package-readmes.md)

docs/issues/open/1885-1669-20-extract-torrust-net-primitives-to-standalone-repo.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ The crate has **zero workspace-path dependencies** — all its runtime deps (`se
5757
The crate is **not yet published on crates.io**; publication from the standalone repository
5858
is part of this issue's scope.
5959

60-
This issue is a subissue of EPIC [#1669](../open/1669-overhaul-packages/EPIC.md)
60+
This issue is a subissue of EPIC [#1669](1669-overhaul-packages/EPIC.md)
6161
(Overhaul: Packages).
6262

6363
## Scope
@@ -160,7 +160,7 @@ Status values: `TODO`, `IN_PROGRESS`, `BLOCKED`, `DONE`.
160160

161161
- [ ] A standalone repository `torrust/torrust-net-primitives` exists on GitHub.
162162
- [ ] The repository contains the crate source (history preservation where practical).
163-
- [ ] CI in the new repository passes.
163+
- [ ] CI in the new repository passes _(may be deferred — see note below)_.
164164
- [ ] `torrust-net-primitives` is published and visible on crates.io.
165165
- [ ] No `Cargo.toml` in the tracker workspace references `torrust-net-primitives` with a path dep.
166166
- [ ] `packages/net-primitives` is absent from the `[workspace]` members list in root `Cargo.toml`.

0 commit comments

Comments
 (0)