Skip to content

fix: limit argument not being passed to view next page#172

Merged
Cedric / ViaDézo1er (viadezo1er) merged 1 commit intomainfrom
cedric/limit-argument-not-passed-to-next-page
May 4, 2026
Merged

fix: limit argument not being passed to view next page#172
Cedric / ViaDézo1er (viadezo1er) merged 1 commit intomainfrom
cedric/limit-argument-not-passed-to-next-page

Conversation

@viadezo1er
Copy link
Copy Markdown
Contributor

@viadezo1er Cedric / ViaDézo1er (viadezo1er) commented May 4, 2026

These projects are from a personal org, you have to select different project ids if you want to test it yourself.

bt view logs --non-interactive --limit 1 --since 2026-05-01T00:00:00Z --object-ref project_logs:b667bd4c-7e55-49e0-a370-f6526e6bbccd
bt view logs [summary]: returned 1 row(s) (object_ref=project_logs:b667bd4c-7e55-49e0-a370-f6526e6bbccd, limit=1, preview_length=125)
  1. created=2026-05-01T20:48:39.870Z root_span_id=6304a076-193f-4bce-9eb9-b37ecf46c2e2 id=6d0e6f48-82f3-44a8-ade1-ec47a4c5b5d9 span_id=6304a076-193f-4bce-9eb9-b37ecf46c2e2 input={"q":"query 59"}

Truncated fields use preview_length=125.
next_cursor: afURqKAzADs
next: bt view logs --object-ref project_logs:b667bd4c-7e55-49e0-a370-f6526e6bbccd --cursor afURqKAzADs --non-interactive

The command to show the next page drops the --limit argument, showing 50 rows (the default).

Now does not drop the argument:

bt view logs --non-interactive --limit 1 --since 2026-05-01T00:00:00Z --object-ref project_logs:b667bd4c-7e55-49e0-a370-f6526e6bbccd
bt view logs [summary]: returned 1 row(s) (object_ref=project_logs:b667bd4c-7e55-49e0-a370-f6526e6bbccd, limit=1, preview_length=125)
  1. created=2026-05-01T20:48:39.870Z root_span_id=6304a076-193f-4bce-9eb9-b37ecf46c2e2 id=6d0e6f48-82f3-44a8-ade1-ec47a4c5b5d9 span_id=6304a076-193f-4bce-9eb9-b37ecf46c2e2 input={"q":"query 59"}

Truncated fields use preview_length=125.
next_cursor: afURqKAzADs
next: bt view logs --object-ref project_logs:b667bd4c-7e55-49e0-a370-f6526e6bbccd --cursor afURqKAzADs --non-interactive --limit 1

bt view trace has the same issue.

Part of #165.

@viadezo1er Cedric / ViaDézo1er (viadezo1er) merged commit 433f380 into main May 4, 2026
32 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