Update builtins to use step-runner BuiltinContext interface

Georgi N. Georgiev | GitLabrequested to merge
ggeorgiev/update-step-runner-ctx into main

Summary

Companion MR for step-runner!475 (merged) which introduces a BuiltinContext interface to isolate builtin functions from step results.

Updates script_legacy and concrete builtins to accept runner.BuiltinContext instead of *runner.StepsContext:

  • script_legacy.Run — replaced stepsCtx.View().Vars with builtinCtx.GetJobVars()
  • concrete.Run — updated signature
  • concrete/run.New — updated to accept runner.BuiltinContext, replaced stepsCtx.View().Vars with builtinCtx.GetJobVars()

Note: go.mod currently uses a local replace directive pointing to the step-runner branch. Once step-runner!475 (merged) is merged, update the require to the released version and remove the replace.

Test plan

  • go build ./... succeeds
  • go test ./functions/... ./commands/steps/... all pass

Merge request reports