fix: stabilize alpha validation gates

(cherry picked from commit 9fcbf360d7)
This commit is contained in:
Tideclaw
2026-06-03 03:17:23 +00:00
parent 85b4314ccc
commit 2154d239e9
2 changed files with 28 additions and 1 deletions

View File

@@ -150,7 +150,8 @@ function resolveGroupTargets(group, exactExcludePaths, forceFileTargets = false)
}
function resolveGroupTargetChunkSize(group, env) {
const override = parsePositiveInt(env[TARGET_CHUNK_SIZE_ENV_KEY]);
const rawOverride = env[TARGET_CHUNK_SIZE_ENV_KEY]?.trim();
const override = rawOverride ? parsePositiveInt(rawOverride, TARGET_CHUNK_SIZE_ENV_KEY) : null;
if (override !== null) {
return override;
}

View File

@@ -741,6 +741,32 @@ describe("scripts/test-extension.mjs", () => {
expect(runParams.targets).toContain("extensions/codex/src/app-server/client.test.ts");
});
it("runs extension batch groups without an optional target chunk size override", async () => {
const runGroup = vi.fn<() => Promise<number>>().mockResolvedValue(0);
const result = await runExtensionBatchPlan(
{
extensionCount: 1,
extensionIds: ["firecrawl"],
estimatedCost: 1,
hasTests: true,
planGroups: [
{
config: "test/vitest/vitest.extensions.config.ts",
estimatedCost: 1,
extensionIds: ["firecrawl"],
roots: [bundledPluginRoot("firecrawl")],
testFileCount: 1,
},
],
testFileCount: 1,
},
{ runGroup },
);
expect(result).toBe(0);
expect(runGroup).toHaveBeenCalledOnce();
});
it("chunks large extension batch groups into separate Vitest processes", async () => {
const runGroup = vi.fn<() => Promise<number>>().mockResolvedValue(0);
const result = await runExtensionBatchPlan(