feat: support required fields (make the others optional)

[evacchi]

• adds isZigOptional(schema, property), checking whether the field is nullable OR it is not declared as required in the parent schema.

requires dylibso/xtp-bindgen-test#28

Result, e.g.:

    CallToolResult:
      required:
        - content
      properties:
        content:
          type: array
          items:
            $ref: "#/components/schemas/Content"
        isError:
          type: boolean
          description: |-
            Whether the tool call ended in an error.

            If not set, this is assumed to be false (the call was successful).

pub const CallToolResult = struct {
    content: []Content,
    /// Whether the tool call ended in an error.
    ///
    /// If not set, this is assumed to be false (the call was successful).
    isError: ?bool = null,

...
}

[evacchi]

related dylibso/mcp.run-servlets#59

[bhelx]

This looks correct to me but maybe let's have @nilslice chime in

[evacchi]

IIRC @nilslice mentioned we'll need to release https://github.com/dylibso/xtp-bindgen-test/ for the CI to pass

[nilslice]

oh yea, feel free to release the dependent library and re-run these tests. Don't let me be a blocker on these, if the tests pass we can push it along

[evacchi]

it looks like the test is failing

xtp plugin test $PLUGIN_NAME/zig-out/bin/plugin.wasm --with test.wasm --mock-host mock.wasm

either the test or the mock... 🤔

[nilslice]

what's the status here? I keep getting reminders but don't know who's on this now (re: test suite etc)

[evacchi]

it's still me I guess

[evacchi]

ok making some progress here

1. we upgraded the JS PDK and the old version was failing with that odd error
2. now all test assertions are failing, but from the logs I can tell they should pass, so I'd say another JS-PDK-related bug

[nilslice]

@evacchi - ok thanks, can you be sure to link this issue to the GitHub ticket to track the JS PDK bug with whatever info you have on it?

[evacchi]

@zshipko fixed the issue, we can merge this and close