Refactor tools to return structured data and enable agent-driven formatting

- Changed all Meta Ads tools to return structured data (lists of dicts) instead of pre-formatted strings
- Updated output_type from "string" to "any" for all 5 tools (GetAdAccountsTool, GetCampaignsTool, GetAdSetsTool, GetAdsTool, GetAdInsightsTool)
- Removed format_as_itemized_list() function and all its usages
- Updated CodeAgent instructions to format structured data into user-friendly markdown (tables, bullet lists)
- Enhanced system message to inform agent about structured data handling capabilities
- Added example query demonstrating data filtering: "List my ad accounts and tell me which ones are active"

Benefits:
- Agent can now filter, sort, and analyze data programmatically
- More flexible and contextual response generation
- Better support for analytical queries
- Cleaner separation between data retrieval and presentation

app.py

@@ -66,7 +66,10 @@ User request: {message}
 Instructions:
 - Write code in <code> tags and execute it immediately
 - Use the available tools: get_ad_accounts(), get_campaigns(), get_adsets(), get_ads(), get_ad_insights()
-- After getting results, call final_answer() with the results
+- The tools return structured data (lists of dictionaries)
+- Format the results in a user-friendly markdown format before calling final_answer()
+- For lists of items, use markdown tables or bullet lists with clear formatting
+- Include relevant fields and make the output easy to read
 - Be direct and efficient"""
         
         # Run the agent
@@ -86,7 +89,7 @@ chatbot = gr.ChatInterface(
     type="messages",
     additional_inputs=[
         gr.Textbox(
-            value="You are a Meta Ads assistant. You can help users retrieve information about their ad accounts, campaigns, ad sets, ads, and performance insights using the Meta Ads API. Use the available tools to answer user queries.",
+            value="You are a Meta Ads assistant. You can help users retrieve information about their ad accounts, campaigns, ad sets, ads, and performance insights using the Meta Ads API. The tools return structured data (lists of dictionaries) that you can process, filter, and analyze. Always format the results in user-friendly markdown (tables, bullet lists, etc.) before presenting them to the user.",
             label="System message",
             lines=3
         ),
@@ -116,6 +119,7 @@ chatbot = gr.ChatInterface(
     """,
     examples=[
         ["Show me all my ad accounts"],
+        ["List my ad accounts and tell me which ones are active"],
         ["Get campaigns for account act_123456789"],
         ["Get ad sets for campaign 123456789"],
         ["What are the insights for my account in the last 30 days?"],

tools/meta_ads_tools.py

@@ -26,20 +26,6 @@ def initialize_facebook_api():
     )
 
 
-def format_as_itemized_list(data_list):
-    """Format a list of dictionaries as an itemized list."""
-    if not data_list:
-        return "No results found."
-    
-    formatted_output = []
-    for idx, item in enumerate(data_list, 1):
-        formatted_output.append(f"\n--- Item {idx} ---")
-        for key, value in item.items():
-            formatted_output.append(f"* {key}: {value}")
-    
-    return "\n".join(formatted_output)
-
-
 class GetAdAccountsTool(Tool):
     name = "get_ad_accounts"
     description = """
@@ -52,7 +38,7 @@ class GetAdAccountsTool(Tool):
         accounts = get_ad_accounts()
     """
     inputs = {}
-    output_type = "string"
+    output_type = "any"
 
     def forward(self):
         """Get all ad accounts."""
@@ -77,7 +63,7 @@ class GetAdAccountsTool(Tool):
                     'currency': account.get(AdAccount.Field.currency, 'N/A'),
                 })
             
-            return format_as_itemized_list(result)
+            return result
         except Exception as e:
             return f"Error retrieving ad accounts: {str(e)}"
 
@@ -102,7 +88,7 @@ class GetCampaignsTool(Tool):
             "description": "The ad account ID (e.g., 'act_123456789')"
         }
     }
-    output_type = "string"
+    output_type = "any"
 
     def forward(self, account_id: str):
         """Get campaigns for an ad account."""
@@ -130,7 +116,7 @@ class GetCampaignsTool(Tool):
                     'lifetime_budget': campaign.get(Campaign.Field.lifetime_budget, 'N/A'),
                 })
             
-            return format_as_itemized_list(result)
+            return result
         except Exception as e:
             return f"Error retrieving campaigns: {str(e)}"
 
@@ -155,7 +141,7 @@ class GetAdSetsTool(Tool):
             "description": "The campaign ID"
         }
     }
-    output_type = "string"
+    output_type = "any"
 
     def forward(self, campaign_id: str):
         """Get ad sets for a campaign."""
@@ -185,7 +171,7 @@ class GetAdSetsTool(Tool):
                     'billing_event': adset.get(AdSet.Field.billing_event, 'N/A'),
                 })
             
-            return format_as_itemized_list(result)
+            return result
         except Exception as e:
             return f"Error retrieving ad sets: {str(e)}"
 
@@ -210,7 +196,7 @@ class GetAdsTool(Tool):
             "description": "The ad set ID"
         }
     }
-    output_type = "string"
+    output_type = "any"
 
     def forward(self, adset_id: str):
         """Get ads for an ad set."""
@@ -234,7 +220,7 @@ class GetAdsTool(Tool):
                     'creative_id': ad.get(Ad.Field.creative, {}).get('id', 'N/A'),
                 })
             
-            return format_as_itemized_list(result)
+            return result
         except Exception as e:
             return f"Error retrieving ads: {str(e)}"
 
@@ -265,7 +251,7 @@ class GetAdInsightsTool(Tool):
             "nullable": True
         }
     }
-    output_type = "string"
+    output_type = "any"
 
     def forward(self, object_id: str, date_preset: str = "last_7d"):
         """Get insights for an ad object."""
@@ -309,7 +295,7 @@ class GetAdInsightsTool(Tool):
                     'actions': insight.get('actions', []),
                 })
             
-            return format_as_itemized_list(result)
+            return result
         except Exception as e:
             return f"Error retrieving insights: {str(e)}"