Skip to content

fix(cli): reject non-positive list limits - #1565

Open
YIKUAIBANZI wants to merge 1 commit into
TokenRhythm:mainfrom
YIKUAIBANZI:fix/positive-list-limits
Open

fix(cli): reject non-positive list limits#1565
YIKUAIBANZI wants to merge 1 commit into
TokenRhythm:mainfrom
YIKUAIBANZI:fix/positive-list-limits

Conversation

@YIKUAIBANZI

Copy link
Copy Markdown

Summary

  • reject non-positive sessions list and cron runs limits before opening a gateway connection
  • emit the standard INVALID_REQUEST JSON envelope and exit code 2 for invalid CLI input
  • reject non-positive operator sessions.list RPC limits while preserving the published guest-limit clamp behavior
  • add regression coverage for both CLI commands and the legacy sessions RPC adapter

Fixes #1558

Verification

  • uv run ruff check src tests
  • 99 focused CLI/gateway tests passed
  • full suite: 25,349 passed, 259 skipped; four host-tooling failures on macOS were re-run successfully with the Linux CI prerequisites (docker command behavior and GNU sed)
  • npm --prefix opensquilla-webui run build
  • uv build --wheel

Third-party origins

none

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.

[Bug]: Sessions and Cron commands accept negative --limit values

1 participant