Skip to content

feat: set GITHUB_COPILOT_INTEGRATION_ID env var for Copilot CLI#24164

Closed
eaftan wants to merge 1 commit intomainfrom
eaftan/integrator-id
Closed

feat: set GITHUB_COPILOT_INTEGRATION_ID env var for Copilot CLI#24164
eaftan wants to merge 1 commit intomainfrom
eaftan/integrator-id

Conversation

@eaftan
Copy link
Copy Markdown
Contributor

@eaftan eaftan commented Apr 2, 2026

When running the Copilot CLI in agentic workflows, set GITHUB_COPILOT_INTEGRATION_ID=agentic-workflows so the CLI knows it is being invoked by gh-aw.

Changes

  • Added CopilotCLIIntegrationIDEnvVar and CopilotCLIIntegrationIDValue constants in pkg/constants/constants.go
  • Set the env var in pkg/workflow/copilot_engine_execution.go for all Copilot engine execution steps
  • Updated golden test files and recompiled all workflow lock files

When running the Copilot CLI in agentic workflows, set
GITHUB_COPILOT_INTEGRATION_ID=agentic-workflows so the CLI
knows it is being invoked by gh-aw.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

Hey @eaftan 👋 — great work on the Copilot CLI integration ID change! Setting GITHUB_COPILOT_INTEGRATION_ID=agentic-workflows so the CLI can identify its caller is a clean, well-scoped improvement, and the implementation follows the existing patterns in pkg/constants/engine_constants.go and pkg/workflow/copilot_engine_execution.go nicely.

Everything here looks solid:

  • ✅ Clear description of what changed and why
  • ✅ Constants properly defined with doc comments in engine_constants.go
  • ✅ Golden test files (basic-copilot.golden, with-imports.golden) updated to reflect the new env var
  • ✅ All 119 lock files recompiled consistently
  • ✅ No new dependencies introduced

This PR looks ready for maintainer review whenever you're happy to mark it out of draft! 🚀

Generated by Contribution Check · ● 2.2M ·

@pelikhan
Copy link
Copy Markdown
Collaborator

pelikhan commented Apr 3, 2026

This will get stale. Will reopen.

@pelikhan pelikhan closed this Apr 3, 2026
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