Skip to content

feat(douban): add short comments reader#1744

Draft
lenovobenben wants to merge 1 commit into
jackwener:mainfrom
lenovobenben:feat/douban-comments
Draft

feat(douban): add short comments reader#1744
lenovobenben wants to merge 1 commit into
jackwener:mainfrom
lenovobenben:feat/douban-comments

Conversation

@lenovobenben
Copy link
Copy Markdown
Contributor

Summary

  • add douban comments for reading short comments on movie, book, and music subjects
  • paginate comment pages up to the requested limit and return user, rating, vote, time, content, and permalink fields
  • add adapter coverage for command registration, comment normalization, host routing, and preserving a 100 item limit

Why

Douban search and subject commands expose entries, but there was no way to inspect the short comment discussion for a selected subject. Short comments are a lightweight way to understand reception across movie, book, and music pages.

Validation

  • npx vitest run --project adapter clis/douban
  • npm run build
  • npm run check:typed-error-lint
  • npm run check:silent-column-drop
  • git diff --check
  • node dist/src/main.js douban comments 26266893 --type movie --limit 100 -f json
  • node dist/src/main.js douban comments 38210508 --type book --limit 100 -f json
  • node dist/src/main.js douban comments 26812952 --type music --limit 100 -f json

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