fix: fail to find local subworkflow of remote workflow

cmd/root.go

@@ -433,7 +433,7 @@ func newRunCommand(ctx context.Context, input *Input) func(*cobra.Command, []str
 		matrixes := parseMatrix(input.matrix)
 		log.Debugf("Evaluated matrix inclusions: %v", matrixes)
 
-		planner, err := model.NewWorkflowPlanner(input.WorkflowsPath(), input.noWorkflowRecurse)
+		planner, err := model.NewWorkflowPlanner("", input.WorkflowsPath(), input.noWorkflowRecurse)
 		if err != nil {
 			return err
 		}

pkg/artifacts/server_test.go

@@ -297,7 +297,7 @@ func runTestJobFile(ctx context.Context, t *testing.T, tjfi TestJobFileInfo) {
 		runner, err := runner.New(runnerConfig)
 		assert.Nil(t, err, tjfi.workflowPath)
 
-		planner, err := model.NewWorkflowPlanner(fullWorkflowPath, true)
+		planner, err := model.NewWorkflowPlanner(workdir, tjfi.workflowPath, true)
 		assert.Nil(t, err, fullWorkflowPath)
 
 		plan, err := planner.PlanEvent(tjfi.eventName)

pkg/model/planner.go

@@ -58,7 +58,13 @@
 // NewWorkflowPlanner will load a specific workflow, all workflows from a directory or all workflows from a directory and its subdirectories
 //
 //nolint:gocyclo
-func NewWorkflowPlanner(path string, noWorkflowRecurse bool) (WorkflowPlanner, error) {
+func NewWorkflowPlanner(directory string, workflow string, noWorkflowRecurse bool) (WorkflowPlanner, error) {
+	path := ""
+	if directory == "" || workflow == "" {
+		path = directory + workflow
+	} else {
+		path = filepath.Join(directory, workflow)
+	}
 	path, err := filepath.Abs(path)
 	if err != nil {
 		return nil, err
@@ -142,7 +148,7 @@
 				_ = f.Close()
 				return nil, fmt.Errorf("error occurring when resetting io pointer in '%s': %w", wf.workflowDirEntry.Name(), err)
 			}
-
+			workflow.RepoPath = directory
 			workflow.File = wf.workflowDirEntry.Name()
 			if workflow.Name == "" {
 				workflow.Name = wf.workflowDirEntry.Name()

pkg/model/planner_test.go

@@ -30,8 +30,7 @@ func TestPlanner(t *testing.T) {
 	workdir, err := filepath.Abs("testdata")
 	assert.NoError(t, err, workdir)
 	for _, table := range tables {
-		fullWorkflowPath := filepath.Join(workdir, table.workflowPath)
-		_, err = NewWorkflowPlanner(fullWorkflowPath, table.noWorkflowRecurse)
+		_, err = NewWorkflowPlanner(workdir, table.workflowPath, table.noWorkflowRecurse)
 		if table.errorMessage == "" {
 			assert.NoError(t, err, "WorkflowPlanner should exit without any error")
 		} else {

pkg/model/workflow.go

@@ -15,6 +15,7 @@ import (
 
 // Workflow is the structure of the files in .github/workflows
 type Workflow struct {
+	RepoPath string
 	File     string
 	Name     string            `yaml:"name"`
 	RawOn    yaml.Node         `yaml:"on"`

pkg/model/workflow_test.go

@@ -334,7 +334,7 @@ jobs:
 }
 
 func TestReadWorkflow_Strategy(t *testing.T) {
-	w, err := NewWorkflowPlanner("testdata/strategy/push.yml", true)
+	w, err := NewWorkflowPlanner("testdata/strategy", "push.yml", true)
 	assert.NoError(t, err)
 
 	p, err := w.PlanJob("strategy-only-max-parallel")

pkg/runner/reusable_workflow.go

@@ -7,8 +7,8 @@
 	"fmt"
 	"io/fs"
 	"os"
-	"path"
 	"regexp"
+	"strings"
 	"sync"
 
 	"github.com/nektos/act/pkg/common"
@@ -17,7 +17,13 @@
 )
 
 func newLocalReusableWorkflowExecutor(rc *RunContext) common.Executor {
-	return newReusableWorkflowExecutor(rc, rc.Config.Workdir, rc.Run.Job().Uses)
+	job := rc.Run.Job()
+	if strings.Index(job.Uses, "./") == 0 {
+		// relative path
+		return newReusableWorkflowExecutor(rc, rc.Run.Workflow.RepoPath, job.Uses)
+	} else {
+		return newReusableWorkflowExecutor(rc, rc.Config.Workdir, job.Uses)
+	}
 }
 
 func newRemoteReusableWorkflowExecutor(rc *RunContext) common.Executor {
@@ -114,7 +120,7 @@
 
 func newReusableWorkflowExecutor(rc *RunContext, directory string, workflow string) common.Executor {
 	return func(ctx context.Context) error {
-		planner, err := model.NewWorkflowPlanner(path.Join(directory, workflow), true)
+		planner, err := model.NewWorkflowPlanner(directory, workflow, true)
 		if err != nil {
 			return err
 		}

pkg/runner/runner_test.go

@@ -50,7 +50,7 @@ func init() {
 }
 
 func TestNoWorkflowsFoundByPlanner(t *testing.T) {
-	planner, err := model.NewWorkflowPlanner("res", true)
+	planner, err := model.NewWorkflowPlanner("res", "", true)
 	assert.NoError(t, err)
 
 	out := log.StandardLogger().Out
@@ -70,7 +70,7 @@ func TestNoWorkflowsFoundByPlanner(t *testing.T) {
 }
 
 func TestGraphMissingEvent(t *testing.T) {
-	planner, err := model.NewWorkflowPlanner("testdata/issue-1595/no-event.yml", true)
+	planner, err := model.NewWorkflowPlanner("testdata/issue-1595", "no-event.yml", true)
 	assert.NoError(t, err)
 
 	out := log.StandardLogger().Out
@@ -88,7 +88,7 @@ func TestGraphMissingEvent(t *testing.T) {
 }
 
 func TestGraphMissingFirst(t *testing.T) {
-	planner, err := model.NewWorkflowPlanner("testdata/issue-1595/no-first.yml", true)
+	planner, err := model.NewWorkflowPlanner("testdata/issue-1595", "no-first.yml", true)
 	assert.NoError(t, err)
 
 	plan, err := planner.PlanEvent("push")
@@ -98,7 +98,7 @@ func TestGraphMissingFirst(t *testing.T) {
 }
 
 func TestGraphWithMissing(t *testing.T) {
-	planner, err := model.NewWorkflowPlanner("testdata/issue-1595/missing.yml", true)
+	planner, err := model.NewWorkflowPlanner("testdata/issue-1595", "missing.yml", true)
 	assert.NoError(t, err)
 
 	out := log.StandardLogger().Out
@@ -117,7 +117,7 @@ func TestGraphWithMissing(t *testing.T) {
 func TestGraphWithSomeMissing(t *testing.T) {
 	log.SetLevel(log.DebugLevel)
 
-	planner, err := model.NewWorkflowPlanner("testdata/issue-1595/", true)
+	planner, err := model.NewWorkflowPlanner("testdata/issue-1595", "", true)
 	assert.NoError(t, err)
 
 	out := log.StandardLogger().Out
@@ -135,7 +135,7 @@ func TestGraphWithSomeMissing(t *testing.T) {
 }
 
 func TestGraphEvent(t *testing.T) {
-	planner, err := model.NewWorkflowPlanner("testdata/basic", true)
+	planner, err := model.NewWorkflowPlanner("testdata/basic", "", true)
 	assert.NoError(t, err)
 
 	plan, err := planner.PlanEvent("push")
@@ -192,7 +192,7 @@ func (j *TestJobFileInfo) runTest(ctx context.Context, t *testing.T, cfg *Config
 	runner, err := New(runnerConfig)
 	assert.Nil(t, err, j.workflowPath)
 
-	planner, err := model.NewWorkflowPlanner(fullWorkflowPath, true)
+	planner, err := model.NewWorkflowPlanner(workdir, j.workflowPath, true)
 	assert.Nil(t, err, fullWorkflowPath)
 
 	plan, err := planner.PlanEvent(j.eventName)
@@ -242,6 +242,7 @@ func TestRunEvent(t *testing.T) {
 		{workdir, "uses-workflow", "pull_request", "", platforms, map[string]string{"secret": "keep_it_private"}},
 		{workdir, "uses-docker-url", "push", "", platforms, secrets},
 		{workdir, "act-composite-env-test", "push", "", platforms, secrets},
+		{workdir, "issue-1875-uses-workflow-with-sub-workflow", "dispatch", "", platforms, secrets},
 
 		// Eval
 		{workdir, "evalmatrix", "push", "", platforms, secrets},

pkg/runner/testdata/issue-1875-uses-workflow-with-sub-workflow/caller-workflow.yml

@@ -0,0 +1,7 @@
+name: Call Reusable Workflow
+on:
+  push:
+
+jobs:
+  call-reusable-workflow-with-sub:
+    uses: wildsheepz/TestReusableWorkflow/.github/workflows/main.yaml@ubuntu_platform