@diff-review-system/drs 2.2.1 → 3.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/README.md +228 -92
- package/dist/ci/runner.d.ts.map +1 -1
- package/dist/ci/runner.js +19 -22
- package/dist/ci/runner.js.map +1 -1
- package/dist/cli/describe-mr.d.ts.map +1 -1
- package/dist/cli/describe-mr.js +39 -20
- package/dist/cli/describe-mr.js.map +1 -1
- package/dist/cli/describe-pr.d.ts.map +1 -1
- package/dist/cli/describe-pr.js +39 -20
- package/dist/cli/describe-pr.js.map +1 -1
- package/dist/cli/index.js +11 -7
- package/dist/cli/index.js.map +1 -1
- package/dist/cli/init.d.ts.map +1 -1
- package/dist/cli/init.js +30 -2
- package/dist/cli/init.js.map +1 -1
- package/dist/cli/post-comments.d.ts.map +1 -1
- package/dist/cli/post-comments.js +5 -5
- package/dist/cli/post-comments.js.map +1 -1
- package/dist/cli/review-local.d.ts.map +1 -1
- package/dist/cli/review-local.integration.test.d.ts +2 -0
- package/dist/cli/review-local.integration.test.d.ts.map +1 -0
- package/dist/cli/review-local.integration.test.js +343 -0
- package/dist/cli/review-local.integration.test.js.map +1 -0
- package/dist/cli/review-local.js +5 -4
- package/dist/cli/review-local.js.map +1 -1
- package/dist/cli/review-local.live.e2e.test.d.ts +2 -0
- package/dist/cli/review-local.live.e2e.test.d.ts.map +1 -0
- package/dist/cli/review-local.live.e2e.test.js +154 -0
- package/dist/cli/review-local.live.e2e.test.js.map +1 -0
- package/dist/cli/review-local.test.d.ts +2 -0
- package/dist/cli/review-local.test.d.ts.map +1 -0
- package/dist/cli/review-local.test.js +164 -0
- package/dist/cli/review-local.test.js.map +1 -0
- package/dist/cli/review-mr.d.ts +1 -1
- package/dist/cli/review-mr.d.ts.map +1 -1
- package/dist/cli/review-mr.js +92 -17
- package/dist/cli/review-mr.js.map +1 -1
- package/dist/cli/review-mr.test.d.ts +2 -0
- package/dist/cli/review-mr.test.d.ts.map +1 -0
- package/dist/cli/review-mr.test.js +142 -0
- package/dist/cli/review-mr.test.js.map +1 -0
- package/dist/cli/review-pr.d.ts +1 -1
- package/dist/cli/review-pr.d.ts.map +1 -1
- package/dist/cli/review-pr.js +96 -13
- package/dist/cli/review-pr.js.map +1 -1
- package/dist/cli/review-pr.test.d.ts +2 -0
- package/dist/cli/review-pr.test.d.ts.map +1 -0
- package/dist/cli/review-pr.test.js +137 -0
- package/dist/cli/review-pr.test.js.map +1 -0
- package/dist/cli/show-changes.js +4 -4
- package/dist/github/platform-adapter.js +2 -2
- package/dist/gitlab/client.js +1 -1
- package/dist/lib/code-quality-report.js +1 -1
- package/dist/lib/comment-formatter.d.ts +2 -1
- package/dist/lib/comment-formatter.d.ts.map +1 -1
- package/dist/lib/comment-formatter.js +33 -1
- package/dist/lib/comment-formatter.js.map +1 -1
- package/dist/lib/comment-formatter.test.js +43 -0
- package/dist/lib/comment-formatter.test.js.map +1 -1
- package/dist/lib/comment-manager.d.ts.map +1 -1
- package/dist/lib/comment-manager.js +4 -3
- package/dist/lib/comment-manager.js.map +1 -1
- package/dist/lib/comment-poster.d.ts +2 -1
- package/dist/lib/comment-poster.d.ts.map +1 -1
- package/dist/lib/comment-poster.js +2 -2
- package/dist/lib/comment-poster.js.map +1 -1
- package/dist/lib/comment-poster.test.js +27 -11
- package/dist/lib/comment-poster.test.js.map +1 -1
- package/dist/lib/config-model-overrides.test.d.ts +1 -1
- package/dist/lib/config-model-overrides.test.js +2 -2
- package/dist/lib/config-model-overrides.test.js.map +1 -1
- package/dist/lib/config.d.ts +34 -7
- package/dist/lib/config.d.ts.map +1 -1
- package/dist/lib/config.js +35 -13
- package/dist/lib/config.js.map +1 -1
- package/dist/lib/config.test.js +16 -0
- package/dist/lib/config.test.js.map +1 -1
- package/dist/lib/context-compression.d.ts +27 -1
- package/dist/lib/context-compression.d.ts.map +1 -1
- package/dist/lib/context-compression.js +106 -4
- package/dist/lib/context-compression.js.map +1 -1
- package/dist/lib/context-compression.test.js +305 -1
- package/dist/lib/context-compression.test.js.map +1 -1
- package/dist/lib/context-loader.d.ts +3 -2
- package/dist/lib/context-loader.d.ts.map +1 -1
- package/dist/lib/context-loader.js +11 -11
- package/dist/lib/context-loader.js.map +1 -1
- package/dist/lib/description-executor.d.ts +19 -2
- package/dist/lib/description-executor.d.ts.map +1 -1
- package/dist/lib/description-executor.js +52 -21
- package/dist/lib/description-executor.js.map +1 -1
- package/dist/lib/description-executor.test.d.ts +2 -0
- package/dist/lib/description-executor.test.d.ts.map +1 -0
- package/dist/lib/description-executor.test.js +120 -0
- package/dist/lib/description-executor.test.js.map +1 -0
- package/dist/lib/description-formatter.d.ts +8 -3
- package/dist/lib/description-formatter.d.ts.map +1 -1
- package/dist/lib/description-formatter.js +88 -13
- package/dist/lib/description-formatter.js.map +1 -1
- package/dist/lib/description-formatter.test.d.ts +2 -0
- package/dist/lib/description-formatter.test.d.ts.map +1 -0
- package/dist/lib/description-formatter.test.js +57 -0
- package/dist/lib/description-formatter.test.js.map +1 -0
- package/dist/lib/diff-parser.test.d.ts +2 -0
- package/dist/lib/diff-parser.test.d.ts.map +1 -0
- package/dist/lib/diff-parser.test.js +335 -0
- package/dist/lib/diff-parser.test.js.map +1 -0
- package/dist/lib/exit.d.ts +35 -0
- package/dist/lib/exit.d.ts.map +1 -0
- package/dist/lib/exit.js +53 -0
- package/dist/lib/exit.js.map +1 -0
- package/dist/lib/exit.test.d.ts +2 -0
- package/dist/lib/exit.test.d.ts.map +1 -0
- package/dist/lib/exit.test.js +120 -0
- package/dist/lib/exit.test.js.map +1 -0
- package/dist/lib/format-utils.d.ts +3 -0
- package/dist/lib/format-utils.d.ts.map +1 -0
- package/dist/lib/format-utils.js +7 -0
- package/dist/lib/format-utils.js.map +1 -0
- package/dist/lib/json-output.d.ts +4 -1
- package/dist/lib/json-output.d.ts.map +1 -1
- package/dist/lib/json-output.js +2 -1
- package/dist/lib/json-output.js.map +1 -1
- package/dist/lib/json-output.test.d.ts +2 -0
- package/dist/lib/json-output.test.d.ts.map +1 -0
- package/dist/lib/json-output.test.js +135 -0
- package/dist/lib/json-output.test.js.map +1 -0
- package/dist/lib/logger.d.ts +10 -2
- package/dist/lib/logger.d.ts.map +1 -1
- package/dist/lib/logger.js +22 -4
- package/dist/lib/logger.js.map +1 -1
- package/dist/lib/logger.test.d.ts +2 -0
- package/dist/lib/logger.test.d.ts.map +1 -0
- package/dist/lib/logger.test.js +324 -0
- package/dist/lib/logger.test.js.map +1 -0
- package/dist/lib/position-validator.test.d.ts +2 -0
- package/dist/lib/position-validator.test.d.ts.map +1 -0
- package/dist/lib/position-validator.test.js +128 -0
- package/dist/lib/position-validator.test.js.map +1 -0
- package/dist/lib/repository-validator.js +1 -1
- package/dist/lib/review-core.d.ts +9 -4
- package/dist/lib/review-core.d.ts.map +1 -1
- package/dist/lib/review-core.js +207 -112
- package/dist/lib/review-core.js.map +1 -1
- package/dist/lib/review-core.test.js +76 -30
- package/dist/lib/review-core.test.js.map +1 -1
- package/dist/lib/review-orchestrator.d.ts +12 -7
- package/dist/lib/review-orchestrator.d.ts.map +1 -1
- package/dist/lib/review-orchestrator.js +78 -22
- package/dist/lib/review-orchestrator.js.map +1 -1
- package/dist/lib/review-orchestrator.test.js +160 -42
- package/dist/lib/review-orchestrator.test.js.map +1 -1
- package/dist/lib/review-parser.test.d.ts +2 -0
- package/dist/lib/review-parser.test.d.ts.map +1 -0
- package/dist/lib/review-parser.test.js +130 -0
- package/dist/lib/review-parser.test.js.map +1 -0
- package/dist/lib/review-usage.d.ts +32 -0
- package/dist/lib/review-usage.d.ts.map +1 -0
- package/dist/lib/review-usage.js +72 -0
- package/dist/lib/review-usage.js.map +1 -0
- package/dist/lib/review-usage.test.d.ts +2 -0
- package/dist/lib/review-usage.test.d.ts.map +1 -0
- package/dist/lib/review-usage.test.js +83 -0
- package/dist/lib/review-usage.test.js.map +1 -0
- package/dist/lib/unified-review-executor.d.ts +6 -2
- package/dist/lib/unified-review-executor.d.ts.map +1 -1
- package/dist/lib/unified-review-executor.js +54 -28
- package/dist/lib/unified-review-executor.js.map +1 -1
- package/dist/lib/unified-review-executor.test.js +138 -16
- package/dist/lib/unified-review-executor.test.js.map +1 -1
- package/dist/lib/write-json-output.test.d.ts +2 -0
- package/dist/lib/write-json-output.test.d.ts.map +1 -0
- package/dist/lib/write-json-output.test.js +259 -0
- package/dist/lib/write-json-output.test.js.map +1 -0
- package/dist/pi/sdk.d.ts +94 -0
- package/dist/pi/sdk.d.ts.map +1 -0
- package/dist/pi/sdk.js +486 -0
- package/dist/pi/sdk.js.map +1 -0
- package/dist/pi/sdk.test.d.ts +2 -0
- package/dist/pi/sdk.test.d.ts.map +1 -0
- package/dist/pi/sdk.test.js +331 -0
- package/dist/pi/sdk.test.js.map +1 -0
- package/dist/{opencode → runtime}/agent-loader.d.ts +7 -5
- package/dist/runtime/agent-loader.d.ts.map +1 -0
- package/dist/{opencode → runtime}/agent-loader.js +24 -19
- package/dist/runtime/agent-loader.js.map +1 -0
- package/dist/runtime/agent-loader.test.d.ts +2 -0
- package/dist/runtime/agent-loader.test.d.ts.map +1 -0
- package/dist/runtime/agent-loader.test.js +280 -0
- package/dist/runtime/agent-loader.test.js.map +1 -0
- package/dist/runtime/built-in-paths.d.ts +2 -0
- package/dist/runtime/built-in-paths.d.ts.map +1 -0
- package/dist/runtime/built-in-paths.js +14 -0
- package/dist/runtime/built-in-paths.js.map +1 -0
- package/dist/{opencode → runtime}/client.d.ts +35 -18
- package/dist/runtime/client.d.ts.map +1 -0
- package/dist/runtime/client.js +486 -0
- package/dist/runtime/client.js.map +1 -0
- package/dist/{opencode → runtime}/client.test.d.ts.map +1 -1
- package/dist/runtime/client.test.js +392 -0
- package/dist/runtime/client.test.js.map +1 -0
- package/dist/runtime/path-config.d.ts +8 -0
- package/dist/runtime/path-config.d.ts.map +1 -0
- package/dist/runtime/path-config.js +68 -0
- package/dist/runtime/path-config.js.map +1 -0
- package/dist/runtime/path-config.test.d.ts +2 -0
- package/dist/runtime/path-config.test.d.ts.map +1 -0
- package/dist/runtime/path-config.test.js +103 -0
- package/dist/runtime/path-config.test.js.map +1 -0
- package/package.json +5 -5
- package/.opencode/opencode.jsonc +0 -15
- package/.opencode/tool/write_json_output.ts +0 -24
- package/.opencode/tools/drs_skill.ts +0 -67
- package/dist/lib/skills-prompt.d.ts +0 -3
- package/dist/lib/skills-prompt.d.ts.map +0 -1
- package/dist/lib/skills-prompt.js +0 -70
- package/dist/lib/skills-prompt.js.map +0 -1
- package/dist/opencode/agent-loader.d.ts.map +0 -1
- package/dist/opencode/agent-loader.js.map +0 -1
- package/dist/opencode/client.d.ts.map +0 -1
- package/dist/opencode/client.js +0 -456
- package/dist/opencode/client.js.map +0 -1
- package/dist/opencode/client.test.js +0 -317
- package/dist/opencode/client.test.js.map +0 -1
- package/dist/opencode/opencode-paths.d.ts +0 -2
- package/dist/opencode/opencode-paths.d.ts.map +0 -1
- package/dist/opencode/opencode-paths.js +0 -7
- package/dist/opencode/opencode-paths.js.map +0 -1
- package/dist/opencode/skill-loader.d.ts +0 -15
- package/dist/opencode/skill-loader.d.ts.map +0 -1
- package/dist/opencode/skill-loader.js +0 -88
- package/dist/opencode/skill-loader.js.map +0 -1
- /package/{.opencode/agent → .pi/agents}/describe/pr-describer.md +0 -0
- /package/{.opencode/agent → .pi/agents}/review/documentation.md +0 -0
- /package/{.opencode/agent → .pi/agents}/review/performance.md +0 -0
- /package/{.opencode/agent → .pi/agents}/review/quality.md +0 -0
- /package/{.opencode/agent → .pi/agents}/review/security.md +0 -0
- /package/{.opencode/agent → .pi/agents}/review/style.md +0 -0
- /package/{.opencode/agent → .pi/agents}/review/unified-reviewer.md +0 -0
- /package/dist/{opencode → runtime}/client.test.d.ts +0 -0
|
@@ -0,0 +1,72 @@
|
|
|
1
|
+
export function createEmptyUsageSummary() {
|
|
2
|
+
return {
|
|
3
|
+
input: 0,
|
|
4
|
+
output: 0,
|
|
5
|
+
cacheRead: 0,
|
|
6
|
+
cacheWrite: 0,
|
|
7
|
+
totalTokens: 0,
|
|
8
|
+
cost: 0,
|
|
9
|
+
};
|
|
10
|
+
}
|
|
11
|
+
export function createEmptyReviewUsageSummary() {
|
|
12
|
+
return {
|
|
13
|
+
total: createEmptyUsageSummary(),
|
|
14
|
+
agents: [],
|
|
15
|
+
};
|
|
16
|
+
}
|
|
17
|
+
export function addUsageSummary(base, delta) {
|
|
18
|
+
const input = base.input + (delta.input ?? 0);
|
|
19
|
+
const output = base.output + (delta.output ?? 0);
|
|
20
|
+
const cacheRead = base.cacheRead + (delta.cacheRead ?? 0);
|
|
21
|
+
const cacheWrite = base.cacheWrite + (delta.cacheWrite ?? 0);
|
|
22
|
+
const totalTokens = base.totalTokens +
|
|
23
|
+
(delta.totalTokens ??
|
|
24
|
+
(delta.input ?? 0) + (delta.output ?? 0) + (delta.cacheRead ?? 0) + (delta.cacheWrite ?? 0));
|
|
25
|
+
const cost = base.cost + (delta.cost ?? 0);
|
|
26
|
+
return {
|
|
27
|
+
input,
|
|
28
|
+
output,
|
|
29
|
+
cacheRead,
|
|
30
|
+
cacheWrite,
|
|
31
|
+
totalTokens,
|
|
32
|
+
cost,
|
|
33
|
+
};
|
|
34
|
+
}
|
|
35
|
+
export function createAgentUsageSummary(agentType) {
|
|
36
|
+
return {
|
|
37
|
+
agentType,
|
|
38
|
+
turns: 0,
|
|
39
|
+
usage: createEmptyUsageSummary(),
|
|
40
|
+
};
|
|
41
|
+
}
|
|
42
|
+
export function applyUsageMessage(agentUsage, message) {
|
|
43
|
+
const model = agentUsage.model ?? formatModelIdentifier(message.provider, message.model);
|
|
44
|
+
const turns = agentUsage.turns + 1;
|
|
45
|
+
if (!message.usage) {
|
|
46
|
+
return {
|
|
47
|
+
...agentUsage,
|
|
48
|
+
model,
|
|
49
|
+
turns,
|
|
50
|
+
};
|
|
51
|
+
}
|
|
52
|
+
return {
|
|
53
|
+
...agentUsage,
|
|
54
|
+
model,
|
|
55
|
+
turns,
|
|
56
|
+
usage: addUsageSummary(agentUsage.usage, message.usage),
|
|
57
|
+
};
|
|
58
|
+
}
|
|
59
|
+
export function aggregateAgentUsage(agents) {
|
|
60
|
+
const total = agents.reduce((acc, agent) => addUsageSummary(acc, agent.usage), createEmptyUsageSummary());
|
|
61
|
+
return {
|
|
62
|
+
total,
|
|
63
|
+
agents,
|
|
64
|
+
};
|
|
65
|
+
}
|
|
66
|
+
export function formatModelIdentifier(provider, model) {
|
|
67
|
+
if (provider && model) {
|
|
68
|
+
return `${provider}/${model}`;
|
|
69
|
+
}
|
|
70
|
+
return model;
|
|
71
|
+
}
|
|
72
|
+
//# sourceMappingURL=review-usage.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-usage.js","sourceRoot":"","sources":["../../src/lib/review-usage.ts"],"names":[],"mappings":"AAsBA,MAAM,UAAU,uBAAuB;IACrC,OAAO;QACL,KAAK,EAAE,CAAC;QACR,MAAM,EAAE,CAAC;QACT,SAAS,EAAE,CAAC;QACZ,UAAU,EAAE,CAAC;QACb,WAAW,EAAE,CAAC;QACd,IAAI,EAAE,CAAC;KACR,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,6BAA6B;IAC3C,OAAO;QACL,KAAK,EAAE,uBAAuB,EAAE;QAChC,MAAM,EAAE,EAAE;KACX,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,eAAe,CAAC,IAAkB,EAAE,KAA4B;IAC9E,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,GAAG,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,CAAC,CAAC;IAC9C,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,GAAG,CAAC,KAAK,CAAC,MAAM,IAAI,CAAC,CAAC,CAAC;IACjD,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,GAAG,CAAC,KAAK,CAAC,SAAS,IAAI,CAAC,CAAC,CAAC;IAC1D,MAAM,UAAU,GAAG,IAAI,CAAC,UAAU,GAAG,CAAC,KAAK,CAAC,UAAU,IAAI,CAAC,CAAC,CAAC;IAC7D,MAAM,WAAW,GACf,IAAI,CAAC,WAAW;QAChB,CAAC,KAAK,CAAC,WAAW;YAChB,CAAC,KAAK,CAAC,KAAK,IAAI,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,IAAI,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,SAAS,IAAI,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,UAAU,IAAI,CAAC,CAAC,CAAC,CAAC;IACjG,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC;IAE3C,OAAO;QACL,KAAK;QACL,MAAM;QACN,SAAS;QACT,UAAU;QACV,WAAW;QACX,IAAI;KACL,CAAC;AACJ,CAAC;AAQD,MAAM,UAAU,uBAAuB,CAAC,SAAiB;IACvD,OAAO;QACL,SAAS;QACT,KAAK,EAAE,CAAC;QACR,KAAK,EAAE,uBAAuB,EAAE;KACjC,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,iBAAiB,CAC/B,UAA6B,EAC7B,OAAyB;IAEzB,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,IAAI,qBAAqB,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,KAAK,CAAC,CAAC;IACzF,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,GAAG,CAAC,CAAC;IAEnC,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,CAAC;QACnB,OAAO;YACL,GAAG,UAAU;YACb,KAAK;YACL,KAAK;SACN,CAAC;IACJ,CAAC;IAED,OAAO;QACL,GAAG,UAAU;QACb,KAAK;QACL,KAAK;QACL,KAAK,EAAE,eAAe,CAAC,UAAU,CAAC,KAAK,EAAE,OAAO,CAAC,KAAK,CAAC;KACxD,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,mBAAmB,CAAC,MAA2B;IAC7D,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CACzB,CAAC,GAAG,EAAE,KAAK,EAAE,EAAE,CAAC,eAAe,CAAC,GAAG,EAAE,KAAK,CAAC,KAAK,CAAC,EACjD,uBAAuB,EAAE,CAC1B,CAAC;IAEF,OAAO;QACL,KAAK;QACL,MAAM;KACP,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,qBAAqB,CAAC,QAAiB,EAAE,KAAc;IACrE,IAAI,QAAQ,IAAI,KAAK,EAAE,CAAC;QACtB,OAAO,GAAG,QAAQ,IAAI,KAAK,EAAE,CAAC;IAChC,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-usage.test.d.ts","sourceRoot":"","sources":["../../src/lib/review-usage.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,83 @@
|
|
|
1
|
+
import { describe, expect, it } from 'vitest';
|
|
2
|
+
import { addUsageSummary, aggregateAgentUsage, applyUsageMessage, createAgentUsageSummary, createEmptyUsageSummary, formatModelIdentifier, } from './review-usage.js';
|
|
3
|
+
describe('review-usage', () => {
|
|
4
|
+
it('adds usage values with sensible defaults', () => {
|
|
5
|
+
const base = createEmptyUsageSummary();
|
|
6
|
+
const total = addUsageSummary(base, {
|
|
7
|
+
input: 10,
|
|
8
|
+
output: 2,
|
|
9
|
+
cacheRead: 3,
|
|
10
|
+
cost: 0.5,
|
|
11
|
+
});
|
|
12
|
+
expect(total).toEqual({
|
|
13
|
+
input: 10,
|
|
14
|
+
output: 2,
|
|
15
|
+
cacheRead: 3,
|
|
16
|
+
cacheWrite: 0,
|
|
17
|
+
totalTokens: 15,
|
|
18
|
+
cost: 0.5,
|
|
19
|
+
});
|
|
20
|
+
});
|
|
21
|
+
it('aggregates per-agent usage into run totals', () => {
|
|
22
|
+
const result = aggregateAgentUsage([
|
|
23
|
+
{
|
|
24
|
+
agentType: 'security',
|
|
25
|
+
turns: 1,
|
|
26
|
+
usage: {
|
|
27
|
+
input: 100,
|
|
28
|
+
output: 10,
|
|
29
|
+
cacheRead: 0,
|
|
30
|
+
cacheWrite: 0,
|
|
31
|
+
totalTokens: 110,
|
|
32
|
+
cost: 0.01,
|
|
33
|
+
},
|
|
34
|
+
},
|
|
35
|
+
{
|
|
36
|
+
agentType: 'quality',
|
|
37
|
+
turns: 1,
|
|
38
|
+
usage: {
|
|
39
|
+
input: 200,
|
|
40
|
+
output: 20,
|
|
41
|
+
cacheRead: 5,
|
|
42
|
+
cacheWrite: 0,
|
|
43
|
+
totalTokens: 225,
|
|
44
|
+
cost: 0.02,
|
|
45
|
+
},
|
|
46
|
+
},
|
|
47
|
+
]);
|
|
48
|
+
expect(result.total).toEqual({
|
|
49
|
+
input: 300,
|
|
50
|
+
output: 30,
|
|
51
|
+
cacheRead: 5,
|
|
52
|
+
cacheWrite: 0,
|
|
53
|
+
totalTokens: 335,
|
|
54
|
+
cost: 0.03,
|
|
55
|
+
});
|
|
56
|
+
expect(result.agents).toHaveLength(2);
|
|
57
|
+
});
|
|
58
|
+
it('applies usage messages and tracks turns/model', () => {
|
|
59
|
+
const base = createAgentUsageSummary('describe/pr-describer');
|
|
60
|
+
const next = applyUsageMessage(base, {
|
|
61
|
+
provider: 'opencode',
|
|
62
|
+
model: 'glm-5-free',
|
|
63
|
+
usage: {
|
|
64
|
+
input: 50,
|
|
65
|
+
output: 10,
|
|
66
|
+
cacheRead: 0,
|
|
67
|
+
cacheWrite: 0,
|
|
68
|
+
totalTokens: 60,
|
|
69
|
+
cost: 0.002,
|
|
70
|
+
},
|
|
71
|
+
});
|
|
72
|
+
expect(next.turns).toBe(1);
|
|
73
|
+
expect(next.model).toBe('opencode/glm-5-free');
|
|
74
|
+
expect(next.usage.totalTokens).toBe(60);
|
|
75
|
+
expect(next.usage.cost).toBe(0.002);
|
|
76
|
+
});
|
|
77
|
+
it('formats provider/model identifiers', () => {
|
|
78
|
+
expect(formatModelIdentifier('opencode', 'glm-5-free')).toBe('opencode/glm-5-free');
|
|
79
|
+
expect(formatModelIdentifier(undefined, 'gpt-4o')).toBe('gpt-4o');
|
|
80
|
+
expect(formatModelIdentifier(undefined, undefined)).toBeUndefined();
|
|
81
|
+
});
|
|
82
|
+
});
|
|
83
|
+
//# sourceMappingURL=review-usage.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-usage.test.js","sourceRoot":"","sources":["../../src/lib/review-usage.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EACL,eAAe,EACf,mBAAmB,EACnB,iBAAiB,EACjB,uBAAuB,EACvB,uBAAuB,EACvB,qBAAqB,GACtB,MAAM,mBAAmB,CAAC;AAE3B,QAAQ,CAAC,cAAc,EAAE,GAAG,EAAE;IAC5B,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,IAAI,GAAG,uBAAuB,EAAE,CAAC;QACvC,MAAM,KAAK,GAAG,eAAe,CAAC,IAAI,EAAE;YAClC,KAAK,EAAE,EAAE;YACT,MAAM,EAAE,CAAC;YACT,SAAS,EAAE,CAAC;YACZ,IAAI,EAAE,GAAG;SACV,CAAC,CAAC;QAEH,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC;YACpB,KAAK,EAAE,EAAE;YACT,MAAM,EAAE,CAAC;YACT,SAAS,EAAE,CAAC;YACZ,UAAU,EAAE,CAAC;YACb,WAAW,EAAE,EAAE;YACf,IAAI,EAAE,GAAG;SACV,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,MAAM,GAAG,mBAAmB,CAAC;YACjC;gBACE,SAAS,EAAE,UAAU;gBACrB,KAAK,EAAE,CAAC;gBACR,KAAK,EAAE;oBACL,KAAK,EAAE,GAAG;oBACV,MAAM,EAAE,EAAE;oBACV,SAAS,EAAE,CAAC;oBACZ,UAAU,EAAE,CAAC;oBACb,WAAW,EAAE,GAAG;oBAChB,IAAI,EAAE,IAAI;iBACX;aACF;YACD;gBACE,SAAS,EAAE,SAAS;gBACpB,KAAK,EAAE,CAAC;gBACR,KAAK,EAAE;oBACL,KAAK,EAAE,GAAG;oBACV,MAAM,EAAE,EAAE;oBACV,SAAS,EAAE,CAAC;oBACZ,UAAU,EAAE,CAAC;oBACb,WAAW,EAAE,GAAG;oBAChB,IAAI,EAAE,IAAI;iBACX;aACF;SACF,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC;YAC3B,KAAK,EAAE,GAAG;YACV,MAAM,EAAE,EAAE;YACV,SAAS,EAAE,CAAC;YACZ,UAAU,EAAE,CAAC;YACb,WAAW,EAAE,GAAG;YAChB,IAAI,EAAE,IAAI;SACX,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,IAAI,GAAG,uBAAuB,CAAC,uBAAuB,CAAC,CAAC;QAE9D,MAAM,IAAI,GAAG,iBAAiB,CAAC,IAAI,EAAE;YACnC,QAAQ,EAAE,UAAU;YACpB,KAAK,EAAE,YAAY;YACnB,KAAK,EAAE;gBACL,KAAK,EAAE,EAAE;gBACT,MAAM,EAAE,EAAE;gBACV,SAAS,EAAE,CAAC;gBACZ,UAAU,EAAE,CAAC;gBACb,WAAW,EAAE,EAAE;gBACf,IAAI,EAAE,KAAK;aACZ;SACF,CAAC,CAAC;QAEH,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC3B,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;QAC/C,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACxC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,CAAC,qBAAqB,CAAC,UAAU,EAAE,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;QACpF,MAAM,CAAC,qBAAqB,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAClE,MAAM,CAAC,qBAAqB,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC,CAAC,aAAa,EAAE,CAAC;IACtE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -9,7 +9,7 @@
|
|
|
9
9
|
*/
|
|
10
10
|
import { type DRSConfig } from './config.js';
|
|
11
11
|
import type { ReviewIssue } from './comment-formatter.js';
|
|
12
|
-
import type { PlatformClient, LineValidator, InlineCommentPosition } from './platform-client.js';
|
|
12
|
+
import type { PlatformClient, PullRequest, FileChange, LineValidator, InlineCommentPosition } from './platform-client.js';
|
|
13
13
|
export { enforceRepoBranchMatch } from './repository-validator.js';
|
|
14
14
|
export { postReviewComments } from './comment-poster.js';
|
|
15
15
|
export interface UnifiedReviewOptions {
|
|
@@ -19,6 +19,10 @@ export interface UnifiedReviewOptions {
|
|
|
19
19
|
projectId: string;
|
|
20
20
|
/** PR/MR number */
|
|
21
21
|
prNumber: number;
|
|
22
|
+
/** Optional pre-fetched PR/MR details (used to avoid duplicate platform calls) */
|
|
23
|
+
pullRequest?: PullRequest;
|
|
24
|
+
/** Optional pre-fetched changed files (used to avoid duplicate platform calls) */
|
|
25
|
+
changedFiles?: FileChange[];
|
|
22
26
|
/** Whether to post comments to the platform */
|
|
23
27
|
postComments: boolean;
|
|
24
28
|
/** Whether to post an error comment if the review fails */
|
|
@@ -41,7 +45,7 @@ export interface UnifiedReviewOptions {
|
|
|
41
45
|
describe?: boolean;
|
|
42
46
|
/** Post generated description during review */
|
|
43
47
|
postDescription?: boolean;
|
|
44
|
-
/** Debug mode - print
|
|
48
|
+
/** Debug mode - print runtime configuration */
|
|
45
49
|
debug?: boolean;
|
|
46
50
|
}
|
|
47
51
|
/**
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"unified-review-executor.d.ts","sourceRoot":"","sources":["../../src/lib/unified-review-executor.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;
|
|
1
|
+
{"version":3,"file":"unified-review-executor.d.ts","sourceRoot":"","sources":["../../src/lib/unified-review-executor.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAMH,OAAO,EAIL,KAAK,SAAS,EACf,MAAM,aAAa,CAAC;AACrB,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAO1D,OAAO,KAAK,EACV,cAAc,EACd,WAAW,EACX,UAAU,EACV,aAAa,EACb,qBAAqB,EACtB,MAAM,sBAAsB,CAAC;AAmB9B,OAAO,EAAE,sBAAsB,EAAE,MAAM,2BAA2B,CAAC;AACnE,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AAEzD,MAAM,WAAW,oBAAoB;IACnC,iDAAiD;IACjD,cAAc,EAAE,cAAc,CAAC;IAC/B,wEAAwE;IACxE,SAAS,EAAE,MAAM,CAAC;IAClB,mBAAmB;IACnB,QAAQ,EAAE,MAAM,CAAC;IACjB,kFAAkF;IAClF,WAAW,CAAC,EAAE,WAAW,CAAC;IAC1B,kFAAkF;IAClF,YAAY,CAAC,EAAE,UAAU,EAAE,CAAC;IAC5B,+CAA+C;IAC/C,YAAY,EAAE,OAAO,CAAC;IACtB,2DAA2D;IAC3D,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,8DAA8D;IAC9D,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,+CAA+C;IAC/C,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,wCAAwC;IACxC,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,uDAAuD;IACvD,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,wEAAwE;IACxE,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,+DAA+D;IAC/D,oBAAoB,CAAC,EAAE,CAAC,KAAK,EAAE,WAAW,EAAE,YAAY,EAAE,OAAO,KAAK,qBAAqB,CAAC;IAC5F,wCAAwC;IACxC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,+CAA+C;IAC/C,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,+CAA+C;IAC/C,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,+CAA+C;IAC/C,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AAED;;GAEG;AACH,wBAAsB,oBAAoB,CACxC,MAAM,EAAE,SAAS,EACjB,OAAO,EAAE,oBAAoB,GAC5B,OAAO,CAAC,IAAI,CAAC,CAgPf"}
|
|
@@ -10,14 +10,17 @@
|
|
|
10
10
|
import chalk from 'chalk';
|
|
11
11
|
import { writeFile } from 'fs/promises';
|
|
12
12
|
import { resolve } from 'path';
|
|
13
|
+
import { exitProcess } from './exit.js';
|
|
13
14
|
import { getDescriberModelOverride, getModelOverrides, getUnifiedModelOverride, } from './config.js';
|
|
14
|
-
import {
|
|
15
|
+
import { connectToRuntime, filterIgnoredFiles, getReviewBudgetModelIds, } from './review-orchestrator.js';
|
|
15
16
|
import { buildBaseInstructions, runReviewPipeline, displayReviewSummary } from './review-core.js';
|
|
16
17
|
import { generateCodeQualityReport, formatCodeQualityReport } from './code-quality-report.js';
|
|
17
18
|
import { formatReviewJson, writeReviewJson, printReviewJson } from './json-output.js';
|
|
18
19
|
import { enforceRepoBranchMatch, resolveBaseBranch, getCanonicalDiffCommand, } from './repository-validator.js';
|
|
19
20
|
import { postReviewComments } from './comment-poster.js';
|
|
21
|
+
import { prepareDiffsForAgent, formatCompressionSummary, resolveCompressionBudget, } from './context-compression.js';
|
|
20
22
|
import { runDescribeIfEnabled } from './description-executor.js';
|
|
23
|
+
import { formatDescribeSummary } from './description-formatter.js';
|
|
21
24
|
import { postErrorComment, removeErrorComment } from './error-comment-poster.js';
|
|
22
25
|
// Re-export functions for backward compatibility
|
|
23
26
|
export { enforceRepoBranchMatch } from './repository-validator.js';
|
|
@@ -27,18 +30,18 @@ export { postReviewComments } from './comment-poster.js';
|
|
|
27
30
|
*/
|
|
28
31
|
export async function executeUnifiedReview(config, options) {
|
|
29
32
|
const { platformClient, projectId, prNumber, postComments } = options;
|
|
30
|
-
// Track
|
|
31
|
-
let
|
|
33
|
+
// Track runtime client for cleanup
|
|
34
|
+
let runtimeClient = null;
|
|
32
35
|
try {
|
|
33
36
|
console.log(chalk.bold.cyan('\n📋 DRS | Code Review Analysis\n'));
|
|
34
37
|
// Fetch PR/MR details
|
|
35
38
|
console.log(chalk.gray(`Fetching PR/MR #${prNumber}...\n`));
|
|
36
|
-
const pr = await platformClient.getPullRequest(projectId, prNumber);
|
|
37
|
-
await enforceRepoBranchMatch(options.workingDir
|
|
39
|
+
const pr = options.pullRequest ?? (await platformClient.getPullRequest(projectId, prNumber));
|
|
40
|
+
await enforceRepoBranchMatch(options.workingDir ?? process.cwd(), projectId, pr, {
|
|
38
41
|
skipRepoCheck: config.review.skipRepoCheck,
|
|
39
42
|
skipBranchCheck: config.review.skipBranchCheck,
|
|
40
43
|
});
|
|
41
|
-
const allFiles = await platformClient.getChangedFiles(projectId, prNumber);
|
|
44
|
+
const allFiles = options.changedFiles ?? (await platformClient.getChangedFiles(projectId, prNumber));
|
|
42
45
|
console.log(chalk.bold(`PR/MR: ${pr.title}`));
|
|
43
46
|
console.log(chalk.gray(`Author: ${pr.author}`));
|
|
44
47
|
console.log(chalk.gray(`Branch: ${pr.sourceBranch} → ${pr.targetBranch}`));
|
|
@@ -66,51 +69,74 @@ export async function executeUnifiedReview(config, options) {
|
|
|
66
69
|
console.log(chalk.yellow('✓ No files to review after filtering\n'));
|
|
67
70
|
return;
|
|
68
71
|
}
|
|
72
|
+
const agentModelOverrides = getModelOverrides(config);
|
|
73
|
+
const unifiedModelOverrides = getUnifiedModelOverride(config);
|
|
69
74
|
const reviewOverrides = {
|
|
70
|
-
...
|
|
71
|
-
...
|
|
75
|
+
...agentModelOverrides,
|
|
76
|
+
...unifiedModelOverrides,
|
|
72
77
|
};
|
|
73
78
|
const describeEnabled = options.describe ?? config.review.describe?.enabled ?? false;
|
|
74
79
|
const postDescriptionEnabled = options.postDescription ?? config.review.describe?.postDescription ?? false;
|
|
75
80
|
const describeOverrides = describeEnabled ? getDescriberModelOverride(config) : {};
|
|
76
81
|
const modelOverrides = { ...reviewOverrides, ...describeOverrides };
|
|
77
|
-
// Connect to
|
|
78
|
-
|
|
82
|
+
// Connect to runtime
|
|
83
|
+
runtimeClient = await connectToRuntime(config, options.workingDir ?? process.cwd(), {
|
|
79
84
|
debug: options.debug,
|
|
80
85
|
modelOverrides,
|
|
81
86
|
});
|
|
82
|
-
const
|
|
87
|
+
const describeFileNames = allFiles
|
|
88
|
+
.filter((file) => file.status !== 'removed')
|
|
89
|
+
.map((file) => file.filename);
|
|
90
|
+
const filteredDescribeFileNames = new Set(filterIgnoredFiles(describeFileNames, config));
|
|
91
|
+
const filesForDescribe = allFiles
|
|
92
|
+
.filter((file) => filteredDescribeFileNames.has(file.filename))
|
|
93
|
+
.map((file) => ({
|
|
83
94
|
filename: file.filename,
|
|
84
95
|
patch: file.patch,
|
|
85
96
|
}));
|
|
97
|
+
let describeResult = null;
|
|
86
98
|
if (describeEnabled) {
|
|
87
|
-
await runDescribeIfEnabled(
|
|
99
|
+
describeResult = await runDescribeIfEnabled(runtimeClient, config, platformClient, projectId, pr, filesForDescribe, postDescriptionEnabled, options.workingDir ?? process.cwd(), options.debug);
|
|
88
100
|
}
|
|
101
|
+
// Build change context from describe output for review agents
|
|
102
|
+
const describeSummary = describeResult ? formatDescribeSummary(describeResult) : undefined;
|
|
89
103
|
// Build instructions for platform review - pass actual diff content from platform
|
|
90
104
|
const reviewLabel = `PR/MR #${prNumber}`;
|
|
91
105
|
const baseBranchResolution = resolveBaseBranch(options.baseBranch, pr.targetBranch);
|
|
92
106
|
const fallbackDiffCommand = getCanonicalDiffCommand(pr, baseBranchResolution);
|
|
93
|
-
const
|
|
94
|
-
|
|
107
|
+
const patchByFilename = new Map(allFiles.map((file) => [file.filename, file.patch]));
|
|
108
|
+
const filesForInstructions = filteredFiles.map((filename) => ({
|
|
109
|
+
filename,
|
|
110
|
+
patch: patchByFilename.get(filename),
|
|
111
|
+
}));
|
|
112
|
+
const reviewModelIds = getReviewBudgetModelIds(config.review.mode, agentModelOverrides, unifiedModelOverrides);
|
|
113
|
+
const contextWindow = runtimeClient.getMinContextWindow(reviewModelIds);
|
|
114
|
+
const compressionOptions = resolveCompressionBudget(contextWindow, config.contextCompression);
|
|
115
|
+
const compression = prepareDiffsForAgent(filesForInstructions, compressionOptions);
|
|
116
|
+
const compressionSummary = formatCompressionSummary(compression);
|
|
117
|
+
if (compressionSummary) {
|
|
118
|
+
console.log(chalk.yellow('⚠ Diff content trimmed to fit token budget.\n'));
|
|
119
|
+
}
|
|
120
|
+
let baseInstructions = buildBaseInstructions(reviewLabel, compression.files, fallbackDiffCommand, compressionSummary);
|
|
95
121
|
if (baseBranchResolution.resolvedBaseBranch) {
|
|
96
122
|
baseInstructions = `${baseInstructions}\n\nBase branch resolved to: ${baseBranchResolution.resolvedBaseBranch} (${baseBranchResolution.source})`;
|
|
97
123
|
}
|
|
98
124
|
// Run agents using shared core logic
|
|
99
|
-
const result = await runReviewPipeline(
|
|
125
|
+
const result = await runReviewPipeline(runtimeClient, config, baseInstructions, reviewLabel, filteredFiles, { prNumber, describeSummary }, options.workingDir ?? process.cwd(), options.debug ?? false);
|
|
100
126
|
// Display summary
|
|
101
127
|
displayReviewSummary(result);
|
|
102
128
|
// Post comments to platform if requested
|
|
103
129
|
if (postComments) {
|
|
104
130
|
// Remove any previous error comment on successful review
|
|
105
131
|
await removeErrorComment(platformClient, projectId, prNumber);
|
|
106
|
-
await postReviewComments(platformClient, projectId, prNumber, result.summary, result.issues, result.changeSummary, pr.platformData, options.lineValidator, options.createInlinePosition);
|
|
132
|
+
await postReviewComments(platformClient, projectId, prNumber, result.summary, result.issues, result.changeSummary, result.usage, pr.platformData, options.lineValidator, options.createInlinePosition);
|
|
107
133
|
}
|
|
108
134
|
// Generate code quality report if requested
|
|
109
135
|
if (options.codeQualityReport) {
|
|
110
|
-
await generateAndWriteCodeQualityReport(result.issues, options.codeQualityReport, options.workingDir
|
|
136
|
+
await generateAndWriteCodeQualityReport(result.issues, options.codeQualityReport, options.workingDir ?? process.cwd());
|
|
111
137
|
}
|
|
112
138
|
// Handle JSON output
|
|
113
|
-
const wantsJsonOutput = options.jsonOutput
|
|
139
|
+
const wantsJsonOutput = options.jsonOutput ?? options.outputPath;
|
|
114
140
|
if (wantsJsonOutput) {
|
|
115
141
|
const jsonOutput = formatReviewJson(result.summary, result.issues, {
|
|
116
142
|
source: `${options.prNumber}`,
|
|
@@ -119,9 +145,9 @@ export async function executeUnifiedReview(config, options) {
|
|
|
119
145
|
source: pr.sourceBranch,
|
|
120
146
|
target: pr.targetBranch,
|
|
121
147
|
},
|
|
122
|
-
});
|
|
148
|
+
}, result.usage);
|
|
123
149
|
if (options.outputPath) {
|
|
124
|
-
await writeReviewJson(jsonOutput, options.outputPath, options.workingDir
|
|
150
|
+
await writeReviewJson(jsonOutput, options.outputPath, options.workingDir ?? process.cwd());
|
|
125
151
|
console.log(chalk.green(`\n✓ Review results written to ${options.outputPath}\n`));
|
|
126
152
|
}
|
|
127
153
|
if (options.jsonOutput) {
|
|
@@ -131,8 +157,8 @@ export async function executeUnifiedReview(config, options) {
|
|
|
131
157
|
// Exit with error code if critical issues found
|
|
132
158
|
if (result.summary.bySeverity.CRITICAL > 0) {
|
|
133
159
|
console.log(chalk.red.bold('⚠️ Critical issues found!\n'));
|
|
134
|
-
await
|
|
135
|
-
|
|
160
|
+
await runtimeClient.shutdown();
|
|
161
|
+
exitProcess(1);
|
|
136
162
|
}
|
|
137
163
|
else if (result.summary.issuesFound === 0) {
|
|
138
164
|
console.log(chalk.green('✓ No issues found! Code looks good.\n'));
|
|
@@ -151,17 +177,17 @@ export async function executeUnifiedReview(config, options) {
|
|
|
151
177
|
}
|
|
152
178
|
// Handle "all agents failed" error
|
|
153
179
|
if (error instanceof Error && error.message === 'All review agents failed') {
|
|
154
|
-
if (
|
|
155
|
-
await
|
|
180
|
+
if (runtimeClient) {
|
|
181
|
+
await runtimeClient.shutdown();
|
|
156
182
|
}
|
|
157
|
-
|
|
183
|
+
exitProcess(1);
|
|
158
184
|
}
|
|
159
185
|
throw error;
|
|
160
186
|
}
|
|
161
187
|
finally {
|
|
162
|
-
// Shutdown
|
|
163
|
-
if (
|
|
164
|
-
await
|
|
188
|
+
// Shutdown runtime client if it was initialized
|
|
189
|
+
if (runtimeClient) {
|
|
190
|
+
await runtimeClient.shutdown();
|
|
165
191
|
}
|
|
166
192
|
}
|
|
167
193
|
}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"unified-review-executor.js","sourceRoot":"","sources":["../../src/lib/unified-review-executor.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,MAAM,OAAO,CAAC;AAC1B,OAAO,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AACxC,OAAO,EAAE,OAAO,EAAE,MAAM,MAAM,CAAC;AAC/B,OAAO,EACL,yBAAyB,EACzB,iBAAiB,EACjB,uBAAuB,GAExB,MAAM,aAAa,CAAC;AAErB,OAAO,
|
|
1
|
+
{"version":3,"file":"unified-review-executor.js","sourceRoot":"","sources":["../../src/lib/unified-review-executor.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,MAAM,OAAO,CAAC;AAC1B,OAAO,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AACxC,OAAO,EAAE,OAAO,EAAE,MAAM,MAAM,CAAC;AAC/B,OAAO,EAAE,WAAW,EAAE,MAAM,WAAW,CAAC;AACxC,OAAO,EACL,yBAAyB,EACzB,iBAAiB,EACjB,uBAAuB,GAExB,MAAM,aAAa,CAAC;AAErB,OAAO,EACL,gBAAgB,EAChB,kBAAkB,EAClB,uBAAuB,GACxB,MAAM,0BAA0B,CAAC;AAClC,OAAO,EAAE,qBAAqB,EAAE,iBAAiB,EAAE,oBAAoB,EAAE,MAAM,kBAAkB,CAAC;AAQlG,OAAO,EAAE,yBAAyB,EAAE,uBAAuB,EAAE,MAAM,0BAA0B,CAAC;AAC9F,OAAO,EAAE,gBAAgB,EAAE,eAAe,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AACtF,OAAO,EACL,sBAAsB,EACtB,iBAAiB,EACjB,uBAAuB,GACxB,MAAM,2BAA2B,CAAC;AACnC,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,EACL,oBAAoB,EACpB,wBAAwB,EACxB,wBAAwB,GACzB,MAAM,0BAA0B,CAAC;AAClC,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACjE,OAAO,EAAE,qBAAqB,EAAoB,MAAM,4BAA4B,CAAC;AACrF,OAAO,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,MAAM,2BAA2B,CAAC;AAEjF,iDAAiD;AACjD,OAAO,EAAE,sBAAsB,EAAE,MAAM,2BAA2B,CAAC;AACnE,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AAuCzD;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,oBAAoB,CACxC,MAAiB,EACjB,OAA6B;IAE7B,MAAM,EAAE,cAAc,EAAE,SAAS,EAAE,QAAQ,EAAE,YAAY,EAAE,GAAG,OAAO,CAAC;IAEtE,mCAAmC;IACnC,IAAI,aAAa,GAAwD,IAAI,CAAC;IAE9E,IAAI,CAAC;QACH,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,mCAAmC,CAAC,CAAC,CAAC;QAElE,sBAAsB;QACtB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,mBAAmB,QAAQ,OAAO,CAAC,CAAC,CAAC;QAE5D,MAAM,EAAE,GAAG,OAAO,CAAC,WAAW,IAAI,CAAC,MAAM,cAAc,CAAC,cAAc,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC,CAAC;QAE7F,MAAM,sBAAsB,CAAC,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,EAAE,EAAE;YAC/E,aAAa,EAAE,MAAM,CAAC,MAAM,CAAC,aAAa;YAC1C,eAAe,EAAE,MAAM,CAAC,MAAM,CAAC,eAAe;SAC/C,CAAC,CAAC;QAEH,MAAM,QAAQ,GACZ,OAAO,CAAC,YAAY,IAAI,CAAC,MAAM,cAAc,CAAC,eAAe,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC,CAAC;QAEtF,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,UAAU,EAAE,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC;QAC9C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAChD,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,YAAY,MAAM,EAAE,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC;QAC3E,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,kBAAkB,QAAQ,CAAC,MAAM,IAAI,CAAC,CAAC,CAAC;QAE/D,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC1B,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,0BAA0B,CAAC,CAAC,CAAC;YACtD,OAAO;QACT,CAAC;QAED,sDAAsD;QACtD,MAAM,gBAAgB,GAAG,QAAQ;aAC9B,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,MAAM,KAAK,SAAS,CAAC;aAC3C,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAEhC,IAAI,gBAAgB,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAClC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,wCAAwC,CAAC,CAAC,CAAC;YACpE,OAAO;QACT,CAAC;QAED,kCAAkC;QAClC,MAAM,iBAAiB,GAAG,kBAAkB,CAAC,gBAAgB,EAAE,MAAM,CAAC,CAAC;QACvE,MAAM,aAAa,GAAG,iBAAiB,CAAC;QACxC,MAAM,YAAY,GAAG,gBAAgB,CAAC,MAAM,GAAG,aAAa,CAAC,MAAM,CAAC;QAEpE,IAAI,YAAY,GAAG,CAAC,EAAE,CAAC;YACrB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,YAAY,YAAY,8BAA8B,CAAC,CAAC,CAAC;QAClF,CAAC;QAED,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC/B,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,wCAAwC,CAAC,CAAC,CAAC;YACpE,OAAO;QACT,CAAC;QAED,MAAM,mBAAmB,GAAG,iBAAiB,CAAC,MAAM,CAAC,CAAC;QACtD,MAAM,qBAAqB,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAC;QAC9D,MAAM,eAAe,GAAG;YACtB,GAAG,mBAAmB;YACtB,GAAG,qBAAqB;SACzB,CAAC;QACF,MAAM,eAAe,GAAG,OAAO,CAAC,QAAQ,IAAI,MAAM,CAAC,MAAM,CAAC,QAAQ,EAAE,OAAO,IAAI,KAAK,CAAC;QACrF,MAAM,sBAAsB,GAC1B,OAAO,CAAC,eAAe,IAAI,MAAM,CAAC,MAAM,CAAC,QAAQ,EAAE,eAAe,IAAI,KAAK,CAAC;QAE9E,MAAM,iBAAiB,GAAG,eAAe,CAAC,CAAC,CAAC,yBAAyB,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QACnF,MAAM,cAAc,GAAG,EAAE,GAAG,eAAe,EAAE,GAAG,iBAAiB,EAAE,CAAC;QAEpE,qBAAqB;QACrB,aAAa,GAAG,MAAM,gBAAgB,CAAC,MAAM,EAAE,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE,EAAE;YAClF,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,cAAc;SACf,CAAC,CAAC;QACH,MAAM,iBAAiB,GAAG,QAAQ;aAC/B,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,MAAM,KAAK,SAAS,CAAC;aAC3C,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAChC,MAAM,yBAAyB,GAAG,IAAI,GAAG,CAAC,kBAAkB,CAAC,iBAAiB,EAAE,MAAM,CAAC,CAAC,CAAC;QACzF,MAAM,gBAAgB,GAAG,QAAQ;aAC9B,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,yBAAyB,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;aAC9D,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACd,QAAQ,EAAE,IAAI,CAAC,QAAQ;YACvB,KAAK,EAAE,IAAI,CAAC,KAAK;SAClB,CAAC,CAAC,CAAC;QACN,IAAI,cAAc,GAAuB,IAAI,CAAC;QAC9C,IAAI,eAAe,EAAE,CAAC;YACpB,cAAc,GAAG,MAAM,oBAAoB,CACzC,aAAa,EACb,MAAM,EACN,cAAc,EACd,SAAS,EACT,EAAE,EACF,gBAAgB,EAChB,sBAAsB,EACtB,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE,EACnC,OAAO,CAAC,KAAK,CACd,CAAC;QACJ,CAAC;QAED,8DAA8D;QAC9D,MAAM,eAAe,GAAG,cAAc,CAAC,CAAC,CAAC,qBAAqB,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;QAE3F,kFAAkF;QAClF,MAAM,WAAW,GAAG,UAAU,QAAQ,EAAE,CAAC;QACzC,MAAM,oBAAoB,GAAG,iBAAiB,CAAC,OAAO,CAAC,UAAU,EAAE,EAAE,CAAC,YAAY,CAAC,CAAC;QACpF,MAAM,mBAAmB,GAAG,uBAAuB,CAAC,EAAE,EAAE,oBAAoB,CAAC,CAAC;QAC9E,MAAM,eAAe,GAAG,IAAI,GAAG,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,QAAQ,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;QACrF,MAAM,oBAAoB,GAAG,aAAa,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC;YAC5D,QAAQ;YACR,KAAK,EAAE,eAAe,CAAC,GAAG,CAAC,QAAQ,CAAC;SACrC,CAAC,CAAC,CAAC;QAEJ,MAAM,cAAc,GAAG,uBAAuB,CAC5C,MAAM,CAAC,MAAM,CAAC,IAAI,EAClB,mBAAmB,EACnB,qBAAqB,CACtB,CAAC;QACF,MAAM,aAAa,GAAG,aAAa,CAAC,mBAAmB,CAAC,cAAc,CAAC,CAAC;QACxE,MAAM,kBAAkB,GAAG,wBAAwB,CAAC,aAAa,EAAE,MAAM,CAAC,kBAAkB,CAAC,CAAC;QAE9F,MAAM,WAAW,GAAG,oBAAoB,CAAC,oBAAoB,EAAE,kBAAkB,CAAC,CAAC;QACnF,MAAM,kBAAkB,GAAG,wBAAwB,CAAC,WAAW,CAAC,CAAC;QAEjE,IAAI,kBAAkB,EAAE,CAAC;YACvB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,+CAA+C,CAAC,CAAC,CAAC;QAC7E,CAAC;QAED,IAAI,gBAAgB,GAAG,qBAAqB,CAC1C,WAAW,EACX,WAAW,CAAC,KAAK,EACjB,mBAAmB,EACnB,kBAAkB,CACnB,CAAC;QACF,IAAI,oBAAoB,CAAC,kBAAkB,EAAE,CAAC;YAC5C,gBAAgB,GAAG,GAAG,gBAAgB,gCAAgC,oBAAoB,CAAC,kBAAkB,KAAK,oBAAoB,CAAC,MAAM,GAAG,CAAC;QACnJ,CAAC;QACD,qCAAqC;QACrC,MAAM,MAAM,GAAG,MAAM,iBAAiB,CACpC,aAAa,EACb,MAAM,EACN,gBAAgB,EAChB,WAAW,EACX,aAAa,EACb,EAAE,QAAQ,EAAE,eAAe,EAAE,EAC7B,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE,EACnC,OAAO,CAAC,KAAK,IAAI,KAAK,CACvB,CAAC;QAEF,kBAAkB;QAClB,oBAAoB,CAAC,MAAM,CAAC,CAAC;QAE7B,yCAAyC;QACzC,IAAI,YAAY,EAAE,CAAC;YACjB,yDAAyD;YACzD,MAAM,kBAAkB,CAAC,cAAc,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC;YAE9D,MAAM,kBAAkB,CACtB,cAAc,EACd,SAAS,EACT,QAAQ,EACR,MAAM,CAAC,OAAO,EACd,MAAM,CAAC,MAAM,EACb,MAAM,CAAC,aAAa,EACpB,MAAM,CAAC,KAAK,EACZ,EAAE,CAAC,YAAY,EACf,OAAO,CAAC,aAAa,EACrB,OAAO,CAAC,oBAAoB,CAC7B,CAAC;QACJ,CAAC;QAED,4CAA4C;QAC5C,IAAI,OAAO,CAAC,iBAAiB,EAAE,CAAC;YAC9B,MAAM,iCAAiC,CACrC,MAAM,CAAC,MAAM,EACb,OAAO,CAAC,iBAAiB,EACzB,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE,CACpC,CAAC;QACJ,CAAC;QAED,qBAAqB;QACrB,MAAM,eAAe,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;QAEjE,IAAI,eAAe,EAAE,CAAC;YACpB,MAAM,UAAU,GAAG,gBAAgB,CACjC,MAAM,CAAC,OAAO,EACd,MAAM,CAAC,MAAM,EACb;gBACE,MAAM,EAAE,GAAG,OAAO,CAAC,QAAQ,EAAE;gBAC7B,OAAO,EAAE,OAAO,CAAC,SAAS;gBAC1B,MAAM,EAAE;oBACN,MAAM,EAAE,EAAE,CAAC,YAAY;oBACvB,MAAM,EAAE,EAAE,CAAC,YAAY;iBACxB;aACF,EACD,MAAM,CAAC,KAAK,CACb,CAAC;YAEF,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;gBACvB,MAAM,eAAe,CAAC,UAAU,EAAE,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;gBAC3F,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,iCAAiC,OAAO,CAAC,UAAU,IAAI,CAAC,CAAC,CAAC;YACpF,CAAC;YAED,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;gBACvB,eAAe,CAAC,UAAU,CAAC,CAAC;YAC9B,CAAC;QACH,CAAC;QAED,gDAAgD;QAChD,IAAI,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,QAAQ,GAAG,CAAC,EAAE,CAAC;YAC3C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,8BAA8B,CAAC,CAAC,CAAC;YAC5D,MAAM,aAAa,CAAC,QAAQ,EAAE,CAAC;YAC/B,WAAW,CAAC,CAAC,CAAC,CAAC;QACjB,CAAC;aAAM,IAAI,MAAM,CAAC,OAAO,CAAC,WAAW,KAAK,CAAC,EAAE,CAAC;YAC5C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,uCAAuC,CAAC,CAAC,CAAC;QACpE,CAAC;IACH,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,uFAAuF;QACvF,IAAI,OAAO,CAAC,gBAAgB,EAAE,CAAC;YAC7B,IAAI,CAAC;gBACH,MAAM,gBAAgB,CAAC,cAAc,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC;YAC9D,CAAC;YAAC,OAAO,SAAS,EAAE,CAAC;gBACnB,MAAM,gBAAgB,GAAG,SAAS,YAAY,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;gBAC5F,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,iCAAiC,gBAAgB,EAAE,CAAC,CAAC,CAAC;YAClF,CAAC;QACH,CAAC;QAED,mCAAmC;QACnC,IAAI,KAAK,YAAY,KAAK,IAAI,KAAK,CAAC,OAAO,KAAK,0BAA0B,EAAE,CAAC;YAC3E,IAAI,aAAa,EAAE,CAAC;gBAClB,MAAM,aAAa,CAAC,QAAQ,EAAE,CAAC;YACjC,CAAC;YACD,WAAW,CAAC,CAAC,CAAC,CAAC;QACjB,CAAC;QACD,MAAM,KAAK,CAAC;IACd,CAAC;YAAS,CAAC;QACT,gDAAgD;QAChD,IAAI,aAAa,EAAE,CAAC;YAClB,MAAM,aAAa,CAAC,QAAQ,EAAE,CAAC;QACjC,CAAC;IACH,CAAC;AACH,CAAC;AAED;;GAEG;AACH,KAAK,UAAU,iCAAiC,CAC9C,MAAqB,EACrB,UAAkB,EAClB,UAAkB;IAElB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,qCAAqC,CAAC,CAAC,CAAC;IAE/D,MAAM,MAAM,GAAG,yBAAyB,CAAC,MAAM,CAAC,CAAC;IACjD,MAAM,WAAW,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAC;IAEpD,MAAM,QAAQ,GAAG,OAAO,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;IACjD,MAAM,SAAS,CAAC,QAAQ,EAAE,WAAW,EAAE,OAAO,CAAC,CAAC;IAEhD,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,oCAAoC,UAAU,EAAE,CAAC,CAAC,CAAC;IAC3E,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,mBAAmB,MAAM,CAAC,MAAM,IAAI,CAAC,CAAC,CAAC;AAChE,CAAC"}
|