Skip to content

chore: apply project inspection report improvements - #35

Merged
mantoujun12 merged 2 commits into
mainfrom
pullfrog/34-inspection-report-improvements
Sep 29, 2026
Merged

mantoujun12 merged 2 commits into
mainfrom
pullfrog/34-inspection-report-improvements

Conversation

@pullfrog

@pullfrog pullfrog Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

概述

响应 #34(项目检查报告 2026-09-29)中的建议改进项,落地其中可操作改进,并说明暂不处理项的原因。

已实施

建议项 改动
2. OG 分享图偏小 新增程序生成的 1200×630 专用分享图 docs/public/og-image.png(深色渐变 + 站点品牌靛蓝配色 + 「25计算机1知识库」标题 + 域名胶囊),config.mts 的 og:image / twitter:image 由 favicon.png 切换为该图,补充 og:image:width/height,并将 twitter:card 升级为 summary_large_image
5. 重复 frontmatter docs/class/7s-original.md 的 title/description/keywords 差异化为「原始版存档」表述,与修订版 7s.md 区分,便于搜索区分两个页面
3. SEO 静态文件手工维护 docs/guide/contributing.md 「新增页面」清单新增第 5 步,提醒同步维护 docs/public/ 下的 sitemap.xml、rss.xml、llms.txt,删除或重命名页面时同样需要清理

按评审意见撤回

  • 4. pnpm 版本差异:评审认为项目为网站类,不使用 corepack 管理 pnpm,对版本不严格,无需锁定版本号。已移除 package.json 的 packageManager 字段,并将 ci.yml 恢复为原先的 pnpm-version matrix 与 version: 输入写法。

暂不处理(附原因)

  • 1. VitePress alpha 版本:属观察项,暂无可执行变更;待稳定版发布后再评估升级。
  • 6. pullfrog.yml 供应商密钥:该文件头部标注 DO NOT EDIT EXCEPT WHERE INDICATED,且当前运行正是经由该工作流(实际生效的密钥无法从仓库内验证)。裁剪环境变量存在破坏后续 agent 运行的风险,建议由管理员在确认仓库 Settings 中实际配置的密钥后自行删减。

验证

  • pnpm docs:build 通过(1.74 s,无死链告警)
  • package.json 与 ci.yml 已恢复与 main 一致;其余页面改动不变

Closes #34

Pullfrog  | View workflow run | via Pullfrog | Using GLM Flash | 𝕏

- add 1200x630 og-image.png and switch og/twitter meta to it
- lock pnpm via packageManager and let ci.yml defer to it
- differentiate 7s-original frontmatter from 7s
- document manual sync of public sitemap/rss/llms files
@pullfrog
pullfrog Bot requested a review from mantoujun12 September 29, 2026 03:47

@mantoujun12 mantoujun12 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

项目为网站类,不使用corepack管理pnpm,对版本不严格,因此不需要锁定版本号。

@pullfrog

pullfrog Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor Author

已按评审意见处理 PR #35:

评审(pullrequestreview-5354559030)要求不锁定 pnpm 版本号。已撤回相关改动——移除 package.json 的 packageManager 字段,ci.yml 恢复原先 pnpm-version matrix 与 version: 输入写法,两文件与 main 一致。其余改动(OG 分享图、7s 原始版 frontmatter 差异化、contributing 静态文件指引)保持不变;PR 描述已同步更新。

验证:pnpm docs:build 通过(1.74 s,无死链告警)。提交 a4340ec 已推送到 pullfrog/34-inspection-report-improvements。

Task list (1/5 completed)

Pullfrog  | View workflow run | via Pullfrog | Using GLM Flash | 𝕏

@mantoujun12
mantoujun12 merged commit 807f779 into main Sep 29, 2026
2 checks passed
@mantoujun12
mantoujun12 deleted the pullfrog/34-inspection-report-improvements branch September 29, 2026 15:22
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.

项目检查报告(2026-09-29)

1 participant