`completed` is an attribute specific to mentoring, it shouldn't be referenced in light_children. Also, generally, this patch is breaking compatibility with the XBlock API.

Why not do it the same way you had persisted the number of attempts attribute on `<mrq>`, before it was moved to `<mentoring>`? You have the logic to do that already implemented no?