[vprashar2929]

Includes the dependency updates from #186 and the compatibility fixes needed to merge them into main ahead of the next mdox release.

• Use *glob.Pattern in transform configuration and pin Colly to the upstream Glob v1 compatibility commit.
• Match Colly's URL normalization for requests, results, and cache keys. mdox deduplicates equivalent destinations while Colly can revisit shared redirect targets. Preserve the original request key and diagnostics through redirected retries.
• Remove resolved reference-definition AST nodes before both formatting passes, preserving mdox's existing inline-link output with Goldmark 1.8.6. Replace deprecated Goldmark text accessors.
• Add local HTTP-server regression coverage for URL variants, deduplication, caching, and redirect targets, plus reference-link formatting and idempotence tests. Verify duplicate links share the final cancellation error.

Colly is pinned to v2.3.1-0.20260902064301-a210c5b4210f because v2.3.0 does not support Glob v1, which the Hugo update requires. This is an upstream pseudo-version containing 35 commits after v2.3.0; it introduces no fork, vendoring, or module replacements. Current main already uses a Colly pseudo-version.

Validation:

• Full Go test suite with race detection: go test -mod=readonly -race -count=1 -timeout=5m ./....
• Latest cancellation assertion: TestValidator_BackoffDoesNotBlockOtherLinks passed 20 runs with race detection.
• Project-pinned make lint GOBIN=/tmp/mdox-pr186-tools, including build, formatting, vet, documentation generation, and clean-tree checks.
• go mod verify.

[bwplotka]

Thanks!

One suggestion only

[bwplotka]

Colly v2.3 now marks redirect targets as visited in checkRedirectFunc when AllowURLRevisit is false, so if an earlier link redirects to /target, a later direct link to /target fails here with AlreadyVisitedError and two links redirecting to /target trigger a 2s retry backoff. Since r.remoteLinks[dest] already deduplicates visits, could we pass colly.AllowURLRevisit() to colly.NewCollector in NewValidator and cover this in TestValidator_RedirectedResult?

[vprashar2929]

Thanks it makes sense. I have added colly.AllowURLRevisit() and extended the existing redirect test to cover both cases.

[bwplotka]

With colly.AllowURLRevisit() enabled in NewValidator, Colly no longer returns AlreadyVisitedError here. Checking that err.Error() contains two "canceled during backoff" entries instead of visitedErr would still verify that the duplicate link shares the final error.

[vprashar2929]

Agreed