-
Notifications
You must be signed in to change notification settings - Fork 33
Fix broken documentation links and add KB-only build support #209
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Conversation
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
- Renamed 5 dot-prefixed .NET dependency files (.net_* → net_*) - Renamed 3 number-prefixed files (0x*, 503_* → error-*, service-*) - Updated internal cross-references to renamed files These filenames were causing invalid URL paths in Docusaurus build. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Removed broken relative path navigation from all 8 KB product index.md files. These files contained links to non-existent hierarchical folders like ./installation/, ./configuration/, etc. that were removed during KB flattening. Replaced with simple product-specific landing pages to eliminate 800+ broken link errors. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Fixed broken cross-references to main documentation: - threatmanager: 2.8 → 3.0 (version does not exist) - threat-prevention: 7.3/stealthintercept/installation → threatprevention/7.4/install These version mismatches were causing additional broken link errors. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Fixed all broken cross-references with hyphenated product names: - data-classification → dataclassification - password-reset → passwordreset - password-policy-enforcer → passwordpolicyenforcer - threat-prevention → threatprevention These hyphenated names don't match actual directory structure and were causing hundreds of broken link errors. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Fixed additional broken cross-references: - auditor/11.x → auditor/10.8 (versions 11.x don't exist) - directory-manager → directorymanager - access-analyzer → accessanalyzer These systematic fixes address the major sources of the 800+ broken link errors reported in the build. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix activity-monitor → activitymonitor in keywords (50+ files) - Fix endpoint-policy-manager → endpointpolicymanager in keywords - Fix log-tracker → logtracker in keywords - Fix access-analyzer path references in cross-links - Fix data-classification path references in cross-links - Fix directory-manager path references in cross-links - Fix endpoint-protector path references in cross-links - Fix password-reset path references in cross-links All product names now consistently use non-hyphenated format to match actual directory structure and prevent broken links. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix auditor version mismatches: 10.0/10.5/10.6/10.7 → 10.8 - Fix .net_dependencies file references with dot prefixes - Fix malformed markdown link syntax (missing brackets) - Fix doubled words: configurationuration → configuration, installationation → installation - Update product versions: dataclassification 5.6.2 → 5.7, passwordreset 3.3 → 3.23, activitymonitor 7.1 → 8.0 - Fix wrong Configure MFA cross-reference to correct recoveryad article Reduces broken /docs/ references from 249 to ~220 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix external twilio URL to use https prefix - Update product versions to match available directories: - passwordpolicyenforcer 10.2 → 11.0 - threatprevention 7.3/7.4 → 7.5 - endpointprotector 2.20/5.9.3 → 5.9.4.2 - privilegesecure 4.0/4.1 → 4.2 - directorymanager 11.0 → 11.1 - Add path fixes to prevent broken admin-guide references Still 235 broken references remaining - mostly path structure issues 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove extra /auditor/ component: /docs/auditor/10.8/auditor/ → /docs/auditor/10.8/ - Fix path names: /access → /accessreviews, /enterpriseauditor/ → /addon/ - Fix admin paths: /admin-guide/ → /admin/ - Fix KB directory names: privilege-secure-access-management → privilegesecure, changetracker → general - Fix KB article filenames: underscores → hyphens for existing files - Fix self-referencing MFA links Reduced broken references from 249 to 232 (fixed 17 structural issues) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
… to 165 - Fixed 84+ broken references using systematic parallel agent approach - Addressed filename issues, path structure problems, and version mismatches - Ready to deploy targeted agents for remaining 165 references by category 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix /do../../auditor/ patterns to /docs/auditor/ - Fix /do../../accessanalyzer/ patterns to /docs/accessanalyzer/ - Fix /do/docs/ patterns to /docs/ - These were causing broken link warnings in docusaurus builds 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Convert internal cross-instance links to external URLs: - KB files can't link directly to other plugin instances - Convert /docs/accessanalyzer/* and /docs/auditor/* to https://docs.netwrix.com URLs - Fixes docusaurus multi-instance link resolution warnings 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Increase Node.js memory limit to 31GB - Disable memory-intensive optimization features: - swcJsLoader, swcJsMinimizer, swcHtmlMinimizer - lightningCssMinimizer, rspackBundler - rspackPersistentCache, ssgWorkerThreads - Keep mdxCrossCompilerCache for essential performance - Address Netlify build failure (exit code 137 - memory exhaustion) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Set DOCS_PRODUCT=auditor to build only Auditor documentation - Add build-full script for complete builds when needed - Dramatically reduces memory usage by filtering products - Should prevent Netlify exit code 137 (memory exhaustion) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add special case for DOCS_PRODUCT=kb to build only Knowledge Base - Drastically reduces memory usage by avoiding all product documentation - Prevents duplicate KB plugin when building KB exclusively - Should allow successful builds on memory-constrained environments 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This reverts commit dd21955.
james-s-anderson
approved these changes
Sep 16, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Test plan
🤖 Generated with Claude Code