Skip to content

Clarify docs-scanner broken link rule#24667

Merged
dvdksn merged 1 commit intodocker:mainfrom
dvdksn:dvdksn/scanner-broken-link-clarification
Apr 9, 2026
Merged

Clarify docs-scanner broken link rule#24667
dvdksn merged 1 commit intodocker:mainfrom
dvdksn:dvdksn/scanner-broken-link-clarification

Conversation

@dvdksn
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn commented Apr 8, 2026

Summary

The scanner's "broken cross-reference context" category was ambiguous enough that the agent filed issues for links to non-existent files — which htmltest already catches. Tightened the broken links rule to explicitly cover links to non-existent files, and clarified that "broken cross-reference context" only applies when the URL resolves but the surrounding prose misdescribes the destination.

Learnings

  • The scanner can reclassify a broken link as a "broken cross-reference" to work around its own "do not file broken links" rule if the distinction isn't spelled out explicitly. The fix is to define what each category covers, not just name it.

Relates to #24661
Relates to #24662

Generated by Claude Code

…cross-reference issues

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@dvdksn dvdksn added the status/review Pull requests that are ready for review label Apr 8, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 8, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit ac4956f
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69d659fcb373d40008c28d9f
😎 Deploy Preview https://deploy-preview-24667--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dvdksn dvdksn requested a review from a team April 8, 2026 13:37
@dvdksn dvdksn merged commit f2ee5e1 into docker:main Apr 9, 2026
14 checks passed
@dvdksn dvdksn deleted the dvdksn/scanner-broken-link-clarification branch April 9, 2026 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants