Skip to content

Verify secure Copilot CLI installation rollout is complete#6698

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/rollout-verified-copilot-installation
Closed

Verify secure Copilot CLI installation rollout is complete#6698
Copilot wants to merge 1 commit into
mainfrom
copilot/rollout-verified-copilot-installation

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 17, 2025

Issue #6675 requested rolling out verified Copilot CLI installation to all affected workflows. Investigation reveals this work was already completed in PR #6691 (merged Dec 17, 2025).

Current State

All 74 Copilot engine workflows use the secure installation pattern:

# Download official installer from GitHub copilot-cli repository
curl -fsSL https://raw.githubusercontent.com/github/copilot-cli/main/install.sh -o /tmp/copilot-install.sh

# Execute with version, then cleanup
export VERSION=0.0.369 && sudo bash /tmp/copilot-install.sh
rm -f /tmp/copilot-install.sh
copilot --version

Verification Results:

  • ✅ 74/74 Copilot workflows use secure pattern (100%)
  • ✅ 0 workflows use insecure curl | sudo bash pattern
  • ✅ All 9 workflows mentioned in issue updated (2 occurrences each)
  • ✅ Implementation in pkg/workflow/copilot_engine.go confirmed
  • ✅ All unit tests passing

Recommendation

Close this issue as already completed by PR #6691. No additional changes needed.

Original prompt

This section details on the original issue you should resolve

<issue_title>[plan] Roll out verified Copilot CLI installation to all affected workflows</issue_title>
<issue_description>## Objective

Apply the verified Copilot CLI installation method (developed in the previous task) to all 20+ workflows that currently use unverified script execution.

Context

After developing and testing the secure installation method, this task rolls it out across the repository. Each affected workflow has TWO occurrences of the installation pattern:

  1. Initial installation step
  2. Cleanup/teardown step

Prerequisites

  • Verified installation method tested and documented (from previous sub-issue)
  • List of affected workflows prepared

Known Affected Workflows (Sample)

Based on the scan report, at minimum:

  • smoke-copilot.md (2 locations)
  • daily-news.md (2 locations)
  • dev.md (2 locations)
  • research.md (2 locations)
  • q.md (2 locations)
  • plan.md (2 locations)
  • mergefest.md (2 locations)
  • tidy.md (2 locations)
  • archie.md (2 locations)

Note: Historical data suggests 40+ total occurrences across 27+ workflows.

Implementation Steps

  1. Get complete list of affected workflows using grep:
    grep -r "curl.*copilot-install" .github/workflows/*.md
  2. For each workflow, update BOTH installation locations
  3. Run make recompile after each batch of changes
  4. Test a representative sample of updated workflows
  5. Run full static analysis to verify all warnings resolved

Acceptance Criteria

AI generated by Plan Command for discussion #6670

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Roll out verified Copilot CLI installation to all workflows Verify secure Copilot CLI installation rollout is complete Dec 17, 2025
Copilot AI requested a review from mnkiefer December 17, 2025 05:19
@pelikhan pelikhan closed this Dec 17, 2025
@github-actions github-actions Bot deleted the copilot/rollout-verified-copilot-installation branch April 2, 2026 13:12
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.

[plan] Roll out verified Copilot CLI installation to all affected workflows

3 participants