I don’t think dumping non-exported names from a package into the current environment is ever a good idea:1 there’s a reason why these functions are not exported, and at the point where you find yourself accessing them in bulk you should stop and rethink your approach.
Given your specific use-case (edit a function from a package that needs to use non-exported names internally), a better solution would be to assign the package namespace as the function’s environment:
the_edited_function = function (…) { … }
environment(the_edited_function) = asNamespace('the_package')
… of course it goes without saying that this is also a horrible hack and should rarely be necessary.
1 It can be done with a single expression:
list2env(as.list(asNamespace('the_package'), all.names = TRUE), envir = environment())
… but, really, don’t do this.