Skip to content

[docs] docs: remove bloat from Network Permissions reference#26151

Merged
pelikhan merged 1 commit intomainfrom
docs/unbloat-network-fc4206a62e4e127f
Apr 14, 2026
Merged

[docs] docs: remove bloat from Network Permissions reference#26151
pelikhan merged 1 commit intomainfrom
docs/unbloat-network-fc4206a62e4e127f

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Targeted unbloating of docs/src/content/docs/reference/network.md.

What was improved

File: docs/src/content/docs/reference/network.md
Reduction: 312 → 302 lines (-10 lines), 63 → 53 bullets (-10 bullets, ~16% reduction)

Changes made

  1. 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.

  2. 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-domains flag, subdomain behavior, wildcard support, audit logging).

  3. 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-urls caution, 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

🗜️ Compressed by Documentation Unbloat · ● 419.2K ·

  • expires on Apr 16, 2026, 3:55 AM UTC

- 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>
@github-actions github-actions bot added automation documentation Improvements or additions to documentation labels Apr 14, 2026
@pelikhan pelikhan marked this pull request as ready for review April 14, 2026 03:56
Copilot AI review requested due to automatic review settings April 14, 2026 03:56
@pelikhan pelikhan merged commit 4fba480 into main Apr 14, 2026
3 checks passed
@pelikhan pelikhan deleted the docs/unbloat-network-fc4206a62e4e127f branch April 14, 2026 03:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants