@urateam/core 0.1.15 → 0.1.18
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/dist/__tests__/agent-stream.test.js +5 -1
- package/dist/__tests__/agent-stream.test.js.map +1 -1
- package/dist/__tests__/agentic-deep-review-provider.test.d.ts +2 -0
- package/dist/__tests__/agentic-deep-review-provider.test.d.ts.map +1 -0
- package/dist/__tests__/agentic-deep-review-provider.test.js +70 -0
- package/dist/__tests__/agentic-deep-review-provider.test.js.map +1 -0
- package/dist/__tests__/audit-immutability.test.js +16 -1
- package/dist/__tests__/audit-immutability.test.js.map +1 -1
- package/dist/__tests__/cost/per-run-multi-model.test.d.ts +2 -0
- package/dist/__tests__/cost/per-run-multi-model.test.d.ts.map +1 -0
- package/dist/__tests__/cost/per-run-multi-model.test.js +102 -0
- package/dist/__tests__/cost/per-run-multi-model.test.js.map +1 -0
- package/dist/__tests__/db-qa-gap-issues.test.d.ts +2 -0
- package/dist/__tests__/db-qa-gap-issues.test.d.ts.map +1 -0
- package/dist/__tests__/db-qa-gap-issues.test.js +113 -0
- package/dist/__tests__/db-qa-gap-issues.test.js.map +1 -0
- package/dist/__tests__/db-release-decisions.test.d.ts +2 -0
- package/dist/__tests__/db-release-decisions.test.d.ts.map +1 -0
- package/dist/__tests__/db-release-decisions.test.js +98 -0
- package/dist/__tests__/db-release-decisions.test.js.map +1 -0
- package/dist/__tests__/db-review-model-runs.test.d.ts +2 -0
- package/dist/__tests__/db-review-model-runs.test.d.ts.map +1 -0
- package/dist/__tests__/db-review-model-runs.test.js +90 -0
- package/dist/__tests__/db-review-model-runs.test.js.map +1 -0
- package/dist/__tests__/e2e-fanout.test.d.ts +17 -0
- package/dist/__tests__/e2e-fanout.test.d.ts.map +1 -0
- package/dist/__tests__/e2e-fanout.test.js +184 -0
- package/dist/__tests__/e2e-fanout.test.js.map +1 -0
- package/dist/__tests__/e2e-pipeline.test.js +1 -0
- package/dist/__tests__/e2e-pipeline.test.js.map +1 -1
- package/dist/__tests__/notifier-discord.test.js +3 -3
- package/dist/__tests__/notifier-discord.test.js.map +1 -1
- package/dist/__tests__/notifier-slack.test.js +2 -2
- package/dist/__tests__/notifier-slack.test.js.map +1 -1
- package/dist/__tests__/notifier.test.js +1 -0
- package/dist/__tests__/notifier.test.js.map +1 -1
- package/dist/__tests__/openrouter-client.test.d.ts +2 -0
- package/dist/__tests__/openrouter-client.test.d.ts.map +1 -0
- package/dist/__tests__/openrouter-client.test.js +56 -0
- package/dist/__tests__/openrouter-client.test.js.map +1 -0
- package/dist/__tests__/openrouter-fanout.test.d.ts +2 -0
- package/dist/__tests__/openrouter-fanout.test.d.ts.map +1 -0
- package/dist/__tests__/openrouter-fanout.test.js +93 -0
- package/dist/__tests__/openrouter-fanout.test.js.map +1 -0
- package/dist/__tests__/post-fanout-comments.test.d.ts +2 -0
- package/dist/__tests__/post-fanout-comments.test.d.ts.map +1 -0
- package/dist/__tests__/post-fanout-comments.test.js +84 -0
- package/dist/__tests__/post-fanout-comments.test.js.map +1 -0
- package/dist/__tests__/qa-audit-events.test.d.ts +2 -0
- package/dist/__tests__/qa-audit-events.test.d.ts.map +1 -0
- package/dist/__tests__/qa-audit-events.test.js +57 -0
- package/dist/__tests__/qa-audit-events.test.js.map +1 -0
- package/dist/__tests__/qa-config.test.d.ts +2 -0
- package/dist/__tests__/qa-config.test.d.ts.map +1 -0
- package/dist/__tests__/qa-config.test.js +80 -0
- package/dist/__tests__/qa-config.test.js.map +1 -0
- package/dist/__tests__/qa-eval.test.d.ts +2 -0
- package/dist/__tests__/qa-eval.test.d.ts.map +1 -0
- package/dist/__tests__/qa-eval.test.js +146 -0
- package/dist/__tests__/qa-eval.test.js.map +1 -0
- package/dist/__tests__/qa-gap.test.d.ts +2 -0
- package/dist/__tests__/qa-gap.test.d.ts.map +1 -0
- package/dist/__tests__/qa-gap.test.js +100 -0
- package/dist/__tests__/qa-gap.test.js.map +1 -0
- package/dist/__tests__/qa-github.test.d.ts +2 -0
- package/dist/__tests__/qa-github.test.d.ts.map +1 -0
- package/dist/__tests__/qa-github.test.js +162 -0
- package/dist/__tests__/qa-github.test.js.map +1 -0
- package/dist/__tests__/ralph-review-fix-regression.test.js +1 -0
- package/dist/__tests__/ralph-review-fix-regression.test.js.map +1 -1
- package/dist/__tests__/release-manager-audit-events.test.d.ts +2 -0
- package/dist/__tests__/release-manager-audit-events.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-audit-events.test.js +64 -0
- package/dist/__tests__/release-manager-audit-events.test.js.map +1 -0
- package/dist/__tests__/release-manager-config.test.d.ts +2 -0
- package/dist/__tests__/release-manager-config.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-config.test.js +56 -0
- package/dist/__tests__/release-manager-config.test.js.map +1 -0
- package/dist/__tests__/release-manager-decide.test.d.ts +2 -0
- package/dist/__tests__/release-manager-decide.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-decide.test.js +102 -0
- package/dist/__tests__/release-manager-decide.test.js.map +1 -0
- package/dist/__tests__/release-manager-github.test.d.ts +2 -0
- package/dist/__tests__/release-manager-github.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-github.test.js +83 -0
- package/dist/__tests__/release-manager-github.test.js.map +1 -0
- package/dist/__tests__/release-manager-license-gate.test.d.ts +2 -0
- package/dist/__tests__/release-manager-license-gate.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-license-gate.test.js +25 -0
- package/dist/__tests__/release-manager-license-gate.test.js.map +1 -0
- package/dist/__tests__/release-manager-scheduler.test.d.ts +2 -0
- package/dist/__tests__/release-manager-scheduler.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-scheduler.test.js +395 -0
- package/dist/__tests__/release-manager-scheduler.test.js.map +1 -0
- package/dist/__tests__/release-manager-slack-handler.test.d.ts +2 -0
- package/dist/__tests__/release-manager-slack-handler.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-slack-handler.test.js +175 -0
- package/dist/__tests__/release-manager-slack-handler.test.js.map +1 -0
- package/dist/__tests__/release-manager-triggers.test.d.ts +2 -0
- package/dist/__tests__/release-manager-triggers.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-triggers.test.js +67 -0
- package/dist/__tests__/release-manager-triggers.test.js.map +1 -0
- package/dist/__tests__/release-manager-versioning.test.d.ts +2 -0
- package/dist/__tests__/release-manager-versioning.test.d.ts.map +1 -0
- package/dist/__tests__/release-manager-versioning.test.js +51 -0
- package/dist/__tests__/release-manager-versioning.test.js.map +1 -0
- package/dist/__tests__/review-feedback.test.js +166 -2
- package/dist/__tests__/review-feedback.test.js.map +1 -1
- package/dist/__tests__/review-prompt.test.d.ts +2 -0
- package/dist/__tests__/review-prompt.test.d.ts.map +1 -0
- package/dist/__tests__/review-prompt.test.js +88 -0
- package/dist/__tests__/review-prompt.test.js.map +1 -0
- package/dist/__tests__/review-provider-registry.test.d.ts +2 -0
- package/dist/__tests__/review-provider-registry.test.d.ts.map +1 -0
- package/dist/__tests__/review-provider-registry.test.js +61 -0
- package/dist/__tests__/review-provider-registry.test.js.map +1 -0
- package/dist/__tests__/runner-fanout-integration.test.d.ts +2 -0
- package/dist/__tests__/runner-fanout-integration.test.d.ts.map +1 -0
- package/dist/__tests__/runner-fanout-integration.test.js +107 -0
- package/dist/__tests__/runner-fanout-integration.test.js.map +1 -0
- package/dist/audit/events.d.ts +54 -0
- package/dist/audit/events.d.ts.map +1 -1
- package/dist/audit/events.js +100 -0
- package/dist/audit/events.js.map +1 -1
- package/dist/audit/writer.d.ts +22 -8
- package/dist/audit/writer.d.ts.map +1 -1
- package/dist/audit/writer.js +22 -8
- package/dist/audit/writer.js.map +1 -1
- package/dist/cost/aggregate.d.ts.map +1 -1
- package/dist/cost/aggregate.js +14 -2
- package/dist/cost/aggregate.js.map +1 -1
- package/dist/cost/csv.d.ts.map +1 -1
- package/dist/cost/csv.js +14 -2
- package/dist/cost/csv.js.map +1 -1
- package/dist/cost/per-run.d.ts +11 -0
- package/dist/cost/per-run.d.ts.map +1 -1
- package/dist/cost/per-run.js +21 -8
- package/dist/cost/per-run.js.map +1 -1
- package/dist/cost/rollup.d.ts.map +1 -1
- package/dist/cost/rollup.js +14 -2
- package/dist/cost/rollup.js.map +1 -1
- package/dist/db/client.d.ts.map +1 -1
- package/dist/db/client.js +20 -1
- package/dist/db/client.js.map +1 -1
- package/dist/db/migrations/postgres/009_review_model_runs.sql +18 -0
- package/dist/db/migrations/postgres/010_release_manager.sql +38 -0
- package/dist/db/migrations/postgres/011_qa_run_columns.sql +8 -0
- package/dist/db/migrations/postgres/012_qa_gap_issues.sql +18 -0
- package/dist/db/migrations/sqlite/008_review_model_runs.sql +18 -0
- package/dist/db/migrations/sqlite/009_release_manager.sql +43 -0
- package/dist/db/migrations/sqlite/010_qa_run_columns.sql +9 -0
- package/dist/db/migrations/sqlite/011_qa_gap_issues.sql +22 -0
- package/dist/db/review-model-runs.d.ts +4 -0
- package/dist/db/review-model-runs.d.ts.map +1 -0
- package/dist/db/review-model-runs.js +25 -0
- package/dist/db/review-model-runs.js.map +1 -0
- package/dist/db/schema.d.ts +761 -0
- package/dist/db/schema.d.ts.map +1 -1
- package/dist/db/schema.js +67 -0
- package/dist/db/schema.js.map +1 -1
- package/dist/executor/executor.d.ts +6 -1
- package/dist/executor/executor.d.ts.map +1 -1
- package/dist/executor/executor.js +3 -1
- package/dist/executor/executor.js.map +1 -1
- package/dist/executor/prompt/assembler.d.ts +5 -2
- package/dist/executor/prompt/assembler.d.ts.map +1 -1
- package/dist/executor/prompt/assembler.js +5 -2
- package/dist/executor/prompt/assembler.js.map +1 -1
- package/dist/executor/prompt/templates.d.ts +2 -2
- package/dist/executor/prompt/templates.d.ts.map +1 -1
- package/dist/executor/prompt/templates.js +21 -2
- package/dist/executor/prompt/templates.js.map +1 -1
- package/dist/executor/review/agentic-deep-review.d.ts +6 -0
- package/dist/executor/review/agentic-deep-review.d.ts.map +1 -0
- package/dist/executor/review/agentic-deep-review.js +37 -0
- package/dist/executor/review/agentic-deep-review.js.map +1 -0
- package/dist/executor/review/openrouter-client.d.ts +23 -0
- package/dist/executor/review/openrouter-client.d.ts.map +1 -0
- package/dist/executor/review/openrouter-client.js +34 -0
- package/dist/executor/review/openrouter-client.js.map +1 -0
- package/dist/executor/review/openrouter-fanout.d.ts +17 -0
- package/dist/executor/review/openrouter-fanout.d.ts.map +1 -0
- package/dist/executor/review/openrouter-fanout.js +84 -0
- package/dist/executor/review/openrouter-fanout.js.map +1 -0
- package/dist/executor/review/post-fanout-comments.d.ts +4 -0
- package/dist/executor/review/post-fanout-comments.d.ts.map +1 -0
- package/dist/executor/review/post-fanout-comments.js +45 -0
- package/dist/executor/review/post-fanout-comments.js.map +1 -0
- package/dist/executor/review/review-prompt.d.ts +21 -0
- package/dist/executor/review/review-prompt.d.ts.map +1 -0
- package/dist/executor/review/review-prompt.js +130 -0
- package/dist/executor/review/review-prompt.js.map +1 -0
- package/dist/executor/review/review-provider.d.ts +27 -0
- package/dist/executor/review/review-provider.d.ts.map +1 -0
- package/dist/executor/review/review-provider.js +36 -0
- package/dist/executor/review/review-provider.js.map +1 -0
- package/dist/executor/review/workdir-snapshot.d.ts +13 -0
- package/dist/executor/review/workdir-snapshot.d.ts.map +1 -0
- package/dist/executor/review/workdir-snapshot.js +24 -0
- package/dist/executor/review/workdir-snapshot.js.map +1 -0
- package/dist/index.d.ts +4 -0
- package/dist/index.d.ts.map +1 -1
- package/dist/index.js +4 -0
- package/dist/index.js.map +1 -1
- package/dist/license.d.ts.map +1 -1
- package/dist/license.js +1 -0
- package/dist/license.js.map +1 -1
- package/dist/pipeline/review-providers-runner.d.ts +31 -0
- package/dist/pipeline/review-providers-runner.d.ts.map +1 -0
- package/dist/pipeline/review-providers-runner.js +64 -0
- package/dist/pipeline/review-providers-runner.js.map +1 -0
- package/dist/pipeline/runner.d.ts +16 -1
- package/dist/pipeline/runner.d.ts.map +1 -1
- package/dist/pipeline/runner.js +137 -43
- package/dist/pipeline/runner.js.map +1 -1
- package/dist/pm/actions/promote.js +2 -2
- package/dist/pm/actions/promote.js.map +1 -1
- package/dist/pm/actions/resolve-approvals.js +3 -3
- package/dist/pm/actions/resolve-approvals.js.map +1 -1
- package/dist/pm/actions/triage.js +2 -2
- package/dist/pm/actions/triage.js.map +1 -1
- package/dist/pm/scheduler.js +3 -3
- package/dist/pm/scheduler.js.map +1 -1
- package/dist/pm/slack-interface.d.ts +8 -0
- package/dist/pm/slack-interface.d.ts.map +1 -1
- package/dist/pm/slack-interface.js +20 -3
- package/dist/pm/slack-interface.js.map +1 -1
- package/dist/qa/gap.d.ts +29 -0
- package/dist/qa/gap.d.ts.map +1 -0
- package/dist/qa/gap.js +91 -0
- package/dist/qa/gap.js.map +1 -0
- package/dist/qa/github.d.ts +77 -0
- package/dist/qa/github.d.ts.map +1 -0
- package/dist/qa/github.js +106 -0
- package/dist/qa/github.js.map +1 -0
- package/dist/qa/index.d.ts +4 -0
- package/dist/qa/index.d.ts.map +1 -0
- package/dist/qa/index.js +4 -0
- package/dist/qa/index.js.map +1 -0
- package/dist/qa/types.d.ts +44 -0
- package/dist/qa/types.d.ts.map +1 -0
- package/dist/qa/types.js +12 -0
- package/dist/qa/types.js.map +1 -0
- package/dist/release-manager/decide.d.ts +26 -0
- package/dist/release-manager/decide.d.ts.map +1 -0
- package/dist/release-manager/decide.js +58 -0
- package/dist/release-manager/decide.js.map +1 -0
- package/dist/release-manager/github.d.ts +47 -0
- package/dist/release-manager/github.d.ts.map +1 -0
- package/dist/release-manager/github.js +66 -0
- package/dist/release-manager/github.js.map +1 -0
- package/dist/release-manager/index.d.ts +9 -0
- package/dist/release-manager/index.d.ts.map +1 -0
- package/dist/release-manager/index.js +9 -0
- package/dist/release-manager/index.js.map +1 -0
- package/dist/release-manager/scheduler.d.ts +30 -0
- package/dist/release-manager/scheduler.d.ts.map +1 -0
- package/dist/release-manager/scheduler.js +427 -0
- package/dist/release-manager/scheduler.js.map +1 -0
- package/dist/release-manager/slack-handler.d.ts +39 -0
- package/dist/release-manager/slack-handler.d.ts.map +1 -0
- package/dist/release-manager/slack-handler.js +124 -0
- package/dist/release-manager/slack-handler.js.map +1 -0
- package/dist/release-manager/state.d.ts +18 -0
- package/dist/release-manager/state.d.ts.map +1 -0
- package/dist/release-manager/state.js +154 -0
- package/dist/release-manager/state.js.map +1 -0
- package/dist/release-manager/triggers.d.ts +32 -0
- package/dist/release-manager/triggers.d.ts.map +1 -0
- package/dist/release-manager/triggers.js +82 -0
- package/dist/release-manager/triggers.js.map +1 -0
- package/dist/release-manager/types.d.ts +79 -0
- package/dist/release-manager/types.d.ts.map +1 -0
- package/dist/release-manager/types.js +48 -0
- package/dist/release-manager/types.js.map +1 -0
- package/dist/release-manager/versioning.d.ts +20 -0
- package/dist/release-manager/versioning.d.ts.map +1 -0
- package/dist/release-manager/versioning.js +60 -0
- package/dist/release-manager/versioning.js.map +1 -0
- package/dist/types.d.ts +52 -0
- package/dist/types.d.ts.map +1 -1
- package/dist/types.js +18 -0
- package/dist/types.js.map +1 -1
- package/package.json +3 -1
|
@@ -0,0 +1,93 @@
|
|
|
1
|
+
import { describe, it, expect, vi, beforeEach } from "vitest";
|
|
2
|
+
const chatCompletion = vi.fn();
|
|
3
|
+
vi.mock("../executor/review/openrouter-client.js", () => ({
|
|
4
|
+
OpenRouterClient: class {
|
|
5
|
+
chatCompletion = chatCompletion;
|
|
6
|
+
},
|
|
7
|
+
}));
|
|
8
|
+
// Stub git diff + file collection so test does not need a real workdir
|
|
9
|
+
vi.mock("../executor/review/workdir-snapshot.js", () => ({
|
|
10
|
+
collectWorkdirSnapshot: async () => ({
|
|
11
|
+
diff: "diff --git a/x b/x\n+y",
|
|
12
|
+
files: [{ path: "x", body: "y" }],
|
|
13
|
+
}),
|
|
14
|
+
}));
|
|
15
|
+
const handoff = {
|
|
16
|
+
runId: "r1",
|
|
17
|
+
issueId: "i1",
|
|
18
|
+
stage: "review",
|
|
19
|
+
timestamp: "2026-04-30T00:00:00Z",
|
|
20
|
+
summary: "",
|
|
21
|
+
filesChanged: ["x"],
|
|
22
|
+
approach: "",
|
|
23
|
+
context: { issueIntent: "do x", constraints: [], assumptions: [] },
|
|
24
|
+
tokenBudget: { contextTokensUsed: 0, recommendedMaxTurns: 0 },
|
|
25
|
+
};
|
|
26
|
+
const ctx = () => ({
|
|
27
|
+
runId: "r1",
|
|
28
|
+
stageRunId: "s1",
|
|
29
|
+
workdir: "/tmp/wd",
|
|
30
|
+
handoff,
|
|
31
|
+
baseRef: "main",
|
|
32
|
+
prNumber: 42,
|
|
33
|
+
});
|
|
34
|
+
describe("OpenRouterFanoutProvider", () => {
|
|
35
|
+
beforeEach(() => { chatCompletion.mockReset(); });
|
|
36
|
+
const validJson = `{"findings":[{"severity":"warning","file":"a","line":1,"category":"quality","description":"d","fix":"f"}]}`;
|
|
37
|
+
it("runs N parallel calls and returns one ReviewModelRun per model", async () => {
|
|
38
|
+
chatCompletion.mockResolvedValue({ content: validJson, inputTokens: 10, outputTokens: 5 });
|
|
39
|
+
const { OpenRouterFanoutProvider } = await import("../executor/review/openrouter-fanout.js");
|
|
40
|
+
const p = new OpenRouterFanoutProvider({
|
|
41
|
+
apiKey: "k",
|
|
42
|
+
baseUrl: "https://x/api/v1",
|
|
43
|
+
models: ["m1", "m2", "m3"],
|
|
44
|
+
timeoutMs: 1000,
|
|
45
|
+
maxInputTokens: 100_000,
|
|
46
|
+
});
|
|
47
|
+
const runs = await p.runReview(ctx());
|
|
48
|
+
expect(runs).toHaveLength(3);
|
|
49
|
+
expect(runs.map((r) => r.modelId).sort()).toEqual(["m1", "m2", "m3"]);
|
|
50
|
+
expect(runs.every((r) => r.status === "completed")).toBe(true);
|
|
51
|
+
expect(chatCompletion).toHaveBeenCalledTimes(3);
|
|
52
|
+
});
|
|
53
|
+
it("partial failure: one model rejects, others complete", async () => {
|
|
54
|
+
chatCompletion
|
|
55
|
+
.mockResolvedValueOnce({ content: validJson, inputTokens: 1, outputTokens: 1 })
|
|
56
|
+
.mockRejectedValueOnce(new Error("boom"))
|
|
57
|
+
.mockResolvedValueOnce({ content: validJson, inputTokens: 1, outputTokens: 1 });
|
|
58
|
+
const { OpenRouterFanoutProvider } = await import("../executor/review/openrouter-fanout.js");
|
|
59
|
+
const p = new OpenRouterFanoutProvider({
|
|
60
|
+
apiKey: "k", baseUrl: "https://x/api/v1",
|
|
61
|
+
models: ["m1", "m2", "m3"], timeoutMs: 1000, maxInputTokens: 100_000,
|
|
62
|
+
});
|
|
63
|
+
const runs = await p.runReview(ctx());
|
|
64
|
+
const failed = runs.filter((r) => r.status === "failed");
|
|
65
|
+
const ok = runs.filter((r) => r.status === "completed");
|
|
66
|
+
expect(failed).toHaveLength(1);
|
|
67
|
+
expect(failed[0].errorMessage).toContain("boom");
|
|
68
|
+
expect(ok).toHaveLength(2);
|
|
69
|
+
});
|
|
70
|
+
it("malformed JSON output → run failed with parser error", async () => {
|
|
71
|
+
chatCompletion.mockResolvedValue({ content: "not json", inputTokens: 1, outputTokens: 1 });
|
|
72
|
+
const { OpenRouterFanoutProvider } = await import("../executor/review/openrouter-fanout.js");
|
|
73
|
+
const p = new OpenRouterFanoutProvider({
|
|
74
|
+
apiKey: "k", baseUrl: "https://x/api/v1",
|
|
75
|
+
models: ["m1"], timeoutMs: 1000, maxInputTokens: 100_000,
|
|
76
|
+
});
|
|
77
|
+
const runs = await p.runReview(ctx());
|
|
78
|
+
expect(runs[0].status).toBe("failed");
|
|
79
|
+
expect(runs[0].errorMessage).toContain("not parseable");
|
|
80
|
+
});
|
|
81
|
+
it("all models fail → returns N failed runs (does not throw)", async () => {
|
|
82
|
+
chatCompletion.mockRejectedValue(new Error("dead"));
|
|
83
|
+
const { OpenRouterFanoutProvider } = await import("../executor/review/openrouter-fanout.js");
|
|
84
|
+
const p = new OpenRouterFanoutProvider({
|
|
85
|
+
apiKey: "k", baseUrl: "https://x/api/v1",
|
|
86
|
+
models: ["m1", "m2"], timeoutMs: 1000, maxInputTokens: 100_000,
|
|
87
|
+
});
|
|
88
|
+
const runs = await p.runReview(ctx());
|
|
89
|
+
expect(runs).toHaveLength(2);
|
|
90
|
+
expect(runs.every((r) => r.status === "failed")).toBe(true);
|
|
91
|
+
});
|
|
92
|
+
});
|
|
93
|
+
//# sourceMappingURL=openrouter-fanout.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"openrouter-fanout.test.js","sourceRoot":"","sources":["../../src/__tests__/openrouter-fanout.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAG9D,MAAM,cAAc,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC/B,EAAE,CAAC,IAAI,CAAC,yCAAyC,EAAE,GAAG,EAAE,CAAC,CAAC;IACxD,gBAAgB,EAAE;QAChB,cAAc,GAAG,cAAc,CAAC;KACjC;CACF,CAAC,CAAC,CAAC;AACJ,uEAAuE;AACvE,EAAE,CAAC,IAAI,CAAC,wCAAwC,EAAE,GAAG,EAAE,CAAC,CAAC;IACvD,sBAAsB,EAAE,KAAK,IAAI,EAAE,CAAC,CAAC;QACnC,IAAI,EAAE,wBAAwB;QAC9B,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC;KAClC,CAAC;CACH,CAAC,CAAC,CAAC;AAEJ,MAAM,OAAO,GAAoB;IAC/B,KAAK,EAAE,IAAI;IACX,OAAO,EAAE,IAAI;IACb,KAAK,EAAE,QAAQ;IACf,SAAS,EAAE,sBAAsB;IACjC,OAAO,EAAE,EAAE;IACX,YAAY,EAAE,CAAC,GAAG,CAAC;IACnB,QAAQ,EAAE,EAAE;IACZ,OAAO,EAAE,EAAE,WAAW,EAAE,MAAM,EAAE,WAAW,EAAE,EAAE,EAAE,WAAW,EAAE,EAAE,EAAE;IAClE,WAAW,EAAE,EAAE,iBAAiB,EAAE,CAAC,EAAE,mBAAmB,EAAE,CAAC,EAAE;CAC9D,CAAC;AAEF,MAAM,GAAG,GAAG,GAAG,EAAE,CAAC,CAAC;IACjB,KAAK,EAAE,IAAI;IACX,UAAU,EAAE,IAAI;IAChB,OAAO,EAAE,SAAS;IAClB,OAAO;IACP,OAAO,EAAE,MAAM;IACf,QAAQ,EAAE,EAAE;CACb,CAAC,CAAC;AAEH,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,UAAU,CAAC,GAAG,EAAE,GAAG,cAAc,CAAC,SAAS,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;IAElD,MAAM,SAAS,GAAG,4GAA4G,CAAC;IAE/H,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC9E,cAAc,CAAC,iBAAiB,CAAC,EAAE,OAAO,EAAE,SAAS,EAAE,WAAW,EAAE,EAAE,EAAE,YAAY,EAAE,CAAC,EAAE,CAAC,CAAC;QAC3F,MAAM,EAAE,wBAAwB,EAAE,GAAG,MAAM,MAAM,CAC/C,yCAAyC,CAC1C,CAAC;QACF,MAAM,CAAC,GAAG,IAAI,wBAAwB,CAAC;YACrC,MAAM,EAAE,GAAG;YACX,OAAO,EAAE,kBAAkB;YAC3B,MAAM,EAAE,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC;YAC1B,SAAS,EAAE,IAAI;YACf,cAAc,EAAE,OAAO;SACxB,CAAC,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,CAAC,CAAC,SAAS,CAAC,GAAG,EAAE,CAAC,CAAC;QACtC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC;QACtE,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC/D,MAAM,CAAC,cAAc,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,cAAc;aACX,qBAAqB,CAAC,EAAE,OAAO,EAAE,SAAS,EAAE,WAAW,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE,CAAC;aAC9E,qBAAqB,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,CAAC;aACxC,qBAAqB,CAAC,EAAE,OAAO,EAAE,SAAS,EAAE,WAAW,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE,CAAC,CAAC;QAClF,MAAM,EAAE,wBAAwB,EAAE,GAAG,MAAM,MAAM,CAC/C,yCAAyC,CAC1C,CAAC;QACF,MAAM,CAAC,GAAG,IAAI,wBAAwB,CAAC;YACrC,MAAM,EAAE,GAAG,EAAE,OAAO,EAAE,kBAAkB;YACxC,MAAM,EAAE,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,EAAE,SAAS,EAAE,IAAI,EAAE,cAAc,EAAE,OAAO;SACrE,CAAC,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,CAAC,CAAC,SAAS,CAAC,GAAG,EAAE,CAAC,CAAC;QACtC,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC;QACzD,MAAM,EAAE,GAAG,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,WAAW,CAAC,CAAC;QACxD,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;QACjD,MAAM,CAAC,EAAE,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,cAAc,CAAC,iBAAiB,CAAC,EAAE,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE,CAAC,CAAC;QAC3F,MAAM,EAAE,wBAAwB,EAAE,GAAG,MAAM,MAAM,CAC/C,yCAAyC,CAC1C,CAAC;QACF,MAAM,CAAC,GAAG,IAAI,wBAAwB,CAAC;YACrC,MAAM,EAAE,GAAG,EAAE,OAAO,EAAE,kBAAkB;YACxC,MAAM,EAAE,CAAC,IAAI,CAAC,EAAE,SAAS,EAAE,IAAI,EAAE,cAAc,EAAE,OAAO;SACzD,CAAC,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,CAAC,CAAC,SAAS,CAAC,GAAG,EAAE,CAAC,CAAC;QACtC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACtC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;QACxE,cAAc,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC;QACpD,MAAM,EAAE,wBAAwB,EAAE,GAAG,MAAM,MAAM,CAC/C,yCAAyC,CAC1C,CAAC;QACF,MAAM,CAAC,GAAG,IAAI,wBAAwB,CAAC;YACrC,MAAM,EAAE,GAAG,EAAE,OAAO,EAAE,kBAAkB;YACxC,MAAM,EAAE,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,SAAS,EAAE,IAAI,EAAE,cAAc,EAAE,OAAO;SAC/D,CAAC,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,CAAC,CAAC,SAAS,CAAC,GAAG,EAAE,CAAC,CAAC;QACtC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC9D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"post-fanout-comments.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/post-fanout-comments.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,84 @@
|
|
|
1
|
+
import { describe, it, expect, vi, beforeEach } from "vitest";
|
|
2
|
+
const { addPRComment } = vi.hoisted(() => ({ addPRComment: vi.fn() }));
|
|
3
|
+
vi.mock("../repo/github.js", () => ({ addPRComment }));
|
|
4
|
+
const completedRun = {
|
|
5
|
+
modelId: "anthropic/claude-3.5-sonnet",
|
|
6
|
+
providerId: "openrouter",
|
|
7
|
+
status: "completed",
|
|
8
|
+
findings: [
|
|
9
|
+
{ severity: "warning", file: "a.ts", line: 1, category: "quality", description: "d", fix: "f" },
|
|
10
|
+
],
|
|
11
|
+
inputTokens: 100,
|
|
12
|
+
outputTokens: 50,
|
|
13
|
+
durationMs: 1000,
|
|
14
|
+
};
|
|
15
|
+
const failedRun = {
|
|
16
|
+
modelId: "openai/gpt-4o",
|
|
17
|
+
providerId: "openrouter",
|
|
18
|
+
status: "failed",
|
|
19
|
+
findings: [],
|
|
20
|
+
inputTokens: 0,
|
|
21
|
+
outputTokens: 0,
|
|
22
|
+
durationMs: 200,
|
|
23
|
+
errorMessage: "rate limited",
|
|
24
|
+
};
|
|
25
|
+
describe("postFanoutCommentsToPR", () => {
|
|
26
|
+
beforeEach(() => {
|
|
27
|
+
addPRComment.mockReset();
|
|
28
|
+
});
|
|
29
|
+
it("posts one comment per run with model id and findings table", async () => {
|
|
30
|
+
addPRComment.mockResolvedValue(undefined);
|
|
31
|
+
const { postFanoutCommentsToPR } = await import("../executor/review/post-fanout-comments.js");
|
|
32
|
+
await postFanoutCommentsToPR({}, "owner", "repo", 42, [completedRun]);
|
|
33
|
+
expect(addPRComment).toHaveBeenCalledOnce();
|
|
34
|
+
const body = addPRComment.mock.calls[0][4];
|
|
35
|
+
expect(body).toContain("anthropic/claude-3.5-sonnet");
|
|
36
|
+
expect(body).toContain("Status: completed");
|
|
37
|
+
expect(body).toContain("warning");
|
|
38
|
+
expect(body).toContain("a.ts");
|
|
39
|
+
expect(body).toContain("Advisory only");
|
|
40
|
+
});
|
|
41
|
+
it("posts a 'failed' comment with errorMessage", async () => {
|
|
42
|
+
addPRComment.mockResolvedValue(undefined);
|
|
43
|
+
const { postFanoutCommentsToPR } = await import("../executor/review/post-fanout-comments.js");
|
|
44
|
+
await postFanoutCommentsToPR({}, "o", "r", 1, [failedRun]);
|
|
45
|
+
const body = addPRComment.mock.calls[0][4];
|
|
46
|
+
expect(body).toContain("Status: failed");
|
|
47
|
+
expect(body).toContain("rate limited");
|
|
48
|
+
});
|
|
49
|
+
it("notes truncated input when truncatedFiles > 0", async () => {
|
|
50
|
+
addPRComment.mockResolvedValue(undefined);
|
|
51
|
+
const { postFanoutCommentsToPR } = await import("../executor/review/post-fanout-comments.js");
|
|
52
|
+
await postFanoutCommentsToPR({}, "o", "r", 1, [
|
|
53
|
+
{ ...completedRun, truncatedFiles: 3 },
|
|
54
|
+
]);
|
|
55
|
+
const body = addPRComment.mock.calls[0][4];
|
|
56
|
+
expect(body).toContain("input truncated");
|
|
57
|
+
expect(body).toContain("3");
|
|
58
|
+
});
|
|
59
|
+
it("escapes newlines and pipes in description cells", async () => {
|
|
60
|
+
const { postFanoutCommentsToPR } = await import("../executor/review/post-fanout-comments.js");
|
|
61
|
+
const runWithNewline = {
|
|
62
|
+
...completedRun,
|
|
63
|
+
findings: [
|
|
64
|
+
{ severity: "warning", file: "a.ts", line: 1, category: "quality",
|
|
65
|
+
description: "first line\nsecond line | with pipe", fix: "f" },
|
|
66
|
+
],
|
|
67
|
+
};
|
|
68
|
+
await postFanoutCommentsToPR({}, "o", "r", 1, [runWithNewline]);
|
|
69
|
+
const body = addPRComment.mock.calls[0][4];
|
|
70
|
+
expect(body).not.toMatch(/first line\nsecond line/); // newline must be replaced
|
|
71
|
+
expect(body).toContain("first line second line \\| with pipe");
|
|
72
|
+
});
|
|
73
|
+
it("continues posting remaining runs when one addPRComment rejects", async () => {
|
|
74
|
+
const { postFanoutCommentsToPR } = await import("../executor/review/post-fanout-comments.js");
|
|
75
|
+
addPRComment
|
|
76
|
+
.mockRejectedValueOnce(new Error("rate limit"))
|
|
77
|
+
.mockResolvedValueOnce(undefined);
|
|
78
|
+
// Should not throw
|
|
79
|
+
await expect(postFanoutCommentsToPR({}, "o", "r", 1, [completedRun, failedRun])).resolves.toBeUndefined();
|
|
80
|
+
// Both addPRComment calls were attempted
|
|
81
|
+
expect(addPRComment).toHaveBeenCalledTimes(2);
|
|
82
|
+
});
|
|
83
|
+
});
|
|
84
|
+
//# sourceMappingURL=post-fanout-comments.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"post-fanout-comments.test.js","sourceRoot":"","sources":["../../src/__tests__/post-fanout-comments.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAG9D,MAAM,EAAE,YAAY,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,YAAY,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC;AACvE,EAAE,CAAC,IAAI,CAAC,mBAAmB,EAAE,GAAG,EAAE,CAAC,CAAC,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC;AAEvD,MAAM,YAAY,GAAmB;IACnC,OAAO,EAAE,6BAA6B;IACtC,UAAU,EAAE,YAAY;IACxB,MAAM,EAAE,WAAW;IACnB,QAAQ,EAAE;QACR,EAAE,QAAQ,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,EAAE,QAAQ,EAAE,SAAS,EAAE,WAAW,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE;KAChG;IACD,WAAW,EAAE,GAAG;IAChB,YAAY,EAAE,EAAE;IAChB,UAAU,EAAE,IAAI;CACjB,CAAC;AAEF,MAAM,SAAS,GAAmB;IAChC,OAAO,EAAE,eAAe;IACxB,UAAU,EAAE,YAAY;IACxB,MAAM,EAAE,QAAQ;IAChB,QAAQ,EAAE,EAAE;IACZ,WAAW,EAAE,CAAC;IACd,YAAY,EAAE,CAAC;IACf,UAAU,EAAE,GAAG;IACf,YAAY,EAAE,cAAc;CAC7B,CAAC;AAEF,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;IACtC,UAAU,CAAC,GAAG,EAAE;QACd,YAAY,CAAC,SAAS,EAAE,CAAC;IAC3B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,YAAY,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAC1C,MAAM,EAAE,sBAAsB,EAAE,GAAG,MAAM,MAAM,CAC7C,4CAA4C,CAC7C,CAAC;QACF,MAAM,sBAAsB,CAAC,EAAW,EAAE,OAAO,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC;QAC/E,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,EAAE,CAAC;QAC5C,MAAM,IAAI,GAAI,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAY,CAAC;QACvD,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,6BAA6B,CAAC,CAAC;QACtD,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;QAC5C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QAClC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;QAC/B,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;QAC1D,YAAY,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAC1C,MAAM,EAAE,sBAAsB,EAAE,GAAG,MAAM,MAAM,CAC7C,4CAA4C,CAC7C,CAAC;QACF,MAAM,sBAAsB,CAAC,EAAW,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,CAAC,SAAS,CAAC,CAAC,CAAC;QACpE,MAAM,IAAI,GAAI,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAY,CAAC;QACvD,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QACzC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,cAAc,CAAC,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,YAAY,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAC1C,MAAM,EAAE,sBAAsB,EAAE,GAAG,MAAM,MAAM,CAC7C,4CAA4C,CAC7C,CAAC;QACF,MAAM,sBAAsB,CAAC,EAAW,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE;YACrD,EAAE,GAAG,YAAY,EAAE,cAAc,EAAE,CAAC,EAAE;SACvC,CAAC,CAAC;QACH,MAAM,IAAI,GAAI,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAY,CAAC;QACvD,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;QAC1C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IAC9B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,MAAM,EAAE,sBAAsB,EAAE,GAAG,MAAM,MAAM,CAC7C,4CAA4C,CAC7C,CAAC;QACF,MAAM,cAAc,GAAmB;YACrC,GAAG,YAAY;YACf,QAAQ,EAAE;gBACR,EAAE,QAAQ,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,EAAE,QAAQ,EAAE,SAAS;oBAC/D,WAAW,EAAE,qCAAqC,EAAE,GAAG,EAAE,GAAG,EAAE;aACjE;SACF,CAAC;QACF,MAAM,sBAAsB,CAAC,EAAW,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,CAAC,cAAc,CAAC,CAAC,CAAC;QACzE,MAAM,IAAI,GAAI,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAY,CAAC;QACvD,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,yBAAyB,CAAC,CAAC,CAAE,2BAA2B;QACjF,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,sCAAsC,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC9E,MAAM,EAAE,sBAAsB,EAAE,GAAG,MAAM,MAAM,CAC7C,4CAA4C,CAC7C,CAAC;QACF,YAAY;aACT,qBAAqB,CAAC,IAAI,KAAK,CAAC,YAAY,CAAC,CAAC;aAC9C,qBAAqB,CAAC,SAAS,CAAC,CAAC;QAEpC,mBAAmB;QACnB,MAAM,MAAM,CACV,sBAAsB,CAAC,EAAW,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,CAAC,YAAY,EAAE,SAAS,CAAC,CAAC,CAC5E,CAAC,QAAQ,CAAC,aAAa,EAAE,CAAC;QAC3B,yCAAyC;QACzC,MAAM,CAAC,YAAY,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"qa-audit-events.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/qa-audit-events.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,57 @@
|
|
|
1
|
+
import { describe, it, expect } from "vitest";
|
|
2
|
+
import { qaRunTriggeredEvent, qaRunCompletedEvent, qaGapIssueFiledEvent, } from "../audit/events.js";
|
|
3
|
+
import { AuditEventSchema } from "../types.js";
|
|
4
|
+
describe("qa audit events", () => {
|
|
5
|
+
it("qaRunTriggeredEvent passes schema validation", () => {
|
|
6
|
+
const evt = qaRunTriggeredEvent({
|
|
7
|
+
repoUrl: "https://github.com/org/repo",
|
|
8
|
+
branch: "main",
|
|
9
|
+
workflow: ".github/workflows/smoke.yml",
|
|
10
|
+
runId: 12345,
|
|
11
|
+
sha: "abcdef0",
|
|
12
|
+
});
|
|
13
|
+
expect(evt.eventType).toBe("qa.run_triggered");
|
|
14
|
+
expect(evt.actor).toBe("release-manager");
|
|
15
|
+
expect(evt.actorType).toBe("release-manager");
|
|
16
|
+
expect(evt.scope).toBe("repo:https://github.com/org/repo");
|
|
17
|
+
expect(() => AuditEventSchema.parse(evt)).not.toThrow();
|
|
18
|
+
expect(evt.payload.runId).toBe(12345);
|
|
19
|
+
});
|
|
20
|
+
it("qaRunCompletedEvent passes schema validation with conclusion", () => {
|
|
21
|
+
const evt = qaRunCompletedEvent({
|
|
22
|
+
repoUrl: "https://github.com/org/repo",
|
|
23
|
+
branch: "main",
|
|
24
|
+
runId: 12345,
|
|
25
|
+
conclusion: "success",
|
|
26
|
+
durationMs: 600_000,
|
|
27
|
+
});
|
|
28
|
+
expect(evt.eventType).toBe("qa.run_completed");
|
|
29
|
+
expect(() => AuditEventSchema.parse(evt)).not.toThrow();
|
|
30
|
+
expect(evt.payload.conclusion).toBe("success");
|
|
31
|
+
expect(evt.payload.durationMs).toBe(600_000);
|
|
32
|
+
});
|
|
33
|
+
it("qaRunCompletedEvent supports synthetic timeout flag", () => {
|
|
34
|
+
const evt = qaRunCompletedEvent({
|
|
35
|
+
repoUrl: "https://github.com/org/repo",
|
|
36
|
+
branch: "main",
|
|
37
|
+
runId: 12345,
|
|
38
|
+
conclusion: "timed_out",
|
|
39
|
+
durationMs: 1_800_000,
|
|
40
|
+
synthetic: true,
|
|
41
|
+
});
|
|
42
|
+
expect(() => AuditEventSchema.parse(evt)).not.toThrow();
|
|
43
|
+
expect(evt.payload.synthetic).toBe(true);
|
|
44
|
+
});
|
|
45
|
+
it("qaGapIssueFiledEvent passes schema validation", () => {
|
|
46
|
+
const evt = qaGapIssueFiledEvent({
|
|
47
|
+
repoUrl: "https://github.com/org/repo",
|
|
48
|
+
branch: "main",
|
|
49
|
+
workflowPath: ".github/workflows/smoke.yml",
|
|
50
|
+
linearIssueId: "BEC-150",
|
|
51
|
+
});
|
|
52
|
+
expect(evt.eventType).toBe("qa.gap_issue_filed");
|
|
53
|
+
expect(() => AuditEventSchema.parse(evt)).not.toThrow();
|
|
54
|
+
expect(evt.payload.linearIssueId).toBe("BEC-150");
|
|
55
|
+
});
|
|
56
|
+
});
|
|
57
|
+
//# sourceMappingURL=qa-audit-events.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"qa-audit-events.test.js","sourceRoot":"","sources":["../../src/__tests__/qa-audit-events.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EACL,mBAAmB,EACnB,mBAAmB,EACnB,oBAAoB,GACrB,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAE/C,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,GAAG,GAAG,mBAAmB,CAAC;YAC9B,OAAO,EAAE,6BAA6B;YACtC,MAAM,EAAE,MAAM;YACd,QAAQ,EAAE,6BAA6B;YACvC,KAAK,EAAE,KAAK;YACZ,GAAG,EAAE,SAAS;SACf,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;QAC/C,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;QAC1C,MAAM,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;QAC9C,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,kCAAkC,CAAC,CAAC;QAC3D,MAAM,CAAC,GAAG,EAAE,CAAC,gBAAgB,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;QACxD,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,MAAM,GAAG,GAAG,mBAAmB,CAAC;YAC9B,OAAO,EAAE,6BAA6B;YACtC,MAAM,EAAE,MAAM;YACd,KAAK,EAAE,KAAK;YACZ,UAAU,EAAE,SAAS;YACrB,UAAU,EAAE,OAAO;SACpB,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;QAC/C,MAAM,CAAC,GAAG,EAAE,CAAC,gBAAgB,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;QACxD,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QAC/C,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC7D,MAAM,GAAG,GAAG,mBAAmB,CAAC;YAC9B,OAAO,EAAE,6BAA6B;YACtC,MAAM,EAAE,MAAM;YACd,KAAK,EAAE,KAAK;YACZ,UAAU,EAAE,WAAW;YACvB,UAAU,EAAE,SAAS;YACrB,SAAS,EAAE,IAAI;SAChB,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,EAAE,CAAC,gBAAgB,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;QACxD,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,GAAG,GAAG,oBAAoB,CAAC;YAC/B,OAAO,EAAE,6BAA6B;YACtC,MAAM,EAAE,MAAM;YACd,YAAY,EAAE,6BAA6B;YAC3C,aAAa,EAAE,SAAS;SACzB,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,oBAAoB,CAAC,CAAC;QACjD,MAAM,CAAC,GAAG,EAAE,CAAC,gBAAgB,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;QACxD,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"qa-config.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/qa-config.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,80 @@
|
|
|
1
|
+
import { describe, it, expect } from "vitest";
|
|
2
|
+
import { QaCheckConfigSchema } from "../qa/types.js";
|
|
3
|
+
import { ReleaseManagerConfigSchema } from "../release-manager/types.js";
|
|
4
|
+
describe("QaCheckConfigSchema", () => {
|
|
5
|
+
it("parses minimal valid config", () => {
|
|
6
|
+
const cfg = QaCheckConfigSchema.parse({
|
|
7
|
+
workflow: ".github/workflows/smoke.yml",
|
|
8
|
+
linearTeamId: "team-uuid-123",
|
|
9
|
+
});
|
|
10
|
+
expect(cfg.workflow).toBe(".github/workflows/smoke.yml");
|
|
11
|
+
expect(cfg.timeoutMinutes).toBe(30); // default
|
|
12
|
+
expect(cfg.linearTeamId).toBe("team-uuid-123");
|
|
13
|
+
expect(cfg.workflowInputs).toBeUndefined();
|
|
14
|
+
});
|
|
15
|
+
it("respects custom timeoutMinutes", () => {
|
|
16
|
+
const cfg = QaCheckConfigSchema.parse({
|
|
17
|
+
workflow: ".github/workflows/smoke.yml",
|
|
18
|
+
linearTeamId: "team-uuid-123",
|
|
19
|
+
timeoutMinutes: 60,
|
|
20
|
+
});
|
|
21
|
+
expect(cfg.timeoutMinutes).toBe(60);
|
|
22
|
+
});
|
|
23
|
+
it("accepts workflowInputs object", () => {
|
|
24
|
+
const cfg = QaCheckConfigSchema.parse({
|
|
25
|
+
workflow: ".github/workflows/smoke.yml",
|
|
26
|
+
linearTeamId: "team-uuid-123",
|
|
27
|
+
workflowInputs: { environment: "preview" },
|
|
28
|
+
});
|
|
29
|
+
expect(cfg.workflowInputs).toEqual({ environment: "preview" });
|
|
30
|
+
});
|
|
31
|
+
it("rejects empty workflow string", () => {
|
|
32
|
+
expect(() => QaCheckConfigSchema.parse({
|
|
33
|
+
workflow: "",
|
|
34
|
+
linearTeamId: "team-uuid-123",
|
|
35
|
+
})).toThrow();
|
|
36
|
+
});
|
|
37
|
+
it("rejects empty linearTeamId", () => {
|
|
38
|
+
expect(() => QaCheckConfigSchema.parse({
|
|
39
|
+
workflow: ".github/workflows/smoke.yml",
|
|
40
|
+
linearTeamId: "",
|
|
41
|
+
})).toThrow();
|
|
42
|
+
});
|
|
43
|
+
it("rejects non-positive timeoutMinutes", () => {
|
|
44
|
+
expect(() => QaCheckConfigSchema.parse({
|
|
45
|
+
workflow: ".github/workflows/smoke.yml",
|
|
46
|
+
linearTeamId: "team-uuid-123",
|
|
47
|
+
timeoutMinutes: 0,
|
|
48
|
+
})).toThrow();
|
|
49
|
+
});
|
|
50
|
+
});
|
|
51
|
+
describe("ReleaseManagerConfigSchema with qaCheck", () => {
|
|
52
|
+
it("accepts a qaCheck trigger alongside existing triggers", () => {
|
|
53
|
+
const cfg = ReleaseManagerConfigSchema.parse({
|
|
54
|
+
enabled: true,
|
|
55
|
+
triggers: {
|
|
56
|
+
mergedPRsSince: 5,
|
|
57
|
+
qaCheck: {
|
|
58
|
+
workflow: ".github/workflows/smoke.yml",
|
|
59
|
+
linearTeamId: "team-uuid-123",
|
|
60
|
+
},
|
|
61
|
+
},
|
|
62
|
+
});
|
|
63
|
+
expect(cfg.triggers.qaCheck?.workflow).toBe(".github/workflows/smoke.yml");
|
|
64
|
+
expect(cfg.triggers.qaCheck?.timeoutMinutes).toBe(30);
|
|
65
|
+
});
|
|
66
|
+
it("treats qaCheck as a valid trigger to satisfy 'at least one trigger' guard", () => {
|
|
67
|
+
// qaCheck alone (no mergedPRsSince/timeSinceLastHours/ciGreenForMinutes/requireSlackApproval) should pass
|
|
68
|
+
const cfg = ReleaseManagerConfigSchema.parse({
|
|
69
|
+
enabled: true,
|
|
70
|
+
triggers: {
|
|
71
|
+
qaCheck: {
|
|
72
|
+
workflow: ".github/workflows/smoke.yml",
|
|
73
|
+
linearTeamId: "team-uuid-123",
|
|
74
|
+
},
|
|
75
|
+
},
|
|
76
|
+
});
|
|
77
|
+
expect(cfg.triggers.qaCheck).toBeDefined();
|
|
78
|
+
});
|
|
79
|
+
});
|
|
80
|
+
//# sourceMappingURL=qa-config.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"qa-config.test.js","sourceRoot":"","sources":["../../src/__tests__/qa-config.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,mBAAmB,EAAE,MAAM,gBAAgB,CAAC;AACrD,OAAO,EAAE,0BAA0B,EAAE,MAAM,6BAA6B,CAAC;AAEzE,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,GAAG,GAAG,mBAAmB,CAAC,KAAK,CAAC;YACpC,QAAQ,EAAE,6BAA6B;YACvC,YAAY,EAAE,eAAe;SAC9B,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAC;QACzD,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC,CAAC,UAAU;QAC/C,MAAM,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;QAC/C,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC,aAAa,EAAE,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;QACxC,MAAM,GAAG,GAAG,mBAAmB,CAAC,KAAK,CAAC;YACpC,QAAQ,EAAE,6BAA6B;YACvC,YAAY,EAAE,eAAe;YAC7B,cAAc,EAAE,EAAE;SACnB,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+BAA+B,EAAE,GAAG,EAAE;QACvC,MAAM,GAAG,GAAG,mBAAmB,CAAC,KAAK,CAAC;YACpC,QAAQ,EAAE,6BAA6B;YACvC,YAAY,EAAE,eAAe;YAC7B,cAAc,EAAE,EAAE,WAAW,EAAE,SAAS,EAAE;SAC3C,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,SAAS,EAAE,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+BAA+B,EAAE,GAAG,EAAE;QACvC,MAAM,CAAC,GAAG,EAAE,CACV,mBAAmB,CAAC,KAAK,CAAC;YACxB,QAAQ,EAAE,EAAE;YACZ,YAAY,EAAE,eAAe;SAC9B,CAAC,CACH,CAAC,OAAO,EAAE,CAAC;IACd,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;QACpC,MAAM,CAAC,GAAG,EAAE,CACV,mBAAmB,CAAC,KAAK,CAAC;YACxB,QAAQ,EAAE,6BAA6B;YACvC,YAAY,EAAE,EAAE;SACjB,CAAC,CACH,CAAC,OAAO,EAAE,CAAC;IACd,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,MAAM,CAAC,GAAG,EAAE,CACV,mBAAmB,CAAC,KAAK,CAAC;YACxB,QAAQ,EAAE,6BAA6B;YACvC,YAAY,EAAE,eAAe;YAC7B,cAAc,EAAE,CAAC;SAClB,CAAC,CACH,CAAC,OAAO,EAAE,CAAC;IACd,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,yCAAyC,EAAE,GAAG,EAAE;IACvD,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;QAC/D,MAAM,GAAG,GAAG,0BAA0B,CAAC,KAAK,CAAC;YAC3C,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE;gBACR,cAAc,EAAE,CAAC;gBACjB,OAAO,EAAE;oBACP,QAAQ,EAAE,6BAA6B;oBACvC,YAAY,EAAE,eAAe;iBAC9B;aACF;SACF,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAC;QAC3E,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,EAAE,cAAc,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2EAA2E,EAAE,GAAG,EAAE;QACnF,0GAA0G;QAC1G,MAAM,GAAG,GAAG,0BAA0B,CAAC,KAAK,CAAC;YAC3C,OAAO,EAAE,IAAI;YACb,QAAQ,EAAE;gBACR,OAAO,EAAE;oBACP,QAAQ,EAAE,6BAA6B;oBACvC,YAAY,EAAE,eAAe;iBAC9B;aACF;SACF,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,WAAW,EAAE,CAAC;IAC7C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"qa-eval.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/qa-eval.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,146 @@
|
|
|
1
|
+
import { describe, it, expect } from "vitest";
|
|
2
|
+
import { evalQaCheck } from "../release-manager/triggers.js";
|
|
3
|
+
const NOW = new Date("2026-05-04T12:00:00Z");
|
|
4
|
+
const cfg = {
|
|
5
|
+
workflow: ".github/workflows/smoke.yml",
|
|
6
|
+
timeoutMinutes: 30,
|
|
7
|
+
linearTeamId: "team-uuid-123",
|
|
8
|
+
};
|
|
9
|
+
describe("evalQaCheck", () => {
|
|
10
|
+
it("returns qa_no_workflow when workflowFileExists=false", () => {
|
|
11
|
+
const result = evalQaCheck({
|
|
12
|
+
qaConfig: cfg,
|
|
13
|
+
headSha: "head_sha_1",
|
|
14
|
+
workflowFileExists: false,
|
|
15
|
+
qaRun: null,
|
|
16
|
+
runConclusion: null,
|
|
17
|
+
now: NOW,
|
|
18
|
+
});
|
|
19
|
+
expect(result.pass).toBe(false);
|
|
20
|
+
expect(result.reason).toBe("qa_no_workflow");
|
|
21
|
+
});
|
|
22
|
+
it("returns qa_needs_trigger when workflow exists but no run for current SHA", () => {
|
|
23
|
+
const result = evalQaCheck({
|
|
24
|
+
qaConfig: cfg,
|
|
25
|
+
headSha: "head_sha_1",
|
|
26
|
+
workflowFileExists: true,
|
|
27
|
+
qaRun: null,
|
|
28
|
+
runConclusion: null,
|
|
29
|
+
now: NOW,
|
|
30
|
+
});
|
|
31
|
+
expect(result.pass).toBe(false);
|
|
32
|
+
expect(result.reason).toBe("qa_needs_trigger");
|
|
33
|
+
});
|
|
34
|
+
it("returns qa_needs_trigger when in-flight run is for stale SHA", () => {
|
|
35
|
+
const qaRun = {
|
|
36
|
+
runId: 99999,
|
|
37
|
+
runSha: "old_sha",
|
|
38
|
+
triggeredAt: new Date(NOW.getTime() - 5 * 60 * 1000),
|
|
39
|
+
};
|
|
40
|
+
const result = evalQaCheck({
|
|
41
|
+
qaConfig: cfg,
|
|
42
|
+
headSha: "head_sha_1",
|
|
43
|
+
workflowFileExists: true,
|
|
44
|
+
qaRun,
|
|
45
|
+
runConclusion: null,
|
|
46
|
+
now: NOW,
|
|
47
|
+
});
|
|
48
|
+
expect(result.pass).toBe(false);
|
|
49
|
+
expect(result.reason).toBe("qa_needs_trigger");
|
|
50
|
+
});
|
|
51
|
+
it("returns qa_running when run is in flight for current SHA, not yet timed out", () => {
|
|
52
|
+
const qaRun = {
|
|
53
|
+
runId: 99999,
|
|
54
|
+
runSha: "head_sha_1",
|
|
55
|
+
triggeredAt: new Date(NOW.getTime() - 5 * 60 * 1000), // 5 min ago, < 30 min timeout
|
|
56
|
+
};
|
|
57
|
+
const result = evalQaCheck({
|
|
58
|
+
qaConfig: cfg,
|
|
59
|
+
headSha: "head_sha_1",
|
|
60
|
+
workflowFileExists: true,
|
|
61
|
+
qaRun,
|
|
62
|
+
runConclusion: null,
|
|
63
|
+
now: NOW,
|
|
64
|
+
});
|
|
65
|
+
expect(result.pass).toBe(false);
|
|
66
|
+
expect(result.reason).toBe("qa_running");
|
|
67
|
+
if (result.pass === false && result.reason === "qa_running") {
|
|
68
|
+
expect(result.runId).toBe(99999);
|
|
69
|
+
}
|
|
70
|
+
});
|
|
71
|
+
it("returns qa_timed_out when run has been running > timeoutMinutes", () => {
|
|
72
|
+
const qaRun = {
|
|
73
|
+
runId: 99999,
|
|
74
|
+
runSha: "head_sha_1",
|
|
75
|
+
triggeredAt: new Date(NOW.getTime() - 35 * 60 * 1000), // 35 min ago, > 30 min timeout
|
|
76
|
+
};
|
|
77
|
+
const result = evalQaCheck({
|
|
78
|
+
qaConfig: cfg,
|
|
79
|
+
headSha: "head_sha_1",
|
|
80
|
+
workflowFileExists: true,
|
|
81
|
+
qaRun,
|
|
82
|
+
runConclusion: null,
|
|
83
|
+
now: NOW,
|
|
84
|
+
});
|
|
85
|
+
expect(result.pass).toBe(false);
|
|
86
|
+
expect(result.reason).toBe("qa_timed_out");
|
|
87
|
+
});
|
|
88
|
+
it("returns pass:true when run completed with conclusion=success", () => {
|
|
89
|
+
const qaRun = {
|
|
90
|
+
runId: 99999,
|
|
91
|
+
runSha: "head_sha_1",
|
|
92
|
+
triggeredAt: new Date(NOW.getTime() - 10 * 60 * 1000),
|
|
93
|
+
};
|
|
94
|
+
const result = evalQaCheck({
|
|
95
|
+
qaConfig: cfg,
|
|
96
|
+
headSha: "head_sha_1",
|
|
97
|
+
workflowFileExists: true,
|
|
98
|
+
qaRun,
|
|
99
|
+
runConclusion: "success",
|
|
100
|
+
now: NOW,
|
|
101
|
+
});
|
|
102
|
+
expect(result.pass).toBe(true);
|
|
103
|
+
expect(result.reason).toMatch(/qa passed/i);
|
|
104
|
+
});
|
|
105
|
+
it("returns qa_failed for conclusion=failure", () => {
|
|
106
|
+
const qaRun = {
|
|
107
|
+
runId: 99999,
|
|
108
|
+
runSha: "head_sha_1",
|
|
109
|
+
triggeredAt: new Date(NOW.getTime() - 10 * 60 * 1000),
|
|
110
|
+
};
|
|
111
|
+
const result = evalQaCheck({
|
|
112
|
+
qaConfig: cfg,
|
|
113
|
+
headSha: "head_sha_1",
|
|
114
|
+
workflowFileExists: true,
|
|
115
|
+
qaRun,
|
|
116
|
+
runConclusion: "failure",
|
|
117
|
+
now: NOW,
|
|
118
|
+
});
|
|
119
|
+
expect(result.pass).toBe(false);
|
|
120
|
+
expect(result.reason).toBe("qa_failed");
|
|
121
|
+
if (result.pass === false && result.reason === "qa_failed") {
|
|
122
|
+
expect(result.conclusion).toBe("failure");
|
|
123
|
+
}
|
|
124
|
+
});
|
|
125
|
+
it("returns qa_failed for conclusion=cancelled", () => {
|
|
126
|
+
const qaRun = {
|
|
127
|
+
runId: 99999,
|
|
128
|
+
runSha: "head_sha_1",
|
|
129
|
+
triggeredAt: new Date(NOW.getTime() - 10 * 60 * 1000),
|
|
130
|
+
};
|
|
131
|
+
const result = evalQaCheck({
|
|
132
|
+
qaConfig: cfg,
|
|
133
|
+
headSha: "head_sha_1",
|
|
134
|
+
workflowFileExists: true,
|
|
135
|
+
qaRun,
|
|
136
|
+
runConclusion: "cancelled",
|
|
137
|
+
now: NOW,
|
|
138
|
+
});
|
|
139
|
+
expect(result.pass).toBe(false);
|
|
140
|
+
expect(result.reason).toBe("qa_failed");
|
|
141
|
+
if (result.pass === false && result.reason === "qa_failed") {
|
|
142
|
+
expect(result.conclusion).toBe("cancelled");
|
|
143
|
+
}
|
|
144
|
+
});
|
|
145
|
+
});
|
|
146
|
+
//# sourceMappingURL=qa-eval.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"qa-eval.test.js","sourceRoot":"","sources":["../../src/__tests__/qa-eval.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,WAAW,EAAE,MAAM,gCAAgC,CAAC;AAG7D,MAAM,GAAG,GAAG,IAAI,IAAI,CAAC,sBAAsB,CAAC,CAAC;AAC7C,MAAM,GAAG,GAAkB;IACzB,QAAQ,EAAE,6BAA6B;IACvC,cAAc,EAAE,EAAE;IAClB,YAAY,EAAE,eAAe;CAC9B,CAAC;AAEF,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;IAC3B,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,MAAM,GAAG,WAAW,CAAC;YACzB,QAAQ,EAAE,GAAG;YACb,OAAO,EAAE,YAAY;YACrB,kBAAkB,EAAE,KAAK;YACzB,KAAK,EAAE,IAAI;YACX,aAAa,EAAE,IAAI;YACnB,GAAG,EAAE,GAAG;SACT,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0EAA0E,EAAE,GAAG,EAAE;QAClF,MAAM,MAAM,GAAG,WAAW,CAAC;YACzB,QAAQ,EAAE,GAAG;YACb,OAAO,EAAE,YAAY;YACrB,kBAAkB,EAAE,IAAI;YACxB,KAAK,EAAE,IAAI;YACX,aAAa,EAAE,IAAI;YACnB,GAAG,EAAE,GAAG;SACT,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,MAAM,KAAK,GAAkB;YAC3B,KAAK,EAAE,KAAK;YACZ,MAAM,EAAE,SAAS;YACjB,WAAW,EAAE,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC;SACrD,CAAC;QACF,MAAM,MAAM,GAAG,WAAW,CAAC;YACzB,QAAQ,EAAE,GAAG;YACb,OAAO,EAAE,YAAY;YACrB,kBAAkB,EAAE,IAAI;YACxB,KAAK;YACL,aAAa,EAAE,IAAI;YACnB,GAAG,EAAE,GAAG;SACT,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,GAAG,EAAE;QACrF,MAAM,KAAK,GAAkB;YAC3B,KAAK,EAAE,KAAK;YACZ,MAAM,EAAE,YAAY;YACpB,WAAW,EAAE,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,EAAE,8BAA8B;SACrF,CAAC;QACF,MAAM,MAAM,GAAG,WAAW,CAAC;YACzB,QAAQ,EAAE,GAAG;YACb,OAAO,EAAE,YAAY;YACrB,kBAAkB,EAAE,IAAI;YACxB,KAAK;YACL,aAAa,EAAE,IAAI;YACnB,GAAG,EAAE,GAAG;SACT,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QACzC,IAAI,MAAM,CAAC,IAAI,KAAK,KAAK,IAAI,MAAM,CAAC,MAAM,KAAK,YAAY,EAAE,CAAC;YAC5D,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACnC,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QACzE,MAAM,KAAK,GAAkB;YAC3B,KAAK,EAAE,KAAK;YACZ,MAAM,EAAE,YAAY;YACpB,WAAW,EAAE,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC,EAAE,+BAA+B;SACvF,CAAC;QACF,MAAM,MAAM,GAAG,WAAW,CAAC;YACzB,QAAQ,EAAE,GAAG;YACb,OAAO,EAAE,YAAY;YACrB,kBAAkB,EAAE,IAAI;YACxB,KAAK;YACL,aAAa,EAAE,IAAI;YACnB,GAAG,EAAE,GAAG;SACT,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,MAAM,KAAK,GAAkB;YAC3B,KAAK,EAAE,KAAK;YACZ,MAAM,EAAE,YAAY;YACpB,WAAW,EAAE,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC;SACtD,CAAC;QACF,MAAM,MAAM,GAAG,WAAW,CAAC;YACzB,QAAQ,EAAE,GAAG;YACb,OAAO,EAAE,YAAY;YACrB,kBAAkB,EAAE,IAAI;YACxB,KAAK;YACL,aAAa,EAAE,SAAS;YACxB,GAAG,EAAE,GAAG;SACT,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,KAAK,GAAkB;YAC3B,KAAK,EAAE,KAAK;YACZ,MAAM,EAAE,YAAY;YACpB,WAAW,EAAE,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC;SACtD,CAAC;QACF,MAAM,MAAM,GAAG,WAAW,CAAC;YACzB,QAAQ,EAAE,GAAG;YACb,OAAO,EAAE,YAAY;YACrB,kBAAkB,EAAE,IAAI;YACxB,KAAK;YACL,aAAa,EAAE,SAAS;YACxB,GAAG,EAAE,GAAG;SACT,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACxC,IAAI,MAAM,CAAC,IAAI,KAAK,KAAK,IAAI,MAAM,CAAC,MAAM,KAAK,WAAW,EAAE,CAAC;YAC3D,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QAC5C,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,KAAK,GAAkB;YAC3B,KAAK,EAAE,KAAK;YACZ,MAAM,EAAE,YAAY;YACpB,WAAW,EAAE,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC;SACtD,CAAC;QACF,MAAM,MAAM,GAAG,WAAW,CAAC;YACzB,QAAQ,EAAE,GAAG;YACb,OAAO,EAAE,YAAY;YACrB,kBAAkB,EAAE,IAAI;YACxB,KAAK;YACL,aAAa,EAAE,WAAW;YAC1B,GAAG,EAAE,GAAG;SACT,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACxC,IAAI,MAAM,CAAC,IAAI,KAAK,KAAK,IAAI,MAAM,CAAC,MAAM,KAAK,WAAW,EAAE,CAAC;YAC3D,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QAC9C,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"qa-gap.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/qa-gap.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,100 @@
|
|
|
1
|
+
import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
|
|
2
|
+
import { randomBytes } from "node:crypto";
|
|
3
|
+
import { unlinkSync } from "node:fs";
|
|
4
|
+
import { eq, and, isNull } from "drizzle-orm";
|
|
5
|
+
import { createDb } from "../db/index.js";
|
|
6
|
+
import { qaGapIssues } from "../db/schema.js";
|
|
7
|
+
import { fileGapIssue } from "../qa/gap.js";
|
|
8
|
+
function tmpDbPath() {
|
|
9
|
+
const id = randomBytes(8).toString("hex");
|
|
10
|
+
return `/tmp/laf-qa-gap-fn-${id}.sqlite`;
|
|
11
|
+
}
|
|
12
|
+
describe("fileGapIssue", () => {
|
|
13
|
+
const paths = [];
|
|
14
|
+
let db;
|
|
15
|
+
const repoUrl = "https://github.com/org/repo";
|
|
16
|
+
const branch = "main";
|
|
17
|
+
const workflowPath = ".github/workflows/smoke.yml";
|
|
18
|
+
beforeEach(async () => {
|
|
19
|
+
const path = tmpDbPath();
|
|
20
|
+
paths.push(path);
|
|
21
|
+
const created = await createDb({ driver: "sqlite", connectionString: path });
|
|
22
|
+
db = created;
|
|
23
|
+
});
|
|
24
|
+
afterEach(() => {
|
|
25
|
+
for (const p of paths) {
|
|
26
|
+
try {
|
|
27
|
+
unlinkSync(p);
|
|
28
|
+
}
|
|
29
|
+
catch { }
|
|
30
|
+
try {
|
|
31
|
+
unlinkSync(p + "-wal");
|
|
32
|
+
}
|
|
33
|
+
catch { }
|
|
34
|
+
try {
|
|
35
|
+
unlinkSync(p + "-shm");
|
|
36
|
+
}
|
|
37
|
+
catch { }
|
|
38
|
+
}
|
|
39
|
+
paths.length = 0;
|
|
40
|
+
});
|
|
41
|
+
function makeMockLinear(over = {}) {
|
|
42
|
+
return {
|
|
43
|
+
createIssue: vi.fn(async () => ({
|
|
44
|
+
issue: Promise.resolve({ identifier: "BEC-150", url: "https://linear.app/beckerspace/issue/BEC-150" }),
|
|
45
|
+
})),
|
|
46
|
+
...over,
|
|
47
|
+
};
|
|
48
|
+
}
|
|
49
|
+
it("files a new issue when none exists, persists qa_gap_issues row", async () => {
|
|
50
|
+
const linear = makeMockLinear();
|
|
51
|
+
const result = await fileGapIssue({
|
|
52
|
+
db,
|
|
53
|
+
linear: linear,
|
|
54
|
+
repoUrl,
|
|
55
|
+
branch,
|
|
56
|
+
workflowPath,
|
|
57
|
+
linearTeamId: "team-uuid-123",
|
|
58
|
+
});
|
|
59
|
+
expect(result.kind).toBe("filed");
|
|
60
|
+
expect(result.kind === "filed" && result.linearIssueId).toBe("BEC-150");
|
|
61
|
+
expect(linear.createIssue).toHaveBeenCalledTimes(1);
|
|
62
|
+
const rows = await db.select().from(qaGapIssues).where(and(eq(qaGapIssues.repoUrl, repoUrl), isNull(qaGapIssues.resolvedAt)));
|
|
63
|
+
expect(rows).toHaveLength(1);
|
|
64
|
+
expect(rows[0].linearIssueId).toBe("BEC-150");
|
|
65
|
+
});
|
|
66
|
+
it("is idempotent — second call with existing open row is a no-op", async () => {
|
|
67
|
+
const linear = makeMockLinear();
|
|
68
|
+
await fileGapIssue({ db, linear: linear, repoUrl, branch, workflowPath, linearTeamId: "team-uuid-123" });
|
|
69
|
+
linear.createIssue.mockClear();
|
|
70
|
+
const result = await fileGapIssue({ db, linear: linear, repoUrl, branch, workflowPath, linearTeamId: "team-uuid-123" });
|
|
71
|
+
expect(result.kind).toBe("already_filed");
|
|
72
|
+
expect(result.kind === "already_filed" && result.linearIssueId).toBe("BEC-150");
|
|
73
|
+
expect(linear.createIssue).not.toHaveBeenCalled();
|
|
74
|
+
});
|
|
75
|
+
it("re-files when the previous row was resolved", async () => {
|
|
76
|
+
const linear = makeMockLinear();
|
|
77
|
+
await fileGapIssue({ db, linear: linear, repoUrl, branch, workflowPath, linearTeamId: "team-uuid-123" });
|
|
78
|
+
// Resolve the existing row
|
|
79
|
+
await db.update(qaGapIssues)
|
|
80
|
+
.set({ resolvedAt: new Date() })
|
|
81
|
+
.where(eq(qaGapIssues.linearIssueId, "BEC-150"));
|
|
82
|
+
// Update the mock to return a new ID for the second filing
|
|
83
|
+
linear.createIssue.mockResolvedValueOnce({
|
|
84
|
+
issue: Promise.resolve({ identifier: "BEC-160", url: "https://linear.app/beckerspace/issue/BEC-160" }),
|
|
85
|
+
});
|
|
86
|
+
const result = await fileGapIssue({ db, linear: linear, repoUrl, branch, workflowPath, linearTeamId: "team-uuid-123" });
|
|
87
|
+
expect(result.kind).toBe("filed");
|
|
88
|
+
expect(result.kind === "filed" && result.linearIssueId).toBe("BEC-160");
|
|
89
|
+
expect(linear.createIssue).toHaveBeenCalledTimes(2);
|
|
90
|
+
});
|
|
91
|
+
it("returns linear_error on Linear API failure", async () => {
|
|
92
|
+
const linear = makeMockLinear({
|
|
93
|
+
createIssue: vi.fn(async () => { throw new Error("Linear unauthorized"); }),
|
|
94
|
+
});
|
|
95
|
+
const result = await fileGapIssue({ db, linear: linear, repoUrl, branch, workflowPath, linearTeamId: "team-uuid-123" });
|
|
96
|
+
expect(result.kind).toBe("linear_error");
|
|
97
|
+
expect(result.kind === "linear_error" && result.message).toMatch(/unauthorized/);
|
|
98
|
+
});
|
|
99
|
+
});
|
|
100
|
+
//# sourceMappingURL=qa-gap.test.js.map
|