feat: add skip_cache option for identity resolver

Author: Ovyerus

.gitignore

@@ -23,4 +23,5 @@ atex-*.tar
 /tmp/
 
 .envrc
-.direnv
+.direnv
+.vscode/

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+  "git.enabled": false
+}

lib/atex/identity_resolver.ex

@@ -2,12 +2,19 @@ defmodule Atex.IdentityResolver do
   alias Atex.IdentityResolver.{Cache, DID, DIDDocument, Handle, Identity}
 
   @handle_strategy Application.compile_env(:atex, :handle_resolver_strategy, :dns_first)
+  @type options() :: {:skip_cache, boolean()}
 
   # TODO: simplify errors
 
-  def resolve(identifier) do
+  @spec resolve(String.t(), list(options())) :: {:ok, Identity.t()} | {:error, any()}
+  def resolve(identifier, opts \\ []) do
+    opts = Keyword.validate!(opts, skip_cache: false)
+    skip_cache = Keyword.get(opts, :skip_cache)
+
+    cache_result = if skip_cache, do: {:error, :not_found}, else: Cache.get(identifier)
+
     # If cache fetch succeeds, then the ok tuple will be retuned by the default `with` behaviour
-    with {:error, :not_found} <- Cache.get(identifier),
+    with {:error, :not_found} <- cache_result,
          {:ok, identity} <- do_resolve(identifier),
          identity <- Cache.insert(identity) do
       {:ok, identity}

lib/atex/identity_resolver/did_document.ex

@@ -68,8 +68,6 @@ defmodule Atex.IdentityResolver.DIDDocument do
   @spec from_json(map()) :: {:ok, t()} | {:error, Peri.Error.t()}
   def from_json(%{} = map) do
     map
-    # TODO: `atomize_keys` instead? Peri doesn't convert nested schemas to atoms but does for the base schema.
-    # Smells like a PR if I've ever smelt one...
     |> Recase.Enumerable.convert_keys(&Recase.to_snake/1)
     |> schema()
     |> case do