critical-gate 2.0.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 +21 -0
- package/README.md +240 -0
- package/dist/cli.d.ts +20 -0
- package/dist/cli.d.ts.map +1 -0
- package/dist/cli.js +224 -0
- package/dist/cli.js.map +1 -0
- package/dist/config/critical-gate-config.d.ts +21 -0
- package/dist/config/critical-gate-config.d.ts.map +1 -0
- package/dist/config/critical-gate-config.js +81 -0
- package/dist/config/critical-gate-config.js.map +1 -0
- package/dist/config/index.d.ts +2 -0
- package/dist/config/index.d.ts.map +1 -0
- package/dist/config/index.js +2 -0
- package/dist/config/index.js.map +1 -0
- package/dist/detectors/api-surface-detector.d.ts +3 -0
- package/dist/detectors/api-surface-detector.d.ts.map +1 -0
- package/dist/detectors/api-surface-detector.js +111 -0
- package/dist/detectors/api-surface-detector.js.map +1 -0
- package/dist/detectors/blast-radius-detector.d.ts +3 -0
- package/dist/detectors/blast-radius-detector.d.ts.map +1 -0
- package/dist/detectors/blast-radius-detector.js +90 -0
- package/dist/detectors/blast-radius-detector.js.map +1 -0
- package/dist/detectors/config-change-detector.d.ts +3 -0
- package/dist/detectors/config-change-detector.d.ts.map +1 -0
- package/dist/detectors/config-change-detector.js +68 -0
- package/dist/detectors/config-change-detector.js.map +1 -0
- package/dist/detectors/dependency-detector.d.ts +3 -0
- package/dist/detectors/dependency-detector.d.ts.map +1 -0
- package/dist/detectors/dependency-detector.js +98 -0
- package/dist/detectors/dependency-detector.js.map +1 -0
- package/dist/detectors/existing-solution-detector.d.ts +3 -0
- package/dist/detectors/existing-solution-detector.d.ts.map +1 -0
- package/dist/detectors/existing-solution-detector.js +87 -0
- package/dist/detectors/existing-solution-detector.js.map +1 -0
- package/dist/detectors/expected-companions-detector.d.ts +3 -0
- package/dist/detectors/expected-companions-detector.d.ts.map +1 -0
- package/dist/detectors/expected-companions-detector.js +65 -0
- package/dist/detectors/expected-companions-detector.js.map +1 -0
- package/dist/detectors/index.d.ts +17 -0
- package/dist/detectors/index.d.ts.map +1 -0
- package/dist/detectors/index.js +16 -0
- package/dist/detectors/index.js.map +1 -0
- package/dist/detectors/intent-verification-detector.d.ts +3 -0
- package/dist/detectors/intent-verification-detector.d.ts.map +1 -0
- package/dist/detectors/intent-verification-detector.js +43 -0
- package/dist/detectors/intent-verification-detector.js.map +1 -0
- package/dist/detectors/pattern-violation-detector.d.ts +3 -0
- package/dist/detectors/pattern-violation-detector.d.ts.map +1 -0
- package/dist/detectors/pattern-violation-detector.js +83 -0
- package/dist/detectors/pattern-violation-detector.js.map +1 -0
- package/dist/detectors/repository-intelligence-detector.d.ts +3 -0
- package/dist/detectors/repository-intelligence-detector.d.ts.map +1 -0
- package/dist/detectors/repository-intelligence-detector.js +51 -0
- package/dist/detectors/repository-intelligence-detector.js.map +1 -0
- package/dist/detectors/rewrite-detector.d.ts +3 -0
- package/dist/detectors/rewrite-detector.d.ts.map +1 -0
- package/dist/detectors/rewrite-detector.js +61 -0
- package/dist/detectors/rewrite-detector.js.map +1 -0
- package/dist/detectors/runner.d.ts +9 -0
- package/dist/detectors/runner.d.ts.map +1 -0
- package/dist/detectors/runner.js +96 -0
- package/dist/detectors/runner.js.map +1 -0
- package/dist/detectors/scope-detector.d.ts +3 -0
- package/dist/detectors/scope-detector.d.ts.map +1 -0
- package/dist/detectors/scope-detector.js +138 -0
- package/dist/detectors/scope-detector.js.map +1 -0
- package/dist/detectors/secret-path-detector.d.ts +5 -0
- package/dist/detectors/secret-path-detector.d.ts.map +1 -0
- package/dist/detectors/secret-path-detector.js +150 -0
- package/dist/detectors/secret-path-detector.js.map +1 -0
- package/dist/detectors/test-weakening-detector.d.ts +3 -0
- package/dist/detectors/test-weakening-detector.d.ts.map +1 -0
- package/dist/detectors/test-weakening-detector.js +91 -0
- package/dist/detectors/test-weakening-detector.js.map +1 -0
- package/dist/detectors/types.d.ts +15 -0
- package/dist/detectors/types.d.ts.map +1 -0
- package/dist/detectors/types.js +2 -0
- package/dist/detectors/types.js.map +1 -0
- package/dist/detectors/utility-reinvention-detector.d.ts +3 -0
- package/dist/detectors/utility-reinvention-detector.d.ts.map +1 -0
- package/dist/detectors/utility-reinvention-detector.js +78 -0
- package/dist/detectors/utility-reinvention-detector.js.map +1 -0
- package/dist/diff/git-diff-reader.d.ts +24 -0
- package/dist/diff/git-diff-reader.d.ts.map +1 -0
- package/dist/diff/git-diff-reader.js +81 -0
- package/dist/diff/git-diff-reader.js.map +1 -0
- package/dist/diff/index.d.ts +4 -0
- package/dist/diff/index.d.ts.map +1 -0
- package/dist/diff/index.js +4 -0
- package/dist/diff/index.js.map +1 -0
- package/dist/diff/parse-unified-diff.d.ts +3 -0
- package/dist/diff/parse-unified-diff.d.ts.map +1 -0
- package/dist/diff/parse-unified-diff.js +153 -0
- package/dist/diff/parse-unified-diff.js.map +1 -0
- package/dist/diff/path-classifier.d.ts +5 -0
- package/dist/diff/path-classifier.d.ts.map +1 -0
- package/dist/diff/path-classifier.js +86 -0
- package/dist/diff/path-classifier.js.map +1 -0
- package/dist/editor/diagnostics.d.ts +35 -0
- package/dist/editor/diagnostics.d.ts.map +1 -0
- package/dist/editor/diagnostics.js +55 -0
- package/dist/editor/diagnostics.js.map +1 -0
- package/dist/editor/index.d.ts +3 -0
- package/dist/editor/index.d.ts.map +1 -0
- package/dist/editor/index.js +2 -0
- package/dist/editor/index.js.map +1 -0
- package/dist/index.d.ts +11 -0
- package/dist/index.d.ts.map +1 -0
- package/dist/index.js +11 -0
- package/dist/index.js.map +1 -0
- package/dist/intent/index.d.ts +5 -0
- package/dist/intent/index.d.ts.map +1 -0
- package/dist/intent/index.js +5 -0
- package/dist/intent/index.js.map +1 -0
- package/dist/intent/intent-core.d.ts +4 -0
- package/dist/intent/intent-core.d.ts.map +1 -0
- package/dist/intent/intent-core.js +58 -0
- package/dist/intent/intent-core.js.map +1 -0
- package/dist/intent/intent-model.d.ts +18 -0
- package/dist/intent/intent-model.d.ts.map +1 -0
- package/dist/intent/intent-model.js +74 -0
- package/dist/intent/intent-model.js.map +1 -0
- package/dist/intent/observed-actions.d.ts +17 -0
- package/dist/intent/observed-actions.d.ts.map +1 -0
- package/dist/intent/observed-actions.js +128 -0
- package/dist/intent/observed-actions.js.map +1 -0
- package/dist/intent/scope-expansion-score.d.ts +13 -0
- package/dist/intent/scope-expansion-score.d.ts.map +1 -0
- package/dist/intent/scope-expansion-score.js +87 -0
- package/dist/intent/scope-expansion-score.js.map +1 -0
- package/dist/intent/task-analysis.d.ts +18 -0
- package/dist/intent/task-analysis.d.ts.map +1 -0
- package/dist/intent/task-analysis.js +46 -0
- package/dist/intent/task-analysis.js.map +1 -0
- package/dist/knowledge/cache.d.ts +29 -0
- package/dist/knowledge/cache.d.ts.map +1 -0
- package/dist/knowledge/cache.js +85 -0
- package/dist/knowledge/cache.js.map +1 -0
- package/dist/knowledge/graph.d.ts +16 -0
- package/dist/knowledge/graph.d.ts.map +1 -0
- package/dist/knowledge/graph.js +139 -0
- package/dist/knowledge/graph.js.map +1 -0
- package/dist/knowledge/history-index.d.ts +20 -0
- package/dist/knowledge/history-index.d.ts.map +1 -0
- package/dist/knowledge/history-index.js +117 -0
- package/dist/knowledge/history-index.js.map +1 -0
- package/dist/knowledge/index.d.ts +9 -0
- package/dist/knowledge/index.d.ts.map +1 -0
- package/dist/knowledge/index.js +8 -0
- package/dist/knowledge/index.js.map +1 -0
- package/dist/knowledge/pattern-index.d.ts +14 -0
- package/dist/knowledge/pattern-index.d.ts.map +1 -0
- package/dist/knowledge/pattern-index.js +94 -0
- package/dist/knowledge/pattern-index.js.map +1 -0
- package/dist/knowledge/provider.d.ts +18 -0
- package/dist/knowledge/provider.d.ts.map +1 -0
- package/dist/knowledge/provider.js +83 -0
- package/dist/knowledge/provider.js.map +1 -0
- package/dist/knowledge/solution-index.d.ts +17 -0
- package/dist/knowledge/solution-index.d.ts.map +1 -0
- package/dist/knowledge/solution-index.js +170 -0
- package/dist/knowledge/solution-index.js.map +1 -0
- package/dist/knowledge/symbol-index.d.ts +4 -0
- package/dist/knowledge/symbol-index.d.ts.map +1 -0
- package/dist/knowledge/symbol-index.js +11 -0
- package/dist/knowledge/symbol-index.js.map +1 -0
- package/dist/knowledge/types.d.ts +100 -0
- package/dist/knowledge/types.d.ts.map +1 -0
- package/dist/knowledge/types.js +2 -0
- package/dist/knowledge/types.js.map +1 -0
- package/dist/llm/artifact.d.ts +6 -0
- package/dist/llm/artifact.d.ts.map +1 -0
- package/dist/llm/artifact.js +86 -0
- package/dist/llm/artifact.js.map +1 -0
- package/dist/llm/cache.d.ts +10 -0
- package/dist/llm/cache.d.ts.map +1 -0
- package/dist/llm/cache.js +24 -0
- package/dist/llm/cache.js.map +1 -0
- package/dist/llm/index.d.ts +8 -0
- package/dist/llm/index.d.ts.map +1 -0
- package/dist/llm/index.js +7 -0
- package/dist/llm/index.js.map +1 -0
- package/dist/llm/prompt.d.ts +7 -0
- package/dist/llm/prompt.d.ts.map +1 -0
- package/dist/llm/prompt.js +23 -0
- package/dist/llm/prompt.js.map +1 -0
- package/dist/llm/provider.d.ts +4 -0
- package/dist/llm/provider.d.ts.map +1 -0
- package/dist/llm/provider.js +43 -0
- package/dist/llm/provider.js.map +1 -0
- package/dist/llm/redaction.d.ts +3 -0
- package/dist/llm/redaction.d.ts.map +1 -0
- package/dist/llm/redaction.js +22 -0
- package/dist/llm/redaction.js.map +1 -0
- package/dist/llm/types.d.ts +89 -0
- package/dist/llm/types.d.ts.map +1 -0
- package/dist/llm/types.js +2 -0
- package/dist/llm/types.js.map +1 -0
- package/dist/reporters/index.d.ts +8 -0
- package/dist/reporters/index.d.ts.map +1 -0
- package/dist/reporters/index.js +21 -0
- package/dist/reporters/index.js.map +1 -0
- package/dist/reporters/json-reporter.d.ts +3 -0
- package/dist/reporters/json-reporter.d.ts.map +1 -0
- package/dist/reporters/json-reporter.js +4 -0
- package/dist/reporters/json-reporter.js.map +1 -0
- package/dist/reporters/markdown-reporter.d.ts +3 -0
- package/dist/reporters/markdown-reporter.d.ts.map +1 -0
- package/dist/reporters/markdown-reporter.js +76 -0
- package/dist/reporters/markdown-reporter.js.map +1 -0
- package/dist/reporters/repair-reporter.d.ts +3 -0
- package/dist/reporters/repair-reporter.d.ts.map +1 -0
- package/dist/reporters/repair-reporter.js +56 -0
- package/dist/reporters/repair-reporter.js.map +1 -0
- package/dist/reporters/sarif-reporter.d.ts +3 -0
- package/dist/reporters/sarif-reporter.d.ts.map +1 -0
- package/dist/reporters/sarif-reporter.js +161 -0
- package/dist/reporters/sarif-reporter.js.map +1 -0
- package/dist/repository/index.d.ts +3 -0
- package/dist/repository/index.d.ts.map +1 -0
- package/dist/repository/index.js +3 -0
- package/dist/repository/index.js.map +1 -0
- package/dist/repository/profile.d.ts +7 -0
- package/dist/repository/profile.d.ts.map +1 -0
- package/dist/repository/profile.js +6 -0
- package/dist/repository/profile.js.map +1 -0
- package/dist/repository/utility-index.d.ts +7 -0
- package/dist/repository/utility-index.d.ts.map +1 -0
- package/dist/repository/utility-index.js +6 -0
- package/dist/repository/utility-index.js.map +1 -0
- package/dist/schema/gate-result.schema.d.ts +514 -0
- package/dist/schema/gate-result.schema.d.ts.map +1 -0
- package/dist/schema/gate-result.schema.js +326 -0
- package/dist/schema/gate-result.schema.js.map +1 -0
- package/dist/schema/index.d.ts +4 -0
- package/dist/schema/index.d.ts.map +1 -0
- package/dist/schema/index.js +3 -0
- package/dist/schema/index.js.map +1 -0
- package/dist/schema/types.d.ts +136 -0
- package/dist/schema/types.d.ts.map +1 -0
- package/dist/schema/types.js +2 -0
- package/dist/schema/types.js.map +1 -0
- package/package.json +51 -0
|
@@ -0,0 +1,111 @@
|
|
|
1
|
+
const apiTaskTerms = [
|
|
2
|
+
"api",
|
|
3
|
+
"public api",
|
|
4
|
+
"export",
|
|
5
|
+
"exports",
|
|
6
|
+
"breaking",
|
|
7
|
+
"signature",
|
|
8
|
+
"changelog",
|
|
9
|
+
"release note",
|
|
10
|
+
"public surface"
|
|
11
|
+
];
|
|
12
|
+
const documentationPathPattern = /(^|\/)(docs?|adr|changelog|changesets?)\/|(^|\/)(CHANGELOG|README|ADR)[^/]*\.md$/i;
|
|
13
|
+
const exportDeclarationPattern = /^\s*export\s+(?:async\s+)?(?:declare\s+)?(?:function|class|interface|type|enum|const|let|var)\s+([A-Za-z_$][\w$]*)/;
|
|
14
|
+
const namedExportPattern = /^\s*export\s*\{([^}]+)\}/;
|
|
15
|
+
const defaultExportPattern = /^\s*export\s+default\s+/;
|
|
16
|
+
export const apiSurfaceDetector = {
|
|
17
|
+
name: "api-surface",
|
|
18
|
+
run: ({ task, diff }) => {
|
|
19
|
+
if (hasVisibleApiAcknowledgement(task.text, diff.files)) {
|
|
20
|
+
return [];
|
|
21
|
+
}
|
|
22
|
+
return diff.files.filter(isSourceFile).flatMap(extractSignals).map(toFinding);
|
|
23
|
+
}
|
|
24
|
+
};
|
|
25
|
+
function isSourceFile(file) {
|
|
26
|
+
return file.role === "source" && /\.(?:[cm]?[jt]sx?)$/.test(file.path);
|
|
27
|
+
}
|
|
28
|
+
function hasVisibleApiAcknowledgement(taskText, files) {
|
|
29
|
+
const normalizedTask = taskText.toLowerCase();
|
|
30
|
+
return (apiTaskTerms.some((term) => normalizedTask.includes(term)) ||
|
|
31
|
+
files.some((file) => documentationPathPattern.test(file.path)));
|
|
32
|
+
}
|
|
33
|
+
function extractSignals(file) {
|
|
34
|
+
return file.hunks.flatMap((hunk) => hunk.lines.flatMap((line) => {
|
|
35
|
+
if (line.kind !== "add" && line.kind !== "delete") {
|
|
36
|
+
return [];
|
|
37
|
+
}
|
|
38
|
+
const exportInfo = getExportInfo(line);
|
|
39
|
+
if (exportInfo === undefined) {
|
|
40
|
+
return [];
|
|
41
|
+
}
|
|
42
|
+
const isRemoval = line.kind === "delete";
|
|
43
|
+
return [
|
|
44
|
+
{
|
|
45
|
+
id: isRemoval ? "removed-export" : "added-export",
|
|
46
|
+
title: isRemoval ? "Public export removed" : "Public export added",
|
|
47
|
+
message: isRemoval
|
|
48
|
+
? "The diff removes an exported symbol without visible API acknowledgement."
|
|
49
|
+
: "The diff adds an exported symbol without visible API acknowledgement.",
|
|
50
|
+
repair: isRemoval
|
|
51
|
+
? "Confirm this is an intended public API change and add changelog, release note, docs, or explicit task/PR acknowledgement."
|
|
52
|
+
: "Confirm the new export is intended public API and document it or keep it internal.",
|
|
53
|
+
severity: isRemoval ? "high" : "medium",
|
|
54
|
+
confidence: exportInfo.kind === "named" ? 0.78 : 0.86,
|
|
55
|
+
path: file.path,
|
|
56
|
+
lineNumber: line.kind === "add" ? line.newLineNumber : line.oldLineNumber,
|
|
57
|
+
symbol: exportInfo.symbol,
|
|
58
|
+
content: line.content
|
|
59
|
+
}
|
|
60
|
+
];
|
|
61
|
+
}));
|
|
62
|
+
}
|
|
63
|
+
function getExportInfo(line) {
|
|
64
|
+
const declaredExport = exportDeclarationPattern.exec(line.content);
|
|
65
|
+
if (declaredExport !== null) {
|
|
66
|
+
return {
|
|
67
|
+
kind: "declared",
|
|
68
|
+
symbol: declaredExport[1]
|
|
69
|
+
};
|
|
70
|
+
}
|
|
71
|
+
const namedExport = namedExportPattern.exec(line.content);
|
|
72
|
+
if (namedExport !== null) {
|
|
73
|
+
return {
|
|
74
|
+
kind: "named",
|
|
75
|
+
symbol: namedExport[1]?.split(",")[0]?.trim()
|
|
76
|
+
};
|
|
77
|
+
}
|
|
78
|
+
if (defaultExportPattern.test(line.content)) {
|
|
79
|
+
return {
|
|
80
|
+
kind: "default",
|
|
81
|
+
symbol: "default"
|
|
82
|
+
};
|
|
83
|
+
}
|
|
84
|
+
return undefined;
|
|
85
|
+
}
|
|
86
|
+
function toFinding(signal) {
|
|
87
|
+
return {
|
|
88
|
+
id: `api-surface:${signal.path}:${signal.id}:${signal.symbol ?? "unknown"}:${signal.lineNumber ?? "unknown"}`,
|
|
89
|
+
detector: "api-surface",
|
|
90
|
+
severity: signal.severity,
|
|
91
|
+
confidence: signal.confidence,
|
|
92
|
+
title: signal.title,
|
|
93
|
+
message: signal.message,
|
|
94
|
+
evidence: [
|
|
95
|
+
{
|
|
96
|
+
kind: "symbol",
|
|
97
|
+
path: signal.path,
|
|
98
|
+
startLine: signal.lineNumber,
|
|
99
|
+
endLine: signal.lineNumber,
|
|
100
|
+
symbol: signal.symbol,
|
|
101
|
+
message: signal.content.trim(),
|
|
102
|
+
data: {
|
|
103
|
+
signal: signal.id
|
|
104
|
+
}
|
|
105
|
+
}
|
|
106
|
+
],
|
|
107
|
+
repair: signal.repair,
|
|
108
|
+
tags: ["api"]
|
|
109
|
+
};
|
|
110
|
+
}
|
|
111
|
+
//# sourceMappingURL=api-surface-detector.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"api-surface-detector.js","sourceRoot":"","sources":["../../src/detectors/api-surface-detector.ts"],"names":[],"mappings":"AAiBA,MAAM,YAAY,GAAG;IACnB,KAAK;IACL,YAAY;IACZ,QAAQ;IACR,SAAS;IACT,UAAU;IACV,WAAW;IACX,WAAW;IACX,cAAc;IACd,gBAAgB;CACjB,CAAC;AAEF,MAAM,wBAAwB,GAC5B,mFAAmF,CAAC;AAEtF,MAAM,wBAAwB,GAC5B,oHAAoH,CAAC;AACvH,MAAM,kBAAkB,GAAG,0BAA0B,CAAC;AACtD,MAAM,oBAAoB,GAAG,yBAAyB,CAAC;AAEvD,MAAM,CAAC,MAAM,kBAAkB,GAAa;IAC1C,IAAI,EAAE,aAAa;IACnB,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE,EAAE;QACtB,IAAI,4BAA4B,CAAC,IAAI,CAAC,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;YACxD,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,OAAO,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,cAAc,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC;IAChF,CAAC;CACF,CAAC;AAEF,SAAS,YAAY,CAAC,IAAc;IAClC,OAAO,IAAI,CAAC,IAAI,KAAK,QAAQ,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACzE,CAAC;AAED,SAAS,4BAA4B,CAAC,QAAgB,EAAE,KAAiB;IACvE,MAAM,cAAc,GAAG,QAAQ,CAAC,WAAW,EAAE,CAAC;IAE9C,OAAO,CACL,YAAY,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,cAAc,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAC1D,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,wBAAwB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAC/D,CAAC;AACJ,CAAC;AAED,SAAS,cAAc,CAAC,IAAc;IACpC,OAAO,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE,CACjC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE;QAC1B,IAAI,IAAI,CAAC,IAAI,KAAK,KAAK,IAAI,IAAI,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;YAClD,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,UAAU,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC;QAEvC,IAAI,UAAU,KAAK,SAAS,EAAE,CAAC;YAC7B,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,KAAK,QAAQ,CAAC;QAEzC,OAAO;YACL;gBACE,EAAE,EAAE,SAAS,CAAC,CAAC,CAAC,gBAAgB,CAAC,CAAC,CAAC,cAAc;gBACjD,KAAK,EAAE,SAAS,CAAC,CAAC,CAAC,uBAAuB,CAAC,CAAC,CAAC,qBAAqB;gBAClE,OAAO,EAAE,SAAS;oBAChB,CAAC,CAAC,0EAA0E;oBAC5E,CAAC,CAAC,uEAAuE;gBAC3E,MAAM,EAAE,SAAS;oBACf,CAAC,CAAC,2HAA2H;oBAC7H,CAAC,CAAC,oFAAoF;gBACxF,QAAQ,EAAE,SAAS,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ;gBACvC,UAAU,EAAE,UAAU,CAAC,IAAI,KAAK,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI;gBACrD,IAAI,EAAE,IAAI,CAAC,IAAI;gBACf,UAAU,EAAE,IAAI,CAAC,IAAI,KAAK,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,aAAa;gBACzE,MAAM,EAAE,UAAU,CAAC,MAAM;gBACzB,OAAO,EAAE,IAAI,CAAC,OAAO;aACtB;SACF,CAAC;IACJ,CAAC,CAAC,CACH,CAAC;AACJ,CAAC;AAED,SAAS,aAAa,CACpB,IAAc;IAEd,MAAM,cAAc,GAAG,wBAAwB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;IAEnE,IAAI,cAAc,KAAK,IAAI,EAAE,CAAC;QAC5B,OAAO;YACL,IAAI,EAAE,UAAU;YAChB,MAAM,EAAE,cAAc,CAAC,CAAC,CAAC;SAC1B,CAAC;IACJ,CAAC;IAED,MAAM,WAAW,GAAG,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;IAE1D,IAAI,WAAW,KAAK,IAAI,EAAE,CAAC;QACzB,OAAO;YACL,IAAI,EAAE,OAAO;YACb,MAAM,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE;SAC9C,CAAC;IACJ,CAAC;IAED,IAAI,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;QAC5C,OAAO;YACL,IAAI,EAAE,SAAS;YACf,MAAM,EAAE,SAAS;SAClB,CAAC;IACJ,CAAC;IAED,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,SAAS,SAAS,CAAC,MAAwB;IACzC,OAAO;QACL,EAAE,EAAE,eAAe,MAAM,CAAC,IAAI,IAAI,MAAM,CAAC,EAAE,IAAI,MAAM,CAAC,MAAM,IAAI,SAAS,IAAI,MAAM,CAAC,UAAU,IAAI,SAAS,EAAE;QAC7G,QAAQ,EAAE,aAAa;QACvB,QAAQ,EAAE,MAAM,CAAC,QAAQ;QACzB,UAAU,EAAE,MAAM,CAAC,UAAU;QAC7B,KAAK,EAAE,MAAM,CAAC,KAAK;QACnB,OAAO,EAAE,MAAM,CAAC,OAAO;QACvB,QAAQ,EAAE;YACR;gBACE,IAAI,EAAE,QAAQ;gBACd,IAAI,EAAE,MAAM,CAAC,IAAI;gBACjB,SAAS,EAAE,MAAM,CAAC,UAAU;gBAC5B,OAAO,EAAE,MAAM,CAAC,UAAU;gBAC1B,MAAM,EAAE,MAAM,CAAC,MAAM;gBACrB,OAAO,EAAE,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE;gBAC9B,IAAI,EAAE;oBACJ,MAAM,EAAE,MAAM,CAAC,EAAE;iBAClB;aACF;SACF;QACD,MAAM,EAAE,MAAM,CAAC,MAAM;QACrB,IAAI,EAAE,CAAC,KAAK,CAAC;KACd,CAAC;AACJ,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"blast-radius-detector.d.ts","sourceRoot":"","sources":["../../src/detectors/blast-radius-detector.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAO3C,eAAO,MAAM,mBAAmB,EAAE,QAsBjC,CAAC"}
|
|
@@ -0,0 +1,90 @@
|
|
|
1
|
+
export const blastRadiusDetector = {
|
|
2
|
+
name: "blast-radius",
|
|
3
|
+
run: ({ diff, context }) => {
|
|
4
|
+
const graph = context?.knowledge?.getFileGraph();
|
|
5
|
+
if (graph === undefined || diff.files.length < 2) {
|
|
6
|
+
return [];
|
|
7
|
+
}
|
|
8
|
+
const clusters = clusterChangedFiles(diff.files, graph);
|
|
9
|
+
if (clusters.length < 2) {
|
|
10
|
+
return [];
|
|
11
|
+
}
|
|
12
|
+
const primaryCluster = clusters[0];
|
|
13
|
+
return clusters
|
|
14
|
+
.slice(1)
|
|
15
|
+
.filter((cluster) => cluster !== primaryCluster)
|
|
16
|
+
.map((cluster, index) => toFinding(cluster, index + 1));
|
|
17
|
+
}
|
|
18
|
+
};
|
|
19
|
+
function clusterChangedFiles(files, graph) {
|
|
20
|
+
const changedByPath = new Map(files.map((file) => [file.path, file]));
|
|
21
|
+
const adjacency = new Map();
|
|
22
|
+
for (const file of files) {
|
|
23
|
+
adjacency.set(file.path, new Set());
|
|
24
|
+
}
|
|
25
|
+
for (const edge of graph.edges) {
|
|
26
|
+
if (!changedByPath.has(edge.from) || !changedByPath.has(edge.to)) {
|
|
27
|
+
continue;
|
|
28
|
+
}
|
|
29
|
+
adjacency.get(edge.from)?.add(edge.to);
|
|
30
|
+
adjacency.get(edge.to)?.add(edge.from);
|
|
31
|
+
}
|
|
32
|
+
const visited = new Set();
|
|
33
|
+
const clusters = [];
|
|
34
|
+
for (const file of files) {
|
|
35
|
+
if (visited.has(file.path)) {
|
|
36
|
+
continue;
|
|
37
|
+
}
|
|
38
|
+
const paths = walkCluster(file.path, adjacency, visited).sort();
|
|
39
|
+
clusters.push({
|
|
40
|
+
paths,
|
|
41
|
+
roles: [...new Set(paths.map((path) => changedByPath.get(path)?.role ?? "unknown"))].sort()
|
|
42
|
+
});
|
|
43
|
+
}
|
|
44
|
+
return clusters.sort((left, right) => right.paths.length - left.paths.length || left.paths[0].localeCompare(right.paths[0]));
|
|
45
|
+
}
|
|
46
|
+
function walkCluster(path, adjacency, visited) {
|
|
47
|
+
const stack = [path];
|
|
48
|
+
const cluster = [];
|
|
49
|
+
while (stack.length > 0) {
|
|
50
|
+
const current = stack.pop();
|
|
51
|
+
if (current === undefined || visited.has(current)) {
|
|
52
|
+
continue;
|
|
53
|
+
}
|
|
54
|
+
visited.add(current);
|
|
55
|
+
cluster.push(current);
|
|
56
|
+
for (const next of adjacency.get(current) ?? []) {
|
|
57
|
+
if (!visited.has(next)) {
|
|
58
|
+
stack.push(next);
|
|
59
|
+
}
|
|
60
|
+
}
|
|
61
|
+
}
|
|
62
|
+
return cluster;
|
|
63
|
+
}
|
|
64
|
+
function toFinding(cluster, index) {
|
|
65
|
+
return {
|
|
66
|
+
id: `blast-radius:unexpected-cluster-${index}:${cluster.paths.join("|")}`,
|
|
67
|
+
detector: "blast-radius",
|
|
68
|
+
severity: getSeverity(cluster),
|
|
69
|
+
confidence: 0.74,
|
|
70
|
+
title: "Unexpected changed-file cluster",
|
|
71
|
+
message: `The diff includes a separate changed-file cluster (${cluster.paths.join(", ")}) outside the primary blast radius.`,
|
|
72
|
+
evidence: cluster.paths.map((path) => ({
|
|
73
|
+
kind: "file",
|
|
74
|
+
path,
|
|
75
|
+
message: `Cluster role(s): ${cluster.roles.join(", ")}.`,
|
|
76
|
+
data: {
|
|
77
|
+
clusterSize: cluster.paths.length,
|
|
78
|
+
roles: cluster.roles
|
|
79
|
+
}
|
|
80
|
+
})),
|
|
81
|
+
repair: "Confirm this separate cluster belongs to the current task, or split it into a separate task with explicit intent.",
|
|
82
|
+
tags: ["scope"]
|
|
83
|
+
};
|
|
84
|
+
}
|
|
85
|
+
function getSeverity(cluster) {
|
|
86
|
+
return cluster.roles.some((role) => role === "config" || role === "manifest" || role === "lockfile")
|
|
87
|
+
? "medium"
|
|
88
|
+
: "low";
|
|
89
|
+
}
|
|
90
|
+
//# sourceMappingURL=blast-radius-detector.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"blast-radius-detector.js","sourceRoot":"","sources":["../../src/detectors/blast-radius-detector.ts"],"names":[],"mappings":"AASA,MAAM,CAAC,MAAM,mBAAmB,GAAa;IAC3C,IAAI,EAAE,cAAc;IACpB,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE,EAAE;QACzB,MAAM,KAAK,GAAG,OAAO,EAAE,SAAS,EAAE,YAAY,EAAE,CAAC;QAEjD,IAAI,KAAK,KAAK,SAAS,IAAI,IAAI,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACjD,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,QAAQ,GAAG,mBAAmB,CAAC,IAAI,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC;QAExD,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACxB,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,cAAc,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC;QAEnC,OAAO,QAAQ;aACZ,KAAK,CAAC,CAAC,CAAC;aACR,MAAM,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,KAAK,cAAc,CAAC;aAC/C,GAAG,CAAC,CAAC,OAAO,EAAE,KAAK,EAAE,EAAE,CAAC,SAAS,CAAC,OAAO,EAAE,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC;IAC5D,CAAC;CACF,CAAC;AAEF,SAAS,mBAAmB,CAAC,KAAiB,EAAE,KAAgB;IAC9D,MAAM,aAAa,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC,CAAC;IACtE,MAAM,SAAS,GAAG,IAAI,GAAG,EAAuB,CAAC;IAEjD,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,EAAE,IAAI,GAAG,EAAE,CAAC,CAAC;IACtC,CAAC;IAED,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;QAC/B,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,CAAC;YACjE,SAAS;QACX,CAAC;QAED,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACvC,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACzC,CAAC;IAED,MAAM,OAAO,GAAG,IAAI,GAAG,EAAU,CAAC;IAClC,MAAM,QAAQ,GAAqB,EAAE,CAAC;IAEtC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YAC3B,SAAS;QACX,CAAC;QAED,MAAM,KAAK,GAAG,WAAW,CAAC,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,OAAO,CAAC,CAAC,IAAI,EAAE,CAAC;QAChE,QAAQ,CAAC,IAAI,CAAC;YACZ,KAAK;YACL,KAAK,EAAE,CAAC,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,IAAI,IAAI,SAAS,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE;SAC5F,CAAC,CAAC;IACL,CAAC;IAED,OAAO,QAAQ,CAAC,IAAI,CAClB,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CACd,KAAK,CAAC,KAAK,CAAC,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,IAAI,IAAI,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,aAAa,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAE,CAAC,CAC1F,CAAC;AACJ,CAAC;AAED,SAAS,WAAW,CAClB,IAAY,EACZ,SAAmC,EACnC,OAAoB;IAEpB,MAAM,KAAK,GAAG,CAAC,IAAI,CAAC,CAAC;IACrB,MAAM,OAAO,GAAa,EAAE,CAAC;IAE7B,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxB,MAAM,OAAO,GAAG,KAAK,CAAC,GAAG,EAAE,CAAC;QAE5B,IAAI,OAAO,KAAK,SAAS,IAAI,OAAO,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;YAClD,SAAS;QACX,CAAC;QAED,OAAO,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QACrB,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAEtB,KAAK,MAAM,IAAI,IAAI,SAAS,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC;YAChD,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;gBACvB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnB,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,SAAS,SAAS,CAAC,OAAuB,EAAE,KAAa;IACvD,OAAO;QACL,EAAE,EAAE,mCAAmC,KAAK,IAAI,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE;QACzE,QAAQ,EAAE,cAAc;QACxB,QAAQ,EAAE,WAAW,CAAC,OAAO,CAAC;QAC9B,UAAU,EAAE,IAAI;QAChB,KAAK,EAAE,iCAAiC;QACxC,OAAO,EAAE,sDAAsD,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,qCAAqC;QAC5H,QAAQ,EAAE,OAAO,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACrC,IAAI,EAAE,MAAM;YACZ,IAAI;YACJ,OAAO,EAAE,oBAAoB,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;YACxD,IAAI,EAAE;gBACJ,WAAW,EAAE,OAAO,CAAC,KAAK,CAAC,MAAM;gBACjC,KAAK,EAAE,OAAO,CAAC,KAAK;aACrB;SACF,CAAC,CAAC;QACH,MAAM,EACJ,mHAAmH;QACrH,IAAI,EAAE,CAAC,OAAO,CAAC;KAChB,CAAC;AACJ,CAAC;AAED,SAAS,WAAW,CAAC,OAAuB;IAC1C,OAAO,OAAO,CAAC,KAAK,CAAC,IAAI,CACvB,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,KAAK,QAAQ,IAAI,IAAI,KAAK,UAAU,IAAI,IAAI,KAAK,UAAU,CAC1E;QACC,CAAC,CAAC,QAAQ;QACV,CAAC,CAAC,KAAK,CAAC;AACZ,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"config-change-detector.d.ts","sourceRoot":"","sources":["../../src/detectors/config-change-detector.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAwB3C,eAAO,MAAM,oBAAoB,EAAE,QAWlC,CAAC"}
|
|
@@ -0,0 +1,68 @@
|
|
|
1
|
+
const configTaskTerms = [
|
|
2
|
+
"config",
|
|
3
|
+
"configuration",
|
|
4
|
+
"ci",
|
|
5
|
+
"workflow",
|
|
6
|
+
"github action",
|
|
7
|
+
"build",
|
|
8
|
+
"bundler",
|
|
9
|
+
"vite",
|
|
10
|
+
"webpack",
|
|
11
|
+
"typescript",
|
|
12
|
+
"tsconfig",
|
|
13
|
+
"eslint",
|
|
14
|
+
"prettier",
|
|
15
|
+
"docker",
|
|
16
|
+
"lint",
|
|
17
|
+
"test runner"
|
|
18
|
+
];
|
|
19
|
+
const documentationPathPattern = /(^|\/)(docs?|adr|changelog|changesets?)\/|(^|\/)(CHANGELOG|README|AGENTS|ADR)[^/]*\.md$/i;
|
|
20
|
+
export const configChangeDetector = {
|
|
21
|
+
name: "config-change",
|
|
22
|
+
run: ({ task, diff }) => {
|
|
23
|
+
const configFiles = diff.files.filter(isConfigChange);
|
|
24
|
+
if (configFiles.length === 0 || hasVisibleConfigExplanation(task, diff.files)) {
|
|
25
|
+
return [];
|
|
26
|
+
}
|
|
27
|
+
return configFiles.map((file) => toFinding(file, task));
|
|
28
|
+
}
|
|
29
|
+
};
|
|
30
|
+
function isConfigChange(file) {
|
|
31
|
+
return file.role === "config" && (file.additions > 0 || file.deletions > 0);
|
|
32
|
+
}
|
|
33
|
+
function hasVisibleConfigExplanation(task, files) {
|
|
34
|
+
const normalizedTask = task.text.toLowerCase();
|
|
35
|
+
return (configTaskTerms.some((term) => normalizedTask.includes(term)) ||
|
|
36
|
+
files.some((file) => documentationPathPattern.test(file.path)));
|
|
37
|
+
}
|
|
38
|
+
function toFinding(file, task) {
|
|
39
|
+
const firstChangedLine = file.hunks
|
|
40
|
+
.flatMap((hunk) => hunk.lines)
|
|
41
|
+
.find((line) => line.kind === "add" || line.kind === "delete");
|
|
42
|
+
const lineNumber = firstChangedLine?.newLineNumber ?? firstChangedLine?.oldLineNumber;
|
|
43
|
+
return {
|
|
44
|
+
id: `config-change:${file.path}`,
|
|
45
|
+
detector: "config-change",
|
|
46
|
+
severity: "medium",
|
|
47
|
+
confidence: 0.82,
|
|
48
|
+
title: "Config changed without visible explanation",
|
|
49
|
+
message: `${file.path} changed, but the task did not mention configuration and no documentation file changed.`,
|
|
50
|
+
evidence: [
|
|
51
|
+
{
|
|
52
|
+
kind: "file",
|
|
53
|
+
path: file.path,
|
|
54
|
+
startLine: lineNumber,
|
|
55
|
+
endLine: lineNumber,
|
|
56
|
+
message: `Config file changed during task: ${task.text}`,
|
|
57
|
+
data: {
|
|
58
|
+
additions: file.additions,
|
|
59
|
+
deletions: file.deletions,
|
|
60
|
+
role: file.role
|
|
61
|
+
}
|
|
62
|
+
}
|
|
63
|
+
],
|
|
64
|
+
repair: "Confirm the config change is required. If it changes team workflow or runtime behavior, add docs, an ADR, changelog entry, or explicit task/PR explanation.",
|
|
65
|
+
tags: ["config"]
|
|
66
|
+
};
|
|
67
|
+
}
|
|
68
|
+
//# sourceMappingURL=config-change-detector.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"config-change-detector.js","sourceRoot":"","sources":["../../src/detectors/config-change-detector.ts"],"names":[],"mappings":"AAIA,MAAM,eAAe,GAAG;IACtB,QAAQ;IACR,eAAe;IACf,IAAI;IACJ,UAAU;IACV,eAAe;IACf,OAAO;IACP,SAAS;IACT,MAAM;IACN,SAAS;IACT,YAAY;IACZ,UAAU;IACV,QAAQ;IACR,UAAU;IACV,QAAQ;IACR,MAAM;IACN,aAAa;CACd,CAAC;AAEF,MAAM,wBAAwB,GAC5B,0FAA0F,CAAC;AAE7F,MAAM,CAAC,MAAM,oBAAoB,GAAa;IAC5C,IAAI,EAAE,eAAe;IACrB,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE,EAAE;QACtB,MAAM,WAAW,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC;QAEtD,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,IAAI,2BAA2B,CAAC,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;YAC9E,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,OAAO,WAAW,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,SAAS,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC;IAC1D,CAAC;CACF,CAAC;AAEF,SAAS,cAAc,CAAC,IAAc;IACpC,OAAO,IAAI,CAAC,IAAI,KAAK,QAAQ,IAAI,CAAC,IAAI,CAAC,SAAS,GAAG,CAAC,IAAI,IAAI,CAAC,SAAS,GAAG,CAAC,CAAC,CAAC;AAC9E,CAAC;AAED,SAAS,2BAA2B,CAAC,IAAgB,EAAE,KAAiB;IACtE,MAAM,cAAc,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;IAE/C,OAAO,CACL,eAAe,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,cAAc,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAC7D,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,wBAAwB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAC/D,CAAC;AACJ,CAAC;AAED,SAAS,SAAS,CAAC,IAAc,EAAE,IAAgB;IACjD,MAAM,gBAAgB,GAAG,IAAI,CAAC,KAAK;SAChC,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC;SAC7B,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,KAAK,KAAK,IAAI,IAAI,CAAC,IAAI,KAAK,QAAQ,CAAC,CAAC;IACjE,MAAM,UAAU,GAAG,gBAAgB,EAAE,aAAa,IAAI,gBAAgB,EAAE,aAAa,CAAC;IAEtF,OAAO;QACL,EAAE,EAAE,iBAAiB,IAAI,CAAC,IAAI,EAAE;QAChC,QAAQ,EAAE,eAAe;QACzB,QAAQ,EAAE,QAAQ;QAClB,UAAU,EAAE,IAAI;QAChB,KAAK,EAAE,4CAA4C;QACnD,OAAO,EAAE,GAAG,IAAI,CAAC,IAAI,yFAAyF;QAC9G,QAAQ,EAAE;YACR;gBACE,IAAI,EAAE,MAAM;gBACZ,IAAI,EAAE,IAAI,CAAC,IAAI;gBACf,SAAS,EAAE,UAAU;gBACrB,OAAO,EAAE,UAAU;gBACnB,OAAO,EAAE,oCAAoC,IAAI,CAAC,IAAI,EAAE;gBACxD,IAAI,EAAE;oBACJ,SAAS,EAAE,IAAI,CAAC,SAAS;oBACzB,SAAS,EAAE,IAAI,CAAC,SAAS;oBACzB,IAAI,EAAE,IAAI,CAAC,IAAI;iBAChB;aACF;SACF;QACD,MAAM,EACJ,6JAA6J;QAC/J,IAAI,EAAE,CAAC,QAAQ,CAAC;KACjB,CAAC;AACJ,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"dependency-detector.d.ts","sourceRoot":"","sources":["../../src/detectors/dependency-detector.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AA2B3C,eAAO,MAAM,kBAAkB,EAAE,QAShC,CAAC"}
|
|
@@ -0,0 +1,98 @@
|
|
|
1
|
+
const dependencySections = new Set([
|
|
2
|
+
"dependencies",
|
|
3
|
+
"devDependencies",
|
|
4
|
+
"peerDependencies",
|
|
5
|
+
"optionalDependencies"
|
|
6
|
+
]);
|
|
7
|
+
const dependencyLinePattern = /^\s*"([^"]+)":\s*"([^"]+)"/;
|
|
8
|
+
const sectionLinePattern = /^\s*"(dependencies|devDependencies|peerDependencies|optionalDependencies)":\s*\{/;
|
|
9
|
+
export const dependencyDetector = {
|
|
10
|
+
name: "dependency-addition",
|
|
11
|
+
run: ({ task, diff }) => {
|
|
12
|
+
const addedDependencies = diff.files.flatMap(extractAddedDependencies);
|
|
13
|
+
return addedDependencies
|
|
14
|
+
.filter((dependency) => !hasVisibleJustification(task.text, dependency.name))
|
|
15
|
+
.map(toFinding);
|
|
16
|
+
}
|
|
17
|
+
};
|
|
18
|
+
function extractAddedDependencies(file) {
|
|
19
|
+
if (file.path !== "package.json" && !file.path.endsWith("/package.json")) {
|
|
20
|
+
return [];
|
|
21
|
+
}
|
|
22
|
+
const dependencies = [];
|
|
23
|
+
for (const hunk of file.hunks) {
|
|
24
|
+
let currentSection;
|
|
25
|
+
for (const line of hunk.lines) {
|
|
26
|
+
const section = getSection(line);
|
|
27
|
+
if (section !== undefined) {
|
|
28
|
+
currentSection = section;
|
|
29
|
+
continue;
|
|
30
|
+
}
|
|
31
|
+
if (line.kind === "context" && line.content.trim() === "}") {
|
|
32
|
+
currentSection = undefined;
|
|
33
|
+
continue;
|
|
34
|
+
}
|
|
35
|
+
if (line.kind !== "add" || currentSection === undefined) {
|
|
36
|
+
continue;
|
|
37
|
+
}
|
|
38
|
+
const dependency = dependencyLinePattern.exec(line.content);
|
|
39
|
+
if (dependency === null) {
|
|
40
|
+
continue;
|
|
41
|
+
}
|
|
42
|
+
dependencies.push({
|
|
43
|
+
name: dependency[1] ?? "",
|
|
44
|
+
version: dependency[2] ?? "",
|
|
45
|
+
section: currentSection,
|
|
46
|
+
path: file.path,
|
|
47
|
+
lineNumber: line.newLineNumber
|
|
48
|
+
});
|
|
49
|
+
}
|
|
50
|
+
}
|
|
51
|
+
return dependencies;
|
|
52
|
+
}
|
|
53
|
+
function getSection(line) {
|
|
54
|
+
const section = sectionLinePattern.exec(line.content)?.[1];
|
|
55
|
+
if (section === undefined || !dependencySections.has(section)) {
|
|
56
|
+
return undefined;
|
|
57
|
+
}
|
|
58
|
+
return section;
|
|
59
|
+
}
|
|
60
|
+
function hasVisibleJustification(taskText, dependencyName) {
|
|
61
|
+
const normalizedTask = taskText.toLowerCase();
|
|
62
|
+
const normalizedDependency = dependencyName.toLowerCase();
|
|
63
|
+
return (normalizedTask.includes(normalizedDependency) ||
|
|
64
|
+
normalizedTask.includes("add dependency") ||
|
|
65
|
+
normalizedTask.includes("new dependency") ||
|
|
66
|
+
normalizedTask.includes("install package") ||
|
|
67
|
+
normalizedTask.includes("install dependency"));
|
|
68
|
+
}
|
|
69
|
+
function toFinding(dependency) {
|
|
70
|
+
const isProductionDependency = dependency.section === "dependencies" || dependency.section === "optionalDependencies";
|
|
71
|
+
const severity = isProductionDependency ? "blocker" : "medium";
|
|
72
|
+
const dependencyType = isProductionDependency ? "production" : "development";
|
|
73
|
+
return {
|
|
74
|
+
id: `dependency-addition:${dependency.path}:${dependency.section}:${dependency.name}`,
|
|
75
|
+
detector: "dependency-addition",
|
|
76
|
+
severity,
|
|
77
|
+
confidence: 0.9,
|
|
78
|
+
title: `Unjustified ${dependencyType} dependency added`,
|
|
79
|
+
message: `${dependency.name}@${dependency.version} was added to ${dependency.section} without visible task justification.`,
|
|
80
|
+
evidence: [
|
|
81
|
+
{
|
|
82
|
+
kind: "manifest",
|
|
83
|
+
path: dependency.path,
|
|
84
|
+
startLine: dependency.lineNumber,
|
|
85
|
+
endLine: dependency.lineNumber,
|
|
86
|
+
message: `${dependency.name} was added to ${dependency.section}.`,
|
|
87
|
+
data: {
|
|
88
|
+
dependency: dependency.name,
|
|
89
|
+
version: dependency.version,
|
|
90
|
+
section: dependency.section
|
|
91
|
+
}
|
|
92
|
+
}
|
|
93
|
+
],
|
|
94
|
+
repair: `Remove ${dependency.name} unless it is required, or update the task/PR context with a clear justification and alternatives considered.`,
|
|
95
|
+
tags: ["dependency"]
|
|
96
|
+
};
|
|
97
|
+
}
|
|
98
|
+
//# sourceMappingURL=dependency-detector.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"dependency-detector.js","sourceRoot":"","sources":["../../src/detectors/dependency-detector.ts"],"names":[],"mappings":"AAkBA,MAAM,kBAAkB,GAAG,IAAI,GAAG,CAAoB;IACpD,cAAc;IACd,iBAAiB;IACjB,kBAAkB;IAClB,sBAAsB;CACvB,CAAC,CAAC;AAEH,MAAM,qBAAqB,GAAG,4BAA4B,CAAC;AAC3D,MAAM,kBAAkB,GACtB,kFAAkF,CAAC;AAErF,MAAM,CAAC,MAAM,kBAAkB,GAAa;IAC1C,IAAI,EAAE,qBAAqB;IAC3B,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE,EAAE;QACtB,MAAM,iBAAiB,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,wBAAwB,CAAC,CAAC;QAEvE,OAAO,iBAAiB;aACrB,MAAM,CAAC,CAAC,UAAU,EAAE,EAAE,CAAC,CAAC,uBAAuB,CAAC,IAAI,CAAC,IAAI,EAAE,UAAU,CAAC,IAAI,CAAC,CAAC;aAC5E,GAAG,CAAC,SAAS,CAAC,CAAC;IACpB,CAAC;CACF,CAAC;AAEF,SAAS,wBAAwB,CAAC,IAAc;IAC9C,IAAI,IAAI,CAAC,IAAI,KAAK,cAAc,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,eAAe,CAAC,EAAE,CAAC;QACzE,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,MAAM,YAAY,GAAsB,EAAE,CAAC;IAE3C,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,EAAE,CAAC;QAC9B,IAAI,cAA6C,CAAC;QAElD,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,EAAE,CAAC;YAC9B,MAAM,OAAO,GAAG,UAAU,CAAC,IAAI,CAAC,CAAC;YAEjC,IAAI,OAAO,KAAK,SAAS,EAAE,CAAC;gBAC1B,cAAc,GAAG,OAAO,CAAC;gBACzB,SAAS;YACX,CAAC;YAED,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;gBAC3D,cAAc,GAAG,SAAS,CAAC;gBAC3B,SAAS;YACX,CAAC;YAED,IAAI,IAAI,CAAC,IAAI,KAAK,KAAK,IAAI,cAAc,KAAK,SAAS,EAAE,CAAC;gBACxD,SAAS;YACX,CAAC;YAED,MAAM,UAAU,GAAG,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;YAE5D,IAAI,UAAU,KAAK,IAAI,EAAE,CAAC;gBACxB,SAAS;YACX,CAAC;YAED,YAAY,CAAC,IAAI,CAAC;gBAChB,IAAI,EAAE,UAAU,CAAC,CAAC,CAAC,IAAI,EAAE;gBACzB,OAAO,EAAE,UAAU,CAAC,CAAC,CAAC,IAAI,EAAE;gBAC5B,OAAO,EAAE,cAAc;gBACvB,IAAI,EAAE,IAAI,CAAC,IAAI;gBACf,UAAU,EAAE,IAAI,CAAC,aAAa;aAC/B,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,OAAO,YAAY,CAAC;AACtB,CAAC;AAED,SAAS,UAAU,CAAC,IAAc;IAChC,MAAM,OAAO,GAAG,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;IAE3D,IAAI,OAAO,KAAK,SAAS,IAAI,CAAC,kBAAkB,CAAC,GAAG,CAAC,OAA4B,CAAC,EAAE,CAAC;QACnF,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,OAAO,OAA4B,CAAC;AACtC,CAAC;AAED,SAAS,uBAAuB,CAAC,QAAgB,EAAE,cAAsB;IACvE,MAAM,cAAc,GAAG,QAAQ,CAAC,WAAW,EAAE,CAAC;IAC9C,MAAM,oBAAoB,GAAG,cAAc,CAAC,WAAW,EAAE,CAAC;IAE1D,OAAO,CACL,cAAc,CAAC,QAAQ,CAAC,oBAAoB,CAAC;QAC7C,cAAc,CAAC,QAAQ,CAAC,gBAAgB,CAAC;QACzC,cAAc,CAAC,QAAQ,CAAC,gBAAgB,CAAC;QACzC,cAAc,CAAC,QAAQ,CAAC,iBAAiB,CAAC;QAC1C,cAAc,CAAC,QAAQ,CAAC,oBAAoB,CAAC,CAC9C,CAAC;AACJ,CAAC;AAED,SAAS,SAAS,CAAC,UAA2B;IAC5C,MAAM,sBAAsB,GAC1B,UAAU,CAAC,OAAO,KAAK,cAAc,IAAI,UAAU,CAAC,OAAO,KAAK,sBAAsB,CAAC;IACzF,MAAM,QAAQ,GAAG,sBAAsB,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,QAAQ,CAAC;IAC/D,MAAM,cAAc,GAAG,sBAAsB,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,aAAa,CAAC;IAE7E,OAAO;QACL,EAAE,EAAE,uBAAuB,UAAU,CAAC,IAAI,IAAI,UAAU,CAAC,OAAO,IAAI,UAAU,CAAC,IAAI,EAAE;QACrF,QAAQ,EAAE,qBAAqB;QAC/B,QAAQ;QACR,UAAU,EAAE,GAAG;QACf,KAAK,EAAE,eAAe,cAAc,mBAAmB;QACvD,OAAO,EAAE,GAAG,UAAU,CAAC,IAAI,IAAI,UAAU,CAAC,OAAO,iBAAiB,UAAU,CAAC,OAAO,sCAAsC;QAC1H,QAAQ,EAAE;YACR;gBACE,IAAI,EAAE,UAAU;gBAChB,IAAI,EAAE,UAAU,CAAC,IAAI;gBACrB,SAAS,EAAE,UAAU,CAAC,UAAU;gBAChC,OAAO,EAAE,UAAU,CAAC,UAAU;gBAC9B,OAAO,EAAE,GAAG,UAAU,CAAC,IAAI,iBAAiB,UAAU,CAAC,OAAO,GAAG;gBACjE,IAAI,EAAE;oBACJ,UAAU,EAAE,UAAU,CAAC,IAAI;oBAC3B,OAAO,EAAE,UAAU,CAAC,OAAO;oBAC3B,OAAO,EAAE,UAAU,CAAC,OAAO;iBAC5B;aACF;SACF;QACD,MAAM,EAAE,UAAU,UAAU,CAAC,IAAI,+GAA+G;QAChJ,IAAI,EAAE,CAAC,YAAY,CAAC;KACrB,CAAC;AACJ,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"existing-solution-detector.d.ts","sourceRoot":"","sources":["../../src/detectors/existing-solution-detector.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAS3C,eAAO,MAAM,wBAAwB,EAAE,QAatC,CAAC"}
|
|
@@ -0,0 +1,87 @@
|
|
|
1
|
+
import { extractSolutionEntries } from "../knowledge/index.js";
|
|
2
|
+
export const existingSolutionDetector = {
|
|
3
|
+
name: "existing-solution",
|
|
4
|
+
run: ({ diff, context }) => {
|
|
5
|
+
const existingSolutions = context?.knowledge?.getSolutionIndex().solutions ?? [];
|
|
6
|
+
if (existingSolutions.length === 0) {
|
|
7
|
+
return [];
|
|
8
|
+
}
|
|
9
|
+
return diff.files
|
|
10
|
+
.filter((file) => file.status === "added" && file.role === "source")
|
|
11
|
+
.flatMap((file) => detectExistingSolutions(file, existingSolutions));
|
|
12
|
+
}
|
|
13
|
+
};
|
|
14
|
+
function detectExistingSolutions(file, existingSolutions) {
|
|
15
|
+
const sourceText = file.hunks
|
|
16
|
+
.flatMap((hunk) => hunk.lines.filter((line) => line.kind === "add").map((line) => line.content))
|
|
17
|
+
.join("\n");
|
|
18
|
+
const addedSolutions = extractSolutionEntries(file.path, sourceText);
|
|
19
|
+
return addedSolutions.flatMap((added) => {
|
|
20
|
+
const match = findBestMatch(added, existingSolutions);
|
|
21
|
+
return match === undefined ? [] : [toFinding(file, match)];
|
|
22
|
+
});
|
|
23
|
+
}
|
|
24
|
+
function findBestMatch(added, existingSolutions) {
|
|
25
|
+
return existingSolutions
|
|
26
|
+
.filter((existing) => existing.path !== added.path && existing.class === added.class)
|
|
27
|
+
.map((existing) => scoreMatch(added, existing))
|
|
28
|
+
.filter((match) => match !== undefined)
|
|
29
|
+
.sort((left, right) => right.confidence - left.confidence)[0];
|
|
30
|
+
}
|
|
31
|
+
function scoreMatch(added, existing) {
|
|
32
|
+
if (added.normalizedName === existing.normalizedName) {
|
|
33
|
+
return { added, existing, confidence: 0.94, reason: "exact normalized name match" };
|
|
34
|
+
}
|
|
35
|
+
if (added.normalizedName.includes(existing.normalizedName) ||
|
|
36
|
+
existing.normalizedName.includes(added.normalizedName)) {
|
|
37
|
+
return { added, existing, confidence: 0.84, reason: "similar normalized name" };
|
|
38
|
+
}
|
|
39
|
+
const domainOverlap = overlapRatio(added.domainTokens, existing.domainTokens);
|
|
40
|
+
const importOverlap = overlapRatio(added.importTokens, existing.importTokens);
|
|
41
|
+
const signatureMatches = added.arity !== undefined &&
|
|
42
|
+
added.arity === existing.arity &&
|
|
43
|
+
added.returnType === existing.returnType;
|
|
44
|
+
if (signatureMatches && domainOverlap >= 0.4) {
|
|
45
|
+
return { added, existing, confidence: 0.78, reason: "matching signature and domain tokens" };
|
|
46
|
+
}
|
|
47
|
+
if (domainOverlap >= 0.5 && importOverlap >= 0.5) {
|
|
48
|
+
return { added, existing, confidence: 0.74, reason: "similar imports and domain tokens" };
|
|
49
|
+
}
|
|
50
|
+
return undefined;
|
|
51
|
+
}
|
|
52
|
+
function overlapRatio(left, right) {
|
|
53
|
+
if (left.length === 0 || right.length === 0) {
|
|
54
|
+
return 0;
|
|
55
|
+
}
|
|
56
|
+
const rightSet = new Set(right);
|
|
57
|
+
const overlap = left.filter((token) => rightSet.has(token)).length;
|
|
58
|
+
return overlap / Math.max(left.length, right.length);
|
|
59
|
+
}
|
|
60
|
+
function toFinding(file, match) {
|
|
61
|
+
return {
|
|
62
|
+
id: `existing-solution:${file.path}:${match.added.exportedName ?? match.added.normalizedName}`,
|
|
63
|
+
detector: "existing-solution",
|
|
64
|
+
severity: "medium",
|
|
65
|
+
confidence: match.confidence,
|
|
66
|
+
title: "New solution may duplicate existing repository solution",
|
|
67
|
+
message: `${match.added.exportedName ?? match.added.normalizedName} was added in ${file.path}, but ${match.existing.exportedName ?? match.existing.normalizedName} already exists in ${match.existing.path}.`,
|
|
68
|
+
evidence: [
|
|
69
|
+
{
|
|
70
|
+
kind: "symbol",
|
|
71
|
+
path: file.path,
|
|
72
|
+
symbol: match.added.exportedName,
|
|
73
|
+
message: `Existing ${match.existing.class} candidate: ${match.existing.path}#${match.existing.exportedName ?? match.existing.normalizedName} (${match.reason}).`,
|
|
74
|
+
data: {
|
|
75
|
+
addedClass: match.added.class,
|
|
76
|
+
existingClass: match.existing.class,
|
|
77
|
+
existingPath: match.existing.path,
|
|
78
|
+
existingExport: match.existing.exportedName,
|
|
79
|
+
reason: match.reason
|
|
80
|
+
}
|
|
81
|
+
}
|
|
82
|
+
],
|
|
83
|
+
repair: `Reuse ${match.existing.exportedName ?? "the existing solution"} from ${match.existing.path}, or document why a separate ${match.added.class} is needed.`,
|
|
84
|
+
tags: ["utility", "convention"]
|
|
85
|
+
};
|
|
86
|
+
}
|
|
87
|
+
//# sourceMappingURL=existing-solution-detector.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"existing-solution-detector.js","sourceRoot":"","sources":["../../src/detectors/existing-solution-detector.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,sBAAsB,EAAsB,MAAM,uBAAuB,CAAC;AAWnF,MAAM,CAAC,MAAM,wBAAwB,GAAa;IAChD,IAAI,EAAE,mBAAmB;IACzB,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE,EAAE;QACzB,MAAM,iBAAiB,GAAG,OAAO,EAAE,SAAS,EAAE,gBAAgB,EAAE,CAAC,SAAS,IAAI,EAAE,CAAC;QAEjF,IAAI,iBAAiB,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACnC,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,OAAO,IAAI,CAAC,KAAK;aACd,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,MAAM,KAAK,OAAO,IAAI,IAAI,CAAC,IAAI,KAAK,QAAQ,CAAC;aACnE,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,uBAAuB,CAAC,IAAI,EAAE,iBAAiB,CAAC,CAAC,CAAC;IACzE,CAAC;CACF,CAAC;AAEF,SAAS,uBAAuB,CAAC,IAAc,EAAE,iBAAkC;IACjF,MAAM,UAAU,GAAG,IAAI,CAAC,KAAK;SAC1B,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,KAAK,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;SAC/F,IAAI,CAAC,IAAI,CAAC,CAAC;IACd,MAAM,cAAc,GAAG,sBAAsB,CAAC,IAAI,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;IAErE,OAAO,cAAc,CAAC,OAAO,CAAC,CAAC,KAAK,EAAE,EAAE;QACtC,MAAM,KAAK,GAAG,aAAa,CAAC,KAAK,EAAE,iBAAiB,CAAC,CAAC;QACtD,OAAO,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;AACL,CAAC;AAED,SAAS,aAAa,CACpB,KAAoB,EACpB,iBAAkC;IAElC,OAAO,iBAAiB;SACrB,MAAM,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,KAAK,KAAK,CAAC,IAAI,IAAI,QAAQ,CAAC,KAAK,KAAK,KAAK,CAAC,KAAK,CAAC;SACpF,GAAG,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC,UAAU,CAAC,KAAK,EAAE,QAAQ,CAAC,CAAC;SAC9C,MAAM,CAAC,CAAC,KAAK,EAA0B,EAAE,CAAC,KAAK,KAAK,SAAS,CAAC;SAC9D,IAAI,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,UAAU,GAAG,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC;AAClE,CAAC;AAED,SAAS,UAAU,CAAC,KAAoB,EAAE,QAAuB;IAC/D,IAAI,KAAK,CAAC,cAAc,KAAK,QAAQ,CAAC,cAAc,EAAE,CAAC;QACrD,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,UAAU,EAAE,IAAI,EAAE,MAAM,EAAE,6BAA6B,EAAE,CAAC;IACtF,CAAC;IAED,IACE,KAAK,CAAC,cAAc,CAAC,QAAQ,CAAC,QAAQ,CAAC,cAAc,CAAC;QACtD,QAAQ,CAAC,cAAc,CAAC,QAAQ,CAAC,KAAK,CAAC,cAAc,CAAC,EACtD,CAAC;QACD,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,UAAU,EAAE,IAAI,EAAE,MAAM,EAAE,yBAAyB,EAAE,CAAC;IAClF,CAAC;IAED,MAAM,aAAa,GAAG,YAAY,CAAC,KAAK,CAAC,YAAY,EAAE,QAAQ,CAAC,YAAY,CAAC,CAAC;IAC9E,MAAM,aAAa,GAAG,YAAY,CAAC,KAAK,CAAC,YAAY,EAAE,QAAQ,CAAC,YAAY,CAAC,CAAC;IAC9E,MAAM,gBAAgB,GACpB,KAAK,CAAC,KAAK,KAAK,SAAS;QACzB,KAAK,CAAC,KAAK,KAAK,QAAQ,CAAC,KAAK;QAC9B,KAAK,CAAC,UAAU,KAAK,QAAQ,CAAC,UAAU,CAAC;IAE3C,IAAI,gBAAgB,IAAI,aAAa,IAAI,GAAG,EAAE,CAAC;QAC7C,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,UAAU,EAAE,IAAI,EAAE,MAAM,EAAE,sCAAsC,EAAE,CAAC;IAC/F,CAAC;IAED,IAAI,aAAa,IAAI,GAAG,IAAI,aAAa,IAAI,GAAG,EAAE,CAAC;QACjD,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,UAAU,EAAE,IAAI,EAAE,MAAM,EAAE,mCAAmC,EAAE,CAAC;IAC5F,CAAC;IAED,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,SAAS,YAAY,CAAC,IAAc,EAAE,KAAe;IACnD,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC5C,OAAO,CAAC,CAAC;IACX,CAAC;IAED,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,CAAC;IAChC,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC;IACnE,OAAO,OAAO,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,MAAM,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC;AACvD,CAAC;AAED,SAAS,SAAS,CAAC,IAAc,EAAE,KAAoB;IACrD,OAAO;QACL,EAAE,EAAE,qBAAqB,IAAI,CAAC,IAAI,IAAI,KAAK,CAAC,KAAK,CAAC,YAAY,IAAI,KAAK,CAAC,KAAK,CAAC,cAAc,EAAE;QAC9F,QAAQ,EAAE,mBAAmB;QAC7B,QAAQ,EAAE,QAAQ;QAClB,UAAU,EAAE,KAAK,CAAC,UAAU;QAC5B,KAAK,EAAE,yDAAyD;QAChE,OAAO,EAAE,GAAG,KAAK,CAAC,KAAK,CAAC,YAAY,IAAI,KAAK,CAAC,KAAK,CAAC,cAAc,iBAAiB,IAAI,CAAC,IAAI,SAAS,KAAK,CAAC,QAAQ,CAAC,YAAY,IAAI,KAAK,CAAC,QAAQ,CAAC,cAAc,sBAAsB,KAAK,CAAC,QAAQ,CAAC,IAAI,GAAG;QAC7M,QAAQ,EAAE;YACR;gBACE,IAAI,EAAE,QAAQ;gBACd,IAAI,EAAE,IAAI,CAAC,IAAI;gBACf,MAAM,EAAE,KAAK,CAAC,KAAK,CAAC,YAAY;gBAChC,OAAO,EAAE,YAAY,KAAK,CAAC,QAAQ,CAAC,KAAK,eAAe,KAAK,CAAC,QAAQ,CAAC,IAAI,IAAI,KAAK,CAAC,QAAQ,CAAC,YAAY,IAAI,KAAK,CAAC,QAAQ,CAAC,cAAc,KAAK,KAAK,CAAC,MAAM,IAAI;gBAChK,IAAI,EAAE;oBACJ,UAAU,EAAE,KAAK,CAAC,KAAK,CAAC,KAAK;oBAC7B,aAAa,EAAE,KAAK,CAAC,QAAQ,CAAC,KAAK;oBACnC,YAAY,EAAE,KAAK,CAAC,QAAQ,CAAC,IAAI;oBACjC,cAAc,EAAE,KAAK,CAAC,QAAQ,CAAC,YAAY;oBAC3C,MAAM,EAAE,KAAK,CAAC,MAAM;iBACrB;aACF;SACF;QACD,MAAM,EAAE,SAAS,KAAK,CAAC,QAAQ,CAAC,YAAY,IAAI,uBAAuB,SAAS,KAAK,CAAC,QAAQ,CAAC,IAAI,gCAAgC,KAAK,CAAC,KAAK,CAAC,KAAK,aAAa;QACjK,IAAI,EAAE,CAAC,SAAS,EAAE,YAAY,CAAC;KAChC,CAAC;AACJ,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"expected-companions-detector.d.ts","sourceRoot":"","sources":["../../src/detectors/expected-companions-detector.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAE3C,eAAO,MAAM,0BAA0B,EAAE,QAiBxC,CAAC"}
|
|
@@ -0,0 +1,65 @@
|
|
|
1
|
+
export const expectedCompanionsDetector = {
|
|
2
|
+
name: "expected-companions",
|
|
3
|
+
run: ({ diff, context }) => {
|
|
4
|
+
const history = context?.knowledge?.getHistoryIndex();
|
|
5
|
+
if (history === undefined) {
|
|
6
|
+
return [];
|
|
7
|
+
}
|
|
8
|
+
const changedPaths = new Set(diff.files.map((file) => file.path));
|
|
9
|
+
const historyFindings = history.companionRules
|
|
10
|
+
.filter((rule) => changedPaths.has(rule.sourcePath) && !changedPaths.has(rule.expectedPath))
|
|
11
|
+
.map((rule) => toHistoryFinding(rule));
|
|
12
|
+
const packageFinding = detectMissingLockfile(diff.files);
|
|
13
|
+
return packageFinding === undefined ? historyFindings : [...historyFindings, packageFinding];
|
|
14
|
+
}
|
|
15
|
+
};
|
|
16
|
+
function detectMissingLockfile(files) {
|
|
17
|
+
const changedPaths = new Set(files.map((file) => file.path));
|
|
18
|
+
const changedPackage = files.find((file) => file.path === "package.json" || file.path.endsWith("/package.json"));
|
|
19
|
+
if (changedPackage === undefined ||
|
|
20
|
+
[...changedPaths].some((path) => /(^|\/)(pnpm-lock\.yaml|package-lock\.json|yarn\.lock|bun\.lockb)$/.test(path))) {
|
|
21
|
+
return undefined;
|
|
22
|
+
}
|
|
23
|
+
return {
|
|
24
|
+
id: `expected-companions:${changedPackage.path}:lockfile`,
|
|
25
|
+
detector: "expected-companions",
|
|
26
|
+
severity: "medium",
|
|
27
|
+
confidence: 0.86,
|
|
28
|
+
title: "Expected companion lockfile missing",
|
|
29
|
+
message: `${changedPackage.path} changed without a corresponding package lockfile change.`,
|
|
30
|
+
evidence: [
|
|
31
|
+
{
|
|
32
|
+
kind: "file",
|
|
33
|
+
path: changedPackage.path,
|
|
34
|
+
message: "Package manifest changed but no lockfile changed in this diff."
|
|
35
|
+
}
|
|
36
|
+
],
|
|
37
|
+
repair: "Update the matching lockfile, or document why the manifest-only change is intentional.",
|
|
38
|
+
tags: ["dependency"]
|
|
39
|
+
};
|
|
40
|
+
}
|
|
41
|
+
function toHistoryFinding(rule) {
|
|
42
|
+
return {
|
|
43
|
+
id: `expected-companions:${rule.sourcePath}:${rule.expectedPath}`,
|
|
44
|
+
detector: "expected-companions",
|
|
45
|
+
severity: "medium",
|
|
46
|
+
confidence: Math.min(0.92, 0.55 + rule.confidence * 0.35),
|
|
47
|
+
title: "Expected companion file missing",
|
|
48
|
+
message: `${rule.sourcePath} changed without historically paired companion ${rule.expectedPath}.`,
|
|
49
|
+
evidence: [
|
|
50
|
+
{
|
|
51
|
+
kind: "history",
|
|
52
|
+
path: rule.sourcePath,
|
|
53
|
+
message: `${rule.expectedPath} has historically changed with ${rule.sourcePath}.`,
|
|
54
|
+
data: {
|
|
55
|
+
expectedPath: rule.expectedPath,
|
|
56
|
+
support: rule.support,
|
|
57
|
+
confidence: rule.confidence
|
|
58
|
+
}
|
|
59
|
+
}
|
|
60
|
+
],
|
|
61
|
+
repair: `Update ${rule.expectedPath}, or document why this change does not need its usual companion.`,
|
|
62
|
+
tags: ["scope"]
|
|
63
|
+
};
|
|
64
|
+
}
|
|
65
|
+
//# sourceMappingURL=expected-companions-detector.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"expected-companions-detector.js","sourceRoot":"","sources":["../../src/detectors/expected-companions-detector.ts"],"names":[],"mappings":"AAIA,MAAM,CAAC,MAAM,0BAA0B,GAAa;IAClD,IAAI,EAAE,qBAAqB;IAC3B,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE,EAAE;QACzB,MAAM,OAAO,GAAG,OAAO,EAAE,SAAS,EAAE,eAAe,EAAE,CAAC;QAEtD,IAAI,OAAO,KAAK,SAAS,EAAE,CAAC;YAC1B,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;QAClE,MAAM,eAAe,GAAG,OAAO,CAAC,cAAc;aAC3C,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;aAC3F,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC;QACzC,MAAM,cAAc,GAAG,qBAAqB,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAEzD,OAAO,cAAc,KAAK,SAAS,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,GAAG,eAAe,EAAE,cAAc,CAAC,CAAC;IAC/F,CAAC;CACF,CAAC;AAEF,SAAS,qBAAqB,CAAC,KAAiB;IAC9C,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;IAC7D,MAAM,cAAc,GAAG,KAAK,CAAC,IAAI,CAC/B,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,KAAK,cAAc,IAAI,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,eAAe,CAAC,CAC9E,CAAC;IAEF,IACE,cAAc,KAAK,SAAS;QAC5B,CAAC,GAAG,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAC9B,mEAAmE,CAAC,IAAI,CAAC,IAAI,CAAC,CAC/E,EACD,CAAC;QACD,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,OAAO;QACL,EAAE,EAAE,uBAAuB,cAAc,CAAC,IAAI,WAAW;QACzD,QAAQ,EAAE,qBAAqB;QAC/B,QAAQ,EAAE,QAAQ;QAClB,UAAU,EAAE,IAAI;QAChB,KAAK,EAAE,qCAAqC;QAC5C,OAAO,EAAE,GAAG,cAAc,CAAC,IAAI,2DAA2D;QAC1F,QAAQ,EAAE;YACR;gBACE,IAAI,EAAE,MAAM;gBACZ,IAAI,EAAE,cAAc,CAAC,IAAI;gBACzB,OAAO,EAAE,gEAAgE;aAC1E;SACF;QACD,MAAM,EACJ,wFAAwF;QAC1F,IAAI,EAAE,CAAC,YAAY,CAAC;KACrB,CAAC;AACJ,CAAC;AAED,SAAS,gBAAgB,CAAC,IAAmB;IAC3C,OAAO;QACL,EAAE,EAAE,uBAAuB,IAAI,CAAC,UAAU,IAAI,IAAI,CAAC,YAAY,EAAE;QACjE,QAAQ,EAAE,qBAAqB;QAC/B,QAAQ,EAAE,QAAQ;QAClB,UAAU,EAAE,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,GAAG,IAAI,CAAC,UAAU,GAAG,IAAI,CAAC;QACzD,KAAK,EAAE,iCAAiC;QACxC,OAAO,EAAE,GAAG,IAAI,CAAC,UAAU,kDAAkD,IAAI,CAAC,YAAY,GAAG;QACjG,QAAQ,EAAE;YACR;gBACE,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,IAAI,CAAC,UAAU;gBACrB,OAAO,EAAE,GAAG,IAAI,CAAC,YAAY,kCAAkC,IAAI,CAAC,UAAU,GAAG;gBACjF,IAAI,EAAE;oBACJ,YAAY,EAAE,IAAI,CAAC,YAAY;oBAC/B,OAAO,EAAE,IAAI,CAAC,OAAO;oBACrB,UAAU,EAAE,IAAI,CAAC,UAAU;iBAC5B;aACF;SACF;QACD,MAAM,EAAE,UAAU,IAAI,CAAC,YAAY,kEAAkE;QACrG,IAAI,EAAE,CAAC,OAAO,CAAC;KAChB,CAAC;AACJ,CAAC"}
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
export { apiSurfaceDetector } from "./api-surface-detector.js";
|
|
2
|
+
export { blastRadiusDetector } from "./blast-radius-detector.js";
|
|
3
|
+
export { configChangeDetector } from "./config-change-detector.js";
|
|
4
|
+
export { dependencyDetector } from "./dependency-detector.js";
|
|
5
|
+
export { existingSolutionDetector } from "./existing-solution-detector.js";
|
|
6
|
+
export { expectedCompanionsDetector } from "./expected-companions-detector.js";
|
|
7
|
+
export { intentVerificationDetector } from "./intent-verification-detector.js";
|
|
8
|
+
export { patternViolationDetector } from "./pattern-violation-detector.js";
|
|
9
|
+
export { rewriteDetector } from "./rewrite-detector.js";
|
|
10
|
+
export { repositoryIntelligenceDetector } from "./repository-intelligence-detector.js";
|
|
11
|
+
export { runDetectors, summarizeFindings, type FindingDecisionPolicy } from "./runner.js";
|
|
12
|
+
export { scanAddedLinesForSecretsAndPaths, secretPathDetector } from "./secret-path-detector.js";
|
|
13
|
+
export { scopeDetector } from "./scope-detector.js";
|
|
14
|
+
export { testWeakeningDetector } from "./test-weakening-detector.js";
|
|
15
|
+
export { utilityReinventionDetector } from "./utility-reinvention-detector.js";
|
|
16
|
+
export type { Detector, DetectorContext } from "./types.js";
|
|
17
|
+
//# sourceMappingURL=index.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/detectors/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,kBAAkB,EAAE,MAAM,2BAA2B,CAAC;AAC/D,OAAO,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAC;AACjE,OAAO,EAAE,oBAAoB,EAAE,MAAM,6BAA6B,CAAC;AACnE,OAAO,EAAE,kBAAkB,EAAE,MAAM,0BAA0B,CAAC;AAC9D,OAAO,EAAE,wBAAwB,EAAE,MAAM,iCAAiC,CAAC;AAC3E,OAAO,EAAE,0BAA0B,EAAE,MAAM,mCAAmC,CAAC;AAC/E,OAAO,EAAE,0BAA0B,EAAE,MAAM,mCAAmC,CAAC;AAC/E,OAAO,EAAE,wBAAwB,EAAE,MAAM,iCAAiC,CAAC;AAC3E,OAAO,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AACxD,OAAO,EAAE,8BAA8B,EAAE,MAAM,uCAAuC,CAAC;AACvF,OAAO,EAAE,YAAY,EAAE,iBAAiB,EAAE,KAAK,qBAAqB,EAAE,MAAM,aAAa,CAAC;AAC1F,OAAO,EAAE,gCAAgC,EAAE,kBAAkB,EAAE,MAAM,2BAA2B,CAAC;AACjG,OAAO,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAC;AACpD,OAAO,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AACrE,OAAO,EAAE,0BAA0B,EAAE,MAAM,mCAAmC,CAAC;AAC/E,YAAY,EAAE,QAAQ,EAAE,eAAe,EAAE,MAAM,YAAY,CAAC"}
|