dsh-pstack 0.2.0-beta.1 → 0.2.0-beta.3

package/dist/pstack/skills/poteto-mode/scripts/watch-pr/cli.test.mjs

@@ -0,0 +1,217 @@
+import { expect } from "../expect.mjs";
+import "../support.mjs";
+import { parsePrNumber } from "./types.mjs";
+import { renderJson, renderPretty } from "./render.mjs";
+import { main, parseArgs } from "./cli.mjs";
+import { fakeReader, passingCheck } from "./fakes.test-helper.mjs";
+import { describe, it } from "node:test";
+// runtime/skills/poteto-mode/scripts/watch-pr/cli.test.ts
+const silentIo = {
+	stdout: () => {},
+	stderr: () => {}
+};
+function testRuntime(reader) {
+	const stdout = [];
+	const stderr = [];
+	return {
+		stdout,
+		stderr,
+		runtime: {
+			reader,
+			clock: {
+				now: () => 0,
+				observedAt: () => "2026-07-26T00:00:00.000Z",
+				async sleep() {
+					throw new Error("test unexpectedly slept");
+				}
+			},
+			stdout: (value) => stdout.push(value),
+			stderr: (value) => stderr.push(value)
+		}
+	};
+}
+describe("parseArgs", () => {
+	it("uses the specified defaults", () => {
+		expect(parseArgs([], silentIo)).toMatchObject({
+			owner: null,
+			repo: null,
+			pr: null,
+			mode: "single",
+			stackPrs: [],
+			statusOnly: false,
+			pretty: false,
+			polling: {
+				interval: 60,
+				sweepInterval: 300,
+				timeout: 0,
+				maxQueryErrors: 5,
+				allowDraft: false
+			}
+		});
+	});
+	it("parses a frozen queued stack bottom-to-top", () => {
+		const parsed = parseArgs([
+			"--queued-stack",
+			"--stack-prs",
+			"#10, 11,#12",
+			"--interval",
+			"2.5",
+			"--sweep-interval",
+			"30",
+			"--timeout",
+			"0",
+			"--max-query-errors",
+			"3",
+			"--allow-draft",
+			"--pretty"
+		], silentIo);
+		expect(parsed.mode).toBe("queued-stack");
+		expect(parsed.stackPrs.map(Number)).toEqual([
+			10,
+			11,
+			12
+		]);
+		expect(parsed.polling).toEqual({
+			interval: 2.5,
+			sweepInterval: 30,
+			timeout: 0,
+			maxQueryErrors: 3,
+			allowDraft: true
+		});
+		expect(parsed.pretty).toBe(true);
+	});
+	it("rejects every invalid mode and numeric shape as usage", async () => {
+		for (const argv of [
+			["--unknown"],
+			["--interval", "0"],
+			["--sweep-interval", "-1"],
+			["--timeout", "-1"],
+			["--max-query-errors", "1.5"],
+			["--stack", "--queued-stack"],
+			["--stack-prs", "1,2"],
+			[
+				"--queued-stack",
+				"--stack-prs",
+				"1,1"
+			]
+		]) {
+			const harness = testRuntime(fakeReader());
+			expect(await main(argv, harness.runtime)).toBe(64);
+			expect(harness.stdout).toEqual([]);
+			expect(harness.stderr.join("")).toContain("error:");
+		}
+	});
+});
+describe("rendering", () => {
+	const context = {
+		owner: "owner",
+		repo: "repo",
+		number: parsePrNumber(1)
+	};
+	const status = {
+		schemaVersion: 1,
+		sequence: 1,
+		observedAt: "2026-07-26T00:00:00.000Z",
+		mode: "single",
+		kind: "STATUS",
+		terminal: true,
+		exitCode: 0,
+		reason: "status-only",
+		rows: [{
+			kind: "merged",
+			context,
+			facts: {
+				context,
+				mergeable: "MERGEABLE",
+				mergeStateStatus: "CLEAN",
+				reviewDecision: "APPROVED",
+				headRefOid: "head",
+				headRefName: "feature",
+				baseRefName: "main",
+				state: "MERGED",
+				mergedAt: "now",
+				isDraft: false
+			}
+		}]
+	};
+	it("emits compact valid JSON by default", () => {
+		const rendered = renderJson(status);
+		expect(rendered.endsWith("\n")).toBe(true);
+		expect(JSON.parse(rendered)).toEqual(status);
+	});
+	it("renders the Markdown table from the same verdict only", () => {
+		const rendered = renderPretty(status);
+		expect(rendered).toContain("| PR | CI | Review | Merge |");
+		expect(rendered).toContain("| [#1](https://github.com/owner/repo/pull/1) | — | — | ✅ merged |");
+	});
+});
+describe("main", () => {
+	it("returns EX_USAGE 64 and writes usage errors only to stderr", async () => {
+		const harness = testRuntime(fakeReader());
+		expect(await main(["--interval", "0"], harness.runtime)).toBe(64);
+		expect(harness.stdout).toEqual([]);
+		expect(harness.stderr.join("")).toContain("option '--interval <seconds>' argument '0' is invalid");
+	});
+	it("bypasses the queue machine for queued-stack status-only", async () => {
+		const harness = testRuntime(fakeReader());
+		const code = await main([
+			"--owner",
+			"owner",
+			"--repo",
+			"repo",
+			"--queued-stack",
+			"--stack-prs",
+			"1",
+			"--status-only"
+		], harness.runtime);
+		expect(code).toBe(0);
+		expect(harness.stdout).toHaveLength(1);
+		const verdict = JSON.parse(harness.stdout[0]);
+		expect(verdict).toMatchObject({
+			kind: "STATUS",
+			terminal: true,
+			exitCode: 0,
+			mode: "queued-stack"
+		});
+		expect(harness.stdout[0]).not.toContain("\"kind\":\"QUEUE\"");
+	});
+	it("returns exit 4 for a hidden GitHub-side CI refusal", async () => {
+		const harness = testRuntime(fakeReader({
+			facts: { mergeStateStatus: "BLOCKED" },
+			fastPath: {
+				kind: "checks",
+				checks: [passingCheck()]
+			},
+			commitRollups: [{
+				oid: "head",
+				state: "FAILURE"
+			}]
+		}));
+		const code = await main([
+			"--owner",
+			"owner",
+			"--repo",
+			"repo",
+			"--pr",
+			"1"
+		], harness.runtime);
+		expect(code).toBe(4);
+		expect(harness.stdout).toHaveLength(1);
+		expect(JSON.parse(harness.stdout[0])).toMatchObject({
+			kind: "BLOCKER",
+			exitCode: 4,
+			blocker: {
+				kind: "failing-checks",
+				ci: { kind: "ci-github-rejected" }
+			}
+		});
+	});
+	it("shows help without touching the reader", async () => {
+		const reader = fakeReader();
+		const harness = testRuntime(reader);
+		expect(await main(["--help"], harness.runtime)).toBe(0);
+		expect(harness.stdout.join("")).toContain("JSON (NDJSON while polling)");
+		expect(reader.calls).toEqual([]);
+	});
+});
+export {};

package/dist/pstack/skills/poteto-mode/scripts/watch-pr/fakes.test-helper.mjs

@@ -0,0 +1,119 @@
+import { parsePrNumber } from "./types.mjs";
+// runtime/skills/poteto-mode/scripts/watch-pr/fakes.test-helper.ts
+function toRawThread(thread) {
+	return {
+		id: thread.id,
+		firstComment: thread.firstComment,
+		resolved: false
+	};
+}
+function passingCheck(name = "ci") {
+	return {
+		kind: "passed",
+		name,
+		reportedState: "SUCCESS",
+		description: "",
+		link: "",
+		workflow: ""
+	};
+}
+function pendingCheck(name = "ci") {
+	return {
+		kind: "pending",
+		name,
+		reportedState: "PENDING",
+		description: "",
+		link: "",
+		workflow: ""
+	};
+}
+function failedCheck(name = "ci") {
+	return {
+		kind: "failed",
+		name,
+		reportedState: "FAILURE",
+		description: "",
+		link: "",
+		workflow: ""
+	};
+}
+function fakeReader(options = {}) {
+	const calls = [];
+	const context = options.current ?? {
+		owner: "owner",
+		repo: "repo",
+		number: parsePrNumber(1)
+	};
+	const defaults = {
+		context,
+		mergeable: "MERGEABLE",
+		mergeStateStatus: "CLEAN",
+		reviewDecision: "APPROVED",
+		headRefOid: "head",
+		headRefName: "feature",
+		baseRefName: "main",
+		state: "OPEN",
+		mergedAt: null,
+		isDraft: false
+	};
+	let page = 0;
+	let threadPage = 0;
+	return {
+		calls,
+		async originRepo() {
+			calls.push("originRepo");
+			return options.origin === void 0 ? {
+				owner: "owner",
+				repo: "repo"
+			} : options.origin;
+		},
+		async currentPr(pr) {
+			calls.push("currentPr");
+			return {
+				...context,
+				number: pr ?? context.number
+			};
+		},
+		async pullRequest(requested) {
+			calls.push("pullRequest");
+			return {
+				...defaults,
+				...options.facts,
+				context: requested
+			};
+		},
+		async openPullRequests() {
+			calls.push("openPullRequests");
+			return options.openPullRequests ?? [];
+		},
+		async checksFastPath() {
+			calls.push("checksFastPath");
+			return options.fastPath ?? {
+				kind: "checks",
+				checks: [passingCheck()]
+			};
+		},
+		async checkRollupPage(_requested, after) {
+			calls.push(`checkRollupPage:${after ?? "null"}`);
+			return options.rollupPages?.[page++] ?? {
+				checks: [],
+				endCursor: null
+			};
+		},
+		async reviewThreadPage(_requested, after) {
+			calls.push(`reviewThreadPage:${after ?? "null"}`);
+			return options.threadPages?.[threadPage++] ?? {
+				threads: (options.threads ?? []).map(toRawThread),
+				endCursor: null
+			};
+		},
+		async commitRollups() {
+			calls.push("commitRollups");
+			return options.commitRollups ?? [{
+				oid: "head",
+				state: "SUCCESS"
+			}];
+		}
+	};
+}
+export { failedCheck, fakeReader, passingCheck, pendingCheck };