forked from nmstate-archive/nmstate-console-plugin
-
Notifications
You must be signed in to change notification settings - Fork 22
Expand file tree
/
Copy path.coderabbit.yaml
More file actions
56 lines (54 loc) · 2.42 KB
/
Copy path.coderabbit.yaml
File metadata and controls
56 lines (54 loc) · 2.42 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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.yaml
language: en-US
tone_instructions: >
Be concise and direct. Focus on correctness, SDK compatibility, and PatternFly
best practices. This is an OpenShift console dynamic plugin — flag any direct
K8s API calls, missing i18n (must use useNMStateTranslation hook or Trans
component), or incorrect extension registration.
reviews:
request_changes_workflow: false
high_level_summary: true
poem: false
review_status: true
collapse_walkthrough: true
path_instructions:
- path: "src/views/**/manifest.ts"
instructions: >
These files register plugin extensions with the OpenShift console.
Verify that EncodedExtension types match the extension type string,
ExposedModules paths resolve to real files, and nav items include
both admin and virtualization perspectives where appropriate.
- path: "src/console-models/**"
instructions: >
K8s model definitions. Verify apiGroup, apiVersion, kind, and plural
are correct for the CRD. Ensure GroupVersionKind and Ref exports exist
and the model is re-exported from index.ts.
- path: "src/views/**/*.tsx"
instructions: >
React components must use PatternFly 6 components and PF utility classes,
functional components with FC typing, and useNMStateTranslation() for
simple user-visible strings or the Trans component for strings with
embedded HTML. No hardcoded English text in JSX. Each file should contain
only a single component.
- path: "src/utils/components/**/*.tsx"
instructions: >
Shared React components must use PatternFly 6 components and PF utility
classes, functional components with FC typing, and useNMStateTranslation()
for simple user-visible strings or the Trans component for strings with
embedded HTML. No hardcoded English text in JSX. Each file should contain
only a single component.
- path: "src/utils/resources/**"
instructions: >
Resource utilities must use SDK hooks (useK8sWatchResource) and SDK
mutation functions (k8sCreate, k8sPatch, k8sDelete). No direct fetch
calls to the K8s API.
path_filters:
- "!package-lock.json"
- "!locales/**"
- "!src/nmstate-types/crds/**"
- "!dist/**"
- "!node_modules/**"
- "!i18n-scripts/**"
auto_review:
enabled: true
drafts: false