Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
74 changes: 66 additions & 8 deletions dist/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -57733,16 +57733,48 @@ function generateMarkdownReport(report) {
lines.push(`| Branch Total Time | ${formatTime(report.totalBranchTime)} |`);
lines.push(`| Base Total Time | ${formatTime(report.totalBaseTime)} |`);
lines.push("");
// Overall status
// Overall status with passing and failing configurations
if (report.diffCount === 0) {
lines.push("## ✅ No API Changes");
lines.push("");
lines.push("No differences detected between the current branch and the comparison ref.");
lines.push("All configurations passed with no differences detected.");
lines.push("");
lines.push("**✅ Passing configurations (no changes detected):**");
for (const result of report.results) {
if (!result.error && !result.diffs.has("error") && !result.hasDifferences) {
lines.push(`- ${result.configName}`);
}
}
}
else {
lines.push("## 📋 API Changes Detected");
lines.push("");
lines.push(`${report.diffCount} configuration(s) have changes. Review below to ensure they are intentional.`);
// List passing configurations first
const passingConfigs = report.results.filter((r) => !r.error && !r.diffs.has("error") && !r.hasDifferences);
if (passingConfigs.length > 0) {
lines.push("**✅ Passing configurations (no changes detected):**");
for (const result of passingConfigs) {
lines.push(`- ${result.configName}`);
}
lines.push("");
}
// List configurations with changes (excluding errors)
const changedConfigs = report.results.filter((r) => r.hasDifferences && !r.error && !r.diffs.has("error"));
if (changedConfigs.length > 0) {
lines.push("**⚠️ Configurations with changes:**");
for (const result of changedConfigs) {
lines.push(`- ${result.configName} (see diff below)`);
}
lines.push("");
}
// List configurations with errors if any
const errorConfigs = report.results.filter((r) => r.error || r.diffs.has("error"));
if (errorConfigs.length > 0) {
lines.push("**❌ Configurations with errors:**");
for (const result of errorConfigs) {
lines.push(`- ${result.configName}`);
}
}
}
lines.push("");
// Per-configuration results
Expand Down Expand Up @@ -57851,18 +57883,44 @@ function generatePRSummary(report) {
lines.push("## ✅ MCP Conformance: No Changes");
lines.push("");
lines.push(`Tested ${report.results.length} configuration(s) - no API changes detected.`);
lines.push("");
lines.push("**✅ Passing configurations:**");
for (const result of report.results.filter((r) => !r.error && !r.diffs.has("error") && !r.hasDifferences)) {
lines.push(`- ${result.configName}`);
}
}
else {
lines.push("## 📋 MCP Conformance: API Changes Detected");
lines.push("");
lines.push(`**${report.diffCount}** of ${report.results.length} configuration(s) have changes.`);
lines.push("");
lines.push("### Changed Endpoints");
lines.push("");
for (const result of report.results.filter((r) => r.hasDifferences)) {
lines.push(`- **${result.configName}:** ${Array.from(result.diffs.keys()).join(", ")}`);
// List passing configurations
const passingConfigs = report.results.filter((r) => !r.error && !r.diffs.has("error") && !r.hasDifferences);
if (passingConfigs.length > 0) {
lines.push("**✅ Passing configurations (no changes):**");
for (const result of passingConfigs) {
lines.push(`- ${result.configName}`);
}
lines.push("");
}
// List configurations with changes (excluding errors)
const changedConfigs = report.results.filter((r) => r.hasDifferences && !r.error && !r.diffs.has("error"));
if (changedConfigs.length > 0) {
lines.push("**⚠️ Changed configurations:**");
for (const result of changedConfigs) {
lines.push(`- **${result.configName}:** ${Array.from(result.diffs.keys()).join(", ")}`);
}
lines.push("");
}
// List configurations with errors if any
const errorConfigs = report.results.filter((r) => r.error || r.diffs.has("error"));
if (errorConfigs.length > 0) {
lines.push("**❌ Configurations with errors:**");
for (const result of errorConfigs) {
lines.push(`- ${result.configName}`);
}
lines.push("");
}
lines.push("");
lines.push("See the full report in the job summary for details.");
}
return lines.join("\n");
Expand Down
Loading
Loading