refactor(driver): share pointer drag throttling between slider and area

[pbroom]

• Add a framework-agnostic createPointerDragController to @color-kit/driver:
  rAF coalescing, dragEpsilon filtering (both axes), and a maxUpdateHz cap
  that defers (rather than drops) over-budget frames; end() force-commits
  the pending point. Scheduling and clock are injected (defaults resolve
  requestAnimationFrame/performance.now at call time; no DOM globals at
  module scope).
• Add normalizeColorAreaPointer; ColorArea drops its inline clamp01 math.
• ColorSlider and ColorArea both drive drags through the controller.
  Slider now flushes the last pending point on pointerup/cancel like the
  area, and ignores non-finite pointer input.
• Hard cut-over: ColorSlider maxPointerRate -> maxUpdateHz (same name as
  ColorArea). Docs updated.
• Unit tests for the controller and area normalization; slider epsilon test
  now uses real pointer coordinates.

Part of a stacked set of PRs (#204 → #221) from the color-kit assessment. Validated per branch with pnpm build and pnpm test; lint, format, typecheck, size, artifact, preprod and agents checks pass at the top of the stack.

🤖 Generated with Claude Code

[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
• 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 👈 (View in Graphite)
• 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.

[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.

[greptile-apps]

Confidence Score: 3/5

[Medium risk] Refactors pointer drag handling into a shared controller.

The PR should not merge until controlled slider drags handle external value updates and the handler-composition test reflects the intended release commit.

Findings

1.  Controlled updates lose pointer moves ▶
2.  Release commit breaks call assertion ▶
3.  Coalesced sample count is understated ▶

Summary

The PR moves slider and area pointer-drag scheduling into a shared driver controller and adds an area pointer-normalization helper.

• Both React controls now use the controller for coalescing, epsilon filtering, rate limiting, and drag completion.
• The slider prop becomes maxUpdateHz, with corresponding documentation and test changes.
• The slider’s epsilon baseline does not follow external controlled-value updates, and a handler-composition test still assumes the former pointerup behavior.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Pointer down] --> B[Controller start and commit]
  C[Pointer moves] --> D[Keep latest pending input]
  D --> E{Animation frame}
  E -->|Within epsilon| F[Discard pending input]
  E -->|Rate budget unavailable| E
  E -->|Eligible| G[Commit normalized point]
  D --> H[Pointer up or cancel]
  H --> I[Force-commit pending input]

Loading

_{Reviews (1) · Last reviewed commit: "refactor(driver): share pointer drag thr..."}

[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:16pm 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:16 PM UTC: Graphite rebased this pull request as part of a merge.
• Oct 10, 7:16 PM UTC: @pbroom merged this pull request with Graphite.