fix: repoint remaining .com refs missed by api#719#721
Conversation
Three references outside api#719's touched-domain test suites: - sendApifyWebhookEmail: the Recoup Chat CTA link in the LLM email prompt now uses getFrontendBaseUrl() (→ chat.recoupable.dev in prod) instead of the dead chat.recoupable.com literal (DRY, same centralizer as the other email links). - registerGetApiKeyTool: the MCP tool description + JSDoc pointed the LLM at the dead api.recoupable.com / developers.recoupable.com → api.recoupable.dev / docs.recoupable.dev. - getCatalogsHandler: JSDoc legacy-endpoint reference → api.recoupable.dev. Refs recoupable/chat#1819. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
Next review available in: 10 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
No issues found across 3 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Architecture diagram
sequenceDiagram
participant Email as Email Service (Resend)
participant Webhook as sendApifyWebhookEmail
participant LLM as LLM Email Generator
participant Config as Config/Env
participant MCP as MCP Tool Registry
participant Docs as Documentation Site
participant API as API Route Handler
Note over Email,API: NEW: Domain Reference Updates Across Components
alt Email Webhook Flow
Webhook->>Webhook: Compose email prompt with CTA link
Webhook->>Config: CHANGED: Call getFrontendBaseUrl()
Config-->>Webhook: Returns chat.recoupable.dev (current env)
Webhook->>LLM: Generate email with CTA: {baseUrl}/?q=...
LLM-->>Webhook: HTML email body
Webhook->>Email: sendEmailWithResend()
end
alt MCP Tool Registration Flow
MCP->>MCP: Register "get_api_key" tool
Note over MCP: Description references:<br/>CHANGED: api.recoupable.dev<br/>CHANGED: docs.recoupable.dev
MCP-->>LLM Agent: Tool description with new domain
Note over LLM Agent: Agent uses description to construct<br/>HTTP requests to api.recoupable.dev
end
alt Catalog Listing Flow
API->>API: GET /api/catalogs?account_id=...
Note over API: JSDoc references legacy endpoint<br/>CHANGED: api.recoupable.dev
API->>Database: selectAccountCatalogs()
Database-->>API: Account catalogs
API-->>Client: Response byte-identical to legacy endpoint
end
Auto-approved: Updates dead .com URL references to .dev equivalents across three files, with no logic changes.
Re-trigger cubic
What
Follow-up to api#719 — three
.comreferences that sat outside its touched-domain test suites (recoupable/chat#1819 deferred §A items).lib/apify/sendApifyWebhookEmail.ts— the "Recoup Chat" CTA link in the LLM email-generation prompt was a hardcodedhttps://chat.recoupable.com/?q=…(dead). Now interpolatesgetFrontendBaseUrl()(→chat.recoupable.devin prod), reusing the same centralizer as the other email links (DRY).lib/mcp/tools/registerGetApiKeyTool.ts— the MCP tool description (shown to the LLM) + JSDoc pointed at the deadapi.recoupable.com/developers.recoupable.com→api.recoupable.dev/docs.recoupable.dev.lib/catalog/getCatalogsHandler.ts— JSDoc legacy-endpoint reference →api.recoupable.dev.Verification
tsc --noEmit: no errors in the changed files.vitest run lib/apify lib/catalog lib/composio: 236 passed / 50 files.recoupable.comadded in the diff.Not included
Brand-email (
@recoupable.com→@recoupable.dev) andsandbox.recoupable.comrefs are tracked separately on #1819 (email pending the sending-domain decision; sandbox deferred — no live.devtarget).🤖 Generated with Claude Code
Summary by cubic
Fix dead links by switching remaining
recoupable.comreferences to.devand routing the Recoup Chat CTA throughgetFrontendBaseUrl(). This prevents broken email CTAs and outdated MCP tool/docs URLs.getFrontendBaseUrl()instead of a hardcodedchat.recoupable.com(resolves tochat.recoupable.devin prod).api.recoupable.devanddocs.recoupable.dev.api.recoupable.dev.Written for commit 52e3958. Summary will update on new commits.