From e251360f05af7c35b989db111b93a58cba7c8138 Mon Sep 17 00:00:00 2001
From: loki5512344 <mauhhhh7@gmail.com>
Date: Mon, 21 Sep 2026 20:52:50 +0200
Subject: [PATCH] fix(counter): migrate old marker and cleanup duplicate

---
 src/feature/counter/task.rs | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/src/feature/counter/task.rs b/src/feature/counter/task.rs
index 10e7094..d9abc19 100644
--- a/src/feature/counter/task.rs
+++ b/src/feature/counter/task.rs
@@ -26,6 +26,28 @@ async fn ensure_channel(ctx: &Context, guild_id: GuildId) -> anyhow::Result<Chan
             let _ = id.edit(&ctx.http, edit).await;
             info!(channel=%id, position=%ch.position, parent=?ch.parent_id, "member counter reconciled to top (outside category, pos 0)");
         }
+        // Cleanup old duplicates with different marker (e.g. "👥｜Members:" with separator)
+        for (oid, oc) in channels.iter().filter(|(oid, _)| *oid != id) {
+            if oc.name.contains("Members:") {
+                // Old counter with different marker or duplicate — delete
+                let _ = oid.delete(&ctx.http).await;
+                info!(channel=%oid, name=%oc.name, "old member counter deleted (migration)");
+            }
+        }
+        return Ok(*id);
+    }
+    // Migration: found old marker but not new — rename/move it instead of creating new
+    if let Some((id, _)) = channels.iter().find(|(_, c)| c.name.contains("Members:")) {
+        let _ = id
+            .edit(
+                &ctx.http,
+                EditChannel::new()
+                    .name(format!("{MARKER} 0"))
+                    .position(0)
+                    .category(Option::<ChannelId>::None),
+            )
+            .await;
+        info!(channel=%id, "old counter migrated to new format");
         return Ok(*id);
     }
     // Create outside category at position 0, deny CONNECT so users can't join, but VIEW remains