impl: prefer agent name over agent id in URI handler

Agent name is easier to resolve in jetbrains gateway module

Author: fioan89

src/main/kotlin/com/coder/gateway/util/LinkHandler.kt

@@ -327,17 +327,16 @@ internal fun getMatchingAgent(
     }
 
     // If the agent is missing and the workspace has only one, use that.
-    // Prefer the ID over the name if both are set.
-    val agent =
-        if (!parameters.agentID().isNullOrBlank()) {
-            agents.firstOrNull { it.id.toString() == parameters.agentID() }
-        } else if (!parameters.agentName().isNullOrBlank()) {
-            agents.firstOrNull { it.name == parameters.agentName() }
-        } else if (agents.size == 1) {
-            agents.first()
-        } else {
-            null
-        }
+    // Prefer the name over the id if both are set.
+    val agent = if (!parameters.agentName().isNullOrBlank()) {
+        agents.firstOrNull { it.name == parameters.agentName() }
+    } else if (!parameters.agentID().isNullOrBlank()) {
+        agents.firstOrNull { it.id.toString() == parameters.agentID() }
+    } else if (agents.size == 1) {
+        agents.first()
+    } else {
+        null
+    }
 
     if (agent == null) {
         if (!parameters.agentID().isNullOrBlank()) {

src/main/kotlin/com/coder/gateway/util/LinkMap.kt

@@ -29,6 +29,7 @@ fun Map<String, String>.owner() = this[OWNER]
 
 fun Map<String, String?>.agentName() = this[AGENT_NAME]
 
+@Deprecated("Use the agent name instead")
 fun Map<String, String?>.agentID() = this[AGENT_ID]
 
 fun Map<String, String>.folder() = this[FOLDER]