mirror of
https://github.com/QwenLM/qwen-code.git
synced 2026-11-07 06:19:43 -10:00
* perf(release): reuse verified native audio prebuilds across releases Every release rebuilds the five native audio prebuilds even when nothing that affects those binaries changed. The build is short; the wait is not. For v0.26.3 the Linux x64 prebuild queued 74 minutes to execute for 50 seconds, and the collection step queued again after it — and `publish` depends on that collection, so the whole release sits behind it. Fingerprint the audio build inputs and reuse an artifact from an earlier successful official main release when the fingerprint matches. The key covers the audio package, the dependency lock, the package manager pin and the contents of the producing workflow and this helper, so the Node pin and the prebuildify target arguments are part of it. The package version is excluded deliberately: no part of the native build reads it, and including it would make reuse impossible between releases. Reuse means executing bytes an earlier run produced, so provenance is checked rather than assumed: exact fingerprint name, unexpired, same repository for both run and head run, and a completed successful release.yml run on main from workflow_dispatch or schedule. The pinned download-artifact already verifies the artifact digest. Retention is checked differently at selection and consumption. Selection requires 48 hours of headroom so the artifact outlives the release; consumers require only that it has not expired. Re-applying the margin against a later clock would reject an artifact the release had already committed to, after the native matrix was skipped and with no fallback. Lookup, trial download and payload validation are all non-fatal, and the reuse ID is published only once a trial download has validated; an empty ID runs the full matrix, which is today's behaviour. The publisher does not trust that decision: it recomputes the fingerprint from its own checkout, re-verifies the source run, and validates the payload again, fatally — by then the matrix was skipped, so a bad artifact must stop the release. This does not depend on ECS_RELEASE_HOST_RUNNER_ENABLED and takes effect with the switch unset. The collector needed three fixes to be a trustworthy producer. Its `rm -rf -- "$GITHUB_WORKSPACE/packages/audio-capture/prebuilds"` resolved through whatever `packages` was on a persistent host: with a leftover symlink it deleted an external tree and exited 0, and the following download wrote through the same symlink. It now wipes through a shared preflight that canonicalises the workspace and refuses paths resolving outside it. It also ran `node` with no Node contract, inheriting Node from the hosted image by accident. And its release-only label pinned `x64` while the runners holding that label are ARM64, which never matches and queues until timeout. Contract tests pin the wiring, each confirmed to fail against an injected regression. Design doc in both languages. * fix(release): reconcile audio reuse workflow contracts Share the pre-checkout wipe, preserve the workflow size cap, and let fingerprint failures rebuild native audio. Co-authored-by: Qwen-Coder <[email protected]> * fix(ci): use block-style release permissions Co-authored-by: Qwen-Coder <[email protected]> * fix(release): drop x64 from publish's release-host expression The review's unresolved item: collect got the ARM64 label fix but publish kept ["self-hosted", "linux", "x64", "ecs-qwen-release-host"], which can never match the ARM64 machines holding that label, so the OIDC-carrying job would queue forever once ECS_RELEASE_HOST_RUNNER_ENABLED is true. Same expression as the collect fix, same one-word drop. * docs(release): clarify audio reuse rollout and validation Co-authored-by: Qwen-Coder <[email protected]> * test(release): cover artifact overwrite and contained symlink guards * test(web-shell): include transcript coverage in recovery fixture Bring the fixture correction from #15883 into this branch to address the observed smoke failure. Co-authored-by: Qwen-Coder <[email protected]> * fix(release): make audio cache seeding best effort Keep the validated same-run artifact usable when retaining a keyed copy fails. Pin the reuse wiring and correct the retention and producer fixtures without changing release trust boundaries. Co-authored-by: Qwen-Coder <[email protected]> * chore(test): apply prettier to the audio prebuild fixtures * docs(release): clarify audio reuse run provenance Co-authored-by: Qwen-Coder <[email protected]> --------- Co-authored-by: Qwen-Coder <[email protected]> Co-authored-by: zzj <[email protected]>
302 lines
11 KiB
JavaScript
302 lines
11 KiB
JavaScript
/**
|
|
* @license
|
|
* Copyright 2026 Qwen Team
|
|
* SPDX-License-Identifier: Apache-2.0
|
|
*/
|
|
import { readFileSync } from 'node:fs';
|
|
import { describe, expect, it } from 'vitest';
|
|
import { parse } from 'yaml';
|
|
|
|
const workflow = (name) =>
|
|
parse(readFileSync(`.github/workflows/${name}.yml`, 'utf8'));
|
|
// Architecture-free jobs can use either reserved ARM64 or x64 slots.
|
|
const routes = [
|
|
['release-vscode-companion', 'prepare'],
|
|
['release-vscode-companion', 'build'],
|
|
['release-vscode-companion', 'publish'],
|
|
['finalize-release', 'finalize'],
|
|
['desktop-release', 'publish'],
|
|
['release-chrome-extension', 'build'],
|
|
['release-chrome-extension', 'publish'],
|
|
['build-and-publish-image', 'build-and-push-to-ghcr'],
|
|
];
|
|
// The VSCode build is a matrix: only the universal leg is architecture-free.
|
|
const matrixLeg = (file, id) =>
|
|
file === 'release-vscode-companion' && id === 'build';
|
|
|
|
describe('downstream release pool', () => {
|
|
it.each(routes)(
|
|
'%s / %s opts in to the isolated pool with a hosted fallback',
|
|
(file, id) => {
|
|
const job = workflow(file).jobs[id];
|
|
const route = (
|
|
enabled,
|
|
repository = 'QwenLM/qwen-code',
|
|
matrix = { os: 'ubuntu-latest', universal: true },
|
|
) =>
|
|
// Evaluate these fixed repository expressions for the supported routing cases.
|
|
Function(
|
|
'github',
|
|
'vars',
|
|
'matrix',
|
|
'fromJSON',
|
|
`return (${job['runs-on'].slice(3, -2)})`,
|
|
)(
|
|
{ repository },
|
|
{ ECS_RELEASE_HOST_RUNNER_ENABLED: enabled },
|
|
matrix,
|
|
JSON.parse,
|
|
);
|
|
expect(route('true')).toEqual([
|
|
'self-hosted',
|
|
'linux',
|
|
'ecs-qwen-release-host',
|
|
]);
|
|
expect([route(''), route('false'), route('true', 'fork/repo')]).toEqual(
|
|
matrixLeg(file, id)
|
|
? ['ubuntu-latest', 'ubuntu-latest', 'ubuntu-latest']
|
|
: [['ubuntu-latest'], ['ubuntu-latest'], ['ubuntu-latest']],
|
|
);
|
|
if (matrixLeg(file, id)) {
|
|
// Native legs keep their own platforms, including the linux-x64 one:
|
|
// it resolves native dependencies for the host it builds on.
|
|
for (const os of ['windows-latest', 'macos-latest', 'macos-15-intel'])
|
|
expect(
|
|
route('true', 'QwenLM/qwen-code', { os, universal: false }),
|
|
).toBe(os);
|
|
expect(
|
|
route('true', 'QwenLM/qwen-code', {
|
|
os: 'ubuntu-latest',
|
|
universal: false,
|
|
}),
|
|
).toBe('ubuntu-latest');
|
|
}
|
|
const init = job.steps.findIndex(
|
|
(s) => s.uses === '$/.github/actions/prepare-release-runner',
|
|
);
|
|
expect(init).toBe(id === 'build-and-push-to-ghcr' ? 1 : 0);
|
|
for (const setup of job.steps.filter((s) =>
|
|
s.uses?.startsWith('actions/setup-node@'),
|
|
)) {
|
|
expect(setup.if).toContain("runner.environment != 'self-hosted'");
|
|
}
|
|
},
|
|
);
|
|
|
|
// The preflight wipes the workspace before actions/checkout, so it must use
|
|
// the '$/' self-repository reference: a './' local action resolves against
|
|
// GITHUB_WORKSPACE and would need a checkout first (and on a persistent
|
|
// machine would execute whatever tree the previous job left behind).
|
|
it('runs the shared preflight from the running commit, before checkout', () => {
|
|
const action = parse(
|
|
readFileSync('.github/actions/prepare-release-runner/action.yml', 'utf8'),
|
|
);
|
|
const [step] = action.runs.steps;
|
|
expect(action.runs.using).toBe('composite');
|
|
expect(step.if).toContain("runner.environment == 'self-hosted'");
|
|
expect(step['working-directory']).toBe('${{ runner.temp }}');
|
|
expect(step.env.WORKFLOW_SHA).toBe('${{ github.workflow_sha }}');
|
|
expect(step.run).toContain('set -euo pipefail');
|
|
expect(step.run).toContain('export GH_CONFIG_DIR="$bootstrap/gh"');
|
|
expect(step.run).toContain('?ref=${WORKFLOW_SHA}');
|
|
expect(step.run).toContain('bash "$bootstrap/prepare-release-runner.sh"');
|
|
expect(step.run).toContain('Release hosts require Node 22');
|
|
expect(step.run).toContain('bash "$bootstrap/check-disk-floor.sh"');
|
|
|
|
for (const [file, id] of routes) {
|
|
const steps = workflow(file).jobs[id].steps;
|
|
const uses = steps.map((s) => s.uses);
|
|
const preflight = uses.indexOf(
|
|
'$/.github/actions/prepare-release-runner',
|
|
);
|
|
const checkout = uses.findIndex((u) =>
|
|
u?.startsWith('actions/checkout@'),
|
|
);
|
|
expect(preflight).toBeGreaterThanOrEqual(0);
|
|
if (checkout !== -1) expect(preflight).toBeLessThan(checkout);
|
|
}
|
|
});
|
|
|
|
it('keeps incompatible native builds and npm provenance publishers hosted', () => {
|
|
expect(workflow('desktop-release').jobs.build['runs-on']).toBe(
|
|
'${{ matrix.os }}',
|
|
);
|
|
expect(workflow('release-sdk').jobs['release-sdk']['runs-on']).toBe(
|
|
'ubuntu-latest',
|
|
);
|
|
expect(workflow('live-host-release').jobs.publish['runs-on']).toBe(
|
|
'ubuntu-latest',
|
|
);
|
|
// These execute a hardcoded linux-amd64 ossutil, so they cannot ride the
|
|
// architecture-free routes. Their migration to the reserved x64 slot is
|
|
// separate from this change.
|
|
for (const [file, id] of [
|
|
['sync-release-to-oss', 'sync'],
|
|
['sync-desktop-to-oss', 'sync'],
|
|
['sync-live-host-to-oss', 'sync'],
|
|
['release-managed-runtime', 'release'],
|
|
]) {
|
|
const text = readFileSync(`.github/workflows/${file}.yml`, 'utf8');
|
|
expect(text).toContain('linux-amd64.zip');
|
|
expect(workflow(file).jobs[id]['runs-on']).toBe('ubuntu-latest');
|
|
}
|
|
});
|
|
|
|
it('isolates persistent tool and credential state', () => {
|
|
const script = readFileSync(
|
|
'.github/scripts/prepare-release-runner.sh',
|
|
'utf8',
|
|
);
|
|
for (const setting of [
|
|
'GIT_CONFIG_GLOBAL',
|
|
'NPM_CONFIG_USERCONFIG',
|
|
'NPM_CONFIG_PREFIX',
|
|
'DOCKER_CONFIG',
|
|
'GH_CONFIG_DIR',
|
|
])
|
|
expect(script).toContain(setting);
|
|
});
|
|
});
|
|
|
|
describe('SDK build/publish handoff', () => {
|
|
const jobs = workflow('release-sdk').jobs;
|
|
const builder = jobs['build-sdk'];
|
|
const publisher = jobs['release-sdk'];
|
|
it('builds on opt-in only and never silently rebuilds after an ECS failure', () => {
|
|
expect(builder.if).toContain(
|
|
"vars.ECS_RELEASE_HOST_RUNNER_ENABLED == 'true'",
|
|
);
|
|
// The builder's label is a literal array rather than a switched
|
|
// expression, so the arch-free contract needs its own assertion: the
|
|
// pool has both architectures and this job only bundles with Node.
|
|
expect(builder['runs-on']).toEqual([
|
|
'self-hosted',
|
|
'linux',
|
|
'ecs-qwen-release-host',
|
|
]);
|
|
expect(builder.permissions).toEqual({ contents: 'read' });
|
|
expect(
|
|
builder.steps.find((s) => s.name === 'Checkout').with[
|
|
'persist-credentials'
|
|
],
|
|
).toBe(false);
|
|
expect(publisher.needs).toEqual(['build-sdk']);
|
|
expect(publisher.if).toContain("needs.build-sdk.result == 'success'");
|
|
expect(publisher.if).toContain(
|
|
"needs.build-sdk.result == 'skipped' && vars.ECS_RELEASE_HOST_RUNNER_ENABLED != 'true'",
|
|
);
|
|
const shared = builder.steps.filter((s) => s.if?.includes('env.BUILD_SDK'));
|
|
for (const step of shared) expect(publisher.steps).toContainEqual(step);
|
|
expect(publisher.env.BUILD_SDK).toBe(
|
|
"${{ needs.build-sdk.result != 'success' }}",
|
|
);
|
|
});
|
|
it('pins source and artifact identity and restores the chosen CLI after tests', () => {
|
|
expect(
|
|
publisher.steps.find((s) => s.name === 'Checkout').with.ref,
|
|
).toContain('needs.build-sdk.outputs.source_sha');
|
|
expect(
|
|
publisher.steps.find((s) => s.name === 'Download the tested SDK package')
|
|
.with['artifact-ids'],
|
|
).toBe('${{ needs.build-sdk.outputs.artifact_id }}');
|
|
const names = builder.steps.map((s) => s.name);
|
|
for (const mode of ['npm', 'source'])
|
|
expect(
|
|
names.indexOf(`Restore CLI into SDK (from ${mode})`),
|
|
).toBeGreaterThan(names.indexOf('Run SDK Integration Tests'));
|
|
expect(
|
|
names.indexOf('Pack the tested SDK without lifecycle scripts'),
|
|
).toBeGreaterThan(names.indexOf('Record bundled CLI version'));
|
|
const publish = publisher.steps.find(
|
|
(s) => s.name === 'Publish @qwen-code/sdk',
|
|
);
|
|
expect(publish.run).toContain(
|
|
'npm publish "$tarball" --ignore-scripts --provenance',
|
|
);
|
|
expect(publish.run).toContain('dist.integrity');
|
|
expect(jobs['report-failure'].if).toContain(
|
|
"needs.build-sdk.result == 'failure'",
|
|
);
|
|
expect(jobs['report-failure'].if).toContain('inputs.dry_run != true');
|
|
});
|
|
});
|
|
|
|
describe('audio release slots', () => {
|
|
const audio = workflow('audio-capture-prebuilds');
|
|
const release = workflow('release');
|
|
const route = (job, matrix, enabled, repository = 'QwenLM/qwen-code') =>
|
|
Function(
|
|
'github',
|
|
'vars',
|
|
'matrix',
|
|
'fromJSON',
|
|
'startsWith',
|
|
'format',
|
|
`return (${job['runs-on'].slice(3, -2)})`,
|
|
)(
|
|
{ repository },
|
|
{ ECS_RELEASE_HOST_RUNNER_ENABLED: enabled },
|
|
matrix,
|
|
JSON.parse,
|
|
(value, prefix) => value.startsWith(prefix),
|
|
(template, value) => template.replace('{0}', value),
|
|
);
|
|
|
|
it('selects matching Linux architecture and preserves hosted fallbacks', () => {
|
|
for (const matrix of audio.jobs.build.strategy.matrix.include) {
|
|
const linux = matrix.os.startsWith('ubuntu');
|
|
expect(route(audio.jobs.build, matrix, 'true')).toEqual(
|
|
linux
|
|
? ['self-hosted', 'linux', matrix.arch, 'ecs-qwen-release-host']
|
|
: matrix.runner,
|
|
);
|
|
expect(route(audio.jobs.build, matrix, '')).toBe(matrix.runner);
|
|
expect(route(audio.jobs.build, matrix, 'false')).toBe(matrix.runner);
|
|
expect(route(audio.jobs.build, matrix, 'true', 'fork/repo')).toBe(
|
|
matrix.runner,
|
|
);
|
|
}
|
|
expect(route(audio.jobs.collect, {}, 'true')).toEqual([
|
|
'self-hosted',
|
|
'linux',
|
|
'ecs-qwen-release-host',
|
|
]);
|
|
expect(route(audio.jobs.collect, {}, '')).toEqual(['ubuntu-latest']);
|
|
});
|
|
|
|
it('builds the selected release commit', () => {
|
|
expect(release.jobs.audio_capture_prebuilds.uses).toBe(
|
|
'./.github/workflows/audio-capture-prebuilds.yml',
|
|
);
|
|
expect(release.jobs.audio_capture_prebuilds.with).toMatchObject({
|
|
ref: '${{ needs.prepare.outputs.release_sha }}',
|
|
});
|
|
expect(
|
|
audio.jobs.build.steps.find((s) =>
|
|
s.uses?.startsWith('actions/checkout@'),
|
|
).with.ref,
|
|
).toBe('${{ inputs.ref || github.sha }}');
|
|
for (const job of [audio.jobs.build, audio.jobs.collect]) {
|
|
expect(job.steps[0].uses).toBe(
|
|
'$/.github/actions/prepare-release-runner',
|
|
);
|
|
}
|
|
});
|
|
|
|
it('keeps newer host glibc out of Linux native binaries', () => {
|
|
const steps = audio.jobs.build.steps;
|
|
const container = steps.find((s) => s.name === 'Build Linux prebuild');
|
|
expect(container.if).toBe("${{ startsWith(matrix.os, 'ubuntu') }}");
|
|
expect(container.run).toContain('node:22.22.3-bookworm');
|
|
expect(container['working-directory']).toBe('.');
|
|
expect(container.run).toContain('--frozen-lockfile --ignore-scripts');
|
|
expect(container.run).toContain('npm run prebuildify -- --target 22.22.3');
|
|
expect(container.run).toContain('--user "$(id -u):$(id -g)"');
|
|
for (const name of ['Install build deps', 'Build prebuild']) {
|
|
expect(steps.find((s) => s.name === name).if).toBe(
|
|
"${{ !startsWith(matrix.os, 'ubuntu') }}",
|
|
);
|
|
}
|
|
});
|
|
});
|