kdm-cli/.coderabbit.yaml at main · KDM-cli/kdm-cli · 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
# # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
# language: "en-US"

# tone_instructions: |
#   Senior Engineer & Security Specialist. Review for: 1. Security (no leaks, URL/port validation). 2. Performance (prefer Promise.allSettled). 3. TUI/UX (intuitive CLI flows, handle cancellations). 4. Tests (full vitest coverage for new logic).

# reviews:
#   profile: "assertive"
#   high_level_summary: true
#   auto_review:
#     enabled: true
#     drafts: false

#   path_filters:
#     - "!dist/**"
#     - "!node_modules/**"
#     - "!package-lock.json"
#     - "src/**"
#     - ".github/workflows/**"

#   path_instructions:
#     - path: "src/commands/**/*.ts"
#       instructions: |
#         - Verify that all commands follow the modular registration pattern.
#         - Ensure errors are handled gracefully and logged using the project's logger.
#         - For TUI interactions, verify that `@vr_patel/tui` tools are used correctly.
#     - path: "src/utils/config.ts"
#       instructions: |
#         - Ensure configuration keys are type-safe.
#         - Verify that sensitive information is not stored in plain text if possible.
#     - path: "src/__tests__/**/*.ts"
#       instructions: |
#         - Ensure mocks are clean and shared correctly.
#         - Verify that tests cover both happy paths and error conditions.

#   pre_merge_checks:
#     linked_issue_assessment:
#       mode: "warning"


# updated

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

language: "en-US"

# ✅ Fix: tone_instructions was > 250 characters, causing a parse error
#    and falling back to defaults on every review.
tone_instructions: "Senior Engineer. Review for: security (no leaks, validate inputs), performance (prefer Promise.allSettled), TUI/UX (handle cancellations), and full vitest test coverage."

reviews:
  profile: "assertive"
  high_level_summary: true
  auto_review:
    enabled: true
    drafts: false
  path_filters:
    - "!dist/**"
    - "!node_modules/**"
    - "!package-lock.json"
    - "src/**"
    - ".github/workflows/**"
  path_instructions:
    - path: "src/commands/**/*.ts"
      instructions: |
        - Verify that all commands follow the modular registration pattern.
        - Ensure errors are handled gracefully and logged using the project's logger.
    - path: "src/utils/config.ts"
      instructions: |
        - Ensure configuration keys are type-safe.
        - Verify that sensitive information is not stored in plain text if possible.
    - path: "src/__tests__/**/*.ts"
      instructions: |
        - Ensure mocks are clean and shared correctly.
        - Verify that tests cover both happy paths and error conditions.

pre_merge_checks:
  linked_issue_assessment:
    mode: "warning"