fix(docs): re-vendor the platform tokens the docs skin had fallen behind - #104
Conversation
`tests/repo/test_brand_assets.py::test_tokens_match_platform` has been red on
main since mklang-platform 1e80671 ("redesign platform web surfaces") added
`--surface-overlay`, `--surface-control` and friends to `web/shared/tokens.css`.
The docs skin's copy is the `:root` block re-vendored as the test's own message
says; the header stays ours.
The test skips on CI (the platform repo is private), which is why only a local
gate with both checkouts saw it.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe CSS token set adds three surface colors, one strong focus-ring style, and two text-measure values. ChangesDesign token additions
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The token additions align with the platform synchronization objective and targeted and full tests pass. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
test_tokens_match_platformhas been failing onmain(noted in #103) since mklang-platform1e80671added six--surface-*tokens toweb/shared/tokens.css. This re-vendors the:rootblock exactly as the test's message instructs; the file header is unchanged.The test skips on CI (private sibling repo), so only a local checkout with both repos could see the drift.
Checks:
tests/repo/test_brand_assets.py4 passed; full suite 682 passed, 1 skipped.🤖 Generated with Claude Code
Summary by CodeRabbit