Revert "fix: restore internal_archive.direct after cycle-break recompile" (#4617)

Reverts bazel-contrib/rules_go#4606

I think this contributes another case to #4607 -- per my investigation
not the *only* path to duplicate import paths, but it does make it
possible.

I'm still investigating, but I think we should revert this patch and "do
it properly", ensuring coherency of direct imports.

I will send a follow-up, more principled fix that also fixes #4607.