chore(core/review): stop emitting unused metadata message

This commit is contained in:
James Peter
2025-10-14 10:46:14 +11:00
parent 90b61f9d78
commit 6a50ea6eae
+2 -38
View File
@@ -3734,7 +3734,7 @@ async fn exit_review_mode(
let event = session.make_event(&task_sub_id, EventMsg::ExitedReviewMode(review_output.clone()));
session.send_event(event).await;
let active_request = session.take_active_review();
let _active_request = session.take_active_review();
let developer_text = match review_output.clone() {
Some(output) => {
@@ -3781,44 +3781,8 @@ async fn exit_review_mode(
content: vec![ContentItem::InputText { text: developer_text.clone() }],
};
let status = if review_output.is_some() {
"complete"
} else {
"no_output"
};
let mut metadata_payload = json!({
"type": "code_review_metadata",
"status": status,
"summary": developer_text,
});
if let Some(request) = active_request {
metadata_payload["prompt"] = json!(request.prompt);
metadata_payload["user_facing_hint"] = json!(request.user_facing_hint);
if let Some(meta) = request.metadata {
metadata_payload["review_context"] = serde_json::to_value(meta).unwrap_or(json!({}));
}
}
if let Some(ref output) = review_output {
metadata_payload["overall_correctness"] = json!(output.overall_correctness);
metadata_payload["overall_confidence_score"] = json!(output.overall_confidence_score);
metadata_payload["finding_count"] = json!(output.findings.len());
metadata_payload["findings"] = serde_json::to_value(&output.findings).unwrap_or(json!([]));
}
let metadata_text = serde_json::to_string_pretty(&metadata_payload)
.unwrap_or_else(|_| metadata_payload.to_string());
let metadata_message = ResponseItem::Message {
id: None,
role: "developer".to_string(),
content: vec![ContentItem::InputText { text: metadata_text }],
};
session
.record_conversation_items(&[developer_message, metadata_message])
.record_conversation_items(&[developer_message])
.await;
}