From df78bab6f487a4e300864b19bf2c575a377cd3fd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Jos=C3=A9=20Guti=C3=A9rrez=20de=20Quevedo=20P=C3=A9?=
 =?UTF-8?q?rez?= <juanjo@gutierrezdequevedo.com>
Date: Fri, 21 Nov 2025 18:49:10 +0100
Subject: [PATCH] fix: filter out (none) tags from patchsets

- Modified cvs_client.py to skip tags with value '(none)' using case-insensitive comparison
- Updated ui.js to also check for '(none)' tags when displaying patchsets
- Patchsets without tags no longer show any tag badge in the UI
---
 cvs_proxy/cvs_client.py | 5 ++++-
 ui/ui.js                | 2 +-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/cvs_proxy/cvs_client.py b/cvs_proxy/cvs_client.py
index 46dc874..aee6488 100644
--- a/cvs_proxy/cvs_client.py
+++ b/cvs_proxy/cvs_client.py
@@ -366,7 +366,10 @@ class CVSClient:
                     current_patchset['author'] = author_match.group(1)
                 
                 if tag_match:
-                    current_patchset['tag'] = tag_match.group(1)
+                    tag_value = tag_match.group(1)
+                    # Don't store tags that are "(none)" or "(None)" - treat them as no tag (case-insensitive)
+                    if tag_value.lower() != '(none)':
+                        current_patchset['tag'] = tag_value
                 
                 if log_match:
                     # Log section starts, capture until next PatchSet or end
diff --git a/ui/ui.js b/ui/ui.js
index 69770f8..5d0c5d5 100644
--- a/ui/ui.js
+++ b/ui/ui.js
@@ -509,7 +509,7 @@ class UIManager {
                 <div class="patchset-number">PatchSet #${ps.patchset}</div>
                 <div class="patchset-log-section">
                     <div class="patchset-log">${ps.log ? this.escapeHtml(ps.log) : ''}</div>
-                    ${ps.tag && ps.tag !== 'N/A' ? `<span class="patchset-tag">${ps.tag}</span>` : ''}
+                    ${ps.tag && ps.tag !== 'N/A' && ps.tag.toLowerCase() !== '(none)' ? `<span class="patchset-tag">${ps.tag}</span>` : ''}
                 </div>
                 <div class="patchset-author">${ps.author || 'Unknown'}</div>
                 <div class="patchset-date" title="${ps.date || 'N/A'}">${this.getRelativeDate(ps.date) || 'N/A'}</div>