Skip to content

Fix the fallout from renaming the rule to done-video-by-ai - #13282

Merged
isaaclombardssw merged 1 commit into
mainfrom
fix-rule-rename
Sep 3, 2026
Merged

Fix the fallout from renaming the rule to done-video-by-ai#13282
isaaclombardssw merged 1 commit into
mainfrom
fix-rule-rename

Conversation

@isaaclombardssw

Copy link
Copy Markdown
Member

#13280 renamed the rule from ai-capture-ui-changes to done-video-by-ai. That left three things behind, plus the two changes from my earlier PR that now need to land on the new path.

The old URL 404s. The rule was live at /ai-capture-ui-changes. Added a redirects entry for it.

Seven rules linked to the old URI. playwright-with-ai, definition-of-done, done-video, write-a-good-pull-request, devtools-design-changes, ai-assisted-desktop-pr-preview and utilize-back-pressure-for-agents all pointed at /ai-capture-ui-changes. Repointed to /done-video-by-ai.

Images were left in the old folder. The rule folder name has to match the uri, so the images moved into done-video-by-ai/.

Clearer PR example screenshots. The demo PR behind the good/bad examples only changed the card hover border, so the pair was hard to read at a glance. It now changes the card title colour too. Recaptured both PR pages, and the images got new names so nothing serves the cached old pair. Demo PRs: #5012 and #5013.

Added Puppeteer to the tooling list.

Frontmatter, check-mdx, malformed-bold, endIntro and category-sync all pass.

🤖 Generated with Claude Code

The rename left the old URL 404ing, seven inbound links pointing at it,
and the images in a folder that no longer matches the uri. Also swaps in
clearer PR example screenshots and adds Puppeteer to the tooling list.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@github-actions github-actions Bot added the Age: 🥚 - New About 2 hours old label Sep 3, 2026
@isaaclombardssw
isaaclombardssw merged commit 8159c4b into main Sep 3, 2026
13 checks passed
@isaaclombardssw
isaaclombardssw deleted the fix-rule-rename branch September 3, 2026 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Age: 🥚 - New About 2 hours old

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants