The result schema of the "oneOf" validation keyword does not match its documentation

[bduminiuc]

I found a commit changing the result of "oneOf" validation.

commit 338d8defdb79eeff5f9801bd443ef9a4ae5bb8e7
Author: Steve Hanson <[email protected]>
Date:   Thu Jun 17 08:58:37 2021 +0100

    initial

I try to parse invalid json with the following schema:

{
    "oneOf": [
        { "type": "integer" },
        { "type": "number" },
        { "type": "string", "pattern": "^z" }
    ]
}

Before the commit, the validator output is:

{"oneOf":{"errors":[{},{},{"type":{"expected":["string"],"actual":"integer","errorCode":20,"instanceRef":"#","schemaRef":"#/oneOf/2"}}],"errorCode":22,"instanceRef":"#","schemaRef":"#"}}

but after:

{"oneOf":{"matches":[0,1],"errorCode":22,"instanceRef":"#","schemaRef":"#"}}

However, I see the documentation says:

allOf, anyOf, and oneOf
errors: required array of at least one object.

So, either it is a mistake in the documentation or the code. But what is the correct one?

Snippet to reproduce:

using namespace rapidjson;

Document sd;
sd.Parse(schema_str);
auto const schema = SchemaDocument{sd};

Document d;
d.Parse("1");  // invalid json

SchemaValidator validator{schema};
if (!d.Accept(validator)) {
    MemoryPoolAllocator<> allocator;
    auto ret = Value(validator.GetError(), allocator);

    StringBuffer sb;
    Writer<StringBuffer> w{sb};
    ret.Accept(w);
    std::cout << sb.GetString();
} else {
    std::cout << "no error";
}