pi-harness-runtime 1.1.439 → 1.1.441

package/packages/jev-judge/tests/question-risk.test.ts

@@ -1,129 +0,0 @@
-/**
- * The risk facts of a pending question.
- *
- * The autonomous decision is only as safe as the facts it rests on: a
- * question that asks for permission for a destructive or irreversible
- * action must never be approved while the user is away.
- */
-
-import { describe, it, expect } from "bun:test";
-import { classifyQuestionRisk } from "../src/question-risk.js";
-
-describe("classifyQuestionRisk", () => {
-	describe("a routine continuation question", () => {
-		it("is harmless enough for an autonomous decision", () => {
-			const risk = classifyQuestionRisk({
-				kind: "routine",
-				text: "Should I continue with the remaining tasks?",
-			});
-
-			expect(risk.requiresExplicitAuthorization).toBe(false);
-			expect(risk.destructive).toBe(false);
-			expect(risk.reversible).toBe(true);
-			expect(risk.external).toBe(false);
-		});
-	});
-
-	describe("a question that asks for permission", () => {
-		it("keeps a benign approval question auto-approvable", () => {
-			const risk = classifyQuestionRisk({
-				kind: "approval",
-				text: "Want me to implement the toggle buttons now?",
-			});
-
-			// The timeout exists for routine, reversible work: the agent
-			// proposing work is not a hard authorization block.
-			expect(risk.requiresExplicitAuthorization).toBe(false);
-			expect(risk.destructive).toBe(false);
-			expect(risk.reversible).toBe(true);
-		});
-
-		it("needs an explicit authorization for an explicit question", () => {
-			const risk = classifyQuestionRisk({
-				kind: "explicit",
-				text: "Shall I push the release?",
-			});
-
-			expect(risk.requiresExplicitAuthorization).toBe(true);
-		});
-
-		it("needs an explicit authorization for a risky approval question", () => {
-			const risk = classifyQuestionRisk({
-				kind: "approval",
-				text: "Want me to remove the old config file now?",
-			});
-
-			expect(risk.requiresExplicitAuthorization).toBe(true);
-			expect(risk.destructive).toBe(true);
-		});
-	});
-
-	describe("a destructive action", () => {
-		it("is destructive even in a routine question", () => {
-			const risk = classifyQuestionRisk({
-				kind: "routine",
-				text: "Should I remove the old config file now?",
-			});
-
-			expect(risk.destructive).toBe(true);
-			expect(risk.requiresExplicitAuthorization).toBe(true);
-			expect(risk.reversible).toBe(false);
-		});
-
-		it("recognizes a dropped database table", () => {
-			const risk = classifyQuestionRisk({
-				kind: "routine",
-				text: "Should I drop table users from the dev database?",
-			});
-
-			expect(risk.destructive).toBe(true);
-			expect(risk.reversible).toBe(false);
-		});
-
-		it("recognizes a force push", () => {
-			const risk = classifyQuestionRisk({
-				kind: "routine",
-				text: "Should I git push --force to the remote?",
-			});
-
-			expect(risk.destructive).toBe(true);
-		});
-	});
-
-	describe("an uncertain question", () => {
-		it("waits when the kind is unknown", () => {
-			const risk = classifyQuestionRisk({ text: "Should I go on?" });
-
-			expect(risk.requiresExplicitAuthorization).toBe(true);
-			expect(risk.signals).toContain("unknown-question");
-		});
-
-		it("waits when the text is empty", () => {
-			const risk = classifyQuestionRisk({ kind: "routine", text: "   " });
-
-			expect(risk.requiresExplicitAuthorization).toBe(true);
-			expect(risk.signals).toContain("unknown-question");
-		});
-	});
-
-	describe("an action that reaches the outside world", () => {
-		it("is external", () => {
-			const risk = classifyQuestionRisk({
-				kind: "routine",
-				text: "Should I deploy to production now?",
-			});
-
-			expect(risk.external).toBe(true);
-			expect(risk.requiresExplicitAuthorization).toBe(true);
-		});
-
-		it("recognizes a message to the users", () => {
-			const risk = classifyQuestionRisk({
-				kind: "routine",
-				text: "Should I send the announcement email to all customers?",
-			});
-
-			expect(risk.external).toBe(true);
-		});
-	});
-});