docs(audit): fix broken internal markdown links#19
Merged
Conversation
Pass 3 of the 2026-05 docs audit: link integrity. Walked all 656 inline markdown links across 31 files (docs/*.md, AGENTS.md, README.md, .github/copilot-instructions.md, CONTRIBUTING.md, CHANGELOG.md). Found 3 broken targets, all pointing at code paths that do not exist. Inventory: - docs/DEVELOPING.md:181 -> ../src/Drederick/Exploit/ICredTool.cs Interface does not exist; cred tools (PasswordSprayTool, KerberoastTool, AsRepRoastTool) currently implement IExploitTool. Fix: removed the broken link and clarified in prose that a dedicated ICredTool marker is not yet split out. - docs/DEVELOPING.md:326 -> ../tests/fixtures/bin/ Directory does not exist (tests/fixtures/ has only data fixtures, no fake-binary replay subdir yet). Fix: un-linked the path; prose still describes the intended layout. - docs/PLUGIN_STRATEGY.md:218 -> ../src/Drederick/Exploit/ICredTool.cs Same as above. Fix: un-linked ICredTool and annotated as planned. Post-fix: 0 broken links across 653 surviving inline references. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
1 participant
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.
Pass 3 of the 2026-05 docs audit: link integrity.
Scope
Walked every inline and reference-style markdown link in:
docs/*.md(26 files)AGENTS.md,README.md,CONTRIBUTING.md,CHANGELOG.md.github/copilot-instructions.mdExternal
http(s)://links skipped. Verified file existence + anchorexistence (explicit
<a id="x">and slugified markdown headings).Results
The PR #17 fix in
docs/LEARNING_LOOP.mdfordocs/AUTOPILOT.mdisverified clean — not re-fixed.
Broken-link inventory
docs/DEVELOPING.md:181../src/Drederick/Exploit/ICredTool.csPasswordSprayTool,KerberoastTool,AsRepRoastTool) implementIExploitToolICredTool, kept prose, clarified that the marker is not yet split outdocs/DEVELOPING.md:326../tests/fixtures/bin/tests/fixtures/contains only data fixtures: nmap XML, NVD mini, GHSA / Tenable / searchsploit samples — no fake-binary replay subdir yet)docs/PLUGIN_STRATEGY.md:218../src/Drederick/Exploit/ICredTool.csWhy these fixes (not deletes / not stubs)
Per the audit fix policy: the
ICredToolinterface is referenced inprose as a real concept but the file does not exist. Cred tools
implement
IExploitTooltoday (verified via grep onPasswordSprayTool.cs/KerberoastTool.cs), so the prose is updatedto reflect actual code.
tests/fixtures/bin/is an aspirational layout— left in prose, link removed so readers don't get a 404.
No target docs were fabricated. No code was changed.
Coordination
Touches only
docs/DEVELOPING.mdanddocs/PLUGIN_STRATEGY.md—disjoint from
docs-audit-llm-tool-coherenceanddocs-audit-root-readme-quickstartzones.Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com