From b773792cd63b20fd17a350355811c509a3d0d127 Mon Sep 17 00:00:00 2001 From: Imani Manyara Date: Fri, 13 Nov 2026 21:43:33 -0500 Subject: [PATCH] Correct the stale mismatch count in the slug resolver The docstring explained why the slug is read from `origin` rather than from the directory name, and illustrated it with `pdf-toolkit` is `laranail/pdf`. That checkout has been renamed to `pdf`, so the example named a directory that no longer exists and the count was one too high. Now six of laranail's 73 checkouts, with `env-tools` is `laranail/env-kit` as the example, plus the command and date the figure came from so the next reader can re-derive it instead of trusting it. The resolver itself is unchanged, and still returns the real slug for a mismatched directory: repo_slug_of('env-tools') gives 'env-kit'. --- scripts/verify-vcs-urls.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/scripts/verify-vcs-urls.py b/scripts/verify-vcs-urls.py index f6ad85c..9fae748 100755 --- a/scripts/verify-vcs-urls.py +++ b/scripts/verify-vcs-urls.py @@ -129,9 +129,14 @@ def repo_slug_of(pkg: str, root: Path | None, org: str) -> str: """The package's real repository slug. In a checkout this is read from `origin`, never from the directory name: - seven directories in the tree are named differently from their repository - (`pdf-toolkit` is `laranail/pdf`), so trusting the name would report a SPLIT + six of laranail's 73 checkouts are named differently from their repository + (`env-tools` is `laranail/env-kit`), so trusting the name would report a SPLIT that is a local naming choice rather than a rename. + + Measured 2026-11-13, comparing each `packages/*/` basename against + `git -C remote get-url origin`. It read seven until that day, when the + `laranail/pdf` checkout was renamed from `pdf-toolkit` to `pdf` and stopped + being one of them; the example moved with it. """ if root is None: return pkg