meta-ads-mcp 0.10.7__tar.gz → 0.11.0__tar.gz

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: meta-ads-mcp
-Version: 0.10.7
+Version: 0.11.0
 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
@@ -176,7 +176,22 @@ For local installation configuration, authentication options, and advanced techn
      - `access_token` (optional): Meta API access token (will use cached token if not provided)
      - `account_id`: Meta Ads account ID (format: act_XXXXXXXXX)
      - `name`: Campaign name
-     - `objective`: Campaign objective (AWARENESS, TRAFFIC, ENGAGEMENT, etc.)
+     - `objective`: Campaign objective (ODAX, outcome-based). Must be one of:
+       - `OUTCOME_AWARENESS`
+       - `OUTCOME_TRAFFIC`
+       - `OUTCOME_ENGAGEMENT`
+       - `OUTCOME_LEADS`
+       - `OUTCOME_SALES`
+       - `OUTCOME_APP_PROMOTION`
+       
+       Note: Legacy objectives such as `BRAND_AWARENESS`, `LINK_CLICKS`, `CONVERSIONS`, `APP_INSTALLS`, etc. are no longer valid for new campaigns and will cause a 400 error. Use the outcome-based values above. Common mappings:
+       - `BRAND_AWARENESS` → `OUTCOME_AWARENESS`
+       - `REACH` → `OUTCOME_AWARENESS`
+       - `LINK_CLICKS`, `TRAFFIC` → `OUTCOME_TRAFFIC`
+       - `POST_ENGAGEMENT`, `PAGE_LIKES`, `EVENT_RESPONSES`, `VIDEO_VIEWS` → `OUTCOME_ENGAGEMENT`
+       - `LEAD_GENERATION` → `OUTCOME_LEADS`
+       - `CONVERSIONS`, `CATALOG_SALES`, `MESSAGES` (sales-focused flows) → `OUTCOME_SALES`
+       - `APP_INSTALLS` → `OUTCOME_APP_PROMOTION`
      - `status`: Initial campaign status (default: PAUSED)
      - `special_ad_categories`: List of special ad categories if applicable
      - `daily_budget`: Daily budget in account currency (in cents)
@@ -184,6 +199,20 @@ For local installation configuration, authentication options, and advanced techn
      - `bid_strategy`: Bid strategy. Must be one of: `LOWEST_COST_WITHOUT_CAP`, `LOWEST_COST_WITH_BID_CAP`, `COST_CAP`, `LOWEST_COST_WITH_MIN_ROAS`.
    - Returns: Confirmation with new campaign details
 
+   - Example:
+     ```json
+     {
+       "name": "2025 - Bedroom Furniture - Awareness",
+       "account_id": "act_123456789012345",
+       "objective": "OUTCOME_AWARENESS",
+       "special_ad_categories": [],
+       "status": "PAUSED",
+       "buying_type": "AUCTION",
+       "bid_strategy": "LOWEST_COST_WITHOUT_CAP",
+       "daily_budget": 10000
+     }
+     ```
+
 7. `mcp_meta_ads_get_adsets`
    - Get ad sets for a Meta Ads account with optional filtering by campaign
    - Inputs:

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/README.md

@@ -151,7 +151,22 @@ For local installation configuration, authentication options, and advanced techn
      - `access_token` (optional): Meta API access token (will use cached token if not provided)
      - `account_id`: Meta Ads account ID (format: act_XXXXXXXXX)
      - `name`: Campaign name
-     - `objective`: Campaign objective (AWARENESS, TRAFFIC, ENGAGEMENT, etc.)
+     - `objective`: Campaign objective (ODAX, outcome-based). Must be one of:
+       - `OUTCOME_AWARENESS`
+       - `OUTCOME_TRAFFIC`
+       - `OUTCOME_ENGAGEMENT`
+       - `OUTCOME_LEADS`
+       - `OUTCOME_SALES`
+       - `OUTCOME_APP_PROMOTION`
+       
+       Note: Legacy objectives such as `BRAND_AWARENESS`, `LINK_CLICKS`, `CONVERSIONS`, `APP_INSTALLS`, etc. are no longer valid for new campaigns and will cause a 400 error. Use the outcome-based values above. Common mappings:
+       - `BRAND_AWARENESS` → `OUTCOME_AWARENESS`
+       - `REACH` → `OUTCOME_AWARENESS`
+       - `LINK_CLICKS`, `TRAFFIC` → `OUTCOME_TRAFFIC`
+       - `POST_ENGAGEMENT`, `PAGE_LIKES`, `EVENT_RESPONSES`, `VIDEO_VIEWS` → `OUTCOME_ENGAGEMENT`
+       - `LEAD_GENERATION` → `OUTCOME_LEADS`
+       - `CONVERSIONS`, `CATALOG_SALES`, `MESSAGES` (sales-focused flows) → `OUTCOME_SALES`
+       - `APP_INSTALLS` → `OUTCOME_APP_PROMOTION`
      - `status`: Initial campaign status (default: PAUSED)
      - `special_ad_categories`: List of special ad categories if applicable
      - `daily_budget`: Daily budget in account currency (in cents)
@@ -159,6 +174,20 @@ For local installation configuration, authentication options, and advanced techn
      - `bid_strategy`: Bid strategy. Must be one of: `LOWEST_COST_WITHOUT_CAP`, `LOWEST_COST_WITH_BID_CAP`, `COST_CAP`, `LOWEST_COST_WITH_MIN_ROAS`.
    - Returns: Confirmation with new campaign details
 
+   - Example:
+     ```json
+     {
+       "name": "2025 - Bedroom Furniture - Awareness",
+       "account_id": "act_123456789012345",
+       "objective": "OUTCOME_AWARENESS",
+       "special_ad_categories": [],
+       "status": "PAUSED",
+       "buying_type": "AUCTION",
+       "bid_strategy": "LOWEST_COST_WITHOUT_CAP",
+       "daily_budget": 10000
+     }
+     ```
+
 7. `mcp_meta_ads_get_adsets`
    - Get ad sets for a Meta Ads account with optional filtering by campaign
    - Inputs:

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/meta_ads_mcp/__init__.py

@@ -1,13 +1,12 @@
 """
 Meta Ads MCP - Python Package
 
-This package provides a Meta Ads Marketing Cloud Platform (MCP) integration
-with the Claude LLM.
+This package provides a Meta Ads MCP integration
 """
 
 from meta_ads_mcp.core.server import main
 
-__version__ = "0.10.7"
+__version__ = "0.11.0"
 
 __all__ = [
     'get_ad_accounts',

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/meta_ads_mcp/core/accounts.py

@@ -1,14 +1,14 @@
 """Account-related functionality for Meta Ads API."""
 
 import json
-from typing import Optional
+from typing import Optional, Dict, Any
 from .api import meta_api_tool, make_api_request
 from .server import mcp_server
 
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_ad_accounts(access_token: str = None, user_id: str = "me", limit: int = 200) -> str:
+async def get_ad_accounts(access_token: Optional[str] = None, user_id: str = "me", limit: int = 200) -> str:
     """
     Get ad accounts accessible by a user.
     
@@ -30,13 +30,13 @@ async def get_ad_accounts(access_token: str = None, user_id: str = "me", limit:
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_account_info(access_token: str = None, account_id: str = None) -> str:
+async def get_account_info(account_id: str, access_token: Optional[str] = None) -> str:
     """
     Get detailed information about a specific ad account.
     
     Args:
+        account_id: Meta Ads account ID (format: act_XXXXXXXXX)
         access_token: Meta API access token (optional - will use cached token if not provided)
-        account_id: Meta Ads account ID (format: act_XXXXXXXXX) - REQUIRED
     """
     if not account_id:
         return {

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/meta_ads_mcp/core/ads.py

@@ -16,27 +16,21 @@ from .server import mcp_server
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_ads(access_token: str = None, account_id: str = None, limit: int = 10, 
+async def get_ads(account_id: str, access_token: Optional[str] = None, limit: int = 10, 
                  campaign_id: str = "", adset_id: str = "") -> str:
     """
     Get ads for a Meta Ads account with optional filtering.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         account_id: Meta Ads account ID (format: act_XXXXXXXXX)
+        access_token: Meta API access token (optional - will use cached token if not provided)
         limit: Maximum number of ads to return (default: 10)
         campaign_id: Optional campaign ID to filter by
         adset_id: Optional ad set ID to filter by
     """
-    # If no account ID is specified, try to get the first one for the user
+    # Require explicit account_id
     if not account_id:
-        accounts_json = await get_ad_accounts("me", json.dumps({"limit": 1}), access_token)
-        accounts_data = json.loads(accounts_json)
-        
-        if "data" in accounts_data and accounts_data["data"]:
-            account_id = accounts_data["data"][0]["id"]
-        else:
-            return json.dumps({"error": "No account ID specified and no accounts found for user"}, indent=2)
+        return json.dumps({"error": "No account ID specified"}, indent=2)
     
     # Prioritize adset_id over campaign_id - use adset-specific endpoint
     if adset_id:
@@ -67,13 +61,13 @@ async def get_ads(access_token: str = None, account_id: str = None, limit: int =
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_ad_details(access_token: str = None, ad_id: str = None) -> str:
+async def get_ad_details(ad_id: str, access_token: Optional[str] = None) -> str:
     """
     Get detailed information about a specific ad.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         ad_id: Meta Ads ad ID
+        access_token: Meta API access token (optional - will use cached token if not provided)
     """
     if not ad_id:
         return json.dumps({"error": "No ad ID provided"}, indent=2)
@@ -91,14 +85,14 @@ async def get_ad_details(access_token: str = None, ad_id: str = None) -> str:
 @mcp_server.tool()
 @meta_api_tool
 async def create_ad(
-    account_id: str = None,
-    name: str = None,
-    adset_id: str = None,
-    creative_id: str = None,
+    account_id: str,
+    name: str,
+    adset_id: str,
+    creative_id: str,
     status: str = "PAUSED",
-    bid_amount = None,
+    bid_amount: Optional[int] = None,
     tracking_specs: Optional[List[Dict[str, Any]]] = None,
-    access_token: str = None
+    access_token: Optional[str] = None
 ) -> str:
     """
     Create a new ad with an existing creative.
@@ -158,13 +152,13 @@ async def create_ad(
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_ad_creatives(access_token: str = None, ad_id: str = None) -> str:
+async def get_ad_creatives(ad_id: str, access_token: Optional[str] = None) -> str:
     """
     Get creative details for a specific ad. Best if combined with get_ad_image to get the full image.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         ad_id: Meta Ads ad ID
+        access_token: Meta API access token (optional - will use cached token if not provided)
     """
     if not ad_id:
         return json.dumps({"error": "No ad ID provided"}, indent=2)
@@ -186,13 +180,13 @@ async def get_ad_creatives(access_token: str = None, ad_id: str = None) -> str:
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_ad_image(access_token: str = None, ad_id: str = None) -> Image:
+async def get_ad_image(ad_id: str, access_token: Optional[str] = None) -> Image:
     """
     Get, download, and visualize a Meta ad image in one step. Useful to see the image in the LLM.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         ad_id: Meta Ads ad ID
+        access_token: Meta API access token (optional - will use cached token if not provided)
     
     Returns:
         The ad image ready for direct visual analysis
@@ -388,13 +382,13 @@ async def get_ad_image(access_token: str = None, ad_id: str = None) -> Image:
 
 @mcp_server.tool()
 @meta_api_tool
-async def save_ad_image_locally(access_token: str = None, ad_id: str = None, output_dir: str = "ad_images") -> str:
+async def save_ad_image_locally(ad_id: str, access_token: Optional[str] = None, output_dir: str = "ad_images") -> str:
     """
     Get, download, and save a Meta ad image locally, returning the file path.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         ad_id: Meta Ads ad ID
+        access_token: Meta API access token (optional - will use cached token if not provided)
         output_dir: Directory to save the image file (default: 'ad_images')
     
     Returns:
@@ -516,11 +510,11 @@ async def save_ad_image_locally(access_token: str = None, ad_id: str = None, out
 @meta_api_tool
 async def update_ad(
     ad_id: str,
-    status: str = None,
-    bid_amount: int = None,
-    tracking_specs = None,
-    creative_id: str = None,
-    access_token: str = None
+    status: Optional[str] = None,
+    bid_amount: Optional[int] = None,
+    tracking_specs: Optional[List[Dict[str, Any]]] = None,
+    creative_id: Optional[str] = None,
+    access_token: Optional[str] = None
 ) -> str:
     """
     Update an ad with new settings.
@@ -562,18 +556,18 @@ async def update_ad(
 @mcp_server.tool()
 @meta_api_tool
 async def upload_ad_image(
-    access_token: str = None,
-    account_id: str = None,
-    file: str = None,
-    image_url: str = None,
-    name: str = None
+    account_id: str,
+    access_token: Optional[str] = None,
+    file: Optional[str] = None,
+    image_url: Optional[str] = None,
+    name: Optional[str] = None
 ) -> str:
     """
     Upload an image to use in Meta Ads creatives.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         account_id: Meta Ads account ID (format: act_XXXXXXXXX)
+        access_token: Meta API access token (optional - will use cached token if not provided)
         file: Data URL or raw base64 string of the image (e.g., "data:image/png;base64,iVBORw0KG...")
         image_url: Direct URL to an image to fetch and upload
         name: Optional name for the image (default: filename)
@@ -671,7 +665,55 @@ async def upload_ad_image(
         print(f"Uploading image to Facebook Ad Account {account_id}")
         data = await make_api_request(endpoint, access_token, params, method="POST")
 
-        return json.dumps(data, indent=2)
+        # Normalize/structure the response for callers (e.g., to easily grab image_hash)
+        # Typical Graph API response shape:
+        # { "images": { "<hash>": { "hash": "<hash>", "url": "...", "width": ..., "height": ..., "name": "...", "status": 1 } } }
+        if isinstance(data, dict) and "images" in data and isinstance(data["images"], dict) and data["images"]:
+            images_dict = data["images"]
+            images_list = []
+            for hash_key, info in images_dict.items():
+                # Some responses may omit the nested hash, so ensure it's present
+                normalized = {
+                    "hash": (info.get("hash") or hash_key),
+                    "url": info.get("url"),
+                    "width": info.get("width"),
+                    "height": info.get("height"),
+                    "name": info.get("name"),
+                }
+                # Drop null/None values
+                normalized = {k: v for k, v in normalized.items() if v is not None}
+                images_list.append(normalized)
+
+            # Sort deterministically by hash
+            images_list.sort(key=lambda i: i.get("hash", ""))
+            primary_hash = images_list[0].get("hash") if images_list else None
+
+            result = {
+                "success": True,
+                "account_id": account_id,
+                "name": final_name,
+                "image_hash": primary_hash,
+                "images_count": len(images_list),
+                "images": images_list
+            }
+            return json.dumps(result, indent=2)
+
+        # If the API returned an error-like structure, surface it consistently
+        if isinstance(data, dict) and "error" in data:
+            return json.dumps({
+                "error": "Failed to upload image",
+                "details": data.get("error"),
+                "account_id": account_id,
+                "name": final_name
+            }, indent=2)
+
+        # Fallback: return a wrapped raw response to avoid breaking callers
+        return json.dumps({
+            "success": True,
+            "account_id": account_id,
+            "name": final_name,
+            "raw_response": data
+        }, indent=2)
 
     except Exception as e:
         return json.dumps({
@@ -683,29 +725,29 @@ async def upload_ad_image(
 @mcp_server.tool()
 @meta_api_tool
 async def create_ad_creative(
-    access_token: str = None,
-    account_id: str = None,
-    name: str = None,
-    image_hash: str = None,
-    page_id: str = None,
-    link_url: str = None,
-    message: str = None,
-    headline: str = None,
-    headlines: List[str] = None,
-    description: str = None,
-    descriptions: List[str] = None,
-    dynamic_creative_spec: Dict[str, Any] = None,
-    call_to_action_type: str = None,
-    instagram_actor_id: str = None
+    account_id: str,
+    image_hash: str,
+    access_token: Optional[str] = None,
+    name: Optional[str] = None,
+    page_id: Optional[str] = None,
+    link_url: Optional[str] = None,
+    message: Optional[str] = None,
+    headline: Optional[str] = None,
+    headlines: Optional[List[str]] = None,
+    description: Optional[str] = None,
+    descriptions: Optional[List[str]] = None,
+    dynamic_creative_spec: Optional[Dict[str, Any]] = None,
+    call_to_action_type: Optional[str] = None,
+    instagram_actor_id: Optional[str] = None
 ) -> str:
     """
     Create a new ad creative using an uploaded image hash.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         account_id: Meta Ads account ID (format: act_XXXXXXXXX)
-        name: Creative name
         image_hash: Hash of the uploaded image
+        access_token: Meta API access token (optional - will use cached token if not provided)
+        name: Creative name
         page_id: Facebook Page ID to be used for the ad
         link_url: Destination URL for the ad
         message: Ad copy/text
@@ -895,23 +937,23 @@ async def create_ad_creative(
 @mcp_server.tool()
 @meta_api_tool
 async def update_ad_creative(
-    access_token: str = None,
-    creative_id: str = None,
-    name: str = None,
-    message: str = None,
-    headline: str = None,
-    headlines: List[str] = None,
-    description: str = None,
-    descriptions: List[str] = None,
-    dynamic_creative_spec: Dict[str, Any] = None,
-    call_to_action_type: str = None
+    creative_id: str,
+    access_token: Optional[str] = None,
+    name: Optional[str] = None,
+    message: Optional[str] = None,
+    headline: Optional[str] = None,
+    headlines: Optional[List[str]] = None,
+    description: Optional[str] = None,
+    descriptions: Optional[List[str]] = None,
+    dynamic_creative_spec: Optional[Dict[str, Any]] = None,
+    call_to_action_type: Optional[str] = None
 ) -> str:
     """
     Update an existing ad creative with new content or settings.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         creative_id: Meta Ads creative ID to update
+        access_token: Meta API access token (optional - will use cached token if not provided)
         name: New creative name
         message: New ad copy/text
         headline: Single headline for simple ads (cannot be used with headlines)
@@ -1204,13 +1246,13 @@ async def _search_pages_by_name_core(access_token: str, account_id: str, search_
 
 @mcp_server.tool()
 @meta_api_tool
-async def search_pages_by_name(access_token: str = None, account_id: str = None, search_term: str = None) -> str:
+async def search_pages_by_name(account_id: str, access_token: Optional[str] = None, search_term: Optional[str] = None) -> str:
     """
     Search for pages by name within an account.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         account_id: Meta Ads account ID (format: act_XXXXXXXXX)
+        access_token: Meta API access token (optional - will use cached token if not provided)
         search_term: Search term to find pages by name (optional - returns all pages if not provided)
     
     Returns:
@@ -1227,13 +1269,13 @@ async def search_pages_by_name(access_token: str = None, account_id: str = None,
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_account_pages(access_token: str = None, account_id: str = None) -> str:
+async def get_account_pages(account_id: str, access_token: Optional[str] = None) -> str:
     """
     Get pages associated with a Meta Ads account.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         account_id: Meta Ads account ID (format: act_XXXXXXXXX)
+        access_token: Meta API access token (optional - will use cached token if not provided)
     
     Returns:
         JSON response with pages associated with the account

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/meta_ads_mcp/core/ads_library.py

@@ -14,10 +14,10 @@ if not DISABLE_ADS_LIBRARY:
     @mcp_server.tool()
     @meta_api_tool
     async def search_ads_archive(
-        access_token: str = None,
-        search_terms: str = None,
+        search_terms: str,
+        ad_reached_countries: List[str],
+        access_token: Optional[str] = None,
         ad_type: str = "ALL",
-        ad_reached_countries: List[str] = None,
         limit: int = 25,  # Default limit, adjust as needed
         fields: str = "ad_creation_time,ad_creative_body,ad_creative_link_caption,ad_creative_link_description,ad_creative_link_title,ad_delivery_start_time,ad_delivery_stop_time,ad_snapshot_url,currency,demographic_distribution,funding_entity,impressions,page_id,page_name,publisher_platform,region_distribution,spend"
     ) -> str:
@@ -25,10 +25,10 @@ if not DISABLE_ADS_LIBRARY:
         Search the Facebook Ads Library archive.
 
         Args:
-            access_token: Meta API access token (optional - will use cached token if not provided).
             search_terms: The search query for ads.
-            ad_type: Type of ads to search for (e.g., POLITICAL_AND_ISSUE_ADS, HOUSING_ADS, ALL).
             ad_reached_countries: List of country codes (e.g., ["US", "GB"]).
+            access_token: Meta API access token (optional - will use cached token if not provided).
+            ad_type: Type of ads to search for (e.g., POLITICAL_AND_ISSUE_ADS, HOUSING_ADS, ALL).
             limit: Maximum number of ads to return.
             fields: Comma-separated string of fields to retrieve for each ad.
 

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/meta_ads_mcp/core/adsets.py

@@ -9,25 +9,19 @@ from .server import mcp_server
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_adsets(access_token: str = None, account_id: str = None, limit: int = 10, campaign_id: str = "") -> str:
+async def get_adsets(account_id: str, access_token: Optional[str] = None, limit: int = 10, campaign_id: str = "") -> str:
     """
     Get ad sets for a Meta Ads account with optional filtering by campaign.
     
     Args:
-        access_token: Meta API access token (optional - will use cached token if not provided)
         account_id: Meta Ads account ID (format: act_XXXXXXXXX)
+        access_token: Meta API access token (optional - will use cached token if not provided)
         limit: Maximum number of ad sets to return (default: 10)
         campaign_id: Optional campaign ID to filter by
     """
-    # If no account ID is specified, try to get the first one for the user
+    # Require explicit account_id
     if not account_id:
-        accounts_json = await get_ad_accounts("me", json.dumps({"limit": 1}), access_token)
-        accounts_data = json.loads(accounts_json)
-        
-        if "data" in accounts_data and accounts_data["data"]:
-            account_id = accounts_data["data"][0]["id"]
-        else:
-            return json.dumps({"error": "No account ID specified and no accounts found for user"}, indent=2)
+        return json.dumps({"error": "No account ID specified"}, indent=2)
     
     # Change endpoint based on whether campaign_id is provided
     if campaign_id:
@@ -53,12 +47,12 @@ async def get_adsets(access_token: str = None, account_id: str = None, limit: in
 
 @mcp_server.tool()
 @meta_api_tool
-async def get_adset_details(access_token: str = None, adset_id: str = None) -> str:
+async def get_adset_details(adset_id: str, access_token: Optional[str] = None) -> str:
     """
     Get detailed information about a specific ad set.
     
     Args:
-        adset_id: Meta Ads ad set ID (required)
+        adset_id: Meta Ads ad set ID
         access_token: Meta API access token (optional - will use cached token if not provided)
     
     Example:
@@ -90,23 +84,23 @@ async def get_adset_details(access_token: str = None, adset_id: str = None) -> s
 @mcp_server.tool()
 @meta_api_tool
 async def create_adset(
-    account_id: str = None, 
-    campaign_id: str = None, 
-    name: str = None,
+    account_id: str, 
+    campaign_id: str, 
+    name: str,
+    optimization_goal: str,
+    billing_event: str,
     status: str = "PAUSED",
-    daily_budget = None,
-    lifetime_budget = None,
-    targeting: Dict[str, Any] = None,
-    optimization_goal: str = None,
-    billing_event: str = None,
-    bid_amount = None,
-    bid_strategy: str = None,
-    start_time: str = None,
-    end_time: str = None,
-    dsa_beneficiary: str = None,
-    promoted_object: Dict[str, Any] = None,
-    destination_type: str = None,
-    access_token: str = None
+    daily_budget: Optional[int] = None,
+    lifetime_budget: Optional[int] = None,
+    targeting: Optional[Dict[str, Any]] = None,
+    bid_amount: Optional[int] = None,
+    bid_strategy: Optional[str] = None,
+    start_time: Optional[str] = None,
+    end_time: Optional[str] = None,
+    dsa_beneficiary: Optional[str] = None,
+    promoted_object: Optional[Dict[str, Any]] = None,
+    destination_type: Optional[str] = None,
+    access_token: Optional[str] = None
 ) -> str:
     """
     Create a new ad set in a Meta Ads account.
@@ -115,13 +109,13 @@ async def create_adset(
         account_id: Meta Ads account ID (format: act_XXXXXXXXX)
         campaign_id: Meta Ads campaign ID this ad set belongs to
         name: Ad set name
+        optimization_goal: Conversion optimization goal (e.g., 'LINK_CLICKS', 'REACH', 'CONVERSIONS', 'APP_INSTALLS')
+        billing_event: How you're charged (e.g., 'IMPRESSIONS', 'LINK_CLICKS')
         status: Initial ad set status (default: PAUSED)
         daily_budget: Daily budget in account currency (in cents) as a string
         lifetime_budget: Lifetime budget in account currency (in cents) as a string
         targeting: Targeting specifications including age, location, interests, etc.
                   Use targeting_automation.advantage_audience=1 for automatic audience finding
-        optimization_goal: Conversion optimization goal (e.g., 'LINK_CLICKS', 'REACH', 'CONVERSIONS', 'APP_INSTALLS')
-        billing_event: How you're charged (e.g., 'IMPRESSIONS', 'LINK_CLICKS')
         bid_amount: Bid amount in account currency (in cents)
         bid_strategy: Bid strategy (e.g., 'LOWEST_COST', 'LOWEST_COST_WITH_BID_CAP')
         start_time: Start time in ISO 8601 format (e.g., '2023-12-01T12:00:00-0800')
@@ -293,10 +287,10 @@ async def create_adset(
 
 @mcp_server.tool()
 @meta_api_tool
-async def update_adset(adset_id: str, frequency_control_specs: List[Dict[str, Any]] = None, bid_strategy: str = None, 
-                        bid_amount: int = None, status: str = None, targeting: Dict[str, Any] = None, 
-                        optimization_goal: str = None, daily_budget = None, lifetime_budget = None, 
-                        access_token: str = None) -> str:
+async def update_adset(adset_id: str, frequency_control_specs: Optional[List[Dict[str, Any]]] = None, bid_strategy: Optional[str] = None, 
+                        bid_amount: Optional[int] = None, status: Optional[str] = None, targeting: Optional[Dict[str, Any]] = None, 
+                        optimization_goal: Optional[str] = None, daily_budget: Optional[int] = None, lifetime_budget: Optional[int] = None, 
+                        access_token: Optional[str] = None) -> str:
     """
     Update an ad set with new settings including frequency caps and budgets.
     

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/meta_ads_mcp/core/api.py

@@ -316,6 +316,6 @@ def meta_api_tool(func):
             return result
         except Exception as e:
             logger.error(f"Error in {func.__name__}: {str(e)}")
-            return {"error": str(e)}
+            return json.dumps({"error": str(e)}, indent=2)
     
     return wrapper 

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/meta_ads_mcp/core/auth.py

@@ -87,7 +87,7 @@ meta_config = MetaConfig()
 
 class TokenInfo:
     """Stores token information including expiration"""
-    def __init__(self, access_token: str, expires_in: int = None, user_id: str = None):
+    def __init__(self, access_token: str, expires_in: Optional[int] = None, user_id: Optional[str] = None):
         self.access_token = access_token
         self.expires_in = expires_in
         self.user_id = user_id

{meta_ads_mcp-0.10.7 → meta_ads_mcp-0.11.0}/meta_ads_mcp/core/authentication.py

@@ -25,6 +25,7 @@ Environment Variables:
 """
 
 import json
+from typing import Optional
 import asyncio
 import os
 from .api import meta_api_tool
@@ -37,7 +38,7 @@ from .pipeboard_auth import pipeboard_auth_manager
 ENABLE_LOGIN_LINK = not bool(os.environ.get("META_ADS_DISABLE_LOGIN_LINK", ""))
 
 
-async def get_login_link(access_token: str = None) -> str:
+async def get_login_link(access_token: Optional[str] = None) -> str:
     """
     Get a clickable login link for Meta Ads authentication.