feat(v2ex): add node, user, member, replies, nodes commands#282
Merged
jackwener merged 7 commits intojackwener:mainfrom Mar 23, 2026
Merged
feat(v2ex): add node, user, member, replies, nodes commands#282jackwener merged 7 commits intojackwener:mainfrom
jackwener merged 7 commits intojackwener:mainfrom
Conversation
Contributor
Author
|
Note: CI failures ( |
Add 5 new public API commands to the v2ex adapter: - node: browse topics by node name - user: list topics by username - member: show user profile - replies: list topic replies - nodes: list all nodes sorted by topic count All commands use strategy: public, browser: false.
…ke tests - Replace isExpectedChineseSiteRestriction with if(code===0) soft guard (V2EX is globally accessible; YAML fetch doesn't throw FETCH_ERROR) - Add value assertions: member username===Livid, limit effectiveness - Add smoke tests for node, member, replies, nodes commands
51e44dd to
2bde569
Compare
…e test - Add url to node.yaml and user.yaml pipeline map steps and columns (V2EX API provides item.url; improves usability for follow-up lookups) - Add v2ex user smoke test (other 4 new commands all had smoke tests; user was missing) - Update E2E assertions to verify url field in node/user results
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add 5 new public API commands to the V2EX adapter, utilizing previously unused V1 API endpoints. All commands are
strategy: public+browser: false— no login or browser required.v2ex node <name>/api/topics/show.json?node_name=v2ex user <username>/api/topics/show.json?username=v2ex member <username>/api/members/show.json?username=v2ex replies <id>/api/replies/show.json?topic_id=v2ex nodes/api/nodes/all.jsonType of Change
Checklist
Documentation (if adding/modifying an adapter)
docs/adapters/(if new adapter) — updated existing v2ex pagedocs/adapters/index.mdtable (if new adapter)docs/.vitepress/config.mts(if new adapter) — N/A, v2ex entry already existsScreenshots / Output