Skip to content

feat: add message_id to Email model - #113

Merged
cpenned merged 4 commits into
mainfrom
cursor/message-id-typing-7179
Jul 8, 2026
Merged

feat: add message_id to Email model#113
cpenned merged 4 commits into
mainfrom
cursor/message-id-typing-7179

Conversation

@cpenned

@cpenned cpenned commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds message_id to the Email model for GET /emails/:id and list responses, matching the Node.js SDK.

Changes

  • Added message_id field to Email with getMessageId() / setMessageId()
  • Updated EmailsTest to verify deserialization

Usage

Email email = resend.emails().get("email-id");
String messageId = email.getMessageId(); // e.g. "<lc2vu8.gpa9o4@email.example.com>"
Open in Web Open in Cursor 

Summary by cubic

Add message_id to the Email model for GET /emails/:id and list responses so you can correlate messages across systems.

  • New Features
    • Email model: added message_id with getMessageId() / setMessageId(); Javadoc aligned; tests updated with example 111-222-333@email.example.com.

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

Review in cubic

cursoragent and others added 2 commits July 8, 2026 19:05
- Add message_id field to Email model for GET /emails/:id and list responses
- Add typed webhook event payload models with message_id in BaseEmailEventData
  and ReceivedEmailEventData, matching the Node.js SDK structure
- Update Webhooks.verify() to return the parsed WebhookEventPayload
- Add email.scheduled and email.suppressed to WebhookEvent enum

Co-authored-by: cpenned <cpenned@users.noreply.github.com>
Remove webhook event payload models, verify() return type change, and
WebhookEvent enum additions. Keep only message_id on Email for GET
/emails/:id and list responses.

Co-authored-by: cpenned <cpenned@users.noreply.github.com>
@cursor cursor Bot changed the title feat: add message_id typing for emails and webhook events feat: add message_id to Email model Jul 8, 2026
cubic-dev-ai[bot]
cubic-dev-ai Bot previously approved these changes Jul 8, 2026

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 33 files (changes from recent commits).

Auto-approved: Adds a new message_id field to the Email model with getter/setter and updates the test to verify deserialization. No logic or API changes, low risk.

Re-trigger cubic

@cpenned
cpenned marked this pull request as ready for review July 8, 2026 19:12
Co-authored-by: cpenned <cpenned@users.noreply.github.com>
cubic-dev-ai[bot]
cubic-dev-ai Bot previously approved these changes Jul 8, 2026

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 1 file (changes from recent commits).

Auto-approved: Adds a field to the Email model and updates a test. No logic changes, low risk.

Re-trigger cubic

Co-authored-by: cpenned <cpenned@users.noreply.github.com>

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 1 file (changes from recent commits).

Auto-approved: Adds a new field to a model class with corresponding tests. Low risk, no logic changes.

Re-trigger cubic

@hermesresend hermesresend left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM — standard getter/setter pattern matching the rest of the model, test asserts the parsed value.

@cpenned
cpenned merged commit e2e1fd6 into main Jul 8, 2026
6 checks passed
@cpenned
cpenned deleted the cursor/message-id-typing-7179 branch July 8, 2026 22:33
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