From ca87cfcc9ef3d7e0383fb2ea6fe5443cfc735d95 Mon Sep 17 00:00:00 2001 From: Jan Krivanek Date: Thu, 2 Apr 2026 01:01:32 +0200 Subject: [PATCH] Fix the nightly report summaries (#1251) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Fix the nightly report summaries * Fix workflow trigger * Fix advisory regex to handle optional variation selector Match ℹ with or without the trailing U+FE0F variation selector so advisory counts are reliable regardless of output encoding. --- .github/workflows/check-pr-target.yml | 2 +- .github/workflows/skill-quality-report.yml | 7 ++++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/.github/workflows/check-pr-target.yml b/.github/workflows/check-pr-target.yml index d0e5834c..05f24fa7 100644 --- a/.github/workflows/check-pr-target.yml +++ b/.github/workflows/check-pr-target.yml @@ -1,7 +1,7 @@ name: Check PR Target Branch on: - pull_request: + pull_request_target: branches: [main] types: [opened] diff --git a/.github/workflows/skill-quality-report.yml b/.github/workflows/skill-quality-report.yml index 61889ff3..9bd036ad 100644 --- a/.github/workflows/skill-quality-report.yml +++ b/.github/workflows/skill-quality-report.yml @@ -158,10 +158,11 @@ jobs: const codeowners = parseCodeowners(); // Count findings + // The skill-validator uses emoji markers: ❌ for errors, ⚠ for warnings, ℹ for advisories const combined = skillsOutput + '\n' + agentsOutput; - const errorCount = (combined.match(/\bError\b/gi) || []).length; - const warningCount = (combined.match(/\bWarning\b/gi) || []).length; - const advisoryCount = (combined.match(/\bAdvisory\b/gi) || []).length; + const errorCount = (combined.match(/❌/g) || []).length; + const warningCount = (combined.match(/⚠/g) || []).length; + const advisoryCount = (combined.match(/ℹ\uFE0F?/g) || []).length; // Count total skills & agents checked let skillDirs = [];