Skip to content

Commit 4f745b7

Browse files
committed
fix(release): keep waived Z.AI probe out of broad shard
Adapt the Z.AI shard portion of upstream c4b2183 (#109272). Exclude the waived API Platform file from the broad agents shard while retaining the dedicated Coding Plan proof. (cherry picked from commit c4b2183)
1 parent fe3014f commit 4f745b7

4 files changed

Lines changed: 13 additions & 10 deletions

File tree

.github/workflows/openclaw-live-and-e2e-checks-reusable.yml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2702,9 +2702,7 @@ jobs:
27022702
include:
27032703
- suite_id: native-live-src-agents
27042704
label: Native live agents
2705-
# Keep the dedicated Coding Plan suite below while temporarily omitting
2706-
# API Platform calls from the broad agents shard.
2707-
command: ZAI_API_KEY= Z_AI_API_KEY= node .release-harness/scripts/test-live-shard.mjs native-live-src-agents
2705+
command: node .release-harness/scripts/test-live-shard.mjs native-live-src-agents
27082706
timeout_minutes: 60
27092707
profile_env_only: false
27102708
profiles: stable full

scripts/test-live-shard.mjs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ const OPTIONAL_LIVE_SHARD_FILE_ENVS = new Map([
3333
["test/image-generation.infer-cli.live.test.ts", ["OPENCLAW_LIVE_INFER_CLI_TEST"]],
3434
]);
3535
const SKIPPED_ASSERTION_STATUSES = new Set(["disabled", "pending", "skipped", "todo"]);
36+
const ZAI_LIVE_TEST_FILE = "src/agents/zai.live.test.ts";
3637

3738
/** Live-test shards included in release validation. */
3839
export const RELEASE_LIVE_TEST_SHARDS = Object.freeze([
@@ -246,9 +247,13 @@ function isMoonshotLiveTest(file) {
246247
export function selectLiveShardFiles(shard, files = collectAllLiveTestFiles()) {
247248
switch (shard) {
248249
case "native-live-src-agents":
249-
return files.filter((file) => file.startsWith("src/agents/") || file.startsWith("src/llm/"));
250+
return files.filter(
251+
(file) =>
252+
file !== ZAI_LIVE_TEST_FILE &&
253+
(file.startsWith("src/agents/") || file.startsWith("src/llm/")),
254+
);
250255
case "native-live-src-agents-zai-coding":
251-
return files.filter((file) => file === "src/agents/zai.live.test.ts");
256+
return files.filter((file) => file === ZAI_LIVE_TEST_FILE);
252257
case "native-live-src-gateway":
253258
return files.filter(
254259
(file) => file.startsWith("src/gateway/") || file.startsWith("src/crestodian/"),

test/scripts/package-acceptance-workflow.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1010,7 +1010,7 @@ describe("package artifact reuse", () => {
10101010
expect(workflow).toContain("suite_id: native-live-src-agents");
10111011
expect(workflow).toContain("Checkout trusted live shard harness");
10121012
expect(workflow).toContain(
1013-
"command: ZAI_API_KEY= Z_AI_API_KEY= node .release-harness/scripts/test-live-shard.mjs native-live-src-agents",
1013+
"command: node .release-harness/scripts/test-live-shard.mjs native-live-src-agents",
10141014
);
10151015
expect(workflow).toContain("suite_id: native-live-src-agents-zai-coding");
10161016
expect(workflow).toContain(

test/scripts/test-live-shard.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,10 +47,7 @@ describe("scripts/test-live-shard", () => {
4747

4848
expect(allFiles.length).toBeGreaterThan(0);
4949
expect([...new Set(selectedFiles)].toSorted((a, b) => a.localeCompare(b))).toEqual(allFiles);
50-
expect(duplicateFiles).toEqual([
51-
"src/agents/zai.live.test.ts",
52-
"extensions/music-generation-providers.live.test.ts",
53-
]);
50+
expect(duplicateFiles).toEqual(["extensions/music-generation-providers.live.test.ts"]);
5451
expect(musicProviderFanout).toEqual([
5552
"native-live-extensions-media-music-google",
5653
"native-live-extensions-media-music-minimax",
@@ -87,6 +84,9 @@ describe("scripts/test-live-shard", () => {
8784
expect(selectLiveShardFiles("native-live-src-agents", allFiles)).toContain(
8885
"src/llm/providers/stream-wrappers/anthropic-family-tool-payload-compat.live.test.ts",
8986
);
87+
expect(selectLiveShardFiles("native-live-src-agents", allFiles)).not.toContain(
88+
"src/agents/zai.live.test.ts",
89+
);
9090
expect(selectLiveShardFiles("native-live-src-agents-zai-coding", allFiles)).toEqual([
9191
"src/agents/zai.live.test.ts",
9292
]);

0 commit comments

Comments
 (0)