Skip to content

[WIP] Add AWF firewall to Copilot agent#2187

Closed
Copilot wants to merge 7 commits intomossaka/gh-aw-firewallfrom
copilot/stack-pr-2161
Closed

[WIP] Add AWF firewall to Copilot agent#2187
Copilot wants to merge 7 commits intomossaka/gh-aw-firewallfrom
copilot/stack-pr-2161

Conversation

Copy link
Contributor

Copilot AI commented Oct 23, 2025

Thanks for the feedback on #2161. I'm creating this stack PR to address your comment and will keep this PR's description up to date as I make progress.

Original PR: #2161
Triggering comment (#2161 (comment)):

@copilot
Add a minimalistic feature flag helper in the compiler .go code that reads the environment variable GH_AW_FEATURES and determine if the "flag" is in the env at string. Make sure to do case insensitive string comparisons. No need for caching.

Use the "firewall" feature test to determine if copilot should enable generating code for the firewall; otherwise use the same generated code.


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

Mossaka and others added 7 commits October 22, 2025 13:48
Signed-off-by: Jiaxiao Zhou <duibao55328@gmail.com>
Signed-off-by: Jiaxiao Zhou <duibao55328@gmail.com>
Signed-off-by: Jiaxiao Zhou <duibao55328@gmail.com>
Signed-off-by: Jiaxiao (mossaka) Zhou <duibao55328@gmail.com>
Signed-off-by: Jiaxiao (mossaka) Zhou <duibao55328@gmail.com>
Signed-off-by: Jiaxiao (mossaka) Zhou <duibao55328@gmail.com>
@Mossaka Mossaka force-pushed the mossaka/gh-aw-firewall branch 2 times, most recently from 45e3d70 to b367af0 Compare October 23, 2025 00:36
@Mossaka Mossaka closed this Oct 23, 2025
Copilot AI requested a review from pelikhan October 23, 2025 00:38
@pelikhan pelikhan deleted the copilot/stack-pr-2161 branch December 15, 2025 04:52
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.

3 participants