@zephyrdeng/pi-review 0.7.0 → 0.8.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +51 -7
- package/README.zh-CN.md +45 -5
- package/dist/args.d.ts +4 -0
- package/dist/args.d.ts.map +1 -1
- package/dist/args.js +82 -5
- package/dist/args.js.map +1 -1
- package/dist/args.test.js +72 -1
- package/dist/args.test.js.map +1 -1
- package/dist/child-process.d.ts +6 -0
- package/dist/child-process.d.ts.map +1 -1
- package/dist/child-process.js +48 -0
- package/dist/child-process.js.map +1 -1
- package/dist/child-process.test.js +55 -1
- package/dist/child-process.test.js.map +1 -1
- package/dist/cli-exit.test.js +14 -5
- package/dist/cli-exit.test.js.map +1 -1
- package/dist/cli.js +8 -3
- package/dist/cli.js.map +1 -1
- package/dist/index.d.ts +13 -6
- package/dist/index.d.ts.map +1 -1
- package/dist/index.js +9 -5
- package/dist/index.js.map +1 -1
- package/dist/install-args.test.js +1 -1
- package/dist/install-args.test.js.map +1 -1
- package/dist/install.d.ts.map +1 -1
- package/dist/install.js +2 -3
- package/dist/install.js.map +1 -1
- package/dist/json-events.d.ts +34 -2
- package/dist/json-events.d.ts.map +1 -1
- package/dist/json-events.js +156 -49
- package/dist/json-events.js.map +1 -1
- package/dist/json-events.test.js +176 -4
- package/dist/json-events.test.js.map +1 -1
- package/dist/loop.d.ts +28 -2
- package/dist/loop.d.ts.map +1 -1
- package/dist/loop.js +45 -3
- package/dist/loop.js.map +1 -1
- package/dist/loop.test.js +24 -1
- package/dist/loop.test.js.map +1 -1
- package/dist/matcher.test.js +1 -1
- package/dist/matcher.test.js.map +1 -1
- package/dist/meta-footer.d.ts +4 -8
- package/dist/meta-footer.d.ts.map +1 -1
- package/dist/meta-footer.js +19 -5
- package/dist/meta-footer.js.map +1 -1
- package/dist/meta-footer.test.js +11 -4
- package/dist/meta-footer.test.js.map +1 -1
- package/dist/panel-aggregate.test.js +1 -1
- package/dist/panel-aggregate.test.js.map +1 -1
- package/dist/panel-cancellation.test.d.ts +2 -0
- package/dist/panel-cancellation.test.d.ts.map +1 -0
- package/dist/panel-cancellation.test.js +55 -0
- package/dist/panel-cancellation.test.js.map +1 -0
- package/dist/panel-cli-ui.test.d.ts +2 -0
- package/dist/panel-cli-ui.test.d.ts.map +1 -0
- package/dist/panel-cli-ui.test.js +91 -0
- package/dist/panel-cli-ui.test.js.map +1 -0
- package/dist/panel-cli.test.js +76 -27
- package/dist/panel-cli.test.js.map +1 -1
- package/dist/panel-config.d.ts +38 -0
- package/dist/panel-config.d.ts.map +1 -1
- package/dist/panel-config.js +137 -4
- package/dist/panel-config.js.map +1 -1
- package/dist/panel-config.test.js +84 -4
- package/dist/panel-config.test.js.map +1 -1
- package/dist/panel-loop.test.js +1 -1
- package/dist/panel-loop.test.js.map +1 -1
- package/dist/panel-tools.test.d.ts +2 -0
- package/dist/panel-tools.test.d.ts.map +1 -0
- package/dist/panel-tools.test.js +12 -0
- package/dist/panel-tools.test.js.map +1 -0
- package/dist/panel-ui.d.ts +20 -0
- package/dist/panel-ui.d.ts.map +1 -0
- package/dist/panel-ui.js +84 -0
- package/dist/panel-ui.js.map +1 -0
- package/dist/panel-ui.test.d.ts +2 -0
- package/dist/panel-ui.test.d.ts.map +1 -0
- package/dist/panel-ui.test.js +122 -0
- package/dist/panel-ui.test.js.map +1 -0
- package/dist/panel-usage.d.ts +6 -0
- package/dist/panel-usage.d.ts.map +1 -0
- package/dist/panel-usage.js +21 -0
- package/dist/panel-usage.js.map +1 -0
- package/dist/panel-view.d.ts +42 -0
- package/dist/panel-view.d.ts.map +1 -0
- package/dist/panel-view.js +124 -0
- package/dist/panel-view.js.map +1 -0
- package/dist/panel-view.test.d.ts +2 -0
- package/dist/panel-view.test.d.ts.map +1 -0
- package/dist/panel-view.test.js +112 -0
- package/dist/panel-view.test.js.map +1 -0
- package/dist/panel.d.ts +22 -2
- package/dist/panel.d.ts.map +1 -1
- package/dist/panel.js +184 -60
- package/dist/panel.js.map +1 -1
- package/dist/prompt.d.ts +2 -0
- package/dist/prompt.d.ts.map +1 -1
- package/dist/prompt.js +34 -3
- package/dist/prompt.js.map +1 -1
- package/dist/prompt.test.js +21 -2
- package/dist/prompt.test.js.map +1 -1
- package/dist/review-events.d.ts +70 -0
- package/dist/review-events.d.ts.map +1 -0
- package/dist/review-events.js +52 -0
- package/dist/review-events.js.map +1 -0
- package/dist/review-events.test.d.ts +2 -0
- package/dist/review-events.test.d.ts.map +1 -0
- package/dist/review-events.test.js +47 -0
- package/dist/review-events.test.js.map +1 -0
- package/dist/review-result.test.js +1 -1
- package/dist/review-result.test.js.map +1 -1
- package/dist/review.js +3 -3
- package/dist/review.js.map +1 -1
- package/dist/review.meta.test.js +1 -1
- package/dist/review.meta.test.js.map +1 -1
- package/dist/skill-protocol.test.js +1 -1
- package/dist/skill-protocol.test.js.map +1 -1
- package/dist/skill-update.test.d.ts +2 -0
- package/dist/skill-update.test.d.ts.map +1 -0
- package/dist/skill-update.test.js +23 -0
- package/dist/skill-update.test.js.map +1 -0
- package/dist/skill.d.ts +22 -0
- package/dist/skill.d.ts.map +1 -1
- package/dist/skill.js +109 -22
- package/dist/skill.js.map +1 -1
- package/dist/types.d.ts +28 -2
- package/dist/types.d.ts.map +1 -1
- package/dist/types.js +2 -0
- package/dist/types.js.map +1 -1
- package/dist/ui-client.d.ts +13 -0
- package/dist/ui-client.d.ts.map +1 -0
- package/dist/ui-client.js +161 -0
- package/dist/ui-client.js.map +1 -0
- package/dist/ui-client.test.d.ts +2 -0
- package/dist/ui-client.test.d.ts.map +1 -0
- package/dist/ui-client.test.js +48 -0
- package/dist/ui-client.test.js.map +1 -0
- package/dist/ui-html.d.ts +7 -0
- package/dist/ui-html.d.ts.map +1 -0
- package/dist/ui-html.js +52 -0
- package/dist/ui-html.js.map +1 -0
- package/dist/ui-html.test.d.ts +2 -0
- package/dist/ui-html.test.d.ts.map +1 -0
- package/dist/ui-html.test.js +26 -0
- package/dist/ui-html.test.js.map +1 -0
- package/dist/ui-server-main.d.ts +3 -0
- package/dist/ui-server-main.d.ts.map +1 -0
- package/dist/ui-server-main.js +128 -0
- package/dist/ui-server-main.js.map +1 -0
- package/dist/ui-server-main.test.d.ts +2 -0
- package/dist/ui-server-main.test.d.ts.map +1 -0
- package/dist/ui-server-main.test.js +123 -0
- package/dist/ui-server-main.test.js.map +1 -0
- package/dist/ui-server.d.ts +42 -0
- package/dist/ui-server.d.ts.map +1 -0
- package/dist/ui-server.js +233 -0
- package/dist/ui-server.js.map +1 -0
- package/dist/ui-server.test.d.ts +2 -0
- package/dist/ui-server.test.d.ts.map +1 -0
- package/dist/ui-server.test.js +236 -0
- package/dist/ui-server.test.js.map +1 -0
- package/dist/update.d.ts.map +1 -1
- package/dist/update.js +48 -19
- package/dist/update.js.map +1 -1
- package/extensions/panel-tool.test.ts +388 -0
- package/extensions/panel-tool.ts +390 -0
- package/extensions/review.test.ts +79 -0
- package/extensions/review.ts +191 -45
- package/extensions/rv-completions.test.ts +177 -18
- package/extensions/rv-completions.ts +490 -60
- package/extensions/rv-interactive.test.ts +562 -0
- package/extensions/rv-interactive.ts +616 -0
- package/extensions/rv-locale.test.ts +12 -4
- package/extensions/rv-locale.ts +44 -11
- package/extensions/rv-model-picker.test.ts +172 -0
- package/extensions/rv-model-picker.ts +313 -0
- package/extensions/rv-model-priorities.test.ts +29 -9
- package/extensions/rv-model-priorities.ts +23 -8
- package/extensions/rv-prompts.test.ts +281 -2
- package/extensions/rv-prompts.ts +480 -32
- package/extensions/rv-resolve.test.ts +189 -0
- package/extensions/rv-resolve.ts +313 -0
- package/extensions/rv-semantic.test.ts +1 -1
- package/extensions/rv-semantic.ts +38 -4
- package/package.json +18 -6
- package/resources/review-presets.json +2 -2
- package/resources/rv-model-priorities.json +18 -9
- package/skills/pi-review/SKILL.md +84 -60
- package/skills/pi-review/references/codex-tools.md +18 -11
- package/skills/pi-review/references/model-selection.md +49 -28
- package/dist/extensions/review.d.ts +0 -2
- package/dist/extensions/review.js +0 -22
- package/dist/extensions/rv-prompts.d.ts +0 -19
- package/dist/extensions/rv-prompts.js +0 -132
|
@@ -0,0 +1,236 @@
|
|
|
1
|
+
import assert from "node:assert/strict";
|
|
2
|
+
import fs from "node:fs";
|
|
3
|
+
import http from "node:http";
|
|
4
|
+
import os from "node:os";
|
|
5
|
+
import path from "node:path";
|
|
6
|
+
import { test, afterEach } from "vitest";
|
|
7
|
+
import { createDashboardRequestListener, createRunDirectory, formatSseEvent, generateCapabilityToken, isAllowedHost, isAllowedOrigin, isValidToken, securityHeaders, splitCompleteLines, writeUrlFileAtomic, } from "./ui-server.js";
|
|
8
|
+
// ---------------------------------------------------------------------------
|
|
9
|
+
// Pure helpers
|
|
10
|
+
// ---------------------------------------------------------------------------
|
|
11
|
+
test("generateCapabilityToken returns high-entropy, unique, URL-safe tokens", () => {
|
|
12
|
+
const a = generateCapabilityToken();
|
|
13
|
+
const b = generateCapabilityToken();
|
|
14
|
+
assert.notEqual(a, b);
|
|
15
|
+
assert.ok(a.length >= 40);
|
|
16
|
+
assert.match(a, /^[A-Za-z0-9_-]+$/);
|
|
17
|
+
});
|
|
18
|
+
test("isValidToken accepts only an exact match and rejects undefined/short candidates", () => {
|
|
19
|
+
const token = generateCapabilityToken();
|
|
20
|
+
assert.equal(isValidToken(token, token), true);
|
|
21
|
+
assert.equal(isValidToken(`${token}x`, token), false);
|
|
22
|
+
assert.equal(isValidToken(token.slice(0, -1), token), false);
|
|
23
|
+
assert.equal(isValidToken(undefined, token), false);
|
|
24
|
+
assert.equal(isValidToken("", token), false);
|
|
25
|
+
});
|
|
26
|
+
test("isAllowedHost accepts only loopback hostnames bound to the server's own port", () => {
|
|
27
|
+
assert.equal(isAllowedHost("127.0.0.1:4100", 4100), true);
|
|
28
|
+
assert.equal(isAllowedHost("localhost:4100", 4100), true);
|
|
29
|
+
assert.equal(isAllowedHost("LOCALHOST:4100", 4100), true);
|
|
30
|
+
assert.equal(isAllowedHost("[::1]:4100", 4100), true);
|
|
31
|
+
assert.equal(isAllowedHost("127.0.0.1:9999", 4100), false);
|
|
32
|
+
assert.equal(isAllowedHost("evil.example:4100", 4100), false);
|
|
33
|
+
assert.equal(isAllowedHost("127.0.0.1", 4100), false);
|
|
34
|
+
assert.equal(isAllowedHost(undefined, 4100), false);
|
|
35
|
+
assert.equal(isAllowedHost("[::1", 4100), false);
|
|
36
|
+
});
|
|
37
|
+
test("isAllowedOrigin allows a missing Origin and exact loopback matches only", () => {
|
|
38
|
+
assert.equal(isAllowedOrigin(undefined, 4100), true);
|
|
39
|
+
assert.equal(isAllowedOrigin("http://127.0.0.1:4100", 4100), true);
|
|
40
|
+
assert.equal(isAllowedOrigin("http://localhost:4100", 4100), true);
|
|
41
|
+
assert.equal(isAllowedOrigin("http://[::1]:4100", 4100), true);
|
|
42
|
+
assert.equal(isAllowedOrigin("http://127.0.0.1:9999", 4100), false);
|
|
43
|
+
assert.equal(isAllowedOrigin("https://evil.example", 4100), false);
|
|
44
|
+
assert.equal(isAllowedOrigin("null", 4100), false);
|
|
45
|
+
});
|
|
46
|
+
test("securityHeaders disables framing, remote assets, and caching", () => {
|
|
47
|
+
const headers = securityHeaders();
|
|
48
|
+
assert.match(headers["Content-Security-Policy"], /default-src 'none'/);
|
|
49
|
+
assert.match(headers["Content-Security-Policy"], /frame-ancestors 'none'/);
|
|
50
|
+
assert.equal(headers["X-Frame-Options"], "DENY");
|
|
51
|
+
assert.equal(headers["Cache-Control"], "no-store");
|
|
52
|
+
});
|
|
53
|
+
test("createRunDirectory creates an owner-only directory", () => {
|
|
54
|
+
const dir = createRunDirectory(`test-${generateCapabilityToken()}`, os.tmpdir());
|
|
55
|
+
try {
|
|
56
|
+
const stat = fs.statSync(dir);
|
|
57
|
+
assert.ok(stat.isDirectory());
|
|
58
|
+
if (process.platform !== "win32") {
|
|
59
|
+
assert.equal(stat.mode & 0o777, 0o700);
|
|
60
|
+
}
|
|
61
|
+
}
|
|
62
|
+
finally {
|
|
63
|
+
fs.rmSync(dir, { recursive: true, force: true });
|
|
64
|
+
}
|
|
65
|
+
});
|
|
66
|
+
test("writeUrlFileAtomic leaves only the final file with no temp remnants", () => {
|
|
67
|
+
const dir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-review-ui-test-"));
|
|
68
|
+
try {
|
|
69
|
+
const target = path.join(dir, "url.txt");
|
|
70
|
+
writeUrlFileAtomic(target, "http://127.0.0.1:4100/run/abc");
|
|
71
|
+
assert.equal(fs.readFileSync(target, "utf8"), "http://127.0.0.1:4100/run/abc\n");
|
|
72
|
+
const remaining = fs.readdirSync(dir);
|
|
73
|
+
assert.deepEqual(remaining, ["url.txt"]);
|
|
74
|
+
}
|
|
75
|
+
finally {
|
|
76
|
+
fs.rmSync(dir, { recursive: true, force: true });
|
|
77
|
+
}
|
|
78
|
+
});
|
|
79
|
+
test("formatSseEvent emits an id: line for Last-Event-ID replay plus a data: line", () => {
|
|
80
|
+
const frame = formatSseEvent(3, { type: "reviewer.started" });
|
|
81
|
+
assert.equal(frame, 'id: 3\ndata: {"type":"reviewer.started"}\n\n');
|
|
82
|
+
});
|
|
83
|
+
test("splitCompleteLines buffers a trailing partial line across chunks", () => {
|
|
84
|
+
const first = splitCompleteLines("", 'line-a\nline-b\npart');
|
|
85
|
+
assert.deepEqual(first.lines, ["line-a", "line-b"]);
|
|
86
|
+
assert.equal(first.leftover, "part");
|
|
87
|
+
const second = splitCompleteLines(first.leftover, "ial\nline-c\n");
|
|
88
|
+
assert.deepEqual(second.lines, ["partial", "line-c"]);
|
|
89
|
+
assert.equal(second.leftover, "");
|
|
90
|
+
});
|
|
91
|
+
function httpGet(port, requestPath, headers = {}) {
|
|
92
|
+
return new Promise((resolve, reject) => {
|
|
93
|
+
const req = http.request({ host: "127.0.0.1", port, path: requestPath, headers }, (res) => {
|
|
94
|
+
let body = "";
|
|
95
|
+
res.on("data", (c) => (body += c));
|
|
96
|
+
res.on("end", () => resolve({ status: res.statusCode ?? 0, headers: res.headers, body }));
|
|
97
|
+
});
|
|
98
|
+
req.on("error", reject);
|
|
99
|
+
req.end();
|
|
100
|
+
});
|
|
101
|
+
}
|
|
102
|
+
/** Collect SSE frames until `minFrames` arrive or the timeout fires, then close the connection. */
|
|
103
|
+
function collectSse(port, requestPath, headers, minFrames) {
|
|
104
|
+
return new Promise((resolve, reject) => {
|
|
105
|
+
const req = http.request({ host: "127.0.0.1", port, path: requestPath, headers }, (res) => {
|
|
106
|
+
let body = "";
|
|
107
|
+
const timer = setTimeout(() => {
|
|
108
|
+
req.destroy();
|
|
109
|
+
resolve(body);
|
|
110
|
+
}, 2000);
|
|
111
|
+
res.on("data", (c) => {
|
|
112
|
+
body += c;
|
|
113
|
+
if ((body.match(/^id: /gm) ?? []).length >= minFrames) {
|
|
114
|
+
clearTimeout(timer);
|
|
115
|
+
req.destroy();
|
|
116
|
+
resolve(body);
|
|
117
|
+
}
|
|
118
|
+
});
|
|
119
|
+
res.on("error", () => resolve(body));
|
|
120
|
+
});
|
|
121
|
+
req.on("error", reject);
|
|
122
|
+
req.end();
|
|
123
|
+
});
|
|
124
|
+
}
|
|
125
|
+
let activeServer;
|
|
126
|
+
let activeDir;
|
|
127
|
+
afterEach(() => {
|
|
128
|
+
if (activeServer) {
|
|
129
|
+
activeServer.closeAllConnections();
|
|
130
|
+
activeServer.close();
|
|
131
|
+
activeServer = undefined;
|
|
132
|
+
}
|
|
133
|
+
if (activeDir) {
|
|
134
|
+
fs.rmSync(activeDir, { recursive: true, force: true });
|
|
135
|
+
activeDir = undefined;
|
|
136
|
+
}
|
|
137
|
+
});
|
|
138
|
+
async function startTestServer(overrides = {}) {
|
|
139
|
+
const dir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-review-ui-test-"));
|
|
140
|
+
activeDir = dir;
|
|
141
|
+
const eventsPath = path.join(dir, "events.jsonl");
|
|
142
|
+
fs.writeFileSync(eventsPath, "");
|
|
143
|
+
const token = generateCapabilityToken();
|
|
144
|
+
const server = http.createServer();
|
|
145
|
+
await new Promise((resolve) => server.listen(0, "127.0.0.1", resolve));
|
|
146
|
+
const port = server.address().port;
|
|
147
|
+
const ctx = {
|
|
148
|
+
token,
|
|
149
|
+
port,
|
|
150
|
+
eventsPath,
|
|
151
|
+
html: "<!doctype html><title>pi-review dashboard</title>",
|
|
152
|
+
staticAssets: { "panel-view.js": { contentType: "text/javascript; charset=utf-8", body: "export const x = 1;" } },
|
|
153
|
+
pollIntervalMs: 20,
|
|
154
|
+
...overrides,
|
|
155
|
+
};
|
|
156
|
+
server.on("request", createDashboardRequestListener(ctx));
|
|
157
|
+
activeServer = server;
|
|
158
|
+
return { port, token, eventsPath };
|
|
159
|
+
}
|
|
160
|
+
test("dashboard serves the HTML page for a valid capability token", async () => {
|
|
161
|
+
const { port, token } = await startTestServer();
|
|
162
|
+
const res = await httpGet(port, `/run/${token}`, { Host: `127.0.0.1:${port}` });
|
|
163
|
+
assert.equal(res.status, 200);
|
|
164
|
+
assert.match(res.headers["content-type"] ?? "", /text\/html/);
|
|
165
|
+
assert.match(res.body, /pi-review dashboard/);
|
|
166
|
+
assert.equal(res.headers["x-frame-options"], "DENY");
|
|
167
|
+
assert.match(String(res.headers["content-security-policy"] ?? ""), /frame-ancestors 'none'/);
|
|
168
|
+
});
|
|
169
|
+
test("dashboard rejects an invalid or wrong-length capability token with 404", async () => {
|
|
170
|
+
const { port, token } = await startTestServer();
|
|
171
|
+
const wrong = await httpGet(port, `/run/${token}x`, { Host: `127.0.0.1:${port}` });
|
|
172
|
+
assert.equal(wrong.status, 404);
|
|
173
|
+
const garbage = await httpGet(port, "/run/not-a-token", { Host: `127.0.0.1:${port}` });
|
|
174
|
+
assert.equal(garbage.status, 404);
|
|
175
|
+
});
|
|
176
|
+
test("dashboard rejects a non-loopback Host header", async () => {
|
|
177
|
+
const { port, token } = await startTestServer();
|
|
178
|
+
const res = await httpGet(port, `/run/${token}`, { Host: "evil.example" });
|
|
179
|
+
assert.equal(res.status, 400);
|
|
180
|
+
});
|
|
181
|
+
test("dashboard rejects a cross-origin request but allows a missing Origin", async () => {
|
|
182
|
+
const { port, token } = await startTestServer();
|
|
183
|
+
const bad = await httpGet(port, `/run/${token}`, { Host: `127.0.0.1:${port}`, Origin: "https://evil.example" });
|
|
184
|
+
assert.equal(bad.status, 403);
|
|
185
|
+
const ok = await httpGet(port, `/run/${token}`, { Host: `127.0.0.1:${port}`, Origin: `http://127.0.0.1:${port}` });
|
|
186
|
+
assert.equal(ok.status, 200);
|
|
187
|
+
const missing = await httpGet(port, `/run/${token}`, { Host: `127.0.0.1:${port}` });
|
|
188
|
+
assert.equal(missing.status, 200);
|
|
189
|
+
});
|
|
190
|
+
test("dashboard serves a static asset under the token boundary and 404s unknown assets", async () => {
|
|
191
|
+
const { port, token } = await startTestServer();
|
|
192
|
+
const res = await httpGet(port, `/run/${token}/static/panel-view.js`, { Host: `127.0.0.1:${port}` });
|
|
193
|
+
assert.equal(res.status, 200);
|
|
194
|
+
assert.match(res.headers["content-type"] ?? "", /javascript/);
|
|
195
|
+
assert.match(res.body, /export const x/);
|
|
196
|
+
const missing = await httpGet(port, `/run/${token}/static/does-not-exist.js`, { Host: `127.0.0.1:${port}` });
|
|
197
|
+
assert.equal(missing.status, 404);
|
|
198
|
+
});
|
|
199
|
+
test("dashboard SSE stream replays existing events then tails new appends in seq order", async () => {
|
|
200
|
+
const { port, token, eventsPath } = await startTestServer();
|
|
201
|
+
fs.appendFileSync(eventsPath, `${JSON.stringify({ v: 1, runId: "r1", seq: 1, at: 1, type: "panel.started" })}\n`);
|
|
202
|
+
fs.appendFileSync(eventsPath, `${JSON.stringify({ v: 1, runId: "r1", seq: 2, at: 2, type: "reviewer.queued" })}\n`);
|
|
203
|
+
const collecting = collectSse(port, `/run/${token}/events`, { Host: `127.0.0.1:${port}` }, 3);
|
|
204
|
+
await new Promise((resolve) => setTimeout(resolve, 60));
|
|
205
|
+
fs.appendFileSync(eventsPath, `${JSON.stringify({ v: 1, runId: "r1", seq: 3, at: 3, type: "reviewer.started" })}\n`);
|
|
206
|
+
const body = await collecting;
|
|
207
|
+
assert.match(body, /id: 1\n/);
|
|
208
|
+
assert.match(body, /id: 2\n/);
|
|
209
|
+
assert.match(body, /id: 3\n/);
|
|
210
|
+
assert.match(body, /"type":"panel.started"/);
|
|
211
|
+
assert.match(body, /"type":"reviewer.started"/);
|
|
212
|
+
});
|
|
213
|
+
test("dashboard SSE stream honors Last-Event-ID and only replays newer events", async () => {
|
|
214
|
+
const { port, token, eventsPath } = await startTestServer();
|
|
215
|
+
fs.appendFileSync(eventsPath, `${JSON.stringify({ v: 1, runId: "r1", seq: 1, at: 1, type: "panel.started" })}\n`);
|
|
216
|
+
fs.appendFileSync(eventsPath, `${JSON.stringify({ v: 1, runId: "r1", seq: 2, at: 2, type: "reviewer.queued" })}\n`);
|
|
217
|
+
fs.appendFileSync(eventsPath, `${JSON.stringify({ v: 1, runId: "r1", seq: 3, at: 3, type: "reviewer.started" })}\n`);
|
|
218
|
+
const body = await collectSse(port, `/run/${token}/events`, { Host: `127.0.0.1:${port}`, "Last-Event-ID": "2" }, 1);
|
|
219
|
+
assert.doesNotMatch(body, /id: 1\n/);
|
|
220
|
+
assert.doesNotMatch(body, /id: 2\n/);
|
|
221
|
+
assert.match(body, /id: 3\n/);
|
|
222
|
+
});
|
|
223
|
+
test("dashboard rejects non-GET/HEAD methods", async () => {
|
|
224
|
+
const { port, token } = await startTestServer();
|
|
225
|
+
const res = await new Promise((resolve, reject) => {
|
|
226
|
+
const req = http.request({ host: "127.0.0.1", port, path: `/run/${token}`, method: "POST", headers: { Host: `127.0.0.1:${port}` } }, (r) => {
|
|
227
|
+
let body = "";
|
|
228
|
+
r.on("data", (c) => (body += c));
|
|
229
|
+
r.on("end", () => resolve({ status: r.statusCode ?? 0, headers: r.headers, body }));
|
|
230
|
+
});
|
|
231
|
+
req.on("error", reject);
|
|
232
|
+
req.end();
|
|
233
|
+
});
|
|
234
|
+
assert.equal(res.status, 405);
|
|
235
|
+
});
|
|
236
|
+
//# sourceMappingURL=ui-server.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"ui-server.test.js","sourceRoot":"","sources":["../src/ui-server.test.ts"],"names":[],"mappings":"AAAA,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACzC,OAAO,EACL,8BAA8B,EAC9B,kBAAkB,EAClB,cAAc,EACd,uBAAuB,EACvB,aAAa,EACb,eAAe,EACf,YAAY,EACZ,eAAe,EACf,kBAAkB,EAClB,kBAAkB,GACnB,MAAM,gBAAgB,CAAC;AAGxB,8EAA8E;AAC9E,eAAe;AACf,8EAA8E;AAE9E,IAAI,CAAC,uEAAuE,EAAE,GAAG,EAAE;IACjF,MAAM,CAAC,GAAG,uBAAuB,EAAE,CAAC;IACpC,MAAM,CAAC,GAAG,uBAAuB,EAAE,CAAC;IACpC,MAAM,CAAC,QAAQ,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;IACtB,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,MAAM,IAAI,EAAE,CAAC,CAAC;IAC1B,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,kBAAkB,CAAC,CAAC;AACtC,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,iFAAiF,EAAE,GAAG,EAAE;IAC3F,MAAM,KAAK,GAAG,uBAAuB,EAAE,CAAC;IACxC,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,EAAE,KAAK,CAAC,EAAE,IAAI,CAAC,CAAC;IAC/C,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,GAAG,KAAK,GAAG,EAAE,KAAK,CAAC,EAAE,KAAK,CAAC,CAAC;IACtD,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,EAAE,KAAK,CAAC,CAAC;IAC7D,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,SAAS,EAAE,KAAK,CAAC,EAAE,KAAK,CAAC,CAAC;IACpD,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,EAAE,EAAE,KAAK,CAAC,EAAE,KAAK,CAAC,CAAC;AAC/C,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,8EAA8E,EAAE,GAAG,EAAE;IACxF,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,gBAAgB,EAAE,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;IAC1D,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,gBAAgB,EAAE,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;IAC1D,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,gBAAgB,EAAE,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;IAC1D,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,YAAY,EAAE,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;IACtD,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,gBAAgB,EAAE,IAAI,CAAC,EAAE,KAAK,CAAC,CAAC;IAC3D,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,mBAAmB,EAAE,IAAI,CAAC,EAAE,KAAK,CAAC,CAAC;IAC9D,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,WAAW,EAAE,IAAI,CAAC,EAAE,KAAK,CAAC,CAAC;IACtD,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,SAAS,EAAE,IAAI,CAAC,EAAE,KAAK,CAAC,CAAC;IACpD,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,MAAM,EAAE,IAAI,CAAC,EAAE,KAAK,CAAC,CAAC;AACnD,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,yEAAyE,EAAE,GAAG,EAAE;IACnF,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,SAAS,EAAE,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;IACrD,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,uBAAuB,EAAE,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;IACnE,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,uBAAuB,EAAE,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;IACnE,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,mBAAmB,EAAE,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;IAC/D,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,uBAAuB,EAAE,IAAI,CAAC,EAAE,KAAK,CAAC,CAAC;IACpE,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,sBAAsB,EAAE,IAAI,CAAC,EAAE,KAAK,CAAC,CAAC;IACnE,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,MAAM,EAAE,IAAI,CAAC,EAAE,KAAK,CAAC,CAAC;AACrD,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,8DAA8D,EAAE,GAAG,EAAE;IACxE,MAAM,OAAO,GAAG,eAAe,EAAE,CAAC;IAClC,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,yBAAyB,CAAC,EAAE,oBAAoB,CAAC,CAAC;IACvE,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,yBAAyB,CAAC,EAAE,wBAAwB,CAAC,CAAC;IAC3E,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,iBAAiB,CAAC,EAAE,MAAM,CAAC,CAAC;IACjD,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,eAAe,CAAC,EAAE,UAAU,CAAC,CAAC;AACrD,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,oDAAoD,EAAE,GAAG,EAAE;IAC9D,MAAM,GAAG,GAAG,kBAAkB,CAAC,QAAQ,uBAAuB,EAAE,EAAE,EAAE,EAAE,CAAC,MAAM,EAAE,CAAC,CAAC;IACjF,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,EAAE,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC;QAC9B,MAAM,CAAC,EAAE,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,CAAC;QAC9B,IAAI,OAAO,CAAC,QAAQ,KAAK,OAAO,EAAE,CAAC;YACjC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,GAAG,KAAK,EAAE,KAAK,CAAC,CAAC;QACzC,CAAC;IACH,CAAC;YAAS,CAAC;QACT,EAAE,CAAC,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACnD,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,qEAAqE,EAAE,GAAG,EAAE;IAC/E,MAAM,GAAG,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,oBAAoB,CAAC,CAAC,CAAC;IACzE,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,SAAS,CAAC,CAAC;QACzC,kBAAkB,CAAC,MAAM,EAAE,+BAA+B,CAAC,CAAC;QAC5D,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,iCAAiC,CAAC,CAAC;QACjF,MAAM,SAAS,GAAG,EAAE,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,CAAC,SAAS,CAAC,SAAS,EAAE,CAAC,SAAS,CAAC,CAAC,CAAC;IAC3C,CAAC;YAAS,CAAC;QACT,EAAE,CAAC,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACnD,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,6EAA6E,EAAE,GAAG,EAAE;IACvF,MAAM,KAAK,GAAG,cAAc,CAAC,CAAC,EAAE,EAAE,IAAI,EAAE,kBAAkB,EAAE,CAAC,CAAC;IAC9D,MAAM,CAAC,KAAK,CAAC,KAAK,EAAE,8CAA8C,CAAC,CAAC;AACtE,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,kEAAkE,EAAE,GAAG,EAAE;IAC5E,MAAM,KAAK,GAAG,kBAAkB,CAAC,EAAE,EAAE,sBAAsB,CAAC,CAAC;IAC7D,MAAM,CAAC,SAAS,CAAC,KAAK,CAAC,KAAK,EAAE,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC,CAAC;IACpD,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;IACrC,MAAM,MAAM,GAAG,kBAAkB,CAAC,KAAK,CAAC,QAAQ,EAAE,eAAe,CAAC,CAAC;IACnE,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC,CAAC;IACtD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC;AACpC,CAAC,CAAC,CAAC;AAYH,SAAS,OAAO,CAAC,IAAY,EAAE,WAAmB,EAAE,UAAkC,EAAE;IACtF,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QACrC,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,OAAO,EAAE,EAAE,CAAC,GAAG,EAAE,EAAE;YACxF,IAAI,IAAI,GAAG,EAAE,CAAC;YACd,GAAG,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,CAAC;YACnC,GAAG,CAAC,EAAE,CAAC,KAAK,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,GAAG,CAAC,UAAU,IAAI,CAAC,EAAE,OAAO,EAAE,GAAG,CAAC,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC;QAC5F,CAAC,CAAC,CAAC;QACH,GAAG,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QACxB,GAAG,CAAC,GAAG,EAAE,CAAC;IACZ,CAAC,CAAC,CAAC;AACL,CAAC;AAED,mGAAmG;AACnG,SAAS,UAAU,CAAC,IAAY,EAAE,WAAmB,EAAE,OAA+B,EAAE,SAAiB;IACvG,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QACrC,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,OAAO,EAAE,EAAE,CAAC,GAAG,EAAE,EAAE;YACxF,IAAI,IAAI,GAAG,EAAE,CAAC;YACd,MAAM,KAAK,GAAG,UAAU,CAAC,GAAG,EAAE;gBAC5B,GAAG,CAAC,OAAO,EAAE,CAAC;gBACd,OAAO,CAAC,IAAI,CAAC,CAAC;YAChB,CAAC,EAAE,IAAI,CAAC,CAAC;YACT,GAAG,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE;gBACnB,IAAI,IAAI,CAAC,CAAC;gBACV,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,IAAI,SAAS,EAAE,CAAC;oBACtD,YAAY,CAAC,KAAK,CAAC,CAAC;oBACpB,GAAG,CAAC,OAAO,EAAE,CAAC;oBACd,OAAO,CAAC,IAAI,CAAC,CAAC;gBAChB,CAAC;YACH,CAAC,CAAC,CAAC;YACH,GAAG,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QACH,GAAG,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QACxB,GAAG,CAAC,GAAG,EAAE,CAAC;IACZ,CAAC,CAAC,CAAC;AACL,CAAC;AAED,IAAI,YAAqC,CAAC;AAC1C,IAAI,SAA6B,CAAC;AAElC,SAAS,CAAC,GAAG,EAAE;IACb,IAAI,YAAY,EAAE,CAAC;QACjB,YAAY,CAAC,mBAAmB,EAAE,CAAC;QACnC,YAAY,CAAC,KAAK,EAAE,CAAC;QACrB,YAAY,GAAG,SAAS,CAAC;IAC3B,CAAC;IACD,IAAI,SAAS,EAAE,CAAC;QACd,EAAE,CAAC,MAAM,CAAC,SAAS,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACvD,SAAS,GAAG,SAAS,CAAC;IACxB,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,KAAK,UAAU,eAAe,CAAC,YAA6C,EAAE;IAC5E,MAAM,GAAG,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,oBAAoB,CAAC,CAAC,CAAC;IACzE,SAAS,GAAG,GAAG,CAAC;IAChB,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;IAClD,EAAE,CAAC,aAAa,CAAC,UAAU,EAAE,EAAE,CAAC,CAAC;IACjC,MAAM,KAAK,GAAG,uBAAuB,EAAE,CAAC;IAExC,MAAM,MAAM,GAAG,IAAI,CAAC,YAAY,EAAE,CAAC;IACnC,MAAM,IAAI,OAAO,CAAO,CAAC,OAAO,EAAE,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,EAAE,WAAW,EAAE,OAAO,CAAC,CAAC,CAAC;IAC7E,MAAM,IAAI,GAAI,MAAM,CAAC,OAAO,EAAuB,CAAC,IAAI,CAAC;IAEzD,MAAM,GAAG,GAA2B;QAClC,KAAK;QACL,IAAI;QACJ,UAAU;QACV,IAAI,EAAE,mDAAmD;QACzD,YAAY,EAAE,EAAE,eAAe,EAAE,EAAE,WAAW,EAAE,gCAAgC,EAAE,IAAI,EAAE,qBAAqB,EAAE,EAAE;QACjH,cAAc,EAAE,EAAE;QAClB,GAAG,SAAS;KACb,CAAC;IACF,MAAM,CAAC,EAAE,CAAC,SAAS,EAAE,8BAA8B,CAAC,GAAG,CAAC,CAAC,CAAC;IAC1D,YAAY,GAAG,MAAM,CAAC;IACtB,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,UAAU,EAAE,CAAC;AACrC,CAAC;AAED,IAAI,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;IAC7E,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,MAAM,eAAe,EAAE,CAAC;IAChD,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,QAAQ,KAAK,EAAE,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,CAAC,CAAC;IAChF,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IAC9B,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,EAAE,YAAY,CAAC,CAAC;IAC9D,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,EAAE,qBAAqB,CAAC,CAAC;IAC9C,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,OAAO,CAAC,iBAAiB,CAAC,EAAE,MAAM,CAAC,CAAC;IACrD,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,yBAAyB,CAAC,IAAI,EAAE,CAAC,EAAE,wBAAwB,CAAC,CAAC;AAC/F,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,wEAAwE,EAAE,KAAK,IAAI,EAAE;IACxF,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,MAAM,eAAe,EAAE,CAAC;IAChD,MAAM,KAAK,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,QAAQ,KAAK,GAAG,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,CAAC,CAAC;IACnF,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IAChC,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,kBAAkB,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,CAAC,CAAC;IACvF,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AACpC,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;IAC9D,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,MAAM,eAAe,EAAE,CAAC;IAChD,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,QAAQ,KAAK,EAAE,EAAE,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,CAAC;IAC3E,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AAChC,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;IACtF,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,MAAM,eAAe,EAAE,CAAC;IAChD,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,QAAQ,KAAK,EAAE,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,MAAM,EAAE,sBAAsB,EAAE,CAAC,CAAC;IAChH,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IAC9B,MAAM,EAAE,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,QAAQ,KAAK,EAAE,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,MAAM,EAAE,oBAAoB,IAAI,EAAE,EAAE,CAAC,CAAC;IACnH,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IAC7B,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,QAAQ,KAAK,EAAE,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,CAAC,CAAC;IACpF,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AACpC,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,kFAAkF,EAAE,KAAK,IAAI,EAAE;IAClG,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,MAAM,eAAe,EAAE,CAAC;IAChD,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,QAAQ,KAAK,uBAAuB,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,CAAC,CAAC;IACrG,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IAC9B,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,OAAO,CAAC,cAAc,CAAC,IAAI,EAAE,EAAE,YAAY,CAAC,CAAC;IAC9D,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,EAAE,gBAAgB,CAAC,CAAC;IACzC,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,IAAI,EAAE,QAAQ,KAAK,2BAA2B,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,CAAC,CAAC;IAC7G,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AACpC,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,kFAAkF,EAAE,KAAK,IAAI,EAAE;IAClG,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,UAAU,EAAE,GAAG,MAAM,eAAe,EAAE,CAAC;IAC5D,EAAE,CAAC,cAAc,CAAC,UAAU,EAAE,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,IAAI,EAAE,eAAe,EAAE,CAAC,IAAI,CAAC,CAAC;IAClH,EAAE,CAAC,cAAc,CAAC,UAAU,EAAE,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,IAAI,EAAE,iBAAiB,EAAE,CAAC,IAAI,CAAC,CAAC;IAEpH,MAAM,UAAU,GAAG,UAAU,CAAC,IAAI,EAAE,QAAQ,KAAK,SAAS,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC;IAC9F,MAAM,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,CAAC;IACxD,EAAE,CAAC,cAAc,CAAC,UAAU,EAAE,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,IAAI,EAAE,kBAAkB,EAAE,CAAC,IAAI,CAAC,CAAC;IAErH,MAAM,IAAI,GAAG,MAAM,UAAU,CAAC;IAC9B,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;IAC9B,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;IAC9B,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;IAC9B,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,wBAAwB,CAAC,CAAC;IAC7C,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,2BAA2B,CAAC,CAAC;AAClD,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;IACzF,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,UAAU,EAAE,GAAG,MAAM,eAAe,EAAE,CAAC;IAC5D,EAAE,CAAC,cAAc,CAAC,UAAU,EAAE,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,IAAI,EAAE,eAAe,EAAE,CAAC,IAAI,CAAC,CAAC;IAClH,EAAE,CAAC,cAAc,CAAC,UAAU,EAAE,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,IAAI,EAAE,iBAAiB,EAAE,CAAC,IAAI,CAAC,CAAC;IACpH,EAAE,CAAC,cAAc,CAAC,UAAU,EAAE,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,IAAI,EAAE,kBAAkB,EAAE,CAAC,IAAI,CAAC,CAAC;IAErH,MAAM,IAAI,GAAG,MAAM,UAAU,CAAC,IAAI,EAAE,QAAQ,KAAK,SAAS,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,eAAe,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC;IACpH,MAAM,CAAC,YAAY,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;IACrC,MAAM,CAAC,YAAY,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;IACrC,MAAM,CAAC,KAAK,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;AAChC,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;IACxD,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,MAAM,eAAe,EAAE,CAAC;IAChD,MAAM,GAAG,GAAG,MAAM,IAAI,OAAO,CAAa,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QAC5D,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CACtB,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,IAAI,EAAE,QAAQ,KAAK,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,aAAa,IAAI,EAAE,EAAE,EAAE,EAC1G,CAAC,CAAC,EAAE,EAAE;YACJ,IAAI,IAAI,GAAG,EAAE,CAAC;YACd,CAAC,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,CAAC;YACjC,CAAC,CAAC,EAAE,CAAC,KAAK,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC,CAAC,UAAU,IAAI,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC;QACtF,CAAC,CACF,CAAC;QACF,GAAG,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QACxB,GAAG,CAAC,GAAG,EAAE,CAAC;IACZ,CAAC,CAAC,CAAC;IACH,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AAChC,CAAC,CAAC,CAAC"}
|
package/dist/update.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../src/update.ts"],"names":[],"mappings":"
|
|
1
|
+
{"version":3,"file":"update.d.ts","sourceRoot":"","sources":["../src/update.ts"],"names":[],"mappings":"AA8CA,wBAAgB,SAAS,IAAI,IAAI,CAqEhC"}
|
package/dist/update.js
CHANGED
|
@@ -1,5 +1,6 @@
|
|
|
1
1
|
import { spawnSync } from "node:child_process";
|
|
2
2
|
import { createRequire } from "node:module";
|
|
3
|
+
import { runUpdateSkill } from "./skill.js";
|
|
3
4
|
const PKG_NAME = "@zephyrdeng/pi-review";
|
|
4
5
|
function getCurrentVersion() {
|
|
5
6
|
const require = createRequire(import.meta.url);
|
|
@@ -22,6 +23,19 @@ function box(lines) {
|
|
|
22
23
|
const body = lines.map((l) => `│ ${pad(l)} │`).join("\n");
|
|
23
24
|
return `${top}\n${empty}\n${body}\n${empty}\n${bot}\n`;
|
|
24
25
|
}
|
|
26
|
+
function refreshSkill() {
|
|
27
|
+
const skill = runUpdateSkill();
|
|
28
|
+
if (!skill.ok) {
|
|
29
|
+
process.stderr.write(box([
|
|
30
|
+
"Skill refresh failed.",
|
|
31
|
+
"",
|
|
32
|
+
"Retry with:",
|
|
33
|
+
" pi-review install-skill",
|
|
34
|
+
]));
|
|
35
|
+
return false;
|
|
36
|
+
}
|
|
37
|
+
return true;
|
|
38
|
+
}
|
|
25
39
|
export function runUpdate() {
|
|
26
40
|
const current = getCurrentVersion();
|
|
27
41
|
const latest = getLatestVersion();
|
|
@@ -29,36 +43,51 @@ export function runUpdate() {
|
|
|
29
43
|
process.stderr.write(box(["Failed to fetch latest version from npm registry"]));
|
|
30
44
|
process.exit(1);
|
|
31
45
|
}
|
|
46
|
+
let packageLine;
|
|
32
47
|
if (current === latest) {
|
|
48
|
+
packageLine = `Package already up to date (v${current})`;
|
|
33
49
|
process.stdout.write(box([
|
|
34
50
|
`pi-review v${current}`,
|
|
35
51
|
"",
|
|
36
|
-
"
|
|
52
|
+
"Package already up to date ✓",
|
|
53
|
+
"Refreshing agent skill...",
|
|
37
54
|
]));
|
|
38
|
-
return;
|
|
39
55
|
}
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
"Updating...",
|
|
44
|
-
]));
|
|
45
|
-
const install = spawnSync("npm", ["install", "-g", `${PKG_NAME}@latest`], {
|
|
46
|
-
encoding: "utf8",
|
|
47
|
-
stdio: "inherit",
|
|
48
|
-
timeout: 60_000,
|
|
49
|
-
});
|
|
50
|
-
if (install.status !== 0) {
|
|
51
|
-
process.stderr.write(box([
|
|
52
|
-
"Update failed. Run manually:",
|
|
56
|
+
else {
|
|
57
|
+
process.stdout.write(box([
|
|
58
|
+
`Update available: ${current} → ${latest}`,
|
|
53
59
|
"",
|
|
54
|
-
|
|
60
|
+
"Updating package...",
|
|
61
|
+
]));
|
|
62
|
+
const install = spawnSync("npm", ["install", "-g", `${PKG_NAME}@latest`], {
|
|
63
|
+
encoding: "utf8",
|
|
64
|
+
stdio: "inherit",
|
|
65
|
+
timeout: 60_000,
|
|
66
|
+
});
|
|
67
|
+
if (install.status !== 0) {
|
|
68
|
+
process.stderr.write(box([
|
|
69
|
+
"Update failed. Run manually:",
|
|
70
|
+
"",
|
|
71
|
+
`npm install -g ${PKG_NAME}@latest`,
|
|
72
|
+
]));
|
|
73
|
+
process.exit(1);
|
|
74
|
+
}
|
|
75
|
+
packageLine = `Package updated: ${current} → ${latest}`;
|
|
76
|
+
process.stdout.write(box([
|
|
77
|
+
packageLine,
|
|
78
|
+
"",
|
|
79
|
+
"Refreshing agent skill...",
|
|
55
80
|
]));
|
|
56
|
-
process.exit(1);
|
|
57
81
|
}
|
|
82
|
+
// skills CLI pulls latest skill content from the source repo; the direct
|
|
83
|
+
// fallback copies from this package tree (refreshed on disk after npm -g).
|
|
84
|
+
const skillOk = refreshSkill();
|
|
58
85
|
process.stdout.write(box([
|
|
59
|
-
|
|
86
|
+
packageLine,
|
|
87
|
+
skillOk ? "Skill refreshed ✓" : "Skill refresh failed",
|
|
60
88
|
"",
|
|
61
|
-
"Done ✓",
|
|
89
|
+
skillOk ? "Done ✓" : "Package ok; skill needs manual install-skill",
|
|
62
90
|
]));
|
|
91
|
+
process.exit(skillOk ? 0 : 1);
|
|
63
92
|
}
|
|
64
93
|
//# sourceMappingURL=update.js.map
|
package/dist/update.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"update.js","sourceRoot":"","sources":["../src/update.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAC/C,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAE5C,MAAM,QAAQ,GAAG,uBAAuB,CAAC;AAEzC,SAAS,iBAAiB;IACxB,MAAM,OAAO,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAC/C,MAAM,GAAG,GAAG,OAAO,CAAC,iBAAiB,
|
|
1
|
+
{"version":3,"file":"update.js","sourceRoot":"","sources":["../src/update.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAC/C,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAC5C,OAAO,EAAE,cAAc,EAAE,MAAM,YAAY,CAAC;AAE5C,MAAM,QAAQ,GAAG,uBAAuB,CAAC;AAEzC,SAAS,iBAAiB;IACxB,MAAM,OAAO,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAC/C,MAAM,GAAG,GAAG,OAAO,CAAC,iBAAiB,CAAwB,CAAC;IAC9D,OAAO,GAAG,CAAC,OAAO,CAAC;AACrB,CAAC;AAED,SAAS,gBAAgB;IACvB,MAAM,MAAM,GAAG,SAAS,CAAC,KAAK,EAAE,CAAC,MAAM,EAAE,QAAQ,EAAE,SAAS,CAAC,EAAE;QAC7D,QAAQ,EAAE,MAAM;QAChB,OAAO,EAAE,MAAM;KAChB,CAAC,CAAC;IACH,OAAO,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;AAC3D,CAAC;AAED,SAAS,GAAG,CAAC,KAAe;IAC1B,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC;IACtD,MAAM,GAAG,GAAG,CAAC,CAAS,EAAE,EAAE,CAAC,CAAC,GAAG,GAAG,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,CAAC,MAAM,CAAC,CAAC;IAC5D,MAAM,GAAG,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,CAAC,GAAG,CAAC;IACzC,MAAM,GAAG,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,CAAC,GAAG,CAAC;IACzC,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,CAAC,GAAG,CAAC;IAC3C,MAAM,IAAI,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC1D,OAAO,GAAG,GAAG,KAAK,KAAK,KAAK,IAAI,KAAK,KAAK,KAAK,GAAG,IAAI,CAAC;AACzD,CAAC;AAED,SAAS,YAAY;IACnB,MAAM,KAAK,GAAG,cAAc,EAAE,CAAC;IAC/B,IAAI,CAAC,KAAK,CAAC,EAAE,EAAE,CAAC;QACd,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,GAAG,CAAC;YACF,uBAAuB;YACvB,EAAE;YACF,aAAa;YACb,2BAA2B;SAC5B,CAAC,CACH,CAAC;QACF,OAAO,KAAK,CAAC;IACf,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,UAAU,SAAS;IACvB,MAAM,OAAO,GAAG,iBAAiB,EAAE,CAAC;IAEpC,MAAM,MAAM,GAAG,gBAAgB,EAAE,CAAC;IAClC,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,kDAAkD,CAAC,CAAC,CAAC,CAAC;QAChF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,IAAI,WAAmB,CAAC;IACxB,IAAI,OAAO,KAAK,MAAM,EAAE,CAAC;QACvB,WAAW,GAAG,gCAAgC,OAAO,GAAG,CAAC;QACzD,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,GAAG,CAAC;YACF,cAAc,OAAO,EAAE;YACvB,EAAE;YACF,8BAA8B;YAC9B,2BAA2B;SAC5B,CAAC,CACH,CAAC;IACJ,CAAC;SAAM,CAAC;QACN,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,GAAG,CAAC;YACF,qBAAqB,OAAO,MAAM,MAAM,EAAE;YAC1C,EAAE;YACF,qBAAqB;SACtB,CAAC,CACH,CAAC;QAEF,MAAM,OAAO,GAAG,SAAS,CAAC,KAAK,EAAE,CAAC,SAAS,EAAE,IAAI,EAAE,GAAG,QAAQ,SAAS,CAAC,EAAE;YACxE,QAAQ,EAAE,MAAM;YAChB,KAAK,EAAE,SAAS;YAChB,OAAO,EAAE,MAAM;SAChB,CAAC,CAAC;QAEH,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACzB,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,GAAG,CAAC;gBACF,8BAA8B;gBAC9B,EAAE;gBACF,kBAAkB,QAAQ,SAAS;aACpC,CAAC,CACH,CAAC;YACF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClB,CAAC;QAED,WAAW,GAAG,oBAAoB,OAAO,MAAM,MAAM,EAAE,CAAC;QACxD,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,GAAG,CAAC;YACF,WAAW;YACX,EAAE;YACF,2BAA2B;SAC5B,CAAC,CACH,CAAC;IACJ,CAAC;IAED,yEAAyE;IACzE,2EAA2E;IAC3E,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;IAE/B,OAAO,CAAC,MAAM,CAAC,KAAK,CAClB,GAAG,CAAC;QACF,WAAW;QACX,OAAO,CAAC,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,sBAAsB;QACtD,EAAE;QACF,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,8CAA8C;KACpE,CAAC,CACH,CAAC;IACF,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;AAChC,CAAC"}
|