forked from open-gsd/gsd-core
-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy path.coderabbit.yaml
More file actions
36 lines (34 loc) · 1.63 KB
/
Copy path.coderabbit.yaml
File metadata and controls
36 lines (34 loc) · 1.63 KB
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
# CodeRabbit configuration — open-gsd/gsd-core
#
# Schema: https://docs.coderabbit.ai/reference/yaml-template/
#
# Project context: GSD ships a CLI tool + an agent runtime, not a documented
# public library. We carry rich JSDoc on internal helpers that warrant it
# (see bin/install.js, gsd-core/bin/lib/*.cjs) but we do not enforce a
# blanket docstring coverage bar — see issue #2932 for rationale.
reviews:
pre_merge_checks:
# Disable docstring coverage check.
#
# The check produces false-positive warnings on PRs whose new code is
# entirely test files: it counts test(...) / beforeEach / afterEach
# arrow-function callbacks as functions and then reports 0% coverage
# because nothing has JSDoc. There is no per-check path filter in CR's
# documented schema that would let us exclude tests/** while keeping
# the check active elsewhere, and the top-level path_filters approach
# would silence ALL CR review on tests (security scans, out-of-scope
# checks, line-level findings) which we want to keep.
#
# All other CR pre-merge checks (out-of-scope, security, title) remain
# at their defaults.
docstrings:
mode: off
tools:
# Disable ESLint. The repo intentionally does not use ESLint — it ships
# its own targeted lint scripts (scripts/lint-no-source-grep.cjs, plus
# the `lint:tests` npm script) that enforce repo-specific test-quality
# invariants. Adding ESLint config purely to satisfy CR would add an
# external dependency (CONTRIBUTING.md: "No external dependencies in
# core") and overlap with the existing custom lint surface.
eslint:
enabled: false