fix: update MCP server and CLI for Vec<&str> size_field_names return type

GiggleLiu · claude · GiggleLiu · commit a64f060d0b86 · 2026-02-26T07:46:22.000+08:00
Fix borrow-after-move in graph show command and use reference in
MCP tools JSON serialization.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/problemreductions-cli/src/commands/graph.rs b/problemreductions-cli/src/commands/graph.rs
@@ -143,7 +143,7 @@ pub fn show(problem: &str, out: &OutputConfig) -> Result<()> {
             "\n{}\n",
             crate::output::fmt_section(&format!("Size fields ({}):", size_fields.len()))
         ));
-        for f in size_fields {
+        for f in &size_fields {
             text.push_str(&format!("  {f}\n"));
         }
     }
diff --git a/problemreductions-cli/src/mcp/tools.rs b/problemreductions-cli/src/mcp/tools.rs
@@ -177,7 +177,7 @@ impl McpServer {
         let mut json = serde_json::json!({
             "name": spec.name,
             "variants": variants,
-            "size_fields": size_fields,
+            "size_fields": &size_fields,
             "reduces_to": outgoing.iter().map(|e| {
                 serde_json::json!({
                     "source": {"name": e.source_name, "variant": e.source_variant},

Original file line number	Diff line number	Diff line change
`@@ -143,7 +143,7 @@ pub fn show(problem: &str, out: &OutputConfig) -> Result<()> {`
`143`	`143`	`"\n{}\n",`
`144`	`144`	`crate::output::fmt_section(&format!("Size fields ({}):", size_fields.len()))`
`145`	`145`	`));`
`146`		`- for f in size_fields {`
	`146`	`+ for f in &size_fields {`
`147`	`147`	`text.push_str(&format!(" {f}\n"));`
`148`	`148`	`}`
`149`	`149`	`}`