npm / arborist / #954

push

fix(add): save packages in the right place

Right now where we end up saving a dep is not very well coupled to where
the user requested it.  We are inferring more than we are checking for
intention.  This moves us to more heavily lean on the `saveType` if
given when adding a package into the dependencies.

We also were not consistently inferring the saveType.  There was code in
two places trying to do this, and either place used slightly different
logic. This moves the logic into one place.

Finally, we are more consistently making certain dependency types
mutually exclusive, for instance dependencies and devDependencies.

3006 of 3036 branches covered (99.01%)

Branch coverage included in aggregate %.

4044 of 4062 relevant lines covered (99.56%)

554.48 hits per line