Proposal: adding `Igniter.copy`

[mayel]

Describe the solution you'd like
Igniter.copy to copy file(s) or directories recursively.

Describe alternatives you've considered

I currently have this in a helper module used by my tasks, but thought it may make sense for it to be supported upstream:

  def igniter_copy(igniter, source, target, opts \\ [])
  def igniter_copy(igniter, sources, target, opts) when is_list(sources) do
    IO.puts("Batch copying #{inspect sources} to #{target}")
    Enum.reduce(sources, igniter, fn source, igniter ->
      igniter_copy(igniter, source, prepare_target_path(source, target), opts)
    end)
  end
  def igniter_copy(igniter, source, target, opts) do
    IO.puts("Copying #{source} to #{target}")

    if File.dir?(source) do

      sources = File.ls!(source)
      |> Enum.map(fn file -> Path.join(source, file) end)

      igniter_copy(igniter, sources, target, opts)

    else
      contents_to_copy = File.read!(source)

      Igniter.create_or_update_file(igniter, target, contents_to_copy, fn source ->
          Rewrite.Source.update(source, :content, contents_to_copy)
      end)
    end
  end

  defp prepare_target_path(source, target) do
    # Remove the repeated part of source from target
    if String.contains?(target, source) do
      String.replace(target, source, "")
      |> String.trim_leading("/")
      |> (&Path.join(target, &1)).()
    else
      Path.join(target, Path.basename(source))
    end
  end

[zachdaniel]

I like it 👍 . Would also be good if it just stored the copies to be made like file moving does. Then we can show cleaner output at the end, like Copying: ... and just show source/dest.

[mayel]

Ah I had modeled it on copy_template, now looking at move_file I see it setting either a moves or a rewrite key on the igniter schema, but where does it do the actual moving? Also how could we support (optionally) overwriting the content if the destination already exists like my code does (edit: I guess we can just update_file in that case instead of printing an error)?

[zachdaniel]

We'd copy files at the very end same place we move them . But also I misunderstood what you were after, you're just looking for a bulk copy template which is not at all what I was describing 🤣 I think yes a function like copy_templates that supported directories and files could make sense.

[mayel]

Ah ok cool I'll improve it a bit and PR 😃