Skip to content

[codex] fix xiaohongshu download image order#1515

Draft
Scofy0123 wants to merge 1 commit into
jackwener:mainfrom
Scofy0123:codex/xiaohongshu-download-display-order
Draft

[codex] fix xiaohongshu download image order#1515
Scofy0123 wants to merge 1 commit into
jackwener:mainfrom
Scofy0123:codex/xiaohongshu-download-display-order

Conversation

@Scofy0123
Copy link
Copy Markdown

Summary

Root Cause

xiaohongshu/download previously collected images from carousel-related DOM selectors and stored them in a Set. That preserved DOM discovery order, not necessarily the order shown in the platform carousel. Hidden slides, preloaded slides, duplicated carousel nodes, or responsive rendering can make the visible cover appear after another DOM image, causing filenames like <noteId>_1.jpg to be misleading.

The shared downloadMedia helper simply names files by the media array index it receives, so the adapter needs to pass media in platform display order.

Validation

  • npm run test:adapter -- clis/xiaohongshu/download.test.js
  • npm run test:adapter -- clis/xiaohongshu/*.test.js
  • npm run typecheck

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] xiaohongshu download may save carousel images in DOM order instead of display order

1 participant