Skip to content

fix(system): validate launcher and timeit integer arguments - #601

Merged
singaraiona merged 1 commit into
RayforceDB:devfrom
belowzeroff:fix/system-arg-validation
Sep 20, 2026
Merged

singaraiona merged 1 commit into
RayforceDB:devfrom
belowzeroff:fix/system-arg-validation

Conversation

@belowzeroff

Copy link
Copy Markdown
Contributor

What changes for users

Malformed numeric startup arguments are rejected instead of being silently truncated or overflowing: -c/--cores, -t/--timeit, and -Q/--querylog now require complete, range-checked integer values. Invalid values fail startup with a non-zero status before the runtime starts.

The same strict parsing is used by .sys.args, .sys.timeit, and .sys.cmd "timeit ...", so values such as 2abc, overflowing core counts, and 1.5 now return an error instead of changing runtime state unexpectedly.

Validation

  • clean full test build with strict warnings
  • 3924/3924 core tests passed
  • release build passed with -Werror
  • targeted runtime and Rayfall system coverage included

All checks pass.

@singaraiona
singaraiona merged commit 0137aa4 into RayforceDB:dev Sep 20, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants