refactor: use serde_urlencoded for query parameters

Replace manual url.query_pairs_mut().append_pair(...) pattern in
list_bugs and list_repos with typed query structs serialized via
serde_urlencoded. This makes the query parameter contract explicit
and type-checked, and removes the direct url crate dependency.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: sachiniyer

Cargo.lock

@@ -19,11 +19,11 @@ clap = { version = "4.5", features = ["derive", "cargo", "env"] }
 # HTTP client
 reqwest = { version = "0.13", features = ["json"] }
 tokio = { version = "1.0", features = ["rt-multi-thread", "macros"] }
-url = "2.5"
 
 # Serialization
 serde = { version = "1.0", features = ["derive"] }
 serde_json = "1.0"
+serde_urlencoded = "0.7"
 csv = "1.4"
 
 # Error handling

Cargo.toml

@@ -1,8 +1,7 @@
 use anyhow::{bail, Context, Result};
 use colored::*;
 use semver::{Version, VersionReq};
-use serde::Deserialize;
-use url::Url;
+use serde::{Deserialize, Serialize};
 
 use super::types::*;
 
@@ -140,22 +139,24 @@ impl ApiClient {
         limit: u32,
         offset: u32,
     ) -> Result<BugsResponse> {
-        let mut url = Url::parse(&format!("{}/public/v1/bugs", self.base_url))?;
-        {
-            let mut query = url.query_pairs_mut();
-            query.append_pair("repo_id", &repo_id.to_string());
-            query.append_pair("limit", &limit.to_string());
-            query.append_pair("offset", &offset.to_string());
-            if let Some(s) = status {
-                let value = serde_json::to_value(s)?;
-                if let Some(status_str) = value.as_str() {
-                    query.append_pair("status", status_str);
-                }
-            }
+        #[derive(Serialize)]
+        struct ListBugsQuery<'a> {
+            repo_id: &'a str,
+            limit: u32,
+            offset: u32,
+            #[serde(skip_serializing_if = "Option::is_none")]
+            status: Option<&'a BugReviewState>,
         }
-        let path_and_query = format!("{}?{}", url.path(), url.query().unwrap_or(""));
-        self.request(reqwest::Method::GET, &path_and_query, None)
-            .await
+
+        let query = ListBugsQuery {
+            repo_id: repo_id.as_str(),
+            limit,
+            offset,
+            status,
+        };
+        let query_string = serde_urlencoded::to_string(&query)?;
+        let path = format!("/public/v1/bugs?{}", query_string);
+        self.request(reqwest::Method::GET, &path, None).await
     }
 
     pub async fn get_bug(&self, bug_id: &BugId) -> Result<Bug> {
@@ -181,15 +182,16 @@ impl ApiClient {
     }
 
     pub async fn list_repos(&self, limit: u32, offset: u32) -> Result<ReposResponse> {
-        let mut url = Url::parse(&format!("{}/public/v1/repos", self.base_url))?;
-        {
-            let mut query = url.query_pairs_mut();
-            query.append_pair("limit", &limit.to_string());
-            query.append_pair("offset", &offset.to_string());
+        #[derive(Serialize)]
+        struct ListReposQuery {
+            limit: u32,
+            offset: u32,
         }
-        let path_and_query = format!("{}?{}", url.path(), url.query().unwrap_or(""));
-        self.request(reqwest::Method::GET, &path_and_query, None)
-            .await
+
+        let query = ListReposQuery { limit, offset };
+        let query_string = serde_urlencoded::to_string(&query)?;
+        let path = format!("/public/v1/repos?{}", query_string);
+        self.request(reqwest::Method::GET, &path, None).await
     }
 }