meta-ads-mcp 0.4.2__py3-none-any.whl → 0.4.4__py3-none-any.whl

meta_ads_mcp/__init__.py

@@ -7,7 +7,7 @@ with the Claude LLM.
 
 from meta_ads_mcp.core.server import main
 
-__version__ = "0.4.2"
+__version__ = "0.4.4"
 
 __all__ = [
     'get_ad_accounts',

meta_ads_mcp/core/ads.py

@@ -38,26 +38,27 @@ async def get_ads(access_token: str = None, account_id: str = None, limit: int =
         else:
             return json.dumps({"error": "No account ID specified and no accounts found for user"}, indent=2)
     
+    # Prioritize adset_id over campaign_id - use adset-specific endpoint
+    if adset_id:
+        endpoint = f"{adset_id}/ads"
+        params = {
+            "fields": "id,name,adset_id,campaign_id,status,creative,created_time,updated_time,bid_amount,conversion_domain,tracking_specs",
+            "limit": limit
+        }
     # Use campaign-specific endpoint if campaign_id is provided
-    if campaign_id:
+    elif campaign_id:
         endpoint = f"{campaign_id}/ads"
         params = {
             "fields": "id,name,adset_id,campaign_id,status,creative,created_time,updated_time,bid_amount,conversion_domain,tracking_specs",
             "limit": limit
         }
-        # Adset ID can still be used to filter within the campaign
-        if adset_id:
-            params["adset_id"] = adset_id
     else:
-        # Default to account-level endpoint if no campaign_id
+        # Default to account-level endpoint if no specific filters
         endpoint = f"{account_id}/ads"
         params = {
             "fields": "id,name,adset_id,campaign_id,status,creative,created_time,updated_time,bid_amount,conversion_domain,tracking_specs",
             "limit": limit
         }
-        # Adset ID can filter at the account level if no campaign specified
-        if adset_id:
-            params["adset_id"] = adset_id
 
     data = await make_api_request(endpoint, access_token, params)
     

meta_ads_mcp/core/adsets.py

@@ -5,9 +5,6 @@ from typing import Optional, Dict, Any, List
 from .api import meta_api_tool, make_api_request
 from .accounts import get_ad_accounts
 from .server import mcp_server
-import asyncio
-from .callback_server import start_callback_server, shutdown_callback_server, update_confirmation
-import urllib.parse
 
 
 @mcp_server.tool()
@@ -211,94 +208,52 @@ async def update_adset(adset_id: str, frequency_control_specs: List[Dict[str, An
         bid_strategy: Bid strategy (e.g., 'LOWEST_COST_WITH_BID_CAP')
         bid_amount: Bid amount in account currency (in cents for USD)
         status: Update ad set status (ACTIVE, PAUSED, etc.)
-        targeting: Targeting specifications including targeting_automation
-                  (e.g. {"targeting_automation":{"advantage_audience":1}})
+        targeting: Complete targeting specifications (will replace existing targeting)
+                  (e.g. {"targeting_automation":{"advantage_audience":1}, "geo_locations": {"countries": ["US"]}})
         optimization_goal: Conversion optimization goal (e.g., 'LINK_CLICKS', 'CONVERSIONS', 'APP_INSTALLS', etc.)
         access_token: Meta API access token (optional - will use cached token if not provided)
     """
     if not adset_id:
         return json.dumps({"error": "No ad set ID provided"}, indent=2)
     
-    changes = {}
+    params = {}
     
     if frequency_control_specs is not None:
-        changes['frequency_control_specs'] = frequency_control_specs
+        params['frequency_control_specs'] = frequency_control_specs
     
     if bid_strategy is not None:
-        changes['bid_strategy'] = bid_strategy
+        params['bid_strategy'] = bid_strategy
         
     if bid_amount is not None:
-        changes['bid_amount'] = bid_amount
+        params['bid_amount'] = str(bid_amount)
         
     if status is not None:
-        changes['status'] = status
+        params['status'] = status
         
     if optimization_goal is not None:
-        changes['optimization_goal'] = optimization_goal
+        params['optimization_goal'] = optimization_goal
         
     if targeting is not None:
-        # Get current ad set details to preserve existing targeting settings
-        current_details_json = await get_adset_details(adset_id=adset_id, access_token=access_token)
-        current_details = json.loads(current_details_json)
-        
-        # Check if the current ad set has targeting information
-        current_targeting = current_details.get('targeting', {})
-        
-        if 'targeting_automation' in targeting:
-            # Only update targeting_automation while preserving other targeting settings
-            if current_targeting:
-                merged_targeting = current_targeting.copy()
-                merged_targeting['targeting_automation'] = targeting['targeting_automation']
-                changes['targeting'] = merged_targeting
-            else:
-                # If there's no existing targeting, we need to create a basic one
-                # Meta requires at least a geo_locations setting
-                basic_targeting = {
-                    'targeting_automation': targeting['targeting_automation'],
-                    'geo_locations': {'countries': ['US']}  # Using US as default location
-                }
-                changes['targeting'] = basic_targeting
+        # Ensure proper JSON encoding for targeting
+        if isinstance(targeting, dict):
+            params['targeting'] = json.dumps(targeting)
         else:
-            # Full targeting replacement
-            changes['targeting'] = targeting
+            params['targeting'] = targeting  # Already a string
     
-    if not changes:
+    if not params:
         return json.dumps({"error": "No update parameters provided"}, indent=2)
+
+    endpoint = f"{adset_id}"
     
-    # Get current ad set details for comparison
-    current_details_json = await get_adset_details(adset_id=adset_id, access_token=access_token)
-    current_details = json.loads(current_details_json)
-    
-    # Start the callback server if not already running
     try:
-        port = start_callback_server()
-        
-        # Generate confirmation URL with properly encoded parameters
-        changes_json = json.dumps(changes)
-        encoded_changes = urllib.parse.quote(changes_json)
-        confirmation_url = f"http://localhost:{port}/confirm-update?adset_id={adset_id}&token={access_token}&changes={encoded_changes}"
+        # Use POST method for updates as per Meta API documentation
+        data = await make_api_request(endpoint, access_token, params, method="POST")
+        return json.dumps(data, indent=2)
     except Exception as e:
+        error_msg = str(e)
+        # Include adset_id in error for better context
         return json.dumps({
-            "error": "Callback server disabled",
-            "message": f"Cannot create confirmation URL: {str(e)}",
-            "suggestion": "Manual update confirmation not available when META_ADS_DISABLE_CALLBACK_SERVER is set",
-            "adset_id": adset_id,
-            "proposed_changes": changes
-        }, indent=2)
-    
-    # Reset the update confirmation
-    update_confirmation.clear()
-    update_confirmation.update({"approved": False})
-    
-    # Return the confirmation link
-    response = {
-        "message": "Please confirm the ad set update",
-        "confirmation_url": confirmation_url,
-        "markdown_link": f"[Click here to confirm ad set update]({confirmation_url})",
-        "current_details": current_details,
-        "proposed_changes": changes,
-        "instructions_for_llm": "You must present this link as clickable Markdown to the user using the markdown_link format provided.",
-        "note": "Click the link to confirm and apply your ad set updates. Refresh the browser page if it doesn't load immediately."
-    }
-    
-    return json.dumps(response, indent=2) 
+            "error": f"Failed to update ad set {adset_id}",
+            "details": error_msg,
+            "params_sent": params
+        }, indent=2)