The class Tool and its subclasses return error with OpenAI StructuredOutput if "strict:true" unless you redefine the schema

[msalafia]

Checked other resources

• I added a very descriptive title to this question.
• I searched the LangChain documentation with the integrated search.
• I used the GitHub search to find a similar question and didn't find it.

Commit to Help

• I commit to help with one of those options 👆

Example Code

// Current Tool definition

export abstract class Tool extends StructuredTool<ZodObjectAny> {
  schema = z
    .object({ input: z.string().optional() })
    .transform((obj) => obj.input);

  constructor(fields?: ToolParams) {
    super(fields);
  }
...
}

Description

The class Tool features the schema property as the follwoing:

export abstract class Tool extends StructuredTool<ZodObjectAny> {
  schema = z
    .object({ input: z.string().optional() })
    .transform((obj) => obj.input);

  constructor(fields?: ToolParams) {
    super(fields);
  }
...
}

If i try to create a custom tool extending Tool or using something like SerpAPI tool (which indeed extends Tool) and using it with ChatOpenAI with response_format: "json_schema" and strict:true i will receive an error because of the optional.

Is this behaviour something you want?

Could this schema be changed with something like

schema = z
    .object({ input: z.string().nullable() })
    .transform((obj) => obj.input);

So the property is always required so OpenAI doesn't complain?

System Info

[email protected]