fix(ci): add unit_tests to coverage job needs to prevent misleading parallel results (#10884)

.forgejo/workflows/ci.yml

@@ -338,7 +338,9 @@ jobs:
         runs-on: docker
         container:
             image: ${{vars.docker_prefix}}python:3.13-slim
-        needs: [lint, typecheck, security, quality]
+        # unit_tests is included so coverage only runs after tests pass,
+        # preventing misleading results when tests are still in-flight or failing.
+        needs: [lint, typecheck, security, quality, unit_tests]
         steps:
             - name: Install system dependencies (nodejs for checkout, git for merge tests)
               run: |

CHANGELOG.md

@@ -7,6 +7,13 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ### Changed
 
+- **CI coverage job now waits for unit_tests** (#10714): Added `unit_tests` to the
+  `needs` list of the `coverage` job in `ci.yml`. Previously the coverage job ran
+  in parallel with unit tests, which could produce misleading pass results when
+  tests were still in-flight or had already failed. Coverage now only starts after
+  unit tests succeed, eliminating redundant parallel test execution and ensuring
+  coverage results are always meaningful.
+
 - **Diagnostics spec examples expanded to all 9 providers** (#5320): Updated the
   `agents diagnostics` command examples in the specification to show all 9 supported
   providers (OpenAI, Anthropic, Google, Gemini, Azure, OpenRouter, Cohere, Groq,