[ekr1]

I often had this issue:

    Tool Call: Local • EditFile

    Error in EditFile: Missing required parameters for EditFile: edits
    NoneType: None

    Tool Call: Local • ReadFile

    range_1:  •  • 

    Error in ReadFile: Missing required parameters for ReadFile: read
    NoneType: None

This would go on and on until the LLM eventually gave up, or managed to provide the parameters correctly. Since working with this change, the issue seems to have gone away for me as far as I can tell.

Detailled (generated) explanation:

Local models sometimes emit tool-call arguments as a bare JSON array (e.g. [{...}, {...}]) instead of wrapping it
under the tool's single required array parameter (EditFile's edits, ReadFile's read). This was silently
collapsed to {}, producing Error in EditFile: Missing required parameters for EditFile: edits.

Fix: parse_tool_arguments now preserves a bare top-level array instead of dropping it, and
BaseTool/ToolValidations auto-wrap it under the tool's required array param (also handling the case where a single
array item is sent directly as a dict). Validation ordering changed so shape-correction runs before the missing-param
check.

Fixed a regression this introduced in the "glued JSON objects" merge path ({"a":1}{}{"b":2} style fragments) by
removing a premature list short-circuit that bypassed merge_glued_json_objects.

[ekr1]

Update: I did work a bit more with these changes active and still have some spurious issues with local command calls, more along the lines of these:

Tool Call: Local • ReadFile

range_1:  •  • 

Errors encountered for 1 operation(s)

I don't know if this is related to this PR. Generally cecli became a bit fickle for me in the recent days; it may be the skills/instructions I use (i.e. caveman) confusing it. Unless you think that the code in this PR is good and useful, best park it for now and I'll come back to it later with more debugging on my side.

Also note that I am always working in dumb terminal mode, not TUI; may be different...