Repository navigation
修复文章预览检查卡住，列表封面默认完整显示 - #93
Open
13304930782 wants to merge 5 commits into
Open
13304930782 wants to merge 5 commits into
13304930782 wants to merge 5 commits into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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
1 participant
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.
修复文章预览因样式请求挂起而为空、“运行发布前检查”一直禁用的问题；文档就绪即挂载，补充初始化超时和重试。列表未设置卡片取景时默认完整显示，手机宽度从 88px 增至 112px；保留已保存的取景并按缩放计算图片 sizes。
按用户明确要求，新增服务器直接拉源码部署入口
scripts/deploy-article-preview-source.sh main|reader。不需要下载部署包或发布 GitHub Release。在独立临时 clone 中安装锁定依赖、类型检查、单元测试并构建前端／SSR；主站更新 dist 和匹配 SSR，仅重启现有 mooncci-api，reader 仅更新静态资源。构建 V8 768 MiB 限制只作用于构建子 shell，PM2 配置不改。可用 RAM+swap 不足 1024 MiB 时停止。直接部署前核对 11 个已部署 PR92 后端文件；备份所有覆盖文件及完整 SSR；逐字节验证备份，拒绝并发变化，验收真实公开 SSR、资源和健康。失败自动恢复；保留业务源码、数据库、迁移、.env、uploads、worker 和原 PM2 参数。DEPLOY.md 记录该显式选择的源码流程。
验证：原修复提交的完整 CI、CodeQL 和 Chromium／WebKit 回归已全部通过。新增部署入口的 7 项本地回归通过（主站／reader、完整回滚、并发变化、校验破坏、后端差异、备份篡改、SSR 依赖／资源缺失），shell 语法与 Node 验收脚本语法通过，并已用真实已构建输出准备主站和 reader 的校验阶段；新增入口已纳入 CI，等待本次提交重新运行。未代为执行生产部署。
修复实际服务器 Python 3.6 兼容性：用 universal_newlines 替代 text，兼容目录复制／文件删除，UUID 替代 time_ns。新增 3 项兼容性回归，总计 10 项本地测试通过。仅更改部署 helper 和测试，应用／构建输入不变，允许经差异检查后复用 06b4be5 已构建输出继续部署，无须重装依赖或编译。未在真实 Python 3.6 解释器执行本地测试；生产失败日志显示出错于 prepare 阶段，尚未替换线上文件。
修复源码部署 SSR 预检误判：有封面文章使用 hero-band，无封面才使用 hero-heading。验收改为验证状态、公开文章数据及完整正文／标题，允许两种头部布局；公开页面也验证 SSR bootstrap 和正文，错误输出资源路径／HTTP 状态及堆栈。真实已构建 SSR 回归覆盖两种文章及首页／列表，缺正文、404、资源错误、缺文章数据均拒绝；部署 10 项回归通过。仅更新验收 helper 和测试，可复用原构建输出。
首页公开验收新增最多 20 秒的有限重试，以覆盖现有 s-maxage=15 缓存窗口；每次仍要求真实 script src 精确匹配新入口和 SSR 数据，持续不匹配仍失败回滚。日志记录预期／实际入口及 Age、Cache-Control、X-Home-Cache、节点／路由头，不记录 Cookie。新增 3 项回归（缓存过期成功、持续旧入口拒绝、SSR 缺失拒绝），与真实 SSR 回归及部署 10 项回归均通过。用户提供的回滚后诊断显示后端与 Nginx 正常返回同一旧入口；尚不能据此确定切换时失败原因。已验证旧离线包的等价修正，无需重新构建。