From 95de4e83d1e37c0f7748092f1d9021030d73cef2 Mon Sep 17 00:00:00 2001 From: Artur Shiriev Date: Fri, 17 Jul 2026 23:19:56 +0300 Subject: [PATCH] chore(benchmarks): note the format_table/format_markdown column sync The two renderers mirror the same columns, precision, and footer facts in different skins (fixed-width vs Markdown). A one-line pointer in each pins the drift hazard without introducing a shared abstraction that two stable renderers don't yet warrant. Co-Authored-By: Claude Opus 4.8 (1M context) --- benchmarks/report.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/benchmarks/report.py b/benchmarks/report.py index ee683e0..fc32293 100644 --- a/benchmarks/report.py +++ b/benchmarks/report.py @@ -106,6 +106,7 @@ def to_baseline(results: list[RunResult]) -> dict[str, typing.Any]: def format_table(results: list[RunResult]) -> str: """Render the runs as a human table with a gated-vs-informational footer.""" + # Columns/precision/footer facts are mirrored in format_markdown -- keep in sync. header = ( f"{'scenario':<22} {'msg/s':>9} {'delete/msg':>11} {'WALrec/msg':>11} " f"{'WALB/msg':>9} {'fpi':>5} {'upd':>7} {'del':>7} {'dead_tup':>9}" @@ -135,6 +136,7 @@ def format_markdown(results: list[RunResult], failures: list[str]) -> str: function only formats -- the exit code is the caller's, so the comment can be posted on both pass and fail. """ + # Columns/precision/footer facts are mirrored in format_table -- keep in sync. verdict = "✅ gate passed" if not failures else "❌ gate FAILED" lines = ["## Benchmark gate", "", verdict, ""] if failures: