Skip to content

docs(claude): tell agents to let heavy gates queue for machine-wide slots - #37057

Merged
mateo-berri merged 4 commits into
litellm_internal_stagingfrom
litellm_claude_md_gate_slot_locks
Aug 15, 2026
Merged

docs(claude): tell agents to let heavy gates queue for machine-wide slots#37057
mateo-berri merged 4 commits into
litellm_internal_stagingfrom
litellm_claude_md_gate_slot_locks

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

TLDR

Problem this solves:

  • Agents don't know heavy gates queue machine-wide
  • A quiet queueing make check looks like a hang
  • So agents kill, retry, or disable the queue
  • Cheap make bootstrap waited behind heavy gates

How it solves it:

  • CLAUDE.md now names the slot queue and its message
  • Says to wait it out with a long timeout
  • Bans LITELLM_GATE_SLOTS=0 and raising the count
  • bootstrap no longer takes a slot, being I/O bound

User Flow

Docs and local-tooling change for contributors in this repo, so no end user flow changes

Relevant issues

Linear ticket

Pre-Submission checklist

  • I have added meaningful tests
  • My PR passes all CI/CD checks (e.g., lint, format, unit tests)
  • My PR's scope is as isolated as possible; it only solves 1 specific problem
  • I have received a Greptile Confidence Score of at least 4/5 before requesting a maintainer review (Greptile reviews automatically once the PR is opened; only comment @greptileai to re-request a review after pushing changes)

Delays in PR merge?

If you're seeing a delay in your PR being merged, ping the LiteLLM Team on Slack (#pr-review).

Screenshots / Proof of Fix

No proxy code paths change. make -n bootstrap now expands straight to the provisioning commands with no gate_slot_lock.py wrapper, while make -n check still starts with python3 scripts/gate_slot_lock.py make check-inner. The lock behavior itself ships in #36988 and is covered by tests/test_litellm/test_gate_slot_lock.py

Type

📖 Documentation

Caveats (if any)

  • Prose rule, so nothing enforces it in CI
  • check-inner / lint-inner still work, just discouraged
  • bootstrap under make check was already unqueued

Link to Devin session: https://app.devin.ai/sessions/13e9cbbb61e742879faed4d9a1f6c19b
Requested by: @mateo-berri

…lots

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@mateo-berri mateo-berri self-assigned this Aug 15, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@greptile-apps

greptile-apps Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR clarifies how machine-wide slots queue heavy contributor checks and removes the I/O-bound bootstrap target from slot management.

  • Documents the queue message and instructs agents to wait rather than bypass slot limits
  • Runs direct make bootstrap provisioning without the slot-lock wrapper
  • Updates Makefile help and slot-lock documentation to list only queued targets

Confidence Score: 5/5

The PR appears safe to merge

No blocking failure remains

Important Files Changed

Filename Overview
CLAUDE.md Adds accurate agent guidance for queued heavy gates; the previously reported bypass-list issue is no longer present
Makefile Makes direct bootstrap execution unqueued while preserving slot locking for check and lint
scripts/gate_slot_lock.py Updates the module documentation to reflect that bootstrap no longer acquires a machine-wide slot

Reviews (4): Last reviewed commit: "chore: make it more concise" | Re-trigger Greptile

Comment thread CLAUDE.md Outdated
Comment thread CLAUDE.md Outdated
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

@greptileai

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

@greptileai

@codecov

codecov Bot commented Aug 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mateo-berri

Copy link
Copy Markdown
Contributor

bugbot run

@cursor

cursor Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

@mateo-berri
mateo-berri enabled auto-merge August 15, 2026 23:38
@mateo-berri

Copy link
Copy Markdown
Contributor

@greptileai

@mateo-berri mateo-berri 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.

LGTM

@mateo-berri
mateo-berri merged commit 26113da into litellm_internal_staging Aug 15, 2026
66 checks passed
@mateo-berri
mateo-berri deleted the litellm_claude_md_gate_slot_locks branch August 15, 2026 23:55
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.

1 participant