[pbroom]

oklab.ts destructured all four OKLab matrices into row constants at
module scope. Bundlers keep top-level destructuring even when nothing
reads it, so any bundle that needed only one direction (contrastRatio
needs OKLab -> linear sRGB; parse needs only the reverse) carried the
other two matrices too. Read the rows inside each kernel instead; the
arithmetic is unchanged, so results are bit-identical.

Gzip, size-limit (on top of #225 in parentheses):
contrastRatio alone: 1310 -> 1096 B (1466 -> 1243 B)
parse alone: 4565 -> 4346 B
parse + toHex + contrastRatio: 5233 -> 5225 B (5358 -> 5354 B)
everyday toolkit: 5281 -> 5275 B (5412 -> 5405 B)
packColors + unpackColors: 2317 -> 2277 B

Part of the color-kit assessment stack (#204 → #227). Build, test, lint, typecheck, format, size and check:artifacts pass.

🤖 Generated with Claude Code

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[pbroom]

• refactor(react)!: remove UI components; ship state and plane hooks only #249
• docs(react): rebuild the React docs around the plane hooks #248
• feat(react): add plane renderer, geometry and adaptive-quality hooks #247
• feat(docs): syntax highlighting palette for code samples #246
• feat(docs): native API map at /api/map #245
• docs: add concept pages, guides and entry-point narratives #244
• feat(docs): live plane hero on home and the example system #243
• feat(docs): generated API reference from TypeDoc JSON #242
• refactor(core)!: derive plane query budgets from step counts and drop PlaneWithSense #241
• fix(react): render input aria-label once, on the inner input #240
• fix(driver): map absolute percents as range positions and keep tiny-step precision #239
• fix(core): uncap saturate chroma and resolve planeHue as an OKLCH hue #238
• fix(driver): reject unknown gamuts and align validation error types #237
• chore(docs): remove Sandpack, the shadcn kit and the legacy docs shell #236
• docs(core,driver,react): document the public API #235
• chore: pin control-kit to a commit with its own pnpm workspace #234
• docs: update implementation backlog for the contrast solver replacement #232
• refactor(react)!: remove ColorPlane's deprecated edge option and webgl renderer alias #231
• fix(driver): validate the gamut in getColorAreaFallbackPoint #230
• feat(driver,react): keep the previous hue when an achromatic color is entered #229
• refactor(core)!: replace the hybrid contrast solver with chroma-column tracing #228
• perf(core): trim what contrastRatio pulls in #227 👈 (View in Graphite)
• perf(core): pack the named-color table #226
• feat(core): point 'p3' users to 'display-p3' #225
• chore: ignore local .claude files in prettier #224
• refactor(core)!: retire the legacy contrast-region engine #223
• fix(core): keep hybrid contrast contours connected through folds #222
• chore: add changesets for the assessment stack #221
• docs: generated API reference and repo polish #220
• chore: adopt changesets for versioning and release notes #219
• chore: fix facade peers, licensing, and lint ignores #218
• refactor(react): reuse core contrast and transfer functions #217
• refactor(core)!: make the contrast-region engine explicit #216
• refactor(core)!: remove pre-launch aliases and trim the root export surface #215
• refactor!: name the Display P3 space 'display-p3' everywhere #214
• refactor(driver): share pointer drag throttling between slider and area #213
• feat(react): keyboard, ARIA, disabled, and focus support for sliders and areas #212
• fix(react): stable refs, WebGL context loss, and handler composition #211
• fix(react): apply controlled updates against the latest snapshot #210
• feat(core): selectable gamut mapping method #209
• fix(core): compute contrast from unrounded values #208
• fix(core): consistent achromatic hue handling and single transfer function #207
• feat(core): parse CSS Color 4 syntax #206
• fix(core): reject malformed color input instead of producing NaN #205
• chore: add fast-check and axe-core test dependencies #204
• docs: graphics and performance APIs demo page #203
• feat(core): tuple and typed-array interop for GPU and three.js #202
• feat(core): allocation-free *Into variants for hot paths #201
• feat(core): interpolation space option for mix/interpolate/generateScale #200
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[greptile-apps]

Confidence Score: 5/5

[Low risk] Moves matrix destructuring inside functions for bundle size.

This PR appears safe to merge; no actionable issues were found.

What we checked:

• Color values stay unchanged: The matrices contain fixed numbers and are not changed by repository code. Both functions keep the same calculations and still read every input before writing the output.

Summary

This PR moves matrix reads inside the two OKLab conversion functions so bundles using only one direction can drop the other direction’s matrices.

• The arithmetic, exports, and input/output behavior stay unchanged.
• No actionable issues were found. Tests and size measurements were not run during this review.

_{Reviews (1) · Last reviewed commit: "perf(core): trim what contrastRatio pull..."}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
color-kit	Ready	Preview	Oct 10, 2026 7:53pm UTC

[pbroom]

Merge activity

• Oct 10, 6:47 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Oct 10, 7:44 PM UTC: Graphite rebased this pull request as part of a merge.
• Oct 10, 7:44 PM UTC: @pbroom merged this pull request with Graphite.