DOC-6909 Teach the feed pipeline and version archiver the /content/ link form by andy-stark-redis · Pull Request #3796 · redis/docs

andy-stark-redis · 2026-08-12T13:30:39Z

Part 2 of 4. Stack: #3795 → #3796 (this) → #3797 → #3798

What this is

Updates the two tools that read relref as a literal string, ahead of any content moving off it. This is @paoloredis's review point from #3732, and it turns out not to be a "before we remove relref" concern — it bites per section, at migration time.

Both changes are inert until content actually uses repo-root-relative links, so this is safe to land on its own.

Tool	Blind spot	Consequence
`process-markdown-content.html`	rewrote only `relref` to absolute URLs	a migrated page ships raw source paths into the AI Markdown/JSON feed, resolving nowhere
`version_archiver.py`	pattern matched only `relref`	links in an archived version keep resolving to latest instead of the frozen copy — silently, no error

The archiver one was found by probe rather than by reasoning, and it's the more expensive: wrong content in a frozen version, discovered late.

Notes for review

Both patterns anchor on a Markdown link destination (](/content/…), not the bare /content/ prefix — otherwise they rewrite the github.com blob URLs that legitimately embed the same path segment. There's a test pinning this.
The archiver shares one replace_link closure across both notations, so the release-notes exemption and the already-versioned guard can't drift apart.
Source-relative links need no archiver rewriting — the whole subtree is copied, so a link between two pages inside it already resolves within the version. test_source_relative_link_is_left_alone asserts the resolution, not just that the text is unchanged.
build/test_version_archiver.py (8 tests) was confirmed to fail with only the new substitution disabled, so it isn't passing vacuously. Like its build/test_*.py siblings it isn't wired into CI.

Not covered

394 .html.md files still leak pre-existing ../ relative links into the feed, plus 21 with rooted non-/content/ links. That defect predates this work entirely and is out of scope here.

Note

Medium Risk
Changes affect archived-version link correctness and AI-facing URLs, but behavior is inert until migrated content uses /content/ links and is backed by targeted archiver tests.

Overview
Teaches the docs pipeline and version archiver to handle repo-root-relative Markdown links (](/content/...) the same way they already handle relref, ahead of DOC-6909 section migrations.

In process-markdown-content.html, new replaceRE rules rewrite those link destinations to https://redis.io/docs/latest/..., including _index.md / index.md handling so section URLs match published paths; anchors stay outside the match.

In version_archiver.py, a second pattern anchored on ](/content/<prefix>/<product>/...) runs through the existing replace_link logic so frozen versions do not keep pointing at latest; release-notes exemption, idempotency, and “don’t touch other products / GitHub blob URLs” behavior stay shared with relref.

Adds build/test_version_archiver.py (8 cases) covering both notations, anchors, source-relative links left alone, and false-positive guards.

^{Reviewed by Cursor Bugbot for commit 522e174. Bugbot is set up for automated code reviews on this repo. Configure here.}

github-actions · 2026-08-12T13:30:54Z

DOC-6909

github-actions · 2026-08-12T13:30:56Z

🧠 Redis Memory

Found 11 related items from repository history:

Commit — DOC-6909 Version plain /content/ links when archiving a version

DOC-6909 Version plain /content/ links when archiving a version
Commit — DOC-6909 Rewrite /content/ links in the AI Markdown and JSON output

DOC-6909 Rewrite /content/ links in the AI Markdown and JSON output
Commit — DOC-6909 Use repo-root-relative /content/ links for redis-py

DOC-6909 Use repo-root-relative /content/ links for redis-py
Commit — DOC-6909 Migrate develop/clients/redis-py to portable Markdown links and callouts

DOC-6909 Migrate develop/clients/redis-py to portable Markdown links and callouts
Commit — DOC-6909 Fix the remaining pre-existing broken links surfaced by the hook

DOC-6909 Fix the remaining pre-existing broken links surfaced by the hook
Commit — DOC-6909 Teach the feed pipeline and version archiver the /content/ link form

DOC-6909 Teach the feed pipeline and version archiver the /content/ link form
PR — DOC-6909 Migrate develop/clients/redis-py to portable Markdown links and callouts

Part 4 of 4. Stack: DOC-6909 Fix pre-existing broken links across the docs #3795 → DOC-6909 Teach the feed pipeline and version archiver the /content/ link form #3796 → DOC-6909 Add link and blockquote render hooks to replace relref #3797 → DOC-6909 Migrate develop/clients/redis-py to portable Markdown links and callouts #3798 (this)
Commit — DOC-6909 Teach the feed pipeline and version archiver the /content/ link form

DOC-6909 Teach the feed pipeline and version archiver the /content/ link form
Commit — DOC-6909 Teach the feed pipeline and version archiver the /content/ link form

DOC-6909 Teach the feed pipeline and version archiver the /content/ link form
Commit — DOC-6909 Teach the feed pipeline and version archiver the /content/ link form

DOC-6909 Teach the feed pipeline and version archiver the /content/ link form
Commit — DOC-6909 Teach the feed pipeline and version archiver the /content/ link form

DOC-6909 Teach the feed pipeline and version archiver the /content/ link form

Memory updated at 522e174

cursor

Cursor Bugbot has reviewed your changes using high effort and found 3 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit c87015c. Configure here.}

cursor · 2026-08-12T15:05:34Z

 {{- $content := .RawContent -}}
 {{- $visited := .Visited | default (slice) -}}

-{{- /* Drop HTML-comment blocks before anything else looks at the content.


HTML comment stripping removed

High Severity

This commit drops the line-anchored HTML-comment strip from process-markdown-content.html, so draft blocks authors park in  now flow into the AI Markdown/JSON feed as published body text. toc-from-markdown.html still strips those comments and documents that the two must stay aligned, so TOC and body also diverge.

^{Reviewed by Cursor Bugbot for commit c87015c. Configure here.}

cursor · 2026-08-12T15:05:34Z

-{{- /* Match non-greedily to the first ">}}": a [^>]* class is defeated by a ">" inside
-       an attribute value (e.g. columnNames="...<br/>...") and leaks the raw shortcode. */ -}}
-{{- $content = $content | replaceRE `(?s)\{\{<\s*/?.*?>\}\}` "" -}}
+{{- $content = $content | replaceRE `\{\{<\s*/?[^>]*>\}\}` "" -}}


Table-children expansion dropped

High Severity

The call to markdown-table-children.html (and its follow-up unescape) was removed from process-markdown-content.html, even though the partial still exists and documents that without it release-notes index bodies are emptied in the feed. Pages whose only body is a table-children shortcode lose their tables and version metadata columns.

^{Reviewed by Cursor Bugbot for commit c87015c. Configure here.}

…ink form Prepares the two tools that read relref as a literal string, ahead of any content moving off it. Both changes are inert until content actually uses repo-root-relative links, so this is safe to land on its own. The AI-facing Markdown and JSON output rewrote only relref to absolute redis.io URLs, so a migrated page would ship raw source paths that resolve nowhere. The version archiver had the same blind spot with worse consequences: links in an archived version would keep resolving to the latest page instead of the frozen copy, silently, with no error. Both patterns anchor on a Markdown link destination rather than the bare content prefix, or they would rewrite the github.com blob URLs that legitimately embed the same path segment. The archiver shares one replace_link closure between notations so the release-notes exemption and the already-versioned guard cannot drift apart. Source-relative links need no archiver rewriting: the whole subtree is copied, so a link between two pages inside it already resolves within the version. The accompanying tests pin that, and were confirmed to fail with only the new substitution disabled. Amended after Bugbot found that the first version of this commit silently reverted two DOC-6939 improvements to the same partial. The stack was carved out of the old single PR by checking each file out wholesale from a tag, which replaces the file rather than applying a delta, so any change main had made to it in the meantime was undone: the line-anchored HTML-comment strip, the table-children expansion, and a catch-all shortcode regex that had been tightened from [^>]* to a non-greedy match. Nothing warned, because reverting those produces silently wrong feed output rather than a build error, and the verification build was green. The rules are now inserted into main's current version of the file, so this commit is purely additive. Constraint: carve a branch out of an old state with a delta, never a wholesale file checkout from a tag, or any change main made to that file since the branch point is silently reverted Ticket: DOC-6909 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

dwdougherty

LGTM, but this definitely needs @paoloredis's eyes.

This was referenced Aug 12, 2026

DOC-6909 Fix pre-existing broken links across the docs #3795

Open

DOC-6909 Migrate develop/clients/redis-py to portable Markdown links and callouts #3798

Open

DOC-6909 Add link and blockquote render hooks to replace relref #3797

Open

andy-stark-redis force-pushed the DOC-6909-2-link-tooling branch from 6ab0ad7 to c87015c Compare August 12, 2026 15:01

andy-stark-redis marked this pull request as ready for review August 12, 2026 15:01

cursor Bot reviewed Aug 12, 2026

View reviewed changes

andy-stark-redis force-pushed the DOC-6909-2-link-tooling branch from c87015c to 7ade771 Compare August 12, 2026 15:25

andy-stark-redis marked this pull request as draft August 12, 2026 15:26

andy-stark-redis force-pushed the DOC-6909-2-link-tooling branch from 7ade771 to 07b7c9f Compare August 12, 2026 15:32

andy-stark-redis marked this pull request as ready for review August 12, 2026 15:45

andy-stark-redis mentioned this pull request Aug 12, 2026

DOC-6909 Add link render hook to replace relref (investigation) #3732

Closed

github-actions Bot mentioned this pull request Aug 13, 2026

DOC-6968 Fix two node-redis step names, and add the missing incr step for nine clients #3809

Merged

andy-stark-redis force-pushed the DOC-6909-2-link-tooling branch from 07b7c9f to b99b3a8 Compare August 14, 2026 13:19

andy-stark-redis force-pushed the DOC-6909-2-link-tooling branch from b99b3a8 to 522e174 Compare August 14, 2026 13:20

andy-stark-redis requested review from dwdougherty and paoloredis August 14, 2026 13:53

andy-stark-redis self-assigned this Aug 14, 2026

andy-stark-redis added the infrastructure label Aug 14, 2026

dwdougherty approved these changes Aug 14, 2026

View reviewed changes

Navigation Menu

Uh oh!

DOC-6909 Teach the feed pipeline and version archiver the /content/ link form - #3796

DOC-6909 Teach the feed pipeline and version archiver the /content/ link form#3796
andy-stark-redis wants to merge 1 commit into
DOC-6909-1-link-fixesfrom
DOC-6909-2-link-tooling

andy-stark-redis commented Aug 12, 2026 •

edited by cursor Bot

Loading

Uh oh!

github-actions Bot commented Aug 12, 2026 •

edited by atlassian Bot

Loading

Uh oh!

github-actions Bot commented Aug 12, 2026 •

edited

Loading

Uh oh!

cursor Bot left a comment

Uh oh!

cursor Bot Aug 12, 2026

Uh oh!

cursor Bot Aug 12, 2026

Uh oh!

Uh oh!

dwdougherty left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

andy-stark-redis commented Aug 12, 2026 • edited by cursor Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What this is

Notes for review

Not covered

Uh oh!

github-actions Bot commented Aug 12, 2026 • edited by atlassian Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Aug 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🧠 Redis Memory

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Aug 12, 2026

Choose a reason for hiding this comment

HTML comment stripping removed

Uh oh!

cursor Bot Aug 12, 2026

Choose a reason for hiding this comment

Table-children expansion dropped

Uh oh!

Uh oh!

dwdougherty left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

andy-stark-redis commented Aug 12, 2026 •

edited by cursor Bot

Loading

github-actions Bot commented Aug 12, 2026 •

edited by atlassian Bot

Loading

github-actions Bot commented Aug 12, 2026 •

edited

Loading