Skip to content

Commit 9040133

Browse files
committed
feat(webapp): allow a fractional ck vtime quantum
The vtime Lua serves weighted fair-queue tags and handles a sub-1 quantum (a finer serve granularity), and ckVtimeRegistration.test.ts exercises it at 0.3, but RUN_ENGINE_CK_VTIME_QUANTUM was schema-constrained to integers. Drop the .int() so the env no longer blocks a capability the engine already has. Default stays 1 (the weight hook is fixed at 1 today). Flagged by Devin on #4367.
1 parent a9a14be commit 9040133

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

apps/webapp/app/env.server.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1224,7 +1224,10 @@ const EnvironmentSchema = z
12241224
// Fair (virtual-time) ordering across concurrency-key variants of a base queue.
12251225
// Off by default; when off the run queue behaves exactly as before.
12261226
RUN_ENGINE_CK_VTIME_SCHEDULING_ENABLED: BoolEnv.default(false),
1227-
RUN_ENGINE_CK_VTIME_QUANTUM: z.coerce.number().int().positive().default(1),
1227+
// Fractional allowed: the vtime Lua serves weighted fair-queue tags, so a sub-1 quantum
1228+
// is a valid finer serve granularity. The weight hook is fixed at 1 today, so 1 stays the
1229+
// default, but the schema no longer blocks the capability the engine already has.
1230+
RUN_ENGINE_CK_VTIME_QUANTUM: z.coerce.number().positive().default(1),
12281231
RUN_ENGINE_CK_VTIME_WINDOW_MULTIPLIER: z.coerce.number().int().positive().default(3),
12291232
RUN_ENGINE_CK_VTIME_STATE_TTL_SECONDS: z.coerce.number().int().positive().default(86400),
12301233

0 commit comments

Comments
 (0)