I looked at the files of the mouse trap: it's OK.
Some remarks from my point of view:
As the first examples on my site show, it is not always necessary to wrap each part in an Asm3 container. This becomes so if we want to predefine the constraint elements.
You could have only had one Asm3 container here.
For your shipping container, it's the same.
To be more rigorous, all the elements of the shipping container except the doors can be in the same Asm3 container (class "box").
Then each opening will be in its own Asm3 container (right door, left door).
A general container will then contain the three sub-assemblies.
To be even more rigorous and especially if you want to predefine the geometric elements to be constrained, then you can put each part in its own container.
Regarding Shapestring I will try to make a short video in a few days.