Skip to content

Skillet: fix sparse checkout path typing in pre-activation skills checkout - #40684

Merged
pelikhan merged 4 commits into
mainfrom
copilot/fix-skills-sparse-checkout
Jun 21, 2026
Merged

Skillet: fix sparse checkout path typing in pre-activation skills checkout#40684
pelikhan merged 4 commits into
mainfrom
copilot/fix-skills-sparse-checkout

Conversation

CopilotAI commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Skilletworkflow_dispatch runs were failing in Checkout skills directory because sparse-checkout (cone mode) included file paths under .github/workflows as directory-style entries, causing git exit 128 before activation. This change normalizes checkout scope to directory paths so pre-activation can proceed to skill matching/agent startup.

  • Problem scope

    • The pre-activation checkout mixed directory and file entries in sparse-checkout:
      • .github/skills (directory)
      • .github/workflows/agentic_commands.yml (file)
      • .github/workflows/agentic-maintenance.yml (file)
      • actions/setup/js/slash_command_matcher.cjs (file)
    • In cone mode, file entries triggered fatal: '... is not a directory'.
  • Workflow update

    • Updated /.github/workflows/skillet.mdCheckout skills directory to include only directory entries:
      • .github/skills
      • .github/workflows
      • actions/setup/js
    • Removed direct file-path entries for agentic_commands.yml, agentic-maintenance.yml, and slash_command_matcher.cjs.
  • Compiled workflow sync

    • Recompiled /.github/workflows/skillet.lock.yml so runtime YAML reflects the corrected sparse-checkout set.
# pre_activation -> Checkout skills directorywith:
sparse-checkout: | .github/skills .github/workflows actions/setup/jspersist-credentials: false

pr-sous-chef: update branch requested for workflow triage
run: 27916230804

Generated by 👨‍🍳 PR Sous Chef · 47.1 AIC · ⌖ 1.04 AIC · ⊞ 17.3K ·

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
CopilotAI changed the title [WIP] Fix skills sparse-checkout step in SkilletSkillet: fix sparse checkout path typing in pre-activation skills checkoutJun 21, 2026
CopilotAI requested a review from pelikhanJune 21, 2026 19:42
@pelikhan
pelikhan marked this pull request as ready for review June 21, 2026 19:42
CopilotAI review requested due to automatic review settings June 21, 2026 19:43

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Skillet workflow’s pre-activation sparse checkout configuration to avoid git sparse-checkout failures in cone mode by replacing workflow file entries under .github/workflows/ with the directory path, and then syncing the compiled workflow output.

Changes:

  • Adjusted Skillet’s pre-activation actions/checkoutsparse-checkout list to include .github/workflows (directory) instead of specific workflow files.
  • Recompiled skillet.lock.yml so the runtime workflow YAML reflects the updated sparse-checkout set.
Show a summary per file
FileDescription
.github/workflows/skillet.mdUpdates pre-activation sparse-checkout paths used before skill matching/agent startup.
.github/workflows/skillet.lock.ymlSyncs the compiled workflow YAML with the updated sparse-checkout configuration.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 1

Comment on lines 45 to 49
sparse-checkout: |
.github/skills
.github/workflows/agentic_commands.yml
.github/workflows/agentic-maintenance.yml
.github/workflows
actions/setup/js/slash_command_matcher.cjs
persist-credentials: false
Comment thread.github/workflows/skillet.md Outdated
.github/workflows/agentic_commands.yml
.github/workflows/agentic-maintenance.yml
.github/workflows
actions/setup/js/slash_command_matcher.cjs

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot this file

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated this in acd8fb5: the sparse-checkout entry now uses the directory path actions/setup/js (cone-mode compatible), and the compiled skillet.lock.yml is synced.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan
pelikhan merged commit 6663925 into mainJun 21, 2026
18 checks passed
@pelikhan
pelikhan deleted the copilot/fix-skills-sparse-checkout branch June 21, 2026 20:27
github-actionsBot added a commit that referenced this pull request Jun 22, 2026
Weekly update covering the week of June 15–22, 2026:
- Compiler +320% performance regression fix (#40662)
- New deferinloop Go linter (#40679)
- gh-aw-detection rollout to 50% of workflows (#40698)
- JSON-RPC error handling fix (#40715)
- Skillet sparse checkout path fix (#40684)
- FNV-1a heredoc delimiter generation (#40696)
- Agent of the Week: delight ✨
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@pelikhan