[app-server] update macro to make renaming methods less boilerplate-y (#6470)

We already do this for notification definitions and it's really nice.

Verified there are no changes to actual exported files by diff'ing
before and after this change.

codex-rs/app-server-protocol/src/protocol/common.rs

@@ -46,7 +46,7 @@ macro_rules! client_request_definitions {
     (
         $(
             $(#[$variant_meta:meta])*
-            $variant:ident {
+            $variant:ident $(=> $wire:literal)? {
                 params: $(#[$params_meta:meta])* $params:ty,
                 response: $response:ty,
             }
@@ -58,6 +58,7 @@ macro_rules! client_request_definitions {
         pub enum ClientRequest {
             $(
                 $(#[$variant_meta])*
+                $(#[serde(rename = $wire)] #[ts(rename = $wire)])?
                 $variant {
                     #[serde(rename = "id")]
                     request_id: RequestId,
@@ -108,94 +109,66 @@ client_request_definitions! {
 
     /// NEW APIs
     // Thread lifecycle
-    #[serde(rename = "thread/start")]
+    ThreadStart => "thread/start" {
-    #[ts(rename = "thread/start")]
-    ThreadStart {
         params: v2::ThreadStartParams,
         response: v2::ThreadStartResponse,
     },
-    #[serde(rename = "thread/resume")]
+    ThreadResume => "thread/resume" {
-    #[ts(rename = "thread/resume")]
-    ThreadResume {
         params: v2::ThreadResumeParams,
         response: v2::ThreadResumeResponse,
     },
-    #[serde(rename = "thread/archive")]
+    ThreadArchive => "thread/archive" {
-    #[ts(rename = "thread/archive")]
-    ThreadArchive {
         params: v2::ThreadArchiveParams,
         response: v2::ThreadArchiveResponse,
     },
-    #[serde(rename = "thread/list")]
+    ThreadList => "thread/list" {
-    #[ts(rename = "thread/list")]
-    ThreadList {
         params: v2::ThreadListParams,
         response: v2::ThreadListResponse,
     },
-    #[serde(rename = "thread/compact")]
+    ThreadCompact => "thread/compact" {
-    #[ts(rename = "thread/compact")]
-    ThreadCompact {
         params: v2::ThreadCompactParams,
         response: v2::ThreadCompactResponse,
     },
-    #[serde(rename = "turn/start")]
+    TurnStart => "turn/start" {
-    #[ts(rename = "turn/start")]
-    TurnStart {
         params: v2::TurnStartParams,
         response: v2::TurnStartResponse,
     },
-    #[serde(rename = "turn/interrupt")]
+    TurnInterrupt => "turn/interrupt" {
-    #[ts(rename = "turn/interrupt")]
-    TurnInterrupt {
         params: v2::TurnInterruptParams,
         response: v2::TurnInterruptResponse,
     },
 
-    #[serde(rename = "model/list")]
+    ModelList => "model/list" {
-    #[ts(rename = "model/list")]
-    ModelList {
         params: v2::ModelListParams,
         response: v2::ModelListResponse,
     },
 
-    #[serde(rename = "account/login/start")]
+    LoginAccount => "account/login/start" {
-    #[ts(rename = "account/login/start")]
-    LoginAccount {
         params: v2::LoginAccountParams,
         response: v2::LoginAccountResponse,
     },
 
-    #[serde(rename = "account/login/cancel")]
+    CancelLoginAccount => "account/login/cancel" {
-    #[ts(rename = "account/login/cancel")]
-    CancelLoginAccount {
         params: v2::CancelLoginAccountParams,
         response: v2::CancelLoginAccountResponse,
     },
 
-    #[serde(rename = "account/logout")]
+    LogoutAccount => "account/logout" {
-    #[ts(rename = "account/logout")]
-    LogoutAccount {
         params: #[ts(type = "undefined")] #[serde(skip_serializing_if = "Option::is_none")] Option<()>,
         response: v2::LogoutAccountResponse,
     },
 
-    #[serde(rename = "account/rateLimits/read")]
+    GetAccountRateLimits => "account/rateLimits/read" {
-    #[ts(rename = "account/rateLimits/read")]
-    GetAccountRateLimits {
         params: #[ts(type = "undefined")] #[serde(skip_serializing_if = "Option::is_none")] Option<()>,
         response: v2::GetAccountRateLimitsResponse,
     },
 
-    #[serde(rename = "feedback/upload")]
+    FeedbackUpload => "feedback/upload" {
-    #[ts(rename = "feedback/upload")]
-    FeedbackUpload {
         params: v2::FeedbackUploadParams,
         response: v2::FeedbackUploadResponse,
     },
 
-    #[serde(rename = "account/read")]
+    GetAccount => "account/read" {
-    #[ts(rename = "account/read")]
-    GetAccount {
         params: v2::GetAccountParams,
         response: v2::GetAccountResponse,
     },