Skip to content

Refresh Remix alpha 6 agent docs#73

Merged
kentcdodds merged 7 commits intomainfrom
cursor/update-remix-alpha6-docs-1a3a
Apr 30, 2026
Merged

Refresh Remix alpha 6 agent docs#73
kentcdodds merged 7 commits intomainfrom
cursor/update-remix-alpha6-docs-1a3a

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented Apr 30, 2026

Summary

  • Upgraded the app to remix@3.0.0-beta.0 and migrated code off deprecated/removed APIs.
  • Updated the client UI runtime from remix/component and legacy host css/on props to remix/ui and css()/on() mixins.
  • Migrated fetch-router route objects from action to handler, data-table definitions to table()/column, and the D1 adapter to the current adapter contract.
  • Fixed chat thread delete confirmation under the new UI event/style model.
  • Deleted docs/agents/remix and replaced it with the Remix CLI packaged skill under .agents/skills/remix.

Testing

  • bun run build
  • bun run typecheck
  • bun run lint
  • bun run test:unit
  • PLAYWRIGHT_PORT=8787 bun run test:e2e
  • bun run test:mcp
  • Targeted Prettier checks for touched source/docs files
  • Remix skill replacement check: old docs removed, 12 packaged skill docs installed, no stale docs/agents/remix references
  • Manual UI smoke test: home counter, signup/login, chat send, and delete confirmation

remix_alpha6_ui_smoke.mp4

Notes

  • bunx remix skills install is not available in remix@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.
  • The default Playwright port 8788 can collide with the local Resend mock worker when another dev server is already running; the passing full E2E run used PLAYWRIGHT_PORT=8787 to target the app worker.
  • A broad bun run format:check still reports unrelated pre-existing formatting issues outside this migration; touched source/docs files were checked with targeted Prettier commands.
Open in Web Open in Cursor 

cursoragent and others added 3 commits April 30, 2026 17:35
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>
@kentcdodds kentcdodds marked this pull request as ready for review April 30, 2026 17:57
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor cursor Bot temporarily deployed to preview-73 April 30, 2026 17:59 Inactive
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

🔎 Preview deployed: https://epicflare-pr-73.kentcdodds.workers.dev

Worker: epicflare-pr-73
D1: epicflare-pr-73-db
KV: epicflare-pr-73-oauth-kv

Mocks:

Comment thread client/double-check.ts Outdated
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor cursor Bot temporarily deployed to preview-73 April 30, 2026 18:19 Inactive
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

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

Comment thread client/routes/chat.tsx
fontSize: typography.fontSize.sm,
fontWeight: typography.fontWeight.semibold,
whiteSpace: 'nowrap',
}),
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit af4a435. Configure here.

Comment thread client/double-check.ts
callAll(onBlur, handlers?.blur),
),
on<HTMLButtonElement, 'click'>('click', onClick),
],
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit af4a435. Configure here.

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor cursor Bot temporarily deployed to preview-73 April 30, 2026 19:54 Inactive
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kentcdodds kentcdodds merged commit 977e397 into main Apr 30, 2026
8 checks passed
@kentcdodds kentcdodds deleted the cursor/update-remix-alpha6-docs-1a3a branch April 30, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants