Fix .env which broke as part of moving cmds to the project service (#493

)

* Fix .env* loading

* Compact logic

* Don't use present progressive

* Load envs when options are applied

internal/runner/client/client.go

@@ -8,6 +8,7 @@ import (
 
 	"github.com/go-git/go-billy/v5/osfs"
 	"github.com/muesli/cancelreader"
+	"github.com/pkg/errors"
 	"github.com/stateful/runme/internal/document"
 	runnerv1 "github.com/stateful/runme/internal/gen/proto/go/runme/runner/v1"
 	"github.com/stateful/runme/internal/project"
@@ -88,8 +89,13 @@ func WithSessionID(id string) RunnerOption {
 }
 
 func WithProject(proj *project.Project) RunnerOption {
-	return withSettings(func(rs *RunnerSettings) {
+	return withSettingsErr(func(rs *RunnerSettings) error {
 		rs.project = proj
+
+		projEnvs, err := rs.project.LoadEnv()
+		rs.envs = append(rs.envs, projEnvs...)
+
+		return errors.Wrap(err, "failed to apply project envs")
 	})
 }
 

internal/runner/client/client_local.go

@@ -95,7 +95,6 @@ func (r *LocalRunner) newExecutable(task project.Task) (runner.Executable, error
 		Logger:  r.logger,
 	}
 
-	// TODO(adamb): what about `r.envs`?
 	cfg.PreEnv, err = r.project.LoadEnv()
 	if err != nil {
 		return nil, err

testdata/script/dotenv.txtar

@@ -0,0 +1,15 @@
+exec runme run env1
+stdout 'SOMETHING=in-my-dot-env'
+! stderr .
+
+-- .env --
+SOMETHING="in-my-dot-env"
+
+-- README.md --
+---
+shell: bash
+---
+
+```sh {"name":"env1"}
+echo SOMETHING=${SOMETHING}
+```