Skip to content

fix(ci): Disable sparse-checkout cone mode for Claude context files#17713

Closed
pratikpugalia wants to merge 1 commit into
facebookincubator:mainfrom
pratikpugalia:fix/ci-failure-comment-sparse-checkout
Closed

fix(ci): Disable sparse-checkout cone mode for Claude context files#17713
pratikpugalia wants to merge 1 commit into
facebookincubator:mainfrom
pratikpugalia:fix/ci-failure-comment-sparse-checkout

Conversation

@pratikpugalia
Copy link
Copy Markdown
Contributor

@pratikpugalia pratikpugalia commented Jun 3, 2026

Summary

The CI Failure Comment workflow has been failing on every run since git 2.54 landed on ubuntu-latest runners (around May 20). The Checkout for Claude context and prompt step passes file paths to sparse-checkout:

sparse-checkout: |
  CLAUDE.md
  .claude/CLAUDE.md
  .claude/skills/ci-failure-analysis/SKILL.md

actions/checkout enables cone mode by default, and git 2.54 hard-rejects non-directory entries in cone mode:

fatal: '.claude/CLAUDE.md' is not a directory; to treat it as a directory anyway, rerun with --skip-checks

Earlier git versions on the runner (e.g. 2.53.0 on May 15 runs) accepted file paths in cone mode, which is why the same YAML used to work.

This change sets sparse-checkout-cone-mode: false on the affected step so file paths are accepted. The earlier .github/scripts checkout stays in cone mode — it is a directory and works fine.

The "Checkout for Claude context and prompt" step lists three files in
its sparse-checkout (CLAUDE.md, .claude/CLAUDE.md,
.claude/skills/ci-failure-analysis/SKILL.md). actions/checkout enables
cone mode by default, and git 2.54 (now on ubuntu-latest runners since
~May 20) hard-rejects non-directory entries in cone mode with:

  fatal: '.claude/CLAUDE.md' is not a directory

Disable cone mode on this step so file paths are accepted. The earlier
sparse-checkout of .github/scripts stays in cone mode — it is a
directory and works.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 3, 2026

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 6a99f96
🔍 Latest deploy log https://app.netlify.com/projects/meta-velox/deploys/6a1fe7131137c900085f25df

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 3, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Build Impact Analysis

No build targets affected by this change.


Fast path • Graph from main@67f7deaf023e581bef67301d0c143a7725888b79

@pratikpugalia
Copy link
Copy Markdown
Contributor Author

@kgpai @czentgr Could you take a look at this minimal fix!

@kgpai
Copy link
Copy Markdown
Contributor

kgpai commented Jun 3, 2026

Thanks @pratikpugalia !

@kgpai kgpai added the ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall label Jun 3, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Jun 3, 2026

@kgpai has imported this pull request. If you are a Meta employee, you can view this in D107456949.

@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Jun 4, 2026

@kgpai merged this pull request in 3552f5e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants