sudeshi-code-review 0.0.0-stage → 1.1.0
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/LICENSE +22 -0
- package/README.md +129 -2
- package/dist/ai/provider.d.ts +12 -0
- package/dist/ai/provider.d.ts.map +1 -0
- package/dist/ai/provider.js +2 -0
- package/dist/ai/provider.js.map +1 -0
- package/dist/ai/sarvamProvider.d.ts +13 -0
- package/dist/ai/sarvamProvider.d.ts.map +1 -0
- package/dist/ai/sarvamProvider.js +162 -0
- package/dist/ai/sarvamProvider.js.map +1 -0
- package/dist/analyzer/analyze.d.ts +5 -0
- package/dist/analyzer/analyze.d.ts.map +1 -0
- package/dist/analyzer/analyze.js +30 -0
- package/dist/analyzer/analyze.js.map +1 -0
- package/dist/analyzer/finding.d.ts +20 -0
- package/dist/analyzer/finding.d.ts.map +1 -0
- package/dist/analyzer/finding.js +2 -0
- package/dist/analyzer/finding.js.map +1 -0
- package/dist/analyzer/rule.d.ts +4 -0
- package/dist/analyzer/rule.d.ts.map +1 -0
- package/dist/analyzer/rule.js +2 -0
- package/dist/analyzer/rule.js.map +1 -0
- package/dist/analyzer/rules/dangerousHtml.d.ts +3 -0
- package/dist/analyzer/rules/dangerousHtml.d.ts.map +1 -0
- package/dist/analyzer/rules/dangerousHtml.js +130 -0
- package/dist/analyzer/rules/dangerousHtml.js.map +1 -0
- package/dist/analyzer/rules/hardcodedSecret.d.ts +3 -0
- package/dist/analyzer/rules/hardcodedSecret.d.ts.map +1 -0
- package/dist/analyzer/rules/hardcodedSecret.js +75 -0
- package/dist/analyzer/rules/hardcodedSecret.js.map +1 -0
- package/dist/analyzer/rules/unsafeLink.d.ts +3 -0
- package/dist/analyzer/rules/unsafeLink.d.ts.map +1 -0
- package/dist/analyzer/rules/unsafeLink.js +68 -0
- package/dist/analyzer/rules/unsafeLink.js.map +1 -0
- package/dist/analyzer/sourceFile.d.ts +5 -0
- package/dist/analyzer/sourceFile.d.ts.map +1 -0
- package/dist/analyzer/sourceFile.js +2 -0
- package/dist/analyzer/sourceFile.js.map +1 -0
- package/dist/cli.d.ts +12 -0
- package/dist/cli.d.ts.map +1 -0
- package/dist/cli.js +134 -0
- package/dist/cli.js.map +1 -0
- package/dist/context/buildContext.d.ts +11 -0
- package/dist/context/buildContext.d.ts.map +1 -0
- package/dist/context/buildContext.js +17 -0
- package/dist/context/buildContext.js.map +1 -0
- package/dist/index.d.ts +3 -0
- package/dist/index.d.ts.map +1 -0
- package/dist/index.js +4 -0
- package/dist/index.js.map +1 -0
- package/dist/reporting/jsonReporter.d.ts +3 -0
- package/dist/reporting/jsonReporter.d.ts.map +1 -0
- package/dist/reporting/jsonReporter.js +4 -0
- package/dist/reporting/jsonReporter.js.map +1 -0
- package/dist/reporting/markdownReporter.d.ts +3 -0
- package/dist/reporting/markdownReporter.d.ts.map +1 -0
- package/dist/reporting/markdownReporter.js +40 -0
- package/dist/reporting/markdownReporter.js.map +1 -0
- package/dist/reporting/terminalReporter.d.ts +3 -0
- package/dist/reporting/terminalReporter.d.ts.map +1 -0
- package/dist/reporting/terminalReporter.js +30 -0
- package/dist/reporting/terminalReporter.js.map +1 -0
- package/dist/review/reviewEngine.d.ts +24 -0
- package/dist/review/reviewEngine.d.ts.map +1 -0
- package/dist/review/reviewEngine.js +116 -0
- package/dist/review/reviewEngine.js.map +1 -0
- package/dist/scanner/scanRepository.d.ts +2 -0
- package/dist/scanner/scanRepository.d.ts.map +1 -0
- package/dist/scanner/scanRepository.js +55 -0
- package/dist/scanner/scanRepository.js.map +1 -0
- package/package.json +47 -4
|
@@ -0,0 +1,75 @@
|
|
|
1
|
+
import ts from 'typescript';
|
|
2
|
+
const SECRET_NAME_PATTERN = /^(apiKey|api_key|paymentApiKey|accessToken|access_token|internalToken|internal_token|databaseUrl|database_url|password|secret|clientSecret|client_secret|token)$/i;
|
|
3
|
+
function isStaticSecretString(node) {
|
|
4
|
+
return ts.isStringLiteral(node) || ts.isNoSubstitutionTemplateLiteral(node);
|
|
5
|
+
}
|
|
6
|
+
function redactSecretEvidence(line, value, literalText) {
|
|
7
|
+
if (!value) {
|
|
8
|
+
return line;
|
|
9
|
+
}
|
|
10
|
+
if (literalText) {
|
|
11
|
+
return line.replace(literalText, literalText[0] + '[REDACTED]' + literalText[0]);
|
|
12
|
+
}
|
|
13
|
+
return line.replace(value, '[REDACTED]');
|
|
14
|
+
}
|
|
15
|
+
export const hardcodedSecretRule = (file) => {
|
|
16
|
+
const findings = [];
|
|
17
|
+
const sourceFile = ts.createSourceFile(file.path, file.content, ts.ScriptTarget.Latest, true, file.path.endsWith('.tsx') ? ts.ScriptKind.TSX : ts.ScriptKind.TS);
|
|
18
|
+
function visit(node) {
|
|
19
|
+
if (ts.isVariableDeclaration(node) &&
|
|
20
|
+
ts.isIdentifier(node.name) &&
|
|
21
|
+
SECRET_NAME_PATTERN.test(node.name.text) &&
|
|
22
|
+
node.initializer &&
|
|
23
|
+
isStaticSecretString(node.initializer)) {
|
|
24
|
+
const { line } = sourceFile.getLineAndCharacterOfPosition(node.getStart(sourceFile));
|
|
25
|
+
const value = node.initializer.text;
|
|
26
|
+
if (value) {
|
|
27
|
+
findings.push({
|
|
28
|
+
id: 'hardcoded-secret',
|
|
29
|
+
category: 'security',
|
|
30
|
+
severity: 'high',
|
|
31
|
+
confidence: 1,
|
|
32
|
+
file: file.path,
|
|
33
|
+
line: line + 1,
|
|
34
|
+
rule: 'hardcoded-secret',
|
|
35
|
+
message: 'A hardcoded secret may be present',
|
|
36
|
+
evidence: redactSecretEvidence(file.content.split('\n')[line]?.trim() ?? '', value, node.initializer.getText(sourceFile)),
|
|
37
|
+
requiresAIReview: true,
|
|
38
|
+
});
|
|
39
|
+
}
|
|
40
|
+
}
|
|
41
|
+
if (ts.isPropertyAssignment(node) &&
|
|
42
|
+
ts.isIdentifier(node.name) &&
|
|
43
|
+
SECRET_NAME_PATTERN.test(node.name.text) &&
|
|
44
|
+
isStaticSecretString(node.initializer)) {
|
|
45
|
+
const propertyName = node.name.text;
|
|
46
|
+
const value = node.initializer.text;
|
|
47
|
+
if (!value) {
|
|
48
|
+
ts.forEachChild(node, visit);
|
|
49
|
+
return;
|
|
50
|
+
}
|
|
51
|
+
if (/^(databaseUrl|database_url)$/i.test(propertyName) &&
|
|
52
|
+
!/^[a-z][a-z0-9+.-]*:\/\/[^/@:]+:[^/@]+@/i.test(value)) {
|
|
53
|
+
ts.forEachChild(node, visit);
|
|
54
|
+
return;
|
|
55
|
+
}
|
|
56
|
+
const { line } = sourceFile.getLineAndCharacterOfPosition(node.getStart(sourceFile));
|
|
57
|
+
findings.push({
|
|
58
|
+
id: 'hardcoded-secret',
|
|
59
|
+
category: 'security',
|
|
60
|
+
severity: 'high',
|
|
61
|
+
confidence: 1,
|
|
62
|
+
file: file.path,
|
|
63
|
+
line: line + 1,
|
|
64
|
+
rule: 'hardcoded-secret',
|
|
65
|
+
message: 'A hardcoded secret may be present',
|
|
66
|
+
evidence: redactSecretEvidence(file.content.split('\n')[line]?.trim() ?? '', value, node.initializer.getText(sourceFile)),
|
|
67
|
+
requiresAIReview: true,
|
|
68
|
+
});
|
|
69
|
+
}
|
|
70
|
+
ts.forEachChild(node, visit);
|
|
71
|
+
}
|
|
72
|
+
visit(sourceFile);
|
|
73
|
+
return findings;
|
|
74
|
+
};
|
|
75
|
+
//# sourceMappingURL=hardcodedSecret.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"hardcodedSecret.js","sourceRoot":"","sources":["../../../src/analyzer/rules/hardcodedSecret.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,YAAY,CAAC;AAI5B,MAAM,mBAAmB,GACvB,mKAAmK,CAAC;AAEtK,SAAS,oBAAoB,CAC3B,IAAmB;IAEnB,OAAO,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,+BAA+B,CAAC,IAAI,CAAC,CAAC;AAC9E,CAAC;AAED,SAAS,oBAAoB,CAC3B,IAAY,EACZ,KAAa,EACb,WAAoB;IAEpB,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,WAAW,EAAE,CAAC;QAChB,OAAO,IAAI,CAAC,OAAO,CACjB,WAAW,EACX,WAAW,CAAC,CAAC,CAAC,GAAG,YAAY,GAAG,WAAW,CAAC,CAAC,CAAC,CAC/C,CAAC;IACJ,CAAC;IAED,OAAO,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,YAAY,CAAC,CAAC;AAC3C,CAAC;AAED,MAAM,CAAC,MAAM,mBAAmB,GAAS,CAAC,IAAI,EAAE,EAAE;IAChD,MAAM,QAAQ,GAAc,EAAE,CAAC;IAC/B,MAAM,UAAU,GAAG,EAAE,CAAC,gBAAgB,CACpC,IAAI,CAAC,IAAI,EACT,IAAI,CAAC,OAAO,EACZ,EAAE,CAAC,YAAY,CAAC,MAAM,EACtB,IAAI,EACJ,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,UAAU,CAAC,EAAE,CAClE,CAAC;IAEF,SAAS,KAAK,CAAC,IAAa;QAC1B,IACE,EAAE,CAAC,qBAAqB,CAAC,IAAI,CAAC;YAC9B,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC;YAC1B,mBAAmB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;YACxC,IAAI,CAAC,WAAW;YAChB,oBAAoB,CAAC,IAAI,CAAC,WAAW,CAAC,EACtC,CAAC;YACD,MAAM,EAAE,IAAI,EAAE,GAAG,UAAU,CAAC,6BAA6B,CACvD,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,CAC1B,CAAC;YACF,MAAM,KAAK,GAAG,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC;YAEpC,IAAI,KAAK,EAAE,CAAC;gBACV,QAAQ,CAAC,IAAI,CAAC;oBACZ,EAAE,EAAE,kBAAkB;oBACtB,QAAQ,EAAE,UAAU;oBACpB,QAAQ,EAAE,MAAM;oBAChB,UAAU,EAAE,CAAC;oBACb,IAAI,EAAE,IAAI,CAAC,IAAI;oBACf,IAAI,EAAE,IAAI,GAAG,CAAC;oBACd,IAAI,EAAE,kBAAkB;oBACxB,OAAO,EAAE,mCAAmC;oBAC5C,QAAQ,EAAE,oBAAoB,CAC5B,IAAI,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,EAC5C,KAAK,EACL,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,UAAU,CAAC,CACrC;oBACD,gBAAgB,EAAE,IAAI;iBACvB,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,IACE,EAAE,CAAC,oBAAoB,CAAC,IAAI,CAAC;YAC7B,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC;YAC1B,mBAAmB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;YACxC,oBAAoB,CAAC,IAAI,CAAC,WAAW,CAAC,EACtC,CAAC;YACD,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;YACpC,MAAM,KAAK,GAAG,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC;YAEpC,IAAI,CAAC,KAAK,EAAE,CAAC;gBACX,EAAE,CAAC,YAAY,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;gBAC7B,OAAO;YACT,CAAC;YAED,IACE,+BAA+B,CAAC,IAAI,CAAC,YAAY,CAAC;gBAClD,CAAC,yCAAyC,CAAC,IAAI,CAAC,KAAK,CAAC,EACtD,CAAC;gBACD,EAAE,CAAC,YAAY,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;gBAC7B,OAAO;YACT,CAAC;YAED,MAAM,EAAE,IAAI,EAAE,GAAG,UAAU,CAAC,6BAA6B,CACvD,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,CAC1B,CAAC;YAEF,QAAQ,CAAC,IAAI,CAAC;gBACZ,EAAE,EAAE,kBAAkB;gBACtB,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,MAAM;gBAChB,UAAU,EAAE,CAAC;gBACb,IAAI,EAAE,IAAI,CAAC,IAAI;gBACf,IAAI,EAAE,IAAI,GAAG,CAAC;gBACd,IAAI,EAAE,kBAAkB;gBACxB,OAAO,EAAE,mCAAmC;gBAC5C,QAAQ,EAAE,oBAAoB,CAC5B,IAAI,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,EAC5C,KAAK,EACL,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,UAAU,CAAC,CACrC;gBACD,gBAAgB,EAAE,IAAI;aACvB,CAAC,CAAC;QACL,CAAC;QAED,EAAE,CAAC,YAAY,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;IAC/B,CAAC;IAED,KAAK,CAAC,UAAU,CAAC,CAAC;IAClB,OAAO,QAAQ,CAAC;AAClB,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"unsafeLink.d.ts","sourceRoot":"","sources":["../../../src/analyzer/rules/unsafeLink.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,YAAY,CAAC;AAEvC,eAAO,MAAM,cAAc,EAAE,IA+F5B,CAAC"}
|
|
@@ -0,0 +1,68 @@
|
|
|
1
|
+
import ts from 'typescript';
|
|
2
|
+
export const unsafeLinkRule = (file) => {
|
|
3
|
+
const findings = [];
|
|
4
|
+
const sourceFile = ts.createSourceFile(file.path, file.content, ts.ScriptTarget.Latest, true, file.path.endsWith('.tsx') ? ts.ScriptKind.TSX : ts.ScriptKind.TS);
|
|
5
|
+
function isDangerousString(node) {
|
|
6
|
+
return ((ts.isStringLiteral(node) || ts.isNoSubstitutionTemplateLiteral(node)) &&
|
|
7
|
+
/^\s*javascript\s*:/i.test(node.text));
|
|
8
|
+
}
|
|
9
|
+
function findVisibleDeclaration(reference) {
|
|
10
|
+
let current = reference.parent;
|
|
11
|
+
while (current) {
|
|
12
|
+
if (ts.isBlock(current) || ts.isSourceFile(current)) {
|
|
13
|
+
for (const statement of current.statements) {
|
|
14
|
+
if (!ts.isVariableStatement(statement))
|
|
15
|
+
continue;
|
|
16
|
+
for (const declaration of statement.declarationList.declarations) {
|
|
17
|
+
if (ts.isIdentifier(declaration.name) &&
|
|
18
|
+
declaration.name.text === reference.text &&
|
|
19
|
+
declaration.initializer &&
|
|
20
|
+
declaration.getStart(sourceFile) < reference.getStart(sourceFile)) {
|
|
21
|
+
return declaration;
|
|
22
|
+
}
|
|
23
|
+
}
|
|
24
|
+
}
|
|
25
|
+
}
|
|
26
|
+
current = current.parent;
|
|
27
|
+
}
|
|
28
|
+
return undefined;
|
|
29
|
+
}
|
|
30
|
+
function visit(node) {
|
|
31
|
+
if (ts.isJsxAttribute(node) &&
|
|
32
|
+
ts.isIdentifier(node.name) &&
|
|
33
|
+
node.name.text === 'href' &&
|
|
34
|
+
node.initializer) {
|
|
35
|
+
let dangerous = false;
|
|
36
|
+
if (ts.isStringLiteral(node.initializer)) {
|
|
37
|
+
dangerous = /^\s*javascript\s*:/i.test(node.initializer.text);
|
|
38
|
+
}
|
|
39
|
+
else if (ts.isJsxExpression(node.initializer)) {
|
|
40
|
+
const expression = node.initializer.expression;
|
|
41
|
+
if (expression && ts.isIdentifier(expression)) {
|
|
42
|
+
const declaration = findVisibleDeclaration(expression);
|
|
43
|
+
dangerous = Boolean(declaration?.initializer &&
|
|
44
|
+
isDangerousString(declaration.initializer));
|
|
45
|
+
}
|
|
46
|
+
}
|
|
47
|
+
if (dangerous) {
|
|
48
|
+
const { line } = sourceFile.getLineAndCharacterOfPosition(node.getStart(sourceFile));
|
|
49
|
+
findings.push({
|
|
50
|
+
id: 'unsafe-link',
|
|
51
|
+
category: 'security',
|
|
52
|
+
severity: 'high',
|
|
53
|
+
confidence: 1,
|
|
54
|
+
file: file.path,
|
|
55
|
+
line: line + 1,
|
|
56
|
+
rule: 'unsafe-link',
|
|
57
|
+
message: 'JavaScript URLs in links can execute code when activated.',
|
|
58
|
+
evidence: file.content.split('\n')[line]?.trim() ?? '',
|
|
59
|
+
requiresAIReview: true,
|
|
60
|
+
});
|
|
61
|
+
}
|
|
62
|
+
}
|
|
63
|
+
ts.forEachChild(node, visit);
|
|
64
|
+
}
|
|
65
|
+
visit(sourceFile);
|
|
66
|
+
return findings;
|
|
67
|
+
};
|
|
68
|
+
//# sourceMappingURL=unsafeLink.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"unsafeLink.js","sourceRoot":"","sources":["../../../src/analyzer/rules/unsafeLink.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,YAAY,CAAC;AAI5B,MAAM,CAAC,MAAM,cAAc,GAAS,CAAC,IAAI,EAAE,EAAE;IAC3C,MAAM,QAAQ,GAAc,EAAE,CAAC;IAE/B,MAAM,UAAU,GAAG,EAAE,CAAC,gBAAgB,CACpC,IAAI,CAAC,IAAI,EACT,IAAI,CAAC,OAAO,EACZ,EAAE,CAAC,YAAY,CAAC,MAAM,EACtB,IAAI,EACJ,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,UAAU,CAAC,EAAE,CAClE,CAAC;IAEF,SAAS,iBAAiB,CAAC,IAAmB;QAC5C,OAAO,CACL,CAAC,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,+BAA+B,CAAC,IAAI,CAAC,CAAC;YACtE,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CACtC,CAAC;IACJ,CAAC;IACD,SAAS,sBAAsB,CAC7B,SAAwB;QAExB,IAAI,OAAO,GAAwB,SAAS,CAAC,MAAM,CAAC;QAEpD,OAAO,OAAO,EAAE,CAAC;YACf,IAAI,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,YAAY,CAAC,OAAO,CAAC,EAAE,CAAC;gBACpD,KAAK,MAAM,SAAS,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;oBAC3C,IAAI,CAAC,EAAE,CAAC,mBAAmB,CAAC,SAAS,CAAC;wBAAE,SAAS;oBAEjD,KAAK,MAAM,WAAW,IAAI,SAAS,CAAC,eAAe,CAAC,YAAY,EAAE,CAAC;wBACjE,IACE,EAAE,CAAC,YAAY,CAAC,WAAW,CAAC,IAAI,CAAC;4BACjC,WAAW,CAAC,IAAI,CAAC,IAAI,KAAK,SAAS,CAAC,IAAI;4BACxC,WAAW,CAAC,WAAW;4BACvB,WAAW,CAAC,QAAQ,CAAC,UAAU,CAAC,GAAG,SAAS,CAAC,QAAQ,CAAC,UAAU,CAAC,EACjE,CAAC;4BACD,OAAO,WAAW,CAAC;wBACrB,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;YAED,OAAO,GAAG,OAAO,CAAC,MAAM,CAAC;QAC3B,CAAC;QAED,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,SAAS,KAAK,CAAC,IAAa;QAC1B,IACE,EAAE,CAAC,cAAc,CAAC,IAAI,CAAC;YACvB,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC;YAC1B,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,MAAM;YACzB,IAAI,CAAC,WAAW,EAChB,CAAC;YACD,IAAI,SAAS,GAAG,KAAK,CAAC;YAEtB,IAAI,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,WAAW,CAAC,EAAE,CAAC;gBACzC,SAAS,GAAG,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC;YAChE,CAAC;iBAAM,IAAI,EAAE,CAAC,eAAe,CAAC,IAAI,CAAC,WAAW,CAAC,EAAE,CAAC;gBAChD,MAAM,UAAU,GAAG,IAAI,CAAC,WAAW,CAAC,UAAU,CAAC;gBAE/C,IAAI,UAAU,IAAI,EAAE,CAAC,YAAY,CAAC,UAAU,CAAC,EAAE,CAAC;oBAC9C,MAAM,WAAW,GAAG,sBAAsB,CAAC,UAAU,CAAC,CAAC;oBAEvD,SAAS,GAAG,OAAO,CACjB,WAAW,EAAE,WAAW;wBACxB,iBAAiB,CAAC,WAAW,CAAC,WAAW,CAAC,CAC3C,CAAC;gBACJ,CAAC;YACH,CAAC;YAED,IAAI,SAAS,EAAE,CAAC;gBACd,MAAM,EAAE,IAAI,EAAE,GAAG,UAAU,CAAC,6BAA6B,CACvD,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,CAC1B,CAAC;gBAEF,QAAQ,CAAC,IAAI,CAAC;oBACZ,EAAE,EAAE,aAAa;oBACjB,QAAQ,EAAE,UAAU;oBACpB,QAAQ,EAAE,MAAM;oBAChB,UAAU,EAAE,CAAC;oBACb,IAAI,EAAE,IAAI,CAAC,IAAI;oBACf,IAAI,EAAE,IAAI,GAAG,CAAC;oBACd,IAAI,EAAE,aAAa;oBACnB,OAAO,EAAE,2DAA2D;oBACpE,QAAQ,EAAE,IAAI,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE;oBACtD,gBAAgB,EAAE,IAAI;iBACvB,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,EAAE,CAAC,YAAY,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;IAC/B,CAAC;IAED,KAAK,CAAC,UAAU,CAAC,CAAC;IAClB,OAAO,QAAQ,CAAC;AAClB,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"sourceFile.d.ts","sourceRoot":"","sources":["../../src/analyzer/sourceFile.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,UAAU,GAAG;IACvB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;CACjB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"sourceFile.js","sourceRoot":"","sources":["../../src/analyzer/sourceFile.ts"],"names":[],"mappings":""}
|
package/dist/cli.d.ts
ADDED
|
@@ -0,0 +1,12 @@
|
|
|
1
|
+
import { scanRepository as defaultScanRepository } from './scanner/scanRepository.js';
|
|
2
|
+
import type { Finding } from './analyzer/finding.js';
|
|
3
|
+
import type { SourceFile } from './analyzer/sourceFile.js';
|
|
4
|
+
import type { ReviewEngine as ReviewEngineType } from './review/reviewEngine.js';
|
|
5
|
+
type CLIDependencies = {
|
|
6
|
+
scanRepository: typeof defaultScanRepository;
|
|
7
|
+
analyze: (files: SourceFile[]) => Finding[];
|
|
8
|
+
reviewEngine?: Pick<ReviewEngineType, 'review'>;
|
|
9
|
+
};
|
|
10
|
+
export declare function runCLI(args: string[], dependencies?: CLIDependencies): Promise<void>;
|
|
11
|
+
export {};
|
|
12
|
+
//# sourceMappingURL=cli.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,cAAc,IAAI,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AACtF,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,uBAAuB,CAAC;AACrD,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,0BAA0B,CAAC;AAK3D,OAAO,KAAK,EAAE,YAAY,IAAI,gBAAgB,EAAE,MAAM,0BAA0B,CAAC;AAQjF,KAAK,eAAe,GAAG;IACrB,cAAc,EAAE,OAAO,qBAAqB,CAAC;IAC7C,OAAO,EAAE,CAAC,KAAK,EAAE,UAAU,EAAE,KAAK,OAAO,EAAE,CAAC;IAC5C,YAAY,CAAC,EAAE,IAAI,CAAC,gBAAgB,EAAE,QAAQ,CAAC,CAAC;CACjD,CAAC;AAQF,wBAAsB,MAAM,CAC1B,IAAI,EAAE,MAAM,EAAE,EACd,YAAY,GAAE,eAQb,GACA,OAAO,CAAC,IAAI,CAAC,CAwIf"}
|
package/dist/cli.js
ADDED
|
@@ -0,0 +1,134 @@
|
|
|
1
|
+
import { existsSync, statSync } from 'node:fs';
|
|
2
|
+
import { readFile, writeFile } from 'node:fs/promises';
|
|
3
|
+
import { resolve } from 'node:path';
|
|
4
|
+
import { scanRepository as defaultScanRepository } from './scanner/scanRepository.js';
|
|
5
|
+
import { analyze as defaultAnalyze } from './analyzer/analyze.js';
|
|
6
|
+
import { dangerousHtmlRule } from './analyzer/rules/dangerousHtml.js';
|
|
7
|
+
import { hardcodedSecretRule } from './analyzer/rules/hardcodedSecret.js';
|
|
8
|
+
import { unsafeLinkRule } from './analyzer/rules/unsafeLink.js';
|
|
9
|
+
import { generateMarkdownReport } from './reporting/markdownReporter.js';
|
|
10
|
+
import { SarvamProvider } from './ai/sarvamProvider.js';
|
|
11
|
+
import { ReviewEngine } from './review/reviewEngine.js';
|
|
12
|
+
import { generateJsonReport } from './reporting/jsonReporter.js';
|
|
13
|
+
import { generateTerminalReport } from './reporting/terminalReporter.js';
|
|
14
|
+
function writeProgress(message) {
|
|
15
|
+
if (process.stderr.isTTY) {
|
|
16
|
+
process.stderr.write(`${message}\n`);
|
|
17
|
+
}
|
|
18
|
+
}
|
|
19
|
+
export async function runCLI(args, dependencies = {
|
|
20
|
+
scanRepository: defaultScanRepository,
|
|
21
|
+
analyze: (files) => defaultAnalyze(files, [
|
|
22
|
+
dangerousHtmlRule,
|
|
23
|
+
hardcodedSecretRule,
|
|
24
|
+
unsafeLinkRule,
|
|
25
|
+
]),
|
|
26
|
+
}) {
|
|
27
|
+
if (args.includes('--help') || args.includes('-h')) {
|
|
28
|
+
process.stdout.write([
|
|
29
|
+
'Sudeshi Code Review',
|
|
30
|
+
'',
|
|
31
|
+
'Usage:',
|
|
32
|
+
' sudeshi-code-review <repository-path>',
|
|
33
|
+
'',
|
|
34
|
+
'Options:',
|
|
35
|
+
' -h, --help Show this help message',
|
|
36
|
+
' -v, --version Show the installed version',
|
|
37
|
+
' --format <format> Report format: markdown or json (default: markdown)',
|
|
38
|
+
' --output <path> Save the report to a file instead of stdout',
|
|
39
|
+
'',
|
|
40
|
+
'Environment:',
|
|
41
|
+
' SARVAM_API_KEY Optional API key for AI-assisted review',
|
|
42
|
+
'',
|
|
43
|
+
].join('\n'));
|
|
44
|
+
return;
|
|
45
|
+
}
|
|
46
|
+
if (args.includes('--version') || args.includes('-v')) {
|
|
47
|
+
const packageJson = JSON.parse(await readFile(new URL('../package.json', import.meta.url), 'utf8'));
|
|
48
|
+
process.stdout.write(`${packageJson.version}\n`);
|
|
49
|
+
return;
|
|
50
|
+
}
|
|
51
|
+
const formatIndex = args.indexOf('--format');
|
|
52
|
+
const format = formatIndex === -1 ? 'markdown' : args[formatIndex + 1];
|
|
53
|
+
if (formatIndex !== -1 && (!format || format.startsWith('--'))) {
|
|
54
|
+
process.stderr.write('Error: --format requires a value.\n');
|
|
55
|
+
process.exit(1);
|
|
56
|
+
return;
|
|
57
|
+
}
|
|
58
|
+
if (format !== 'markdown' && format !== 'json') {
|
|
59
|
+
process.stderr.write('Error: --format must be either markdown or json.\n');
|
|
60
|
+
process.exit(1);
|
|
61
|
+
return;
|
|
62
|
+
}
|
|
63
|
+
const outputIndex = args.indexOf('--output');
|
|
64
|
+
const outputPath = outputIndex === -1 ? undefined : args[outputIndex + 1];
|
|
65
|
+
if (outputIndex !== -1 && (!outputPath || outputPath.startsWith('--'))) {
|
|
66
|
+
process.stderr.write('Error: --output requires a file path.\n');
|
|
67
|
+
process.exit(1);
|
|
68
|
+
return;
|
|
69
|
+
}
|
|
70
|
+
const repositoryPath = args[0];
|
|
71
|
+
if (!repositoryPath) {
|
|
72
|
+
process.stderr.write('Error: repository path is required.\n');
|
|
73
|
+
process.exit(1);
|
|
74
|
+
return;
|
|
75
|
+
}
|
|
76
|
+
const absolutePath = resolve(repositoryPath);
|
|
77
|
+
if (!existsSync(absolutePath)) {
|
|
78
|
+
process.stderr.write('Error: repository path does not exist.\n');
|
|
79
|
+
process.exit(1);
|
|
80
|
+
return;
|
|
81
|
+
}
|
|
82
|
+
if (!statSync(absolutePath).isDirectory()) {
|
|
83
|
+
process.stderr.write('Error: repository path must be a directory.\n');
|
|
84
|
+
process.exit(1);
|
|
85
|
+
return;
|
|
86
|
+
}
|
|
87
|
+
writeProgress('Scanning repository...');
|
|
88
|
+
const filePaths = await dependencies.scanRepository(absolutePath);
|
|
89
|
+
const files = await Promise.all(filePaths.map(async (filePath) => ({
|
|
90
|
+
path: filePath,
|
|
91
|
+
content: await readFile(resolve(absolutePath, filePath), 'utf8'),
|
|
92
|
+
})));
|
|
93
|
+
writeProgress('Analyzing files...');
|
|
94
|
+
const findings = dependencies.analyze(files);
|
|
95
|
+
const apiKey = process.env.SARVAM_API_KEY;
|
|
96
|
+
let reviewEngine = dependencies.reviewEngine;
|
|
97
|
+
if (!reviewEngine && apiKey) {
|
|
98
|
+
writeProgress('Initializing AI review...');
|
|
99
|
+
reviewEngine = new ReviewEngine({
|
|
100
|
+
provider: new SarvamProvider({ apiKey }),
|
|
101
|
+
});
|
|
102
|
+
}
|
|
103
|
+
let result;
|
|
104
|
+
if (reviewEngine) {
|
|
105
|
+
writeProgress('Running AI review...');
|
|
106
|
+
result = await reviewEngine.review({ files, findings });
|
|
107
|
+
}
|
|
108
|
+
else {
|
|
109
|
+
result = {
|
|
110
|
+
findings,
|
|
111
|
+
filesScanned: files.length,
|
|
112
|
+
durationMs: 0,
|
|
113
|
+
};
|
|
114
|
+
}
|
|
115
|
+
writeProgress('Generating report...');
|
|
116
|
+
if (outputPath) {
|
|
117
|
+
const report = format === 'json'
|
|
118
|
+
? generateJsonReport(result)
|
|
119
|
+
: generateMarkdownReport(result);
|
|
120
|
+
await writeFile(resolve(outputPath), `${report}\n`, 'utf8');
|
|
121
|
+
return;
|
|
122
|
+
}
|
|
123
|
+
if (format === 'json') {
|
|
124
|
+
process.stdout.write(`${generateJsonReport(result)}\n`);
|
|
125
|
+
return;
|
|
126
|
+
}
|
|
127
|
+
if (process.stdout.isTTY) {
|
|
128
|
+
process.stdout.write(`${generateTerminalReport(result)}\n`);
|
|
129
|
+
}
|
|
130
|
+
else {
|
|
131
|
+
process.stdout.write(`${generateMarkdownReport(result)}\n`);
|
|
132
|
+
}
|
|
133
|
+
}
|
|
134
|
+
//# sourceMappingURL=cli.js.map
|
package/dist/cli.js.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"cli.js","sourceRoot":"","sources":["../src/cli.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AAC/C,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AACvD,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AACpC,OAAO,EAAE,cAAc,IAAI,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AAGtF,OAAO,EAAE,OAAO,IAAI,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAClE,OAAO,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AACtE,OAAO,EAAE,mBAAmB,EAAE,MAAM,qCAAqC,CAAC;AAC1E,OAAO,EAAE,cAAc,EAAE,MAAM,gCAAgC,CAAC;AAGhE,OAAO,EAAE,sBAAsB,EAAE,MAAM,iCAAiC,CAAC;AACzE,OAAO,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AACxD,OAAO,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AACxD,OAAO,EAAE,kBAAkB,EAAE,MAAM,6BAA6B,CAAC;AACjE,OAAO,EAAE,sBAAsB,EAAE,MAAM,iCAAiC,CAAC;AAQzE,SAAS,aAAa,CAAC,OAAe;IACpC,IAAI,OAAO,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC;QACzB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,OAAO,IAAI,CAAC,CAAC;IACvC,CAAC;AACH,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,MAAM,CAC1B,IAAc,EACd,eAAgC;IAC9B,cAAc,EAAE,qBAAqB;IACrC,OAAO,EAAE,CAAC,KAAK,EAAE,EAAE,CACjB,cAAc,CAAC,KAAK,EAAE;QACpB,iBAAiB;QACjB,mBAAmB;QACnB,cAAc;KACf,CAAC;CACL;IAED,IAAI,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;QACnD,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB;YACE,qBAAqB;YACrB,EAAE;YACF,QAAQ;YACR,yCAAyC;YACzC,EAAE;YACF,UAAU;YACV,2CAA2C;YAC3C,+CAA+C;YAC/C,0EAA0E;YAC1E,kEAAkE;YAClE,EAAE;YACF,cAAc;YACd,4DAA4D;YAC5D,EAAE;SACH,CAAC,IAAI,CAAC,IAAI,CAAC,CACb,CAAC;QACF,OAAO;IACT,CAAC;IAED,IAAI,IAAI,CAAC,QAAQ,CAAC,WAAW,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;QACtD,MAAM,WAAW,GAAG,IAAI,CAAC,KAAK,CAC5B,MAAM,QAAQ,CAAC,IAAI,GAAG,CAAC,iBAAiB,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,MAAM,CAAC,CAC7C,CAAC;QAEzB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,WAAW,CAAC,OAAO,IAAI,CAAC,CAAC;QACjD,OAAO;IACT,CAAC;IAED,MAAM,WAAW,GAAG,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;IAC7C,MAAM,MAAM,GAAG,WAAW,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,WAAW,GAAG,CAAC,CAAC,CAAC;IAEvE,IAAI,WAAW,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,IAAI,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,EAAE,CAAC;QAC/D,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,qCAAqC,CAAC,CAAC;QAC5D,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAChB,OAAO;IACT,CAAC;IAED,IAAI,MAAM,KAAK,UAAU,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;QAC/C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,oDAAoD,CAAC,CAAC;QAC3E,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAChB,OAAO;IACT,CAAC;IAED,MAAM,WAAW,GAAG,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;IAC7C,MAAM,UAAU,GAAG,WAAW,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,WAAW,GAAG,CAAC,CAAC,CAAC;IAE1E,IAAI,WAAW,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,IAAI,UAAU,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,EAAE,CAAC;QACvE,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,yCAAyC,CAAC,CAAC;QAChE,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAChB,OAAO;IACT,CAAC;IAED,MAAM,cAAc,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;IAE/B,IAAI,CAAC,cAAc,EAAE,CAAC;QACpB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,uCAAuC,CAAC,CAAC;QAC9D,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAChB,OAAO;IACT,CAAC;IAED,MAAM,YAAY,GAAG,OAAO,CAAC,cAAc,CAAC,CAAC;IAE7C,IAAI,CAAC,UAAU,CAAC,YAAY,CAAC,EAAE,CAAC;QAC9B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,0CAA0C,CAAC,CAAC;QACjE,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAChB,OAAO;IACT,CAAC;IAED,IAAI,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,WAAW,EAAE,EAAE,CAAC;QAC1C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,+CAA+C,CAAC,CAAC;QACtE,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAChB,OAAO;IACT,CAAC;IAED,aAAa,CAAC,wBAAwB,CAAC,CAAC;IACxC,MAAM,SAAS,GAAG,MAAM,YAAY,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC;IAElE,MAAM,KAAK,GAAiB,MAAM,OAAO,CAAC,GAAG,CAC3C,SAAS,CAAC,GAAG,CAAC,KAAK,EAAE,QAAQ,EAAE,EAAE,CAAC,CAAC;QACjC,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,MAAM,QAAQ,CAAC,OAAO,CAAC,YAAY,EAAE,QAAQ,CAAC,EAAE,MAAM,CAAC;KACjE,CAAC,CAAC,CACJ,CAAC;IAEF,aAAa,CAAC,oBAAoB,CAAC,CAAC;IACpC,MAAM,QAAQ,GAAG,YAAY,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC;IAE7C,MAAM,MAAM,GAAG,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC;IAC1C,IAAI,YAAY,GAAG,YAAY,CAAC,YAAY,CAAC;IAE7C,IAAI,CAAC,YAAY,IAAI,MAAM,EAAE,CAAC;QAC5B,aAAa,CAAC,2BAA2B,CAAC,CAAC;QAC3C,YAAY,GAAG,IAAI,YAAY,CAAC;YAC9B,QAAQ,EAAE,IAAI,cAAc,CAAC,EAAE,MAAM,EAAE,CAAC;SACzC,CAAC,CAAC;IACL,CAAC;IAED,IAAI,MAAoB,CAAC;IAEzB,IAAI,YAAY,EAAE,CAAC;QACjB,aAAa,CAAC,sBAAsB,CAAC,CAAC;QACtC,MAAM,GAAG,MAAM,YAAY,CAAC,MAAM,CAAC,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,CAAC;IAC1D,CAAC;SAAM,CAAC;QACN,MAAM,GAAG;YACP,QAAQ;YACR,YAAY,EAAE,KAAK,CAAC,MAAM;YAC1B,UAAU,EAAE,CAAC;SACd,CAAC;IACJ,CAAC;IAED,aAAa,CAAC,sBAAsB,CAAC,CAAC;IAEtC,IAAI,UAAU,EAAE,CAAC;QACf,MAAM,MAAM,GACV,MAAM,KAAK,MAAM;YACf,CAAC,CAAC,kBAAkB,CAAC,MAAM,CAAC;YAC5B,CAAC,CAAC,sBAAsB,CAAC,MAAM,CAAC,CAAC;QAErC,MAAM,SAAS,CAAC,OAAO,CAAC,UAAU,CAAC,EAAE,GAAG,MAAM,IAAI,EAAE,MAAM,CAAC,CAAC;QAC5D,OAAO;IACT,CAAC;IAED,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;QACtB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,kBAAkB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;QACxD,OAAO;IACT,CAAC;IAED,IAAI,OAAO,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC;QACzB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,sBAAsB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;IAC9D,CAAC;SAAM,CAAC;QACN,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,sBAAsB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;IAC9D,CAAC;AACH,CAAC"}
|
|
@@ -0,0 +1,11 @@
|
|
|
1
|
+
import type { Finding } from '../analyzer/finding.js';
|
|
2
|
+
import type { SourceFile } from '../analyzer/sourceFile.js';
|
|
3
|
+
export type ReviewContext = {
|
|
4
|
+
finding: Finding;
|
|
5
|
+
code: string;
|
|
6
|
+
startLine: number;
|
|
7
|
+
endLine: number;
|
|
8
|
+
imports: string[];
|
|
9
|
+
};
|
|
10
|
+
export declare function buildContext(sourceFile: SourceFile, finding: Finding): ReviewContext;
|
|
11
|
+
//# sourceMappingURL=buildContext.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"buildContext.d.ts","sourceRoot":"","sources":["../../src/context/buildContext.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAC;AACtD,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,2BAA2B,CAAC;AAE5D,MAAM,MAAM,aAAa,GAAG;IAC1B,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,EAAE,CAAC;CACnB,CAAC;AAIF,wBAAgB,YAAY,CAC1B,UAAU,EAAE,UAAU,EACtB,OAAO,EAAE,OAAO,GACf,aAAa,CAiBf"}
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
const CONTEXT_LINES = 3;
|
|
2
|
+
export function buildContext(sourceFile, finding) {
|
|
3
|
+
const lines = sourceFile.content.split('\n');
|
|
4
|
+
const startIndex = Math.max(0, finding.line - 1 - CONTEXT_LINES);
|
|
5
|
+
const endIndex = Math.min(lines.length, finding.line + CONTEXT_LINES);
|
|
6
|
+
const imports = lines
|
|
7
|
+
.map((line) => line.trim())
|
|
8
|
+
.filter((line) => line.startsWith('import '));
|
|
9
|
+
return {
|
|
10
|
+
finding,
|
|
11
|
+
code: lines.slice(startIndex, endIndex).join('\n'),
|
|
12
|
+
startLine: startIndex + 1,
|
|
13
|
+
endLine: endIndex,
|
|
14
|
+
imports,
|
|
15
|
+
};
|
|
16
|
+
}
|
|
17
|
+
//# sourceMappingURL=buildContext.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"buildContext.js","sourceRoot":"","sources":["../../src/context/buildContext.ts"],"names":[],"mappings":"AAWA,MAAM,aAAa,GAAG,CAAC,CAAC;AAExB,MAAM,UAAU,YAAY,CAC1B,UAAsB,EACtB,OAAgB;IAEhB,MAAM,KAAK,GAAG,UAAU,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAE7C,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,OAAO,CAAC,IAAI,GAAG,CAAC,GAAG,aAAa,CAAC,CAAC;IACjE,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,EAAE,OAAO,CAAC,IAAI,GAAG,aAAa,CAAC,CAAC;IAEtE,MAAM,OAAO,GAAG,KAAK;SAClB,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;SAC1B,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,SAAS,CAAC,CAAC,CAAC;IAEhD,OAAO;QACL,OAAO;QACP,IAAI,EAAE,KAAK,CAAC,KAAK,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;QAClD,SAAS,EAAE,UAAU,GAAG,CAAC;QACzB,OAAO,EAAE,QAAQ;QACjB,OAAO;KACR,CAAC;AACJ,CAAC"}
|
package/dist/index.d.ts
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":""}
|
package/dist/index.js
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AAEA,OAAO,EAAE,MAAM,EAAE,MAAM,UAAU,CAAC;AAElC,MAAM,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"jsonReporter.d.ts","sourceRoot":"","sources":["../../src/reporting/jsonReporter.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AAE9D,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,YAAY,GAAG,MAAM,CAE/D"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"jsonReporter.js","sourceRoot":"","sources":["../../src/reporting/jsonReporter.ts"],"names":[],"mappings":"AAEA,MAAM,UAAU,kBAAkB,CAAC,MAAoB;IACrD,OAAO,IAAI,CAAC,SAAS,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC;AACzC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"markdownReporter.d.ts","sourceRoot":"","sources":["../../src/reporting/markdownReporter.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AAE9D,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,YAAY,GAAG,MAAM,CA0DnE"}
|
|
@@ -0,0 +1,40 @@
|
|
|
1
|
+
export function generateMarkdownReport(result) {
|
|
2
|
+
const highCount = result.findings.filter((finding) => finding.severity === 'high').length;
|
|
3
|
+
const mediumCount = result.findings.filter((finding) => finding.severity === 'medium').length;
|
|
4
|
+
const lowCount = result.findings.filter((finding) => finding.severity === 'low').length;
|
|
5
|
+
const summary = [
|
|
6
|
+
'## Summary',
|
|
7
|
+
'',
|
|
8
|
+
`- Files scanned: ${result.filesScanned}`,
|
|
9
|
+
`- Total findings: ${result.findings.length}`,
|
|
10
|
+
`- High severity: ${highCount}`,
|
|
11
|
+
`- Medium severity: ${mediumCount}`,
|
|
12
|
+
`- Low severity: ${lowCount}`,
|
|
13
|
+
].join('\n');
|
|
14
|
+
const findings = result.findings
|
|
15
|
+
.map((finding) => {
|
|
16
|
+
const sections = [
|
|
17
|
+
`### ${finding.severity.toUpperCase()} - ${finding.file}:${finding.line}`,
|
|
18
|
+
finding.message,
|
|
19
|
+
`**Rule confidence:** ${Math.round(finding.confidence * 100)}%`,
|
|
20
|
+
];
|
|
21
|
+
if (finding.evidence) {
|
|
22
|
+
sections.push(`**Evidence:**\n\n\`\`\`tsx\n${finding.evidence}\n\`\`\``);
|
|
23
|
+
}
|
|
24
|
+
if (finding.aiReview) {
|
|
25
|
+
sections.push([
|
|
26
|
+
'### AI Review',
|
|
27
|
+
`- **Valid finding:** ${finding.aiReview.valid}`,
|
|
28
|
+
`- **AI confidence:** ${Math.round(finding.aiReview.confidence * 100)}%`,
|
|
29
|
+
`- **Reason:** ${finding.aiReview.reason}`,
|
|
30
|
+
`- **Recommendation:** ${finding.aiReview.recommendation}`,
|
|
31
|
+
`- **Assessment:** ${finding.aiReview.valid ? 'Valid finding' : 'False positive'}`,
|
|
32
|
+
].join('\n'));
|
|
33
|
+
}
|
|
34
|
+
return sections.join('\n\n');
|
|
35
|
+
})
|
|
36
|
+
.join('\n\n');
|
|
37
|
+
const findingsSection = result.findings.length === 0 ? 'No findings detected.' : findings;
|
|
38
|
+
return `# Sudeshi Code Review\n\n${summary}\n\n${findingsSection}`;
|
|
39
|
+
}
|
|
40
|
+
//# sourceMappingURL=markdownReporter.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"markdownReporter.js","sourceRoot":"","sources":["../../src/reporting/markdownReporter.ts"],"names":[],"mappings":"AAEA,MAAM,UAAU,sBAAsB,CAAC,MAAoB;IACzD,MAAM,SAAS,GAAG,MAAM,CAAC,QAAQ,CAAC,MAAM,CACtC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,KAAK,MAAM,CACzC,CAAC,MAAM,CAAC;IACT,MAAM,WAAW,GAAG,MAAM,CAAC,QAAQ,CAAC,MAAM,CACxC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,KAAK,QAAQ,CAC3C,CAAC,MAAM,CAAC;IACT,MAAM,QAAQ,GAAG,MAAM,CAAC,QAAQ,CAAC,MAAM,CACrC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,KAAK,KAAK,CACxC,CAAC,MAAM,CAAC;IAET,MAAM,OAAO,GAAG;QACd,YAAY;QACZ,EAAE;QACF,oBAAoB,MAAM,CAAC,YAAY,EAAE;QACzC,qBAAqB,MAAM,CAAC,QAAQ,CAAC,MAAM,EAAE;QAC7C,oBAAoB,SAAS,EAAE;QAC/B,sBAAsB,WAAW,EAAE;QACnC,mBAAmB,QAAQ,EAAE;KAC9B,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAEb,MAAM,QAAQ,GAAG,MAAM,CAAC,QAAQ;SAC7B,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE;QACf,MAAM,QAAQ,GAAG;YACf,OAAO,OAAO,CAAC,QAAQ,CAAC,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,IAAI,OAAO,CAAC,IAAI,EAAE;YACzE,OAAO,CAAC,OAAO;YACf,wBAAwB,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,UAAU,GAAG,GAAG,CAAC,GAAG;SAChE,CAAC;QAEF,IAAI,OAAO,CAAC,QAAQ,EAAE,CAAC;YACrB,QAAQ,CAAC,IAAI,CACX,+BAA+B,OAAO,CAAC,QAAQ,UAAU,CAC1D,CAAC;QACJ,CAAC;QAED,IAAI,OAAO,CAAC,QAAQ,EAAE,CAAC;YACrB,QAAQ,CAAC,IAAI,CACX;gBACE,eAAe;gBACf,wBAAwB,OAAO,CAAC,QAAQ,CAAC,KAAK,EAAE;gBAChD,wBAAwB,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,QAAQ,CAAC,UAAU,GAAG,GAAG,CAAC,GAAG;gBACxE,iBAAiB,OAAO,CAAC,QAAQ,CAAC,MAAM,EAAE;gBAC1C,yBAAyB,OAAO,CAAC,QAAQ,CAAC,cAAc,EAAE;gBAC1D,qBACE,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,gBAC7C,EAAE;aACH,CAAC,IAAI,CAAC,IAAI,CAAC,CACb,CAAC;QACJ,CAAC;QAED,OAAO,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC/B,CAAC,CAAC;SACD,IAAI,CAAC,MAAM,CAAC,CAAC;IAEhB,MAAM,eAAe,GACnB,MAAM,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,uBAAuB,CAAC,CAAC,CAAC,QAAQ,CAAC;IAEpE,OAAO,4BAA4B,OAAO,OAAO,eAAe,EAAE,CAAC;AACrE,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"terminalReporter.d.ts","sourceRoot":"","sources":["../../src/reporting/terminalReporter.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AAE9D,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,YAAY,GAAG,MAAM,CA2CnE"}
|
|
@@ -0,0 +1,30 @@
|
|
|
1
|
+
export function generateTerminalReport(result) {
|
|
2
|
+
const counts = {
|
|
3
|
+
high: result.findings.filter((finding) => finding.severity === 'high')
|
|
4
|
+
.length,
|
|
5
|
+
medium: result.findings.filter((finding) => finding.severity === 'medium')
|
|
6
|
+
.length,
|
|
7
|
+
low: result.findings.filter((finding) => finding.severity === 'low').length,
|
|
8
|
+
};
|
|
9
|
+
const lines = [
|
|
10
|
+
'SUDESHI CODE REVIEW',
|
|
11
|
+
'===================',
|
|
12
|
+
'',
|
|
13
|
+
`Review complete in ${(result.durationMs / 1000).toFixed(2)}s`,
|
|
14
|
+
`${result.filesScanned} files scanned · ${result.findings.length} findings`,
|
|
15
|
+
'',
|
|
16
|
+
`Severity: ${counts.high} high · ${counts.medium} medium · ${counts.low} low`,
|
|
17
|
+
];
|
|
18
|
+
if (result.findings.length === 0) {
|
|
19
|
+
lines.push('', 'No findings detected.');
|
|
20
|
+
return lines.join('\n');
|
|
21
|
+
}
|
|
22
|
+
for (const finding of result.findings) {
|
|
23
|
+
lines.push('', `[${finding.severity.toUpperCase()}] ${finding.rule}`, ` ${finding.file}:${finding.line}`, ` ${finding.message}`, ` Confidence: ${Math.round(finding.confidence * 100)}%`);
|
|
24
|
+
if (finding.aiReview) {
|
|
25
|
+
lines.push(` AI assessment: ${finding.aiReview.valid ? 'Valid finding' : 'Possible false positive'}`, ` AI confidence: ${Math.round(finding.aiReview.confidence * 100)}%`, ` Recommendation: ${finding.aiReview.recommendation}`);
|
|
26
|
+
}
|
|
27
|
+
}
|
|
28
|
+
return lines.join('\n');
|
|
29
|
+
}
|
|
30
|
+
//# sourceMappingURL=terminalReporter.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"terminalReporter.js","sourceRoot":"","sources":["../../src/reporting/terminalReporter.ts"],"names":[],"mappings":"AAEA,MAAM,UAAU,sBAAsB,CAAC,MAAoB;IACzD,MAAM,MAAM,GAAG;QACb,IAAI,EAAE,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,KAAK,MAAM,CAAC;aACnE,MAAM;QACT,MAAM,EAAE,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,KAAK,QAAQ,CAAC;aACvE,MAAM;QACT,GAAG,EAAE,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,KAAK,KAAK,CAAC,CAAC,MAAM;KAC5E,CAAC;IAEF,MAAM,KAAK,GAAG;QACZ,qBAAqB;QACrB,qBAAqB;QACrB,EAAE;QACF,sBAAsB,CAAC,MAAM,CAAC,UAAU,GAAG,IAAI,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG;QAC9D,GAAG,MAAM,CAAC,YAAY,oBAAoB,MAAM,CAAC,QAAQ,CAAC,MAAM,WAAW;QAC3E,EAAE;QACF,aAAa,MAAM,CAAC,IAAI,WAAW,MAAM,CAAC,MAAM,aAAa,MAAM,CAAC,GAAG,MAAM;KAC9E,CAAC;IAEF,IAAI,MAAM,CAAC,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACjC,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,uBAAuB,CAAC,CAAC;QACxC,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC1B,CAAC;IAED,KAAK,MAAM,OAAO,IAAI,MAAM,CAAC,QAAQ,EAAE,CAAC;QACtC,KAAK,CAAC,IAAI,CACR,EAAE,EACF,IAAI,OAAO,CAAC,QAAQ,CAAC,WAAW,EAAE,KAAK,OAAO,CAAC,IAAI,EAAE,EACrD,KAAK,OAAO,CAAC,IAAI,IAAI,OAAO,CAAC,IAAI,EAAE,EACnC,KAAK,OAAO,CAAC,OAAO,EAAE,EACtB,iBAAiB,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,UAAU,GAAG,GAAG,CAAC,GAAG,CACzD,CAAC;QAEF,IAAI,OAAO,CAAC,QAAQ,EAAE,CAAC;YACrB,KAAK,CAAC,IAAI,CACR,oBAAoB,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,yBAAyB,EAAE,EAC1F,oBAAoB,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,QAAQ,CAAC,UAAU,GAAG,GAAG,CAAC,GAAG,EACpE,qBAAqB,OAAO,CAAC,QAAQ,CAAC,cAAc,EAAE,CACvD,CAAC;QACJ,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}
|
|
@@ -0,0 +1,24 @@
|
|
|
1
|
+
import type { AIProvider } from '../ai/provider.js';
|
|
2
|
+
import type { Finding } from '../analyzer/finding.js';
|
|
3
|
+
import type { SourceFile } from '../analyzer/sourceFile.js';
|
|
4
|
+
export type ReviewResult = {
|
|
5
|
+
findings: Finding[];
|
|
6
|
+
filesScanned: number;
|
|
7
|
+
durationMs: number;
|
|
8
|
+
};
|
|
9
|
+
type ReviewEngineConfig = {
|
|
10
|
+
provider: AIProvider;
|
|
11
|
+
};
|
|
12
|
+
type ReviewInput = {
|
|
13
|
+
files: SourceFile[];
|
|
14
|
+
findings: Finding[];
|
|
15
|
+
};
|
|
16
|
+
export declare class ReviewEngine {
|
|
17
|
+
private readonly config;
|
|
18
|
+
constructor(config: ReviewEngineConfig);
|
|
19
|
+
private redactHardcodedSecret;
|
|
20
|
+
private redactSecretValue;
|
|
21
|
+
review(input: ReviewInput): Promise<ReviewResult>;
|
|
22
|
+
}
|
|
23
|
+
export {};
|
|
24
|
+
//# sourceMappingURL=reviewEngine.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"reviewEngine.d.ts","sourceRoot":"","sources":["../../src/review/reviewEngine.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,mBAAmB,CAAC;AACpD,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAC;AACtD,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,2BAA2B,CAAC;AAG5D,MAAM,MAAM,YAAY,GAAG;IACzB,QAAQ,EAAE,OAAO,EAAE,CAAC;IACpB,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;CACpB,CAAC;AAEF,KAAK,kBAAkB,GAAG;IACxB,QAAQ,EAAE,UAAU,CAAC;CACtB,CAAC;AAEF,KAAK,WAAW,GAAG;IACjB,KAAK,EAAE,UAAU,EAAE,CAAC;IACpB,QAAQ,EAAE,OAAO,EAAE,CAAC;CACrB,CAAC;AAaF,qBAAa,YAAY;IACX,OAAO,CAAC,QAAQ,CAAC,MAAM;gBAAN,MAAM,EAAE,kBAAkB;IAEvD,OAAO,CAAC,qBAAqB;IAoE7B,OAAO,CAAC,iBAAiB;IAcnB,MAAM,CAAC,KAAK,EAAE,WAAW,GAAG,OAAO,CAAC,YAAY,CAAC;CA8DxD"}
|