Refine validmind-docs-coverage skill: var-in-heading anchors + author-prefixed branches#1437
Merged
Merged
Conversation
Warn that {{< var >}} shortcodes inside headings produce unusable
auto-generated ids, and generalize the branch-naming convention from
codex/ to author-prefixed branches.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
|
Contributor
Lighthouse check results✓ INFO: No site pages to audit in this PR. Commit SHA: 56234a3 |
cachafla
approved these changes
Jul 24, 2026
Contributor
Validate docs site✓ INFO: A live preview of the docs site is available — Open the preview |
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
3 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.
Pull Request Description
What and why?
Implements the two refinements proposed in #1435, both verified empirically (see issue comment):
{{< var >}}shortcodes render unusable auto-generated ids (reproduced onmain:manage-b58fc729-...7b7b3c20...-runs), and to add an explicit{#id}to any heading used as a link target.codex/sc-<story-id>-<slug>to author-prefixed<author>/sc-<story-id>-<slug>, matching existing practice ([SC-16822] Documentation: Risk Tiering #1425juan/..., [SC-16823] Documentation: Document Checker runs, publishing, and artifacts #1434 and [SC-16959] Documentation: workflow execution taken path and Step Detail panels #1436panchicore/...) —pr_previews/<branch>/works identically.Closes #1435.
How to test
Skill-text-only change; no site sources touched. Both behaviors verified against current
mainas documented in the issue comment.What needs special review?
Wording only — @cachafla as skill author.
Dependencies, breaking changes, and deployment notes
None.
Release notes
Internal tooling change; no release notes.
Checklist
🤖 Generated with Claude Code