Reformatted

Author: scohen

apps/engine/lib/engine/analyzer.ex

@@ -115,7 +115,7 @@ defmodule Engine.Analyzer do
           {:ok, module()} | :error
   def expand_alias([_ | _] = segments, %Analysis{} = analysis, %Position{} = position) do
     with %Analysis{valid?: true} = analysis <- Forge.Ast.reanalyze_to(analysis, position),
-         aliases <- aliases_at(analysis, position),
+         aliases = aliases_at(analysis, position),
          {:ok, resolved} <- resolve_alias(segments, aliases) do
       {:ok, Module.concat(resolved)}
     else

apps/engine/lib/engine/api/proxy.ex

@@ -29,20 +29,18 @@ defmodule Engine.Api.Proxy do
 
   """
 
-  alias Forge.Document
-  alias Forge.Document.Changes
+  @behaviour :gen_statem
+
+  import Engine.Api.Proxy.Records, only: :macros
+  import Record
 
   alias Engine.Api.Proxy.BufferingState
   alias Engine.Api.Proxy.DrainingState
   alias Engine.Api.Proxy.ProxyingState
-  alias Engine.Api.Proxy.Records
   alias Engine.CodeMod
   alias Engine.Commands
-
-  import Record
-  import Records, only: :macros
-
-  @behaviour :gen_statem
+  alias Forge.Document
+  alias Forge.Document.Changes
 
   defrecord :buffer, contents: nil, return: :ok
   defrecord :drop, contents: nil, return: :ok

apps/engine/lib/engine/api/proxy/buffering_state.ex

@@ -1,11 +1,10 @@
 defmodule Engine.Api.Proxy.BufferingState do
-  alias Forge.Document
+  import Engine.Api.Proxy.Records
+  import Forge.EngineApi.Messages
 
   alias Engine.Build
   alias Engine.Commands
-
-  import Forge.EngineApi.Messages
-  import Engine.Api.Proxy.Records
+  alias Forge.Document
 
   defstruct initiator_pid: nil, buffer: []
 

apps/engine/lib/engine/api/proxy/draining_state.ex

@@ -1,9 +1,8 @@
 defmodule Engine.Api.Proxy.DrainingState do
+  import Engine.Api.Proxy.Records
+
   alias Engine.Api.Proxy.BufferingState
   alias Engine.Api.Proxy.ProxyingState
-  alias Engine.Api.Proxy.Records
-
-  import Records
 
   defstruct [:proxying_state, :buffering_state]
 

apps/engine/lib/engine/api/proxy/proxying_state.ex

@@ -1,10 +1,8 @@
 defmodule Engine.Api.Proxy.ProxyingState do
-  alias Engine.Api.Proxy.Records
+  import Engine.Api.Proxy.Records
 
   defstruct refs_to_from: %{}
 
-  import Records
-
   def new do
     %__MODULE__{}
   end

apps/engine/lib/engine/api/proxy/records.ex

@@ -1,8 +1,8 @@
 defmodule Engine.Api.Proxy.Records do
-  alias Forge.Formats
-
   import Record
 
+  alias Forge.Formats
+
   defrecord :mfa, module: nil, function: nil, arguments: [], seq: nil
 
   def mfa(module, function, arguments) do
@@ -22,7 +22,7 @@ defmodule Engine.Api.Proxy.Records do
     Code.ensure_compiled!(module)
     Code.ensure_loaded!(module)
 
-    unless function_exported?(module, f, arity) do
+    if !function_exported?(module, f, arity) do
       mfa = Formats.mfa(module, f, arity)
 
       raise CompileError.message(%{

apps/engine/lib/engine/bootstrap.ex

@@ -111,7 +111,7 @@ defmodule Engine.Bootstrap do
         |> Path.expand()
       end)
 
-    unless current_dir == configured_root do
+    if current_dir != configured_root do
       File.cd!(configured_root)
     end
   end

apps/engine/lib/engine/build.ex

@@ -1,11 +1,12 @@
 defmodule Engine.Build do
+  use GenServer
+
   alias Engine.Build.Document.Compilers.HEEx
   alias Engine.Build.State
   alias Forge.Document
   alias Forge.Project
 
   require Logger
-  use GenServer
 
   @timeout_interval_millis 50
 

apps/engine/lib/engine/build/capture_io.ex

@@ -88,16 +88,14 @@ defmodule Engine.Build.CaptureIO do
   end
 
   defp do_capture_gl(string_io, fun) do
-    try do
-      fun.()
-    catch
-      kind, reason ->
-        {:ok, {_input, output}} = StringIO.close(string_io)
-        {output, {kind, reason}}
-    else
-      result ->
-        {:ok, {_input, output}} = StringIO.close(string_io)
-        {output, result}
-    end
+    fun.()
+  catch
+    kind, reason ->
+      {:ok, {_input, output}} = StringIO.close(string_io)
+      {output, {kind, reason}}
+  else
+    result ->
+      {:ok, {_input, output}} = StringIO.close(string_io)
+      {output, result}
   end
 end

apps/engine/lib/engine/build/capture_server.ex

@@ -1,11 +1,11 @@
 defmodule Engine.Build.CaptureServer do
   @moduledoc false
+  use GenServer
+
   @compile {:no_warn_undefined, Logger}
   @timeout :infinity
   @name __MODULE__
 
-  use GenServer
-
   def start_link(_opts) do
     GenServer.start_link(__MODULE__, :ok, name: @name)
   end