ci: split qa smoke into dedicated check

RomneyDa · RomneyDa · commit 0455bddb70a0 · 2026-06-24T09:38:30.000-07:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -284,6 +284,7 @@ jobs:
           if (runNodeFull) {
             checksFastCoreTasks.push(
               { check_name: "checks-fast-bundled-protocol", runtime: "node", task: "bundled-protocol" },
+              { check_name: "QA Smoke CI", runtime: "node", task: "qa-smoke-ci" },
               { check_name: "checks-fast-bun-launcher", runtime: "bun", task: "bun-launcher" },
             );
           } else {
@@ -920,6 +921,8 @@ jobs:
             bundled-protocol)
               pnpm test:bundled
               pnpm protocol:check
+              ;;
+            qa-smoke-ci)
               output_dir=".artifacts/qa-e2e/smoke-ci-profile"
               export OPENCLAW_BUILD_PRIVATE_QA=1
               export OPENCLAW_ENABLE_PRIVATE_QA_CLI=1
@@ -956,7 +959,7 @@ jobs:
           esac
 
       - name: Upload QA smoke profile evidence
-        if: always() && matrix.task == 'bundled-protocol'
+        if: always() && matrix.task == 'qa-smoke-ci'
         uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
         with:
           name: qa-smoke-profile-${{ github.run_id }}-${{ github.run_attempt }}
diff --git a/docs/ci.md b/docs/ci.md
@@ -30,7 +30,7 @@ or an explicit manual dispatch.
 | `security-fast`                    | Private key detection, changed-workflow audit via `zizmor`, and production lockfile audit                 | Always on non-draft pushes and PRs                  |
 | `check-dependencies`               | Production Knip dependency-only pass plus the unused-file allowlist guard                                 | Node-relevant changes                               |
 | `build-artifacts`                  | Build `dist/`, Control UI, built-CLI smoke checks, embedded built-artifact checks, and reusable artifacts | Node-relevant changes                               |
-| `checks-fast-core`                 | Fast Linux correctness lanes such as bundled, protocol, and CI-routing checks                             | Node-relevant changes                               |
+| `checks-fast-core`                 | Fast Linux correctness lanes such as bundled, protocol, QA Smoke CI, and CI-routing checks                | Node-relevant changes                               |
 | `checks-fast-contracts-plugins-*`  | Two sharded plugin contract checks                                                                        | Node-relevant changes                               |
 | `checks-fast-contracts-channels-*` | Two sharded channel contract checks                                                                       | Node-relevant changes                               |
 | `checks-node-core-*`               | Core Node test shards, excluding channel, bundled, contract, and extension lanes                          | Node-relevant changes                               |
diff --git a/test/scripts/ci-workflow-guards.test.ts b/test/scripts/ci-workflow-guards.test.ts
@@ -822,8 +822,12 @@ describe("ci workflow guards", () => {
       expect(ciWorkflowText).not.toContain(`"${categoryId}"`);
     }
     expect(runStep.run).toContain("bundled-protocol)");
+    expect(runStep.run).toContain("qa-smoke-ci)");
     expect(runStep.run).toContain("contracts-plugins-ci-routing)");
     expect(runStep.run).toContain("ci-routing)");
+    expect(ciWorkflowText).toContain(
+      '{ check_name: "QA Smoke CI", runtime: "node", task: "qa-smoke-ci" }',
+    );
     expect(runStep.run).toContain("--qa-profile smoke-ci");
     expect(runStep.run).toContain("--concurrency 8");
     expect(runStep.run).not.toContain("--category");
@@ -832,7 +836,10 @@ describe("ci workflow guards", () => {
     expect(runStep.run).toContain('exit "$qa_exit_code"');
     expect(runStep.run).toContain("scripts/build-all.mjs qaRuntime");
     expect(runStep.run).not.toContain("OPENAI_API_KEY");
-    expect(uploadStep.if).toBe("always() && matrix.task == 'bundled-protocol'");
+    expect(runStep.run).toMatch(
+      /bundled-protocol\)\s+pnpm test:bundled\s+pnpm protocol:check\s+;;\s+qa-smoke-ci\)/,
+    );
+    expect(uploadStep.if).toBe("always() && matrix.task == 'qa-smoke-ci'");
     expect(uploadStep.with).toMatchObject({
       path: ".artifacts/qa-e2e/smoke-ci-profile/",
       "if-no-files-found": "warn",
