fix(gateway): 对齐计划审批错误码

保留计划审批拒绝当前 revision 后隐藏的产品语义，并将 Web 文案调整为需要修改。

Author: wynxing

internal/gateway/protocol/jsonrpc.go

@@ -1544,7 +1544,7 @@ func decodeApprovePlanParams(raw json.RawMessage) (ApprovePlanParams, *JSONRPCEr
 			return NewJSONRPCError(JSONRPCCodeInvalidParams, "missing required field: params.plan_id", GatewayCodeMissingRequiredField)
 		}
 		if p.Revision <= 0 {
-			return NewJSONRPCError(JSONRPCCodeInvalidParams, "invalid field: params.revision", GatewayCodeInvalidFrame)
+			return NewJSONRPCError(JSONRPCCodeInvalidParams, "invalid field: params.revision", GatewayCodeInvalidAction)
 		}
 		return nil
 	})

internal/gateway/protocol/jsonrpc_test.go

@@ -343,12 +343,25 @@ func TestNormalizeJSONRPCRequestApprovePlan(t *testing.T) {
 	})
 
 	tests := []struct {
-		name   string
-		params string
+		name            string
+		params          string
+		wantGatewayCode string
 	}{
-		{name: "missing session", params: `{"session_id":" ","plan_id":"plan-1","revision":1}`},
-		{name: "missing plan", params: `{"session_id":"session-1","plan_id":" ","revision":1}`},
-		{name: "invalid revision", params: `{"session_id":"session-1","plan_id":"plan-1","revision":0}`},
+		{
+			name:            "missing session",
+			params:          `{"session_id":" ","plan_id":"plan-1","revision":1}`,
+			wantGatewayCode: GatewayCodeMissingRequiredField,
+		},
+		{
+			name:            "missing plan",
+			params:          `{"session_id":"session-1","plan_id":" ","revision":1}`,
+			wantGatewayCode: GatewayCodeMissingRequiredField,
+		},
+		{
+			name:            "invalid revision",
+			params:          `{"session_id":"session-1","plan_id":"plan-1","revision":0}`,
+			wantGatewayCode: GatewayCodeInvalidAction,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
@@ -361,6 +374,9 @@ func TestNormalizeJSONRPCRequestApprovePlan(t *testing.T) {
 			if rpcErr == nil || rpcErr.Code != JSONRPCCodeInvalidParams {
 				t.Fatalf("expected invalid params error, got %#v", rpcErr)
 			}
+			if gatewayCode := GatewayCodeFromJSONRPCError(rpcErr); gatewayCode != tt.wantGatewayCode {
+				t.Fatalf("gateway_code = %q, want %q", gatewayCode, tt.wantGatewayCode)
+			}
 		})
 	}
 }

web/src/components/chat/ChatPanel.test.tsx

@@ -341,7 +341,7 @@ describe('ChatPanel', () => {
     expect(screen.getByText('计划审批')).toBeInTheDocument()
     expect(screen.getByRole('button', { name: /同意并以 bypass 执行/ })).toBeInTheDocument()
     expect(screen.getByRole('button', { name: /同意并以 default 执行/ })).toBeInTheDocument()
-    expect(screen.getByRole('button', { name: /拒绝\/其它想法/ })).toBeInTheDocument()
+    expect(screen.getByRole('button', { name: /需要修改/ })).toBeInTheDocument()
   })
 
   it('approves plan with bypass mode and starts build run', async () => {
@@ -398,13 +398,13 @@ describe('ChatPanel', () => {
     expect(useChatStore.getState().permissionMode).toBe('default')
   })
 
-  it('dismisses current plan revision and shows again for a new revision', async () => {
+  it('rejects current plan revision and shows approval again for a new revision', async () => {
     useChatStore.setState({
       messages: [draftPlanMessage(1)],
     } as any)
 
     render(<ChatPanel />)
-    fireEvent.click(screen.getByRole('button', { name: /拒绝\/其它想法/ }))
+    fireEvent.click(screen.getByRole('button', { name: /需要修改/ }))
 
     await waitFor(() => {
       expect(screen.queryByText('计划审批')).not.toBeInTheDocument()

web/src/components/chat/ChatPanel.tsx

@@ -684,7 +684,7 @@ export default function ChatPanel() {
                 className="permission-btn reject"
                 style={{ opacity: isApprovingPlan ? 0.6 : 1 }}
               >
-                <X size={13} /> 拒绝/其它想法
+                <X size={13} /> 需要修改
               </button>
               <button
                 onClick={() => handleApprovePlan('default')}