fix(feishu): support drive folder pagination#101572
Merged
Merged
Conversation
Co-authored-by: 张贵萍0668001030 <[email protected]>
Contributor
Author
|
Land-ready proof for exact head
No known code or CI blockers remain. Contributor credit is preserved in the co-authored commit and PR body. |
Contributor
Author
|
Merged via squash.
|
This was referenced Jul 7, 2026
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
Jul 8, 2026
Co-authored-by: 张贵萍0668001030 <[email protected]>
giodl73-repo
pushed a commit
to giodl73-repo/openclaw
that referenced
this pull request
Jul 8, 2026
Co-authored-by: 张贵萍0668001030 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What Problem This Solves
Feishu Drive folder listings return
next_page_token, but thefeishu_drivetool could not accept that cursor, leaving every page after the first unreachable.This is the clean current-main replacement for #101249 and #101567. The original contributor branch was 159 main commits and 903 files behind; the first replacement then inherited merge-base noise when the verified GitHub commit path appended its rebased tree. This branch contains exactly the four intended Feishu files in one co-authored commit.
Why This Change Was Made
page_sizeandpage_tokeninputs to the existinglistaction.page_sizeat runtime, not only in the model schema."0"folder tokens.The change stays plugin-local and preserves the existing response shape. It does not broaden into automatic recursive pagination or the separate
infolookup behavior tracked by #93928.User Impact
Agents can retrieve subsequent pages of a shared Feishu Drive folder by passing the returned
next_page_tokenaspage_tokenwith the samefolder_token. Invalid page sizes fail before an API request.Evidence
@larksuiteoapi/node-sdksource confirmsdrive.file.listacceptsfolder_token,page_size, andpage_token, and returnsnext_page_token/has_more./tools/invokethrough the bundled tool and captured the expected SDK-boundary continuation request.cbx_5bcffb4b507b, runrun_5b84b967a00e; 20/20 Feishu Drive tests passed.oxfmton all touched files;git diff --check; skill YAML frontmatter parse.Closes the
listcontinuation portion of #93928. Supersedes #101249 and #101567.