fix(react-doctor): rename generic handlers

[bntvllnt]

Summary

• Renamed all react-doctor/no-generic-handler-names handler definitions in packages/ui to action-oriented names while preserving DOM event prop names.
• Regenerated the registry component shims so the public /r/* sources mirror the package changes.
• Kept behavior unchanged, including consumer onClick ordering, defaultPrevented guards, callback payloads, and event propagation behavior.

Closes #276
Related to #279

Validation

• pnpm doctor:json && jq '[.diagnostics[] | select(.rule == "no-generic-handler-names")] | length' .react-doctor.json PASS (0)
• pnpm -F @vllnt/ui exec eslint <changed package files> PASS
• pnpm -F @vllnt/ui lint PASS
• pnpm -F @vllnt/ui exec tsc --noEmit --project tsconfig.build.json PASS
• pnpm build PASS
• pnpm test:once PASS (216 files, 1215 tests)
• git diff --check PASS

Notes: pnpm build emitted the existing Next.js multiple-lockfile warning for this nested worktree. Vitest emitted existing jsdom navigation, duplicate-key, and Radix dialog-description warnings while passing.

Design Deviations

None.

[vllnt-pilot]

Preview ready · pr-339-ui-registry

Service	Status	Preview
ui-registry	Ready	https://pr-339-ui-registry.preview.vllnt.ai

Inspect

• Deployed to vllnt-cluster from 4f4cc45
• Reply with /clean to destroy this preview now

[bntvllnt]

Review — clean / manual approval recommended

BLOCKING

None.

WARN

None.

VERIFIED CLEAN

• Reviewed current head 4f4cc4560efa062af78680293c2f58cc3e23a59c for PR #339 (fix/issue-276-handler-names).
• Reviewed all 39 changed files: 18 registry shims, 20 package component files, and 1 package test file.
• The diff is a narrow react-doctor/no-generic-handler-names cleanup: generic local handler definitions were renamed to action-oriented names while DOM event prop names and external callback props stayed intact.
• Behavior is preserved: consumer onClick ordering, event.defaultPrevented guards, callback payloads, and event propagation behavior remain equivalent.
• Registry mirrors match the package-side rename intent for the included components.
• PR metadata is current: it closes #276, references #279, and its file count / validation claims match the live diff and check state.
• Post-review head drift from 8a11c6bb4a7c70fefbd3ad0ce5f0511f0704053f to this head is non-product drift only: the PR file list is unchanged; the branch only absorbed main-side CI/deploy metadata (.github/workflows/build.yml, Dockerfile, manifests/lockfile) outside the PR delta.

VALIDATION

• Live GitHub checks re-fetched before publication: 13 visible checks, all pass or skipping; no failures.
• Local targeted rule gate run from the review worktree: pnpm doctor:json; exit 0. react-doctor/no-generic-handler-names diagnostics count is 0.
• Local grep spot-check found only handleSubmit in knowledge-check, which is not flagged by react-doctor and is semantically submit-specific.
• Viewed-state coverage: marked all 39 changed files viewed successfully.
• Not rerun locally in this review pass: full pnpm -F @vllnt/ui lint, tsc, pnpm build, pnpm test:once; relied on live green GitHub checks plus PR-body preserved evidence for those full gates.

Verdict: approval recommended, but final approval remains reserved for bntvllnt/manual review.