docs: remove stale verification map and restore LCS/LPC boundaries - #334
Conversation
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 24 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
WalkthroughPR уточняет документацию о статусе LCS/LPC, контрастных компонентах и источниках verification checks. Также добавлены live-repository проверки дрейфа терминологии, ручных verification-утверждений и неподтверждённых runtime-оценок. ChangesКонтракт LCS/LPC и контрастные компоненты
Источники reference и feasibility-проверок
Сканирование repository claims
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@coderabbitai review |
1 similar comment
|
@coderabbitai review |
✅ Action performedReview finished.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 9
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@crates/labcolors-core/src/semantic.rs`:
- Around line 133-137: Обновите rustdoc у `RoleChroma::Curve`, заменив
утверждение о «постоянной перцептивной красочности» описанием фиксированной
CAM16-UCS-координаты `M'` (`TINT_TARGET_MP`). Явно укажите, что эта координата
является выбранной design-policy величиной и не представляет универсальную шкалу
перцептивной цветности.
In `@crates/labcolors-core/src/spaces/cam16.rs`:
- Around line 244-247: Update the CAM16-UCS helper comment above lcs and lpc to
describe the transforms as analytically mutually inverse, not exactly or
bit-exactly invertible; explicitly state that binary64/f64 behavior is validated
within the 1e-12 tolerance used by ucs_rescale_round_trips.
In `@docs/decisions/0003-hk-scope.md`:
- Around line 90-95: В описании APCA/LPC переведите англоязычные термины на
русский: замените “channel power” на русский эквивалент, “readability-путь” на
“путь читаемости”, “legacy-WCAG luminance” на русское описание устаревшей
яркости WCAG, а также переведите “brightness” и “appearance” при их наличии в
этом фрагменте. Сохраните идентификаторы APCA, Ys, LPC и названия стандартов без
изменений.
In `@docs/whitepaper.md`:
- Around line 409-418: Переведите англоязычные термины в секции 3.1 на русские
формулировки: замените candidate, readability, legacy-WCAG luminance,
apparent-brightness, characterization-кандидаты, full LPC evaluator и shipped
продукт, сохранив только необходимые названия API и стандартов вроде LPC, APCA,
WCAG и CAM16.
- Around line 33-43: Переведите добавленный текст в docs/whitepaper.md на
русский, заменив англоязычные термины design-system, candidate contrast paths,
admitted contract, perceived-contrast solver и conformance естественными
русскими описаниями. Сохраните без перевода только имена LCS/LPC и ссылки на
исходники, не изменяя технический смысл разделов 1.2 и окружающего контекста.
- Around line 33-43: Перепишите определения `J'` и `s` в §1.2 как описания
текущих представлений и implementation inputs, убрав утверждения о
«перцептуальной яркости» и «насыщенности». Не представляйте `s` как CAM16
saturation или часть завершённого LCS/LPC-контракта; сохраните только смысл,
подтверждённый опубликованными данными и документацией `lcs.rs`.
In `@packages/colors/test/public-claims.test.mjs`:
- Around line 20-21: Расширьте регулярное выражение REPOSITORY_TEXT_EXT, добавив
поддерживаемые расширения jsx, go, java, kt и mdx, чтобы repository-wide
сканирование охватывало все форматы, указанные в правилах; сохраните
существующие расширения и текущую проверку границы имени файла.
In `@README.md`:
- Around line 5-8: Переведите описательный текст фрагмента README на русский,
заменив английское выражение “perceptual superiority” на «перцептуальное
превосходство». Сохраните технические идентификаторы LCS, LPC, CAM16, Oklab и
APCA-shaped без изменений.
- Line 232: Обновите русскоязычную формулировку в README.md, заменив
англо-русское «appearance-результат» на однозначное русское выражение, например
«результат модели внешнего вида», не изменяя остальную часть утверждения.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 0d69508b-3ac5-4581-89d9-b0aac495578e
📒 Files selected for processing (17)
README.mdcrates/labcolors-core/README.mdcrates/labcolors-core/src/golden_tests.rscrates/labcolors-core/src/lcs.rscrates/labcolors-core/src/lib.rscrates/labcolors-core/src/lpc.rscrates/labcolors-core/src/pair_label_tests.rscrates/labcolors-core/src/reference_vectors_deep.rscrates/labcolors-core/src/semantic.rscrates/labcolors-core/src/spaces/cam16.rscrates/labcolors-core/tests/reference_vectors.rscrates/labcolors-wasm/src/lib.rsdocs/decisions/0003-hk-scope.mddocs/verification-map.mddocs/whitepaper.mdpackages/colors/README.mdpackages/colors/test/public-claims.test.mjs
💤 Files with no reviewable changes (1)
- docs/verification-map.md
1 participant
Outcome
Verification
No product bytes or public function signatures are intentionally changed.
Summary by CodeRabbit
Документация
Тесты