[docs] docs: remove bloat from Network Permissions reference#26151
Merged
[docs] docs: remove bloat from Network Permissions reference#26151
Conversation
- Fix Access Levels section: was "four access levels" but listed five items; removed misclassified domain-matching behaviors (items 4-5) and added a cross-reference to the dedicated Wildcard Domain Patterns section - Condense AWF feature list from 6 bullets to one sentence - Condense SSL bump Security Considerations from 4 bullets to prose Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
pelikhan
approved these changes
Apr 14, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR reduces redundancy and corrects a factual inconsistency in the Network Permissions reference documentation.
Changes:
- Corrected the “Access Levels” section to describe three access levels (and removed two items that were domain-matching behaviors, not access levels), adding a cross-reference to the wildcard section.
- Condensed the AWF “When enabled” bullet list into a single sentence while retaining key behavioral details.
- Converted the SSL bump “Security Considerations” bullets into concise prose without dropping technical meaning.
Show a summary per file
| File | Description |
|---|---|
| docs/src/content/docs/reference/network.md | Removes repetitive bullets, fixes an incorrect access-level count, and adds a clearer cross-reference to wildcard behavior. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 0
This was referenced Apr 14, 2026
Closed
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.
Targeted unbloating of
docs/src/content/docs/reference/network.md.What was improved
File:
docs/src/content/docs/reference/network.mdReduction: 312 → 302 lines (-10 lines), 63 → 53 bullets (-10 bullets, ~16% reduction)
Changes made
Fixed Access Levels section (factual error + bloat): The section claimed "four access levels" but listed five items. Items 4–5 ("Automatic Subdomain Matching" and "Wildcard Patterns") are domain-matching behaviors, not access levels, and are already covered in the dedicated "Wildcard Domain Patterns" section lower on the page. Removed the two misclassified items, corrected the count to three, and added a cross-reference link.
Condensed AWF feature list (6 bullets → 1 sentence): The "When enabled, AWF:" block listed six bullets that mostly restated what a firewall does. Condensed to a single inline sentence preserving the key specifics (
--allow-domainsflag, subdomain behavior, wildcard support, audit logging).Condensed SSL bump Security Considerations (4 bullets → prose): Four bullets converted to two sentences, retaining all technical content (man-in-the-middle nature, when to use it,
allow-urlscaution, AWF version requirement, SRT exclusion).Screenshots
Visual screenshot capture was unavailable due to network isolation between the Playwright container and the agent container in this AWF configuration. Page content was verified via
curl— the rendered "Access Levels" section correctly shows "three access levels" with 3 items and a cross-reference link to the Wildcard Domain Patterns section.Blocked domains
N/A — screenshots were not captured.
References: §24379858317