feat: use Err type rather than a tuple

- No need override IO::MultiWriter IO type

Author: initdc

README.md

@@ -10,7 +10,7 @@
    dependencies:
      exec:
        github: initdc/exec.cr
-       version: 0.2.1
+       version: 0.2.2
    ```
 
 2. Run `shards install`

shard.yml

@@ -1,5 +1,5 @@
 name: exec
-version: 0.2.1
+version: 0.2.2
 
 authors:
   - initdc <initd@outlook.com>

spec/exec_spec.cr

@@ -35,10 +35,9 @@ describe Exec do
     when String
       r.chomp.should eq "good"
     else
-      o, e, s = r
-      o.chomp.should eq "good"
-      e.chomp.should eq "bad"
-      s.exit_code.should eq 1
+      r.stdout.chomp.should eq "good"
+      r.stderr.chomp.should eq "bad"
+      r.status.exit_code.should eq 1
     end
   end
 

src/exec.cr

@@ -1,31 +1,24 @@
 class Exec < Process
-  VERSION = "0.2.1"
+  VERSION = "0.2.2"
 
-  class IO
-    class MultiWriter < ::IO::MultiWriter
-      alias IO = ::IO | ::IO::FileDescriptor | ::String::Builder
-
-      def read(slice : Bytes) : NoReturn
-        raise ::IO::Error.new("Can't read from IO::MultiWriter")
-      end
-    end
-  end
+  # https://github.com/crystal-lang/crystal/blob/1.19.1/src/compiler/crystal/macros/macros.cr#L68
+  record Err, stdout : String, stderr : String, status : Process::Status
 
   def self.run(command : String, args = nil, env : Env = nil, clear_env : Bool = false, shell : Bool = true,
-               input : Stdio = Redirect::Inherit, output : Stdio = Redirect::Inherit, error : Stdio = Redirect::Inherit, chdir : Path | String? = nil) : String | {String, String, Process::Status}
+               input : Stdio = Redirect::Inherit, output : Stdio = Redirect::Inherit, error : Stdio = Redirect::Inherit, chdir : Path | String? = nil) : String | Err
     output_strio = String::Builder.new
     error_strio = String::Builder.new
 
     output_writer = if output.is_a?(Redirect)
-                      output == Redirect::Close ? output : Exec::IO::MultiWriter.new(STDOUT, output_strio)
+                      output == Redirect::Close ? output : IO::MultiWriter.new(STDOUT, output_strio)
                     else
-                      output != STDOUT ? Exec::IO::MultiWriter.new(STDOUT, output, output_strio) : Exec::IO::MultiWriter.new(STDOUT, output_strio)
+                      output != STDOUT ? IO::MultiWriter.new(STDOUT, output, output_strio) : IO::MultiWriter.new(STDOUT, output_strio)
                     end
 
     error_writer = if error.is_a?(Redirect)
-                     error == Redirect::Close ? error : Exec::IO::MultiWriter.new(STDERR, error_strio)
+                     error == Redirect::Close ? error : IO::MultiWriter.new(STDERR, error_strio)
                    else
-                     error != STDERR ? Exec::IO::MultiWriter.new(STDERR, error, error_strio) : Exec::IO::MultiWriter.new(STDERR, error_strio)
+                     error != STDERR ? IO::MultiWriter.new(STDERR, error, error_strio) : IO::MultiWriter.new(STDERR, error_strio)
                    end
 
     status = new(command, args, env, clear_env, shell, input, output_writer, error_writer, chdir).wait
@@ -40,7 +33,7 @@ class Exec < Process
     when true
       output_strio.to_s
     else
-      {output_strio.to_s, error_strio.to_s, status}
+      Err.new(output_strio.to_s, error_strio.to_s, status)
     end
   end