Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions docs/issues/open/1669-overhaul-packages/EPIC.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ epic: null
github-issue: 1669
spec-path: docs/issues/open/1669-overhaul-packages/EPIC.md
epic-owner: josecelano
last-updated-utc: "2026-10-07 14:54"
last-updated-utc: "2026-10-07 16:39"
semantic-links:
skill-links:
- create-issue
Expand Down Expand Up @@ -668,7 +668,7 @@ Status: TODO unless noted.
- [ ] [#2454](https://github.com/torrust/torrust-tracker/issues/2454) Mark public error enums `#[non_exhaustive]` before first publish _(Pre-publish API checklist; per crate, before its first publish)_
- [ ] Decouple published `test-helpers` from unpublished crates _(draft; finding 1 of the 2026-10-06 coupling report)_
- [ ] Gate server `testing` modules behind a Cargo feature _(draft; finding 2 of the 2026-10-06 coupling report; DEC-13 follow-up)_
- [ ] Fix coupling-tool module name resolution for renamed dependencies and custom library names _(draft; finding 3 of the 2026-10-06 coupling report)_
- [ ] [#2475](https://github.com/torrust/torrust-tracker/issues/2475) Fix coupling-tool module name resolution for renamed dependencies and custom library names _(finding 3 of the 2026-10-06 coupling report)_
- [x] [#1881](https://github.com/torrust/torrust-tracker/issues/1881) SI-16: Migrate `contrib/bencode` to `torrust/torrust-bittorrent` as `torrust-bencode` _(Rule E; no blockers within this EPIC)_
- [x] Extract `torrust-clock` to standalone repository — [#1879](https://github.com/torrust/torrust-tracker/issues/1879) _(Rule E; requires completed clock rename and type move work)_
- [x] Extract `torrust-located-error` to standalone repository — [#1894](https://github.com/torrust/torrust-tracker/issues/1894) _(Rule E; requires completed rename SI-10 #1823)_ — **DONE**
Expand Down Expand Up @@ -709,7 +709,7 @@ Details:
| Tracker client extraction | #TBD — Extract `torrust-tracker-client` to standalone repository | [docs/issues/drafts/1669-extract-torrust-tracker-client-to-standalone-repo/ISSUE.md](../../drafts/1669-extract-torrust-tracker-client-to-standalone-repo/ISSUE.md) | TODO | Rule E; blocked by publication of `udp-protocol`, `http-protocol`, and `tracker-client` lib |
| Test-helpers decoupling | #TBD — Decouple published `test-helpers` from unpublished crates | [docs/issues/drafts/1669-decouple-test-helpers-from-unpublished-crates/ISSUE.md](../../drafts/1669-decouple-test-helpers-from-unpublished-crates/ISSUE.md) | TODO | Draft; finding 1 of the 2026-10-06 coupling report (#2446) |
| Gate server testing | #TBD — Gate server `testing` modules behind a Cargo feature | [docs/issues/drafts/1669-gate-server-testing-modules-behind-feature/ISSUE.md](../../drafts/1669-gate-server-testing-modules-behind-feature/ISSUE.md) | TODO | Draft; finding 2 of the 2026-10-06 coupling report (#2446); DEC-13 follow-up |
| Coupling tool name fix | #TBD — Coupling tool misses renamed dependencies and custom library names | [docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md](../../drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md) | TODO | Draft; finding 3 of the 2026-10-06 coupling report (#2446) |
| Coupling tool name fix | [#2475](https://github.com/torrust/torrust-tracker/issues/2475) — Coupling tool misses renamed dependencies and custom library names | [docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md](../2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md) | TODO | Finding 3 of the 2026-10-06 coupling report (#2446) |
| UDP trait abstractions | [#1924](https://github.com/torrust/torrust-tracker/issues/1924) SI-30: Extract UDP trait abstractions for REST API (`BanningStats`, `UdpCoreStatsRepository`, `UdpServerStatsRepository`) | [docs/issues/closed/1924-1669-si-30-decouple-rest-api-core-from-udp-internals/ISSUE.md](../../closed/1924-1669-si-30-decouple-rest-api-core-from-udp-internals/ISSUE.md) | DONE | UDP-side only; REST-side wiring deferred to #1930; MAX_CONNECTION_ID_ERRORS_PER_IP → config option |
| Cargo deny enforcement | [#1925](https://github.com/torrust/torrust-tracker/issues/1925) SI-31: Configure `cargo deny` for workspace layer boundary enforcement | [docs/issues/closed/1925-1669-si-31-configure-cargo-deny-for-layer-boundary-enforcement/ISSUE.md](../../closed/1925-1669-si-31-configure-cargo-deny-for-layer-boundary-enforcement/ISSUE.md) | DONE | Tooling; create deny.toml with bans for all forbidden edges; add to CI and hooks |
| Versioning policy | [#1926](https://github.com/torrust/torrust-tracker/issues/1926) SI-32: Define package versioning strategy | [docs/issues/closed/1926-1669-si-32-define-package-versioning-strategy/ISSUE.md](../../closed/1926-1669-si-32-define-package-versioning-strategy/ISSUE.md) | DONE | Policy; all packages version independently; path deps make linked versions unnecessary |
Expand Down Expand Up @@ -753,7 +753,7 @@ After SI-14, there is a proposal to evaluate a dedicated repository for protocol
- [docs/issues/drafts/1669-extract-torrust-tracker-client-to-standalone-repo/ISSUE.md](../../drafts/1669-extract-torrust-tracker-client-to-standalone-repo/ISSUE.md)
- [docs/issues/drafts/1669-decouple-test-helpers-from-unpublished-crates/ISSUE.md](../../drafts/1669-decouple-test-helpers-from-unpublished-crates/ISSUE.md)
- [docs/issues/drafts/1669-gate-server-testing-modules-behind-feature/ISSUE.md](../../drafts/1669-gate-server-testing-modules-behind-feature/ISSUE.md)
- [docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md](../../drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md)
- [docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md](../2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md)
- [docs/issues/closed/1930-1669-si-33-rest-api-contract-first-architecture/ISSUE.md](../../closed/1930-1669-si-33-rest-api-contract-first-architecture/ISSUE.md)
- [docs/issues/closed/1910-1669-si-29-rename-udp-and-http-core-protocol-crates-to-remove-redundant-tracker/ISSUE.md](../../closed/1910-1669-si-29-rename-udp-and-http-core-protocol-crates-to-remove-redundant-tracker/ISSUE.md)
> New subissues are created as analysis reveals the next improvement. The EPIC is never
Expand Down Expand Up @@ -990,6 +990,8 @@ Previously referenced tools (screenshots from CodeScene already in the issue com
publishes of a package whose public error API changed, so item 4 applies (PR #2456 review).
- 2026-10-07 14:54 UTC - GitHub Copilot - #2446 closed by PR #2462; its spec moved to
`docs/issues/closed/` and the Details table and Specs Index now link there.
- 2026-10-07 16:39 UTC - GitHub Copilot - Created #2475 from the coupling-tool draft (finding 3),
linked it as a sub-issue, and moved its spec to `docs/issues/open/`.

## Acceptance Criteria

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1222,8 +1222,8 @@ five edges that are used:
(already noted in the previous report).

Reading the library target name and the dependency rename from `cargo metadata` would fix
this. Draft subissue:
[`1669-coupling-tool-resolve-lib-names-and-renames`](../../drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md).
this. Subissue: #2475
([`2475-1669-coupling-tool-resolve-lib-names-and-renames`](../2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md)).

#### Acceptable thin dependencies (not worth addressing)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,20 +2,20 @@
schema-version: 1
doc-type: issue
issue-type: bug
status: draft
status: planned
priority: p3
epic: 1669
github-issue: null
spec-path: docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md
branch: "{issue-number}-1669-coupling-tool-resolve-lib-names-and-renames"
github-issue: 2475
spec-path: docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md
branch: "2475-1669-coupling-tool-resolve-lib-names-and-renames-spec"
related-pr: null
last-updated-utc: "2026-10-07 12:39"
last-updated-utc: "2026-10-07 16:39"
semantic-links:
skill-links:
- create-issue
- fix-bug
related-artifacts:
- docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/manual-verification-evidence.md
- docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/manual-verification-evidence.md
- contrib/dev-tools/analysis/workspace-coupling/src/main.rs
- contrib/dev-tools/analysis/workspace-coupling/tests/parse_imports.rs
- docs/issues/open/1669-overhaul-packages/workspace-coupling-report-2026-10-06.md
Expand All @@ -24,7 +24,7 @@ semantic-links:

<!-- skill-link: create-issue -->

# Issue #[To be assigned] - Coupling tool misses renamed dependencies and custom library names
# Issue #2475 - Coupling tool misses renamed dependencies and custom library names

Subissue of EPIC [#1669](../../open/1669-overhaul-packages/EPIC.md) (Overhaul: Packages).

Expand Down Expand Up @@ -142,9 +142,9 @@ Use a Conventional Commit message with the issue reference and sign every commit

- [x] Spec drafted in `docs/issues/drafts/`
- [x] Bug reproduced and classified in `manual-verification-evidence.md` before review (R1)
- [ ] Spec reviewed and approved by user/maintainer
- [ ] GitHub issue created and issue number added to this spec
- [ ] Spec moved to `docs/issues/open/` with issue number prefix
- [x] Spec reviewed and approved by user/maintainer
- [x] GitHub issue created and issue number added to this spec
- [x] Spec moved to `docs/issues/open/` with issue number prefix
- [ ] Implementation completed
- [ ] Automatic verification completed (`linter all`, relevant tests and pre-push checks)
- [ ] Manual verification scenarios executed and recorded in issue-local
Expand All @@ -165,6 +165,9 @@ Use a Conventional Commit message with the issue reference and sign every commit
per PR #2462 review finding F15. Backfilled at 12:39 UTC, per review finding F20.
- 2026-10-07 12:29 UTC - GitHub Copilot - The link to finding 3 now says it lists all five
edges, per PR #2462 review finding F21.
- 2026-10-07 16:39 UTC - GitHub Copilot - Maintainer approved the draft (reviewed in PR #2462).
Created issue #2475, linked it as a subissue of EPIC #1669, and moved the spec to
`docs/issues/open/`.

## Acceptance Criteria

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
doc-type: manual-verification-evidence
issue-spec: docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md
last-updated-utc: 2026-10-07 08:48
issue-spec: docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md
last-updated-utc: 2026-10-07 16:39
---

<!-- cspell:ignore splitlines startswith -->
Expand Down
Loading