Skip to content

Add jefere9901/rss-feed-reader#1831

Open
jefere9901 wants to merge 1 commit into
siyuan-note:mainfrom
jefere9901:main
Open

Add jefere9901/rss-feed-reader#1831
jefere9901 wants to merge 1 commit into
siyuan-note:mainfrom
jefere9901:main

Conversation

@jefere9901
Copy link
Copy Markdown

@jefere9901 jefere9901 commented May 7, 2026

If you are submitting a new bazaar package for the first time, please confirm the following information.

  • The repository is public
  • Include the appropriate open source license file LICENSE
  • Does not involve infringing content, such as non-commercial font files

If your marketplace package is closed-source, please grant @TCOTC, @88250, and @Vanessa219 access to the source code so that we can complete the review.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

Pull Request Check Report

Add 1 Plugin Repo

jefere9901/rss-feed-reader


@TCOTC TCOTC changed the title Add rss-feed-reader Add jefere9901/rss-feed-reader May 7, 2026
@TCOTC
Copy link
Copy Markdown
Collaborator

TCOTC commented May 7, 2026

压缩包必须命名为 package.zip

@jefere9901
Copy link
Copy Markdown
Author

jefere9901 commented May 8, 2026 via email

@TCOTC TCOTC added the Check Run PR Check workflow label May 8, 2026
@github-actions github-actions Bot removed the Check Run PR Check workflow label May 8, 2026
@TCOTC
Copy link
Copy Markdown
Collaborator

TCOTC commented May 8, 2026

初步审核

以下是 AI 的审核结论,请开发者确认并修复之后回复,然后维护者会进行人工审核。

注意:审核基于最新源代码(main 分支 commit a7605a0),而非 v1.0.1 发布包。

版本: v1.0.1 (commit: 20875e0) / 最新源码: a7605a0

发现的问题

  • 缺少 LICENSE 文件 — 仓库根目录没有 LICENSE 文件,必须在仓库中添加合适的开源许可证。
  • 缺少 preview.png(CI 已标记)— 需要在仓库和 package.zip 中添加预览截图。
  • README 中包含相对路径图片引用README.mdREADME_en.md 中使用了相对路径的 HTML <img> 标签,应将需要展示的图片上传到 GitHub raw 或 CDN,使用绝对 https:// URL:
    • <img src="rss-feed-screen.png">
    • <img src="icon.png">
    • <img src="wx.jpg">
    • <img src="zfb.jpg">
  • funding 字段为空plugin.json"funding": {"custom": []}custom 为空数组,应删除整个 funding 字段。
  • 缺少 disabledInPublish 字段 — 插件通过 saveData() 存储 AI API 密钥等敏感数据,应在 plugin.json 中添加 "disabledInPublish": true
  • 缺少 onunload() 生命周期方法 — 当前 src/index.ts 没有 onunload(),导致以下运行时资源无法释放:
    • dock.ts 中通过 new MutationObserver(...) 创建的 DOM 属性监听器(应在 onunload() 中调用 .disconnect()
  • 使用 index.css 文件 — 当前在 onload() 中通过 document.head.appendChild(style) 添加 <style> 元素,应将样式打包为 index.css 由思源自动加载而不是插件手动插入
  • 缺少 uninstall() 生命周期方法 — 插件通过 saveData("rss-feed-reader-data", ...) 存储数据到 petal,但从未调用 removeData()。完全卸载时 petal 中的数据不会被清理。应在 uninstall() 中添加 this.removeData("rss-feed-reader-data")
  • 热键格式不符合思源规范src/index.ts:42 使用 "Ctrl+Shift+R",应改为思源标准格式如 "⌃⇧R"
  • reader-view.ts 中全局 Escape 键监听器未清理 — 第 141 行 document.addEventListener("keydown", ...) 使用 {once: true},但如果用户通过点击遮罩关闭阅读器(而非按 Escape),该监听器会滞留,闭包持有整个 ReaderView 实例导致内存泄漏。应在 close() 方法中移除该监听器。
  • .gitignore 中出现乱码 — 文件末尾存在编码损坏的行,可能导致 git 规则失效。

需要维护者人工验证

  • 安装并测试所有功能:订阅添加/删除、文章阅读/下载、OPML 导入导出、文件夹管理
  • AI 摘要/翻译/问答功能(需新版发布后测试,当前 v1.0.1 package.zip 不包含 AI 模块)
  • 暗色/亮色主题切换、折叠面板、数据重置
  • 验证所有声明的前端(desktop、browser-desktop、mobile)和平台(windows、linux、darwin、docker、android、ios)兼容性
  • 验证插件重载/更新周期是否正常,关闭/重开插件后数据是否保留
  • 验证反限制模式(bypass paywall)在不同网站的抓取效果

检查通过

  • icon.png 大小 7.9KB(< 20KB)
  • 使用自定义 SVG 图标 iconRSS(非思源内置图标)
  • CSS 无模板残留样式(无 .plugin-sample
  • 所有路径使用 / 分隔符
  • window.location.reload() 调用
  • 无 i18n 混合问题(插件不使用 i18n 模块)
  • package.zip 中无多余文件(node_modules、.git 等)
  • loadData() 仅在初始化时调用一次
  • console.log 仅在生命周期/初始化函数中使用
  • 无 EventBus 监听器需要清理
  • 关键词不含 "siyuan"
  • displayName 不含 "Siyuan"
  • name 与仓库名一致
  • readme 字段与实际文件匹配
  • package.zip 内部路径全部使用 /
  • 仓库 plugin.json 已添加 networks 白名单(为 AI 功能准备)

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