[8/N] fix(zh): rewrite content and explanatory pages / 重写内容与说明页面中文文案 - #839
[8/N] fix(zh): rewrite content and explanatory pages / 重写内容与说明页面中文文案#839edwingao28 wants to merge 11 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Claude finished @edwingao28's task in 2m 25s —— View job Review complete
🔴 1 blocking issue — a genuine CI failure not covered by the acknowledged timing guard.
Runtime verification (Playwright, real DB): Chinese copy (advisory, non-blocking): the touched strings are all legitimate fidelity/terminology fixes against the English source — Test coverage: new lib/content changes ship colocated tests ( Fix the hardcoded post count and the unit suite goes green. Everything else LGTM. |
This comment has been minimized.
This comment has been minimized.
Correct localized routes and terminology, preserve source scope, contain API schema overflow on mobile, and add focused desktop/mobile regression coverage. 中文:修正本地化路由与术语,保持英文原文的语义范围,解决 API 数据结构在移动端的横向溢出,并补齐桌面端与移动端回归覆盖。
Use the established speculative-decoding term and keep the audited DeepSeek V4 and GB300 inline code byte-identical to their English sources. 中文:统一使用既定的“投机解码”术语,并确保 DeepSeek V4 与 GB300 审核段落中的内联代码与英文原文逐字节一致。
Parse multiline blog tags and restore complete locale metadata parity. Cover API schema containment plus every localized Blog and Glossary listing-to-detail journey at desktop and mobile widths. Correct the TCO glossary translation to preserve software and workload scope. 中文:修复博客多行标签解析并恢复中英文 metadata 完整一致;覆盖 API 数据结构布局,以及 Blog 和 Glossary 在桌面端与移动端的全部双语列表到详情页点击链路;同时修正 TCO 术语页中软件实现与工作负载范围的翻译。
Translate each infrastructure relationship as operational rather than constructional, with a focused fidelity guard. Move exhaustive localized Blog and Glossary navigation matrices into a dedicated integration spec so smoke coverage stays lean. 中文:将三处基础设施与传统领地的关系准确表达为“运行于”,避免误写成建设关系,并加入聚焦语义范围的回归测试;同时把 Blog 与 Glossary 的完整双语导航矩阵移入独立集成测试,保持 smoke 测试轻量。
Localize Blog table-of-contents and heading-link controls, preserve the English rendering, and cover mobile layout plus copy analytics in content-owned tests. 中文:完善 Blog 目录与标题链接控件的中文显示,保持英文界面不变,并在内容分支覆盖移动端布局和复制操作埋点。
f947086 to
0748d30
Compare
|
Claude finished @edwingao28's task in 1m 56s —— View job Re-review complete
No new commits since my previous review. This sync appears to be a base/re-trigger, not new code, so there's nothing new to review — but the previously-flagged blocker is still unresolved and 🔴 Still blocking — unchanged from last review
Everything else
Bottom line: fix the |
Master grew the blog to 31 locale pairs and the glossary to 118 entries, so the hardcoded 19/54 counts in blog-content.test.ts and content-journeys.cy.ts were stale after the rebase. Also register the new content-journeys.cy.ts spec in timings.json (and restore its sort order) so the cypress-split baseline test passes.
|
Claude finished @edwingao28's task in 1m 49s —— View job Re-review complete
✅ The 🔴 blocker from my previous review is fixed. Commit What the new commit does (test-only, verified)
CI status
The unit blocker and the No new concernsThe commit only touches two test files and 🟢 Re-review: no blocking issues. Prior blocker resolved; recommend confirming all E2E shards go green before merge. |
After a client-side navigation Next.js updates the document body before the streamed <link rel="alternate"> tags replace the listing page's, so asserting have.attr on the first matching alternate raced the head update and flaked on the first article/term of a journey. Query for the fully-updated link (hreflang + href) and retry until it exists.
|
Claude finished @edwingao28's task in 55s —— View job Re-review complete
🟢 Re-review: no blocking issues. The single new commit is test-only and correctly reasoned. What
|
| Check | Status |
|---|---|
| Tests (Unit) | ✅ pass |
| Component tests / Code Quality | ✅ pass |
| Lint & Format | ✅ pass |
| Tests (E2E) | ⏳ in progress |
Nothing else changed
Only one E2E test file was touched — no UI/runtime surface changed since the prior review, so the earlier Playwright verification (/zh/about, /zh/blog/..., localized BlogToc/HeadingLink, 0 console errors) still stands, and the Chinese-copy changes remain the same legitimate fidelity/terminology fixes (advisory, non-blocking).
Prior blocker (hardcoded 19 post count) stays resolved. Recommend confirming all E2E shards go green before merge.
· feat/zh-content-pages
中文:优化关于页面、常见问题与闭环并发术语,并移除不符合测试规范的内联超时。
中文:将 master 的最新本地化与内容页面分支同步,确保 PR 预览与最终合并状态一致。
|
Claude finished @edwingao28's task in 1m 59s —— View job Re-review complete
🟢 Re-review: no blocking issues. Two new commits since my prior review:
|
| Check | Status |
|---|---|
| Tests (Unit) | ✅ pass (blocker from earlier reviews stays resolved) |
| Lint & Format | ✅ pass |
| Code Quality | ⏳ in progress |
| Tests (E2E) | ⏳ in progress |
The E2E run is the one thing to confirm green before merge, since 9ac9f6cc shortens the content-journeys alternate-link wait from 15 s to the 6 s global timeout — the retrying selector should cover it, but the shard result is the proof.
Prior blocker (hardcoded 19 post count) remains resolved. No new blocking issues.
· feat/zh-content-pages
中文:优化 About 可复现性说明、FAQ 测试配置术语和 Glossary 提示框用词。
中文:同步 master 的最新模型页面与应用改动,并保留内容页面的已观测 Cypress 时间基线。
|
Claude finished @edwingao28's task in 1m 32s —— View job Re-review complete
🟢 Re-review: no blocking issues. One code-bearing commit since my prior review —
|
| Check | Status |
|---|---|
| Tests (Unit) | ✅ pass |
| Lint & Format | ✅ pass |
| Code Quality | ✅ pass |
| Tests (E2E) | ⏳ in progress |
The prior blocker (hardcoded 19 post count) remains resolved. Confirm the E2E shards go green before merge; no new blocking issues.
· feat/zh-content-pages
Part of #823 / 属于 #823 路线图的一部分
What changed
Scope boundary
Full article-by-article Chinese editorial rewrites remain on hold. This PR only fixes high-confidence protected-content and fidelity regressions in existing article translations.
Validation
mastersync.This PR is Draft for the Chinese maintainer's rendered-page and sentence-by-sentence review before merge.
中文说明
范围边界
逐篇文章的完整中文编辑重写继续暂缓。本 PR 只修正现有文章翻译中高置信度的受保护内容和语义保真问题。
验证结果
master后,GitHub CI 正在针对最终提交重新运行。本 PR 保持 Draft,等待中文维护者在合并前完成真实页面和逐句人工审核。
Note
Medium Risk
The changes touch many localized user-facing strings and add long-running Cypress journeys (content-journeys), which increases CI time and flakiness risk without altering core inference or auth logic.
Overview
This PR audits and rewrites Simplified Chinese copy on About, FAQ, Glossary, API reference, Land Acknowledgement, and selected blog passages—aligned to English sources while preserving protected inline code, tag parity (e.g.
cann), and terminology (推测解码 → 投机解码, 评测 → 评估, tooltip → 提示框). The Chinese About page now links to/zh/blogarticles instead of English URLs withhreflang="en".Blog reading chrome gains locale-aware TOC and heading-link copy (Chinese aria-labels, “链接已复制”), with analytics events including
locale.API docs UI adds
min-w-0anddata-testid="api-schema-note"so wide schema blocks scroll inside the layout without horizontal page overflow on mobile.Test coverage is the other major theme: stable
data-testid/ slug attributes on blog and glossary list/detail pages; new Cypress suites that click through all 31 blog pairs and 118 glossary entries at desktop and mobile widths; API overflow checks; expandedzh-pagesjourneys; and Vitest guards for FAQ parity, glossary fidelity, land-acknowledgement wording, and blog inline-code preservation.Reviewed by Cursor Bugbot for commit ab10607. Bugbot is set up for automated code reviews on this repo. Configure here.