From 5f05fd97e619f7b868a926b2c2bf8d15ac2ed089 Mon Sep 17 00:00:00 2001
From: CJ van den Berg <cj@vdbonline.com>
Date: Tue, 8 Apr 2025 18:00:49 +0200
Subject: [PATCH] refactor: remove duplicate implementation of command.get_id

---
 src/command.zig | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/src/command.zig b/src/command.zig
index 26a4dcc..e2b5ef4 100644
--- a/src/command.zig
+++ b/src/command.zig
@@ -150,13 +150,8 @@ pub fn execute(id: ID, ctx: Context) tp.result {
 }
 
 pub fn get_id(name: []const u8) ?ID {
-    for (commands.items) |cmd| {
-        if (cmd) |p|
-            if (std.mem.eql(u8, p.name, name))
-                return p.id;
-    }
-    tp.trace(tp.channel.debug, .{ "command", "get_id", "failed", name });
-    return null;
+    var id: ?ID = null;
+    return get_id_cache(name, &id);
 }
 
 pub fn get_name(id: ID) ?[]const u8 {
@@ -178,6 +173,7 @@ pub fn get_id_cache(name: []const u8, id: *?ID) ?ID {
                 return p.id;
             };
     }
+    tp.trace(tp.channel.debug, .{ "command", "get_id_cache", "failed", name });
     return null;
 }