Test by sweetmantech · Pull Request #108 · recoupable/api

sweetmantech · 2026-01-13T13:28:51Z

Summary by CodeRabbit

New Features
- Enhanced room ID extraction from email messages with improved HTML content parsing capabilities
Tests
- Added comprehensive unit test coverage for room ID extraction from email HTML content
- Covers multiple email client formats and various edge cases including URL encoding and special formatting
- Added tests for improved multi-source room ID lookup with HTML content verification

_{✏️ Tip: You can customize this high-level summary in your review settings.}

The room_id parameter is now required to ensure all outbound emails include the chat link footer, enabling email thread continuity. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

…-email-tool-room_id-required feat: make room_id required in send_email tool

- Add createChatHandler in lib/chats/ - Add POST route at app/api/chats/ - Account ID inferred from API key - Optional artistId and chatId params - chatId auto-generated if not provided Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

- Document validate function pattern using Zod in CLAUDE.md - Add validateCreateChatBody.ts for POST /api/chats - Update createChatHandler to use the validate function Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

- Create safeParseJson helper that returns {} if body is empty/invalid - Use in createChatHandler so body is not required - All params are optional, so empty body should work Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

…-rooms-post-chats Add POST /api/chats endpoint

…107) Superhuman email client inserts <wbr /> tags in link text which breaks plain text extraction. Added extractRoomIdFromHtml function as secondary fallback in getEmailRoomId to handle this case. Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

vercel · 2026-01-13T13:28:55Z

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

Project	Deployment	Review	Updated (UTC)
recoup-api	Building	Preview	Jan 13, 2026 1:28pm

coderabbitai · 2026-01-13T13:29:02Z

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Introduces a new extractRoomIdFromHtml function to extract room IDs from email HTML content, handling various email client formats and special characters like <wbr/> tags. Integrates this into the existing getEmailRoomId extraction flow between text-based and header-based fallback checks. Includes comprehensive unit tests for all scenarios.

Changes

Cohort / File(s)	Summary
New room ID extraction from HTML `lib/emails/inbound/extractRoomIdFromHtml.ts`	Exports `extractRoomIdFromHtml(html)` function with regex-based room UUID extraction. Handles URL-encoded redirects, `<wbr/>` tag stripping, and returns first valid UUID or undefined.
Integration into email room ID extraction `lib/emails/inbound/getEmailRoomId.ts`	Imports and calls `extractRoomIdFromHtml` after text-based extraction but before References header fallback. Updates function documentation to reflect new extraction order.
Unit tests for HTML extraction `lib/emails/inbound/__tests__/extractRoomIdFromHtml.test.ts`	Comprehensive test suite (+168 lines) covering Superhuman/Gmail reply formats, URL encoding, `<wbr/>` tags, case sensitivity, multiple links, and no-ID edge cases.
Integration tests for getEmailRoomId `lib/emails/inbound/__tests__/getEmailRoomId.test.ts`	Adds 41 lines of tests under "secondary: extracting from email HTML" block, verifying HTML extraction with `<wbr/>` tags and text priority over HTML. Updates fallback tests for References header usage.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~28 minutes

Possibly related PRs

Test #88: Modifies getEmailRoomId.ts control flow for room-id extraction, affecting the same function's structure that this PR extends.
Test #101: Adds related room ID extraction utilities and modifies getEmailRoomId.ts, introducing similar email processing enhancements.

Poem

A rabbit hops through HTML's maze,
With <wbr/> tags in a tangled haze,
Finding room IDs, left and right,
Each regex pattern shining bright! ✨🐰

✨ Finishing touches

📝 Generate docstrings

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dec68d0 and a84ec73.

📒 Files selected for processing (4)

lib/emails/inbound/__tests__/extractRoomIdFromHtml.test.ts
lib/emails/inbound/__tests__/getEmailRoomId.test.ts
lib/emails/inbound/extractRoomIdFromHtml.ts
lib/emails/inbound/getEmailRoomId.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

sweetmantech and others added 9 commits January 8, 2026 10:56

Merge pull request #102 from Recoupable-com/sweetmantech/myc-3876-api…

5354e16

…-email-tool-room_id-required feat: make room_id required in send_email tool

Add input validation pattern to CLAUDE.md and chats endpoint

f8baa3a

- Document validate function pattern using Zod in CLAUDE.md - Add validateCreateChatBody.ts for POST /api/chats - Update createChatHandler to use the validate function Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Remove try-catch around request.json(), follow existing pattern

2f03d7f

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Add safeParseJson utility for optional request bodies

aeee4dd

- Create safeParseJson helper that returns {} if body is empty/invalid - Use in createChatHandler so body is not required - All params are optional, so empty body should work Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Merge pull request #104 from Recoupable-com/sweetmantech/myc-3881-api…

dbee139

…-rooms-post-chats Add POST /api/chats endpoint

Merge branch 'main' of github.com:Recoupable-com/Recoup-API into test

1ca467f

sweetmantech merged commit b12ebe8 into main Jan 13, 2026
1 of 2 checks passed

vercel bot deployed to Preview January 13, 2026 13:29 View deployment

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Test#108

Test#108
sweetmantech merged 9 commits intomainfrom
test

sweetmantech commented Jan 13, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

vercel bot commented Jan 13, 2026

Uh oh!

Uh oh!

coderabbitai bot commented Jan 13, 2026 •

edited

Loading

Review failed

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Comments

Conversation

sweetmantech commented Jan 13, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

vercel bot commented Jan 13, 2026

Uh oh!

Uh oh!

coderabbitai bot commented Jan 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Comments

sweetmantech commented Jan 13, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jan 13, 2026 •

edited

Loading