ewilliams28/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-08-25 03:45:46 -06:00
Code Issues Packages Projects Releases Wiki Activity

perf(mantis): build proof lanes runtime-only (#128007)

Browse Source 
The proof gateways execute runtime JS only; declarations forced a ~177s
unified rebuild per lane because the declaration cache key is an
aggregate source hash. OPENCLAW_RUN_NODE_SKIP_DTS_BUILD=1 on profile
full now selects the runtime artifact surface via the uncached generic
tsdown graph, both Mantis lanes pass it through, and the baseline
archive moves to an isolated mantis-runtime-v1 namespace. Measured
candidate build: 177.1s -> 33.05s.
This commit is contained in:
Ayaan Zaidi
2026-08-23 00:15:24 +05:30
committed by GitHub
parent 7d18cc4ccb
commit 2eecf3d3f2
5 changed files with 86 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/scripts/build-all.test.ts
+34 -5
View File
@@ -587,6 +587,30 @@ describe("resolveBuildAllSteps", () => {
]);
});
it("uses the full runtime artifact surface without declaration work when DTS is disabled", () => {
const steps = resolveBuildAllSteps("full", {
OPENCLAW_RUN_NODE_SKIP_DTS_BUILD: "1",
});
const labels = steps.map((step) => step.label);
expect(labels).toEqual([
"plugins:assets:build",
"tsdown",
"external-plugins:local-dist",
"check-cli-bootstrap-imports",
"plugins:assets:copy",
"runtime-postbuild",
"build-stamp",
"runtime-postbuild-stamp",
"ui:build",
"write-build-info",
"write-cli-startup-metadata",
]);
expect(steps.find((step) => step.label === "tsdown")?.cache).toBeUndefined();
expect(labels).not.toContain("write-plugin-sdk-entry-dts");
expect(labels).not.toContain("check-plugin-sdk-exports");
});
it("uses a source performance profile with QA assets and immutable build provenance", () => {
expect(resolveBuildAllSteps("sourcePerformance").map((step) => step.label)).toEqual([
"plugins:assets:build",
@@ -1121,27 +1145,32 @@ describe("resolveBuildAllStepCacheState", () => {
});
});
it("marks cacheable steps stale when a tracked env input changes", () => {
it("never reuses a runtime-only cache as a declaration-complete full-build cache", () => {
withBuildCacheFixture(({ rootDir, step }) => {
const envStep = {
...step,
cache: {
...step.cache,
env: ["OPENCLAW_BUILD_PRIVATE_QA"],
env: ["OPENCLAW_RUN_NODE_SKIP_DTS_BUILD"],
restore: "always" as const,
},
};
const cacheState = resolveBuildAllStepCacheState(envStep, {
rootDir,
env: { OPENCLAW_BUILD_PRIVATE_QA: "0" },
env: { OPENCLAW_RUN_NODE_SKIP_DTS_BUILD: "1" },
});
writeBuildAllStepCacheStamp(envStep, cacheState, {
rootDir,
env: { OPENCLAW_RUN_NODE_SKIP_DTS_BUILD: "1" },
});
writeBuildAllStepCacheStamp(envStep, cacheState, { rootDir });
const stale = resolveBuildAllStepCacheState(envStep, {
rootDir,
env: { OPENCLAW_BUILD_PRIVATE_QA: "1" },
env: {},
});
expect(stale.cacheable).toBe(true);
expect(stale.fresh).toBe(false);
expect(stale.restorable).toBe(false);
expect(stale.reason).toBe("stale");
});
});
test/scripts/mantis-telegram-desktop-proof-workflow.test.ts
+11 -5
View File
@@ -901,12 +901,13 @@ describe("Mantis Telegram Desktop proof workflow", () => {
expect(restore.with?.key).toContain("steps.proof_worktrees.outputs.lockfile_sha256");
expect(restore.with?.key).toContain("steps.proof_worktrees.outputs.node_version");
expect(restore.with?.key).toContain("steps.proof_worktrees.outputs.pnpm_version");
expect(restore.with?.key).toContain("mantis-baseline-v3");
expect(restore.with?.key).toContain("mantis-runtime-v1");
expect(restore.with?.key).toMatch(/pnpm_version.*baseline_revision/u);
expect(restore.with?.["restore-keys"]).toBe(
"${{ runner.os }}-${{ runner.arch }}-mantis-baseline-v3-${{ steps.proof_worktrees.outputs.lockfile_sha256 }}-${{ steps.proof_worktrees.outputs.node_version }}-${{ steps.proof_worktrees.outputs.pnpm_version }}-\n",
"${{ runner.os }}-${{ runner.arch }}-mantis-runtime-v1-${{ steps.proof_worktrees.outputs.lockfile_sha256 }}-${{ steps.proof_worktrees.outputs.node_version }}-${{ steps.proof_worktrees.outputs.pnpm_version }}-\n",
);
expect(restore.with?.path).toBe(".artifacts/mantis-baseline-build.tar");
expect(restore.with?.path).toBe(".artifacts/mantis-runtime-build.tar");
expect(setup.with?.["build-all-cache-scope"]).toBeUndefined();
expect(builds.if).toBeUndefined();
expect(builds.env?.HOST_PNPM_STORE).toBe(
"${{ steps.setup-node-env.outputs.pnpm-store-cache-path }}",
@@ -916,10 +917,13 @@ describe("Mantis Telegram Desktop proof workflow", () => {
expect(buildRun).toContain("baseline_archive_restored=true");
expect(buildRun).toContain('"$toolchain_dir/pnpm" install --frozen-lockfile');
expect(buildRun).toContain('if [[ "$BASELINE_BUILD_CACHE_HIT" != "true" ]]');
expect(buildRun).toContain('"$toolchain_dir/pnpm" build');
expect(buildRun).toMatch(
/OPENCLAW_RUN_NODE_SKIP_DTS_BUILD=1 \\\n\s+PATH="\$toolchain_dir:\/usr\/bin:\/bin" \\\n\s+"\$toolchain_dir\/pnpm" build/u,
);
expect(buildRun).toContain('mv -T "$baseline_archive_new" "$BASELINE_BUILD_ARCHIVE"');
expect(buildRun).toContain('mkdir -p "$baseline_root/.artifacts/build-all-cache"');
expect(buildRun).toContain(".artifacts/build-all-cache");
expect(buildRun).toContain("for phase in tsdown-ai tsdown-packages tsdown-unified");
expect(buildRun).not.toContain("for phase in tsdown-ai tsdown-packages tsdown-unified");
expect(buildRun).toContain("-type f -links +1");
expect(save.if).toContain("steps.baseline_build_cache.outputs.cache-hit != 'true'");
expect(save.uses).toContain("actions/cache/save@");
@@ -1253,6 +1257,8 @@ describe("Mantis Telegram Desktop proof workflow", () => {
expect(wrapper).toContain("--memory 8g");
expect(wrapper).toContain("--cpus 4");
expect(wrapper).toContain("--memory 16g");
expect(wrapper).toContain("OPENCLAW_RUN_NODE_SKIP_DTS_BUILD=1 corepack pnpm build");
expect(wrapper).not.toContain("\n corepack pnpm build\n");
expect(sutScript).not.toContain("CODEX_HOME");
expect(sutScript).not.toContain("codexProxyPort");
expect(wrapper).toContain('connects("runner-host", 9)');