Skip to content

feat: add blocklist support for domain filtering#114

Merged
Mossaka merged 3 commits intomainfrom
copilot/add-support-for-blocklist
Dec 19, 2025
Merged

feat: add blocklist support for domain filtering#114
Mossaka merged 3 commits intomainfrom
copilot/add-support-for-blocklist

Conversation

Copy link
Contributor

Copilot AI commented Dec 19, 2025

  • Understand current codebase architecture
  • Add blocklist support to types (WrapperConfig, SquidConfig)
  • Add --block-domains CLI option
  • Add --block-domains-file CLI option for file-based blocklist
  • Update squid-config.ts to generate ACL rules for blocked domains
  • Add unit tests for blocklist functionality (10 new tests)
  • Merge main branch
  • Update docs-site CLI reference with blocklist options
  • Update docs/usage.md with blocklist documentation and wildcard patterns
  • Run code review and codeql_checker
Original prompt

This section details on the original issue you should resolve

<issue_title>Add supprort for blocklist</issue_title>
<issue_description></issue_description>

<agent_instructions>Plan first and then add support for blocklist</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 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] Add support for blocklist feature feat: add blocklist support for domain filtering Dec 19, 2025
Copilot AI requested a review from Mossaka December 19, 2025 08:35
@Mossaka Mossaka marked this pull request as ready for review December 19, 2025 08:40
@github-actions
Copy link

Test Coverage Report

Metric Coverage Covered/Total
Lines 65.85% 733/1113
Statements 66.02% 754/1142
Functions 71.42% 90/126
Branches 61.02% 249/408
Coverage Thresholds

The project has the following coverage thresholds configured:

  • Lines: 38%
  • Statements: 38%
  • Functions: 35%
  • Branches: 30%

Coverage report generated by `npm run test:coverage`

@Mossaka
Copy link
Collaborator

Mossaka commented Dec 19, 2025

@copilot merge main and please update the docs https://githubnext.github.io/gh-aw-firewall/

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

Copilot AI commented Dec 19, 2025

@copilot merge main and please update the docs https://githubnext.github.io/gh-aw-firewall/

Merged main and updated the documentation. The CLI reference at docs-site and docs/usage.md now include the --block-domains and --block-domains-file options with usage examples. (ee14a84)

@github-actions
Copy link

💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...]

@github-actions
Copy link

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link

Smoke Test Results

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved PR data
  • ✅ File Writing: Created test file
  • ✅ Bash Tool: Verified file contents

Status: PASS

📰 BREAKING: Report filed by Smoke Copilot fer issue #114 🗺️

@github-actions
Copy link

Smoke Test Results - Claude Engine

✅ GitHub MCP - Last 2 merged PRs reviewed
✅ File Writing - Created test file successfully
❌ Bash Tool - File read verified
❌ Playwright MCP - Page loaded but asset errors (tunnel connection failed)

Recent PRs:

Status: PARTIAL PASS - Core tools working, Playwright has network issues

💥 [THE END] — Illustrated by Smoke Claude fer issue #114 🗺️

@Mossaka Mossaka merged commit dfd5209 into main Dec 19, 2025
27 checks passed
@Mossaka Mossaka deleted the copilot/add-support-for-blocklist branch December 19, 2025 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add supprort for blocklist

2 participants