PKG -- [fcl] Respect service method for discovery (#1375)

* PKG -- [fcl] Respect service method for discovery

Addresses: #1374

* PKG -- [fcl] Add changeset

Co-authored-by: Chase Fleming <1666730+chasefleming@users.noreply.github.com>

.changeset/brown-bears-remain.md

@@ -0,0 +1,5 @@
+---
+"@onflow/fcl": patch
+---
+
+Fix issue where authenticating a Discovery API service would not respect service method.

packages/fcl/src/discovery/utils.js

@@ -0,0 +1,38 @@
+import {config} from "@onflow/config"
+import {invariant} from "@onflow/util-invariant"
+import {serviceRegistry} from "../current-user/exec-service/plugins"
+
+export const makeDiscoveryServices = async () => {
+  const extensionServices = window?.fcl_extensions || []
+  return [...extensionServices, ...serviceRegistry.getServices()]
+}
+
+export async function getDiscoveryService(service) {
+  const discoveryAuthnInclude = await config.get("discovery.authn.include", [])
+  const discoveryWalletMethod = await config.first([
+    "discovery.wallet.method",
+    "discovery.wallet.method.default",
+  ])
+  const method = service?.method
+    ? service.method
+    : discoveryWalletMethod
+  const endpoint =
+    service?.endpoint ??
+    (await config.first(["discovery.wallet", "challenge.handshake"]))
+
+  invariant(
+    endpoint,
+    `
+    If no service is passed to "authenticate," then "discovery.wallet" must be defined in fcl config.
+    See: "https://docs.onflow.org/fcl/reference/api/#setting-configuration-values"
+    `
+  )
+
+  return {
+    ...service,
+    type: "authn",
+    endpoint,
+    method,
+    discoveryAuthnInclude,
+  }
+}