Fix relrefs for .md files in redisvl latest release#3079
Fix relrefs for .md files in redisvl latest release#3079paoloredis wants to merge 1 commit intomainfrom
Conversation
🛡️ Jit Security Scan Results✅ No security findings were detected in this PR
Security scan by Jit
|
|
This change fixes relrefs for the concepts content, e.g. the links at the bottom of this page https://redis.io/docs/latest/develop/ai/redisvl/concepts/architecture/ |
|
@paoloredis The regexes don't seem to be quite right. For example, when I click the link to Search & Indexing at the bottom of the Architecture page, the URL is ...which gives a 404. The actual link should be https://redis.io/docs/staging/DOC-6504/develop/ai/redisvl/concepts/search-and-indexing (ie, no |
|
Yeah, that's expected. Right now the redisvl workflow isn't running as part of staging builds so pages won't be updated |
andy-stark-redis
left a comment
There was a problem hiding this comment.
OK, then - approved.
Note
Medium Risk
Changes the docs sync workflow’s link-rewrite rules, which could affect many generated pages if the new regexes mis-handle edge cases or anchors.
Overview
Updates the
redisvl_docs_syncGitHub Actions workflow to rewrite intra-doc*.mdlinks into Hugorelrefshortcodes during the markdown formatting step, including support for stripping numeric filename prefixes (e.g.01_) and handling both path-based and same-directory links.This reduces broken navigation in the generated RedisVL “latest” docs by ensuring cross-page links resolve correctly after files are renamed/organized for Hugo.
Reviewed by Cursor Bugbot for commit e782c05. Bugbot is set up for automated code reviews on this repo. Configure here.