refactor: prefix step index to step id to keep execution order steps …

…with custom step ids

common/model/step.go

@@ -6,6 +6,7 @@ import "strings"
 //
 // See: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idsteps
 type Step struct {
+ Index string `yaml:"-"` // Index is the index of the step in the job context.
  ID string `yaml:"id,omitempty"` // ID is the unique identifier of the step.
  If string `yaml:"if,omitempty"` // If is the conditional expression to run the step.
  Name string `yaml:"name,omitempty"` // Name is the name of the step.

ghx/src/context/paths.go

@@ -66,7 +66,7 @@ func (c *Context) GetStepRunPath() (string, error) {
  return "", errors.New("no step is set")
  }
 
- return EnsureDir(c.GhxConfig.HomeDir, "run", "jobs", c.Execution.JobRun.Job.ID, "steps", c.Execution.StepRun.Step.ID)
+ return EnsureDir(c.GhxConfig.HomeDir, "run", "jobs", c.Execution.JobRun.Job.ID, "steps", c.Execution.StepRun.Step.Index+"."+c.Execution.StepRun.Step.ID)
 }
 
 // EnsureDir return the joined path and ensures that the directory exists. and returns the joined path.

ghx/src/main.go

@@ -91,8 +91,10 @@ func LoadWorkflow(cfg context.GhxConfig, path string) (model.Workflow, error) {
 
  // update step IDs if not provided
  for ids, step := range job.Steps {
+ step.Index = fmt.Sprintf("%d", ids)
+
  if step.ID == "" {
- step.ID = fmt.Sprintf("%d", ids)
+ step.ID = step.Index
  }
 
  job.Steps[ids] = step