Skip to content

improvement(integrations): tighten sixtyfour, agentmail, agentphone outputs#4765

Merged
waleedlatif1 merged 4 commits into
stagingfrom
waleedlatif1/verify-sixtyfour-integration
May 27, 2026
Merged

improvement(integrations): tighten sixtyfour, agentmail, agentphone outputs#4765
waleedlatif1 merged 4 commits into
stagingfrom
waleedlatif1/verify-sixtyfour-integration

Conversation

@waleedlatif1
Copy link
Copy Markdown
Collaborator

Summary

  • sixtyfour: capture org_chart from enrich-company, add typed properties to emails/personalEmails block outputs
  • agentmail: rename replyTo to "Override Recipients" (comma-separated), expose username as advanced, add timestamp output
  • agentphone: include mediaUrls array on conversation messages, document list_contacts max=200

Type of Change

  • Improvement

Testing

Tested manually against live API docs (sixtyfour OpenAPI spec).

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment May 27, 2026 11:34pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented May 27, 2026

PR Summary

Low Risk
Documentation and additive output-field mappings for third-party integrations; no auth, billing, or core execution path changes.

Overview
Updates validate-integration guidance in three places (.agents, .claude, .cursor): properties / nested items apply to tool outputs only; block json outputs must document shape in description strings, and properties on block outputs is forbidden (TypeScript build failure).

AgentMail — Tools and types now surface labels and timestamp on get/list message and get-thread payloads; the block adds timestamp, clarifies labels, renames reply override UI to Override Recipients (comma-separated), and marks create inbox username as advanced.

AgentPhone — Conversation message mapping and schemas add mediaUrls (array) alongside mediaUrl; list_contacts documents limit max 200.

Sixtyfourenrich_company maps org_chartorgChart in tool output and types; block documents email JSON shapes and adds orgChart when fullOrgChart is used.

Reviewed by Cursor Bugbot for commit fce28d3. Configure here.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 27, 2026

Greptile Summary

This PR tightens the output contracts for three integrations — sixtyfour, agentmail, and agentphone — by surfacing previously-hidden or undocumented API fields and improving field descriptions to match the codebase's updated documentation standard.

  • sixtyfour: emails/personalEmails block outputs now document their {address, status, type} shape; orgChart is captured from the enrich-company response and exposed as a new block + tool output.
  • agentmail: timestamp and labels are added to get_message, get_thread, and list_messages tools with matching type-interface updates; the replyTo UI field is renamed and its placeholder clarified for comma-separated recipients; username is moved to advanced mode.
  • agentphone: mediaUrls (string array) is added to all three conversation-message tools (get_conversation, get_conversation_messages, update_conversation) and the block output; list_contacts documents the API's max=200 limit.

Confidence Score: 5/5

Safe to merge — all changes are additive output-contract improvements with no breaking changes to existing fields.

Every new field has a safe default in the error and happy-path branches ([] for arrays, null for optional strings), TypeScript interfaces are kept in sync with transformation logic, and no existing API parameters or block inputs are removed or renamed in a breaking way.

No files require special attention.

Important Files Changed

Filename Overview
apps/sim/blocks/blocks/agentmail.ts Renames replyTo UI field to "Override Recipients", updates placeholder to note comma-separated input, hides username behind advanced mode, and adds timestamp to block outputs.
apps/sim/blocks/blocks/agentphone.ts Adds mediaUrls (json array) to block outputs and updates the messages description to include the new field.
apps/sim/blocks/blocks/sixtyfour.ts Improves emails/personalEmails descriptions with field-level documentation and adds orgChart output; orgChart description and personalEmails enum hints are less complete than emails.
apps/sim/tools/agentmail/types.ts Adds labels: string[] and `timestamp: string
apps/sim/tools/agentmail/get_message.ts Exposes labels and timestamp from the API response; both the happy path and the error fallback path are correctly updated with defaults ([] / null).
apps/sim/tools/agentphone/types.ts Adds mediaUrls: string[] to AgentPhoneConversationMessage; non-optional with [] as the safe default matches the transformation logic in all three conversation tools.
apps/sim/tools/sixtyfour/enrich_company.ts Captures org_chart from the API response and maps it to orgChart; the ?? null fallback and the flexible `Record<string, unknown>
apps/sim/tools/agentphone/list_contacts.ts Documentation-only update: adds max 200 to the limit param description to surface the API constraint to users.

Reviews (2): Last reviewed commit: "chore(skills): clarify validate-integrat..." | Re-trigger Greptile

Comment thread apps/sim/tools/agentphone/get_conversation.ts
@waleedlatif1
Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1
Copy link
Copy Markdown
Collaborator Author

@cursor review

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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit fce28d3. Configure here.

@waleedlatif1 waleedlatif1 merged commit b197bb0 into staging May 27, 2026
14 checks passed
@waleedlatif1 waleedlatif1 deleted the waleedlatif1/verify-sixtyfour-integration branch May 27, 2026 23:36
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.

1 participant