Skip to content

fix(douban): classify tv search results correctly#979

Merged
jackwener merged 1 commit intojackwener:mainfrom
Astro-Han:fix/douban-search-type
Apr 13, 2026
Merged

fix(douban): classify tv search results correctly#979
jackwener merged 1 commit intojackwener:mainfrom
Astro-Han:fix/douban-search-type

Conversation

@Astro-Han
Copy link
Copy Markdown
Contributor

Description

Fix douban search so TV series results returned from movie search are classified as tvshow instead of always inheriting the search type movie.

Related issue: #978

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

Screenshots / Output

Relevant verification:

node dist/src/main.js validate
# PASS

npx tsc --noEmit
# PASS

npx vitest run --project adapter clis/douban/*.test.js
# PASS

Note: npm test currently fails on an unrelated existing adapter test:

clis/zsxq/topic.test.js > zsxq topic command > maps topic detail 404 responses to NOT_FOUND before fetching comments
TypeError: Cannot read properties of undefined (reading 'status')

@jackwener jackwener force-pushed the fix/douban-search-type branch from 2b2b91c to 8deb88b Compare April 13, 2026 00:37
@jackwener jackwener merged commit 5dcbf92 into jackwener:main Apr 13, 2026
11 checks passed
@Astro-Han Astro-Han deleted the fix/douban-search-type branch April 13, 2026 10:18
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.

2 participants