fix(xiaohongshu+rednote/search): fall back to href-based note cards when section.note-item class is dropped

[Benjamin-eecs]

Description

Issue #1506 reports opencli xiaohongshu search returning [] even though the search results page visibly has notes. Trace evidence in the issue: xhs ships a render variant where each note card is a bare <section> with no note-item class, so the three section.note-item selectors in clis/xiaohongshu/search.js all match zero elements.

The three call sites in the shared search IIFEs (WAIT_FOR_CONTENT_JS, buildScrollUntilJs, buildSearchExtractJs) now use the same defensive strategy: try the legacy section.note-item class first, then fall back to any <section> that wraps a /search_result/... or /explore/... link. The change is in the xiaohongshu file so the rednote adapter (which imports buildSearchExtractJs and buildScrollUntilJs from here) picks it up automatically.

Extraction-side title selector also gains a fallback: when no .title / .note-title element matches, read the first <span> inside the search-result link, which is where the bare-section render puts the caption per the trace.

Related issue: #1506. Also incidentally addresses the EMPTY_RESULT symptom reported in #1500 (autofix's "page.evaluate envelope" diagnosis is incorrect; rednote uses the same code path and returns proper arrays without unwrapping).

Type of Change

• 🐛 Bug fix
• ✨ New feature
• 🌐 New site adapter
• 📝 Documentation
• ♻️ Refactor
• 🔧 CI / build / tooling

Checklist

• I ran the checks relevant to this PR
• I updated tests or docs if needed
• I included output or screenshots when useful

Documentation (if adding/modifying an adapter)

• Added doc page under docs/adapters/ (if new adapter)
• Updated docs/adapters/index.md table (if new adapter)
• Updated sidebar in docs/.vitepress/config.mts (if new adapter)
• Updated README.md / README.zh-CN.md when command discoverability changed
• Used positional args for the command's primary subject unless a named flag is clearly better
• Normalized expected adapter failures to CliError subclasses instead of raw Error

(Selector logic only. Command surface and error contract unchanged.)

Screenshots / Output

npx vitest run --project adapter clis/xiaohongshu/: 105/105 green (existing test suite unchanged, passes on both legacy and fallback paths).

Live verify on rednote (same imported code path, account-safe to test):

$ opencli rednote search "美食" --limit 3 -f json
[
  { "rank": 1, "title": "在朋友家吃过一次，被惊艳到了！",      "author": "小明教做菜",   "likes": "2344", ... },
  { "rank": 2, "title": "我的15💰晚餐｜一碗南瓜焖饭",         "author": "下班吃点啥",   "likes": "2万",  ... },
  { "rank": 3, "title": "干净饮食🫛牛肉炒口蘑豌豆米饭拼盘🍳", "author": "豆豆子",       "likes": "1.2万", ... }
]

Legacy section.note-item path is exercised here (rednote still renders the class). The fallback is dormant on rednote but selectable through the same :has() query if rednote's DOM follows xhs's lead.

Live verify on xiaohongshu was not performed: the test machine has no logged-in xhs session, and the project's operational guidance flags xhs as account-ban-sensitive. The fix is structural; the bare <section> shape the issue reporter traced is reachable through the new fallback, and the existing tests keep the legacy path green.

npx tsc --noEmit clean. npm run build 815 manifest entries, unchanged shape. silent-column-drop / typed-error-lint baselines unchanged.

Closes #1506
Refs #1500

[Copilot]

Pull request overview

This PR hardens the xiaohongshu search DOM extraction logic so it can still detect and extract note cards when XHS drops the legacy section.note-item class (as reported in #1506), and improves title extraction for the new “bare <section>” render shape.

Changes:

• Adds a fallback note-card detection strategy based on <section> elements that contain /search_result/ or /explore/ links (used in wait, scroll-until, and extraction paths).
• Adds a title fallback that reads the first <span> inside the detail link when class-based title selectors don’t match.
• Updates extraction loop control flow to work with the new fallback card collection logic.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[jackwener]

深度 review 完毕，合入 ✅。

第一性原理 review

• 触发条件确凿：issue [BUG] xiaohongshu search returns empty array due to outdated DOM selectors #1506 现场抓到 xhs 把 section.note-item class drop 掉。defensive selector fallback 是正解。
• 三层 selector 一致回退：WAIT_FOR_CONTENT_JS / buildScrollUntilJs.collectNoteCards / buildSearchExtractJs.collectNoteCards 都按"class first → fallback to section:has(a[href*=/search_result/]) / section:has(a[href*=/explore/])"同一规则，没漏点。
• .forEach(el => return) → for (const el of ...) { continue }：必要重写，因为 collectNoteCards() 现在返回 NodeList 或 Set，forEach 在 Set 上是 OK 但 .return 早退语义不同；改成 for...of + continue 顺手统一两种容器的退出语义，没引入 silent skip。
• Title fallback：<section> 裸 render 下，note caption 落在 search_result anchor 的首个 <span> 里。titleEl?.textContent || '' 空时再去捞 detailLinkEl?.querySelector('span') 是合理的二层兜底；不会把 author / likes 当 title 误抓（搜索器固定）。
• el.classList?.contains optional chain：fallback 路径下 <section> 不一定带 classList getter，加 ?. 防 TypeError。
• 没有引入新 audit 违规：typed-error-lint 189/189，silent-column-drop 103/103。tests 125/125（xiaohongshu + rednote）。

Nitpick（不阻塞 merge）

两份 collectNoteCards 内联 closure 完全相同（19 行 × 2）。是 ≥2 sibling 抽 utils.js 的候选；但 #1391 设的 lazy-extract 阈值是"出现第 3 个 sibling 才抽"，目前 2 个内联是可接受边界，留 follow-up 不阻塞。

合入：squash → 2babed84e9539cb5c6f3389858e2d55b378ba102。