facebookincubator / TTPForge / 36603140004

53%

main: 67%

Pull #643

github

Forward named environment variables to remote steps (#643)

Summary:
Pull Request resolved: https://github.com/facebookincubator/TTPForge/pull/643

## Context
- Remote commands do not inherit the runner's environment. Callers need to forward selected values, such as run identifiers, across SSH without editing each TTP.
- Forwarding is requested through a CLI flag so an older binary rejects unsupported usage before executing the TTP.
- GitHub CI exposed an existing Go version mismatch: `new("baz")` in `pkg/args/choices_test.go` requires Go 1.26, while the module selected Go 1.25.5.
## This Diff
- Add repeatable `--forward-env NAME` for remote `inline:` and `file:` commands.
- Trim and deduplicate names in argument order. Require nonempty names to match `[A-Za-z_][A-Za-z0-9_]*`; reject invalid names before remote command dispatch, even if unset. Skip unset variables and preserve set-but-empty values.
- Pass forwarded values literally, including `$forge.` references. Continue expanding TTP-level and step-level `env`, with forwarded < TTP < step precedence.
- Document the CLI interface, version requirement, validation, and precedence in the remote execution guide and schema reference.
- Cover forwarding and precedence with 14 authenticated loopback SSH cases, 12 invalid-name helper cases, and 6 executor rejection cases across POSIX, PowerShell, and cmd configurations.
- Set the module minimum to Go 1.26.0 and select Go 1.26.5. Have GitHub pre-commit derive its Go version from `go.mod`.

Reviewed By: d0n601

Differential Revision: D120420789

26 of 27 new or added lines in 3 files covered. (96.3%)

3271 of 6225 relevant lines covered (52.55%)

15.1 hits per line