DOC-7059: Migrate develop/programmability+pubsub+using-commands to render hooks - #3983
Merged
andy-stark-redis merged 1 commit intoSep 11, 2026
Merged
Conversation
…nder hooks Converts relref shortcode links to plain Markdown resolved by render-link.html, per DOC-6909/DOC-7047. Ran build/migrate_shortcode_links.py (all stages) over every .md file under content/develop/programmability, content/develop/pubsub, and content/develop/using-commands. 12 of 13 files changed (337 relref links converted, all to the canonical /content/<path>.md[#anchor] form). multi-key-operations.md was a no-op: it contains zero Hugo shortcodes at all, not even relref, so there was nothing to convert. No {{< note >}}/{{% note %}}/warning/tip/info/alert callouts existed in this unit -- confirmed zero `> [!TYPE]` blockquotes were introduced, so the -spacer adjacency check from DOC-6909/DOC-7047 didn't apply here. One intentional non-rewrite: `[commands](/commands)` in programmability/_index.md stays as a bare relative link rather than being canonicalized, because content/commands has no _index.md/index.md file for linkify's _find_content_file to resolve against (it's a generated section, not a plain content page). This matches the script's own "a missed rewrite is fine, a wrong one is not" rule -- individual command pages like /commands/eval do have real files and did get canonicalized normally. Post-conversion grep across the unit for any remaining {{< relref|note|warning|tip|info|alert >}} (either delimiter form) found nothing left, and a diff scan confirmed no shortcode types outside relref/note/warning/tip/info/alert were touched (clients-example, multitabs, embed-md etc. untouched, as required). Gotcha for future sessions: running the migration script via a shell command that `cd`s to the shared main checkout path (rather than using relative/worktree-local paths) can silently write through to that shared checkout instead of the isolated worktree, because the Bash sandbox's write-allowlist includes cwd ("."). Always run this script with paths relative to the worktree's own cwd, never via `cd <shared-checkout> && ...`. Caught and fully reverted before this commit by restoring the 12 affected shared-checkout files from their git blobs at the shared checkout's actual HEAD commit, verified byte-identical before redoing the conversion correctly in the worktree. Central href-diff and build-warning verification against main, plus human review, are still pending -- to be done across this migration batch's units together in the main checkout, not per-unit. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at bd62376 |
This was referenced Sep 11, 2026
2 tasks
Contributor
Author
|
Thanks @dwdougherty ! |
andy-stark-redis
deleted the
DOC-7059-programmability-pubsub-using-commands
branch
September 11, 2026 13:32
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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
]({{< relref "X" >}})links to plain](X), canonicalized to repo-root-relative/content/<path>.md[#anchor]form, resolved bylayouts/_default/_markup/render-link.html. Part of the DOC-6909/DOC-7047 render-hook rollout, continued under DOC-7059.content/develop/programmability/,content/develop/pubsub/,content/develop/using-commands/(file-disjoint unit; other agents are converting otherdevelop/subdirectories in parallel).build/migrate_shortcode_links.py all <file>(relref-to-plain, callouts-to-blockquote, linkify) over every.mdfile in the three directories.Scope details
content/develop/using-commands/multi-key-operations.mdwas a no-op: it has zero Hugo shortcodes at all (not even relref), so nothing to convert.{{< note >}}/{{% note %}}/warning/tip/info/alert callouts existed in this unit, so zero> [!TYPE]blockquotes were introduced — the -spacer adjacency fix from earlier units didn't apply here.[commands](/commands)inprogrammability/_index.mdstays as a bare relative link becausecontent/commandshas no_index.md/index.mdfile for the script's resolver to match against (it's a generated section). This follows the script's own "a missed rewrite is fine, a wrong one is not" rule — individual command pages like/commands/evaldo resolve and were canonicalized normally.{{< relref|note|warning|tip|info|alert >}}(either delimiter form) found nothing left. Diff scan confirmed no other shortcode types (clients-example,multitabs,embed-md, etc.) were touched.Test plan
main(done across this migration batch's units together, in the main checkout — not part of this PR)main(same batch-wide process)🤖 Generated with Claude Code
Note
Low Risk
Documentation-only link syntax migration with no runtime or product behavior changes; main risk is a broken internal link if render-link resolution differs from the old relref behavior.
Overview
This PR replaces Hugo
relrefshortcodes with plain Markdown links acrosscontent/develop/programmability/,content/develop/pubsub/, andcontent/develop/using-commands/solayouts/_default/_markup/render-link.htmlcan resolve them at build time.Roughly 337 link-only edits in 12 files: targets are canonicalized to repo-root paths like
/content/develop/programmability/lua-api.md#script_flagsand/content/commands/eval.md, with anchors preserved where they existed before. Prose and examples are unchanged.The
/commandsindex link stays a bare/commandspath (no/content/commands/_index.md) because that section has no index page for the migrator to target—individual command pages still use/content/commands/<name>.md.multi-key-operations.mdhad no shortcodes and is untouched.Reviewed by Cursor Bugbot for commit bd62376. Bugbot is set up for automated code reviews on this repo. Configure here.