IsaacTeleop/.coderabbit.yaml at main · NVIDIA/IsaacTeleop · GitHub

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
# SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
# SPDX-License-Identifier: Apache-2.0

# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

# CodeRabbit config for Isaac Teleop
# Goal: one review round, critical issues only, no spam.

language: "en-US"

tone_instructions: >-
  Only flag critical bugs, security vulnerabilities, logic errors, and
  concurrency issues. Skip style, formatting, naming, and minor suggestions.
  Be concise and actionable.

reviews:
  profile: "chill"

  # --- Reduce noise: disable decorative / low-value sections ---
  high_level_summary: true
  poem: false
  sequence_diagrams: true
  collapse_walkthrough: true
  changed_files_summary: false
  suggested_labels: false
  suggested_reviewers: false
  related_issues: true
  related_prs: true
  estimate_code_review_effort: true
  in_progress_fortune: false
  assess_linked_issues: true

  # --- One round and done ---
  auto_review:
    enabled: true
    auto_incremental_review: false   # Do NOT re-review on every push
    drafts: false
    ignore_title_keywords:
      - "WIP"
      - "DO NOT MERGE"

  # --- Per-path review focus (tell the reviewer what matters) ---
  path_instructions:
    - path: "**/*.{cpp,h,hpp,cxx,hxx}"
      instructions: >-
        C++20 code. Focus on memory safety, thread safety, resource leaks, and correctness.
        Ignore style, clang-format enforces it at build time.
    - path: "**/*.py"
      instructions: >-
        Python code (bindings, retargeters, examples, scripts). Focus on logic errors and security
        issues only. Ignore formatting and import order, ruff handles those via pre-commit.
    - path: ".github/workflows/**"
      instructions: >-
        CI/CD workflows. Focus on security (secret handling, permissions) and
        correctness of build/test steps.
    - path: "docs/**"
      instructions: "Only flag factual errors, broken links, or spelling mistakes."

  # --- Disable tools that overlap with pre-commit / clang-format ---
  tools:
    ruff:
      enabled: false           # Pre-commit: ruff lint + ruff format
    markdownlint:
      enabled: false           # Low value, noisy on docs
    languagetool:
      enabled: false           # Spelling/grammar on code comments is noise
    biome:
      enabled: false           # Not used in this repo
    eslint:
      enabled: false           # No eslint config in repo
    # Keep these — not covered by pre-commit:
    shellcheck:
      enabled: true
    hadolint:
      enabled: true            # Dockerfile linting
    actionlint:
      enabled: true            # GitHub Actions linting
    cppcheck:
      enabled: true            # C++ static analysis
    github-checks:
      enabled: true
      timeout_ms: 90000

# --- Stop the back-and-forth in PR comment threads ---
chat:
  auto_reply: false            # Only respond when explicitly @mentioned

knowledge_base:
  opt_out: false
  learnings:
    scope: "local"