Skip to content

Commit ca87cfc

Browse files
authored
Fix the nightly report summaries (#1251)
* 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.
1 parent d791831 commit ca87cfc

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

.github/workflows/check-pr-target.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
name: Check PR Target Branch
22

33
on:
4-
pull_request:
4+
pull_request_target:
55
branches: [main]
66
types: [opened]
77

.github/workflows/skill-quality-report.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -158,10 +158,11 @@ jobs:
158158
const codeowners = parseCodeowners();
159159
160160
// Count findings
161+
// The skill-validator uses emoji markers: ❌ for errors, ⚠ for warnings, ℹ for advisories
161162
const combined = skillsOutput + '\n' + agentsOutput;
162-
const errorCount = (combined.match(/\bError\b/gi) || []).length;
163-
const warningCount = (combined.match(/\bWarning\b/gi) || []).length;
164-
const advisoryCount = (combined.match(/\bAdvisory\b/gi) || []).length;
163+
const errorCount = (combined.match(/❌/g) || []).length;
164+
const warningCount = (combined.match(/⚠/g) || []).length;
165+
const advisoryCount = (combined.match(/ℹ\uFE0F?/g) || []).length;
165166
166167
// Count total skills & agents checked
167168
let skillDirs = [];

0 commit comments

Comments
 (0)