fix(release): apply 2026.9.8 live shard waivers (#163621)

This commit is contained in:
Dallin Romney 2026-10-02 09:17:01 -07:00 • committed by GitHub
parent b2c302324f
commit bfeca6756c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 48 additions and 1 deletions

View file

@ -317,6 +317,39 @@ function isMoonshotLiveTest(file: string) {
return file.startsWith("extensions/moonshot/");
}
// The frozen 2026.9.8 candidate retains three intentionally skipped single-case
// live files. The trusted tooling checkout owns shard selection, so omit those
// candidate files here rather than weakening the per-file passing-assertion guard.
const RELEASE_2026_9_8_WAIVED_LIVE_FILES = new Set([
"src/gateway/gateway-progress-refresh.live.test.ts",
"src/agents/embedded-agent-runner.responses-output-limit.live.test.ts",
"test/gateway-subagent-restart.live.test.ts",
]);
const RELEASE_WAIVED_LIVE_FILES = new Map<string, ReadonlySet<string>>([
["2026.9.8", RELEASE_2026_9_8_WAIVED_LIVE_FILES],
]);
export function withoutReleaseWaivedLiveFiles(
files: string[],
candidateVersion: string | undefined,
) {
const waived = candidateVersion ? RELEASE_WAIVED_LIVE_FILES.get(candidateVersion) : undefined;
return waived ? files.filter((file) => !waived.has(file)) : files;
}
function readCandidateVersion(repoRoot = process.cwd()) {
try {
const manifest: unknown = JSON.parse(
fs.readFileSync(path.join(repoRoot, "package.json"), "utf8"),
);
return isUnknownRecord(manifest) && typeof manifest.version === "string"
? manifest.version
: undefined;
} catch {
return undefined;
}
}
/**
* Selects the live test files belonging to one shard name.
*/
@ -781,7 +814,7 @@ if (process.argv[1] && path.resolve(process.argv[1]) === fileURLToPath(import.me
let files;
try {
files = selectLiveShardFiles(shard);
files = withoutReleaseWaivedLiveFiles(selectLiveShardFiles(shard), readCandidateVersion());
} catch (error) {
console.error(error instanceof Error ? error.message : String(error));
usage();

View file

@ -28,6 +28,7 @@ import {
resolveLiveShardPreparation,
selectLiveShardFiles,
validateLiveShardReportPayload,
withoutReleaseWaivedLiveFiles,
} from "../../scripts/test-live-shard.mts";
import { resolveRuntimeWorkerUrl } from "../../src/infra/runtime-worker-url.js";
import { expectNoReaddirSyncDuring } from "../../src/test-utils/fs-scan-assertions.js";
@ -37,6 +38,19 @@ import { preparedScriptWrapperEnv } from "./prepared-script-wrapper.test-support
describe("scripts/test-live-shard", () => {
const allFiles = collectAllLiveTestFiles();
it("drops release-waived single-case live files only for the waived candidate version", () => {
const files = [
"src/gateway/gateway-progress-refresh.live.test.ts",
"src/gateway/gateway-codex-harness.live.test.ts",
"test/gateway-subagent-restart.live.test.ts",
];
expect(withoutReleaseWaivedLiveFiles(files, "2026.9.8")).toEqual([
"src/gateway/gateway-codex-harness.live.test.ts",
]);
expect(withoutReleaseWaivedLiveFiles(files, "2026.9.9")).toEqual(files);
expect(withoutReleaseWaivedLiveFiles(files, undefined)).toEqual(files);
});
it("discovers live tests without scanning source roots in-process", () => {
expectNoReaddirSyncDuring(() => {
const files = collectAllLiveTestFiles();