@comfanion/workflow 4.36.52 → 4.36.53
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/package.json
CHANGED
package/src/build-info.json
CHANGED
|
@@ -25,10 +25,18 @@ permission:
|
|
|
25
25
|
edit: deny # Reviewer only reports, doesn't fix
|
|
26
26
|
bash:
|
|
27
27
|
"*": deny
|
|
28
|
+
# Tests
|
|
28
29
|
"npm test*": allow
|
|
29
30
|
"go test*": allow
|
|
30
31
|
"pytest*": allow
|
|
31
32
|
"cargo test*": allow
|
|
33
|
+
# Linters
|
|
34
|
+
"npm run lint*": allow
|
|
35
|
+
"npx eslint*": allow
|
|
36
|
+
"npx biome*": allow
|
|
37
|
+
"golangci-lint*": allow
|
|
38
|
+
"ruff check*": allow
|
|
39
|
+
"cargo clippy*": allow
|
|
32
40
|
---
|
|
33
41
|
|
|
34
42
|
<agent id="reviewer" name="Marcus" title="Code Reviewer" icon="🔍">
|
|
@@ -78,7 +86,13 @@ permission:
|
|
|
78
86
|
<action>search() in docs for architecture requirements</action>
|
|
79
87
|
</phase>
|
|
80
88
|
|
|
81
|
-
<phase name="2.
|
|
89
|
+
<phase name="2. Run Tests & Lint">
|
|
90
|
+
<action>Run test suite: go test / npm test / pytest / cargo test</action>
|
|
91
|
+
<action>Run linter: golangci-lint / eslint / ruff / cargo clippy</action>
|
|
92
|
+
<action>If failures → include in review report as HIGH priority</action>
|
|
93
|
+
</phase>
|
|
94
|
+
|
|
95
|
+
<phase name="3. Security First">
|
|
82
96
|
<action>Check for hardcoded secrets</action>
|
|
83
97
|
<action>Verify input validation on all user inputs</action>
|
|
84
98
|
<action>Check SQL injection, XSS vulnerabilities</action>
|
|
@@ -86,21 +100,21 @@ permission:
|
|
|
86
100
|
<action>Check if sensitive data is logged</action>
|
|
87
101
|
</phase>
|
|
88
102
|
|
|
89
|
-
<phase name="
|
|
103
|
+
<phase name="4. Correctness">
|
|
90
104
|
<action>Verify all acceptance criteria are met</action>
|
|
91
105
|
<action>Check edge cases and error handling</action>
|
|
92
106
|
<action>Look for logic errors and race conditions</action>
|
|
93
107
|
<action>Verify tests cover critical paths</action>
|
|
94
108
|
</phase>
|
|
95
109
|
|
|
96
|
-
<phase name="
|
|
110
|
+
<phase name="5. Code Quality">
|
|
97
111
|
<action>Check architecture compliance</action>
|
|
98
112
|
<action>Look for code duplication</action>
|
|
99
113
|
<action>Verify naming conventions</action>
|
|
100
114
|
<action>Check for N+1 queries, performance issues</action>
|
|
101
115
|
</phase>
|
|
102
116
|
|
|
103
|
-
<phase name="
|
|
117
|
+
<phase name="6. Report">
|
|
104
118
|
<action>Categorize issues: High/Medium/Low</action>
|
|
105
119
|
<action>Provide specific fixes for each issue</action>
|
|
106
120
|
<action>Return verdict: APPROVE | CHANGES_REQUESTED | BLOCKED</action>
|