Skip to content

[codex] Fix Xiaohongshu search navigation#2011

Draft
ppop123 wants to merge 1 commit into
jackwener:mainfrom
ppop123:codex/fix-xhs-search-route
Draft

[codex] Fix Xiaohongshu search navigation#2011
ppop123 wants to merge 1 commit into
jackwener:mainfrom
ppop123:codex/fix-xhs-search-route

Conversation

@ppop123

@ppop123 ppop123 commented Jun 24, 2026

Copy link
Copy Markdown

Summary

  • submit Xiaohongshu searches through the visible web search box instead of direct-linking the search results URL
  • wait until the page reaches search_result / search_result_ai before extracting note cards
  • surface Xiaohongshu security restriction pages (安全限制, 300011, Account abnormal) as typed command errors instead of empty results

Root Cause

xiaohongshu/search navigated directly to /search_result?...source=web_search_result_notes. In current Xiaohongshu web sessions that route can redirect to /website-login/error with 300011 Account abnormal, while manual searches from the web UI still work. The adapter also treated non-result pages as empty data, so callers saw (no data) instead of a clear failure.

Validation

  • npx vitest run --project adapter clis/xiaohongshu/search.test.js
  • npm run typecheck

Local Live Check

Before preparing this PR, I verified the same change against a logged-in local Chrome/OpenCLI Adapter session with:

opencli xiaohongshu search "增额寿险靠谱吗" --limit 5

The command returned relevant search results instead of (no data).

wearzdk added a commit to wearzdk/OpenCLI that referenced this pull request Jun 26, 2026
wearzdk added a commit to wearzdk/OpenCLI that referenced this pull request Jun 26, 2026
从 jackwener/opencli 123 个开放 PR 中筛选并入与本产品(内容发布)相关的:
- B 核心/daemon/扩展/pipeline 可靠性 23 个
- A twitter/小红书/公众号/tiktok 发布写修复 7 个
- C instagram/linkedin 内容平台读取增强 3 个
跳过 jackwener#2026 jackwener#1960(已被我们 d4e65ab + jackwener#2011 + rednote 适配器取代)。
本地化解决了 4 处冲突,删除 jackwener#1972 冗余 per-command strip。
全量测试 5808 passed,无新增失败(6 个 cli.test.ts 浏览器/守护进程用例为合并前既有、环境相关)。
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