From c3d2129472d764c81a663fa4bbca4d5258cea111 Mon Sep 17 00:00:00 2001
From: Mohamed Boudra <boudra.moha@gmail.com>
Date: Fri, 1 May 2026 21:22:00 +0700
Subject: [PATCH] refactor(server): extract AgentActionResponsePayloadSchema

Five set_agent_*_response and update_agent_response payloads were
pixel-identical. Share one named schema so the concept is explicit and
new agent-action responses don't copy-paste the same four fields.
Wire format and inferred types are unchanged.
---
 packages/server/src/shared/messages.ts | 42 ++++++++------------------
 1 file changed, 12 insertions(+), 30 deletions(-)

diff --git a/packages/server/src/shared/messages.ts b/packages/server/src/shared/messages.ts
index 70d744f86..2b0977183 100644
--- a/packages/server/src/shared/messages.ts
+++ b/packages/server/src/shared/messages.ts
@@ -1095,14 +1095,16 @@ export const SetAgentModeRequestMessageSchema = z.object({
   requestId: z.string(),
 });
 
+const AgentActionResponsePayloadSchema = z.object({
+  requestId: z.string(),
+  agentId: z.string(),
+  accepted: z.boolean(),
+  error: z.string().nullable(),
+});
+
 export const SetAgentModeResponseMessageSchema = z.object({
   type: z.literal("set_agent_mode_response"),
-  payload: z.object({
-    requestId: z.string(),
-    agentId: z.string(),
-    accepted: z.boolean(),
-    error: z.string().nullable(),
-  }),
+  payload: AgentActionResponsePayloadSchema,
 });
 
 export const SetAgentModelRequestMessageSchema = z.object({
@@ -1114,12 +1116,7 @@ export const SetAgentModelRequestMessageSchema = z.object({
 
 export const SetAgentModelResponseMessageSchema = z.object({
   type: z.literal("set_agent_model_response"),
-  payload: z.object({
-    requestId: z.string(),
-    agentId: z.string(),
-    accepted: z.boolean(),
-    error: z.string().nullable(),
-  }),
+  payload: AgentActionResponsePayloadSchema,
 });
 
 export const SetAgentThinkingRequestMessageSchema = z.object({
@@ -1131,12 +1128,7 @@ export const SetAgentThinkingRequestMessageSchema = z.object({
 
 export const SetAgentThinkingResponseMessageSchema = z.object({
   type: z.literal("set_agent_thinking_response"),
-  payload: z.object({
-    requestId: z.string(),
-    agentId: z.string(),
-    accepted: z.boolean(),
-    error: z.string().nullable(),
-  }),
+  payload: AgentActionResponsePayloadSchema,
 });
 
 export const SetAgentFeatureRequestMessageSchema = z.object({
@@ -1149,22 +1141,12 @@ export const SetAgentFeatureRequestMessageSchema = z.object({
 
 export const SetAgentFeatureResponseMessageSchema = z.object({
   type: z.literal("set_agent_feature_response"),
-  payload: z.object({
-    requestId: z.string(),
-    agentId: z.string(),
-    accepted: z.boolean(),
-    error: z.string().nullable(),
-  }),
+  payload: AgentActionResponsePayloadSchema,
 });
 
 export const UpdateAgentResponseMessageSchema = z.object({
   type: z.literal("update_agent_response"),
-  payload: z.object({
-    requestId: z.string(),
-    agentId: z.string(),
-    accepted: z.boolean(),
-    error: z.string().nullable(),
-  }),
+  payload: AgentActionResponsePayloadSchema,
 });
 
 export const SetVoiceModeResponseMessageSchema = z.object({