seb/deployHook
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(api): escape Markdown and adjust compare link formatting

Browse Source 
Ensure parentheses are escaped in section headers, switch to en dash in
heading, and avoid Markdown links by outputting escaped text and URL.
Also improve line breaks for consistent layout.
This commit is contained in:
sebseb7
2025-08-04 10:34:29 +02:00
parent e4a01f387b
commit ab4964815c
1 changed files with 11 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
x.js
View File

@@ -246,7 +246,7 @@ function formatCommitMessage(payload) {
let filesList = '';
if (allAdded.size > 0) {
filesList += `\n➕ *Added (${allAdded.size}):*\n`;
filesList += `\n➕ *Added \\(${allAdded.size}\\):*\n`;
Array.from(allAdded).slice(0, 10).forEach(file => {
filesList += ` • \`${escapeInlineCode(file)}\`\n`;
});
@@ -256,7 +256,7 @@ function formatCommitMessage(payload) {
}
if (allModified.size > 0) {
filesList += `\n📝 *Modified (${allModified.size}):*\n`;
filesList += `\n📝 *Modified \\(${allModified.size}\\):*\n`;
Array.from(allModified).slice(0, 10).forEach(file => {
filesList += ` • \`${escapeInlineCode(file)}\`\n`;
});
@@ -266,7 +266,7 @@ function formatCommitMessage(payload) {
}
if (allRemoved.size > 0) {
filesList += `\n❌ *Removed (${allRemoved.size}):*\n`;
filesList += `\n❌ *Removed \\(${allRemoved.size}\\):*\n`;
Array.from(allRemoved).slice(0, 10).forEach(file => {
filesList += ` • \`${escapeInlineCode(file)}\`\n`;
});
@@ -284,13 +284,15 @@ function formatCommitMessage(payload) {
// Escape heading line content and link text/url appropriately
const repoEsc = escapeMdV2(repo);
const heading = `🚀 *${repoEsc}* - ${totalCommits} ${commitWord} pushed`;
// Use en dash and keep the rest escaped via escapeMdV2 for safety
const heading = `🚀 *${repoEsc}* – ${totalCommits} ${commitWord} pushed`;
let compare = '';
if (payload.compare_url) {
// Safer to avoid Markdown link syntax; print escaped text and URL
const linkText = escapeMdV2('Compare Changes');
const linkUrl = escapeMdV2(payload.compare_url);
compare = `[${linkText}](${linkUrl})`;
compare = `${linkText}: ${linkUrl}`;
}
// Try to append a German executive summary of the most recent commit.
@@ -308,10 +310,11 @@ function formatCommitMessage(payload) {
// already logged inside summarizer; keep silent here
}
// Ensure each section starts on its own line; compare may be plain URL text.
return `${heading}
${commitsText}
${filesList}
${compare}${summaryBlock}`;
${compare ? `\n${compare}` : ''}${summaryBlock}`;
})();
}