Skip to content

Conversation

@cpenned
Copy link
Contributor

@cpenned cpenned commented Jan 23, 2026


Summary by cubic

Reorganized the React Email skill under skills/react-email and expanded the docs for clearer setup, Tailwind usage, rendering, and sending. Added guidance for existing projects, assets, and common pitfalls.

  • Refactors

    • Moved SKILL.md to skills/react-email/SKILL.md.
    • Consolidated references under skills/react-email/references/.
    • Added skills/react-email/README.md describing the skill structure and progressive loading.
  • New Features

    • Added “existing project” setup with required deps (@react-email/preview-server) and package.json scripts.
    • Included Tailwind usage in templates and asset guidance.
    • Added common mistakes and best practices, plus a basic template and i18n guidance.
    • Clarified sending with Resend and linked the SENDING guide.

Written for commit e21776f. Summary will update on new commits.

@changeset-bot
Copy link

changeset-bot bot commented Jan 23, 2026

⚠️ No Changeset found

Latest commit: e21776f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Jan 23, 2026

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

Project Deployment Review Updated (UTC)
react-email Ready Ready Preview, Comment Jan 27, 2026 4:50pm
react-email-demo Ready Ready Preview, Comment Jan 27, 2026 4:50pm

Request Review

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 7 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="skills/react-email/SKILL.md">

<violation number="1" location="skills/react-email/SKILL.md:408">
P2: SKILL.md exceeds the documented <350 line limit for core skill instructions, which breaks the progressive disclosure constraints. Consider moving detailed sections to references and trimming the core file below 350 lines.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="skills/react-email/SKILL.md">

<violation number="1" location="skills/react-email/SKILL.md:273">
P2: The Tailwind config example imports only Tailwind and Img but uses Body, Container, and Button, so the snippet is not runnable. Include the missing components in the import to keep the example correct.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
@cpenned cpenned merged commit 6961d40 into canary Jan 27, 2026
16 checks passed
@cpenned cpenned deleted the feat/react-email-skill-improvements branch January 27, 2026 17:10
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.

3 participants