Replace positional boolean params with named option structs in workflow helpers

[Copilot]

Two workflow helpers relied on consecutive positional booleans, making call sites easy to misread and silently break when argument order changed. This updates those APIs to use named structs so intent is explicit at each call site.

• Copilot step env flags

  • Replaced the buildCopilotStepEnv boolean cluster with a private copilotStepEnvFlags struct.
  • Keeps the helper local/private while making the call site self-describing.

• npm install step options

  • Introduced NPMInstallOptions for GenerateNpmInstallSteps and GenerateNpmInstallStepsWithScope.
  • Replaced the includeNodeSetup / isGlobal / runInstallScripts / cooldownEnabled positional tail with named fields.
  • Avoids boolean transposition bugs and makes future option growth safer.

• Call site cleanup

  • Updated engine and tool install paths to pass explicit named options:
    • Claude
    • Pi
    • behavior-defined engines
    • Playwright CLI
  • Preserves existing behavior while making intent obvious inline.

• Focused test updates

  • Updated existing npm install tests to use the new options struct form, keeping coverage aligned with the new API shape.

Example:

steps:=GenerateNpmInstallStepsWithScope(
"@playwright/cli",
version,
"Install Playwright CLI",
"playwright-cli",
NPMInstallOptions{
IncludeNodeSetup: needsNodeSetup,
IsGlobal: true,
RunInstallScripts: true,
CooldownEnabled: resolveRuntimeCooldown(workflowData, "node"),
},
)

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[pelikhan]

@copilot run pr-finisher skill

[gh-aw-bot]

@copilot please run the pr-finisher skill, address any unresolved review feedback, investigate the failed checks listed on this PR if applicable, and rerun checks once the branch is ready.

Generated by 👨‍🍳 PR Sous Chef · 8.18 AIC · ⌖ 7.29 AIC · ⊞ 7.5K · ◷
_{Comment /souschef to run again}

[Copilot]

@copilot please run the pr-finisher skill, address any unresolved review feedback, investigate the failed checks listed on this PR if applicable, and rerun checks once the branch is ready.

...

Checked PR #46064 at 9c7ef5b. There are no unresolved review threads to address. I re-validated locally with make fmt, make lint, and go test -v ./pkg/workflow/..., all of which passed. The failed checks currently listed on the PR are older automation failures: copilot-pull-request-reviewer is a 400 Bad Request fetching AGENTS.md, and Smoke CI failed in its agent job without evidence implicating this helper-refactor diff. I can’t rerun GitHub checks from this agent; a maintainer will need to re-trigger CI once you want a fresh PR status.

[github-actions]

🎉 This pull request is included in a new release.

Release: v0.82.12