Skip to content

Add 'make recompile' to copilot agent setup workflow#1551

Merged
pelikhan merged 2 commits intomainfrom
copilot/add-make-recompile-workflow
Oct 11, 2025
Merged

Add 'make recompile' to copilot agent setup workflow#1551
pelikhan merged 2 commits intomainfrom
copilot/add-make-recompile-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 11, 2025

Summary

Adds a make recompile step to the .github/workflows/copilot-setup-steps.yml workflow to ensure workflow files are properly compiled during Copilot agent setup.

Background

The copilot-setup-steps.yml workflow was introduced to mirror the CI workflow's build job setup steps, providing Copilot agents with a fully configured development environment. However, it was missing the make recompile step that exists in the CI build job.

Changes

  • Added a "Recompile workflows" step after the "Build code" step in copilot-setup-steps.yml
  • The step runs make recompile with continue-on-error: true to maintain consistency with other setup steps
  • Also includes auto-generated lock file updates from running make recompile (Copilot CLI version bump to 0.0.339 and Claude CLI version bump to 2.0.14, plus MCP schema updates)

Why This Matters

According to AGENTS.md documentation:

  • "ALWAYS RUN RECOMPILE BEFORE COMMITTING CHANGES"
  • "ALWAYS RUN MAKE RECOMPILE TO ENSURE JAVASCRIPT IS PROPERLY FORMATTED"

The make recompile command:

  1. Recompiles all workflow markdown files to their .lock.yml counterparts
  2. Ensures JavaScript files are properly formatted
  3. Validates workflow configurations

By including this step in the setup, Copilot agents will have a development environment that truly mirrors the CI build job, with all workflows properly compiled and ready for agent operations.

Testing

  • ✅ Validated workflow file syntax with actionlint
  • ✅ Verified make build and make recompile run successfully
  • ✅ All unit tests pass
  • ✅ Change follows the same pattern as the CI workflow build job
Original prompt

Add 'make recompile' to copilot agent setup workflow


💡 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.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review October 11, 2025 16:52
Copilot AI changed the title [WIP] Add make recompile to copilot agent setup workflow Add 'make recompile' to copilot agent setup workflow Oct 11, 2025
Copilot AI requested a review from pelikhan October 11, 2025 16:52
@github-actions
Copy link
Copy Markdown
Contributor

Agentic Changeset Generator triggered by this pull request

@pelikhan pelikhan merged commit 62bb606 into main Oct 11, 2025
8 checks passed
@pelikhan pelikhan deleted the copilot/add-make-recompile-workflow branch October 11, 2025 16:52
Copilot AI added a commit that referenced this pull request Mar 23, 2026
Fixes #1551

- Create .github/workflows/shared/pr-code-review-config.md with shared config:
  cache-memory, github toolsets [pull_requests, repos], create-pull-request-review-comment
  side RIGHT, and submit-pull-request-review max 1
- Update grumpy-reviewer.md to import shared component, removing duplicate config
- Update pr-nitpick-reviewer.md similarly
- Update security-review.md similarly
- Regenerate lock files for all three workflows

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/98001808-6936-4837-9236-331ed885a406
pelikhan added a commit that referenced this pull request Mar 23, 2026
* Initial plan

* feat: extract PR Code Review base configuration into shared component

Fixes #1551

- Create .github/workflows/shared/pr-code-review-config.md with shared config:
  cache-memory, github toolsets [pull_requests, repos], create-pull-request-review-comment
  side RIGHT, and submit-pull-request-review max 1
- Update grumpy-reviewer.md to import shared component, removing duplicate config
- Update pr-nitpick-reviewer.md similarly
- Update security-review.md similarly
- Regenerate lock files for all three workflows

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/98001808-6936-4837-9236-331ed885a406

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
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