fix(intercom): expose manage-conversation options as static props for MCP v3 (#21753)

* fix(intercom): expose manage-conversation options as static props for MCP v3

`Manage A Conversation` revealed `type`, `assigneeId`, `teamAssigneeId`, `body`
and `snoozedUntil` through `additionalProps()`, driven by `reloadProps` on
`messageType` and `type`. An MCP tool resolves its JSON Schema once at
registration, so props that only appear after a parent prop is set never reach
an agent.

As the issue notes, this action fetches nothing: `additionalProps()` only
flipped `hidden` on props that were already declared, then returned `{}`. All
five are now declared plainly and the mechanism is gone.

Three of them -- `type`, `assigneeId` and `teamAssigneeId` -- were hidden but
not marked optional, so simply unhiding them would have made them required and
broken every existing configuration. They are now explicitly `optional: true`.
`body` and `snoozedUntil` were already optional. Nothing became required and no
default changed.

Since the form no longer hides the props that do not apply, each description now
says which `messageType` it belongs to, and the action description lists the
mapping. That is also what an agent reads to call the tool correctly.

Also fixes a bug this mechanism was hiding: `run()` never read
`teamAssigneeId`. Assigning to a team set `type: "team"` and then sent
`assignee_id: assigneeId`, which `additionalProps()` had just hidden and left
undefined -- so team assignment silently sent no assignee at all. `assignee_id`
now takes the team id when `type` is `team`. The two ids stay separate props
because they list different options (admins vs teams).

* fix(intercom): add a concrete ISO 8601 example to the snoozedUntil description

Date prop descriptions must carry a concrete inline example, per the component
guidelines. The format string is replaced with a real timestamp.

* fixing the coderabbit review and changes related to manage conversation

---------

Co-authored-by: vetrivigneshwaran <vicky@pipedream.com>

Author: DeepakJaggupalli

components/intercom/actions/manage-conversation/manage-conversation.mjs

@@ -1,3 +1,4 @@
+import { ConfigurationError } from "@pipedream/platform";
 import {
   MESSAGE_TYPE_OPTIONS,
   TYPE_OPTIONS,
@@ -7,8 +8,8 @@ import intercom from "../../intercom.app.mjs";
 export default {
   key: "intercom-manage-conversation",
   name: "Manage A Conversation",
-  description: "Close/Snooze/Open/Assign a conversation by its ID. [See the documentation](https://developers.intercom.com/docs/references/2.12/rest-api/api.intercom.io/conversations/manageconversation).",
-  version: "0.0.1",
+  description: "Close, snooze, open, or assign a conversation by its ID. Which of the optional props apply depends on **Message Type**: `close` uses **Body**, `snoozed` uses **Snoozed Until**, `assignment` uses **Type** together with **Assignee ID** or **Team Assignee ID**, and `open` uses none of them. [See the documentation](https://developers.intercom.com/docs/references/2.12/rest-api/api.intercom.io/conversations/manageconversation).",
+  version: "0.1.0",
   annotations: {
     destructiveHint: false,
     openWorldHint: true,
@@ -26,84 +27,53 @@ export default {
     messageType: {
       type: "string",
       label: "Message Type",
-      description: "The kind of message being created.",
-      reloadProps: true,
+      description: "The kind of message being created, which determines the operation performed on the conversation. Use `close` to close it, `snoozed` to snooze it, `open` to reopen it, or `assignment` to assign it.",
       options: MESSAGE_TYPE_OPTIONS,
     },
-    type: {
-      type: "string",
-      label: "Type",
-      description: "The type of the message being created.",
-      options: TYPE_OPTIONS,
-      reloadProps: true,
-      hidden: true,
-    },
     adminId: {
       propDefinition: [
         intercom,
         "adminId",
       ],
     },
+    type: {
+      type: "string",
+      label: "Type",
+      description: "Whether an assignment targets an admin or a team. Only used when **Message Type** is `assignment`. Set `admin` to assign using **Assignee ID**, or `team` to assign using **Team Assignee ID**.",
+      options: TYPE_OPTIONS,
+      optional: true,
+    },
     assigneeId: {
       propDefinition: [
         intercom,
         "adminId",
       ],
       label: "Assignee ID",
-      description: "The `id` of the `admin` which will be assigned the conversation. A conversation can be assigned both an admin and a team.\nSet `0` if you want this assign to no admin (ie. Unassigned).",
-      hidden: true,
+      description: "The `id` of the `admin` which will be assigned the conversation. Only used when **Message Type** is `assignment` and **Type** is `admin`. Set `0` to assign to no admin (ie. Unassigned).",
+      optional: true,
     },
     teamAssigneeId: {
       propDefinition: [
         intercom,
         "teamAssigneeId",
       ],
-      hidden: true,
+      label: "Team Assignee ID",
+      description: "The `id` of the `team` which will be assigned the conversation. Only used when **Message Type** is `assignment` and **Type** is `team`. Set `0` to assign to no team (ie. Unassigned).",
+      optional: true,
     },
     body: {
       type: "string",
       label: "Body",
-      description: "The text body of the comment.",
-      hidden: true,
+      description: "The text body of the comment. Used when **Message Type** is `close` or `assignment`.",
       optional: true,
     },
     snoozedUntil: {
       type: "string",
       label: "Snoozed Until",
-      description: "The date and time the conversation will be snoozed until. Format: YYYY-MM-DDTHH:MM:SSZ",
-      hidden: true,
+      description: "The date and time the conversation will be snoozed until, as an ISO 8601 timestamp — for example `2026-08-24T18:30:00Z`. Only used when **Message Type** is `snoozed`.",
       optional: true,
     },
   },
-  additionalProps(props) {
-    props.type.hidden = true;
-    props.body.hidden = true;
-    props.snoozedUntil.hidden = true;
-    props.assigneeId.hidden = true;
-
-    switch (this.messageType) {
-    case "snoozed":
-      props.snoozedUntil.hidden = false;
-      break;
-    case "close":
-      props.body.hidden = false;
-      break;
-    case "assignment":
-      props.type.hidden = false;
-      props.body.hidden = false;
-      props.assigneeId.hidden = false;
-
-      if (this.type === "team") {
-        props.teamAssigneeId.hidden = false;
-        props.assigneeId.hidden = true;
-      } else if (this.type === "admin") {
-        props.teamAssigneeId.hidden = true;
-        props.assigneeId.hidden = false;
-      }
-      break;
-    }
-    return {};
-  },
   methods: {
     manageConversation({
       conversationId, ...args
@@ -125,8 +95,30 @@ export default {
       messageType,
       snoozedUntil,
       assigneeId,
+      teamAssigneeId,
     } = this;
 
+    let snoozedUntilTimestamp;
+    if (snoozedUntil) {
+      const parsed = Date.parse(snoozedUntil);
+      if (Number.isNaN(parsed)) {
+        throw new ConfigurationError("`Snoozed Until` must be a valid ISO 8601 timestamp");
+      }
+      snoozedUntilTimestamp = parsed / 1000;
+    }
+
+    if (messageType === "assignment") {
+      if (type !== "admin" && type !== "team") {
+        throw new ConfigurationError("`Type` must be `admin` or `team` when `Message Type` is `assignment`");
+      }
+      if (type === "admin" && assigneeId === undefined) {
+        throw new ConfigurationError("`Assignee ID` is required when `Type` is `admin`");
+      }
+      if (type === "team" && teamAssigneeId === undefined) {
+        throw new ConfigurationError("`Team Assignee ID` is required when `Type` is `team`");
+      }
+    }
+
     const response = await manageConversation({
       $,
       conversationId,
@@ -137,12 +129,16 @@ export default {
         type: messageType === "close"
           ? "admin"
           : type,
-        snoozed_until: snoozedUntil && Date.parse(snoozedUntil) / 1000,
-        assignee_id: assigneeId,
+        snoozed_until: snoozedUntilTimestamp,
+        // A team assignment carries the team's id in `assignee_id`; the admin and
+        // team ids come from different props because they list different options.
+        assignee_id: type === "team"
+          ? teamAssigneeId
+          : assigneeId,
       },
     });
 
-    $.export("$summary", "Conversation managed successfully");
+    $.export("$summary", `Conversation ${conversationId} updated with message type "${messageType}"`);
     return response;
   },
 };

components/intercom/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pipedream/intercom",
-  "version": "0.9.0",
+  "version": "0.10.0",
   "description": "Pipedream Intercom Components",
   "main": "intercom.app.mjs",
   "keywords": [