semantic diff review: ast aware hunks, risk scoring, review packs sized for a context window.
1 // Score de risque d'un hunk. Le but n'est pas de noter le style, c'est de
2 // dire a un relecteur ou regarder en premier quand il a peu de contexte.
3
4 import type { Hunk } from './hunks';
5
6 export interface Risk {
7 score: number;
8 reasons: string[];
9 }
10
11 export function scoreHunk(hunk: Hunk): Risk {
12 const reasons: string[] = [];
13 let score = 0;
14
15 // Un changement de comportement sans changement de test est le motif que
16 // les relecteurs, agents comme humains, ratent le plus souvent.
17 if (hunk.touchesPublicApi && !hunk.touchesTests) {
18 score += 40;
19 reasons.push('public behaviour changed, no test delta');
20 }
21
22 if (hunk.touchesAuth) {
23 score += 30;
24 reasons.push('auth surface');
25 }
26
27 // Un lockfile seul ne merite pas de reveiller qui que ce soit.
28 if (hunk.isLockfileOnly) {
29 return { score: 0, reasons: ['lockfile only'] };
30 }
31
32 // Renommer ou commenter n'est pas risque, et gonfler le score sur ces
33 // hunks noie le signal utile.
34 if (hunk.isRenameOnly || hunk.isCommentOnly) {
35 score = Math.min(score, 5);
36 reasons.push('rename or comment only');
37 }
38
39 return { score: Math.min(score, 100), reasons };
40 }