fix: dialyzer and compiler warnings

.github/workflows/ci.yml

@@ -12,9 +12,12 @@ jobs:
   lint:
     runs-on: ubuntu-latest
 
+    env:
+      MIX_ENV: test
+
     strategy:
       matrix:
-        elixir: [1.17.0]
+        elixir: [1.18.1]
         otp: [27.0]
 
     steps:
@@ -27,8 +30,30 @@ jobs:
           elixir-version: ${{ matrix.elixir }}
           otp-version: ${{ matrix.otp }}
 
-      - name: Install dependencies
-        run: mix deps.get
+      - name: Cache Elixir deps
+        uses: actions/cache@v1
+        id: deps-cache
+        with:
+          path: deps
+          key: ${{ runner.os }}-mix-${{ env.MIX_ENV }}-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
+
+      - name: Cache Elixir _build
+        uses: actions/cache@v1
+        id: build-cache
+        with:
+          path: _build
+          key: ${{ runner.os }}-build-${{ env.MIX_ENV }}-${{ matrix.otp }}-${{ matrix.elixir }}-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
+
+      - name: Install deps
+        if: steps.deps-cache.outputs.cache-hit != 'true'
+        run: |
+          mix local.rebar --force
+          mix local.hex --force
+          mix deps.get --only ${{ env.MIX_ENV }}
+
+      - name: Compile deps
+        if: steps.build-cache.outputs.cache-hit != 'true'
+        run: mix deps.compile --warnings-as-errors
 
       - name: Clean build
         run: mix clean
@@ -38,3 +63,127 @@ jobs:
 
       - name: Run Credo
         run: mix credo --strict
+
+  static-analisys:
+    runs-on: ubuntu-latest
+
+    env:
+      MIX_ENV: test
+
+    strategy:
+      matrix:
+        elixir: [1.18.1]
+        otp: [27.0]
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Set up Elixir
+        uses: erlef/setup-beam@v1
+        with:
+          elixir-version: ${{ matrix.elixir }}
+          otp-version: ${{ matrix.otp }}
+
+      - name: Cache Elixir deps
+        uses: actions/cache@v1
+        id: deps-cache
+        with:
+          path: deps
+          key: ${{ runner.os }}-mix-${{ env.MIX_ENV }}-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
+
+      - name: Cache Elixir _build
+        uses: actions/cache@v1
+        id: build-cache
+        with:
+          path: _build
+          key: ${{ runner.os }}-build-${{ env.MIX_ENV }}-${{ matrix.otp }}-${{ matrix.elixir }}-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
+
+      - name: Install deps
+        if: steps.deps-cache.outputs.cache-hit != 'true'
+        run: |
+          mix local.rebar --force
+          mix local.hex --force
+          mix deps.get --only ${{ env.MIX_ENV }}
+
+      - name: Compile deps
+        if: steps.build-cache.outputs.cache-hit != 'true'
+        run: mix deps.compile --warnings-as-errors
+
+      # Don't cache PLTs based on mix.lock hash, as Dialyzer can incrementally update even old ones
+      # Cache key based on Elixir & Erlang version (also useful when running in matrix)
+      - name: Restore PLT cache
+        uses: actions/cache/restore@v3
+        id: plt_cache
+        with:
+          key: ${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-plt
+          restore-keys: ${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-plt
+          path: priv/plts
+
+      # Create PLTs if no cache was found
+      - name: Create PLTs
+        if: steps.plt_cache.outputs.cache-hit != 'true'
+        run: mix dialyzer --plt
+
+      - name: Save PLT cache
+        uses: actions/cache/save@v3
+        if: steps.plt_cache.outputs.cache-hit != 'true'
+        id: plt_cache_save
+        with:
+          key: ${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-plt
+          path: priv/plts
+
+      - name: Run dialyzer
+        run: mix dialyzer --format github
+
+  test:
+    runs-on: ubuntu-latest
+
+    env:
+      MIX_ENV: test
+
+    strategy:
+      matrix:
+        elixir: [1.18.1]
+        otp: [27.0]
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Set up Elixir
+        uses: erlef/setup-beam@v1
+        with:
+          elixir-version: ${{ matrix.elixir }}
+          otp-version: ${{ matrix.otp }}
+
+      - name: Cache Elixir deps
+        uses: actions/cache@v1
+        id: deps-cache
+        with:
+          path: deps
+          key: ${{ runner.os }}-mix-${{ env.MIX_ENV }}-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
+
+      - name: Cache Elixir _build
+        uses: actions/cache@v1
+        id: build-cache
+        with:
+          path: _build
+          key: ${{ runner.os }}-build-${{ env.MIX_ENV }}-${{ matrix.otp }}-${{ matrix.elixir }}-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
+
+      - name: Install deps
+        if: steps.deps-cache.outputs.cache-hit != 'true'
+        run: |
+          mix local.rebar --force
+          mix local.hex --force
+          mix deps.get --only ${{ env.MIX_ENV }}
+
+      - name: Compile deps
+        if: steps.build-cache.outputs.cache-hit != 'true'
+        run: mix deps.compile --warnings-as-errors
+
+      - name: Clean build
+        run: mix clean
+
+      - name: Run tests
+        run: mix test

.gitignore

@@ -39,3 +39,5 @@ result
 .elixir_ls/
 .elixir-tools/
 .lexical/
+
+/priv/plts/

README.md

@@ -7,8 +7,8 @@
 ```elixir
 def deps do
   [
-    {:supabase_potion, "~> 0.5"},
-    {:supabase_gotrue, "~> 0.3"}
+    {:supabase_potion, "~> 0.6"},
+    {:supabase_gotrue, "~> 0.4"}
   ]
 end
 ```

lib/supabase/go_true.ex

@@ -39,7 +39,7 @@ defmodule Supabase.GoTrue do
   @impl true
   def get_user(%Client{} = client, %Session{} = session) do
     with {:ok, response} <- UserHandler.get_user(client, session.access_token) do
-      User.parse(response)
+      User.parse(response.body)
     end
   end
 
@@ -116,7 +116,7 @@ defmodule Supabase.GoTrue do
   @impl true
   def verify_otp(%Client{} = client, params) do
     with{:ok, response} <- UserHandler.verify_otp(client, params) do
-      Session.parse(response)
+      Session.parse(response.body)
     end
   end
 
@@ -155,7 +155,7 @@ defmodule Supabase.GoTrue do
   def sign_in_with_password(%Client{} = client, credentials) do
     with{:ok, credentials} <- SignInWithPassword.parse(credentials),
          {:ok, response} <- UserHandler.sign_in_with_password(client, credentials) do
-      Session.parse(response)
+      Session.parse(response.body)
     end
   end
 

lib/supabase/go_true/admin.ex

@@ -36,8 +36,8 @@ defmodule Supabase.GoTrue.Admin do
   def sign_out(%Client{} = client, %Session{} = session, scope) when scope in @scopes do
       case AdminHandler.sign_out(client, session.access_token, scope) do
         {:ok, _} -> :ok
-        {:error, :not_found} -> :ok
-        {:error, :unauthorized} -> :ok
+        {:error, %{code: :not_found}} -> :ok
+        {:error, %{code: :unauthorized}} -> :ok
         err -> err
       end
   end
@@ -57,7 +57,7 @@ defmodule Supabase.GoTrue.Admin do
   def invite_user_by_email(%Client{} = client, email, options \\ %{}) do
     with {:ok, options} <- InviteUserParams.parse(options),
          {:ok, response} <- AdminHandler.invite_user(client, email, options) do
-      User.parse(response)
+      User.parse(response.body)
     end
   end
 
@@ -75,7 +75,7 @@ defmodule Supabase.GoTrue.Admin do
   def generate_link(%Client{} = client, attrs) do
     with {:ok, params} <- GenerateLink.parse(attrs),
          {:ok, response} <- AdminHandler.generate_link(client, params) do
-      GenerateLink.properties(response)
+      GenerateLink.properties(response.body)
     end
   end
 
@@ -93,7 +93,7 @@ defmodule Supabase.GoTrue.Admin do
   def create_user(%Client{} = client, attrs) do
     with {:ok, params} <- AdminUserParams.parse(attrs),
          {:ok, response} <- AdminHandler.create_user(client, params) do
-      User.parse(response)
+      User.parse(response.body)
     end
   end
 
@@ -128,7 +128,7 @@ defmodule Supabase.GoTrue.Admin do
   @impl true
   def get_user_by_id(%Client{} = client, user_id) do
     with {:ok, response} <- AdminHandler.get_user(client, user_id) do
-      User.parse(response)
+      User.parse(response.body)
     end
   end
 
@@ -199,7 +199,7 @@ defmodule Supabase.GoTrue.Admin do
   def update_user_by_id(%Client{} = client, user_id, attrs) do
     with {:ok, params} <- AdminUserParams.parse_update(attrs),
          {:ok, response} <- AdminHandler.update_user(client, user_id, params) do
-      User.parse(response)
+      User.parse(response.body)
     end
   end
 end

lib/supabase/go_true/admin_behaviour.ex

@@ -8,15 +8,14 @@ defmodule Supabase.GoTrue.AdminBehaviour do
   @type scope :: :global | :local | :others
   @type invite_options :: %{data: map, redirect_to: String.t()}
 
-  @callback sign_out(Client.t(), Session.t(), scope) :: :ok | {:error, atom}
-  @callback invite_user_by_email(Client.t(), email, invite_options) :: :ok | {:error, atom}
+  @callback sign_out(Client.t(), Session.t(), scope) :: :ok | {:error, Supabase.Error.t()}
+  @callback invite_user_by_email(Client.t(), email, invite_options) ::
+              :ok | {:error, Supabase.Error.t()}
             when email: String.t()
-  @callback generate_link(Client.t(), map) :: {:ok, String.t()} | {:error, atom}
-  @callback create_user(Client.t(), map) :: {:ok, User.t()} | {:error, atom}
-  @callback list_users(Client.t()) :: {:ok, list(User.t())} | {:error, atom}
-  @callback get_user_by_id(Client.t(), Ecto.UUID.t()) :: {:ok, User.t()} | {:error, atom}
-  @callback update_user_by_id(Client.t(), Ecto.UUID.t(), map) ::
-              {:ok, User.t()} | {:error, atom}
-  @callback delete_user(Client.t(), Ecto.UUID.t(), keyword) ::
-              {:ok, User.t()} | {:error, atom}
+  @callback generate_link(Client.t(), map) :: Supabase.result(String.t())
+  @callback create_user(Client.t(), map) :: Supabase.result(User.t())
+  @callback list_users(Client.t()) :: Supabase.result(list(User.t()))
+  @callback get_user_by_id(Client.t(), Ecto.UUID.t()) :: Supabase.result(User.t())
+  @callback update_user_by_id(Client.t(), Ecto.UUID.t(), map) :: Supabase.result(User.t())
+  @callback delete_user(Client.t(), Ecto.UUID.t(), keyword) :: Supabase.result(User.t())
 end

lib/supabase/go_true/auth.ex

@@ -53,7 +53,7 @@ defmodule Supabase.GoTrue.Auth do
   end
 
   defp network_failure_changeset(failure, attrs) do
-    cast(failure, attrs, ~w[max_retries max_interval])
+    cast(failure, attrs, ~w[max_retries max_interval]a)
   end
 
   def parse_mfa(attrs) do

lib/supabase/go_true/schemas/sign_in_with_password.ex

@@ -15,6 +15,18 @@ defmodule Supabase.GoTrue.Schemas.SignInWithPassword do
 
   import Ecto.Changeset
 
+  @type t :: %__MODULE__{
+          email: String.t() | nil,
+          phone: String.t() | nil,
+          password: String.t() | nil,
+          options:
+            %__MODULE__.Options{
+              data: map | nil,
+              captcha_token: String.t() | nil
+            }
+            | nil
+        }
+
   @primary_key false
   embedded_schema do
     field(:email, :string)
@@ -51,6 +63,6 @@ defmodule Supabase.GoTrue.Schemas.SignInWithPassword do
   end
 
   defp options_changeset(options, attrs) do
-    cast(options, attrs, ~w[email_redirect_to data captcha_token]a)
+    cast(options, attrs, ~w[data captcha_token]a)
   end
 end

lib/supabase/go_true/schemas/user_params.ex

@@ -13,6 +13,15 @@ defmodule Supabase.GoTrue.Schemas.UserParams do
 
   import Ecto.Changeset
 
+  @type t :: %{
+          data: map | nil,
+          email: String.t() | nil,
+          phone: String.t() | nil,
+          password: String.t() | nil,
+          nonce: String.t() | nil,
+          email_redirect_to: String.t() | nil
+        }
+
   @types %{
     data: :map,
     email: :string,