It's a perfectly fine way of doing things as long as it's consistent and the spec is clear.
HTTP is a transport layer. You don't have to use its codes for your application layer. It's often done that way but it's not the only way.
In the example above the transport layer is saying "OK I've delivered your output" which is technically correct. It's not concerned with logical errors inside what it was transporting, just with the delivery itself.