fix: preserve client_metadata.scope if already set

[owendevereaux]

Summary

Don't override client_metadata.scope if it was explicitly set by the client.

Problem

As reported in #2317, the scope selection strategy in async_auth_flow unconditionally overwrites any scope that was explicitly set by the client. This prevents clients from:

• Requesting fewer permissions than available
• Working with servers that reject requests for unauthorized scopes (e.g., SalesForce)

Solution

Made the scope assignment conditional:

# Step 3: Apply scope selection strategy (only if not already set)
if self.context.client_metadata.scope is None:
    self.context.client_metadata.scope = get_client_metadata_scopes(...)

This preserves the existing behavior when no scope is set, while respecting explicit scope configuration.

Testing

• Existing tests continue to pass (scope is None by default)
• Clients that explicitly set client_metadata.scope will now have that value preserved

Fixes #2317

[maxisbey]

Thanks for the PR! The linked issue (#2317) hasn't been triaged yet — per our CONTRIBUTING.md, please wait for maintainer feedback before starting work. Closing for now; feel free to reopen once the issue is labeled ready for work.

_{AI Disclaimer}