Skip to content

fix: ロググループ・ストリームの全ページ取得に対応 - #9

Merged
myuron merged 1 commit into
mainfrom
fix/pagination-fetch-all-pages
Apr 14, 2026
Merged

myuron merged 1 commit into
mainfrom
fix/pagination-fetch-all-pages

Conversation

@myuron

@myuron myuron commented Apr 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • fetchLogGroups/fetchLogStreams が1ページ目しか取得していなかった問題を修正
  • ListLogGroupsPage/ListLogStreamsPage(NextToken対応)に切り替え
  • カーソルがリスト末尾に到達した時点で次ページを非同期追加取得(maybeFetchMore)

Test plan

  • 既存テスト全通過(go test ./...)
  • NextToken保存・nil時の挙動テスト追加
  • 追加ページのappendテスト追加
  • カーソル末尾到達時のfetchトリガーテスト追加
  • 実環境でストリームが多いロググループを選択し、スクロールで全件取得されることを確認

🤖 Generated with Claude Code

fetchLogGroups/fetchLogStreamsが1ページ目しか取得していなかったため、
ListLogGroupsPage/ListLogStreamsPage(NextToken対応)に切り替え。
カーソルがリスト末尾に到達した時点で次ページを非同期追加取得する。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@myuron
myuron merged commit ee106d1 into main Apr 14, 2026
@myuron
myuron deleted the fix/pagination-fetch-all-pages branch April 15, 2026 09:47
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.

1 participant