diff --git a/docs/issues/open/1669-overhaul-packages/EPIC.md b/docs/issues/open/1669-overhaul-packages/EPIC.md index b7f368f87..eb4cc0121 100644 --- a/docs/issues/open/1669-overhaul-packages/EPIC.md +++ b/docs/issues/open/1669-overhaul-packages/EPIC.md @@ -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 @@ -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** @@ -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 | @@ -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 @@ -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 diff --git a/docs/issues/open/1669-overhaul-packages/workspace-coupling-report-2026-10-06.md b/docs/issues/open/1669-overhaul-packages/workspace-coupling-report-2026-10-06.md index f90ddc318..78a924ebd 100644 --- a/docs/issues/open/1669-overhaul-packages/workspace-coupling-report-2026-10-06.md +++ b/docs/issues/open/1669-overhaul-packages/workspace-coupling-report-2026-10-06.md @@ -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) diff --git a/docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md b/docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md similarity index 93% rename from docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md rename to docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md index c89a1b0bf..d4975fa4b 100644 --- a/docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md +++ b/docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/ISSUE.md @@ -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 @@ -24,7 +24,7 @@ semantic-links: -# 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). @@ -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 @@ -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 diff --git a/docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/manual-verification-evidence.md b/docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/manual-verification-evidence.md similarity index 97% rename from docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/manual-verification-evidence.md rename to docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/manual-verification-evidence.md index a38abd381..9f6ed4fbf 100644 --- a/docs/issues/drafts/1669-coupling-tool-resolve-lib-names-and-renames/manual-verification-evidence.md +++ b/docs/issues/open/2475-1669-coupling-tool-resolve-lib-names-and-renames/manual-verification-evidence.md @@ -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 ---