@urateam/core 0.1.16 → 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__/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-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.map +1 -1
- package/dist/executor/executor.js +2 -0
- package/dist/executor/executor.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.map +1 -1
- package/dist/pipeline/runner.js +99 -7
- 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 +48 -0
- package/dist/types.d.ts.map +1 -1
- package/dist/types.js +8 -0
- package/dist/types.js.map +1 -1
- package/package.json +3 -1
|
@@ -0,0 +1,175 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
|
|
2
|
+
import { randomBytes } from "node:crypto";
|
|
3
|
+
import { unlinkSync } from "node:fs";
|
|
4
|
+
import { eq, isNull, and } from "drizzle-orm";
|
|
5
|
+
import { createDb } from "../db/index.js";
|
|
6
|
+
import { releaseApprovals, releaseDecisions } from "../db/schema.js";
|
|
7
|
+
import { parseReleaseSubcommand, handleReleaseSubcommand, } from "../release-manager/slack-handler.js";
|
|
8
|
+
function tmpDbPath() {
|
|
9
|
+
const id = randomBytes(8).toString("hex");
|
|
10
|
+
return `/tmp/laf-rm-slack-${id}.sqlite`;
|
|
11
|
+
}
|
|
12
|
+
describe("parseReleaseSubcommand", () => {
|
|
13
|
+
it("parses 'approve'", () => {
|
|
14
|
+
expect(parseReleaseSubcommand("approve")).toEqual({ kind: "approve" });
|
|
15
|
+
expect(parseReleaseSubcommand(" APPROVE ")).toEqual({ kind: "approve" });
|
|
16
|
+
});
|
|
17
|
+
it("parses 'skip <reason>'", () => {
|
|
18
|
+
expect(parseReleaseSubcommand("skip the world is on fire")).toEqual({
|
|
19
|
+
kind: "skip",
|
|
20
|
+
reason: "the world is on fire",
|
|
21
|
+
});
|
|
22
|
+
});
|
|
23
|
+
it("returns help on bare 'skip' (no reason)", () => {
|
|
24
|
+
expect(parseReleaseSubcommand("skip")).toEqual({ kind: "unknown", original: "skip" });
|
|
25
|
+
});
|
|
26
|
+
it("parses 'status'", () => {
|
|
27
|
+
expect(parseReleaseSubcommand("status")).toEqual({ kind: "status" });
|
|
28
|
+
});
|
|
29
|
+
it("returns unknown for empty / garbage", () => {
|
|
30
|
+
expect(parseReleaseSubcommand("")).toEqual({ kind: "unknown", original: "" });
|
|
31
|
+
expect(parseReleaseSubcommand("foo bar")).toEqual({ kind: "unknown", original: "foo bar" });
|
|
32
|
+
});
|
|
33
|
+
});
|
|
34
|
+
describe("handleReleaseSubcommand", () => {
|
|
35
|
+
const paths = [];
|
|
36
|
+
let db;
|
|
37
|
+
const repoUrl = "https://github.com/org/repo";
|
|
38
|
+
const branch = "main";
|
|
39
|
+
beforeEach(async () => {
|
|
40
|
+
const path = tmpDbPath();
|
|
41
|
+
paths.push(path);
|
|
42
|
+
const created = await createDb({ driver: "sqlite", connectionString: path });
|
|
43
|
+
db = created;
|
|
44
|
+
});
|
|
45
|
+
afterEach(() => {
|
|
46
|
+
for (const p of paths) {
|
|
47
|
+
try {
|
|
48
|
+
unlinkSync(p);
|
|
49
|
+
}
|
|
50
|
+
catch { }
|
|
51
|
+
try {
|
|
52
|
+
unlinkSync(p + "-wal");
|
|
53
|
+
}
|
|
54
|
+
catch { }
|
|
55
|
+
try {
|
|
56
|
+
unlinkSync(p + "-shm");
|
|
57
|
+
}
|
|
58
|
+
catch { }
|
|
59
|
+
}
|
|
60
|
+
paths.length = 0;
|
|
61
|
+
});
|
|
62
|
+
it("approve writes a release_approvals row and returns confirmation", async () => {
|
|
63
|
+
const r = await handleReleaseSubcommand({
|
|
64
|
+
cmd: { kind: "approve" },
|
|
65
|
+
db,
|
|
66
|
+
repoUrl,
|
|
67
|
+
branch,
|
|
68
|
+
slackUserId: "U123",
|
|
69
|
+
});
|
|
70
|
+
expect(r.text).toMatch(/Approved/i);
|
|
71
|
+
expect(r.responseType).toBe("in_channel");
|
|
72
|
+
const rows = await db.select().from(releaseApprovals).where(and(eq(releaseApprovals.repoUrl, repoUrl), isNull(releaseApprovals.consumedAt)));
|
|
73
|
+
expect(rows).toHaveLength(1);
|
|
74
|
+
expect(rows[0].approvedBy).toBe("U123");
|
|
75
|
+
});
|
|
76
|
+
it("approve is idempotent — second approve from same user returns the same friendly response (UNIQUE catches it)", async () => {
|
|
77
|
+
await handleReleaseSubcommand({
|
|
78
|
+
cmd: { kind: "approve" }, db, repoUrl, branch, slackUserId: "U123",
|
|
79
|
+
});
|
|
80
|
+
const r = await handleReleaseSubcommand({
|
|
81
|
+
cmd: { kind: "approve" }, db, repoUrl, branch, slackUserId: "U123",
|
|
82
|
+
});
|
|
83
|
+
expect(r.text).toMatch(/already approved|Approved/i);
|
|
84
|
+
});
|
|
85
|
+
it("skip <reason> writes a release_decisions row and signals the scheduler to pause", async () => {
|
|
86
|
+
const r = await handleReleaseSubcommand({
|
|
87
|
+
cmd: { kind: "skip", reason: "deployment freeze" },
|
|
88
|
+
db,
|
|
89
|
+
repoUrl,
|
|
90
|
+
branch,
|
|
91
|
+
slackUserId: "U123",
|
|
92
|
+
});
|
|
93
|
+
expect(r.text).toMatch(/skipped/i);
|
|
94
|
+
expect(r.text).toMatch(/deployment freeze/);
|
|
95
|
+
expect(r.responseType).toBe("in_channel");
|
|
96
|
+
const rows = await db.select().from(releaseDecisions).where(eq(releaseDecisions.repoUrl, repoUrl));
|
|
97
|
+
expect(rows).toHaveLength(1);
|
|
98
|
+
expect(rows[0].decision).toBe("skip");
|
|
99
|
+
expect(rows[0].reason).toMatch(/manual:deployment freeze/);
|
|
100
|
+
});
|
|
101
|
+
it("status returns ephemeral with last 5 decisions", async () => {
|
|
102
|
+
// Seed 6 decision rows
|
|
103
|
+
for (let i = 0; i < 6; i++) {
|
|
104
|
+
await db.insert(releaseDecisions).values({
|
|
105
|
+
id: `rd_${i}`,
|
|
106
|
+
repoUrl,
|
|
107
|
+
branch,
|
|
108
|
+
decidedAt: new Date(Date.now() - (6 - i) * 60_000),
|
|
109
|
+
decision: "skip",
|
|
110
|
+
reason: `reason_${i}`,
|
|
111
|
+
triggerStateJson: "{}",
|
|
112
|
+
attemptCount: 0,
|
|
113
|
+
});
|
|
114
|
+
}
|
|
115
|
+
const r = await handleReleaseSubcommand({
|
|
116
|
+
cmd: { kind: "status" },
|
|
117
|
+
db,
|
|
118
|
+
repoUrl,
|
|
119
|
+
branch,
|
|
120
|
+
slackUserId: "U123",
|
|
121
|
+
});
|
|
122
|
+
expect(r.responseType).toBe("ephemeral");
|
|
123
|
+
expect(r.text).toMatch(/Recent decisions/);
|
|
124
|
+
// Most recent 5 — reason_5..reason_1 — are present, reason_0 is not
|
|
125
|
+
expect(r.text).toMatch(/reason_5/);
|
|
126
|
+
expect(r.text).not.toMatch(/reason_0/);
|
|
127
|
+
});
|
|
128
|
+
it("unknown returns a help message", async () => {
|
|
129
|
+
const r = await handleReleaseSubcommand({
|
|
130
|
+
cmd: { kind: "unknown", original: "frobnicate" },
|
|
131
|
+
db,
|
|
132
|
+
repoUrl,
|
|
133
|
+
branch,
|
|
134
|
+
slackUserId: "U123",
|
|
135
|
+
});
|
|
136
|
+
expect(r.text).toMatch(/Try.*approve.*skip.*status/i);
|
|
137
|
+
});
|
|
138
|
+
});
|
|
139
|
+
import { createSlackInterface } from "../pm/slack-interface.js";
|
|
140
|
+
describe("slack-interface /release dispatcher", () => {
|
|
141
|
+
it("routes /release approve to releaseHandler", async () => {
|
|
142
|
+
const releaseHandler = vi.fn(async () => ({ text: "ok-handler", responseType: "in_channel" }));
|
|
143
|
+
// We bypass signature verification by stubbing it via a captured signing secret.
|
|
144
|
+
// In a real test we'd sign the request; here we assert call routing only by
|
|
145
|
+
// crafting a body and invoking the Hono router.
|
|
146
|
+
const { router } = createSlackInterface({
|
|
147
|
+
signingSecret: "test-secret-1234567890",
|
|
148
|
+
botToken: "xoxb-test",
|
|
149
|
+
channelId: "C123",
|
|
150
|
+
releaseHandler,
|
|
151
|
+
});
|
|
152
|
+
// Build a valid signature for the body so the request passes the check.
|
|
153
|
+
const body = "command=%2Frelease&text=approve&user_id=U123&response_url=";
|
|
154
|
+
const ts = Math.floor(Date.now() / 1000).toString();
|
|
155
|
+
const crypto = await import("crypto");
|
|
156
|
+
const sig = "v0=" +
|
|
157
|
+
crypto.createHmac("sha256", "test-secret-1234567890")
|
|
158
|
+
.update(`v0:${ts}:${body}`)
|
|
159
|
+
.digest("hex");
|
|
160
|
+
const res = await router.fetch(new Request("http://localhost/slack/commands", {
|
|
161
|
+
method: "POST",
|
|
162
|
+
headers: {
|
|
163
|
+
"X-Slack-Request-Timestamp": ts,
|
|
164
|
+
"X-Slack-Signature": sig,
|
|
165
|
+
"Content-Type": "application/x-www-form-urlencoded",
|
|
166
|
+
},
|
|
167
|
+
body,
|
|
168
|
+
}));
|
|
169
|
+
expect(res.status).toBe(200);
|
|
170
|
+
const json = await res.json();
|
|
171
|
+
expect(json.text).toBe("ok-handler");
|
|
172
|
+
expect(releaseHandler).toHaveBeenCalledWith({ text: "approve", userId: "U123" });
|
|
173
|
+
});
|
|
174
|
+
});
|
|
175
|
+
//# sourceMappingURL=release-manager-slack-handler.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"release-manager-slack-handler.test.js","sourceRoot":"","sources":["../../src/__tests__/release-manager-slack-handler.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AACzE,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAC1C,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,EAAE,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,aAAa,CAAC;AAC9C,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAC1C,OAAO,EAAE,gBAAgB,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AACrE,OAAO,EACL,sBAAsB,EACtB,uBAAuB,GACxB,MAAM,qCAAqC,CAAC;AAE7C,SAAS,SAAS;IAChB,MAAM,EAAE,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;IAC1C,OAAO,qBAAqB,EAAE,SAAS,CAAC;AAC1C,CAAC;AAED,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;IACtC,EAAE,CAAC,kBAAkB,EAAE,GAAG,EAAE;QAC1B,MAAM,CAAC,sBAAsB,CAAC,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;QACvE,MAAM,CAAC,sBAAsB,CAAC,YAAY,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,wBAAwB,EAAE,GAAG,EAAE;QAChC,MAAM,CAAC,sBAAsB,CAAC,2BAA2B,CAAC,CAAC,CAAC,OAAO,CAAC;YAClE,IAAI,EAAE,MAAM;YACZ,MAAM,EAAE,sBAAsB;SAC/B,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,CAAC,sBAAsB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,CAAC,CAAC;IACxF,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,iBAAiB,EAAE,GAAG,EAAE;QACzB,MAAM,CAAC,sBAAsB,CAAC,QAAQ,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAC;IACvE,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,MAAM,CAAC,sBAAsB,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,EAAE,EAAE,CAAC,CAAC;QAC9E,MAAM,CAAC,sBAAsB,CAAC,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,CAAC,CAAC;IAC9F,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACvC,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,IAAI,EAAO,CAAC;IACZ,MAAM,OAAO,GAAG,6BAA6B,CAAC;IAC9C,MAAM,MAAM,GAAG,MAAM,CAAC;IAEtB,UAAU,CAAC,KAAK,IAAI,EAAE;QACpB,MAAM,IAAI,GAAG,SAAS,EAAE,CAAC;QACzB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjB,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,gBAAgB,EAAE,IAAI,EAAE,CAAC,CAAC;QAC7E,EAAE,GAAG,OAAc,CAAC;IACtB,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;YACtB,IAAI,CAAC;gBAAC,UAAU,CAAC,CAAC,CAAC,CAAC;YAAC,CAAC;YAAC,MAAM,CAAC,CAAA,CAAC;YAC/B,IAAI,CAAC;gBAAC,UAAU,CAAC,CAAC,GAAG,MAAM,CAAC,CAAC;YAAC,CAAC;YAAC,MAAM,CAAC,CAAA,CAAC;YACxC,IAAI,CAAC;gBAAC,UAAU,CAAC,CAAC,GAAG,MAAM,CAAC,CAAC;YAAC,CAAC;YAAC,MAAM,CAAC,CAAA,CAAC;QAC1C,CAAC;QACD,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC;IACnB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;QAC/E,MAAM,CAAC,GAAG,MAAM,uBAAuB,CAAC;YACtC,GAAG,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE;YACxB,EAAE;YACF,OAAO;YACP,MAAM;YACN,WAAW,EAAE,MAAM;SACpB,CAAC,CAAC;QACH,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;QACpC,MAAM,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QAC1C,MAAM,IAAI,GAAG,MAAM,EAAE,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC,KAAK,CACzD,GAAG,CAAC,EAAE,CAAC,gBAAgB,CAAC,OAAO,EAAE,OAAO,CAAC,EAAE,MAAM,CAAC,gBAAgB,CAAC,UAAU,CAAC,CAAC,CAChF,CAAC;QACF,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8GAA8G,EAAE,KAAK,IAAI,EAAE;QAC5H,MAAM,uBAAuB,CAAC;YAC5B,GAAG,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,EAAE,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM;SACnE,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,MAAM,uBAAuB,CAAC;YACtC,GAAG,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,EAAE,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM;SACnE,CAAC,CAAC;QACH,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,4BAA4B,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iFAAiF,EAAE,KAAK,IAAI,EAAE;QAC/F,MAAM,CAAC,GAAG,MAAM,uBAAuB,CAAC;YACtC,GAAG,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,mBAAmB,EAAE;YAClD,EAAE;YACF,OAAO;YACP,MAAM;YACN,WAAW,EAAE,MAAM;SACpB,CAAC,CAAC;QACH,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;QACnC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC;QAC5C,MAAM,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QAC1C,MAAM,IAAI,GAAG,MAAM,EAAE,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,gBAAgB,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;QACnG,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACtC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,0BAA0B,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,uBAAuB;QACvB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAC3B,MAAM,EAAE,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,MAAM,CAAC;gBACvC,EAAE,EAAE,MAAM,CAAC,EAAE;gBACb,OAAO;gBACP,MAAM;gBACN,SAAS,EAAE,IAAI,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,GAAG,MAAM,CAAC;gBAClD,QAAQ,EAAE,MAAM;gBAChB,MAAM,EAAE,UAAU,CAAC,EAAE;gBACrB,gBAAgB,EAAE,IAAI;gBACtB,YAAY,EAAE,CAAC;aAChB,CAAC,CAAC;QACL,CAAC;QACD,MAAM,CAAC,GAAG,MAAM,uBAAuB,CAAC;YACtC,GAAG,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;YACvB,EAAE;YACF,OAAO;YACP,MAAM;YACN,WAAW,EAAE,MAAM;SACpB,CAAC,CAAC;QACH,MAAM,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACzC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;QAC3C,oEAAoE;QACpE,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;QACnC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;QAC9C,MAAM,CAAC,GAAG,MAAM,uBAAuB,CAAC;YACtC,GAAG,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,QAAQ,EAAE,YAAY,EAAE;YAChD,EAAE;YACF,OAAO;YACP,MAAM;YACN,WAAW,EAAE,MAAM;SACpB,CAAC,CAAC;QACH,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,6BAA6B,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,OAAO,EAAE,oBAAoB,EAAE,MAAM,0BAA0B,CAAC;AAEhE,QAAQ,CAAC,qCAAqC,EAAE,GAAG,EAAE;IACnD,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;QACzD,MAAM,cAAc,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE,YAAY,EAAE,YAAqB,EAAE,CAAC,CAAC,CAAC;QACxG,iFAAiF;QACjF,4EAA4E;QAC5E,gDAAgD;QAChD,MAAM,EAAE,MAAM,EAAE,GAAG,oBAAoB,CAAC;YACtC,aAAa,EAAE,wBAAwB;YACvC,QAAQ,EAAE,WAAW;YACrB,SAAS,EAAE,MAAM;YACjB,cAAc;SACf,CAAC,CAAC;QACH,wEAAwE;QACxE,MAAM,IAAI,GAAG,4DAA4D,CAAC;QAC1E,MAAM,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC,QAAQ,EAAE,CAAC;QACpD,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,QAAQ,CAAC,CAAC;QACtC,MAAM,GAAG,GACP,KAAK;YACL,MAAM,CAAC,UAAU,CAAC,QAAQ,EAAE,wBAAwB,CAAC;iBAClD,MAAM,CAAC,MAAM,EAAE,IAAI,IAAI,EAAE,CAAC;iBAC1B,MAAM,CAAC,KAAK,CAAC,CAAC;QACnB,MAAM,GAAG,GAAG,MAAM,MAAM,CAAC,KAAK,CAAC,IAAI,OAAO,CAAC,iCAAiC,EAAE;YAC5E,MAAM,EAAE,MAAM;YACd,OAAO,EAAE;gBACP,2BAA2B,EAAE,EAAE;gBAC/B,mBAAmB,EAAE,GAAG;gBACxB,cAAc,EAAE,mCAAmC;aACpD;YACD,IAAI;SACL,CAAC,CAAC,CAAC;QACJ,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC7B,MAAM,IAAI,GAAQ,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;QACnC,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QACrC,MAAM,CAAC,cAAc,CAAC,CAAC,oBAAoB,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC,CAAC;IACnF,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"release-manager-triggers.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/release-manager-triggers.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,67 @@
|
|
|
1
|
+
import { describe, it, expect } from "vitest";
|
|
2
|
+
import { evalMergedPRsSince, evalTimeSinceLastHours, evalCiGreenForMinutes, evalRequireSlackApproval, } from "../release-manager/triggers.js";
|
|
3
|
+
describe("evalMergedPRsSince", () => {
|
|
4
|
+
it("passes when count meets threshold", () => {
|
|
5
|
+
expect(evalMergedPRsSince(5, 5)).toEqual({ pass: true, reason: "mergedPRsSince=5 (have 5)" });
|
|
6
|
+
expect(evalMergedPRsSince(7, 5)).toEqual({ pass: true, reason: "mergedPRsSince=5 (have 7)" });
|
|
7
|
+
});
|
|
8
|
+
it("fails when count is below threshold", () => {
|
|
9
|
+
expect(evalMergedPRsSince(3, 5)).toEqual({ pass: false, reason: "mergedPRsSince not met (3/5)" });
|
|
10
|
+
expect(evalMergedPRsSince(0, 1)).toEqual({ pass: false, reason: "mergedPRsSince not met (0/1)" });
|
|
11
|
+
});
|
|
12
|
+
});
|
|
13
|
+
describe("evalTimeSinceLastHours", () => {
|
|
14
|
+
const now = new Date("2026-05-01T12:00:00Z");
|
|
15
|
+
it("passes when no last tag exists (initial release)", () => {
|
|
16
|
+
expect(evalTimeSinceLastHours(null, 24, now)).toEqual({ pass: true, reason: "no prior tag" });
|
|
17
|
+
});
|
|
18
|
+
it("passes when elapsed >= threshold", () => {
|
|
19
|
+
const lastTag = new Date(now.getTime() - 25 * 3600 * 1000);
|
|
20
|
+
const r = evalTimeSinceLastHours(lastTag, 24, now);
|
|
21
|
+
expect(r.pass).toBe(true);
|
|
22
|
+
});
|
|
23
|
+
it("fails when elapsed < threshold", () => {
|
|
24
|
+
const lastTag = new Date(now.getTime() - 2 * 3600 * 1000);
|
|
25
|
+
const r = evalTimeSinceLastHours(lastTag, 24, now);
|
|
26
|
+
expect(r.pass).toBe(false);
|
|
27
|
+
expect(r.reason).toMatch(/timeSinceLastHours not met/);
|
|
28
|
+
});
|
|
29
|
+
});
|
|
30
|
+
describe("evalCiGreenForMinutes", () => {
|
|
31
|
+
const now = new Date("2026-05-01T12:00:00Z");
|
|
32
|
+
it("fails when CI is not green", () => {
|
|
33
|
+
expect(evalCiGreenForMinutes("not-green", null, 30, now)).toEqual({
|
|
34
|
+
pass: false,
|
|
35
|
+
reason: "ci_not_green",
|
|
36
|
+
});
|
|
37
|
+
});
|
|
38
|
+
it("fails when CI status is unavailable", () => {
|
|
39
|
+
expect(evalCiGreenForMinutes("unavailable", null, 30, now)).toEqual({
|
|
40
|
+
pass: false,
|
|
41
|
+
reason: "ci_check_unavailable",
|
|
42
|
+
});
|
|
43
|
+
});
|
|
44
|
+
it("fails when green-since is too recent", () => {
|
|
45
|
+
const greenSince = new Date(now.getTime() - 10 * 60 * 1000); // 10 min ago
|
|
46
|
+
const r = evalCiGreenForMinutes("green", greenSince, 30, now);
|
|
47
|
+
expect(r.pass).toBe(false);
|
|
48
|
+
expect(r.reason).toMatch(/ciGreenForMinutes not met/);
|
|
49
|
+
});
|
|
50
|
+
it("passes when green long enough", () => {
|
|
51
|
+
const greenSince = new Date(now.getTime() - 45 * 60 * 1000);
|
|
52
|
+
const r = evalCiGreenForMinutes("green", greenSince, 30, now);
|
|
53
|
+
expect(r.pass).toBe(true);
|
|
54
|
+
});
|
|
55
|
+
});
|
|
56
|
+
describe("evalRequireSlackApproval", () => {
|
|
57
|
+
it("passes when require=false (no-op)", () => {
|
|
58
|
+
expect(evalRequireSlackApproval(false, false)).toEqual({ pass: true, reason: "approval not required" });
|
|
59
|
+
});
|
|
60
|
+
it("passes when require=true and approval is fresh", () => {
|
|
61
|
+
expect(evalRequireSlackApproval(true, true)).toEqual({ pass: true, reason: "approval is fresh" });
|
|
62
|
+
});
|
|
63
|
+
it("fails when require=true and no fresh approval", () => {
|
|
64
|
+
expect(evalRequireSlackApproval(true, false)).toEqual({ pass: false, reason: "no_fresh_approval" });
|
|
65
|
+
});
|
|
66
|
+
});
|
|
67
|
+
//# sourceMappingURL=release-manager-triggers.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"release-manager-triggers.test.js","sourceRoot":"","sources":["../../src/__tests__/release-manager-triggers.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EACL,kBAAkB,EAClB,sBAAsB,EACtB,qBAAqB,EACrB,wBAAwB,GACzB,MAAM,gCAAgC,CAAC;AAExC,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,CAAC,kBAAkB,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,EAAE,2BAA2B,EAAE,CAAC,CAAC;QAC9F,MAAM,CAAC,kBAAkB,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,EAAE,2BAA2B,EAAE,CAAC,CAAC;IAChG,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,MAAM,CAAC,kBAAkB,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,EAAE,8BAA8B,EAAE,CAAC,CAAC;QAClG,MAAM,CAAC,kBAAkB,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,EAAE,8BAA8B,EAAE,CAAC,CAAC;IACpG,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;IACtC,MAAM,GAAG,GAAG,IAAI,IAAI,CAAC,sBAAsB,CAAC,CAAC;IAE7C,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,CAAC,sBAAsB,CAAC,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,EAAE,cAAc,EAAE,CAAC,CAAC;IAChG,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC1C,MAAM,OAAO,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,EAAE,GAAG,IAAI,GAAG,IAAI,CAAC,CAAC;QAC3D,MAAM,CAAC,GAAG,sBAAsB,CAAC,OAAO,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC;QACnD,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;QACxC,MAAM,OAAO,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,CAAC,GAAG,IAAI,GAAG,IAAI,CAAC,CAAC;QAC1D,MAAM,CAAC,GAAG,sBAAsB,CAAC,OAAO,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC;QACnD,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC3B,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,4BAA4B,CAAC,CAAC;IACzD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,MAAM,GAAG,GAAG,IAAI,IAAI,CAAC,sBAAsB,CAAC,CAAC;IAE7C,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;QACpC,MAAM,CAAC,qBAAqB,CAAC,WAAW,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC;YAChE,IAAI,EAAE,KAAK;YACX,MAAM,EAAE,cAAc;SACvB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,MAAM,CAAC,qBAAqB,CAAC,aAAa,EAAE,IAAI,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC;YAClE,IAAI,EAAE,KAAK;YACX,MAAM,EAAE,sBAAsB;SAC/B,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,MAAM,UAAU,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC,CAAC,aAAa;QAC1E,MAAM,CAAC,GAAG,qBAAqB,CAAC,OAAO,EAAE,UAAU,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC;QAC9D,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC3B,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,2BAA2B,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,+BAA+B,EAAE,GAAG,EAAE;QACvC,MAAM,UAAU,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC;QAC5D,MAAM,CAAC,GAAG,qBAAqB,CAAC,OAAO,EAAE,UAAU,EAAE,EAAE,EAAE,GAAG,CAAC,CAAC;QAC9D,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,CAAC,wBAAwB,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,EAAE,uBAAuB,EAAE,CAAC,CAAC;IAC1G,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,CAAC,wBAAwB,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,EAAE,mBAAmB,EAAE,CAAC,CAAC;IACpG,CAAC,CAAC,CAAC;IACH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,CAAC,wBAAwB,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,EAAE,mBAAmB,EAAE,CAAC,CAAC;IACtG,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"release-manager-versioning.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/release-manager-versioning.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,51 @@
|
|
|
1
|
+
import { describe, it, expect } from "vitest";
|
|
2
|
+
import { bumpFromConfigAndCommits } from "../release-manager/versioning.js";
|
|
3
|
+
describe("bumpFromConfigAndCommits", () => {
|
|
4
|
+
describe("policy=patch", () => {
|
|
5
|
+
it("always bumps patch", () => {
|
|
6
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "feat: anything" }], "patch")).toBe("v1.2.4");
|
|
7
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "BREAKING CHANGE: removed X" }], "patch")).toBe("v1.2.4");
|
|
8
|
+
expect(bumpFromConfigAndCommits("0.1.16", [], "patch")).toBe("v0.1.17");
|
|
9
|
+
});
|
|
10
|
+
it("bumps from null/missing tag to v0.0.1", () => {
|
|
11
|
+
expect(bumpFromConfigAndCommits(null, [], "patch")).toBe("v0.0.1");
|
|
12
|
+
});
|
|
13
|
+
});
|
|
14
|
+
describe("policy=minor", () => {
|
|
15
|
+
it("always bumps minor and resets patch to 0", () => {
|
|
16
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "fix: x" }], "minor")).toBe("v1.3.0");
|
|
17
|
+
expect(bumpFromConfigAndCommits("v0.1.16", [{ message: "feat!: breaking" }], "minor")).toBe("v0.2.0");
|
|
18
|
+
});
|
|
19
|
+
it("bumps from null/missing tag to v0.1.0", () => {
|
|
20
|
+
expect(bumpFromConfigAndCommits(null, [], "minor")).toBe("v0.1.0");
|
|
21
|
+
});
|
|
22
|
+
});
|
|
23
|
+
describe("policy=conventional-commits", () => {
|
|
24
|
+
it("returns major when any commit has BREAKING CHANGE in body", () => {
|
|
25
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "feat: foo\n\nBREAKING CHANGE: removed flag" }], "conventional-commits")).toBe("v2.0.0");
|
|
26
|
+
});
|
|
27
|
+
it("returns major when any commit subject has '!:'", () => {
|
|
28
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "feat!: rewrite" }], "conventional-commits")).toBe("v2.0.0");
|
|
29
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "fix(api)!: change request shape" }], "conventional-commits")).toBe("v2.0.0");
|
|
30
|
+
});
|
|
31
|
+
it("returns minor when any commit is feat: but none are breaking", () => {
|
|
32
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "feat: add X" }, { message: "fix: bug" }], "conventional-commits")).toBe("v1.3.0");
|
|
33
|
+
});
|
|
34
|
+
it("returns minor for feat(scope):", () => {
|
|
35
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "feat(api): new endpoint" }], "conventional-commits")).toBe("v1.3.0");
|
|
36
|
+
});
|
|
37
|
+
it("returns patch for fix:/refactor:/perf: only", () => {
|
|
38
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "fix: a" }, { message: "perf: b" }, { message: "refactor: c" }], "conventional-commits")).toBe("v1.2.4");
|
|
39
|
+
});
|
|
40
|
+
it("returns patch for non-conforming commits (no error)", () => {
|
|
41
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [{ message: "wip" }, { message: "merge branch 'foo'" }], "conventional-commits")).toBe("v1.2.4");
|
|
42
|
+
});
|
|
43
|
+
});
|
|
44
|
+
describe("leading-v handling", () => {
|
|
45
|
+
it("strips leading v on input and always emits leading v", () => {
|
|
46
|
+
expect(bumpFromConfigAndCommits("1.2.3", [], "patch")).toBe("v1.2.4");
|
|
47
|
+
expect(bumpFromConfigAndCommits("v1.2.3", [], "patch")).toBe("v1.2.4");
|
|
48
|
+
});
|
|
49
|
+
});
|
|
50
|
+
});
|
|
51
|
+
//# sourceMappingURL=release-manager-versioning.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"release-manager-versioning.test.js","sourceRoot":"","sources":["../../src/__tests__/release-manager-versioning.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,wBAAwB,EAAE,MAAM,kCAAkC,CAAC;AAE5E,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,QAAQ,CAAC,cAAc,EAAE,GAAG,EAAE;QAC5B,EAAE,CAAC,oBAAoB,EAAE,GAAG,EAAE;YAC5B,MAAM,CAAC,wBAAwB,CAAC,QAAQ,EAAE,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACpG,MAAM,CAAC,wBAAwB,CAAC,QAAQ,EAAE,CAAC,EAAE,OAAO,EAAE,4BAA4B,EAAE,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YAChH,MAAM,CAAC,wBAAwB,CAAC,QAAQ,EAAE,EAAE,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QAC1E,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,CAAC,wBAAwB,CAAC,IAAI,EAAE,EAAE,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACrE,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,cAAc,EAAE,GAAG,EAAE;QAC5B,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,CAAC,wBAAwB,CAAC,QAAQ,EAAE,CAAC,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YAC5F,MAAM,CAAC,wBAAwB,CAAC,SAAS,EAAE,CAAC,EAAE,OAAO,EAAE,iBAAiB,EAAE,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACxG,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,CAAC,wBAAwB,CAAC,IAAI,EAAE,EAAE,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACrE,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,6BAA6B,EAAE,GAAG,EAAE;QAC3C,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;YACnE,MAAM,CACJ,wBAAwB,CACtB,QAAQ,EACR,CAAC,EAAE,OAAO,EAAE,4CAA4C,EAAE,CAAC,EAC3D,sBAAsB,CACvB,CACF,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACnB,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,CACJ,wBAAwB,CAAC,QAAQ,EAAE,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC,EAAE,sBAAsB,CAAC,CAC5F,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACjB,MAAM,CACJ,wBAAwB,CACtB,QAAQ,EACR,CAAC,EAAE,OAAO,EAAE,iCAAiC,EAAE,CAAC,EAChD,sBAAsB,CACvB,CACF,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACnB,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;YACtE,MAAM,CACJ,wBAAwB,CACtB,QAAQ,EACR,CAAC,EAAE,OAAO,EAAE,aAAa,EAAE,EAAE,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,EACrD,sBAAsB,CACvB,CACF,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACnB,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;YACxC,MAAM,CACJ,wBAAwB,CAAC,QAAQ,EAAE,CAAC,EAAE,OAAO,EAAE,yBAAyB,EAAE,CAAC,EAAE,sBAAsB,CAAC,CACrG,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACnB,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;YACrD,MAAM,CACJ,wBAAwB,CACtB,QAAQ,EACR,CAAC,EAAE,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,OAAO,EAAE,SAAS,EAAE,EAAE,EAAE,OAAO,EAAE,aAAa,EAAE,CAAC,EAC3E,sBAAsB,CACvB,CACF,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACnB,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;YAC7D,MAAM,CACJ,wBAAwB,CACtB,QAAQ,EACR,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,EAAE,EAAE,OAAO,EAAE,oBAAoB,EAAE,CAAC,EACvD,sBAAsB,CACvB,CACF,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACnB,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAClC,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;YAC9D,MAAM,CAAC,wBAAwB,CAAC,OAAO,EAAE,EAAE,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACtE,MAAM,CAAC,wBAAwB,CAAC,QAAQ,EAAE,EAAE,EAAE,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACzE,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-prompt.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/review-prompt.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,88 @@
|
|
|
1
|
+
import { describe, it, expect } from "vitest";
|
|
2
|
+
import { buildReviewPrompt, parseReviewFindings, estimateTokens, } from "../executor/review/review-prompt.js";
|
|
3
|
+
const handoff = {
|
|
4
|
+
runId: "r1",
|
|
5
|
+
issueId: "i1",
|
|
6
|
+
stage: "review",
|
|
7
|
+
timestamp: "2026-04-30T00:00:00Z",
|
|
8
|
+
summary: "",
|
|
9
|
+
filesChanged: ["src/foo.ts"],
|
|
10
|
+
approach: "",
|
|
11
|
+
context: {
|
|
12
|
+
issueIntent: "Fix bug X",
|
|
13
|
+
constraints: ["no new deps"],
|
|
14
|
+
assumptions: ["node 20"],
|
|
15
|
+
},
|
|
16
|
+
tokenBudget: { contextTokensUsed: 0, recommendedMaxTurns: 0 },
|
|
17
|
+
};
|
|
18
|
+
describe("buildReviewPrompt", () => {
|
|
19
|
+
it("includes intent, constraints, diff, and JSON-output instruction", () => {
|
|
20
|
+
const prompt = buildReviewPrompt({
|
|
21
|
+
handoff,
|
|
22
|
+
diff: "diff --git a/src/foo.ts b/src/foo.ts\n+++ b/src/foo.ts\n@@\n+x",
|
|
23
|
+
files: [{ path: "src/foo.ts", body: "export const x = 1;" }],
|
|
24
|
+
maxInputTokens: 100_000,
|
|
25
|
+
});
|
|
26
|
+
expect(prompt.messages).toHaveLength(2);
|
|
27
|
+
expect(prompt.messages[0].role).toBe("system");
|
|
28
|
+
expect(prompt.messages[1].role).toBe("user");
|
|
29
|
+
expect(prompt.messages[1].content).toContain("Fix bug X");
|
|
30
|
+
expect(prompt.messages[1].content).toContain("no new deps");
|
|
31
|
+
expect(prompt.messages[1].content).toContain("diff --git");
|
|
32
|
+
expect(prompt.messages[1].content).toContain("export const x = 1;");
|
|
33
|
+
expect(prompt.messages[0].content).toContain("findings"); // schema instruction
|
|
34
|
+
expect(prompt.truncatedFiles).toBe(0);
|
|
35
|
+
});
|
|
36
|
+
it("drops file bodies tail-first when over budget but keeps the diff", () => {
|
|
37
|
+
const big = "x".repeat(20_000);
|
|
38
|
+
const out = buildReviewPrompt({
|
|
39
|
+
handoff,
|
|
40
|
+
diff: "diff --git a/foo b/foo\n+y",
|
|
41
|
+
files: [
|
|
42
|
+
{ path: "a.ts", body: big },
|
|
43
|
+
{ path: "b.ts", body: big },
|
|
44
|
+
{ path: "c.ts", body: big },
|
|
45
|
+
],
|
|
46
|
+
maxInputTokens: 6_000, // forces truncation
|
|
47
|
+
});
|
|
48
|
+
expect(out.truncatedFiles).toBeGreaterThan(0);
|
|
49
|
+
expect(out.messages[1].content).toContain("diff --git");
|
|
50
|
+
});
|
|
51
|
+
});
|
|
52
|
+
describe("parseReviewFindings", () => {
|
|
53
|
+
it("extracts the first balanced JSON object and validates against schema", () => {
|
|
54
|
+
const raw = `Sure, here is the review:
|
|
55
|
+
\`\`\`json
|
|
56
|
+
{ "findings": [
|
|
57
|
+
{ "severity": "warning", "file": "a.ts", "line": 1, "category": "x", "description": "d", "fix": "f" }
|
|
58
|
+
] }
|
|
59
|
+
\`\`\`
|
|
60
|
+
End.`;
|
|
61
|
+
const findings = parseReviewFindings(raw);
|
|
62
|
+
expect(findings).toHaveLength(1);
|
|
63
|
+
expect(findings[0].severity).toBe("warning");
|
|
64
|
+
});
|
|
65
|
+
it("throws on malformed JSON", () => {
|
|
66
|
+
expect(() => parseReviewFindings("not json at all")).toThrow();
|
|
67
|
+
});
|
|
68
|
+
it("throws when schema validation fails", () => {
|
|
69
|
+
expect(() => parseReviewFindings('{"findings":[{"bad":true}]}')).toThrow();
|
|
70
|
+
});
|
|
71
|
+
it("ignores { inside string literals before the real envelope", () => {
|
|
72
|
+
const raw = `I noticed "a {problem}" here. {"findings":[]}`;
|
|
73
|
+
expect(parseReviewFindings(raw)).toEqual([]);
|
|
74
|
+
});
|
|
75
|
+
it("handles \\uXXXX escapes inside string literals", () => {
|
|
76
|
+
// " is a double-quote; the scanner must not terminate the string here.
|
|
77
|
+
const raw = `Model output: { "findings": [{ "severity": "warning", "file": "a\\u0022.ts", "line": 1, "category": "x", "description": "d", "fix": "f" }] }`;
|
|
78
|
+
const findings = parseReviewFindings(raw);
|
|
79
|
+
expect(findings).toHaveLength(1);
|
|
80
|
+
expect(findings[0].file).toContain('"');
|
|
81
|
+
});
|
|
82
|
+
});
|
|
83
|
+
describe("estimateTokens", () => {
|
|
84
|
+
it("returns a roughly char/4 estimate", () => {
|
|
85
|
+
expect(estimateTokens("a".repeat(40))).toBe(10);
|
|
86
|
+
});
|
|
87
|
+
});
|
|
88
|
+
//# sourceMappingURL=review-prompt.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-prompt.test.js","sourceRoot":"","sources":["../../src/__tests__/review-prompt.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EACL,iBAAiB,EACjB,mBAAmB,EACnB,cAAc,GACf,MAAM,qCAAqC,CAAC;AAG7C,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,YAAY,CAAC;IAC5B,QAAQ,EAAE,EAAE;IACZ,OAAO,EAAE;QACP,WAAW,EAAE,WAAW;QACxB,WAAW,EAAE,CAAC,aAAa,CAAC;QAC5B,WAAW,EAAE,CAAC,SAAS,CAAC;KACzB;IACD,WAAW,EAAE,EAAE,iBAAiB,EAAE,CAAC,EAAE,mBAAmB,EAAE,CAAC,EAAE;CAC9D,CAAC;AAEF,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QACzE,MAAM,MAAM,GAAG,iBAAiB,CAAC;YAC/B,OAAO;YACP,IAAI,EAAE,gEAAgE;YACtE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE,IAAI,EAAE,qBAAqB,EAAE,CAAC;YAC5D,cAAc,EAAE,OAAO;SACxB,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAC/C,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QAC7C,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;QAC1D,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;QAC5D,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QAC3D,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;QACpE,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC,CAAE,qBAAqB;QAChF,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;QAC1E,MAAM,GAAG,GAAG,GAAG,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC;QAC/B,MAAM,GAAG,GAAG,iBAAiB,CAAC;YAC5B,OAAO;YACP,IAAI,EAAE,4BAA4B;YAClC,KAAK,EAAE;gBACL,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,EAAE;gBAC3B,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,EAAE;gBAC3B,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,EAAE;aAC5B;YACD,cAAc,EAAE,KAAK,EAAE,oBAAoB;SAC5C,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;QAC9C,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,sEAAsE,EAAE,GAAG,EAAE;QAC9E,MAAM,GAAG,GAAG;;;;;;KAMX,CAAC;QACF,MAAM,QAAQ,GAAG,mBAAmB,CAAC,GAAG,CAAC,CAAC;QAC1C,MAAM,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACjC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0BAA0B,EAAE,GAAG,EAAE;QAClC,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,iBAAiB,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,6BAA6B,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;IAC7E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,MAAM,GAAG,GAAG,+CAA+C,CAAC;QAC5D,MAAM,CAAC,mBAAmB,CAAC,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,uEAAuE;QACvE,MAAM,GAAG,GAAG,8IAA8I,CAAC;QAC3J,MAAM,QAAQ,GAAG,mBAAmB,CAAC,GAAG,CAAC,CAAC;QAC1C,MAAM,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACjC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;IAC9B,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,CAAC,cAAc,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-provider-registry.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/review-provider-registry.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,61 @@
|
|
|
1
|
+
import { describe, it, expect } from "vitest";
|
|
2
|
+
import { getEnabledProviders, } from "../executor/review/review-provider.js";
|
|
3
|
+
describe("review-provider registry", () => {
|
|
4
|
+
it("exports the ReviewProvider interface and ReviewModelRun type", () => {
|
|
5
|
+
// Compile-time existence check via type assignment
|
|
6
|
+
const _checkRun = {
|
|
7
|
+
modelId: "x",
|
|
8
|
+
providerId: "agentic",
|
|
9
|
+
status: "completed",
|
|
10
|
+
findings: [],
|
|
11
|
+
inputTokens: 0,
|
|
12
|
+
outputTokens: 0,
|
|
13
|
+
durationMs: 0,
|
|
14
|
+
};
|
|
15
|
+
expect(_checkRun.modelId).toBe("x");
|
|
16
|
+
});
|
|
17
|
+
it("returns at least the agentic provider when env is empty", () => {
|
|
18
|
+
const providers = getEnabledProviders({});
|
|
19
|
+
expect(providers.length).toBeGreaterThanOrEqual(1);
|
|
20
|
+
expect(providers.some((p) => p.id === "agentic")).toBe(true);
|
|
21
|
+
});
|
|
22
|
+
it("ReviewProvider has runReview signature", () => {
|
|
23
|
+
const providers = getEnabledProviders({});
|
|
24
|
+
const p = providers[0];
|
|
25
|
+
expect(typeof p.runReview).toBe("function");
|
|
26
|
+
});
|
|
27
|
+
});
|
|
28
|
+
describe("review-provider registry — fanout selection", () => {
|
|
29
|
+
it("returns only agentic when REVIEW_MODELS unset", () => {
|
|
30
|
+
const ps = getEnabledProviders({});
|
|
31
|
+
expect(ps.map((p) => p.id)).toEqual(["agentic"]);
|
|
32
|
+
});
|
|
33
|
+
it("adds openrouter when both vars set", () => {
|
|
34
|
+
const ps = getEnabledProviders({
|
|
35
|
+
REVIEW_MODELS: "anthropic/claude-3.5-sonnet,openai/gpt-4o",
|
|
36
|
+
OPENROUTER_API_KEY: "sk-or-x",
|
|
37
|
+
});
|
|
38
|
+
expect(ps.map((p) => p.id).sort()).toEqual(["agentic", "openrouter"]);
|
|
39
|
+
});
|
|
40
|
+
it("throws when REVIEW_MODELS set but OPENROUTER_API_KEY missing", () => {
|
|
41
|
+
expect(() => getEnabledProviders({ REVIEW_MODELS: "x/y" })).toThrow(/OPENROUTER_API_KEY/);
|
|
42
|
+
});
|
|
43
|
+
it("throws when OPENROUTER_API_KEY set but REVIEW_MODELS missing", () => {
|
|
44
|
+
expect(() => getEnabledProviders({ OPENROUTER_API_KEY: "sk" })).toThrow(/REVIEW_MODELS/);
|
|
45
|
+
});
|
|
46
|
+
it("treats whitespace-only REVIEW_MODELS as unset (and throws since OPENROUTER_API_KEY is set without effective models)", () => {
|
|
47
|
+
expect(() => getEnabledProviders({ REVIEW_MODELS: " , , ", OPENROUTER_API_KEY: "sk" })).toThrow(/REVIEW_MODELS/);
|
|
48
|
+
});
|
|
49
|
+
it("trims whitespace and drops empty entries from REVIEW_MODELS", () => {
|
|
50
|
+
const ps = getEnabledProviders({
|
|
51
|
+
REVIEW_MODELS: " m1 , , m2 ,",
|
|
52
|
+
OPENROUTER_API_KEY: "sk",
|
|
53
|
+
});
|
|
54
|
+
const fanout = ps.find((p) => p.id === "openrouter");
|
|
55
|
+
expect(fanout).toBeDefined();
|
|
56
|
+
// White-box: read the configured models off the provider.
|
|
57
|
+
const models = fanout.cfg.models;
|
|
58
|
+
expect(models).toEqual(["m1", "m2"]);
|
|
59
|
+
});
|
|
60
|
+
});
|
|
61
|
+
//# sourceMappingURL=review-provider-registry.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-provider-registry.test.js","sourceRoot":"","sources":["../../src/__tests__/review-provider-registry.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EACL,mBAAmB,GAIpB,MAAM,uCAAuC,CAAC;AAE/C,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,mDAAmD;QACnD,MAAM,SAAS,GAAmB;YAChC,OAAO,EAAE,GAAG;YACZ,UAAU,EAAE,SAAS;YACrB,MAAM,EAAE,WAAW;YACnB,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,CAAC;YACd,YAAY,EAAE,CAAC;YACf,UAAU,EAAE,CAAC;SACd,CAAC;QACF,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;QACjE,MAAM,SAAS,GAAG,mBAAmB,CAAC,EAAE,CAAC,CAAC;QAC1C,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAC;QACnD,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC/D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,SAAS,GAAG,mBAAmB,CAAC,EAAE,CAAC,CAAC;QAC1C,MAAM,CAAC,GAAmB,SAAS,CAAC,CAAC,CAAC,CAAC;QACvC,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,6CAA6C,EAAE,GAAG,EAAE;IAC3D,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,EAAE,GAAG,mBAAmB,CAAC,EAAE,CAAC,CAAC;QACnC,MAAM,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,EAAE,GAAG,mBAAmB,CAAC;YAC7B,aAAa,EAAE,2CAA2C;YAC1D,kBAAkB,EAAE,SAAS;SAC9B,CAAC,CAAC;QACH,MAAM,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,EAAE,YAAY,CAAC,CAAC,CAAC;IACxE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,EAAE,aAAa,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,OAAO,CACjE,oBAAoB,CACrB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,EAAE,kBAAkB,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,OAAO,CACrE,eAAe,CAChB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qHAAqH,EAAE,GAAG,EAAE;QAC7H,MAAM,CAAC,GAAG,EAAE,CACV,mBAAmB,CAAC,EAAE,aAAa,EAAE,OAAO,EAAE,kBAAkB,EAAE,IAAI,EAAE,CAAC,CAC1E,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,MAAM,EAAE,GAAG,mBAAmB,CAAC;YAC7B,aAAa,EAAE,cAAc;YAC7B,kBAAkB,EAAE,IAAI;SACzB,CAAC,CAAC;QACH,MAAM,MAAM,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,YAAY,CAAC,CAAC;QACrD,MAAM,CAAC,MAAM,CAAC,CAAC,WAAW,EAAE,CAAC;QAC7B,0DAA0D;QAC1D,MAAM,MAAM,GAAI,MAAmD,CAAC,GAAG,CAAC,MAAM,CAAC;QAC/E,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"runner-fanout-integration.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/runner-fanout-integration.test.ts"],"names":[],"mappings":""}
|