Add docs to Mix tasks

- mix help will now list the available atomvm task
- mix doc will generate ex docs

Author: joustava

README.md

@@ -1,18 +1,6 @@
 defmodule ExAtomVM do
-  @moduledoc """
-  Documentation for ExAtomVM.
-  """
-
-  @doc """
-  Hello world.
-
-  ## Examples
-
-      iex> ExAtomVM.hello()
-      :world
-
-  """
-  def hello do
-    :world
-  end
+  @moduledoc "README.md"
+             |> File.read!()
+             |> String.split("<!-- MDOC !-->")
+             |> Enum.fetch!(1)
 end

lib/ex_atom_vm.ex

@@ -1,5 +1,8 @@
 defmodule Mix.Tasks.Atomvm.Check do
+  @shortdoc "Check application code for use of unsupported instructions"
+
   alias Mix.Project
+  use Mix.Task
 
   def run(args) do
     Mix.Tasks.Compile.run(args)
@@ -201,6 +204,7 @@ defmodule Mix.Tasks.Atomvm.Check do
           you must remove the parentheses: map.field
         """)
       end
+
       IO.puts("error: following missing instructions are used:")
       print_list(missing_instructions)
       IO.puts("")

lib/mix/tasks/check.ex

@@ -1,5 +1,59 @@
 defmodule Mix.Tasks.Atomvm.Esp32.Flash do
   use Mix.Task
+
+  @shortdoc "Flash the application to an ESP32 micro-controller"
+
+  @moduledoc """
+  Flashes the application to an ESP32 micro-controller.
+
+  > #### Info {: .info}
+  >
+  > Note. Before running this task, you must flash the AtomVM virtual machine to the target device.
+  >
+  > ### ESP32 Build Requirements
+  >
+  > This tasks depends on a host installation of the IDF_SDK and Espressif Xtensa tool chains, see [Building for ESP32](https://www.atomvm.net/doc/main/build-instructions.html#building-for-esp32)
+  >
+  > If the IDF_PATH environment variable is set, then the esptool.py from the IDF SDK installation will be used to flash the application to the ESP32 device.
+  > Otherwise, this plugin will attempt to use the esptool.py program from the user's PATH environment variable.
+  > The ESP Tool Python3 application can be installed from source or via many popular package managers. Consult your local OS documentation for more information.
+
+  ## Usage example
+
+  Within your AtomVM mix project run
+
+  `
+  $ mix atomvm.esp32.flash
+  `
+
+  Or with optional flags (which will override the config in mix.exs)
+
+  `
+  $ mix atomvm.esp32.flash --port /dev/tty.usbserial-0001
+  `
+
+  ## Configuration
+
+  ExAtomVM can be configured from the mix.ex file and supports the following settings for the
+  `atomvm.esp32.flash` task.
+
+    * `:flash_offset` - The start address of the flash to write the application to in hexademical format,
+      defaults to `0x250000`.
+
+    * `:chip` - Chip type, defaults to `:esp32`.
+
+    * `:port` - The port to which device is connected on the host computer, defaults to `/dev/ttyUSB0`.
+
+    * `:baud` - The BAUD rate used when flashing to device, defaults to `115200`.
+
+  ## Command line options
+
+  Properties in the mix.exs file may be over-ridden on the command line using long-style flags (prefixed by --) by the same name
+  as the supported properties key listed in .
+
+  For example, you can use the `--port` option to specify or override the port property.
+  """
+
   alias Mix.Project
   alias Mix.Tasks.Atomvm.Packbeam
 

lib/mix/tasks/esp32_flash.ex

@@ -1,5 +1,35 @@
 defmodule Mix.Tasks.Atomvm.Packbeam do
   use Mix.Task
+
+  @shortdoc "Bundle the application into an AVM file"
+
+  @moduledoc """
+  Bundle an application into an AVM file that can be flashed to a micro-controller and executed by the AtomVM virtual machine.
+
+
+  ## Usage example
+
+  Within your AtomVM mix project run
+
+  `
+  $ mix atomvm.packbeam --start MyProject
+  `
+
+  ## Configuration
+
+  ExAtomVM can be configured from the mix.ex file and supports the following settings for the
+  `atomvm.packbeam` task.
+
+    * `:start` - The name of the module containing the start/0 entrypoint function.
+
+  ## Command line options
+
+  Properties in the mix.exs file may be over-ridden on the command line using long-style flags (prefixed by --) by the same name
+  as the supported properties key listed in .
+
+  For example, you can use the `--start` option to specify or override the `start` property.
+  """
+
   alias ExAtomVM.PackBEAM
   alias Mix.Project
   alias Mix.Tasks.Atomvm.Check

lib/mix/tasks/packbeam.ex

@@ -1,5 +1,53 @@
 defmodule Mix.Tasks.Atomvm.Pico.Flash do
   use Mix.Task
+
+  @shortdoc "Flash the application to a pico micro-controller"
+
+  @moduledoc """
+  Flashes the application to a Raspberry Pi RP2
+
+  You can build with all boards supported by Raspberry Pi pico SDK, including Pico, Pico-W and Pico2.
+  AtomVM also works with clones such as RP2040 Zero.
+
+  > #### Info {: .info}
+  >
+  > Note. Before running this task, you must flash the AtomVM virtual machine to the target device.
+  >
+  > This tasks depends on a host installation of pico supported tooling, see [Building for Raspberry Pi RP2](https://www.atomvm.net/doc/master/build-instructions.html#building-for-raspberry-pi-rp2)
+
+  ## Usage example
+
+  Within your AtomVM mix project run
+
+  `
+  $ mix atomvm.pico.flash
+  `
+
+  Or with optional flags (which will override the config in mix.exs)
+
+  `
+  $ mix atomvm.pico.flash --picotool /some/path
+  `
+
+  ## Configuration
+
+  ExAtomVM can be configured from the mix.ex file and supports the following settings for the
+  `atomvm.pico.flash` task.
+
+    * `:pico_path` - The full path to the pico mount point, defaults to `"/run/media/${USER}/RPI-RP2"` on linux; `"/Volumes/RPI-RP2"` on darwin (Mac)
+
+    * `:pico_reset` - The full path to the pico device to reset if required, default `"/dev/ttyACM*"` on linux; `"/dev/cu.usbmodem14*"` on darwin (Mac)
+
+    * `:picotool` - The full path to picotool executable (currently optional), default `undefined`
+
+  ## Command line options
+
+  Properties in the mix.exs file may be over-ridden on the command line using long-style flags (prefixed by --) by the same name
+  as the supported properties key listed in .
+
+  For example, you can use the `--picotool` option to specify or override the `picotool` property.
+  """
+
   alias Mix.Project
   alias Mix.Tasks.Atomvm.Uf2create
 
@@ -9,12 +57,38 @@ defmodule Mix.Tasks.Atomvm.Pico.Flash do
     with {:atomvm, {:ok, avm_config}} <- {:atomvm, Keyword.fetch(config, :atomvm)},
          {:args, {:ok, options}} <- {:args, parse_args(args)},
          {:uf2, :ok} <- {:uf2, Uf2create.run(args)} do
-      pico_path = 
-        Map.get(options, :pico_path, Keyword.get(avm_config, :pico_path, System.get_env("ATOMVM_PICO_MOUNT_PATH", get_default_mount())))
+      pico_path =
+        Map.get(
+          options,
+          :pico_path,
+          Keyword.get(
+            avm_config,
+            :pico_path,
+            System.get_env("ATOMVM_PICO_MOUNT_PATH", get_default_mount())
+          )
+        )
+
       pico_reset =
-        Map.get(options, :pico_reset, Keyword.get(avm_config, :pico_reset, System.get_env("ATOMVM_PICO_RESET_DEV", get_reset_base())))
+        Map.get(
+          options,
+          :pico_reset,
+          Keyword.get(
+            avm_config,
+            :pico_reset,
+            System.get_env("ATOMVM_PICO_RESET_DEV", get_reset_base())
+          )
+        )
+
       picotool =
-        Map.get(options, :picotool, Keyword.get(avm_config, :picotool, System.get_env("ATOMVM_PICOTOOL_PATH", "#{:os.find_executable(~c"picotool")}")))
+        Map.get(
+          options,
+          :picotool,
+          Keyword.get(
+            avm_config,
+            :picotool,
+            System.get_env("ATOMVM_PICOTOOL_PATH", "#{:os.find_executable(~c"picotool")}")
+          )
+        )
 
       do_flash(pico_path, pico_reset, picotool)
     else
@@ -70,13 +144,16 @@ defmodule Mix.Tasks.Atomvm.Pico.Flash do
         case Map.get(filestat, :type) do
           :directory ->
             :ok
+
           _ ->
             IO.puts("Object found at #{mount} is not a directory")
             exit({:shutdown, 1})
         end
+
       {:error, :enoent} ->
         Process.sleep(1000)
         wait_for_mount(mount, count + 1)
+
       error ->
         IO.puts("unexpected error: #{error} while checking pico mount path.")
         exit({:shutdown, 1})
@@ -94,10 +171,12 @@ defmodule Mix.Tasks.Atomvm.Pico.Flash do
         case Map.get(info, :type) do
           :directory ->
             :ok
+
           _ ->
             IO.puts("error: object at pico mount path not a directory. Abort!")
             exit({:shutdown, 1})
         end
+
       _ ->
         IO.puts("error: Pico not mounted. Abort!")
         exit({:shutdown, 1})
@@ -123,16 +202,19 @@ defmodule Mix.Tasks.Atomvm.Pico.Flash do
     case Path.wildcard(resetdev) do
       [] ->
         false
+
       [device | _t] ->
         case File.stat(device) do
           {:ok, info} ->
             case Map.get(info, :type) do
               :device -> {true, device}
               _ -> false
             end
+
           _ ->
             false
         end
+
       _ ->
         false
     end
@@ -146,25 +228,40 @@ defmodule Mix.Tasks.Atomvm.Pico.Flash do
       {"", 0} ->
         # Pause to let the device settle
         Process.sleep(200)
+
       error ->
         case picotool do
           false ->
-            IO.puts("Error: #{error}\nUnable to locate 'picotool', close the serial monitor before flashing, or install picotool for automatic disconnect and BOOTSEL mode.")
+            IO.puts(
+              "Error: #{error}\nUnable to locate 'picotool', close the serial monitor before flashing, or install picotool for automatic disconnect and BOOTSEL mode."
+            )
+
             exit({:shutdown, 1})
+
           _ ->
-            IO.puts("Warning: #{error}\nFor faster flashing remember to disconnect serial monitor first.")
+            IO.puts(
+              "Warning: #{error}\nFor faster flashing remember to disconnect serial monitor first."
+            )
+
             reset_args = ["reboot", "-f", "-u"]
-            IO.puts("Disconnecting serial monitor with `picotool #{:lists.join(" ", reset_args)}` in 5 seconds...")
+
+            IO.puts(
+              "Disconnecting serial monitor with `picotool #{:lists.join(" ", reset_args)}` in 5 seconds..."
+            )
+
             Process.sleep(5000)
+
             case :string.trim(System.cmd(picotool, reset_args)) do
               {status, 0} ->
                 case status do
                   "The device was asked to reboot into BOOTSEL mode." ->
                     :ok
+
                   pt_error ->
                     IO.puts("Failed to prepare pico for flashing: #{pt_error}")
                     exit({:shutdown, 1})
                 end
+
               _ ->
                 IO.puts("Failed to prepare pico for flashing: #{error}")
             end
@@ -176,14 +273,22 @@ defmodule Mix.Tasks.Atomvm.Pico.Flash do
     case needs_reset(pico_reset) do
       false ->
         :ok
+
       {true, reset_port} ->
         do_reset(reset_port, picotool)
         IO.puts("Waiting for the device at path #{pico_path} to settle and mount...")
         wait_for_mount(pico_path, 0)
     end
 
     check_pico_mount(pico_path)
-    _bytes = File.copy!("#{Project.config()[:app]}.uf2", "#{pico_path}/#{Project.config()[:app]}.uf2", :infinity)
+
+    _bytes =
+      File.copy!(
+        "#{Project.config()[:app]}.uf2",
+        "#{pico_path}/#{Project.config()[:app]}.uf2",
+        :infinity
+      )
+
     IO.puts("Successfully loaded #{Project.config()[:app]} to the pico device at #{pico_path}.")
   end
 end