mirror of
https://github.com/storytold/deckcraft.git
synced 2026-10-31 05:56:55 -06:00
marshiyar
e83f964077
What was wrong: the insert_chart tool had no x, y, w or h in its schema, so the server refused them as unknown arguments, while insert_picture and insert_table take them. A chart could only be placed at the default box, [144, 81, 672, 378] on a 960x540 slide, and then had to be moved with a second command. What changed: insert_chart declares x, y, w and h and folds them into the engine's `rect` when all four are given, as insert_picture and insert_table do. Without all four the chart keeps its default place. How it was tested: insert_chart_takes_a_box failed before the change (the arguments were refused) and passes after it. It checks the box of the placed chart, that a partial box leaves the default place, and that an extreme box is answered without a panic. The rest of the deckcraft-mcp tests pass, and clippy is clean with -D warnings. Fixes #1151