Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 13, 2026

The Firewall Activity section used a heading outside the <details> tag, making it non-collapsible unlike other report sections (Safe Inputs MCP Server Logs, MCP Gateway).

Changes

  • src/logs/stats-formatter.ts: Moved "Firewall Activity" into <summary> tag, added prefix to stats line
  • src/logs/stats-formatter.test.ts: Updated test expectations
  • docs-site/src/content/docs/reference/cli-reference.md: Updated example output

Before/After

Before:

### Firewall Activity

<details>
<summary>46 requests | 36 allowed | 10 blocked | 9 unique domains</summary>

After:

<details>
<summary>Firewall Activity</summary>

▼ 46 requests | 36 allowed | 10 blocked | 9 unique domains

Screenshot

Format comparison showing collapsible section

Original prompt

This section details on the original issue you should resolve

<issue_title>markdown match style of other reports</issue_title>
<issue_description>Show as collapsed details section

Image</issue_description>

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


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update markdown to match style of other reports feat: make firewall activity collapsible like other report sections Jan 13, 2026
Copilot AI requested a review from Mossaka January 13, 2026 08:02
@Mossaka Mossaka marked this pull request as ready for review January 14, 2026 18:11
@Mossaka Mossaka requested a review from Copilot January 14, 2026 18:11
@github-actions
Copy link

Test Coverage Report

Metric Coverage Covered/Total
Lines 77.12% 1281/1661
Statements 77.17% 1312/1700
Functions 77.04% 141/183
Branches 69.29% 431/622
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`

Copy link

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 makes the Firewall Activity section collapsible by moving the heading inside the <summary> tag, matching the design pattern used by other report sections like Safe Inputs MCP Server Logs and MCP Gateway.

Changes:

  • Moved "Firewall Activity" heading from outside to inside the <summary> tag
  • Added prefix to the stats line and moved it outside the <summary> tag
  • Updated all test expectations to match the new format

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/logs/stats-formatter.ts Restructured markdown output to make the section properly collapsible with title in summary tag
src/logs/stats-formatter.test.ts Updated test expectations to verify the new collapsible format
docs-site/src/content/docs/reference/cli-reference.md Updated example output documentation to reflect the new format

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Mossaka Mossaka merged commit edfee3f into main Jan 15, 2026
48 checks passed
@Mossaka Mossaka deleted the copilot/update-markdown-style branch January 15, 2026 01:53
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.

markdown match style of other reports

2 participants