Support for bulk operations - #149

I am not sure about this one, the formulation is ambiguous.

status The HTTP response status code for the requested operation.
When indicating an error, the "response" attribute MUST contain
the detail error response as per Section 3.12.

I think the detail error response refers to the content of response being an error, more than the error detail attribute that should be set, or it would have been error response detail attribute.

What do you think?

The 3.7.3 - Error handling section also mentions this description attribute.

If an error occurred, the status MUST also include the description attribute containing a human-readable explanation of the error.

It is followed by an example with the "detail" attribute with the description of the error. So I guessed this is what they're talking about.

Ok this is more convincing👌. Maybe you can also add this citation in comment?

Also I think the .get will break if the response is an Error object instead of the dict representing it:

BulkOperation.model_validate(
    {"method": "POST", "bulkId": "q", "status": 400,
     "response": Error(status=400, detail="bad request")},
    scim_ctx=Context.RESOURCE_CREATION_RESPONSE,
)

Ok this is more convincing👌. Maybe you can also add this citation in comment?

Actually, I changed my mind. Actually this sentence appears to be a relic of the SCIM 1 draft, where description was a real attribute of status.

If an error occurred the status MUST also
include the description attribute containing a human readable
explanation of the error.

"status": {
"code": "201"
}

The following is an example of a status in a failed operation.

"status": {
"code": "400",
"description": "Request is unparseable, syntactically incorrect, or violates schema."
}

Neither description nor status are valid in SCIM2. There is even an errata to RFC7644 which removes the status line which makes no sense here. This makes me think this in an error.

TL;DR: please allow empty detail parameters in error responses, but check that the response is actually an error object.

Actually, I think that BulkOperation BulkRequest and BulkResponse should be generic like it is done in patch for instance:

This would allow to give a better type to data than Any, Probably something like ResourceT | PatchOp[ResourceT] | None

Same thing here, making the classes generic would allow for ResourceT | Error | None instead of Any