Merge pull request #14 from WebexCommunity/claude/webex-message-edit-attachments-SnnRP

adamweeks · web-flow · commit fe40bacef948 · 2026-05-24T23:18:37.000-04:00
diff --git a/src/webex_bot_mcp/main.py b/src/webex_bot_mcp/main.py
@@ -36,6 +36,7 @@
     # Message functions
     send_webex_message, send_webex_message_with_mentions,
     list_webex_messages, delete_webex_message,
+    update_webex_message, get_webex_attachment_action,
     # Space message aliases
     send_webex_space_message, list_webex_space_messages,
     # Adaptive card tools
@@ -105,6 +106,8 @@ async def dispatch(self, request, call_next):
 mcp.tool()(send_webex_message_with_mentions)
 mcp.tool()(list_webex_messages)
 mcp.tool()(delete_webex_message)
+mcp.tool()(update_webex_message)
+mcp.tool()(get_webex_attachment_action)
 
 # Space message aliases
 mcp.tool()(send_webex_space_message)
@@ -1117,6 +1120,124 @@ def webex_design_adaptive_card_prompt():
     }
 
 
+@mcp.prompt("webex-handle-card-submission")
+def webex_handle_card_submission_prompt():
+    """Process an Adaptive Card form submission received by the bot application"""
+    return {
+        "name": "Handle Card Submission",
+        "description": (
+            "Given an action_id from an attachmentActions webhook your bot application "
+            "already received, fetch the submitted form data and act on it — reply to "
+            "the room, update the original message, or route the inputs to a workflow."
+        ),
+        "arguments": [
+            {
+                "name": "action_id",
+                "description": (
+                    "The attachment action ID from payload[\"data\"][\"id\"] in the "
+                    "webhook POST your bot application received from Webex"
+                ),
+                "required": True
+            },
+            {
+                "name": "intended_action",
+                "description": (
+                    "What to do with the submitted data, e.g. "
+                    "\"approve the request and notify the room\", "
+                    "\"store the feedback and confirm to the user\", "
+                    "\"update the original card message with the outcome\""
+                ),
+                "required": True
+            }
+        ],
+        "template": """Process an Adaptive Card form submission.
+
+IMPORTANT — how action_id reaches you:
+This MCP server cannot receive inbound webhook events. Your separate bot
+application received a POST from Webex at its registered webhook endpoint and
+extracted the action_id from payload["data"]["id"]. That ID is what you are
+working with now.
+
+Action ID: {action_id}
+Intended action: {intended_action}
+
+Steps:
+
+1. Call get_webex_attachment_action(action_id="{action_id}") to retrieve the
+   full submission. The response data contains:
+   - inputs: dict of form field values the user submitted
+   - messageId: the card message that was interacted with
+   - roomId: the room where the interaction happened
+   - personId: the person who submitted the card
+
+2. Inspect the inputs dict and summarise what the user submitted.
+
+3. Carry out the intended action: {intended_action}
+   Common patterns:
+   - Send a confirmation: call send_webex_message(room_id=roomId, markdown="...")
+   - Update the original card message: call update_webex_message(
+       message_id=messageId, markdown="...outcome summary...")
+   - Both: update the card to show it is resolved, then send a threaded reply
+     with details using parent_id=messageId
+
+4. Report what was done: the action_id, the inputs received, and the actions taken.
+
+If get_webex_attachment_action returns a 404, the action_id may be wrong or
+the bot token may not have access to that action. Confirm the ID came from a
+webhook payload for the attachmentActions resource (event: created) and that
+the webhook was registered via create_webex_webhook."""
+    }
+
+
+@mcp.prompt("webex-edit-message")
+def webex_edit_message_prompt():
+    """Correct or update an existing Webex message by its ID"""
+    return {
+        "name": "Edit Message",
+        "description": (
+            "Update the content of an existing Webex message — fix a typo, "
+            "change a status, or replace stale information — using update_webex_message."
+        ),
+        "arguments": [
+            {
+                "name": "message_id",
+                "description": "ID of the message to edit",
+                "required": True
+            },
+            {
+                "name": "new_content",
+                "description": "The replacement text or markdown to put in the message",
+                "required": True
+            },
+            {
+                "name": "reason",
+                "description": "Optional: why the message is being edited (for your own context)",
+                "required": False
+            }
+        ],
+        "template": """Edit an existing Webex message.
+
+Message ID:   {message_id}
+New content:  {new_content}
+Reason:       {reason or "not specified"}
+
+Steps:
+
+1. Call update_webex_message with:
+   - message_id: "{message_id}"
+   - markdown: the corrected content (preferred), or text if plain text is sufficient
+
+   Note: only the bot or user that sent the original message can edit it.
+   A 403 means the bot did not send that message; a 404 means the ID is wrong
+   or the message has been deleted.
+
+2. Confirm the update succeeded and show the new content that was set.
+
+If the new content contains formatting (bold, lists, links), pass it as
+markdown. If it is plain text only, either parameter works."""
+    }
+
+
 @mcp.prompt("webex-setup-webhook")
 def webex_setup_webhook_prompt():
     """Register a new Webex webhook for a specific resource and event"""
@@ -1310,9 +1431,9 @@ def server_version():
             "streamable-http", "stdio",
             "error-handling", "versioning"
         ],
-        "tools_count": 40,
+        "tools_count": 42,
         "resources_count": 11,
-        "prompts_count": 11,
+        "prompts_count": 13,
         "breaking_changes": {
             "1.0.0": [
                 "Initial release with structured error handling",
diff --git a/src/webex_bot_mcp/tools/__init__.py b/src/webex_bot_mcp/tools/__init__.py
@@ -12,6 +12,7 @@
 from .messages import (
     send_webex_message, send_webex_message_with_mentions,
     list_webex_messages, delete_webex_message,
+    update_webex_message, get_webex_attachment_action,
     send_webex_space_message, list_webex_space_messages,
     send_webex_adaptive_card, send_webex_space_adaptive_card,
     build_webex_adaptive_card,
@@ -52,6 +53,7 @@
     # Message functions
     'send_webex_message', 'send_webex_message_with_mentions',
     'list_webex_messages', 'delete_webex_message',
+    'update_webex_message', 'get_webex_attachment_action',
     # Space message aliases
     'send_webex_space_message', 'list_webex_space_messages',
     # Adaptive card tools
diff --git a/src/webex_bot_mcp/tools/messages.py b/src/webex_bot_mcp/tools/messages.py
@@ -553,6 +553,123 @@ def send_webex_adaptive_card(
         return _map_exception_to_error(e)
 
 
+def update_webex_message(
+    message_id: str,
+    text: Optional[str] = None,
+    markdown: Optional[str] = None,
+) -> Dict[str, Any]:
+    """
+    Edit an existing Webex message.
+
+    Only the bot or user that sent the original message may edit it.
+    At least one of text or markdown must be provided.
+
+    Args:
+        message_id: ID of the message to edit (required)
+        text: New plain text content
+        markdown: New markdown content
+
+    Returns:
+        Standardized response dictionary with success/error information
+    """
+    try:
+        if not message_id:
+            return create_error_response(
+                error_code=WebexErrorCodes.INVALID_ARGUMENTS,
+                message="message_id is required"
+            )
+        if not (text or markdown):
+            return create_error_response(
+                error_code=WebexErrorCodes.MISSING_REQUIRED_FIELD,
+                message="At least one of text or markdown is required",
+                details={"required_one_of": ["text", "markdown"]}
+            )
+
+        params: Dict[str, Any] = {}
+        if text:
+            params['text'] = text
+        if markdown:
+            params['markdown'] = markdown
+
+        message = get_webex_api().messages.update(messageId=message_id, **params)
+
+        return create_success_response(
+            data=_message_to_dict(message),
+            metadata={
+                'operation': 'update_message',
+                'message_id': message_id,
+                'content_type': 'markdown' if markdown else 'text',
+            }
+        )
+
+    except Exception as e:
+        return _map_exception_to_error(e)
+
+
+def get_webex_attachment_action(action_id: str) -> Dict[str, Any]:
+    """
+    Retrieve the form data submitted when a user clicks an Adaptive Card button.
+
+    This MCP server makes outbound calls to Webex — it does NOT receive inbound
+    webhook events. The action_id must be obtained externally: your own bot
+    application (a separate HTTP server with a public HTTPS endpoint) receives
+    the webhook POST from Webex, extracts the "id" field from the payload, and
+    then supplies that action_id here so this tool can fetch the full submission.
+
+    Flow:
+        1. User clicks a card button in Webex.
+        2. Webex POSTs the event to your bot's registered webhook endpoint.
+        3. Your bot extracts action_id from payload["data"]["id"].
+        4. Your bot (or an AI agent) calls this tool with that action_id.
+        5. This tool calls GET /attachment/actions/{id} and returns the inputs.
+
+    A webhook must be registered via create_webex_webhook with
+    resource="attachmentActions" and event="created" for step 2 to occur —
+    without it, card submissions are silently dropped by Webex.
+
+    Args:
+        action_id: ID of the attachment action to retrieve (required).
+                   Comes from payload["data"]["id"] in the webhook POST your
+                   bot application receives from Webex.
+
+    Returns:
+        Standardized response dictionary with success/error information.
+        The data dict contains the submitted form inputs under the "inputs" key,
+        along with the associated messageId, roomId, and personId.
+    """
+    try:
+        if not action_id:
+            return create_error_response(
+                error_code=WebexErrorCodes.INVALID_ARGUMENTS,
+                message="action_id is required"
+            )
+
+        action = get_webex_api().attachment_actions.get(action_id)
+
+        action_dict: Dict[str, Any] = {
+            'id': action.id,
+            'type': getattr(action, 'type', None),
+            'messageId': getattr(action, 'messageId', None),
+            'inputs': getattr(action, 'inputs', {}),
+            'roomId': getattr(action, 'roomId', None),
+            'personId': getattr(action, 'personId', None),
+            'created': (
+                action.created.isoformat()
+                if hasattr(action.created, 'isoformat')
+                else str(action.created)
+            ),
+        }
+        action_dict = {k: v for k, v in action_dict.items() if v is not None}
+
+        return create_success_response(
+            data=action_dict,
+            metadata={'operation': 'get_attachment_action', 'action_id': action_id}
+        )
+
+    except Exception as e:
+        return _map_exception_to_error(e)
+
+
 # Space message aliases — "room" and "space" are synonymous in Webex
 
 def send_webex_space_message(
diff --git a/tests/test_messages.py b/tests/test_messages.py
