From 15b03e83d4eb153e51b5d554a9071873582bcbcb Mon Sep 17 00:00:00 2001 From: rob Date: Tue, 4 Nov 2025 22:37:08 -0400 Subject: [PATCH] fix: suppress marker warnings for uninitialized summary files - Only show marker missing warnings if file has OTHER markers - If file has no markers at all, it's uninitialized and warnings are noise - Prevents confusing warnings on first commit for new discussions - All 41 tests passing --- automation/summary.py | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/automation/summary.py b/automation/summary.py index cc170c8..1d9b58e 100644 --- a/automation/summary.py +++ b/automation/summary.py @@ -42,12 +42,16 @@ def update_marker_block( updated = re.sub(pattern, replacer, content, flags=re.DOTALL) - # If no replacement happened, the markers might not exist + # If no replacement happened, the markers might not exist. + # Only warn if the file has OTHER markers (indicating it's initialized but missing this one). + # If there are NO markers at all, the file is uninitialized and warnings are noise. if updated == content: - sys.stderr.write( - f"[summary] note: markers for {marker_name} not found " - "(summary file likely not initialized yet)\n" - ) + has_any_markers = "