RSSHub icon indicating copy to clipboard operation
RSSHub copied to clipboard

twitter 的 list timeline 因为过滤了 conversation 导致可能有推文丢失

Open winoros opened this issue 4 months ago • 2 comments

路由地址

/twitter/list/:id/:routeParams?

完整路由地址

任意一个包含了对话的 list

相关文档

https://docs.rsshub.app/routes/social-media#list-timeline

预期是什么?

我看了下代码,只要在 gatherLegacyFromData 里加一个 listConversation 的 filter 传参就可以了,不过要不要和现有的参数 includeReplies 结合起来用传参控制呢

实际发生了什么?

这个主要问题,先不说跨用户的 reply,哪怕是本人对自己原推的补充,都会使原推和这个追加对话合进一个 listConversation 里导致某个原推再也不可能在解析的 rss 流里出现了。。还是稍微有些危险的

部署

自建

部署相关信息

No response

额外信息

因为也不难改,所以虽然我不太懂 ts 但也能改,主要是行为上要不要受 `includeReplies` 控制

这不是重复的 issue

  • [x] 我已经搜索了 现有 issue,以确保该错误尚未被报告。

winoros avatar Aug 10 '25 05:08 winoros

Searching for maintainers:
  • /twitter/list/:id/:routeParams?: @DIYgod @xyqfer @pseudoyu

To maintainers: if you are not willing to be disturbed, list your username in scripts/workflow/test-issue/call-maintainer.js. In this way, your username will be wrapped in an inline code block when tagged so you will not be notified.

If all routes can not be found, the issue will be closed automatically. Please use NOROUTE for a route-irrelevant issue or leave a comment if it is a mistake. 如果所有路由都无法匹配,issue 将会被自动关闭。如果 issue 和路由无关,请使用 NOROUTE 关键词,或者留下评论。我们会重新审核。

github-actions[bot] avatar Aug 10 '25 05:08 github-actions[bot]

Hi @winoros! I'm Dosu and I’m helping the RSSHub team.

目前 /twitter/list/:id/:routeParams? 路由没有提供用户控制 conversation(如 listConversation)过滤的参数,includeReplies 也只在 user timeline 路由支持,文档和代码都没有为 list timeline 实现相关逻辑 1 2

你可以参考 user timeline 的实现方式,在 list timeline 的 handler 里解析一个新的参数(比如 includeConversations 或直接复用 includeReplies),然后把它作为 filterNested 传给 gatherLegacyFromData,比如 ['listConversation']。这样就能让用户决定是否过滤 conversation,避免推文丢失 3

建议不要直接复用 includeReplies,因为它在 list 路由并未文档化,容易让用户混淆。可以单独加一个参数,文档里也要说明清楚行为和区别。

如需代码参考,可以看 user timeline 路由如何传递 filterNested 给 gatherLegacyFromData,然后在 list timeline 路由做类似处理即可。这样扩展起来很简单,代码结构也支持这种做法。

To reply, just mention @dosu.


How did I do? Good | Irrelevant | Incorrect | Verbose | Hallucination | Report 🐛 | Other  Chat with Dosu Join Discord Share on X

dosubot[bot] avatar Aug 10 '25 05:08 dosubot[bot]