camel-ai 0.2.76a2__py3-none-any.whl → 0.2.76a4__py3-none-any.whl

camel/__init__.py

@@ -14,7 +14,7 @@
 
 from camel.logger import disable_logging, enable_logging, set_log_level
 
-__version__ = '0.2.76a2'
+__version__ = '0.2.76a4'
 
 __all__ = [
     '__version__',

camel/societies/workforce/workforce.py

@@ -2075,8 +2075,40 @@ class Workforce(BaseNode):
             TaskAssignResult: Assignment result containing task assignments
                 with their dependencies.
         """
+        # Wait for workers to be ready before assignment with exponential
+        # backoff
+        worker_readiness_timeout = 2.0  # Maximum wait time in seconds
+        worker_readiness_check_interval = 0.05  # Initial check interval
+        start_time = time.time()
+        check_interval = worker_readiness_check_interval
+        backoff_multiplier = 1.5  # Exponential backoff factor
+        max_interval = 0.5  # Cap the maximum interval
+
+        while (time.time() - start_time) < worker_readiness_timeout:
+            valid_worker_ids = self._get_valid_worker_ids()
+            if len(valid_worker_ids) > 0:
+                elapsed = time.time() - start_time
+                logger.debug(
+                    f"Workers ready after {elapsed:.3f}s: "
+                    f"{len(valid_worker_ids)} workers available"
+                )
+                break
+
+            await asyncio.sleep(check_interval)
+            # Exponential backoff with cap
+            check_interval = min(
+                check_interval * backoff_multiplier, max_interval
+            )
+        else:
+            # Timeout reached, log warning but continue
+            logger.warning(
+                f"Worker readiness timeout after "
+                f"{worker_readiness_timeout}s, "
+                f"proceeding with {len(self._children)} children"
+            )
+            valid_worker_ids = self._get_valid_worker_ids()
+
         self.coordinator_agent.reset()
-        valid_worker_ids = self._get_valid_worker_ids()
 
         logger.debug(
             f"Sending batch assignment request to coordinator "
@@ -2110,7 +2142,24 @@ class Workforce(BaseNode):
                 invalid_assignments, tasks, valid_worker_ids
             )
         )
-        all_assignments = valid_assignments + retry_and_fallback_assignments
+
+        # Combine assignments with deduplication, prioritizing retry results
+        assignment_map = {a.task_id: a for a in valid_assignments}
+        assignment_map.update(
+            {a.task_id: a for a in retry_and_fallback_assignments}
+        )
+        all_assignments = list(assignment_map.values())
+
+        # Log any overwrites for debugging
+        valid_task_ids = {a.task_id for a in valid_assignments}
+        retry_task_ids = {a.task_id for a in retry_and_fallback_assignments}
+        overlap_task_ids = valid_task_ids & retry_task_ids
+
+        if overlap_task_ids:
+            logger.warning(
+                f"Retry assignments overrode {len(overlap_task_ids)} "
+                f"valid assignments for tasks: {sorted(overlap_task_ids)}"
+            )
 
         # Update Task.dependencies for all final assignments
         self._update_task_dependencies_from_assignments(all_assignments, tasks)

camel/toolkits/mcp_toolkit.py

@@ -18,6 +18,8 @@ import warnings
 from contextlib import AsyncExitStack
 from typing import Any, Dict, List, Optional
 
+from typing_extensions import TypeGuard
+
 from camel.logger import get_logger
 from camel.toolkits import BaseToolkit, FunctionTool
 from camel.utils.commons import run_async
@@ -43,6 +45,187 @@ class MCPToolError(Exception):
     pass
 
 
+_EMPTY_SCHEMA = {
+    "additionalProperties": False,
+    "type": "object",
+    "properties": {},
+    "required": [],
+}
+
+
+def ensure_strict_json_schema(schema: dict[str, Any]) -> dict[str, Any]:
+    r"""Mutates the given JSON schema to ensure it conforms to the
+    `strict` standard that the OpenAI API expects.
+    """
+    if schema == {}:
+        return _EMPTY_SCHEMA
+    return _ensure_strict_json_schema(schema, path=(), root=schema)
+
+
+def _ensure_strict_json_schema(
+    json_schema: object,
+    *,
+    path: tuple[str, ...],
+    root: dict[str, object],
+) -> dict[str, Any]:
+    if not is_dict(json_schema):
+        raise TypeError(
+            f"Expected {json_schema} to be a dictionary; path={path}"
+        )
+
+    defs = json_schema.get("$defs")
+    if is_dict(defs):
+        for def_name, def_schema in defs.items():
+            _ensure_strict_json_schema(
+                def_schema, path=(*path, "$defs", def_name), root=root
+            )
+
+    definitions = json_schema.get("definitions")
+    if is_dict(definitions):
+        for definition_name, definition_schema in definitions.items():
+            _ensure_strict_json_schema(
+                definition_schema,
+                path=(*path, "definitions", definition_name),
+                root=root,
+            )
+
+    typ = json_schema.get("type")
+    if typ == "object" and "additionalProperties" not in json_schema:
+        json_schema["additionalProperties"] = False
+    elif (
+        typ == "object"
+        and "additionalProperties" in json_schema
+        and json_schema["additionalProperties"]
+    ):
+        raise ValueError(
+            "additionalProperties should not be set for object types. This "
+            "could be because you're using an older version of Pydantic, or "
+            "because you configured additional properties to be allowed. If "
+            "you really need this, update the function or output tool "
+            "to not use a strict schema."
+        )
+
+    # object types
+    # { 'type': 'object', 'properties': { 'a':  {...} } }
+    properties = json_schema.get("properties")
+    if is_dict(properties):
+        json_schema["required"] = list(properties.keys())
+        json_schema["properties"] = {
+            key: _ensure_strict_json_schema(
+                prop_schema, path=(*path, "properties", key), root=root
+            )
+            for key, prop_schema in properties.items()
+        }
+
+    # arrays
+    # { 'type': 'array', 'items': {...} }
+    items = json_schema.get("items")
+    if is_dict(items):
+        json_schema["items"] = _ensure_strict_json_schema(
+            items, path=(*path, "items"), root=root
+        )
+
+    # unions
+    any_of = json_schema.get("anyOf")
+    if is_list(any_of):
+        json_schema["anyOf"] = [
+            _ensure_strict_json_schema(
+                variant, path=(*path, "anyOf", str(i)), root=root
+            )
+            for i, variant in enumerate(any_of)
+        ]
+
+    # intersections
+    all_of = json_schema.get("allOf")
+    if is_list(all_of):
+        if len(all_of) == 1:
+            json_schema.update(
+                _ensure_strict_json_schema(
+                    all_of[0], path=(*path, "allOf", "0"), root=root
+                )
+            )
+            json_schema.pop("allOf")
+        else:
+            json_schema["allOf"] = [
+                _ensure_strict_json_schema(
+                    entry, path=(*path, "allOf", str(i)), root=root
+                )
+                for i, entry in enumerate(all_of)
+            ]
+
+    # strip `None` defaults as there's no meaningful distinction here
+    # the schema will still be `nullable` and the model will default
+    # to using `None` anyway
+    if json_schema.get("default", None) is None:
+        json_schema.pop("default", None)
+
+    # we can't use `$ref`s if there are also other properties defined, e.g.
+    # `{"$ref": "...", "description": "my description"}`
+    #
+    # so we unravel the ref
+    # `{"type": "string", "description": "my description"}`
+    ref = json_schema.get("$ref")
+    if ref and has_more_than_n_keys(json_schema, 1):
+        assert isinstance(ref, str), f"Received non-string $ref - {ref}"
+
+        resolved = resolve_ref(root=root, ref=ref)
+        if not is_dict(resolved):
+            raise ValueError(
+                f"Expected `$ref: {ref}` to resolved to a dictionary but got "
+                f"{resolved}"
+            )
+
+        # properties from the json schema take priority
+        # over the ones on the `$ref`
+        json_schema.update({**resolved, **json_schema})
+        json_schema.pop("$ref")
+        # Since the schema expanded from `$ref` might not
+        # have `additionalProperties: false` applied
+        # we call `_ensure_strict_json_schema` again to fix the inlined
+        # schema and ensure it's valid
+        return _ensure_strict_json_schema(json_schema, path=path, root=root)
+
+    return json_schema
+
+
+def resolve_ref(*, root: dict[str, object], ref: str) -> object:
+    if not ref.startswith("#/"):
+        raise ValueError(
+            f"Unexpected $ref format {ref!r}; Does not start with #/"
+        )
+
+    path = ref[2:].split("/")
+    resolved = root
+    for key in path:
+        value = resolved[key]
+        assert is_dict(value), (
+            f"encountered non-dictionary entry while resolving {ref} - "
+            f"{resolved}"
+        )
+        resolved = value
+
+    return resolved
+
+
+def is_dict(obj: object) -> TypeGuard[dict[str, object]]:
+    # just pretend that we know there are only `str` keys
+    # as that check is not worth the performance cost
+    return isinstance(obj, dict)
+
+
+def is_list(obj: object) -> TypeGuard[list[object]]:
+    return isinstance(obj, list)
+
+
+def has_more_than_n_keys(obj: dict[str, object], n: int) -> bool:
+    i = 0
+    for _ in obj.keys():
+        i += 1
+        if i > n:
+            return True
+    return False
+
+
 class MCPToolkit(BaseToolkit):
     r"""MCPToolkit provides a unified interface for managing multiple
     MCP server connections and their tools.
@@ -476,367 +659,149 @@ class MCPToolkit(BaseToolkit):
             raise ValueError(error_msg) from e
 
     def _ensure_strict_tool_schema(self, tool: FunctionTool) -> FunctionTool:
-        r"""Ensure a tool has a strict schema compatible with OpenAI's
-        requirements according to the structured outputs specification.
-
-        Args:
-            tool (FunctionTool): The tool to check and update if necessary.
-
-        Returns:
-            FunctionTool: The tool with a strict schema.
+        r"""Ensure a tool has a strict schema compatible with
+        OpenAI's requirements.
+
+        Strategy:
+        - Ensure parameters exist with at least an empty properties object
+            (OpenAI requirement).
+        - Try converting parameters to strict using ensure_strict_json_schema.
+        - If conversion fails, mark function.strict = False and
+            keep best-effort parameters.
         """
         try:
             schema = tool.get_openai_tool_schema()
 
-            # Helper functions for validation and transformation
-            def _validate_and_fix_schema(obj, path="", in_root=True):
-                r"""Recursively validate and fix schema to meet strict
-                requirements.
-                """
-                if isinstance(obj, dict):
-                    # Check if this is the root object
-                    if in_root and path == "":
-                        # Root must be an object, not anyOf
-                        if "anyOf" in obj and "type" not in obj:
-                            raise ValueError(
-                                "Root object must not be anyOf and must "
-                                "be an object"
-                            )
-                        if obj.get("type") and obj["type"] != "object":
-                            raise ValueError(
-                                "Root object must have type 'object'"
-                            )
-
-                    # Handle object types
-                    if obj.get("type") == "object":
-                        # Ensure additionalProperties is false
-                        obj["additionalProperties"] = False
-
-                        # Process properties
-                        if "properties" in obj:
-                            props = obj["properties"]
-                            # Only set required if it doesn't exist or needs
-                            # updating
-                            if "required" not in obj:
-                                # If no required field exists, make all fields
-                                # required
-                                obj["required"] = list(props.keys())
-                            else:
-                                # Ensure required field only contains valid
-                                # property names
-                                existing_required = obj.get("required", [])
-                                valid_required = [
-                                    req
-                                    for req in existing_required
-                                    if req in props
-                                ]
-                                # Add any missing properties to required
-                                for prop_name in props:
-                                    if prop_name not in valid_required:
-                                        valid_required.append(prop_name)
-                                obj["required"] = valid_required
-
-                            # Recursively process each property
-                            for prop_name, prop_schema in props.items():
-                                _validate_and_fix_schema(
-                                    prop_schema, f"{path}.{prop_name}", False
-                                )
-
-                    # Handle arrays
-                    elif obj.get("type") == "array":
-                        if "items" in obj:
-                            _validate_and_fix_schema(
-                                obj["items"], f"{path}.items", False
-                            )
+            def _has_strict_mode_incompatible_features(json_schema):
+                r"""Check if schema has features incompatible
+                with OpenAI strict mode."""
 
-                    # Handle anyOf
-                    elif "anyOf" in obj:
-                        # Validate anyOf schemas
-                        for i, schema in enumerate(obj["anyOf"]):
-                            _validate_and_fix_schema(
-                                schema, f"{path}.anyOf[{i}]", False
-                            )
-
-                    # Handle string format validation
-                    elif obj.get("type") == "string":
-                        if "format" in obj:
-                            allowed_formats = [
-                                "date-time",
-                                "time",
-                                "date",
-                                "duration",
-                                "email",
-                                "hostname",
-                                "ipv4",
-                                "ipv6",
-                                "uuid",
-                            ]
-                            if obj["format"] not in allowed_formats:
-                                del obj["format"]  # Remove unsupported format
-
-                    # Handle number/integer validation
-                    elif obj.get("type") in ["number", "integer"]:
-                        # These properties are supported
-                        supported_props = [
-                            "multipleOf",
-                            "maximum",
-                            "exclusiveMaximum",
-                            "minimum",
-                            "exclusiveMinimum",
-                        ]
-                        # Remove any unsupported properties
-                        for key in list(obj.keys()):
-                            if key not in [
-                                *supported_props,
-                                "type",
-                                "description",
-                                "default",
-                            ]:
-                                del obj[key]
-
-                    # Process nested structures
-                    for key in ["allOf", "oneOf", "$defs", "definitions"]:
-                        if key in obj:
-                            if isinstance(obj[key], list):
-                                for i, item in enumerate(obj[key]):
-                                    _validate_and_fix_schema(
-                                        item, f"{path}.{key}[{i}]", False
-                                    )
-                            elif isinstance(obj[key], dict):
-                                for def_name, def_schema in obj[key].items():
-                                    _validate_and_fix_schema(
-                                        def_schema,
-                                        f"{path}.{key}.{def_name}",
-                                        False,
-                                    )
-
-                elif isinstance(obj, list):
-                    for i, item in enumerate(obj):
-                        _validate_and_fix_schema(item, f"{path}[{i}]", False)
-
-            def _check_schema_limits(obj, counts=None):
-                r"""Check if schema exceeds OpenAI limits."""
-                if counts is None:
-                    counts = {
-                        "properties": 0,
-                        "depth": 0,
-                        "enums": 0,
-                        "string_length": 0,
-                    }
+                def _check_incompatible(obj, path=""):
+                    if not isinstance(obj, dict):
+                        return False
 
-                def _count_properties(o, depth=0):
-                    if isinstance(o, dict):
-                        if depth > 5:
-                            raise ValueError(
-                                "Schema exceeds maximum nesting depth of 5"
+                    # Check for allOf in array items (known to cause issues)
+                    if "items" in obj and isinstance(obj["items"], dict):
+                        items_schema = obj["items"]
+                        if "allOf" in items_schema:
+                            logger.debug(
+                                f"Found allOf in array items at {path}"
                             )
+                            return True
+                        # Recursively check items schema
+                        if _check_incompatible(items_schema, f"{path}.items"):
+                            return True
+
+                    # Check for other potentially problematic patterns
+                    # anyOf/oneOf in certain contexts can also cause issues
+                    if (
+                        "anyOf" in obj and len(obj["anyOf"]) > 10
+                    ):  # Large unions can be problematic
+                        return True
+
+                    # Recursively check nested objects
+                    for key in [
+                        "properties",
+                        "additionalProperties",
+                        "patternProperties",
+                    ]:
+                        if key in obj and isinstance(obj[key], dict):
+                            if key == "properties":
+                                for prop_name, prop_schema in obj[key].items():
+                                    if isinstance(
+                                        prop_schema, dict
+                                    ) and _check_incompatible(
+                                        prop_schema,
+                                        f"{path}.{key}.{prop_name}",
+                                    ):
+                                        return True
+                            elif _check_incompatible(
+                                obj[key], f"{path}.{key}"
+                            ):
+                                return True
 
-                        if o.get("type") == "object" and "properties" in o:
-                            counts["properties"] += len(o["properties"])
-                            for prop in o["properties"].values():
-                                _count_properties(prop, depth + 1)
-
-                        if "enum" in o:
-                            counts["enums"] += len(o["enum"])
-                            if isinstance(o["enum"], list):
-                                for val in o["enum"]:
-                                    if isinstance(val, str):
-                                        counts["string_length"] += len(val)
-
-                        # Count property names
-                        if "properties" in o:
-                            for name in o["properties"].keys():
-                                counts["string_length"] += len(name)
-
-                        # Process nested structures
-                        for key in ["items", "allOf", "oneOf", "anyOf"]:
-                            if key in o:
-                                if isinstance(o[key], dict):
-                                    _count_properties(o[key], depth)
-                                elif isinstance(o[key], list):
-                                    for item in o[key]:
-                                        _count_properties(item, depth)
-
-                _count_properties(obj)
-
-                # Check limits, reference: https://platform.openai.com/docs/guides/structured-outputs?api-mode=responses#objects-have-limitations-on-nesting-depth-and-size # noqa: E501
-                if counts["properties"] > 5000:
-                    raise ValueError(
-                        "Schema exceeds maximum of 5000 properties"
-                    )
-                if counts["enums"] > 1000:
-                    raise ValueError(
-                        "Schema exceeds maximum of 1000 enum values"
-                    )
-                if counts["string_length"] > 120000:
-                    raise ValueError(
-                        "Schema exceeds maximum total string length of 120000"
-                    )
-
-                return True
+                    # Check arrays and unions
+                    for key in ["allOf", "anyOf", "oneOf"]:
+                        if key in obj and isinstance(obj[key], list):
+                            for i, item in enumerate(obj[key]):
+                                if isinstance(
+                                    item, dict
+                                ) and _check_incompatible(
+                                    item, f"{path}.{key}[{i}]"
+                                ):
+                                    return True
 
-            # Check if schema has any issues that prevent strict mode
-            def _has_strict_mode_issues(obj):
-                r"""Check for any issues that would prevent strict mode."""
-                issues = []
+                    return False
 
-                def _check_issues(o, path=""):
-                    if isinstance(o, dict):
-                        # Check for additionalProperties: true
-                        if o.get("additionalProperties") is True:
-                            issues.append(
-                                f"additionalProperties: true at {path}"
-                            )
+                return _check_incompatible(json_schema)
 
-                        # Check for unsupported keywords
-                        unsupported = [
-                            "not",
-                            "dependentRequired",
-                            "dependentSchemas",
-                            "if",
-                            "then",
-                            "else",
-                            "patternProperties",
-                        ]
-                        for keyword in unsupported:
-                            if keyword in o:
-                                issues.append(
-                                    f"Unsupported keyword '{keyword}' "
-                                    f"at {path}"
-                                )
-
-                        # Recursively check
-                        for key, value in o.items():
-                            if isinstance(value, (dict, list)):
-                                _check_issues(value, f"{path}.{key}")
-
-                    elif isinstance(o, list):
-                        for i, item in enumerate(o):
-                            _check_issues(item, f"{path}[{i}]")
-
-                _check_issues(obj)
-                return issues
-
-            # Check if already strict and compliant
-            if schema.get("function", {}).get("strict") is True:
-                # Validate it's actually compliant
-                try:
-                    params = schema["function"].get("parameters", {})
-                    if params:
-                        _validate_and_fix_schema(params)
-                        _check_schema_limits(params)
-                    return tool
-                except Exception:
-                    # Not actually compliant, continue to fix it
-                    pass
-
-            # Apply sanitization first to handle optional fields properly
+            # Apply sanitization if available
             if "function" in schema:
-                # Apply the sanitization function first
-                from camel.toolkits.function_tool import (
-                    sanitize_and_enforce_required,
-                )
+                try:
+                    from camel.toolkits.function_tool import (
+                        sanitize_and_enforce_required,
+                    )
 
-                schema = sanitize_and_enforce_required(schema)
-
-                # Special handling for schemas with additionalProperties that
-                # aren't false These can't use strict mode
-                def _has_open_props(obj, path=""):
-                    """Check if any object has additionalProperties that
-                    isn't false."""
-                    if isinstance(obj, dict):
-                        if (
-                            obj.get("type") == "object"
-                            and "additionalProperties" in obj
-                        ):
-                            if obj["additionalProperties"] is not False:
-                                return True
+                    schema = sanitize_and_enforce_required(schema)
+                except ImportError:
+                    logger.debug("sanitize_and_enforce_required not available")
 
-                        # Recurse through the schema
-                        for key, value in obj.items():
-                            if key in [
-                                "properties",
-                                "items",
-                                "allOf",
-                                "oneOf",
-                                "anyOf",
-                            ]:
-                                if isinstance(value, dict):
-                                    if _has_open_props(value, f"{path}.{key}"):
-                                        return True
-                                elif isinstance(value, list):
-                                    for i, item in enumerate(value):
-                                        if _has_open_props(
-                                            item,
-                                            f"{path}.{key}[{i}]",
-                                        ):
-                                            return True
-                            elif isinstance(value, dict) and key not in [
-                                "description",
-                                "type",
-                                "enum",
-                            ]:
-                                if _has_open_props(value, f"{path}.{key}"):
-                                    return True
-                    return False
+                parameters = schema["function"].get("parameters", {})
+                if not parameters:
+                    # Empty parameters - use minimal valid schema
+                    parameters = {
+                        "type": "object",
+                        "properties": {},
+                        "additionalProperties": False,
+                    }
+                    schema["function"]["parameters"] = parameters
 
-                # Check if schema has dynamic additionalProperties
-                if _has_open_props(schema["function"].get("parameters", {})):
-                    # Can't use strict mode with dynamic additionalProperties
-                    schema["function"]["strict"] = False
-                    tool.set_openai_tool_schema(schema)
-                    logger.warning(
-                        f"Tool '{tool.get_function_name()}' has "
-                        f"dynamic additionalProperties and cannot use "
-                        f"strict mode"
-                    )
-                    return tool
+                # MCP spec doesn't require 'properties', but OpenAI spec does
+                if (
+                    parameters.get("type") == "object"
+                    and "properties" not in parameters
+                ):
+                    parameters["properties"] = {}
 
-                # Now check for blocking issues after sanitization
-                issues = _has_strict_mode_issues(schema)
-                if issues:
-                    # Can't use strict mode
+                try:
+                    # _check_schema_limits(parameters)
+
+                    # Check for OpenAI strict mode incompatible features
+                    if _has_strict_mode_incompatible_features(parameters):
+                        raise ValueError(
+                            "Schema contains features "
+                            "incompatible with strict mode"
+                        )
+
+                    strict_params = ensure_strict_json_schema(parameters)
+                    schema["function"]["parameters"] = strict_params
+                    schema["function"]["strict"] = True
+                except Exception as e:
+                    # Fallback to non-strict mode on any failure
                     schema["function"]["strict"] = False
-                    tool.set_openai_tool_schema(schema)
                     logger.warning(
-                        f"Tool '{tool.get_function_name()}' has "
-                        f"issues preventing strict mode: "
-                        f"{'; '.join(issues[:3])}{'...' if len(issues) > 3 else ''}"  # noqa: E501
+                        f"Tool '{tool.get_function_name()}' "
+                        f"cannot use strict mode: {e}"
                     )
-                    return tool
-
-                # Enable strict mode
-                schema["function"]["strict"] = True
-
-                parameters = schema["function"].get("parameters", {})
-                if parameters:
-                    # Validate and fix the parameters schema
-                    _validate_and_fix_schema(parameters)
-
-                    # Check schema limits
-                    _check_schema_limits(parameters)
 
             tool.set_openai_tool_schema(schema)
-            logger.debug(
-                f"Updated tool '{tool.get_function_name()}' to strict mode"
-            )
 
         except Exception as e:
-            # If we can't make it strict, disable strict mode
+            # Final fallback - ensure tool still works
             try:
-                if "function" in schema:
-                    schema["function"]["strict"] = False
-                tool.set_openai_tool_schema(schema)
+                current_schema = tool.get_openai_tool_schema()
+                if "function" in current_schema:
+                    current_schema["function"]["strict"] = False
+                tool.set_openai_tool_schema(current_schema)
                 logger.warning(
-                    f"Failed to ensure strict schema for "
-                    f"tool '{tool.get_function_name()}': {str(e)[:100]}. "
-                    f"Setting strict=False."
+                    f"Error processing schema for tool "
+                    f"'{tool.get_function_name()}': {str(e)[:100]}. "
+                    f"Using non-strict mode."
                 )
             except Exception as inner_e:
-                # If even setting strict=False fails, log the error
                 logger.error(
-                    f"Critical error processing "
-                    f"tool '{tool.get_function_name()}': {inner_e}. "
+                    f"Critical error processing tool "
+                    f"'{tool.get_function_name()}': {inner_e}. "
                     f"Tool may not function correctly."
                 )
 
@@ -879,6 +844,7 @@ class MCPToolkit(BaseToolkit):
             )
 
         all_tools = []
+        seen_names: set[str] = set()
         for i, client in enumerate(self.clients):
             try:
                 client_tools = client.get_tools()
@@ -887,6 +853,14 @@ class MCPToolkit(BaseToolkit):
                 strict_tools = []
                 for tool in client_tools:
                     strict_tool = self._ensure_strict_tool_schema(tool)
+                    name = strict_tool.get_function_name()
+                    if name in seen_names:
+                        logger.warning(
+                            f"Duplicate tool name detected and "
+                            f"skipped: '{name}' from client {i+1}"
+                        )
+                        continue
+                    seen_names.add(name)
                     strict_tools.append(strict_tool)
 
                 all_tools.extend(strict_tools)

camel/utils/context_utils.py

@@ -393,3 +393,55 @@ class ContextUtility:
             str: The session ID.
         """
         return self.session_id
+
+    def load_markdown_context_to_memory(
+        self, agent: "ChatAgent", filename: str
+    ) -> str:
+        r"""Load context from a markdown file and append it to agent memory.
+        Preserves existing conversation history without wiping it.
+
+        Args:
+            agent (ChatAgent): The agent to append context to.
+            filename (str): Name of the markdown file (without .md extension).
+
+        Returns:
+            str: Status message indicating success or failure with details.
+        """
+        try:
+            content = self.load_markdown_file(filename)
+
+            if not content.strip():
+                return f"Context file not found or empty: {filename}"
+
+            from camel.messages import BaseMessage
+            from camel.types import OpenAIBackendRole
+
+            prefix_prompt = (
+                "The following is the context from a previous "
+                "session or workflow. This information might help you "
+                "understand the background, choose which tools to use, "
+                "and plan your next steps."
+            )
+
+            # TODO: change to system message once multi-system-message
+            # is supported
+            context_message = BaseMessage.make_assistant_message(
+                role_name="Assistant",
+                content=f"{prefix_prompt}\n\n{content}",
+            )
+
+            agent.update_memory(context_message, OpenAIBackendRole.USER)
+
+            char_count = len(content)
+            log_msg = (
+                f"Context appended to agent {agent.agent_id} "
+                f"({char_count} characters)"
+            )
+            logger.info(log_msg)
+
+            return log_msg
+
+        except Exception as e:
+            error_msg = f"Failed to load markdown context to memory: {e}"
+            logger.error(error_msg)
+            return error_msg

{camel_ai-0.2.76a2.dist-info → camel_ai-0.2.76a4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: camel-ai
-Version: 0.2.76a2
+Version: 0.2.76a4
 Summary: Communicative Agents for AI Society Study
 Project-URL: Homepage, https://www.camel-ai.org/
 Project-URL: Repository, https://github.com/camel-ai/camel

{camel_ai-0.2.76a2.dist-info → camel_ai-0.2.76a4.dist-info}/RECORD

@@ -1,4 +1,4 @@
-camel/__init__.py,sha256=kLbq9fj3ffO20PS_HeFWEBQFjG_e0tk2Grxj_hGCu58,901
+camel/__init__.py,sha256=VEPqtodQz0M5XRxGR-gpj2ZYNaBX-IYXD71NCkHVyn4,901
 camel/generators.py,sha256=JRqj9_m1PF4qT6UtybzTQ-KBT9MJQt18OAAYvQ_fr2o,13844
 camel/human.py,sha256=Xg8x1cS5KK4bQ1SDByiHZnzsRpvRP-KZViNvmu38xo4,5475
 camel/logger.py,sha256=WgEwael_eT6D-lVAKHpKIpwXSTjvLbny5jbV1Ab8lnA,5760
@@ -285,7 +285,7 @@ camel/societies/workforce/structured_output_handler.py,sha256=xr8szFN86hg3jQ825a
 camel/societies/workforce/task_channel.py,sha256=TXRwiqtmRPdelEmFCVN3jhd5XpgaSLwy9uHPtGecujA,11418
 camel/societies/workforce/utils.py,sha256=THgNHSeZsNVnjTzQTur3qCJhi72MrDS8X2gPET174cI,8434
 camel/societies/workforce/worker.py,sha256=MtUqYkTC9V-PIIRwSkKiB9w_YSu92iOpoha2rktEiQ0,6248
-camel/societies/workforce/workforce.py,sha256=h7Df4zoo5UuB-GpYTHotdcIwFDS6ESANLQLWzo56urE,142583
+camel/societies/workforce/workforce.py,sha256=m5m2d2aZdKlxek8xTDrPHqvur9rg5LGYcUecZrR0ym8,144628
 camel/societies/workforce/workforce_logger.py,sha256=0YT__ys48Bgn0IICKIZBmSWhON-eA1KShebjCdn5ppE,24525
 camel/storages/__init__.py,sha256=RwpEyvxpMbJzVDZJJygeBg4AzyYMkTjjkfB53hTuqGo,2141
 camel/storages/graph_storages/__init__.py,sha256=G29BNn651C0WTOpjCl4QnVM-4B9tcNh8DdmsCiONH8Y,948
@@ -354,7 +354,7 @@ camel/toolkits/klavis_toolkit.py,sha256=ZKerhgz5e-AV-iv0ftf07HgWikknIHjB3EOQswfu
 camel/toolkits/linkedin_toolkit.py,sha256=wn4eXwYYlVA7doTna7k7WYhUqTBF83W79S-UJs_IQr0,8065
 camel/toolkits/markitdown_toolkit.py,sha256=lwN6qQY8TLZkNWOqzeKZG3Fku-HMpGFrdRwhtPaJSlw,3844
 camel/toolkits/math_toolkit.py,sha256=SJbzT6akHRlmqo1QwCj1f7-6pEv0sNKJbcYvYAylHQw,5439
-camel/toolkits/mcp_toolkit.py,sha256=FbBKyz7f6y2hA-lBVsh3K-_gyz2rVDwe3uvaK8OsuRc,41442
+camel/toolkits/mcp_toolkit.py,sha256=Y6bwq_Wy1FeBi7uVjiR6SrN1GQeJJDqgDt6Hcl26gOw,37530
 camel/toolkits/memory_toolkit.py,sha256=TeKYd5UMwgjVpuS2orb-ocFL13eUNKujvrFOruDCpm8,4436
 camel/toolkits/meshy_toolkit.py,sha256=NbgdOBD3FYLtZf-AfonIv6-Q8-8DW129jsaP1PqI2rs,7126
 camel/toolkits/message_agent_toolkit.py,sha256=yWvAaxoxAvDEtD7NH7IkkHIyfWIYK47WZhn5E_RaxKo,22661
@@ -459,7 +459,7 @@ camel/utils/__init__.py,sha256=qQeMHZJ8Bbgpm4tBu-LWc_P3iFjXBlVfALdKTiD_s8I,3305
 camel/utils/async_func.py,sha256=KqoktGSWjZBuAMQ2CV0X6FRgHGlzCKLfeaWvp-f1Qz8,1568
 camel/utils/commons.py,sha256=xEhN__xkM1AT0dvvlAHZiPkGvfpwpj9BhCAWD51qJQ0,37163
 camel/utils/constants.py,sha256=cqnxmpUeOwrtsR-tRO4bbOc6ZP19TLj7avjm3FONMJs,1410
-camel/utils/context_utils.py,sha256=PIYCAAb9WGJo9GGC644rVuPDmRMlqEoeDAuQayr4a9E,13609
+camel/utils/context_utils.py,sha256=oJ6mdN2q4NKoHV-KBtlGhCEiKGrOtdwwmZpYbLL-2Ek,15466
 camel/utils/deduplication.py,sha256=UHikAtOW1TTDunf2t_wa2kFbmkrXWf7HfOKwLvwCxzo,8958
 camel/utils/filename.py,sha256=HYNc1wbSCgNR1CN21cwHxdAhpnsf5ySJ6jUDfeqOK20,2532
 camel/utils/langfuse.py,sha256=OowR6A790XG-b0UHiTYduYvS18PvSGFdmqki2Poogo0,8578
@@ -479,7 +479,7 @@ camel/verifiers/math_verifier.py,sha256=tA1D4S0sm8nsWISevxSN0hvSVtIUpqmJhzqfbuMo
 camel/verifiers/models.py,sha256=GdxYPr7UxNrR1577yW4kyroRcLGfd-H1GXgv8potDWU,2471
 camel/verifiers/physics_verifier.py,sha256=c1grrRddcrVN7szkxhv2QirwY9viIRSITWeWFF5HmLs,30187
 camel/verifiers/python_verifier.py,sha256=ogTz77wODfEcDN4tMVtiSkRQyoiZbHPY2fKybn59lHw,20558
-camel_ai-0.2.76a2.dist-info/METADATA,sha256=BC0_IwTj6y7726aIiEeEwakNxBmS0gvjZPkaj4POw0s,54897
-camel_ai-0.2.76a2.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-camel_ai-0.2.76a2.dist-info/licenses/LICENSE,sha256=id0nB2my5kG0xXeimIu5zZrbHLS6EQvxvkKkzIHaT2k,11343
-camel_ai-0.2.76a2.dist-info/RECORD,,
+camel_ai-0.2.76a4.dist-info/METADATA,sha256=RpkQkSkR6m7JLdYutzU7R0qy2319g-mAOcWTZgmfgK4,54897
+camel_ai-0.2.76a4.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+camel_ai-0.2.76a4.dist-info/licenses/LICENSE,sha256=id0nB2my5kG0xXeimIu5zZrbHLS6EQvxvkKkzIHaT2k,11343
+camel_ai-0.2.76a4.dist-info/RECORD,,