aj/OpenNest
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fix(mcp): add null guards and safe parsing to pack_plate

Browse Source 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
AJ Isaacs
2026-03-08 15:52:10 -04:00
parent 0c56a9049a
commit e9ea290907
1 changed files with 14 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
OpenNest.Mcp/Tools/NestingTools.cs
View File

@@ -140,8 +140,21 @@ namespace OpenNest.Mcp.Tools
if (plate == null) if (plate == null)
return $"Error: plate {plateIndex} not found"; return $"Error: plate {plateIndex} not found";
if (string.IsNullOrWhiteSpace(drawingNames))
return "Error: drawingNames is required";
if (string.IsNullOrWhiteSpace(quantities))
return "Error: quantities is required";
var names = drawingNames.Split(',').Select(n => n.Trim()).ToArray(); var names = drawingNames.Split(',').Select(n => n.Trim()).ToArray();
var qtys = quantities.Split(',').Select(q => int.Parse(q.Trim())).ToArray(); var qtyStrings = quantities.Split(',').Select(q => q.Trim()).ToArray();
var qtys = new int[qtyStrings.Length];
for (var i = 0; i < qtyStrings.Length; i++)
{
if (!int.TryParse(qtyStrings[i], out qtys[i]))
return $"Error: '{qtyStrings[i]}' is not a valid quantity";
}
if (names.Length != qtys.Length) if (names.Length != qtys.Length)
return $"Error: drawing names count ({names.Length}) does not match quantities count ({qtys.Length})"; return $"Error: drawing names count ({names.Length}) does not match quantities count ({qtys.Length})";