lib/credo/check/readability/alias_as.ex

defmodule Credo.Check.Readability.AliasAs do
  use Credo.Check,
    id: "EX3001",
    base_priority: :low,
    tags: [:experimental],
    explanations: [
      check: """
      Aliases can be "renamed" using the `:as` option, but that sometimes
      makes the code more difficult to read.

          # preferred

          def MyModule do
            alias MyApp.Module1

            def my_function(foo) do
              Module1.run(foo)
            end
          end

          # NOT preferred

          def MyModule do
            alias MyApp.Module1, as: M1

            def my_function(foo) do
              # what the heck is `M1`?
              M1.run(foo)
            end
          end

      Please note that you might want to deactivate this check for cases in which you have an alias that
      is used tons throughout your codebase.

      If, for example, you are using a third-party module named `FlupsyTopsyDataRetentionServiceServer`
      in half your modules, it is of course reasonable to alias it to `Server`.

      Like all `Readability` issues, this one is not a technical concern.
      But you can improve the odds of others reading and liking your code by making
      it easier to follow.
      """
    ]

  @doc false
  @impl true
  def run(%SourceFile{} = source_file, params) do
    source_file
    |> Credo.Code.prewalk(&traverse(&1, &2, IssueMeta.for(source_file, params)))
    |> Enum.reverse()
  end

  defp traverse(ast, issues, issue_meta), do: {ast, add_issue(issues, issue(ast, issue_meta))}

  defp add_issue(issues, nil), do: issues
  defp add_issue(issues, issue), do: [issue | issues]

  defp issue({:alias, _, [{:__MODULE__, _, nil}, [as: {_, meta, _}]]}, issue_meta),
    do: issue_for(issue_meta, meta[:line])

  defp issue({:alias, _, [{_, _, _}, [as: {_, meta, _}]]}, issue_meta) do
    issue_for(issue_meta, meta[:line])
  end

  defp issue(_ast, _issue_meta), do: nil

  defp issue_for(issue_meta, line_no) do
    format_issue(
      issue_meta,
      message: "Avoid using the `:as` option with `alias`.",
      trigger: "as:",
      line_no: line_no
    )
  end
end