Decode HTML entities in action log output (#479)

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Pascal Birchler <pascalb@google.com>

Author: Copilot

dist/index.js

@@ -11,6 +11,7 @@ import { context } from '@actions/github';
 import { existsSync, readFileSync } from 'node:fs';
 import { PRCommentManager } from './pr-comment-manager';
 import { PRCommentFormatter, CheckResult } from './pr-comment-formatter';
+import { decodeHtmlEntities } from './utils';
 
 const args = process.argv.slice(2);
 
@@ -49,7 +50,7 @@ for (let i = 0; i < fileContents.length - 1; i++) {
 			result.type === 'ERROR' || process.env.STRICT === 'true'
 				? error
 				: warning;
-		func(result.message, {
+		func(decodeHtmlEntities(result.message), {
 			title: result.code,
 			file: fileName,
 			startLine: result.line,

dist/index.js.map

@@ -1,3 +1,5 @@
+import { decodeHtmlEntities } from './utils';
+
 export interface CheckResult {
 	line: number;
 	column: number;
@@ -208,7 +210,11 @@ ${sections.join('\n\n')}`;
 
 		const rows = results.map(result => {
 			const line = result.line > 0 ? `\`${result.line}\`` : '`0`';
-			return this.tableRow([line, result.code, result.message]);
+			return this.tableRow([
+				line,
+				result.code,
+				decodeHtmlEntities(result.message),
+			]);
 		});
 
 		return [header, separator, ...rows].join('\n');

src/main.ts

@@ -0,0 +1,37 @@
+/**
+ * Decodes HTML entities in a string.
+ * This handles common HTML entities found in WordPress Plugin Check output.
+ *
+ * @param text - The text containing HTML entities
+ * @returns The decoded text
+ */
+export function decodeHtmlEntities(text: string): string {
+	const namedEntities: Record<string, string> = {
+		'&quot;': '"',
+		'&apos;': "'",
+		'&amp;': '&',
+		'&lt;': '<',
+		'&gt;': '>',
+		'&nbsp;': ' ',
+	};
+
+	return text.replace(
+		/&(?:#x([0-9a-fA-F]+)|#(\d+)|([a-zA-Z]+));/g,
+		(match, hex, dec, named) => {
+			if (hex) {
+				// Hexadecimal entity
+				return String.fromCharCode(parseInt(hex, 16));
+			}
+			if (dec) {
+				// Decimal entity
+				return String.fromCharCode(parseInt(dec, 10));
+			}
+			if (named && namedEntities[`&${named};`]) {
+				// Named entity
+				return namedEntities[`&${named};`];
+			}
+			// Unknown entity, return as-is
+			return match;
+		},
+	);
+}