no embedding headers

Author: divyegala

.coderabbit.yaml

@@ -0,0 +1,77 @@
+# SPDX-FileCopyrightText: Copyright (c) 2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# SPDX-License-Identifier: Apache-2.0
+
+reviews:
+  profile: chill
+  high_level_summary: true
+  high_level_summary_in_walkthrough: true
+  poem: false
+  auto_review:
+    enabled: true
+    drafts: false
+    base_branches:
+      - "^main$"
+      - "^release/.*"
+      - "^hotfix/.*"
+    ignore_usernames: ["rapids-bot", "GPUtester", "nv-automation-bot", "copy-pr-bot"]
+  tools:
+    markdownlint:
+      enabled: true
+    shellcheck:
+      enabled: true
+    gitleaks:
+      enabled: true
+  sequence_diagrams: false
+  collapse_walkthrough: true
+
+  # Reduce noise from status messages
+  request_changes_workflow: false
+  review_status: false
+
+  # Path-specific review instructions
+  # Note: Detailed C++/CUDA and Python review guidelines are in cpp/REVIEW_GUIDELINES.md and python/REVIEW_GUIDELINES.md
+  path_instructions:
+    - path: "docs/**/*"
+      instructions: |
+        For documentation changes, focus on:
+        - Accuracy: Verify code examples compile and run correctly
+        - Completeness: Check if API changes (parameters, return values, errors) are documented
+        - Clarity: Flag confusing explanations, missing prerequisites, or unclear examples
+        - Consistency: Version numbers, parameter types, and terminology match code
+        - Missing docs: If PR changes public APIs without updating docs, flag as HIGH priority
+
+    - path: "c/include/cuvs/**/*"
+      instructions: |
+        For public C API headers, additionally check:
+        - Doxygen documentation for all public functions/classes
+        - API changes flagged for docs/ updates
+        - Breaking changes require deprecation warnings and migration guide updates
+
+    - path: "cpp/include/cuvs/**/*"
+      instructions: |
+        For public C++ API headers, additionally check:
+        - Doxygen documentation for all public functions/classes
+        - API changes flagged for docs/ updates
+        - Breaking changes require deprecation warnings and migration guide updates
+
+    - path: "notebooks/**/*"
+      instructions: |
+        For example notebooks:
+        - Verify code examples match current API (parameter names, return types)
+        - Flag outdated imports or deprecated API usage
+
+    - path: "ci/**/*"
+      instructions: |
+        For CI/build scripts:
+        - Check for proper conda environment handling
+        - Verify GPU availability checks before tests
+        - Check for proper error handling and meaningful error messages
+
+knowledge_base:
+  opt_out: false
+  code_guidelines:
+    filePatterns:
+      - "cpp/REVIEW_GUIDELINES.md"
+      - "python/REVIEW_GUIDELINES.md"
+      - "docs/source/contributing.md"
+      - "docs/source/developer_guide.md"

.github/CODEOWNERS

@@ -31,6 +31,7 @@ cpp/scripts/run-cmake-format.sh @rapidsai/cuvs-cmake-codeowners
 /.github/                @rapidsai/ci-codeowners
 /ci/                     @rapidsai/ci-codeowners
 /.shellcheckrc           @rapidsai/ci-codeowners
+/.coderabbit.yaml        @rapidsai/ci-codeowners
 
 #packaging code owners
 /.pre-commit-config.yaml @rapidsai/packaging-codeowners

.github/workflows/pr.yaml

@@ -68,6 +68,8 @@ jobs:
       files_yaml: |
         build_docs:
           - '**'
+          - '!**/*/REVIEW_GUIDELINES.md'
+          - '!.coderabbit.yaml'
           - '!.devcontainer/**'
           - '!.github/CODEOWNERS'
           - '!.github/ISSUE_TEMPLATE/**'
@@ -90,6 +92,8 @@ jobs:
           - '!thirdparty/LICENSES/**'
         test_cpp:
           - '**'
+          - '!**/*/REVIEW_GUIDELINES.md'
+          - '!.coderabbit.yaml'
           - '!.devcontainer/**'
           - '!.github/CODEOWNERS'
           - '!.github/ISSUE_TEMPLATE/**'
@@ -124,6 +128,8 @@ jobs:
           - '!thirdparty/LICENSES/**'
         test_java:
           - '**'
+          - '!**/*/REVIEW_GUIDELINES.md'
+          - '!.coderabbit.yaml'
           - '!.devcontainer/**'
           - '!.github/CODEOWNERS'
           - '!.github/ISSUE_TEMPLATE/**'
@@ -157,6 +163,8 @@ jobs:
           - '!thirdparty/LICENSES/**'
         test_python_conda:
           - '**'
+          - '!**/*/REVIEW_GUIDELINES.md'
+          - '!.coderabbit.yaml'
           - '!.devcontainer/**'
           - '!.github/CODEOWNERS'
           - '!.github/ISSUE_TEMPLATE/**'
@@ -194,6 +202,8 @@ jobs:
           - '!thirdparty/LICENSES/**'
         test_python_wheels:
           - '**'
+          - '!**/*/REVIEW_GUIDELINES.md'
+          - '!.coderabbit.yaml'
           - '!.devcontainer/**'
           - '!.github/CODEOWNERS'
           - '!.github/ISSUE_TEMPLATE/**'
@@ -236,6 +246,8 @@ jobs:
           - '!thirdparty/LICENSES/**'
         test_rust:
           - '**'
+          - '!**/*/REVIEW_GUIDELINES.md'
+          - '!.coderabbit.yaml'
           - '!.devcontainer/**'
           - '!.github/CODEOWNERS'
           - '!.github/ISSUE_TEMPLATE/**'
@@ -273,6 +285,8 @@ jobs:
           - '!thirdparty/LICENSES/**'
         test_go:
           - '**'
+          - '!**/*/REVIEW_GUIDELINES.md'
+          - '!.coderabbit.yaml'
           - '!.devcontainer/**'
           - '!.github/CODEOWNERS'
           - '!.github/ISSUE_TEMPLATE/**'

.pre-commit-config.yaml

@@ -43,7 +43,7 @@ repos:
                 additional_dependencies: [toml]
                 args: ["--config=pyproject.toml"]
       - repo: https://github.com/pre-commit/mirrors-clang-format
-        rev: v20.1.4
+        rev: v20.1.8
         hooks:
               - id: clang-format
                 types_or: [c, c++, cuda]
@@ -96,10 +96,14 @@ repos:
                 name: clang-format-with-cmake-placeholders
                 entry: python3 ci/checks/clang_format_with_cmake_placeholders.py
                 language: python
-                additional_dependencies: [clang_format==20.1.4]
+                additional_dependencies: [clang_format==20.1.8]
                 files: |
                   (?x)
                     [.](cpp|cu|hpp|cuh)[.]in$
+                # clang-format struggles with one of the placeholders in register_fatbin.cpp.in, so exclude it
+                exclude: |
+                  (?x)
+                    ^cpp/cmake/modules/register_fatbin[.]cpp[.]in$
       - repo: https://github.com/codespell-project/codespell
         rev: v2.4.1
         hooks:

ci/validate_wheel.sh

@@ -21,11 +21,11 @@ PYDISTCHECK_ARGS=(
 if [[ "${package_dir}" == "python/libcuvs" ]]; then
     if [[ "${RAPIDS_CUDA_MAJOR}" == "12" ]]; then
         PYDISTCHECK_ARGS+=(
-            --max-allowed-size-compressed '800Mi'
+            --max-allowed-size-compressed '400Mi'
         )
     else
         PYDISTCHECK_ARGS+=(
-            --max-allowed-size-compressed '450Mi'
+            --max-allowed-size-compressed '250Mi'
         )
     fi
 elif [[ "${package_dir}" != "python/cuvs" ]]; then

conda/environments/all_cuda-129_arch-aarch64.yaml

@@ -8,8 +8,8 @@ dependencies:
 - _go_select *=cgo
 - breathe>=4.35.0
 - c-compiler
-- clang-tools==20.1.4
-- clang==20.1.4
+- clang-tools==20.1.8
+- clang==20.1.8
 - cmake>=3.30.4
 - cuda-nvcc
 - cuda-nvrtc-dev
@@ -26,7 +26,7 @@ dependencies:
 - go
 - graphviz
 - ipython
-- libclang==20.1.4
+- libclang==20.1.8
 - libcublas-dev
 - libcurand-dev
 - libcusolver-dev

conda/environments/all_cuda-129_arch-x86_64.yaml

@@ -8,8 +8,8 @@ dependencies:
 - _go_select *=cgo
 - breathe>=4.35.0
 - c-compiler
-- clang-tools==20.1.4
-- clang==20.1.4
+- clang-tools==20.1.8
+- clang==20.1.8
 - cmake>=3.30.4
 - cuda-nvcc
 - cuda-nvrtc-dev
@@ -26,7 +26,7 @@ dependencies:
 - go
 - graphviz
 - ipython
-- libclang==20.1.4
+- libclang==20.1.8
 - libcublas-dev
 - libcurand-dev
 - libcusolver-dev

conda/environments/all_cuda-131_arch-aarch64.yaml

@@ -8,8 +8,8 @@ dependencies:
 - _go_select *=cgo
 - breathe>=4.35.0
 - c-compiler
-- clang-tools==20.1.4
-- clang==20.1.4
+- clang-tools==20.1.8
+- clang==20.1.8
 - cmake>=3.30.4
 - cuda-nvcc
 - cuda-nvrtc-dev
@@ -26,7 +26,7 @@ dependencies:
 - go
 - graphviz
 - ipython
-- libclang==20.1.4
+- libclang==20.1.8
 - libcublas-dev
 - libcurand-dev
 - libcusolver-dev

conda/environments/all_cuda-131_arch-x86_64.yaml

@@ -8,8 +8,8 @@ dependencies:
 - _go_select *=cgo
 - breathe>=4.35.0
 - c-compiler
-- clang-tools==20.1.4
-- clang==20.1.4
+- clang-tools==20.1.8
+- clang==20.1.8
 - cmake>=3.30.4
 - cuda-nvcc
 - cuda-nvrtc-dev
@@ -26,7 +26,7 @@ dependencies:
 - go
 - graphviz
 - ipython
-- libclang==20.1.4
+- libclang==20.1.8
 - libcublas-dev
 - libcurand-dev
 - libcusolver-dev

conda/environments/bench_ann_cuda-129_arch-aarch64.yaml

@@ -6,8 +6,8 @@ channels:
 - conda-forge
 dependencies:
 - c-compiler
-- clang-tools==20.1.4
-- clang==20.1.4
+- clang-tools==20.1.8
+- clang==20.1.8
 - click
 - cmake>=3.30.4
 - cuda-nvcc
@@ -24,7 +24,7 @@ dependencies:
 - gcc_linux-aarch64=14.*
 - glog>=0.6.0
 - h5py>=3.8.0
-- libclang==20.1.4
+- libclang==20.1.8
 - libcublas-dev
 - libcurand-dev
 - libcusolver-dev