@@ -96,13 +96,6 @@ describe("evaluatePullRequestWatch", () => {
     assert.deepEqual(evaluatePullRequestWatch(watch(), plain, noRemarks).changes, []);
   });
 
-  it("reports passed when a check was skipped and the rest succeeded", () => {
-    const green = detail({ checks: [check("checks", "success"), check("deploy", "skipped")] });
-    assert.deepEqual(evaluatePullRequestWatch(watch(), green, noRemarks).changes, [
-      { kind: "checks-passed", count: 2, required: false },
-    ]);
-  });
-
   it("reports passed again when a required check shows up already passed", () => {
     const required = (name: string, status: PullRequestCheck["status"]) => ({
       ...check(name, status),
 
@@ -164,6 +164,67 @@ it.effect("reads the newest Forgejo checks without repository or viewer requests
   );
 });
 
+it.effect("reads Forgejo jobs that did not run as finished checks", () =>
+  Effect.gen(function* () {
+    const provider = yield* ForgejoPullRequestProvider.make;
+    const read = provider.getChangeRequestChecks;
+    if (read === undefined) return yield* Effect.die("checks read missing");
+    const result = yield* read({
+      cwd: "/repo",
+      repository: "acme/web",
+      host: "forgejo.test",
+      number: 1,
+    });
+    assert.deepStrictEqual(
+      result.checks.map((check) => [check.name, check.status]),
+      [
+        ["deploy", "skipped"],
+        ["audit", "neutral"],
+        ["docs", "pending"],
+      ],
+    );
+  }).pipe(
+    Effect.provide(
+      Layer.mock(ForgejoCli.ForgejoCli)({
+        api: (input) => {
+          const status = (context: string, state: string) => ({
+            context,
+            status: state,
+            description: null,
+            target_url: null,
+            updated_at: "2026-10-11T00:00:00Z",
+          });
+          const body = input.path.endsWith("pulls/1")
+            ? {
+                number: 1,
+                title: "Checks",
+                body: "",
+                html_url: "https://forgejo.test/acme/web/pulls/1",
+                user: null,
+                state: "open",
+                merged: false,
+                head: { ref: "feature", sha: "head", repo: null },
+                base: { ref: "main", sha: "base", repo: null },
+                created_at: "2026-10-11T00:00:00Z",
+                updated_at: "2026-10-11T00:00:00Z",
+                closed_at: null,
+                merged_at: null,
+                labels: [],
+              }
+            : input.path.endsWith("page=1")
+              ? [
+                  status("deploy", "skipped"),
+                  status("audit", "warning"),
+                  status("docs", "future-state"),
+                ]
+              : [];
+          return encodeJsonEffect(body).pipe(Effect.orDie, Effect.map(processOutput));
+        },
+      }),
+    ),
+  ),
+);
+
 it.effect("opens Forgejo Actions check links on the pull request's server", () =>
   Effect.gen(function* () {
     const provider = yield* ForgejoPullRequestProvider.make;
 
@@ -213,8 +213,9 @@ export function forgejoCommit(commit: typeof ForgejoCommit.Type): PullRequestCom
   };
 }
 
-// Forgejo's commit status states; Actions reports a job that did not run as `skipped` and a
-// cancelled job as `failure`. A skipped job is done, so it must not hold back "checks passed".
+// Forgejo Actions reports a job that did not run as `skipped` and a cancelled one as `failure`,
+// and `warning` is finished without being a pass. Only `pending` and states we do not know may
+// hold a watch open.
 function forgejoCheckStatus(state: string): PullRequestCheckStatus {
   switch (state) {
     case "success":