Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 4, 2026

Reduce README bloat by keeping “Get started fast” terse while still showing installation and a minimal usage example.

  • README cleanup
    • Keep a two-line quick start (install + basic awf usage) and add a one-sentence note explaining the -- separator.
curl -sSL https://raw.githubusercontent.com/github/gh-aw-firewall/main/install.sh | sudo bash
sudo awf --allow-domains github.com -- curl https://api.github.com

The -- separator divides firewall options from the command to run.

README get started fast section update

Custom agent used: technical-doc-writer
AI technical documentation writer for awf library using Astro Starlight

Original prompt

unbloat the readme, get started fast section should just be one line to get started. no more than that

Custom agent used: technical-doc-writer
AI technical documentation writer for awf library using Astro Starlight


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor 'Get Started Fast' section of README docs: condense get started fast section Feb 4, 2026
Copilot AI requested a review from Mossaka February 4, 2026 00:36
Copy link
Collaborator

@Mossaka Mossaka left a comment

Choose a reason for hiding this comment

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

Review: REQUEST_CHANGES

@copilot Please fix the following issues:

Problem

The "Get started fast" section now only shows installation but removes the usage example entirely. A "Get started fast" section that only shows how to install - but not how to USE - the tool defeats its purpose.

What was removed that should be kept:

  • The example command showing how to actually run awf after installation
  • The explanation about the -- separator (critical syntax)

Required Fix

Keep at least one usage example. A minimal but complete version:

## Get started fast

```bash
curl -sSL https://raw.githubusercontent.com/github/gh-aw-firewall/main/install.sh | sudo bash
sudo awf --allow-domains github.com -- curl https://api.github.com

This keeps it terse (two lines) while showing both installation AND basic usage.

The prerequisite note about Docker can optionally be removed, but the usage example is essential.

Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 4, 2026

Review: REQUEST_CHANGES

@copilot Please fix the following issues:...

Added a minimal usage example and a one-sentence -- separator note to the Get started fast section. Screenshot: https://github.com/user-attachments/assets/7db60ed1-c454-4607-b26c-9e26477e6792. Commit: 5b0fd21.

Copilot AI requested a review from Mossaka February 4, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants