meta-ads-mcp 1.0.13__tar.gz → 1.0.15__tar.gz

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: meta-ads-mcp
-Version: 1.0.13
+Version: 1.0.15
 Summary: Model Context Protocol (MCP) server for interacting with Meta Ads API
 Project-URL: Homepage, https://github.com/pipeboard-co/meta-ads-mcp
 Project-URL: Bug Tracker, https://github.com/pipeboard-co/meta-ads-mcp/issues
@@ -25,12 +25,14 @@ Description-Content-Type: text/markdown
 
 # Meta Ads MCP
 
-A [Model Context Protocol (MCP)](https://modelcontextprotocol.io/) server for interacting with Meta Ads API. This tool enables AI models to access, analyze, and manage Meta advertising campaigns through a standardized interface, allowing LLMs to retrieve performance data, visualize ad creatives, and provide strategic insights for Facebook, Instagram, and other Meta platforms.
+A [Model Context Protocol (MCP)](https://modelcontextprotocol.io/) server for interacting with Meta Ads. Analyze, manage and optimize Meta advertising campaigns through an AI interface. Use an LLM to retrieve performance data, visualize ad creatives, and provide strategic insights for your ads on Facebook, Instagram, and other Meta platforms.
 
 > **DISCLAIMER:** This is an unofficial third-party tool and is not associated with, endorsed by, or affiliated with Meta in any way. This project is maintained independently and uses Meta's public APIs according to their terms of service. Meta, Facebook, Instagram, and other Meta brand names are trademarks of their respective owners.
 
 [![Meta Ads MCP Server Demo](https://github.com/user-attachments/assets/3e605cee-d289-414b-814c-6299e7f3383e)](https://github.com/user-attachments/assets/3e605cee-d289-414b-814c-6299e7f3383e)
 
+[![MCP Badge](https://lobehub.com/badge/mcp/nictuku-meta-ads-mcp)](https://lobehub.com/mcp/nictuku-meta-ads-mcp)
+
 mcp-name: co.pipeboard/meta-ads-mcp
 
 ## Community & Support

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/README.md

@@ -1,11 +1,13 @@
 # Meta Ads MCP
 
-A [Model Context Protocol (MCP)](https://modelcontextprotocol.io/) server for interacting with Meta Ads API. This tool enables AI models to access, analyze, and manage Meta advertising campaigns through a standardized interface, allowing LLMs to retrieve performance data, visualize ad creatives, and provide strategic insights for Facebook, Instagram, and other Meta platforms.
+A [Model Context Protocol (MCP)](https://modelcontextprotocol.io/) server for interacting with Meta Ads. Analyze, manage and optimize Meta advertising campaigns through an AI interface. Use an LLM to retrieve performance data, visualize ad creatives, and provide strategic insights for your ads on Facebook, Instagram, and other Meta platforms.
 
 > **DISCLAIMER:** This is an unofficial third-party tool and is not associated with, endorsed by, or affiliated with Meta in any way. This project is maintained independently and uses Meta's public APIs according to their terms of service. Meta, Facebook, Instagram, and other Meta brand names are trademarks of their respective owners.
 
 [![Meta Ads MCP Server Demo](https://github.com/user-attachments/assets/3e605cee-d289-414b-814c-6299e7f3383e)](https://github.com/user-attachments/assets/3e605cee-d289-414b-814c-6299e7f3383e)
 
+[![MCP Badge](https://lobehub.com/badge/mcp/nictuku-meta-ads-mcp)](https://lobehub.com/mcp/nictuku-meta-ads-mcp)
+
 mcp-name: co.pipeboard/meta-ads-mcp
 
 ## Community & Support

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/meta_ads_mcp/__init__.py

@@ -6,7 +6,7 @@ This package provides a Meta Ads MCP integration
 
 from meta_ads_mcp.core.server import main
 
-__version__ = "1.0.13"
+__version__ = "1.0.15"
 
 __all__ = [
     'get_ad_accounts',

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/meta_ads_mcp/core/ads.py

@@ -831,32 +831,18 @@ async def create_ad_creative(
     if description and descriptions:
         return json.dumps({"error": "Cannot specify both 'description' and 'descriptions'. Use 'description' for single description or 'descriptions' for multiple."}, indent=2)
     
-    # Convert simple parameters to complex format for internal processing
-    final_headlines = None
-    final_descriptions = None
-    
-    if headline:
-        final_headlines = [headline]
-    elif headlines:
-        final_headlines = headlines
-        
-    if description:
-        final_descriptions = [description]
-    elif descriptions:
-        final_descriptions = descriptions
-    
-    # Validate dynamic creative parameters
-    if final_headlines:
-        if len(final_headlines) > 5:
+    # Validate dynamic creative parameters (plural forms only)
+    if headlines:
+        if len(headlines) > 5:
             return json.dumps({"error": "Maximum 5 headlines allowed for dynamic creatives"}, indent=2)
-        for i, h in enumerate(final_headlines):
+        for i, h in enumerate(headlines):
             if len(h) > 40:
                 return json.dumps({"error": f"Headline {i+1} exceeds 40 character limit"}, indent=2)
     
-    if final_descriptions:
-        if len(final_descriptions) > 5:
+    if descriptions:
+        if len(descriptions) > 5:
             return json.dumps({"error": "Maximum 5 descriptions allowed for dynamic creatives"}, indent=2)
-        for i, d in enumerate(final_descriptions):
+        for i, d in enumerate(descriptions):
             if len(d) > 125:
                 return json.dumps({"error": f"Description {i+1} exceeds 125 character limit"}, indent=2)
     
@@ -866,8 +852,9 @@ async def create_ad_creative(
     }
     
     # Choose between asset_feed_spec (dynamic creative) or object_story_spec (traditional)
-    if final_headlines or final_descriptions:
-        # Use asset_feed_spec for dynamic creatives
+    # ONLY use asset_feed_spec when user explicitly provides plural parameters (headlines/descriptions)
+    if headlines or descriptions:
+        # Use asset_feed_spec for dynamic creatives with multiple variants
         asset_feed_spec = {
             "ad_formats": ["SINGLE_IMAGE"],
             "images": [{"hash": image_hash}],
@@ -875,12 +862,12 @@ async def create_ad_creative(
         }
         
         # Handle headlines
-        if final_headlines:
-            asset_feed_spec["headlines"] = [{"text": headline_text} for headline_text in final_headlines]
+        if headlines:
+            asset_feed_spec["headlines"] = [{"text": headline_text} for headline_text in headlines]
             
         # Handle descriptions  
-        if final_descriptions:
-            asset_feed_spec["descriptions"] = [{"text": description_text} for description_text in final_descriptions]
+        if descriptions:
+            asset_feed_spec["descriptions"] = [{"text": description_text} for description_text in descriptions]
         
         # Add message as primary_texts if provided
         if message:
@@ -897,7 +884,7 @@ async def create_ad_creative(
             "page_id": page_id
         }
     else:
-        # Use traditional object_story_spec for single creative
+        # Use traditional object_story_spec with link_data for simple creatives
         creative_data["object_story_spec"] = {
             "page_id": page_id,
             "link_data": {
@@ -909,17 +896,25 @@ async def create_ad_creative(
         # Add optional parameters if provided
         if message:
             creative_data["object_story_spec"]["link_data"]["message"] = message
+        
+        # Add headline (singular) to link_data
+        if headline:
+            creative_data["object_story_spec"]["link_data"]["name"] = headline
+        
+        # Add description (singular) to link_data
+        if description:
+            creative_data["object_story_spec"]["link_data"]["description"] = description
+        
+        # Add call_to_action to link_data for simple creatives
+        if call_to_action_type:
+            creative_data["object_story_spec"]["link_data"]["call_to_action"] = {
+                "type": call_to_action_type
+            }
     
     # Add dynamic creative spec if provided
     if dynamic_creative_spec:
         creative_data["dynamic_creative_spec"] = dynamic_creative_spec
     
-    # Only add call_to_action to object_story_spec if we're not using asset_feed_spec
-    if call_to_action_type and "asset_feed_spec" not in creative_data:
-        creative_data["object_story_spec"]["link_data"]["call_to_action"] = {
-            "type": call_to_action_type
-        }
-    
     if instagram_actor_id:
         creative_data["instagram_actor_id"] = instagram_actor_id
     
@@ -998,32 +993,18 @@ async def update_ad_creative(
     if description and descriptions:
         return json.dumps({"error": "Cannot specify both 'description' and 'descriptions'. Use 'description' for single description or 'descriptions' for multiple."}, indent=2)
     
-    # Convert simple parameters to complex format for internal processing
-    final_headlines = None
-    final_descriptions = None
-    
-    if headline:
-        final_headlines = [headline]
-    elif headlines:
-        final_headlines = headlines
-        
-    if description:
-        final_descriptions = [description]
-    elif descriptions:
-        final_descriptions = descriptions
-    
-    # Validate dynamic creative parameters
-    if final_headlines:
-        if len(final_headlines) > 5:
+    # Validate dynamic creative parameters (plural forms only)
+    if headlines:
+        if len(headlines) > 5:
             return json.dumps({"error": "Maximum 5 headlines allowed for dynamic creatives"}, indent=2)
-        for i, h in enumerate(final_headlines):
+        for i, h in enumerate(headlines):
             if len(h) > 40:
                 return json.dumps({"error": f"Headline {i+1} exceeds 40 character limit"}, indent=2)
     
-    if final_descriptions:
-        if len(final_descriptions) > 5:
+    if descriptions:
+        if len(descriptions) > 5:
             return json.dumps({"error": "Maximum 5 descriptions allowed for dynamic creatives"}, indent=2)
-        for i, d in enumerate(final_descriptions):
+        for i, d in enumerate(descriptions):
             if len(d) > 125:
                 return json.dumps({"error": f"Description {i+1} exceeds 125 character limit"}, indent=2)
     
@@ -1033,45 +1014,58 @@ async def update_ad_creative(
     if name:
         update_data["name"] = name
     
-    if message:
-        update_data["object_story_spec"] = {"link_data": {"message": message}}
-    
-    # Handle dynamic creative assets via asset_feed_spec
-    if final_headlines or final_descriptions or dynamic_creative_spec:
+    # Choose between asset_feed_spec (dynamic creative) or object_story_spec (traditional)
+    # ONLY use asset_feed_spec when user explicitly provides plural parameters (headlines/descriptions)
+    if headlines or descriptions or dynamic_creative_spec:
+        # Handle dynamic creative assets via asset_feed_spec
         asset_feed_spec = {}
         
         # Add required ad_formats field for dynamic creatives
         asset_feed_spec["ad_formats"] = ["SINGLE_IMAGE"]
         
         # Handle headlines
-        if final_headlines:
-            asset_feed_spec["headlines"] = [{"text": headline_text} for headline_text in final_headlines]
+        if headlines:
+            asset_feed_spec["headlines"] = [{"text": headline_text} for headline_text in headlines]
             
         # Handle descriptions  
-        if final_descriptions:
-            asset_feed_spec["descriptions"] = [{"text": description_text} for description_text in final_descriptions]
+        if descriptions:
+            asset_feed_spec["descriptions"] = [{"text": description_text} for description_text in descriptions]
         
         # Add message as primary_texts if provided
         if message:
             asset_feed_spec["primary_texts"] = [{"text": message}]
         
+        # Add call_to_action_types if provided
+        if call_to_action_type:
+            asset_feed_spec["call_to_action_types"] = [call_to_action_type]
+        
         update_data["asset_feed_spec"] = asset_feed_spec
+    else:
+        # Use traditional object_story_spec with link_data for simple creatives
+        if message or headline or description or call_to_action_type:
+            update_data["object_story_spec"] = {"link_data": {}}
+            
+            if message:
+                update_data["object_story_spec"]["link_data"]["message"] = message
+            
+            # Add headline (singular) to link_data
+            if headline:
+                update_data["object_story_spec"]["link_data"]["name"] = headline
+            
+            # Add description (singular) to link_data
+            if description:
+                update_data["object_story_spec"]["link_data"]["description"] = description
+            
+            # Add call_to_action to link_data for simple creatives
+            if call_to_action_type:
+                update_data["object_story_spec"]["link_data"]["call_to_action"] = {
+                    "type": call_to_action_type
+                }
     
     # Add dynamic creative spec if provided
     if dynamic_creative_spec:
         update_data["dynamic_creative_spec"] = dynamic_creative_spec
     
-    # Handle call_to_action - add to asset_feed_spec if using dynamic creative, otherwise to object_story_spec
-    if call_to_action_type:
-        if "asset_feed_spec" in update_data:
-            update_data["asset_feed_spec"]["call_to_action_types"] = [call_to_action_type]
-        else:
-            if "object_story_spec" not in update_data:
-                update_data["object_story_spec"] = {"link_data": {}}
-            update_data["object_story_spec"]["link_data"]["call_to_action"] = {
-                "type": call_to_action_type
-            }
-    
     # Prepare the API endpoint for updating the creative
     endpoint = f"{creative_id}"
     

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/meta_ads_mcp/core/api.py

@@ -6,7 +6,8 @@ import httpx
 import asyncio
 import functools
 import os
-from .auth import needs_authentication, get_current_access_token, auth_manager, start_callback_server, shutdown_callback_server
+from . import auth
+from .auth import needs_authentication, auth_manager, start_callback_server, shutdown_callback_server
 from .utils import logger
 
 # Constants
@@ -203,7 +204,7 @@ def meta_api_tool(func):
             # If access_token is not in kwargs or not kwargs['access_token'], try to get it from auth_manager
             if 'access_token' not in kwargs or not kwargs['access_token']:
                 try:
-                    access_token = await get_current_access_token()
+                    access_token = await auth.get_current_access_token()
                     if access_token:
                         kwargs['access_token'] = access_token
                         logger.debug("Using access token from auth_manager")

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/meta_ads_mcp/core/authentication.py

@@ -29,7 +29,8 @@ from typing import Optional
 import asyncio
 import os
 from .api import meta_api_tool
-from .auth import start_callback_server, shutdown_callback_server, auth_manager, get_current_access_token
+from . import auth
+from .auth import start_callback_server, shutdown_callback_server, auth_manager
 from .server import mcp_server
 from .utils import logger, META_APP_SECRET
 from .pipeboard_auth import pipeboard_auth_manager

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/meta_ads_mcp/core/duplication.py

@@ -6,7 +6,7 @@ import httpx
 from typing import Optional, Dict, Any, List, Union
 from .server import mcp_server
 from .api import meta_api_tool
-from .auth import get_current_access_token
+from . import auth
 from .http_auth_integration import FastMCPAuthIntegration
 
 
@@ -204,7 +204,7 @@ async def _forward_duplication_request(resource_type: str, resource_id: str, acc
         
         # Use provided access_token parameter if no Facebook token found in context
         if not facebook_token:
-            facebook_token = access_token if access_token else await get_current_access_token()
+            facebook_token = access_token if access_token else await auth.get_current_access_token()
         
         # Validate we have both required tokens
         if not pipeboard_token:

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "meta-ads-mcp"
-version = "1.0.13"
+version = "1.0.15"
 description = "Model Context Protocol (MCP) server for interacting with Meta Ads API"
 readme = "README.md"
 requires-python = ">=3.10"

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/server.json

@@ -2,7 +2,7 @@
   "$schema": "https://static.modelcontextprotocol.io/schemas/2025-09-16/server.schema.json",
   "name": "co.pipeboard/meta-ads-mcp",
   "description": "Facebook / Meta Ads automation with AI: analyze performance, test creatives, optimize spend.",
-  "version": "1.0.13",
+  "version": "1.0.15",
   "remotes": [
     {
       "type": "streamable-http",
@@ -13,7 +13,7 @@
     {
       "registryType": "pypi",
       "identifier": "meta-ads-mcp",
-      "version": "1.0.13",
+      "version": "1.0.15",
       "transport": {
         "type": "stdio"
       }

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/tests/test_account_info_access_fix.py

@@ -38,7 +38,7 @@ class TestAccountInfoAccessFix:
         }
         
         with patch('meta_ads_mcp.core.accounts.make_api_request', new_callable=AsyncMock) as mock_api:
-            with patch('meta_ads_mcp.core.api.get_current_access_token', new_callable=AsyncMock) as mock_auth:
+            with patch('meta_ads_mcp.core.auth.get_current_access_token', new_callable=AsyncMock) as mock_auth:
                 mock_auth.return_value = "test_access_token"
                 mock_api.return_value = mock_account_response
                 
@@ -95,7 +95,7 @@ class TestAccountInfoAccessFix:
         }
         
         with patch('meta_ads_mcp.core.accounts.make_api_request', new_callable=AsyncMock) as mock_api:
-            with patch('meta_ads_mcp.core.api.get_current_access_token', new_callable=AsyncMock) as mock_auth:
+            with patch('meta_ads_mcp.core.auth.get_current_access_token', new_callable=AsyncMock) as mock_auth:
                 mock_auth.return_value = "test_access_token"
                 
                 # First call returns permission error, second call returns accessible accounts
@@ -158,7 +158,7 @@ class TestAccountInfoAccessFix:
         }
         
         with patch('meta_ads_mcp.core.accounts.make_api_request', new_callable=AsyncMock) as mock_api:
-            with patch('meta_ads_mcp.core.api.get_current_access_token', new_callable=AsyncMock) as mock_auth:
+            with patch('meta_ads_mcp.core.auth.get_current_access_token', new_callable=AsyncMock) as mock_auth:
                 mock_auth.return_value = "test_access_token"
                 mock_api.return_value = mock_error_response
                 
@@ -180,7 +180,7 @@ class TestAccountInfoAccessFix:
     async def test_account_info_missing_account_id_error(self):
         """Test that missing account_id parameter returns appropriate error"""
         
-        with patch('meta_ads_mcp.core.api.get_current_access_token', new_callable=AsyncMock) as mock_auth:
+        with patch('meta_ads_mcp.core.auth.get_current_access_token', new_callable=AsyncMock) as mock_auth:
             mock_auth.return_value = "test_access_token"
             
             result = await get_account_info(account_id=None)
@@ -207,7 +207,7 @@ class TestAccountInfoAccessFix:
         }
         
         with patch('meta_ads_mcp.core.accounts.make_api_request', new_callable=AsyncMock) as mock_api:
-            with patch('meta_ads_mcp.core.api.get_current_access_token', new_callable=AsyncMock) as mock_auth:
+            with patch('meta_ads_mcp.core.auth.get_current_access_token', new_callable=AsyncMock) as mock_auth:
                 mock_auth.return_value = "test_access_token"
                 mock_api.return_value = mock_account_response
                 
@@ -236,7 +236,7 @@ class TestAccountInfoAccessFix:
         }
         
         with patch('meta_ads_mcp.core.accounts.make_api_request', new_callable=AsyncMock) as mock_api:
-            with patch('meta_ads_mcp.core.api.get_current_access_token', new_callable=AsyncMock) as mock_auth:
+            with patch('meta_ads_mcp.core.auth.get_current_access_token', new_callable=AsyncMock) as mock_auth:
                 mock_auth.return_value = "test_access_token"
                 mock_api.return_value = mock_account_response
                 
@@ -272,7 +272,7 @@ class TestAccountInfoAccessRegression:
         }
         
         with patch('meta_ads_mcp.core.accounts.make_api_request', new_callable=AsyncMock) as mock_api:
-            with patch('meta_ads_mcp.core.api.get_current_access_token', new_callable=AsyncMock) as mock_auth:
+            with patch('meta_ads_mcp.core.auth.get_current_access_token', new_callable=AsyncMock) as mock_auth:
                 mock_auth.return_value = "test_access_token"
                 mock_api.return_value = mock_account_response
                 

{meta_ads_mcp-1.0.13 → meta_ads_mcp-1.0.15}/tests/test_budget_update.py

@@ -40,7 +40,7 @@ class TestBudgetUpdateFunctionality:
     def mock_auth_manager(self):
         """Mock for the authentication manager"""
         with patch('meta_ads_mcp.core.api.auth_manager') as mock, \
-             patch('meta_ads_mcp.core.api.get_current_access_token') as mock_get_token:
+             patch('meta_ads_mcp.core.auth.get_current_access_token') as mock_get_token:
             # Mock a valid access token
             mock.get_current_access_token.return_value = "test_access_token"
             mock.is_token_valid.return_value = True
@@ -433,7 +433,7 @@ class TestBudgetUpdateIntegration:
         # Test that the function accepts the new parameters
         with patch('meta_ads_mcp.core.adsets.make_api_request') as mock_api, \
              patch('meta_ads_mcp.core.api.auth_manager') as mock_auth, \
-             patch('meta_ads_mcp.core.api.get_current_access_token') as mock_get_token:
+             patch('meta_ads_mcp.core.auth.get_current_access_token') as mock_get_token:
             
             mock_api.return_value = {"id": "test_id", "daily_budget": "5000"}
             mock_auth.get_current_access_token.return_value = "test_access_token"

meta_ads_mcp-1.0.15/tests/test_create_ad_creative_simple.py

@@ -0,0 +1,127 @@
+"""Test that create_ad_creative handles simple creatives correctly."""
+
+import pytest
+import json
+from unittest.mock import AsyncMock, patch
+from meta_ads_mcp.core.ads import create_ad_creative
+
+
+@pytest.mark.asyncio
+async def test_simple_creative_uses_object_story_spec():
+    """Test that singular headline/description uses object_story_spec, not asset_feed_spec."""
+    
+    # Mock the make_api_request function
+    with patch('meta_ads_mcp.core.ads.make_api_request') as mock_api, \
+         patch('meta_ads_mcp.core.ads._discover_pages_for_account') as mock_discover:
+        
+        # Mock page discovery
+        mock_discover.return_value = {
+            "success": True,
+            "page_id": "123456789",
+            "page_name": "Test Page"
+        }
+        
+        # Mock creative creation response
+        mock_api.side_effect = [
+            # First call: Create creative
+            {"id": "creative_123"},
+            # Second call: Get creative details
+            {
+                "id": "creative_123",
+                "name": "Test Creative",
+                "status": "ACTIVE"
+            }
+        ]
+        
+        # Call create_ad_creative with singular headline and description
+        result = await create_ad_creative(
+            account_id="act_701351919139047",
+            image_hash="test_hash_123",
+            name="Math Problem - Hormozi",
+            link_url="https://adrocketx.ai/",
+            message="If you're spending 4+ hours per campaign...",
+            headline="Stop paying yourself $12.50/hour",
+            description="AI builds campaigns in 3min. 156% higher conversions. Free beta.",
+            call_to_action_type="LEARN_MORE",
+            access_token="test_token"
+        )
+        
+        # Check that make_api_request was called
+        assert mock_api.call_count == 2
+        
+        # Get the creative_data that was sent to the API
+        create_call_args = mock_api.call_args_list[0]
+        endpoint = create_call_args[0][0]
+        creative_data = create_call_args[0][2]
+        
+        print("Creative data sent to API:")
+        print(json.dumps(creative_data, indent=2))
+        
+        # Verify it uses object_story_spec, NOT asset_feed_spec
+        assert "object_story_spec" in creative_data, "Should use object_story_spec for simple creatives"
+        assert "asset_feed_spec" not in creative_data, "Should NOT use asset_feed_spec for simple creatives"
+        
+        # Verify object_story_spec structure
+        assert "link_data" in creative_data["object_story_spec"]
+        link_data = creative_data["object_story_spec"]["link_data"]
+        
+        # Verify simple creative fields are in link_data
+        assert link_data["image_hash"] == "test_hash_123"
+        assert link_data["link"] == "https://adrocketx.ai/"
+        assert link_data["message"] == "If you're spending 4+ hours per campaign..."
+        
+        # The issue: headline and description should be in link_data for simple creatives
+        # Not in asset_feed_spec
+        print("\nlink_data structure:")
+        print(json.dumps(link_data, indent=2))
+
+
+@pytest.mark.asyncio
+async def test_dynamic_creative_uses_asset_feed_spec():
+    """Test that plural headlines/descriptions uses asset_feed_spec."""
+    
+    with patch('meta_ads_mcp.core.ads.make_api_request') as mock_api, \
+         patch('meta_ads_mcp.core.ads._discover_pages_for_account') as mock_discover:
+        
+        # Mock page discovery
+        mock_discover.return_value = {
+            "success": True,
+            "page_id": "123456789",
+            "page_name": "Test Page"
+        }
+        
+        # Mock creative creation response
+        mock_api.side_effect = [
+            {"id": "creative_456"},
+            {"id": "creative_456", "name": "Dynamic Creative", "status": "ACTIVE"}
+        ]
+        
+        # Call with PLURAL headlines and descriptions (dynamic creative)
+        result = await create_ad_creative(
+            account_id="act_701351919139047",
+            image_hash="test_hash_456",
+            name="Dynamic Creative Test",
+            link_url="https://example.com/",
+            message="Test message",
+            headlines=["Headline 1", "Headline 2"],
+            descriptions=["Description 1", "Description 2"],
+            access_token="test_token"
+        )
+        
+        # Get the creative_data that was sent to the API
+        create_call_args = mock_api.call_args_list[0]
+        creative_data = create_call_args[0][2]
+        
+        print("\nDynamic creative data sent to API:")
+        print(json.dumps(creative_data, indent=2))
+        
+        # Verify it uses asset_feed_spec for dynamic creatives
+        assert "asset_feed_spec" in creative_data, "Should use asset_feed_spec for dynamic creatives"
+        
+        # Verify asset_feed_spec structure
+        asset_feed_spec = creative_data["asset_feed_spec"]
+        assert "headlines" in asset_feed_spec
+        assert len(asset_feed_spec["headlines"]) == 2
+        assert "descriptions" in asset_feed_spec
+        assert len(asset_feed_spec["descriptions"]) == 2
+

meta_ads_mcp-1.0.15/tests/test_create_simple_creative_e2e.py

@@ -0,0 +1,49 @@
+"""End-to-end test for creating simple creatives with singular headline/description."""
+
+import pytest
+import json
+import os
+from meta_ads_mcp.core.ads import create_ad_creative
+
+
+@pytest.mark.skip(reason="Requires authentication - run manually with: pytest tests/test_create_simple_creative_e2e.py -v")
+@pytest.mark.asyncio
+async def test_create_simple_creative_with_real_api():
+    """Test creating a simple creative with singular headline/description using real Meta API."""
+    
+    # Account and image details from user
+    account_id = "act_3182643988557192"
+    image_hash = "ca228ac8ff3a66dca9435c90dd6953d6"
+    
+    # Create a simple creative with singular headline and description
+    result = await create_ad_creative(
+        account_id=account_id,
+        image_hash=image_hash,
+        name="E2E Test - Simple Creative",
+        link_url="https://example.com/",
+        message="This is a test message for the ad.",
+        headline="Test Headline",
+        description="Test description for ad.",
+        call_to_action_type="LEARN_MORE"
+    )
+    
+    print("\n=== API Response ===")
+    print(result)
+    
+    result_data = json.loads(result)
+    
+    # Check if there's an error
+    if "error" in result_data:
+        pytest.fail(f"Creative creation failed: {result_data['error']}")
+    
+    # Verify success
+    assert "success" in result_data or "creative_id" in result_data or "id" in result_data, \
+        f"Expected success response, got: {result_data}"
+    
+    print("\n✅ Simple creative created successfully!")
+    
+    if "creative_id" in result_data:
+        print(f"Creative ID: {result_data['creative_id']}")
+    elif "details" in result_data and "id" in result_data["details"]:
+        print(f"Creative ID: {result_data['details']['id']}")
+