fix(ci): preserve mutation annotation punctuation

Author: roomote

scripts/stryker-diff.mjs

@@ -422,13 +422,16 @@ export function mutantCounts(report) {
 	return counts
 }
 
-function escapeWorkflowValue(value) {
-	return String(value)
-		.replaceAll("%", "%25")
-		.replaceAll("\r", "%0D")
-		.replaceAll("\n", "%0A")
-		.replaceAll(":", "%3A")
-		.replaceAll(",", "%2C")
+function escapeWorkflowData(value) {
+	return String(value).replaceAll("%", "%25").replaceAll("\r", "%0D").replaceAll("\n", "%0A")
+}
+
+function escapeWorkflowProperty(value) {
+	return escapeWorkflowData(value).replaceAll(":", "%3A").replaceAll(",", "%2C")
+}
+
+export function formatAnnotationCommand(annotation) {
+	return `::error file=${escapeWorkflowProperty(annotation.file)},line=${annotation.line},title=Mutation test gap::${escapeWorkflowData(annotation.message)}`
 }
 
 export function formatAnnotations(blockingMutants, packageRoot, state = { total: 0, perFile: new Map() }) {
@@ -682,9 +685,7 @@ export function runManifest(repoRoot, manifest, reportRoot) {
 				packageEntry.runRoot ?? packageEntry.root,
 				annotationState,
 			)) {
-				console.log(
-					`::error file=${escapeWorkflowValue(annotation.file)},line=${annotation.line},title=Mutation test gap::${escapeWorkflowValue(annotation.message)}`,
-				)
+				console.log(formatAnnotationCommand(annotation))
 			}
 			evaluateReport(report, packageEntry)
 			rows.push({

scripts/stryker-diff.test.mjs

@@ -15,6 +15,7 @@ import {
 	evaluateReport,
 	executableChangedLines,
 	formatAnnotations,
+	formatAnnotationCommand,
 	formatBlockingMutants,
 	formatSummary,
 	mutantCounts,
@@ -432,6 +433,19 @@ describe("failure output", () => {
 		assert.equal(state.total, 20)
 	})
 
+	it("preserves punctuation in annotation messages while escaping properties", () => {
+		const command = formatAnnotationCommand({
+			file: "src/value:one,two.ts",
+			line: 4,
+			message: "Survived mutant (replacement: left, right). 100% reproducible.",
+		})
+
+		assert.equal(
+			command,
+			"::error file=src/value%3Aone%2Ctwo.ts,line=4,title=Mutation test gap::Survived mutant (replacement: left, right). 100%25 reproducible.",
+		)
+	})
+
 	it("uses the actual tests recorded by Stryker", () => {
 		assert.deepEqual(
 			testsFromMutationReport({ testFiles: { "src/value.test.ts": {}, "src/other.spec.ts": {} } }, [