Skip to content

fix(cli): reject pagination modifiers with --count - #69

Closed
yearth wants to merge 1 commit into
yusukebe:mainfrom
yearth:codex/reject-count-pagination-modifiers
Closed

fix(cli): reject pagination modifiers with --count#69
yearth wants to merge 1 commit into
yusukebe:mainfrom
yearth:codex/reject-count-pagination-modifiers

Conversation

@yearth

@yearth yearth commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • reject explicit --limit, --offset, --budget, and --all with --count
  • fail before file or network I/O with an actionable hint
  • preserve standalone count behavior and existing error precedence

Tests

  • bunx tsc --noEmit
  • bunx oxfmt --check .
  • bun test --timeout 30000 (277 pass)
  • bun run build

@yusukebe

yusukebe commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Hi @yearth

I think silent is fine here, like -L and -s.

@yearth

yearth commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, understood. I over-applied the never-silent rule here. Since --count reports total matches, treating pagination modifiers as harmless no-ops is consistent with the existing curl-compatible flags. I’ll close this.

@yearth yearth closed this Aug 10, 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.

2 participants