Skip to content

[q] fix(difc): use <b> for bold in details/summary and show #unknown entries with tool name (#22264)#22291

Merged
pelikhan merged 2 commits intomainfrom
q/fix-difc-filtered-message-3994dfa1d733529b
Mar 22, 2026
Merged

[q] fix(difc): use <b> for bold in details/summary and show #unknown entries with tool name (#22264)#22291
pelikhan merged 2 commits intomainfrom
q/fix-difc-filtered-message-3994dfa1d733529b

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Mar 22, 2026

Fixes two issues with the integrity filter blocked message generated by gateway_difc_filtered.cjs.

Changes

1. Use <b> HTML element for bold in <details><summary>

GitHub does not render **markdown bold** inside <summary> tags. Changed to <b>HTML bold</b>:

Before:

<summary>**🔒 Integrity filter blocked N items**</summary>

After:

<summary><b>🔒 Integrity filter blocked N items</b></summary>

2. Show #unknown entries but hide the #unknown text

Events whose resolved reference is #unknown (e.g. description: "github:#unknown") are unresolvable GitHub entities. These entries are now shown in the blocked list, but the #unknown text is hidden — the entry's tool_name is used as the reference label instead (falling back to - if absent). The count in the summary includes these entries.

Tests

All 30 existing tests pass. Updated 2 tests to reflect the new behavior:

  • should show events with #unknown description using tool_name instead
  • should show entry using tool_name when description is #unknown

Triggered by: @pelikhan on #22264

🎩 Equipped by Q ·

  • expires on Mar 24, 2026, 4:34 PM UTC

⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

- Replace **markdown bold** with <b>HTML bold</b> in the details/summary
  element since GitHub doesn't render markdown inside summary tags
- Filter out events whose resolved reference is #unknown — these are
  unresolvable GitHub entities and are not valid entries to surface
- If all events are #unknown the section is suppressed entirely

Fixes #22264

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@pelikhan
Copy link
Contributor

@copilot show the unknown entries, but hide the "#unknown" text

Copy link
Contributor

Copilot AI commented Mar 22, 2026

@copilot show the unknown entries, but hide the "#unknown" text

Done in 18ddccd. #unknown entries are now shown in the list, but the #unknown text is hidden — the entry's tool_name is used as the reference label instead (e.g. `search_issues` `search_issues`: reason).

Copilot AI changed the title [q] fix(difc): use <b> for bold in details/summary and hide #unknown entries (#22264) [q] fix(difc): use <b> for bold in details/summary and show #unknown entries with tool name (#22264) Mar 22, 2026
Copilot AI requested a review from pelikhan March 22, 2026 16:44
@pelikhan pelikhan merged commit 18ec96b into main Mar 22, 2026
@pelikhan pelikhan deleted the q/fix-difc-filtered-message-3994dfa1d733529b branch March 22, 2026 16: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.

2 participants