@isentinel/jest-roblox 0.3.9 → 0.3.10

package/dist/cli.mjs

@@ -1,4 +1,4 @@
-import { B as parseGameOutput, J as formatBanner, N as outputMultiResult, P as outputSingleResult, bt as version, ct as loadConfig, ht as VALID_BACKENDS, n as runJestRoblox, st as mergeCliWithConfig, tt as LuauScriptError, v as formatMissingScopes, vt as isValidBackend, y as walkErrorChain, yt as ConfigError } from "./run-BM5sqclu.mjs";
+import { B as parseGameOutput, J as formatBanner, N as outputMultiResult, P as outputSingleResult, bt as version, ct as loadConfig, ht as VALID_BACKENDS, n as runJestRoblox, st as mergeCliWithConfig, tt as LuauScriptError, v as formatMissingScopes, vt as isValidBackend, y as walkErrorChain, yt as ConfigError } from "./run-BKP7mem2.mjs";
 import { OpenCloudError } from "@bedrock-rbx/ocale";
 import process from "node:process";
 import { parseArgs as parseArgs$1 } from "node:util";

package/dist/index.d.mts

@@ -461,9 +461,25 @@ interface OpenCloudOptions {
 }
 declare class OpenCloudBackend implements Backend {
   private readonly runner;
-  readonly kind = "open-cloud";
+  /** One-shot per run so parallel raced tasks don't repeat the warning. */
+  private raceWarned;
+  readonly kind: "open-cloud";
   constructor(credentials: OpenCloudCredentials, options?: OpenCloudOptions);
   runTests(options: BackendOptions): Promise<BackendResult>;
+  /**
+  * Optimistic version pinning. Pinned tasks
+  * (`/versions/{v}/luau-execution-session-tasks`) miss the warm-server pool
+  * whenever no server holds the freshly-uploaded version yet, costing a cold
+  * place boot per task (~10-45s, scaling with place size). Unpinned tasks
+  * boot the latest saved version from the warm pool, so the first attempt
+  * runs unpinned with a guard prepended: if the booted server is not on this
+  * run's version (a concurrent upload won the boot race), the task returns
+  * {@link PLACE_VERSION_RACE_SENTINEL} instead of running. On the sentinel,
+  * the task is retried once, pinned — correct by construction, no re-upload
+  * (the version exists even when it is no longer head), and no unpinned
+  * retry loop for a concurrent uploader to keep winning against.
+  */
+  private executeGuarded;
   private runBucket;
   private runStaticBuckets;
   private runWorkStealing;
@@ -606,7 +622,7 @@ declare class StudioCliBackend implements Backend {
   private readonly launch;
   private readonly studioPath?;
   private readonly timeout;
-  readonly kind = "studio-cli";
+  readonly kind: "studio-cli";
   constructor(options?: StudioCliOptions);
   runTests(options: BackendOptions): Promise<BackendResult>;
   /**
@@ -635,7 +651,7 @@ declare class StudioBackend implements Backend {
   private readonly timeout;
   private preConnected?;
   private wss?;
-  readonly kind = "studio";
+  readonly kind: "studio";
   constructor(options: StudioOptions);
   close(): void;
   runTests(options: BackendOptions): Promise<BackendResult>;

package/dist/index.mjs

@@ -1,4 +1,4 @@
-import { $ as writeManifest, A as buildManifestSchema, B as parseGameOutput, C as COVERAGE_MANIFEST_PATH, D as visitStatement, E as visitExpression, F as formatAnnotations, G as formatFailure, H as createTimingCollector, I as formatJobSummary, K as formatResult, L as formatExecuteOutput, M as readBuildManifest, O as buildPlace, Q as readManifest, R as runProjects, S as COVERAGE_BUILD_MANIFEST_PATH, T as visitBlock, U as formatJson, V as writeGameOutput, W as writeJsonFile, X as MANIFEST_VERSION, Y as hashFile, Z as manifestSchema, _ as createOpenCloudBackend, _t as defineProject, a as collectStubMounts, at as parseCoverageEnvelope, b as buildJestArgv, c as runTypecheck, ct as loadConfig, d as resolveAllProjects, dt as GLOBAL_TEST_KEYS, et as applyAttribution, f as StudioBackend, ft as JEST_ARGV_EXCLUDED_KEYS, g as OpenCloudBackend, gt as defineConfig, h as createStudioCliBackend, i as buildImplicitProject, it as normalizeRawCoverage, j as emitBuildManifest, k as BUILD_MANIFEST_VERSION, l as cleanLeftoverStubs, lt as resolveConfig, m as StudioCliBackend, mt as SHARED_TEST_KEYS, n as runJestRoblox, nt as extractJsonFromOutput, o as loadRojoTree, ot as mergeRawCoverage, p as createStudioBackend, pt as ROOT_CLI_KEYS, q as formatTestSummary, r as runSingleOrMulti, rt as parseJestOutput, s as prepareBakedCoverage, st as mergeCliWithConfig, t as getRawProjects, u as generateProjectStubs, ut as DEFAULT_CONFIG, w as findRojoProject, x as generateTestScript, z as formatGameOutputNotice } from "./run-BM5sqclu.mjs";
+import { $ as writeManifest, A as buildManifestSchema, B as parseGameOutput, C as COVERAGE_MANIFEST_PATH, D as visitStatement, E as visitExpression, F as formatAnnotations, G as formatFailure, H as createTimingCollector, I as formatJobSummary, K as formatResult, L as formatExecuteOutput, M as readBuildManifest, O as buildPlace, Q as readManifest, R as runProjects, S as COVERAGE_BUILD_MANIFEST_PATH, T as visitBlock, U as formatJson, V as writeGameOutput, W as writeJsonFile, X as MANIFEST_VERSION, Y as hashFile, Z as manifestSchema, _ as createOpenCloudBackend, _t as defineProject, a as collectStubMounts, at as parseCoverageEnvelope, b as buildJestArgv, c as runTypecheck, ct as loadConfig, d as resolveAllProjects, dt as GLOBAL_TEST_KEYS, et as applyAttribution, f as StudioBackend, ft as JEST_ARGV_EXCLUDED_KEYS, g as OpenCloudBackend, gt as defineConfig, h as createStudioCliBackend, i as buildImplicitProject, it as normalizeRawCoverage, j as emitBuildManifest, k as BUILD_MANIFEST_VERSION, l as cleanLeftoverStubs, lt as resolveConfig, m as StudioCliBackend, mt as SHARED_TEST_KEYS, n as runJestRoblox, nt as extractJsonFromOutput, o as loadRojoTree, ot as mergeRawCoverage, p as createStudioBackend, pt as ROOT_CLI_KEYS, q as formatTestSummary, r as runSingleOrMulti, rt as parseJestOutput, s as prepareBakedCoverage, st as mergeCliWithConfig, t as getRawProjects, u as generateProjectStubs, ut as DEFAULT_CONFIG, w as findRojoProject, x as generateTestScript, z as formatGameOutputNotice } from "./run-BKP7mem2.mjs";
 import * as path$1 from "node:path";
 //#region src/artifacts/build-coverage-place.ts
 const CACHE_DIR$1 = path$1.join(".jest-roblox", "cache");

package/dist/{run-BM5sqclu.mjs → run-BKP7mem2.mjs}

@@ -36,7 +36,7 @@ import { once } from "node:events";
 import { parseJSONC, parseYAML } from "confbox";
 import { Visitor, parseSync } from "oxc-parser";
 //#region package.json
-var version = "0.3.9";
+var version = "0.3.10";
 //#endregion
 //#region src/config/errors.ts
 var ConfigError = class extends Error {
@@ -7554,6 +7554,15 @@ function readStringProperty(err, key) {
 }
 //#endregion
 //#region src/backends/open-cloud.ts
+/**
+* The value the version guard returns when the booted server is not running
+* the version this run uploaded — i.e. a concurrent upload won the boot race.
+* The backend retries that task once, pinned to the uploaded version.
+*
+* Embedded verbatim in a Luau double-quoted string literal, so it must not
+* contain backslashes, double quotes, or newlines.
+*/
+const PLACE_VERSION_RACE_SENTINEL = "__JEST_ROBLOX_PLACE_VERSION_RACE__";
 const PARALLEL_AUTO_CAP = 3;
 const BASE_URL_ENV = "JEST_ROBLOX_OPEN_CLOUD_BASE_URL";
 const MAX_RETRIES_ENV = "JEST_ROBLOX_OCALE_MAX_RETRIES";
@@ -7561,11 +7570,14 @@ const DEFAULT_STREAM_POLL_MS = 250;
 const TrailingSlashesPattern = /\/+$/;
 var OpenCloudBackend = class {
 	runner;
+	/** One-shot per run so parallel raced tasks don't repeat the warning. */
+	raceWarned = false;
 	kind = "open-cloud";
 	constructor(credentials, options) {
 		this.runner = options?.runner ?? new OcaleRunner(credentials, resolveRunnerOptions());
 	}
 	async runTests(options) {
+		this.raceWarned = false;
 		const { jobs, parallel, scriptOverride, streaming, workStealing } = options;
 		if (jobs.length === 0) throw new Error("OpenCloudBackend requires at least one job");
 		if (workStealing === true && scriptOverride === void 0) throw new Error("OpenCloudBackend work-stealing mode requires scriptOverride");
@@ -7588,6 +7600,37 @@ var OpenCloudBackend = class {
 			}
 		};
 	}
+	/**
+	* Optimistic version pinning. Pinned tasks
+	* (`/versions/{v}/luau-execution-session-tasks`) miss the warm-server pool
+	* whenever no server holds the freshly-uploaded version yet, costing a cold
+	* place boot per task (~10-45s, scaling with place size). Unpinned tasks
+	* boot the latest saved version from the warm pool, so the first attempt
+	* runs unpinned with a guard prepended: if the booted server is not on this
+	* run's version (a concurrent upload won the boot race), the task returns
+	* {@link PLACE_VERSION_RACE_SENTINEL} instead of running. On the sentinel,
+	* the task is retried once, pinned — correct by construction, no re-upload
+	* (the version exists even when it is no longer head), and no unpinned
+	* retry loop for a concurrent uploader to keep winning against.
+	*/
+	async executeGuarded(options) {
+		const { placeVersion, script, timeout } = options;
+		const guarded = injectVersionGuard(script, placeVersion);
+		const first = await this.runner.executeScript({
+			script: guarded,
+			timeout
+		});
+		if (first.outputs[0] !== "__JEST_ROBLOX_PLACE_VERSION_RACE__") return first;
+		if (!this.raceWarned) {
+			this.raceWarned = true;
+			process.stderr.write("Warning: place version raced by a concurrent upload — raced tasks retried pinned (slower, cold place boot).\n");
+		}
+		return this.runner.executeScript({
+			placeVersion,
+			script,
+			timeout
+		});
+	}
 	async runBucket(bucket, placeVersion, scriptOverride) {
 		const { indices, jobs } = bucket;
 		const primary = jobs[0];
@@ -7598,7 +7641,7 @@ var OpenCloudBackend = class {
 			};
 		});
 		const script = scriptOverride ?? generateTestScript(inputs);
-		const scriptResult = await this.runner.executeScript({
+		const scriptResult = await this.executeGuarded({
 			placeVersion,
 			script,
 			timeout: primary.config.timeout
@@ -7643,7 +7686,7 @@ var OpenCloudBackend = class {
 			},
 			places: [{ runTask: async () => {
 				launched += 1;
-				return this.runner.executeScript({
+				return this.executeGuarded({
 					placeVersion,
 					script: scriptOverride,
 					timeout: primaryConfig.timeout
@@ -7743,6 +7786,22 @@ async function sleep(ms) {
 		setTimeout(resolve, ms);
 	});
 }
+/**
+* Insert the version guard after any leading `--!` directive lines — Luau
+* honors directives only in the leading comment block, so a plain line-1
+* prepend would silently disable a caller's `--!strict`/`--!native`/etc.
+*/
+function injectVersionGuard(script, placeVersion) {
+	const guard = `if game.PlaceVersion ~= ${String(placeVersion)} then return "${PLACE_VERSION_RACE_SENTINEL}" end`;
+	const lines = script.split("\n");
+	let insertAt = 0;
+	for (const line of lines) {
+		if (!line.startsWith("--!")) break;
+		insertAt += 1;
+	}
+	lines.splice(insertAt, 0, guard);
+	return lines.join("\n");
+}
 function resolveRunnerOptions() {
 	const baseUrl = resolveOpenCloudBaseUrl();
 	const maxRetries = resolveOcaleMaxRetries();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@isentinel/jest-roblox",
-  "version": "0.3.9",
+  "version": "0.3.10",
   "description": "Jest-compatible CLI for running roblox-ts tests via Roblox Open Cloud",
   "keywords": [
     "jest",
@@ -48,7 +48,7 @@
     "!dist/**/*.tsbuildinfo"
   ],
   "dependencies": {
-    "@bedrock-rbx/ocale": "0.1.0-beta.19",
+    "@bedrock-rbx/ocale": "0.1.0",
     "@jridgewell/trace-mapping": "0.3.31",
     "arktype": "2.2.0",
     "c12": "4.0.0-beta.5",
@@ -67,7 +67,7 @@
   },
   "devDependencies": {
     "@isentinel/eslint-config": "6.0.0-beta.3",
-    "@isentinel/roblox-ts": "4.0.10",
+    "@isentinel/roblox-ts": "4.0.11",
     "@isentinel/tsconfig": "1.2.0",
     "@isentinel/weld": "0.2.0",
     "@oxc-project/types": "0.123.0",