Skip to content

解决CLI某页面过程中遇到的重型SPA超时问题,然后修复了几个小问题(流水线失败看起来像是E2E的环境问题)#793

Open
YoungCan-Wang wants to merge 12 commits intojackwener:mainfrom
YoungCan-Wang:main
Open

Conversation

@YoungCan-Wang
Copy link
Copy Markdown
Contributor

  • P0 navigateBefore 选项对象 — 解决实际痛点(重型 SPA 超时),改动集中且向后兼容
  • P1 Error 子类 — 填补 icons 和 class 的已有缺口,但不影响现有功能
  • P2 evaluate() 类型对齐 — 3 行改动,修的是语义不一致,不是 bug
  • P2 evaluateFetch() — 95 行新增代码,但目前没有任何现有适配器在用它,是"空投"的工具函数,不清楚各站点开发节奏,不敢全量修改 怕与其他人冲突。

YoungCan-Wang and others added 10 commits April 4, 2026 15:16
feat: support image replies in twitter reply
…dd size limit

- Fix attachReplyImage fallback: use uploaded flag instead of checking
  page.setFileInput existence, so base64 fallback actually runs when
  CDP setFileInput throws "Unknown action"
- Restore execCommand('insertText') as primary text input method for
  Twitter's Draft.js editor, with paste event as fallback
- Add 20MB size limit for remote image downloads to prevent OOM
- Remove unsafe buttons[0] fallback that could click invisible buttons
…rning

Local images were not validated for size — a 100MB file would fail only
at upload time. Remote images already had MAX_IMAGE_SIZE_BYTES checks.
Also add a console.warn when using the base64 fallback with large
payloads, consistent with xiaohongshu/publish.ts behavior.
…rror 子类。适配器遇到这三类错误只能 throw new

  Error(...),CLI 渲染器无法匹配到正确图标,全部显示为通用的 💥。
…valuateFetch() 工具函数,目前每个站点各自实现,功能重叠但质量参差。提取为共享工具
@YoungCan-Wang
Copy link
Copy Markdown
Contributor Author

大佬 这个是流水线失败了?

@YoungCan-Wang
Copy link
Copy Markdown
Contributor Author

大佬 看起来是E2E 环境问题

@YoungCan-Wang YoungCan-Wang changed the title 解决CLI某页面过程中遇到的重型SPA超时问题,然后修复了几个小问题 解决CLI某页面过程中遇到的重型SPA超时问题,然后修复了几个小问题(流水线失败看起来像是E2E的环境问题) Apr 5, 2026
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