Fix insert-block tool call

[jamwil]

The check for mutual exclusivity on the position argument did not handle empty strings.

[jamwil]

There are some other similar bugs. Still making edits.

[jamwil]

@dwash96 Does the coder provide the LLM with the docstring from the execute method on the tool? GPT-5.2 takes the schema super literally on InsertBlock and I think the tool manual (if that is the docstring or something else) needs to make it more clear to the LLM that three of the parameters are mutually exclusive.

[dwash96]

Hmm it doesn't no, but I feel like these things are JSON schemas at the end of the day so I'll merge this and add the oneOf sub schema property to InsertBlock and ShowNumberedContext:

https://json-schema.org/understanding-json-schema/reference/combining

[jamwil]

Hmm it doesn't no, but I feel like these things are JSON schemas at the end of the day so I'll merge this and add the oneOf sub schema property to InsertBlock and ShowNumberedContext:

https://json-schema.org/understanding-json-schema/reference/combining

I'm getting an error against openrouter/openai: \"Invalid schema for function 'InsertBlock': schema must have type 'object' and not have 'oneOf'/'anyOf'/'allOf'/'enum'/'not' at the top level.\"

I guess they only support a sub-set of JSON schema.