feat(agent): bundle multi-pwsh with Windows installer - #1904
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Let maintainers know that an action is required on their side
|
There was a problem hiding this comment.
Pull request overview
Bundles multi-pwsh with Windows Agent MSI packages.
Changes:
- Adds architecture-specific
multi-pwshdownloading. - Integrates the executable into CI and release packaging.
- Documents local MSI build setup.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.agents/skills/build-agent-msi/SKILL.md |
Documents download and environment setup. |
.github/workflows/ci.yml |
Downloads multi-pwsh for Windows builds. |
.github/workflows/package.yml |
Supplies it during release packaging. |
ci/download-multi-pwsh.ps1 |
Downloads and extracts release assets. |
package/AgentWindowsManaged/Program.cs |
Includes the executable in the MSI. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Pass GITHUB_TOKEN to multi-pwsh download steps, stop on script errors, and add a DEBUG default path for local MSI builds. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This pull request adds support for downloading and integrating the
multi-pwshexecutable into the Windows agent build and packaging process. The changes automate fetching the correctmulti-pwshbinary for the target architecture, ensure the executable is available as a build artifact, and update the MSI packaging to include it.Integration of multi-pwsh into build and packaging:
ci/download-multi-pwsh.ps1to download and extract the appropriatemulti-pwshbinary for the specified architecture and destination..github/workflows/ci.ymland.github/workflows/package.ymlto run the new script, set theDAGENT_MULTI_PWSH_EXECUTABLEenvironment variable, and ensure the executable is available during packaging. [1] [2] [3] [4] [5]Documentation updates:
.agents/skills/build-agent-msi/SKILL.mdto include the new step for downloadingmulti-pwshand to document the new environment variable. [1] [2]MSI packaging changes:
package/AgentWindowsManaged/Program.csto resolve and include themulti-pwsh.exebinary as part of the agent MSI package. [1] [2]