mirror of
https://github.com/huggingface/lerobot.git
synced 2026-05-15 08:39:49 +00:00
Pepijn
7a8b02cd32
CLAUDE.md is CI-only config — moving it to .github/ ensures it is not visible at the repo root when contributors clone lerobot. Both workflows now explicitly reference .github/CLAUDE.md in their prompt/system-prompt. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
50 lines
2.0 KiB
YAML
50 lines
2.0 KiB
YAML
name: Claude Code Review
|
|
|
|
on:
|
|
pull_request:
|
|
types: [opened, synchronize, ready_for_review, reopened]
|
|
|
|
jobs:
|
|
claude-review:
|
|
runs-on: ubuntu-latest
|
|
permissions:
|
|
contents: read
|
|
pull-requests: write
|
|
issues: read
|
|
id-token: write
|
|
actions: read
|
|
env:
|
|
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true
|
|
|
|
steps:
|
|
- name: Checkout repository
|
|
uses: actions/checkout@v4
|
|
with:
|
|
fetch-depth: 1
|
|
persist-credentials: false
|
|
|
|
- name: Run Claude Code Review
|
|
id: claude-review
|
|
uses: anthropics/claude-code-action@26ddc358fe3befff50c5ec2f80304c90c763f6f8 # v1
|
|
with:
|
|
anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
|
|
use_sticky_comment: true
|
|
prompt: |
|
|
Read `.github/CLAUDE.md` for lerobot-specific conventions, then review this PR.
|
|
Provide structured, actionable feedback.
|
|
|
|
Focus areas (in priority order):
|
|
1. **Correctness**: Logic errors, off-by-ones, wrong tensor shapes, incorrect loss functions
|
|
2. **Type safety**: All new/modified Python code must pass `mypy --strict`; check for missing annotations
|
|
3. **Backwards compatibility**: Does this break `LeRobotDataset`, `Policy`, `Robot`, `Teleoperator`, `Env`, or `Processor` public APIs?
|
|
4. **Tests**: New features must have tests; no silent behavioral changes
|
|
5. **Code style**: Explicit over magic, no unnecessary abstractions, no decorative comments
|
|
6. **HF integration**: Dataset streaming, `push_to_hub`, HF Hub compatibility preserved?
|
|
7. **pre-commit**: Would `pre-commit run -a` pass? (ruff, mypy, typos, zizmor)
|
|
|
|
Format findings as P1 (must fix) / P2 (should fix) / P3 (nice to have).
|
|
Skip P3 if the PR is already high quality.
|
|
claude_args: '--model claude-opus-4-6'
|
|
# See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
|
|
# or https://code.claude.com/docs/en/cli-reference for available options
|