Skip to content

Commit 8e13aa7

Browse files
author
Fr4gm3nt3d_sh
committed
Update .github/workflows/llmcodereview.yml
1 parent 5789f56 commit 8e13aa7

File tree

1 file changed

+15
-15
lines changed

1 file changed

+15
-15
lines changed

‎.github/workflows/llmcodereview.yml‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -78,24 +78,24 @@ jobs:
7878
};
7979
8080
const markdownToHtml = (md) => {
81-
// Extract code blocks and inline code first
8281
const codeBlocks = [];
8382
const inlineCode = [];
8483
84+
// Extract code blocks FIRST - don't escape them yet
8585
let html = md.replace(/```[\s\S]*?```/g, (match) => {
8686
const code = match.substring(3, match.length - 3).trim();
87-
const idx = codeBlocks.length;
8887
codeBlocks.push(code);
89-
return '\n__CODEBLOCK' + idx + '__\n';
88+
return 'XYZCODEBLOCX' + (codeBlocks.length - 1) + 'XYZCODEBLOCX';
9089
});
9190
91+
// Extract inline code
9292
html = html.replace(/`[^`]+`/g, (match) => {
9393
const code = match.substring(1, match.length - 1);
94-
const idx = inlineCode.length;
9594
inlineCode.push(code);
96-
return '__INLINE' + idx + '__';
95+
return 'XYZINLINEX' + (inlineCode.length - 1) + 'XYZINLINEX';
9796
});
9897
98+
// NOW escape the remaining HTML
9999
html = escapeHtml(html);
100100
101101
// Headers
@@ -105,7 +105,7 @@ jobs:
105105
html = html.replace(/^## (.*?)$/gm, '<h2>$1</h2>');
106106
html = html.replace(/^# (.*?)$/gm, '<h1>$1</h1>');
107107
108-
// Bold only (no italics)
108+
// Bold only
109109
html = html.replace(/\*\*(.*?)\*\*/g, '<strong>$1</strong>');
110110
html = html.replace(/__(.*?)__/g, '<strong>$1</strong>');
111111
@@ -126,19 +126,19 @@ jobs:
126126
html = html.replace(/(<\/h[1-6]>)<\/p>/g, '$1');
127127
html = html.replace(/<p>(<ul>)/g, '$1');
128128
html = html.replace(/(<\/ul>)<\/p>/g, '$1');
129-
html = html.replace(/<p>\n__CODEBLOCK/g, '\n__CODEBLOCK');
130-
html = html.replace(/__CODEBLOCK\d+__\n<\/p>/g, (match) => match.replace('<\/p>', ''));
129+
html = html.replace(/<p>XYZCODEBLOCX/g, 'XYZCODEBLOCX');
130+
html = html.replace(/XYZCODEBLOCX<\/p>/g, 'XYZCODEBLOCX');
131131
html = html.replace(/<p><\/p>/g, '');
132132
133133
// Restore inline code
134-
inlineCode.forEach((code, i) => {
135-
html = html.replace(new RegExp('__INLINE' + i + '__', 'g'), '<code>' + escapeHtml(code) + '</code>');
136-
});
134+
for (let i = 0; i < inlineCode.length; i++) {
135+
html = html.split('XYZINLINEX' + i + 'XYZINLINEX').join('<code>' + escapeHtml(inlineCode[i]) + '</code>');
136+
}
137137
138138
// Restore code blocks
139-
codeBlocks.forEach((code, i) => {
140-
html = html.replace(new RegExp('\n?__CODEBLOCK' + i + '__\n?', 'g'), '<pre><code>' + escapeHtml(code) + '</code></pre>');
141-
});
139+
for (let i = 0; i < codeBlocks.length; i++) {
140+
html = html.split('XYZCODEBLOCX' + i + 'XYZCODEBLOCX').join('<pre><code>' + escapeHtml(codeBlocks[i]) + '</code></pre>');
141+
}
142142
143143
return html;
144144
};
@@ -147,7 +147,7 @@ jobs:
147147
const reportName = "report-" + timestamp + ".html";
148148
const date = new Date();
149149
150-
const html = '<!DOCTYPE html><html lang="en" data-theme="dark"><head><meta charset="UTF-8"><meta name="viewport" content="width=device-width, initial-scale=1.0"><title>Gemini Code Review</title><link rel="stylesheet" href="../style.css"><style>.report-container { max-width: 800px; margin: 1.5rem auto; background: var(--bg-light); border-radius: 6px; border: 1px solid var(--border); overflow: hidden; }.report-header { background: var(--bg); color: var(--text); padding: 2rem 1.5rem; text-align: center; border-bottom: 1px solid var(--border); }.report-header h1 { font-size: 2rem; margin-bottom: 0.3rem; font-weight: 700; color: var(--primary); }.report-header p { font-size: 1rem; color: var(--text-muted); }.report-timestamp { font-size: 0.85rem; margin-top: 0.5rem; color: var(--text-muted); }.report-content { padding: 1.5rem; font-size: 1rem; line-height: 1.6; }.report-content h1 { font-size: 1.8rem; color: var(--primary); margin-top: 1.5rem; margin-bottom: 0.8rem; border-left: 3px solid var(--primary); padding-left: 0.8rem; }.report-content h2 { font-size: 1.6rem; color: var(--primary); margin-top: 1.5rem; margin-bottom: 0.8rem; border-left: 3px solid var(--primary); padding-left: 0.8rem; }.report-content h3 { font-size: 1.3rem; color: var(--secondary); margin-top: 1rem; margin-bottom: 0.5rem; }.report-content h4 { font-size: 1.1rem; color: var(--secondary); margin-top: 0.8rem; margin-bottom: 0.4rem; }.report-content p { margin-bottom: 0.8rem; color: var(--text); }.report-content ul { margin-left: 1.5rem; margin-bottom: 0.8rem; }.report-content li { margin-bottom: 0.3rem; color: var(--text); }.report-content code { background: var(--bg); padding: 0.15rem 0.4rem; border-radius: 3px; font-family: monospace; color: var(--secondary); font-size: 0.95em; }.report-content pre { background: var(--bg-dark); color: var(--text); padding: 1rem; border-radius: 4px; overflow-x: auto; margin: 0.8rem 0; font-family: monospace; font-size: 0.9rem; border: 1px solid var(--border); }.report-footer { background: var(--bg); padding: 1rem; text-align: center; color: var(--text-muted); border-top: 1px solid var(--border); font-size: 0.9rem; }.report-footer a { color: var(--primary); text-decoration: none; }.report-footer a:hover { color: var(--secondary); }</style></head><body><div class="report-container"><div class="report-header"><h1>🔍 Code Review</h1><p>Gemini Analysis</p><p class="report-timestamp">' + date.toUTCString() + '</p></div><div class="report-content">' + markdownToHtml(report) + '</div><div class="report-footer"><p><a href="index.html">← Back to Reports</a></p></div></div></body></html>';
150+
const html = '<!DOCTYPE html><html lang="en" data-theme="dark"><head><meta charset="UTF-8"><meta name="viewport" content="width=device-width, initial-scale=1.0"><title>Gemini Code Review</title><link rel="stylesheet" href="../style.css"><style>.report-container { max-width: 800px; margin: 1.5rem auto; background: var(--bg-light); border-radius: 6px; border: 1px solid var(--border); overflow: hidden; }.report-header { background: var(--bg); color: var(--text); padding: 2rem 1.5rem; text-align: center; border-bottom: 1px solid var(--border); }.report-header h1 { font-size: 2rem; margin-bottom: 0.3rem; font-weight: 700; color: var(--primary); }.report-header p { font-size: 1rem; color: var(--text-muted); }.report-timestamp { font-size: 0.85rem; margin-top: 0.5rem; color: var(--text-muted); }.report-content { padding: 1.5rem; font-size: 0.95rem; line-height: 1.6; }.report-content h1 { font-size: 1.8rem; color: var(--primary); margin-top: 1.5rem; margin-bottom: 0.8rem; border-left: 3px solid var(--primary); padding-left: 0.8rem; }.report-content h2 { font-size: 1.6rem; color: var(--primary); margin-top: 1.5rem; margin-bottom: 0.8rem; border-left: 3px solid var(--primary); padding-left: 0.8rem; }.report-content h3 { font-size: 1.3rem; color: var(--secondary); margin-top: 1rem; margin-bottom: 0.5rem; }.report-content h4 { font-size: 1.1rem; color: var(--secondary); margin-top: 0.8rem; margin-bottom: 0.4rem; }.report-content p { margin-bottom: 0.8rem; color: var(--text); }.report-content ul { margin-left: 1.5rem; margin-bottom: 0.8rem; }.report-content li { margin-bottom: 0.3rem; color: var(--text); }.report-content code { background: var(--bg); padding: 0.15rem 0.4rem; border-radius: 3px; font-family: monospace; color: var(--secondary); font-size: 0.95em; }.report-content pre { background: var(--bg-dark); color: var(--text); padding: 1rem; border-radius: 4px; overflow-x: auto; margin: 0.8rem 0; font-family: monospace; font-size: 0.9rem; border: 1px solid var(--border); }.report-footer { background: var(--bg); padding: 1rem; text-align: center; color: var(--text-muted); border-top: 1px solid var(--border); font-size: 0.9rem; }.report-footer a { color: var(--primary); text-decoration: none; }.report-footer a:hover { color: var(--secondary); }</style></head><body><div class="report-container"><div class="report-header"><h1>🔍 Code Review</h1><p>Gemini Analysis</p><p class="report-timestamp">' + date.toUTCString() + '</p></div><div class="report-content">' + markdownToHtml(report) + '</div><div class="report-footer"><p><a href="index.html">← Back to Reports</a></p></div></div></body></html>';
151151
152152
if (!fs.existsSync("reports")) {
153153
fs.mkdirSync("reports", { recursive: true });

0 commit comments

Comments
 (0)