ok so (1) you should be able to have duplicate dependencies just fine in

ns/config/BUILD

and

ns/config/tests/BUILD

. in fact, that's somewhat preferred over putting it in just the one parent target. (2) the cycle should only happen if there's a legitimate dependency cycle between dependencies declared in BUILD files, which you should be able to see. there is a shadow of a chance that this is a nondeterministic pantsd error, which you can check by running

./pants kill-pantsd

and then re-running the command