nitpicks

lib/plugins/why.ex

@@ -90,6 +90,7 @@ defmodule Plugins.Why do
   end
 
   defmodule Debugging do
+    @moduledoc false
     def probably_a_traceback(str) when is_binary(str),
       do: String.contains?(str, "We asked")
 

lib/service.ex

@@ -47,7 +47,9 @@ defmodule Service do
     quote do
       @behaviour unquote(__MODULE__)
 
-      # in order to use the default schema in doc strings, use this attribute, as the function will not be found when the compiler is compiling the docstring.
+      # in order to use the default schema in doc strings, use this attribute,
+      # as docstrings are made at compile-time and the function will not be
+      # ready yet.
       # For example:
       #
       #   @moduledoc """
@@ -115,7 +117,6 @@ defmodule Service do
   # Some common functions make more sense being abbreviated here
   # Usually rearranging the arguments for better piping
 
-  # TODO: move this into service-generic Stampede.Logger
   def txt_format(blk, type, :logger),
     do: TxtBlock.Md.format(blk, type)
 

lib/stampede.ex

@@ -102,7 +102,7 @@ defmodule Stampede do
         |> List.to_string()
         |> String.starts_with?(to_string(module_name) <> ".")
         |> if do
-          MapSet.put(acc, List.to_atom(name))
+          MapSet.put(acc, List.to_existing_atom(name))
         else
           acc
         end

lib/stampede/interaction_form.ex

@@ -9,7 +9,6 @@ defmodule Stampede.InteractionForm do
   use TypeCheck.Defstruct
 
   defstruct!(
-    # TODO: rename to "chosen_plugin"
     plugin: _ :: any(),
     service: _ :: atom(),
     msg: _ :: MsgReceived.t(),

lib/stampede/scheduler.ex

@@ -1,3 +1,4 @@
 defmodule Stampede.Scheduler do
+  @moduledoc false
   use Quantum, otp_app: :stampede
 end

lib/stampede/tables.ex

@@ -1,6 +1,7 @@
 defmodule Stampede.Tables do
-  @compile [:bin_opt_info, :recv_opt_info]
   # Manage various Mnesia tables. Generally avoiding keeping business logic here
+  @moduledoc false
+  @compile [:bin_opt_info, :recv_opt_info]
   require Logger
   use TypeCheck
   alias Stampede, as: S

lib/txt_block.ex

@@ -87,7 +87,8 @@ defmodule TxtBlock do
 
   def plain_indent_io(str, prefix, bp) when is_binary(prefix) do
     IO.iodata_to_binary(str)
-    # TODO: this is being recompiled every time. figure out how to precompile binary patterns without needing application state
+    # TODO: this is being recompiled every time. figure out how to
+    # precompile binary patterns without needing application state
     |> String.split(bp || bp_newline(), trim: true)
     |> Enum.flat_map(&[prefix, &1, "\n"])
   end
@@ -96,6 +97,7 @@ defmodule TxtBlock do
 end
 
 defmodule TxtBlock.Debugging do
+  @moduledoc false
   use TypeCheck
 
   @spec! all_formats_example() :: TxtBlock.t()