Obviate need for provider IDs in two places using (resolve)

Somewhat more maintainable

src/cljs/wish/providers.cljs

@@ -16,8 +16,6 @@
             [wish.util :refer [>evt]]
             [wish.views.widgets :as widgets]))
 
-; NOTE: If you add a provider, make sure to also add
-; its IDs to wish.providers.util/provider-id?
 (def ^:private providers
   {:gdrive
    {:id :gdrive

src/cljs/wish/providers/util.cljs

@@ -2,10 +2,19 @@
       :doc "Providers util"}
   wish.providers.util)
 
+; lazy resolve to avoid circular dependencies;
+; we need to know if a keyword is a provider-id for, eg checking if a
+; keyword value is a sheet ID in subs-util, but all sorts of things depend
+; on subscriptions—including wish.providers (indirectly) so to prevent
+; circular dependencies, we resolve it (and memoize the result) at runtime.
+; We *could* just hard-code the provider IDs here, but that duplication
+; seems unnecessarily dumb
+(def ^:private get-info-ref (delay (resolve 'wish.providers/get-info)))
+
 (defn provider-id?
   "Check if the given keyword is the ID of a provider"
   [kw]
   (when kw
-    (contains?
-      #{:gdrive :wish}
-      kw)))
+    (not (nil?
+           (@get-info-ref
+             kw)))))