Conversation
|
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
There was a problem hiding this comment.
Pull Request Overview
This pull request fixes broken documentation links throughout the MWAP project documentation. The changes address 31 broken internal links that were identified during documentation validation, ensuring all cross-references point to existing files and updating archive references appropriately.
Key changes include:
- Updated OAuth documentation references to use correct filenames (oauth-security.md instead of oauth-security-guide.md)
- Fixed archive folder references for deprecated documentation
- Updated configuration to properly exclude archive directories from validation
Reviewed Changes
Copilot reviewed 13 out of 35 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/README.md | Updated main navigation links for OAuth guides and removed references to non-existent AI-Agents files |
| docs/LINK-VALIDATION-SUMMARY.md | Added comprehensive validation summary documenting all 31 link fixes |
| docs/09-Reports-and-History/*.md | Updated OAuth guide references in historical reports to point to correct files or archives |
| docs/06-Guides/*.md | Fixed OAuth security guide references and updated archive paths |
| docs/06-Guides/archive/COMPLETE-IMPLEMENTATION-SUMMARY.md | Removed duplicate archived file from main guides directory |
| docs/04-Backend/api-quickreference.md | Fixed relative path to OAuth integration guide |
| .docs-config.json | Updated validation configuration with correct directory structure and enhanced archive exclusion patterns |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| - [Performance Guide](./06-Guides/performance-guide.md) - API configuration and optimization strategies | ||
| - [Testing Guide](./06-Guides/testing-guide.md) - Complete testing strategies and implementation | ||
| - [OAuth Security Guide](./06-Guides/oauth-security-guide.md) - OAuth callback security and PKCE implementation | ||
| - [OAuth Security Guide](./06-Guides/oauth-security.md) - OAuth security threat models and best practices |
There was a problem hiding this comment.
The PR title contains a spelling error: 'docuemntaiton' should be 'documentation'.
| ], | ||
| "excludePatterns": [ | ||
| "docs/**/archive/**", | ||
| "docs/archive/**", |
There was a problem hiding this comment.
[nitpick] The pattern 'docs//archive/' should cover 'docs/archive/**' as well, making the second pattern redundant. Consider removing the more specific pattern to reduce duplication.
| "docs/archive/**", |
Pull Request
Description
Brief description of the changes in this PR.
Type of Change
Related Issues
Closes #(issue number)
Changes Made
Documentation Impact
Testing
npm run docs:validateto check documentation linksSecurity Considerations
Architecture Compliance
Checklist
Screenshots (if applicable)
Add screenshots to help explain your changes.
Additional Notes
Any additional information that reviewers should know.