Skip to content

Commit 3d1b8bc

Browse files
committed
Keep evaluated benchmark module imports free of entry side effects
1 parent 5b8c4eb commit 3d1b8bc

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

‎docs/Features/BenchmarkComparisons/LiveProgress.md‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,5 +64,8 @@ new capture filter, and an existing cache receipt acceptance regression failed.
6464
The [run 37212016437](https://github.com/managedcode/KeyLoad/actions/runs/37212016437)
6565
capture artifact has 28/29 passed: native logger capture rejected Aspire's UTC
6666
timestamp prefix. These failures are retained as evidence; the timestamp, module
67-
mode and filter inventory corrections require another genuine Linux run. Full
67+
mode and filter inventory corrections require another genuine Linux run. An
68+
evaluated Node import must not trigger the script's production entry merely
69+
because its first argument contains the module path; the real child regressions
70+
also verify this boundary. Full
6871
unit/scalar, recovery, RF3 and complete native workload gates are not qualified.

‎scripts/Features/BenchmarkComparisons/run-workload.mjs‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,8 @@ export async function runWorkload() {
134134
], root, progress);
135135
}
136136

137-
if (process.argv[1] && path.resolve(process.argv[1]) === fileURLToPath(import.meta.url)) {
137+
const evaluated = process.execArgv.some(argument => argument === '--eval' || argument === '-e');
138+
if (!evaluated && process.argv[1] && path.resolve(process.argv[1]) === fileURLToPath(import.meta.url)) {
138139
runWorkload().then(code => { process.exitCode = code; }).catch(() => {
139140
process.stderr.write('Native benchmark entry failed.\n');
140141
process.exitCode = 1;

0 commit comments

Comments
 (0)