From 2b12eff87c146aff9e4586b0d180b96829d1b34c Mon Sep 17 00:00:00 2001
From: Jonathan Liebig <jonathan.liebig@gmail.com>
Date: Thu, 29 Oct 2026 13:21:22 +0100
Subject: [PATCH] ci: remove redundant qualification result job

---
 .github/workflows/ci.yml | 22 +++++-----------------
 README.md                |  2 +-
 2 files changed, 6 insertions(+), 18 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index c45b737..3b39bb2 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -101,23 +101,11 @@ jobs:
       - if: ${{ always() && github.event_name == 'pull_request' && (github.event.pull_request.head.repo.full_name != github.repository || github.actor == 'dependabot[bot]') }}
         run: echo 'Owned artifact uploads are unavailable for fork and Dependabot checks.' >> "$GITHUB_STEP_SUMMARY"
 
-  result:
-    name: Qualification result
-    if: ${{ always() }}
-    needs: correctness
-    runs-on: [self-hosted, Linux, X64, nextide-ci-docker]
-    timeout-minutes: 5
-    steps:
-      - name: Require selected groups
+      - name: Qualification summary
+        if: ${{ always() }}
         env:
           PROFILE: ${{ inputs.profile || 'full' }}
-          SOURCE_SHA: ${{ needs.correctness.outputs.sha || inputs.source_sha || github.sha }}
-          CORRECTNESS: ${{ needs.correctness.result }}
+          SOURCE_SHA: ${{ steps.source.outputs.sha || inputs.source_sha || github.sha }}
+          CORRECTNESS: ${{ job.status }}
         run: |
-          node <<'NODE'
-          const { PROFILE, SOURCE_SHA, CORRECTNESS } = process.env;
-          console.log(`Qualification ${PROFILE}: ${SOURCE_SHA}; Correctness=${CORRECTNESS}`);
-          if (!["full", "deploy"].includes(PROFILE) || CORRECTNESS !== "success") {
-            process.exit(1);
-          }
-          NODE
+          printf 'Qualification %s: %s; Correctness=%s\n' "$PROFILE" "$SOURCE_SHA" "$CORRECTNESS" | tee -a "$GITHUB_STEP_SUMMARY"
diff --git a/README.md b/README.md
index 3c7ab5e..c263211 100644
--- a/README.md
+++ b/README.md
@@ -62,7 +62,7 @@ Pull requests, main pushes, and the weekly **Qualification** workflow run full q
 `full` or `deploy`; deploy deliberately skips **Quality**. Both profiles share one
 Node installation in **Correctness / Package and browser**. Full runs execute
 Quality first; its full lint also covers correctness lint and the watchlist.
-**Qualification result** requires the selected checks to succeed and reports the validated commit. The
+That job requires the selected checks to succeed and reports the profile, validated commit, and status. The
 package has no service deployment or database integration lane; its integration
 surface is the packed consumer and playground browser suite.