From a241c491dfc6108283ed4bc94e8aa3a725615440 Mon Sep 17 00:00:00 2001
From: Ajay Kidave <ajayvk@clace.io>
Date: Sat, 30 Nov 2024 19:47:31 -0800
Subject: [PATCH] Added exec redirect to temp file

---
 internal/app/action/result.go.html | 11 ++++++-----
 plugins/exec.go                    | 31 ++++++++++++++++++++++++------
 2 files changed, 31 insertions(+), 11 deletions(-)

diff --git a/internal/app/action/result.go.html b/internal/app/action/result.go.html
index e7662ea..61684c6 100644
--- a/internal/app/action/result.go.html
+++ b/internal/app/action/result.go.html
@@ -83,11 +83,12 @@
   <div id="action_result" hx-swap-oob="true" hx-swap="outerHTML">
     <div class="divider text-lg text-secondary">Output File</div>
 
-    {{ range . }}
-      <h2>
-        <a class="link link-primary" href="{{ .url }}"> Download {{ .name }}</a>
-      </h2>
-    {{ end }}
+    <h2 class="text-center">
+      {{ range . }}
+        Download <a class="link link-primary" href="{{ .url }}">{{ .name }}</a>
+        <br />
+      {{ end }}
+    </h2>
   </div>
 {{ end }}
 
diff --git a/plugins/exec.go b/plugins/exec.go
index a4379ba..ba2b300 100644
--- a/plugins/exec.go
+++ b/plugins/exec.go
@@ -8,6 +8,7 @@ import (
 	"bytes"
 	"fmt"
 	"io"
+	"os"
 	"os/exec"
 
 	"github.com/claceio/clace/internal/app"
@@ -34,12 +35,13 @@ func NewExecPlugin(_ *types.PluginContext) (any, error) {
 
 func (e *ExecPlugin) Run(thread *starlark.Thread, _ *starlark.Builtin, args starlark.Tuple, kwargs []starlark.Tuple) (starlark.Value, error) {
 	var (
-		path           starlark.String
-		cmdArgs        *starlark.List
-		env            *starlark.List
-		processPartial starlark.Bool
+		path                         starlark.String
+		cmdArgs                      *starlark.List
+		env                          *starlark.List
+		processPartial, stdoutToFile starlark.Bool
 	)
-	if err := starlark.UnpackArgs("run", args, kwargs, "path", &path, "args?", &cmdArgs, "env?", &env, "process_partial?", &processPartial); err != nil {
+	if err := starlark.UnpackArgs("run", args, kwargs, "path", &path, "args?", &cmdArgs, "env?", &env,
+		"process_partial?", &processPartial, "stdout_file", &stdoutToFile); err != nil {
 		return nil, err
 	}
 	if cmdArgs == nil {
@@ -53,6 +55,7 @@ func (e *ExecPlugin) Run(thread *starlark.Thread, _ *starlark.Builtin, args star
 	argsList := make([]string, 0, cmdArgs.Len())
 	envList := make([]string, 0, env.Len())
 	processPartialBool := bool(processPartial)
+	stdoutToFileBool := bool(stdoutToFile)
 
 	for i := 0; i < cmdArgs.Len(); i++ {
 		value, ok := cmdArgs.Index(i).(starlark.String)
@@ -84,7 +87,19 @@ func (e *ExecPlugin) Run(thread *starlark.Thread, _ *starlark.Builtin, args star
 	}
 
 	var buf bytes.Buffer
-	_, err = io.CopyN(&buf, stdout, MAX_BYTES_STDOUT)
+	var tempFile *os.File
+
+	if stdoutToFileBool {
+		tempFile, err = os.CreateTemp("", "clace-exec-stdout-*")
+		if err != nil {
+			return nil, fmt.Errorf("error creating temporary file: %w", err)
+		}
+		defer tempFile.Close()
+		_, err = io.Copy(tempFile, stdout)
+	} else {
+		_, err = io.CopyN(&buf, stdout, MAX_BYTES_STDOUT)
+	}
+
 	if err != nil && err != io.EOF {
 		return nil, err
 	}
@@ -97,6 +112,10 @@ func (e *ExecPlugin) Run(thread *starlark.Thread, _ *starlark.Builtin, args star
 		return nil, runErr
 	}
 
+	if stdoutToFileBool {
+		return app.NewResponse(starlark.String(tempFile.Name())), nil
+	}
+
 	lines := starlark.NewList([]starlark.Value{})
 	scanner := bufio.NewScanner(bytes.NewReader(buf.Bytes()))
 	for scanner.Scan() {