Refresh Remix alpha 6 agent docs#73
Conversation
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
|
🔎 Preview deployed: https://epicflare-pr-73.kentcdodds.workers.dev Worker: Mocks:
|
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit af4a435. Configure here.
| fontSize: typography.fontSize.sm, | ||
| fontWeight: typography.fontWeight.semibold, | ||
| whiteSpace: 'nowrap', | ||
| }), |
There was a problem hiding this comment.
Missing pointerEvents:'none' on invisible delete button
Medium Severity
The delete button's CSS has opacity: 0 but is missing pointerEvents: 'none', which was present in the old code. The parent container's hover rule sets pointerEvents: 'auto' (line 1049) to re-enable clicks on hover — but without the base pointerEvents: 'none' on the button itself, the invisible button is always clickable. Users can accidentally trigger the delete confirmation (or deletion) by clicking the area where the hidden button sits.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit af4a435. Configure here.
| callAll(onBlur, handlers?.blur), | ||
| ), | ||
| on<HTMLButtonElement, 'click'>('click', onClick), | ||
| ], |
There was a problem hiding this comment.
Unused exported createDoubleCheck function after refactor
Low Severity
createDoubleCheck is exported from double-check.ts but has zero consumers in the codebase. Its only caller in chat.tsx was removed in this PR, replaced by the inline confirmingDeleteThreadIds Set. The entire file (including its imports of on, ElementProps, Handle, MixinDescriptor, and the helper functions callAll and normalizeMix) is now dead code.
Reviewed by Cursor Bugbot for commit af4a435. Configure here.
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>


Summary
remix@3.0.0-beta.0and migrated code off deprecated/removed APIs.remix/componentand legacy hostcss/onprops toremix/uiandcss()/on()mixins.actiontohandler, data-table definitions totable()/column, and the D1 adapter to the current adapter contract.docs/agents/remixand replaced it with the Remix CLI packaged skill under.agents/skills/remix.Testing
bun run buildbun run typecheckbun run lintbun run test:unitPLAYWRIGHT_PORT=8787 bun run test:e2ebun run test:mcpdocs/agents/remixreferencesNotes
bunx remix skills installis not available inremix@3.0.0-beta.0(RMX_UNKNOWN_COMMAND); the beta CLI package contains the scaffolded Remix skill at@remix-run/cli/bootstrap/.agents/skills/remix, which is what this PR installs.8788can collide with the local Resend mock worker when another dev server is already running; the passing full E2E run usedPLAYWRIGHT_PORT=8787to target the app worker.bun run format:checkstill reports unrelated pre-existing formatting issues outside this migration; touched source/docs files were checked with targeted Prettier commands.