Run configureInstaller after WaitForCompletion in WithXYZPackageInstallation methods

[afscrome]

Closes #760

Run configureInstaller after WaitForCompletion in the WithXYZPackageInstallation methods

PR Checklist

• Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
• Based off latest main branch of toolkit
• PR doesn't include merge commits (always rebase on top of our main, if needed)
• N/A New integration
  • Docs are written
  • Added description of major feature to project description for NuGet package (4000 total character limit, so don't push entire description over that)
• N/A Tests for the changes have been added (for bug fixes / features) (if applicable)
• Contains NO breaking changes
• Every new API (including internal ones) has full XML docs
• Code follows all style conventions

Other information

[aaronpowell]

Would it not make more sense to introduce an overload like public static IResourceBuilder<NodeAppResource> WithNpmPackageInstallation(this IResourceBuilder<NodeAppResource> resource, bool useCI = false, bool waitForCompletion = true, Action<IResourceBuilder<NpmInstallerResource>>? configureInstaller = null), and if waitForCompletion is false then we just skip the method call.

[afscrome]

Either way, I think the configureInstaller function should still be the last thing to be invoked.

As for introducing a new optional arg overload, that sort of feels like we're fighting the natural builder pattern, and hsould instead go for more of a fluent API:

frontEnd.WithNpmPackageInstallation(x => x
    .WithCi()
    .WithoutWait());