Refactor default decision point values from collect.md to includable files#1013
Merged
ahouseholder merged 20 commits intomainfrom Oct 14, 2025
Merged
Conversation
Co-authored-by: ahouseholder <[email protected]>
Copilot
AI
changed the title
[WIP] Refactor default exploitation values to an includable
Refactor default exploitation values from collect.md to an includable file
Oct 10, 2025
Co-authored-by: ahouseholder <[email protected]>
Co-authored-by: ahouseholder <[email protected]>
Co-authored-by: ahouseholder <[email protected]>
Co-authored-by: ahouseholder <[email protected]>
Co-authored-by: ahouseholder <[email protected]>
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the "Default Exploitation Values" tip from a single location into a reusable include file to improve content maintainability and consistency across the documentation.
- Created a new include file
docs/_includes/default_exploitation_values.mdcontaining the tip content - Replaced the original tip in
collect.mdwith an include statement - Added the include to two additional documentation pages for broader coverage
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/_includes/default_exploitation_values.md | New include file containing the default exploitation values tip with relative paths |
| docs/howto/bootstrap/collect.md | Replaced inline tip content with include statement |
| docs/reference/decision_points/exploitation.md | Added include for the default exploitation values tip |
| docs/howto/gathering_info/exploitation.md | Added include for the default exploitation values tip |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: ahouseholder <[email protected]>
sei-vsarvepalli
approved these changes
Oct 14, 2025
Contributor
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Removed an include statement for default safety values and clarified the expansive view of safety in relation to well-being.
…ation-values' into copilot/refactor-default-exploitation-values
This was
linked to
issues
Oct 14, 2025
This was referenced Oct 14, 2025
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
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.
This pull request refactors how default values for several decision points are documented by moving their content into dedicated include files and updating various documentation pages to use these includes. This improves maintainability and consistency across the documentation. Additionally, it updates the markdown include plugin configuration to better handle relative links.
Documentation refactoring:
Exploitation,System Exposure,Safety Impact, andMission Impactin thedocs/_includesdirectory, consolidating their content for reuse. [1] [2] [3] [4]collect.md,gathering_info/exploitation.md,gathering_info/mission_impact.md,gathering_info/system_exposure.md,decision_points/exploitation.md,decision_points/mission_impact.md,decision_points/safety_impact.md,decision_points/system_exposure.md) to use the new markdown includes for default decision point values, replacing inline content. [1] [2] [3] [4] [5] [6] [7] [8]Plugin configuration improvements:
include-markdownplugin configuration inmkdocs.ymlto enablerewrite_relative_links, ensuring that relative links in included markdown files resolve correctly.Minor cleanup:
mkdocs.ymlfor tidiness.