Skip to content

fix: classify xianyu item auth and blocked states#726

Merged
jackwener merged 2 commits intomainfrom
fix/xianyu-item-auth-and-blocked-state
Apr 3, 2026
Merged

fix: classify xianyu item auth and blocked states#726
jackwener merged 2 commits intomainfrom
fix/xianyu-item-auth-and-blocked-state

Conversation

@jackwener
Copy link
Copy Markdown
Owner

Description

This follow-up PR restores the xianyu item auth/risk-control classification fix that did not land in #696 and also updates the test mock to match the current IPage interface so typecheck stays green.

What changed

  • classify login walls as AuthRequiredError before waiting on window.lib.mtop
  • classify verification/risk-control pages as EmptyResultError
  • keep true mtop-not-ready failures as SelectorError
  • align the xianyu item test page mock with the current IPage contract

Verification

  • npm run test:adapter -- src/clis/xianyu/item.test.ts src/clis/xianyu/search.test.ts src/clis/xianyu/chat.test.ts
  • npm run typecheck

Related: #696

@jackwener jackwener closed this Apr 3, 2026
@jackwener jackwener reopened this Apr 3, 2026
@jackwener jackwener merged commit fc818b3 into main Apr 3, 2026
11 checks passed
@jackwener jackwener deleted the fix/xianyu-item-auth-and-blocked-state branch April 3, 2026 04:50
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