Uh oh!
There was an error while loading. Please reload this page.
Use link targets that resolve for the aggregation cross-references - #65
Merged
guanzhousongmicrosoft merged 1 commit intoAug 3, 2026
Conversation
Seventeen links across six pages point at relative .md paths and every one of them 404s. On $search the link to $vectorSearch renders as href="./%24vectorsearch.md", which resolves against the page's own directory - documentdb.io serves these with a trailing slash - and lands at /operators/aggregation/$search/%24vectorsearch.md. Wrong depth and a leaked extension, the two failure modes documentdb#57 catalogued, in the same href. They were introduced together. $vectorSearch, $project, $limit and $graphLookup arrived in documentdb#59, $search in documentdb#64 following the convention it found on the page next to it, and $meta links back to $vectorSearch the same way. None of them render, so the pages read as cross-linked while every cross-link is dead. Rewritten to the absolute form the rest of the reference already uses - https://documentdb.io/docs/reference/operators/aggregation/%24bucket/ - which is what documentdb#57 settled on for exactly this reason: it does not depend on how the site resolves a relative path, and it survives a page moving between directories. All thirteen distinct targets were requested against the live site and return 200, and no relative .md link remains anywhere in the repository.
guanzhousongmicrosoft
approved these changes
Aug 3, 2026
guanzhousongmicrosoft
left a comment
Contributor
There was a problem hiding this comment.
Confirmed against the live site: the relative form 404s, all 13 absolute targets return 200, and no relative .md link remains in the repo.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Seventeen cross-reference links across six pages point at relative
.mdpaths, and every one of them 404s.On
$search, the link to$vectorSearchrenders as:documentdb.io serves these pages with a trailing slash, so that resolves against the page's own directory and lands at
/operators/aggregation/$search/%24vectorsearch.md— wrong depth and a leaked extension, the two failure modes #57 catalogued, in a single href. Confirmed 404 against the live site.Affected
$search.md$project.md$vectorsearch.md$graphlookup.md$limit.md$meta.mdThey were introduced together:
$vectorSearch,$project,$limitand$graphLookuparrived in #59,$searchin #64 following the convention it found on the page beside it, and$metalinks back the same way. None of them render, so those pages read as cross-linked while every cross-link is dead.Fix
Rewritten to the absolute form the rest of the reference already uses:
That is what #57 settled on for exactly this reason — it does not depend on how the site resolves a relative path, and it survives a page moving between directories.
Verification
All 13 distinct targets were requested against the live site and return 200. No relative
.mdlink remains anywhere in the repository:Related: this is the class of #38, which stays open for the wider
.md-in-links question.