Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 6, 2026

  • Add feature flag constant to Constants.Runner.Features
  • Wrap ACTIONS_ORCHESTRATION_ID setting with feature flag check in NodeScriptActionHandler.cs
  • Format code
  • Build code to validate changes
  • Request code review

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update to address feedback on actions orchestration ID Add feature flag guard for ACTIONS_ORCHESTRATION_ID environment variable Jan 6, 2026
Copilot AI requested a review from TingluoHuang January 6, 2026 17:21
@TingluoHuang
Copy link
Member

@copilot where is the changes?

- Add SetOrchestrationIdEnvForNodeActions feature flag constant
- Wrap ACTIONS_ORCHESTRATION_ID env var setting with feature flag check
- Follows same pattern as ACTIONS_CACHE_SERVICE_V2

Co-authored-by: TingluoHuang <1750815+TingluoHuang@users.noreply.github.com>
@TingluoHuang TingluoHuang marked this pull request as ready for review January 6, 2026 18:13
@TingluoHuang TingluoHuang requested a review from a team as a code owner January 6, 2026 18:13
@TingluoHuang TingluoHuang merged commit 4822c46 into users/tihuang/setorchidenv Jan 6, 2026
1 check passed
@TingluoHuang TingluoHuang deleted the copilot/sub-pr-4178 branch January 6, 2026 18:13
Copy link
Contributor Author

Copilot AI commented Jan 6, 2026

@copilot where is the changes?

The changes have been implemented in commit d63009f. The implementation adds:

  1. A new feature flag constant SetOrchestrationIdEnvForNodeActions in Constants.Runner.Features
  2. A feature flag check wrapping the ACTIONS_ORCHESTRATION_ID environment variable assignment in NodeScriptActionHandler.cs

This follows the project's coding guidelines to safeguard changes with feature flags.

Copilot stopped work on behalf of TingluoHuang due to an error January 6, 2026 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants