gentle-pi 2.1.2 → 2.3.0

package/tests/fixtures/native-review-cli/v2.1.3/sdd-status.json

@@ -1,200 +0,0 @@
-{
-  "schemaName": "gentle-ai.sdd-status",
-  "schemaVersion": 1,
-  "changeName": "native-review-authority-parity",
-  "artifactStore": "openspec",
-  "planningHome": {
-    "mode": "repo-local",
-    "path": "/tmp/gentle-ai-v212-contract-9XGIB4/openspec"
-  },
-  "changeRoot": "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity",
-  "artifactPaths": {
-    "proposal": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/proposal.md"
-    ],
-    "specs": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/specs/review/spec.md"
-    ],
-    "design": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/design.md"
-    ],
-    "tasks": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/tasks.md"
-    ],
-    "applyProgress": [],
-    "verifyReport": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/verify-report.md"
-    ],
-    "reviewPolicy": [],
-    "reviewLedger": [],
-    "reviewReceipt": [],
-    "reviewBundle": [],
-    "reviewContext": [],
-    "reviewState": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/reviews/transaction.json"
-    ]
-  },
-  "contextFiles": {
-    "proposal": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/proposal.md"
-    ],
-    "specs": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/specs/review/spec.md"
-    ],
-    "design": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/design.md"
-    ],
-    "tasks": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/tasks.md"
-    ],
-    "applyProgress": [],
-    "verifyReport": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/verify-report.md"
-    ],
-    "reviewPolicy": [],
-    "reviewLedger": [],
-    "reviewReceipt": [],
-    "reviewBundle": [],
-    "reviewContext": [],
-    "reviewState": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4/openspec/changes/native-review-authority-parity/reviews/transaction.json"
-    ]
-  },
-  "artifacts": {
-    "applyProgress": "missing",
-    "design": "done",
-    "proposal": "done",
-    "reviewBundle": "missing",
-    "reviewContext": "missing",
-    "reviewLedger": "missing",
-    "reviewReceipt": "missing",
-    "reviewState": "done",
-    "specs": "done",
-    "tasks": "done",
-    "verifyReport": "done"
-  },
-  "taskProgress": {
-    "total": 1,
-    "completed": 1,
-    "pending": 0,
-    "allComplete": true
-  },
-  "dependencies": {
-    "proposal": "all_done",
-    "specs": "all_done",
-    "design": "all_done",
-    "tasks": "all_done",
-    "apply": "all_done",
-    "verify": "all_done",
-    "archive": "ready"
-  },
-  "applyState": "all_done",
-  "actionContext": {
-    "mode": "repo-local",
-    "workspaceRoot": "/tmp/gentle-ai-v212-contract-9XGIB4",
-    "allowedEditRoots": [
-      "/tmp/gentle-ai-v212-contract-9XGIB4"
-    ]
-  },
-  "relationships": {
-    "dependsOn": [],
-    "supersedes": [],
-    "amends": [],
-    "conflictsWith": [],
-    "sameDomainActiveChanges": []
-  },
-  "remediationState": {
-    "required": false,
-    "complete": false,
-    "failedEvidenceRevision": "",
-    "lineageId": "",
-    "generation": 0,
-    "fixBatch": 0,
-    "reason": ""
-  },
-  "reviewGate": {
-    "result": "allow",
-    "reason": "explicit bound compact authority exactly matches the current repository"
-  },
-  "reviewTransaction": {
-    "schema": "gentle-ai.review-transaction/v1",
-    "lineage_id": "issue-1099-corrected-candidate-review-20260710",
-    "mode": "ordinary_4r",
-    "generation": 1,
-    "state": "reviewing",
-    "snapshot": {
-      "kind": "current-changes",
-      "base_tree": "ba040f17cdcd2a5960361971eaff79684c48847a",
-      "candidate_tree": "5bc2f4bdd086d064bc3ada255f16c631facc6839",
-      "paths_digest": "sha256:4a52771035fe6ef25f0155b11c052a661879b1cd1af4bbe5cd337118a9dd29dd",
-      "intended_untracked": [
-        "assets/agents/gentle-ai-explore.md",
-        "assets/agents/gentle-ai-verify.md"
-      ],
-      "intended_untracked_proof": "sha256:ea65650186e3d14ed53ae4912a2a30127863fe9aec0e4c14050e4fa154b0bfa3",
-      "paths": [
-        "assets/agents/gentle-ai-explore.md",
-        "assets/agents/gentle-ai-verify.md",
-        "assets/orchestrator-delegation.md",
-        "assets/orchestrator.md",
-        "tests/package-manifest.test.ts",
-        "tests/sdd-agent-tools.test.ts"
-      ],
-      "identity": "sha256:d88627d276b790d1ed58abc44118d1d95a9620d9755d5e61c8fbda0f97cea53c"
-    },
-    "base_tree": "ba040f17cdcd2a5960361971eaff79684c48847a",
-    "paths_digest": "sha256:4a52771035fe6ef25f0155b11c052a661879b1cd1af4bbe5cd337118a9dd29dd",
-    "initial_review_tree": "5bc2f4bdd086d064bc3ada255f16c631facc6839",
-    "final_candidate_tree": "5bc2f4bdd086d064bc3ada255f16c631facc6839",
-    "fix_delta_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
-    "policy_hash": "sha256:9197a98b8c5ceac34da4e8d213908d68023f215e891ecb85dfe9fb0b355f9482",
-    "ledger_hash": "",
-    "ledger_findings_hash": "",
-    "evidence_hash": "",
-    "judge_proofs": [],
-    "counters": {
-      "full_reviews": 1,
-      "refuter_batches": 0,
-      "fix_batches": 0,
-      "scoped_fix_validations": 0,
-      "final_verifications": 0,
-      "fix_rounds": 0,
-      "scoped_rejudgments": 0,
-      "judge_executions": 0
-    },
-    "findings": [],
-    "classifications": {},
-    "outcomes": {},
-    "fix_finding_ids": [],
-    "pending_refuter_ids": [],
-    "fix_caused_findings": [],
-    "follow_ups": []
-  },
-  "phaseInstructions": {
-    "apply": [
-      "Change: native-review-authority-parity",
-      "State: all_done",
-      "Read proposal, specs, design, and tasks before editing.",
-      "Implement only unchecked tasks and update tasks.md checkboxes as work completes."
-    ],
-    "verify": [
-      "Change: native-review-authority-parity",
-      "State: all_done",
-      "Verify implementation against proposal, specs, design, and task completion.",
-      "Run final verification only after every task is complete; apply-progress never makes final verification ready."
-    ],
-    "remediate": [
-      "Change: native-review-authority-parity",
-      "Remediation is allowed only when the persisted review transaction has remaining mode-specific budget.",
-      "Bind focused tests, runtime harness evidence, and rollback evidence to the exact failed evidence revision.",
-      "A bare remediation envelope or stale failed revision never completes remediation."
-    ],
-    "archive": [
-      "Change: native-review-authority-parity",
-      "State: ready",
-      "Archive only when verify-report.md exists and every task checkbox is complete."
-    ]
-  },
-  "nextRecommended": "archive",
-  "blockedReasons": []
-}

package/tests/git-commit-transaction.test.ts

@@ -1,302 +0,0 @@
-import assert from "node:assert/strict";
-import { execFileSync } from "node:child_process";
-import { chmodSync, existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import test from "node:test";
-import { setTimeout as delay } from "node:timers/promises";
-import {
-	COMMIT_TRANSACTION_STATE,
-	assertNoUnresolvedCommitTransaction,
-	inspectCommitTransaction,
-	prepareCommitTransactionInvocation,
-	reconcileCommitTransaction,
-	runGitCommitTransaction,
-	verifyCommitTransactionResult,
-} from "../lib/git-commit-transaction.ts";
-import type { NativeReviewCli, NativeValidateResult } from "../lib/native-review-cli.ts";
-
-function git(cwd: string, ...arguments_: string[]): string {
-	return execFileSync("git", arguments_, { cwd, encoding: "utf8" }).trim();
-}
-
-function repository(t: test.TestContext): string {
-	const cwd = mkdtempSync(join(tmpdir(), "gentle-pi-commit-transaction-"));
-	t.after(() => rmSync(cwd, { recursive: true, force: true }));
-	git(cwd, "init", "-b", "main");
-	git(cwd, "config", "user.name", "Commit Transaction Test");
-	git(cwd, "config", "user.email", "commit-transaction@example.invalid");
-	writeFileSync(join(cwd, "tracked.txt"), "base\n");
-	git(cwd, "add", "tracked.txt");
-	git(cwd, "commit", "-m", "base");
-	return cwd;
-}
-
-function installHook(cwd: string, name: string, body: string): void {
-	const hooks = git(cwd, "rev-parse", "--path-format=absolute", "--git-path", "hooks");
-	mkdirSync(hooks, { recursive: true });
-	const path = join(hooks, name);
-	writeFileSync(path, `#!/bin/sh\nset -eu\n${body}\n`);
-	chmodSync(path, 0o700);
-}
-
-function stage(cwd: string, content = "candidate\n"): string {
-	writeFileSync(join(cwd, "tracked.txt"), content);
-	git(cwd, "add", "tracked.txt");
-	return git(cwd, "write-tree");
-}
-
-function invocation(cwd: string, lineageId: string, arguments_: readonly string[] = ["-m", "candidate"]) {
-	const intendedTree = git(cwd, "write-tree");
-	const command = `git commit ${arguments_.map((value) => JSON.stringify(value)).join(" ")}`;
-	return prepareCommitTransactionInvocation({
-		command,
-		cwd,
-		arguments: arguments_,
-		authorization: {
-			lineageId,
-			storeRevision: "sha256:" + "a".repeat(64),
-			fingerprint: "sha256:" + "b".repeat(64),
-			intendedTree,
-		},
-	});
-}
-
-function native(cwd: string, lineageId: string, result: NativeValidateResult["result"] = "allow"): NativeReviewCli {
-	return {
-		async validate(request) {
-			const tree = git(cwd, "write-tree");
-			return {
-				allowed: result === "allow",
-				result,
-				action: result === "allow" ? "continue" : result === "scope-changed" ? "create-new-lineage" : "explicit-maintainer-action",
-				reason: result === "allow" ? "receipt allows exact tree" : "post-hook tree differs from receipt",
-				gateContext: {
-					lineageId,
-					storeRevision: "sha256:" + "c".repeat(64),
-					raw: { gate: request.gate, lineage_id: lineageId, candidate_tree: tree },
-				},
-			};
-		},
-	} as NativeReviewCli;
-}
-
-test("a non-mutating pre-commit hook runs once and HEAD proves the native-authorized tree", async (t) => {
-	const cwd = repository(t);
-	stage(cwd);
-	const count = join(cwd, ".git", "hook-count");
-	installHook(cwd, "pre-commit", `printf '1\\n' >> ${JSON.stringify(count)}`);
-	const before = git(cwd, "rev-parse", "HEAD");
-	const result = await runGitCommitTransaction(invocation(cwd, "non-mutating"), { nativeReviewCli: native(cwd, "non-mutating") });
-	assert.notEqual(result.head, before);
-	assert.equal(result.tree, git(cwd, "rev-parse", "HEAD^{tree}"));
-	assert.equal(readFileSync(count, "utf8"), "1\n");
-	assert.deepEqual(inspectCommitTransaction(cwd), { status: "clean" });
-});
-
-test("a mutating hook creates no commit until the post-hook tree is reviewed, then exact retry skips the hook", async (t) => {
-	const cwd = repository(t);
-	stage(cwd, "unformatted\n");
-	const count = join(cwd, ".git", "hook-count");
-	installHook(cwd, "pre-commit", `printf 'formatted\\n' > tracked.txt\ngit add tracked.txt\nprintf '1\\n' >> ${JSON.stringify(count)}`);
-	const before = git(cwd, "rev-parse", "HEAD");
-	await assert.rejects(
-		runGitCommitTransaction(invocation(cwd, "before-format"), { nativeReviewCli: native(cwd, "before-format", "scope-changed") }),
-		/not the authorized tree/,
-	);
-	assert.equal(git(cwd, "rev-parse", "HEAD"), before);
-	assert.equal(inspectCommitTransaction(cwd).record?.state, COMMIT_TRANSACTION_STATE.AWAITING_REVIEW);
-	assert.throws(() => assertNoUnresolvedCommitTransaction(cwd), /publication is blocked/);
-	const result = await runGitCommitTransaction(invocation(cwd, "after-format"), { nativeReviewCli: native(cwd, "after-format") });
-	assert.equal(result.tree, git(cwd, "rev-parse", "HEAD^{tree}"));
-	assert.equal(readFileSync(count, "utf8"), "1\n");
-});
-
-test("a failing pre-commit hook creates no commit and leaves explicit recovery state", async (t) => {
-	const cwd = repository(t);
-	stage(cwd);
-	installHook(cwd, "pre-commit", "exit 23");
-	const before = git(cwd, "rev-parse", "HEAD");
-	await assert.rejects(runGitCommitTransaction(invocation(cwd, "hook-failure"), { nativeReviewCli: native(cwd, "hook-failure") }), /hook failed/);
-	assert.equal(git(cwd, "rev-parse", "HEAD"), before);
-	assert.equal(inspectCommitTransaction(cwd).record?.state, COMMIT_TRANSACTION_STATE.HOOK_FAILED);
-	assert.match(inspectCommitTransaction(cwd).record?.error ?? "", /exit 23/);
-});
-
-test("a native scope-changed denial on the unmutated authorized tree awaits explicit review", async (t) => {
-	const cwd = repository(t);
-	const authorizedTree = stage(cwd);
-	const before = git(cwd, "rev-parse", "HEAD");
-	let validations = 0;
-	const denying = native(cwd, "scope-changed-denial", "scope-changed");
-	const counting = {
-		async validate(request) {
-			validations += 1;
-			return denying.validate(request);
-		},
-	} as NativeReviewCli;
-	await assert.rejects(
-		runGitCommitTransaction(invocation(cwd, "scope-changed-denial"), { nativeReviewCli: counting }),
-		/native pre-commit validation denied the post-hook tree: scope-changed/,
-	);
-	assert.equal(validations, 1, "the unmutated tree must pass the mutation guard and reach native validation");
-	assert.equal(git(cwd, "rev-parse", "HEAD"), before);
-	assert.equal(git(cwd, "write-tree"), authorizedTree, "the denied index stays exactly as authorized");
-	const inspection = inspectCommitTransaction(cwd);
-	assert.equal(inspection.record?.state, COMMIT_TRANSACTION_STATE.AWAITING_REVIEW);
-	assert.equal(inspection.record?.error, "post-hook tree differs from receipt");
-	assert.equal(typeof inspection.record?.native_result, "object", "the native denial must be recorded on the transaction");
-});
-
-test("a non-scope-changed native denial fails validation and requires explicit recovery", async (t) => {
-	const cwd = repository(t);
-	stage(cwd);
-	const before = git(cwd, "rev-parse", "HEAD");
-	await assert.rejects(
-		runGitCommitTransaction(invocation(cwd, "invalidated-denial"), { nativeReviewCli: native(cwd, "invalidated-denial", "invalidated") }),
-		/native pre-commit validation denied the post-hook tree: invalidated/,
-	);
-	assert.equal(git(cwd, "rev-parse", "HEAD"), before);
-	assert.equal(inspectCommitTransaction(cwd).record?.state, COMMIT_TRANSACTION_STATE.VALIDATION_FAILED);
-	await assert.rejects(
-		runGitCommitTransaction(invocation(cwd, "invalidated-denial"), { nativeReviewCli: native(cwd, "invalidated-denial") }),
-		/requires explicit recovery from state validation-failed/,
-	);
-});
-
-test("a mutating pre-commit hook fails closed even when native validation would allow the post-hook tree", async (t) => {
-	const cwd = repository(t);
-	const authorizedTree = stage(cwd, "unformatted\n");
-	const count = join(cwd, ".git", "hook-count");
-	installHook(cwd, "pre-commit", `printf 'formatted\\n' > tracked.txt\ngit add tracked.txt\nprintf '1\\n' >> ${JSON.stringify(count)}`);
-	const before = git(cwd, "rev-parse", "HEAD");
-	// The fake native CLI allows whatever tree the index holds, so the only
-	// defense against committing the hook-mutated tree is the transaction's
-	// own binding to the invocation's authorized tree.
-	await assert.rejects(
-		runGitCommitTransaction(invocation(cwd, "permissive-native"), { nativeReviewCli: native(cwd, "permissive-native") }),
-		/pre-commit hook mutated the staged candidate/,
-	);
-	assert.equal(git(cwd, "rev-parse", "HEAD"), before, "no commit object may be created from the mutated index");
-	assert.equal(readFileSync(count, "utf8"), "1\n", "the mutating hook must have run exactly once");
-	assert.notEqual(git(cwd, "write-tree"), authorizedTree, "the mutated index is preserved for inspection");
-	const inspection = inspectCommitTransaction(cwd);
-	assert.equal(inspection.record?.state, COMMIT_TRANSACTION_STATE.AWAITING_REVIEW);
-	assert.match(inspection.record?.error ?? "", /normalize sources and re-run review explicitly, or make the hook convergent/);
-	assert.throws(() => assertNoUnresolvedCommitTransaction(cwd), /publication is blocked/);
-});
-
-test("a convergent pre-commit hook runs exactly once and commits the exact authorized tree", async (t) => {
-	const cwd = repository(t);
-	const authorizedTree = stage(cwd, "formatted\n");
-	const count = join(cwd, ".git", "hook-count");
-	installHook(cwd, "pre-commit", `printf 'formatted\\n' > tracked.txt\ngit add tracked.txt\nprintf '1\\n' >> ${JSON.stringify(count)}`);
-	const before = git(cwd, "rev-parse", "HEAD");
-	const result = await runGitCommitTransaction(invocation(cwd, "convergent"), { nativeReviewCli: native(cwd, "convergent") });
-	assert.notEqual(result.head, before);
-	assert.equal(result.tree, authorizedTree);
-	assert.equal(git(cwd, "rev-parse", "HEAD^{tree}"), authorizedTree);
-	assert.equal(readFileSync(count, "utf8"), "1\n", "the convergent hook must have run exactly once");
-	assert.deepEqual(inspectCommitTransaction(cwd), { status: "clean" });
-});
-
-test("a repository with no hooks commits the authorized tree unchanged", async (t) => {
-	const cwd = repository(t);
-	const authorizedTree = stage(cwd);
-	const before = git(cwd, "rev-parse", "HEAD");
-	const result = await runGitCommitTransaction(invocation(cwd, "no-hooks"), { nativeReviewCli: native(cwd, "no-hooks") });
-	assert.notEqual(result.head, before);
-	assert.equal(result.tree, authorizedTree);
-	assert.equal(git(cwd, "rev-parse", "HEAD^{tree}"), authorizedTree);
-	assert.deepEqual(inspectCommitTransaction(cwd), { status: "clean" });
-});
-
-test("the Git-created HEAD tree is proven equal to the authorized tree on success", async (t) => {
-	const cwd = repository(t);
-	const authorizedTree = stage(cwd);
-	const result = await runGitCommitTransaction(invocation(cwd, "head-proof"), { nativeReviewCli: native(cwd, "head-proof") });
-	assert.equal(result.tree, authorizedTree);
-	assert.equal(git(cwd, "rev-parse", "HEAD^{tree}"), authorizedTree);
-	const verified = verifyCommitTransactionResult(cwd, result.transactionId);
-	assert.deepEqual(verified, { transactionId: result.transactionId, status: "committed", head: result.head, tree: authorizedTree });
-});
-
-test("message hooks cannot change the index after native authorization", async (t) => {
-	const cwd = repository(t);
-	stage(cwd);
-	installHook(cwd, "prepare-commit-msg", "printf 'late mutation\\n' > tracked.txt\ngit add tracked.txt");
-	const before = git(cwd, "rev-parse", "HEAD");
-	await assert.rejects(runGitCommitTransaction(invocation(cwd, "late-mutation"), { nativeReviewCli: native(cwd, "late-mutation") }), /Git commit failed/);
-	assert.equal(git(cwd, "rev-parse", "HEAD"), before);
-	assert.equal(inspectCommitTransaction(cwd).record?.state, COMMIT_TRANSACTION_STATE.COMMIT_FAILED);
-});
-
-test("amend and post-commit crash reconciliation preserve the exact authorized tree", async (t) => {
-	const cwd = repository(t);
-	const authorizedTree = stage(cwd, "amended\n");
-	const amended = await runGitCommitTransaction(invocation(cwd, "amend", ["--amend", "--no-edit"]), { nativeReviewCli: native(cwd, "amend") });
-	assert.equal(amended.tree, authorizedTree);
-	stage(cwd, "after-crash\n");
-	await assert.rejects(
-		runGitCommitTransaction(invocation(cwd, "crash"), { nativeReviewCli: native(cwd, "crash"), failpoint: "after-commit-before-proof" }),
-		/test interruption/,
-	);
-	assert.equal(inspectCommitTransaction(cwd).record?.state, COMMIT_TRANSACTION_STATE.COMMIT_RUNNING);
-	assert.deepEqual(reconcileCommitTransaction(cwd), { status: "clean" });
-	assert.deepEqual(inspectCommitTransaction(cwd), { status: "clean" });
-});
-
-test("a post-commit hook cannot replace the exact commit created by Git", async (t) => {
-	const cwd = repository(t);
-	stage(cwd, "intermediate\n");
-	git(cwd, "commit", "-m", "intermediate");
-	stage(cwd);
-	installHook(cwd, "post-commit", [
-		"original=$(git rev-parse HEAD)",
-		"tree=$(git rev-parse HEAD^{tree})",
-		"alternate_parent=$(git rev-parse HEAD~2)",
-		"replacement=$(printf 'replacement\\n' | git commit-tree \"$tree\" -p \"$alternate_parent\")",
-		"git update-ref HEAD \"$replacement\" \"$original\"",
-	].join("\n"));
-	await assert.rejects(
-		runGitCommitTransaction(invocation(cwd, "post-commit-replacement"), { nativeReviewCli: native(cwd, "post-commit-replacement") }),
-		/different commit|identity changed/,
-	);
-	assert.equal(inspectCommitTransaction(cwd).record?.state, COMMIT_TRANSACTION_STATE.INCIDENT);
-});
-
-test("cancellation cannot strand a commit after HEAD advances", async (t) => {
-	const cwd = repository(t);
-	stage(cwd);
-	// Deterministic cancellation ordering (issue #178): a wall-clock
-	// AbortSignal.timeout raced Git's own progress on loaded CI runners and
-	// could abort before HEAD advanced. Git only runs the post-commit hook
-	// after the commit object exists and HEAD has advanced, so the hook
-	// reports that it started and then blocks until the test releases it.
-	// Aborting between those two events guarantees the cancellation lands
-	// while the commit process is still running and strictly after HEAD
-	// advanced, on every runner.
-	const started = join(cwd, ".git", "post-commit-started");
-	const release = join(cwd, ".git", "post-commit-release");
-	installHook(cwd, "post-commit", [
-		`printf '1\\n' > ${JSON.stringify(started)}`,
-		`while [ ! -e ${JSON.stringify(release)} ]; do sleep 0.02; done`,
-	].join("\n"));
-	const before = git(cwd, "rev-parse", "HEAD");
-	const abort = new AbortController();
-	const pending = runGitCommitTransaction(invocation(cwd, "commit-cancellation"), {
-		nativeReviewCli: native(cwd, "commit-cancellation"),
-		signal: abort.signal,
-	});
-	let settled = false;
-	void pending.then(() => { settled = true; }, () => { settled = true; });
-	while (!existsSync(started) && !settled) await delay(10);
-	assert.equal(settled, false, "commit transaction finished before the post-commit hook confirmed HEAD advanced");
-	abort.abort();
-	writeFileSync(release, "release\n");
-	const result = await pending;
-	assert.equal(abort.signal.aborted, true);
-	assert.notEqual(result.head, before);
-	assert.equal(result.status, "committed");
-	assert.deepEqual(inspectCommitTransaction(cwd), { status: "clean" });
-});