.coderabbit.yaml (5)

95-101: Fixture example uses @fixture without import; prefer @pytest.fixture for clarity.

As written, it may confuse contributors unless they import fixture explicitly. Recommend using the canonical decorator.

Apply this diff:

-        - Pytest fixtures should define the name argument when applying the pytest.fixture decorator. The fixture
-          function being decorated should be named using the `fixture_` prefix, using snake_case. Example:
-            @fixture(name="my_fixture")
+        - Pytest fixtures should define the `name` argument when applying the `@pytest.fixture` decorator. The fixture
+          function being decorated should be named using the `fixture_` prefix, using snake_case. Example:
+            @pytest.fixture(name="my_fixture")
             def fixture_my_fixture():
                 pass

---

131-135: “external” label rule hard-codes a team; use author_association to avoid org/team drift.

Team names and membership change; GitHub exposes author_association (OWNER, MEMBER, COLLABORATOR, CONTRIBUTOR, FIRST_TIMER, NONE). Suggest basing guidance on that.

Apply this diff:

-    - label: external
-      instructions: >-
-        Apply this label when the PR was submitted by someone who is not a member of the @NVIDIA/agentiq-developers
-        GitHub team.
+    - label: external
+      instructions: >-
+        Apply this label when the PR author is not an organization OWNER or MEMBER (i.e., author_association is
+        COLLABORATOR, CONTRIBUTOR, FIRST_TIMER, FIRST_TIME_CONTRIBUTOR, or NONE).

---

33-33: Consider enabling incremental reviews to reduce noise on subsequent pushes.

auto_incremental_review: false disables incremental passes; enabling it usually shortens feedback cycles on active PRs.

Apply this diff:

-    auto_incremental_review: false
+    auto_incremental_review: true

---

25-25: Tweak title guidance grammar and be explicit about length.

Small clarity improvement; no behavior change.

Apply this diff:

-      requirements: "Title should be concise, and descriptive, and use imperative mood (max ~72 chars)."
+      requirements: "Title should be concise, descriptive, and use imperative mood (≤72 characters)."

---

8-8: Prefer HTTPS for the Apache 2.0 license URL.

Minor but standardizes on TLS links.

Apply this diff:

-# http://www.apache.org/licenses/LICENSE-2.0
+# https://www.apache.org/licenses/LICENSE-2.0

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.