Skip to content

Validate positive tokens_per_minute#152

Merged
pbdeuchler merged 1 commit into
masterfrom
fix/tokens-per-minute-positive
Jun 15, 2026
Merged

Validate positive tokens_per_minute#152
pbdeuchler merged 1 commit into
masterfrom
fix/tokens-per-minute-positive

Conversation

@pbdeuchler

Copy link
Copy Markdown
Owner

Closes #130.

Summary:

  • Reject explicit tokens_per_minute = 0 during model config validation.
  • Mark tokens_per_minute with a JSON Schema minimum of 1.
  • Document that tokens_per_minute must be positive when set.

Verification:

  • cargo fmt --all -- --check
  • cargo test -p halter-config
  • cargo run -p halter-cli --bin halter-cli -- config schema
  • cargo check --workspace

@pbdeuchler pbdeuchler merged commit ba02804 into master Jun 15, 2026
2 checks passed
pbdeuchler added a commit that referenced this pull request Jun 16, 2026
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.

[L28] tokens_per_minute = 0 is accepted with undefined semantics

1 participant