From 4fb0d9acedfb5712135f30cc351cc46edbead6a8 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 21 Mar 2026 18:10:13 +0000 Subject: [PATCH 1/3] Initial plan From 15d493eefa96b4025b58dbc8c979b46e43446620 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 21 Mar 2026 18:15:03 +0000 Subject: [PATCH 2/3] fix: update integrity filter message - remove min- prefix and info link Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Agent-Logs-Url: https://github.com/github/gh-aw/sessions/6741fb4e-31c9-451c-af8d-3301d527d8e3 --- actions/setup/js/gateway_difc_filtered.cjs | 2 +- actions/setup/js/gateway_difc_filtered.test.cjs | 3 +-- actions/setup/md/integrity_filter_remediation.md | 1 - 3 files changed, 2 insertions(+), 4 deletions(-) diff --git a/actions/setup/js/gateway_difc_filtered.cjs b/actions/setup/js/gateway_difc_filtered.cjs index c03da87abd..4b00e585cf 100644 --- a/actions/setup/js/gateway_difc_filtered.cjs +++ b/actions/setup/js/gateway_difc_filtered.cjs @@ -96,7 +96,7 @@ function generateDifcFilteredSection(filteredEvents) { section += `>
\n`; section += `> 🔒 Integrity filter blocked ${count} ${itemWord}\n`; section += `>\n`; - section += `> The following ${itemWord} were blocked because they don't meet the GitHub [\`min-integrity\`](https://github.github.com/gh-aw/reference/integrity/) level.\n`; + section += `> The following ${itemWord} were blocked because they don't meet the GitHub integrity level.\n`; section += `>\n`; const maxItems = 16; diff --git a/actions/setup/js/gateway_difc_filtered.test.cjs b/actions/setup/js/gateway_difc_filtered.test.cjs index 4542e4f133..b769bdc9cb 100644 --- a/actions/setup/js/gateway_difc_filtered.test.cjs +++ b/actions/setup/js/gateway_difc_filtered.test.cjs @@ -260,7 +260,7 @@ describe("gateway_difc_filtered.cjs", () => { const result = generateDifcFilteredSection(events); expect(result).toContain("blocked because they don't meet"); - expect(result).toContain("min-integrity"); + expect(result).toContain("GitHub integrity level"); }); it("should include frontmatter yaml sample for adjusting min-integrity", () => { @@ -269,7 +269,6 @@ describe("gateway_difc_filtered.cjs", () => { expect(result).toContain("```yaml"); expect(result).toContain("min-integrity:"); - expect(result).toContain("Integrity Filtering"); }); it("should render item lines without parentheses around tool and reason", () => { diff --git a/actions/setup/md/integrity_filter_remediation.md b/actions/setup/md/integrity_filter_remediation.md index 51b95331ba..83861a76b9 100644 --- a/actions/setup/md/integrity_filter_remediation.md +++ b/actions/setup/md/integrity_filter_remediation.md @@ -6,4 +6,3 @@ tools: min-integrity: approved # merged | approved | unapproved | none ``` -See [Integrity Filtering](https://github.github.com/gh-aw/reference/integrity/) for more information. From d7df56a2b8488ee49c4d3b100bf40086cfe2eeab Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 21 Mar 2026 18:34:47 +0000 Subject: [PATCH 3/3] fix: make details summary text bold in integrity filter and firewall blocked sections Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Agent-Logs-Url: https://github.com/github/gh-aw/sessions/430547cc-7977-463f-b7ff-a7e718b43a1d --- actions/setup/js/firewall_blocked_domains.cjs | 2 +- actions/setup/js/firewall_blocked_domains.test.cjs | 4 ++-- actions/setup/js/gateway_difc_filtered.cjs | 2 +- actions/setup/js/gateway_difc_filtered.test.cjs | 8 ++++---- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/actions/setup/js/firewall_blocked_domains.cjs b/actions/setup/js/firewall_blocked_domains.cjs index 01bc24f44b..9de978474b 100644 --- a/actions/setup/js/firewall_blocked_domains.cjs +++ b/actions/setup/js/firewall_blocked_domains.cjs @@ -197,7 +197,7 @@ function generateBlockedDomainsSection(blockedDomains) { let section = "\n\n> [!WARNING]\n"; section += `>
\n`; - section += `> ⚠️ Firewall blocked ${domainCount} ${domainWord}\n`; + section += `> **⚠️ Firewall blocked ${domainCount} ${domainWord}**\n`; section += `>\n`; section += `> The following ${domainWord} ${domainCount === 1 ? "was" : "were"} blocked by the firewall during workflow execution:\n`; section += `>\n`; diff --git a/actions/setup/js/firewall_blocked_domains.test.cjs b/actions/setup/js/firewall_blocked_domains.test.cjs index b23068698f..6da6aeba50 100644 --- a/actions/setup/js/firewall_blocked_domains.test.cjs +++ b/actions/setup/js/firewall_blocked_domains.test.cjs @@ -311,7 +311,7 @@ describe("firewall_blocked_domains.cjs", () => { expect(result).toContain("> [!WARNING]"); expect(result).toContain(">
"); expect(result).toContain(">
"); - expect(result).toContain("> ⚠️ Firewall blocked 1 domain"); + expect(result).toContain("> **⚠️ Firewall blocked 1 domain**"); expect(result).toContain("> - `blocked.example.com`"); expect(result).toContain("> The following domain was blocked by the firewall during workflow execution:"); expect(result).toContain('> ```yaml\n> network:\n> allowed:\n> - defaults\n> - "blocked.example.com"\n> ```'); @@ -325,7 +325,7 @@ describe("firewall_blocked_domains.cjs", () => { expect(result).toContain("> [!WARNING]"); expect(result).toContain(">
"); expect(result).toContain(">
"); - expect(result).toContain("> ⚠️ Firewall blocked 3 domains"); + expect(result).toContain("> **⚠️ Firewall blocked 3 domains**"); expect(result).toContain("> - `alpha.example.com`"); expect(result).toContain("> - `beta.example.com`"); expect(result).toContain("> - `gamma.example.com`"); diff --git a/actions/setup/js/gateway_difc_filtered.cjs b/actions/setup/js/gateway_difc_filtered.cjs index 4b00e585cf..56a7285fed 100644 --- a/actions/setup/js/gateway_difc_filtered.cjs +++ b/actions/setup/js/gateway_difc_filtered.cjs @@ -94,7 +94,7 @@ function generateDifcFilteredSection(filteredEvents) { let section = "\n\n> [!NOTE]\n"; section += `>
\n`; - section += `> 🔒 Integrity filter blocked ${count} ${itemWord}\n`; + section += `> **🔒 Integrity filter blocked ${count} ${itemWord}**\n`; section += `>\n`; section += `> The following ${itemWord} were blocked because they don't meet the GitHub integrity level.\n`; section += `>\n`; diff --git a/actions/setup/js/gateway_difc_filtered.test.cjs b/actions/setup/js/gateway_difc_filtered.test.cjs index b769bdc9cb..414e8b689a 100644 --- a/actions/setup/js/gateway_difc_filtered.test.cjs +++ b/actions/setup/js/gateway_difc_filtered.test.cjs @@ -175,7 +175,7 @@ describe("gateway_difc_filtered.cjs", () => { expect(result).toContain("> [!NOTE]"); expect(result).toContain(">
"); expect(result).toContain(">
"); - expect(result).toContain("> 🔒 Integrity filter blocked 1 item"); + expect(result).toContain("> **🔒 Integrity filter blocked 1 item**"); expect(result).toContain("[#42](https://github.com/org/repo/issues/42)"); expect(result).toContain("`list_issues`"); expect(result).toContain("Integrity check failed"); @@ -202,7 +202,7 @@ describe("gateway_difc_filtered.cjs", () => { const result = generateDifcFilteredSection(events); expect(result).toContain("> [!NOTE]"); - expect(result).toContain("> 🔒 Integrity filter blocked 2 items"); + expect(result).toContain("> **🔒 Integrity filter blocked 2 items**"); expect(result).toContain("[#42](https://github.com/org/repo/issues/42)"); expect(result).toContain("[#99](https://github.com/org/repo/issues/99)"); }); @@ -318,7 +318,7 @@ describe("gateway_difc_filtered.cjs", () => { const result = generateDifcFilteredSection(events); - expect(result).toContain("> 🔒 Integrity filter blocked 2 items"); + expect(result).toContain("> **🔒 Integrity filter blocked 2 items**"); expect(result).toContain("[#42](https://github.com/org/repo/issues/42)"); expect(result).toContain("[#99](https://github.com/org/repo/issues/99)"); }); @@ -363,7 +363,7 @@ describe("gateway_difc_filtered.cjs", () => { const result = generateDifcFilteredSection(events); // Summary still shows the total count - expect(result).toContain("> 🔒 Integrity filter blocked 20 items"); + expect(result).toContain("> **🔒 Integrity filter blocked 20 items**"); // First 16 items rendered expect(result).toContain("[#1](https://github.com/org/repo/issues/1)"); expect(result).toContain("[#16](https://github.com/org/repo/issues/16)");