Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…capture complete review pages
…request filtering
ECC Tools / Security EvidenceCommit: Security evidence gate passed (success) No security-sensitive scanner-evidence gap detected. Mode: enforce Scanned 60 changed file(s). No missing scanner-evidence signal was detected. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Risk TaxonomyCommit: PR taxonomy review recommended (neutral) Detected 2 PR taxonomy bucket(s): Security Evidence, CI/CD Recommendation. Scanned 60 changed file(s). Roadmap taxonomy buckets: Security EvidenceSecurity-sensitive changes should carry explicit scanner, code-scanning, or focused regression evidence. Signals:
Paths:
CI/CD RecommendationCI, dependency, coverage, and contract signals should be routed into follow-up checks or verification work. Signals:
Paths:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Reference Set ReadinessCommit: Reference set readiness gaps detected (neutral) Reference evidence present for 1/7 areas (14%) across 60 changed file(s). This check is based on files changed in this PR. Repository-level readiness is still reported by
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Hosted Promotion ReadinessCommit: Hosted promotion readiness passed (success) No hosted promotion evidence gaps detected across 60 changed file(s); 0 corpus scenarios had matching evidence. This check compares PR file changes against the evaluator/RAG promotion corpus in No evaluator corpus scenarios matched this PR. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Config AuditCommit: No changed-config issues detected (success) Scanned 1 config file(s) present at this commit across 1 changed config path(s) and found no issues in the supported security rules. Changed config files:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Harness AuditCommit: No harness issues detected (success) Scanned 1 changed config file(s) and found no harness issues. Changed config files:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Security EvidenceCommit: Security evidence gate passed (success) No security-sensitive scanner-evidence gap detected. Mode: enforce Scanned 60 changed file(s). No missing scanner-evidence signal was detected. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Risk TaxonomyCommit: PR taxonomy review recommended (neutral) Detected 2 PR taxonomy bucket(s): Security Evidence, CI/CD Recommendation. Scanned 60 changed file(s). Roadmap taxonomy buckets: Security EvidenceSecurity-sensitive changes should carry explicit scanner, code-scanning, or focused regression evidence. Signals:
Paths:
CI/CD RecommendationCI, dependency, coverage, and contract signals should be routed into follow-up checks or verification work. Signals:
Paths:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Reference Set ReadinessCommit: Reference set readiness gaps detected (neutral) Reference evidence present for 1/7 areas (14%) across 60 changed file(s). This check is based on files changed in this PR. Repository-level readiness is still reported by
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Hosted Promotion ReadinessCommit: Hosted promotion readiness passed (success) No hosted promotion evidence gaps detected across 60 changed file(s); 0 corpus scenarios had matching evidence. This check compares PR file changes against the evaluator/RAG promotion corpus in No evaluator corpus scenarios matched this PR. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Config AuditCommit: No changed-config issues detected (success) Scanned 1 config file(s) present at this commit across 1 changed config path(s) and found no issues in the supported security rules. Changed config files:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Harness AuditCommit: No harness issues detected (success) Scanned 1 changed config file(s) and found no harness issues. Changed config files:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Security EvidenceCommit: Security evidence gate passed (success) No security-sensitive scanner-evidence gap detected. Mode: enforce Scanned 60 changed file(s). No missing scanner-evidence signal was detected. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Risk TaxonomyCommit: PR taxonomy review recommended (neutral) Detected 2 PR taxonomy bucket(s): Security Evidence, CI/CD Recommendation. Scanned 60 changed file(s). Roadmap taxonomy buckets: Security EvidenceSecurity-sensitive changes should carry explicit scanner, code-scanning, or focused regression evidence. Signals:
Paths:
CI/CD RecommendationCI, dependency, coverage, and contract signals should be routed into follow-up checks or verification work. Signals:
Paths:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Reference Set ReadinessCommit: Reference set readiness gaps detected (neutral) Reference evidence present for 1/7 areas (14%) across 60 changed file(s). This check is based on files changed in this PR. Repository-level readiness is still reported by
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Hosted Promotion ReadinessCommit: Hosted promotion readiness passed (success) No hosted promotion evidence gaps detected across 60 changed file(s); 0 corpus scenarios had matching evidence. This check compares PR file changes against the evaluator/RAG promotion corpus in No evaluator corpus scenarios matched this PR. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Config AuditCommit: No changed-config issues detected (success) Scanned 1 config file(s) present at this commit across 1 changed config path(s) and found no issues in the supported security rules. Changed config files:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Harness AuditCommit: No harness issues detected (success) Scanned 1 changed config file(s) and found no harness issues. Changed config files:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Security EvidenceCommit: Security evidence gate passed (success) No security-sensitive scanner-evidence gap detected. Mode: enforce Scanned 60 changed file(s). No missing scanner-evidence signal was detected. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Risk TaxonomyCommit: PR taxonomy review recommended (neutral) Detected 2 PR taxonomy bucket(s): Security Evidence, CI/CD Recommendation. Scanned 60 changed file(s). Roadmap taxonomy buckets: Security EvidenceSecurity-sensitive changes should carry explicit scanner, code-scanning, or focused regression evidence. Signals:
Paths:
CI/CD RecommendationCI, dependency, coverage, and contract signals should be routed into follow-up checks or verification work. Signals:
Paths:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Reference Set ReadinessCommit: Reference set readiness gaps detected (neutral) Reference evidence present for 1/7 areas (14%) across 60 changed file(s). This check is based on files changed in this PR. Repository-level readiness is still reported by
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Hosted Promotion ReadinessCommit: Hosted promotion readiness passed (success) No hosted promotion evidence gaps detected across 60 changed file(s); 0 corpus scenarios had matching evidence. This check compares PR file changes against the evaluator/RAG promotion corpus in No evaluator corpus scenarios matched this PR. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Config AuditCommit: No changed-config issues detected (success) Scanned 1 config file(s) present at this commit across 1 changed config path(s) and found no issues in the supported security rules. Changed config files:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Harness AuditCommit: No harness issues detected (success) Scanned 1 changed config file(s) and found no harness issues. Changed config files:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
维护者边界
本 PR 只交付待维护者审查的改版。不得合并，不开启自动合并，不部署。最终由维护者查看截图和检查结果后决定。功能检查通过不等于所有性能目标已经稳定达到。
改版范围
保留 Astro、Lora 与 Mochi、暖色纸张视觉。重构中英文首页、作品页、四个项目案例、实验室、关于、联系、正在构建与收藏。文章列表默认展开，增加配套阅读资料，保留旧文章地址、原文语言、项目锚点、搜索、简历和完整仓库档案。
来源选择与验收规则是可操作的固定教学样例，不声称是真实模型运行。案例说明依据已读取的仓库 README 整理。Glassbox 使用画布工作台定位。现有原创角色图和 Zhihu Threads 双语图片继续使用，没有重新生成整套角色或伪造产品截图。没有手工修改 GitHub 同步快照。
2026 年 9 月 10 日验收补修
基线提交 24e2bef 的阅读检查有两项失败，分别发生在正常动态和减少动态模式。手机目录的焦点处理漏掉原生 summary，并把关闭的 details 内部链接误算为可聚焦控件。
本轮只修改 src/layouts/ContentLayout.astro 与 scripts/test-studio-reading.mjs。纳入原生折叠控件，排除隐藏、禁用和负 tabindex 控件，修复正向与反向 Tab 循环。目录打开后隔离背景交互，关闭时恢复各元素原有 inert 状态。选择章节后把焦点交给真实标题，切换到桌面宽度时移除遮罩与模态状态，并处理移动关闭按钮消失后的焦点。
扩展英文用例时，曾错误假设英文目录也有 summary。核对实际 DOM 后，分别验证中文折叠目录与英文直列目录。每组重新加载页面，避免前一项失败传递到下一项。没有删除原有回归或把误报算成网站缺陷。
最终版本与工作流
最终 head 为 662d3c6。
GitHub Actions 使用临时预览提交 5f40400。通过 Git Data API 核对，该提交与 head 的树完全一致，都是 b128fd493747fd61ae322e8b11124b4ce472ff62。临时 CI 预览不表示已经合并到 main。
原有回归工作流已成功完成：https://github.com/lora-sys/loraSys/actions/runs/34430253529
语言、改版、阅读及性能审计工作流已成功完成：https://github.com/lora-sys/loraSys/actions/runs/34430253500
锁定依赖安装、同步快照校验、Astro 检查和生产构建通过。检查结果如下：
静态扫描覆盖 93 个 HTML、4800 个站内引用和 287 个语言替代引用，未发现已定义检查错误。改版专项访问 30 条路径，分别使用 360、390、768、1440 像素宽度，共 120 个页面场景。不同检查组分别覆盖中英文、深浅主题、正常动态和减少动态，不把这些数量描述为互不重复的独立测试。
截图证据
浏览器使用 GitHub Actions 中的完整 Chromium，读取当前 PR 的生产构建。全页及阅读检查记录的浏览器版本为 145.0.7632.6。截图不是线上发布效果，也不是设计稿。
两个最终产物共保留 330 张原始 PNG。本次另整理 142 张截图为独立 HTML 图库，可按页面、宽度、语言、主题及状态筛选，点击查看完整图片。原始尺寸和 SHA256 记录在图库清单中。图库和经过筛选的证据包在对话附件交付，不加入站点生产代码。
审计产物 ID 为 10134328237，回归产物 ID 为 10134244848，可在上述工作流查看。人工检查了重点页面的实际截图，包括桌面首页、作品、写作、Glassbox、两个实验、双语手机首页、双语目录和深色目录。程序截取全部页面不等于逐张进行了人工全文审核。
性能结果与保留项
保留全部 12 次 Lighthouse 默认模拟移动限速样本，没有只挑最高分。生产文本使用 gzip。
全部样本 CLS 为 0。自动无障碍、最佳实践和 SEO 分数均为 100，但不是完整无障碍认证。
首页仍存在分数及总阻塞时间波动，不能声称所有样本都达到 98 或 TBT 均接近 0。#36 继续保留，不在本轮关闭。不把不同工作流机器间的波动归因于某一条代码改动。
验证边界
没有测试实体手机、Safari、Firefox或读屏器。没有调用真实模型，也未验收第三方登录与评论服务。本地浏览器访问被环境策略拦截，因此使用上述 CI 浏览器执行验收，没有把本地启动尝试算作通过。
本轮没有合并、没有启用自动合并、没有向 main 推送或部署。截图、报告、依赖和字体不进入仓库；对话证据包不包含字体或浏览器轨迹中的资源副本。