honeycomb-api 0.1.0__py3-none-any.whl → 0.5.3__py3-none-any.whl

honeycomb/tools/descriptions.py

@@ -70,6 +70,9 @@ SLO_DESCRIPTIONS = {
         "Use this when defining reliability targets for services, such as 99.9% availability or p99 latency targets. "
         "Requires a dataset, SLO name, target (as percentage, per-million, or nines), time period in days, and an SLI definition. "
         "For the SLI, provide an alias and optionally an expression - if expression is provided, a new derived column is created inline; if only alias is provided, it uses an existing derived column. "
+        "SLI EXPRESSION SYNTAX: Must return boolean (1/0 coerced). Use $ prefix for columns (case-sensitive). "
+        "Example: LT($status_code, 500) or AND(LT($status_code, 500), LT($duration_ms, 1000)). "
+        "See honeycomb_create_derived_column for full expression syntax (conditionals, comparisons, regex, etc.). "
         "You can also add burn alerts inline to notify when error budget depletes too quickly. "
         "Supports both single-dataset and multi-dataset SLOs."
     ),
@@ -251,6 +254,55 @@ RECIPIENT_DESCRIPTIONS = {
 # Derived Columns
 # ==============================================================================
 
+# Shared syntax reference for derived column descriptions
+_DERIVED_COLUMN_SYNTAX_GUIDE = """
+EXPRESSION SYNTAX REFERENCE:
+- Column refs: $column_name or $"column with spaces" (CASE-SENSITIVE)
+- Strings: double quotes ("value")
+- Regex patterns: backticks (`pattern`)
+- DCs operate on ONE ROW at a time - they are NOT aggregation functions
+
+CONDITIONALS:
+- IF(cond, true_val, false_val) - e.g., IF(LT($status, 400), "ok", "error")
+- SWITCH($field, "case1", result1, "case2", result2, default) - string matching
+- COALESCE(a, b, c) - first non-null value
+
+COMPARISONS (return bool):
+- LT, LTE, GT, GTE - e.g., LT($duration_ms, 1000)
+- EQUALS($a, $b) - e.g., EQUALS($method, "GET")
+- IN($field, val1, val2, ...) - e.g., IN($status, 200, 201, 204)
+
+BOOLEAN:
+- EXISTS($field) - true if field is non-null
+- NOT(cond), AND(a, b, ...), OR(a, b, ...)
+
+MATH: MIN, MAX, SUM, SUB, MUL, DIV, MOD, LOG10
+
+STRING:
+- CONCAT($a, " ", $b) - join strings
+- STARTS_WITH($str, "prefix"), ENDS_WITH, CONTAINS - return bool
+- TO_LOWER($str), LENGTH($str)
+
+REGEX (use backticks for patterns):
+- REG_MATCH($str, `pattern`) - returns bool
+- REG_VALUE($str, `^/api/(.+)`) - extracts first capture group
+- REG_COUNT($str, `pattern`) - count matches
+
+TIME:
+- EVENT_TIMESTAMP() - event time as Unix timestamp
+- UNIX_TIMESTAMP($field) - parse field as timestamp
+- FORMAT_TIME("%Y-%m-%d", $timestamp_field) - strftime formatting
+
+DATA: BUCKET($val, size) - numeric bucketing
+
+TYPE CONVERSION: INT(), FLOAT(), BOOL(), STRING()
+
+COMMON PATTERNS:
+- SLI for SLOs (MUST return boolean; 1/0 coerced to true/false): LT($status_code, 500)
+- Root span detection: NOT(EXISTS($trace.parent_id))
+- Error classification: IF(GTE($status, 500), "server_error", IF(GTE($status, 400), "client_error", "success"))
+"""
+
 DERIVED_COLUMN_DESCRIPTIONS = {
     "honeycomb_list_derived_columns": (
         "Lists all derived columns (calculated fields) in a dataset. "
@@ -268,15 +320,16 @@ DERIVED_COLUMN_DESCRIPTIONS = {
         "Creates a new standalone derived column that calculates values from event fields using expressions. "
         "Use this for general-purpose computed metrics, data normalization, or calculations that will be used in multiple queries. "
         "NOTE: If you are creating a derived column specifically for an SLO, use honeycomb_create_slo instead with an inline SLI expression - it creates both in one operation. "
-        "Requires the dataset slug (use '__all__' for environment-wide), an alias (the column name), and an expression using Honeycomb's query language. "
-        "Common expression functions include IF() for conditionals, EQUALS/LT/GT for comparisons, and field references with $ prefix (e.g., $status_code). "
-        "Optional description parameter documents the column's purpose for team members."
+        "Requires the dataset slug (use '__all__' for environment-wide), an alias (the column name), and an expression. "
+        "Optional description parameter documents the column's purpose for team members. "
+        + _DERIVED_COLUMN_SYNTAX_GUIDE
     ),
     "honeycomb_update_derived_column": (
         "Updates an existing derived column's alias, expression, or description. "
         "Use this to fix calculation logic, rename computed fields, or improve documentation as your understanding evolves. "
         "Requires the dataset slug, derived column ID, and the complete updated configuration. "
-        "Note: Changing the expression only affects new queries - existing query results are not recalculated."
+        "Note: Changing the expression only affects new queries - existing query results are not recalculated. "
+        + _DERIVED_COLUMN_SYNTAX_GUIDE
     ),
     "honeycomb_delete_derived_column": (
         "Permanently deletes a derived column from a dataset. "
@@ -297,6 +350,7 @@ QUERY_DESCRIPTIONS = {
         "Requires the dataset slug and query specification including time_range and calculations. "
         "Optional annotation_name parameter creates the query with a display name for easier identification in the UI. "
         "The query specification supports multiple calculations (unlike triggers which allow only one), filters, breakdowns, orders, havings, and limits for comprehensive data analysis."
+        "Queries can include calculated_fields (derived columns) - see honeycomb_create_derived_column for expression syntax. "
     ),
     "honeycomb_get_query": (
         "Retrieves a saved query's configuration by its ID. "
@@ -331,12 +385,16 @@ BOARD_DESCRIPTIONS = {
         "Returns the complete board configuration including all panel definitions, layout mode (auto or manual), tags, and links to visualizations."
     ),
     "honeycomb_create_board": (
-        "Creates a new board (dashboard) with inline query panels, SLO panels, and text panels in a single operation. "
+        "Creates a new board (dashboard) with inline query panels, SLO panels, text panels, board views, and preset filters in a single operation. "
         "Use this to build comprehensive dashboards for service monitoring, create SRE views, or consolidate related visualizations. "
-        "Requires a name and supports inline_query_panels (array of query definitions that will be created automatically), text_panels (markdown content), and slo_panels (SLO IDs). "
+        "Requires a name and supports inline_query_panels (array of query definitions that will be created automatically), text_panels (markdown content), slo_panels (SLO IDs), views (filtered perspectives), and preset_filters (dynamic filters). "
         "Each inline query panel needs a name, dataset, time_range, and calculations - optionally include filters, breakdowns, orders, and limit. "
+        "Query panels can include calculated_fields (derived columns) - see honeycomb_create_derived_column for expression syntax. "
+        "For inline SLO panels with SLI expressions: must return boolean, use $ prefix for columns. Example: LT($status_code, 500). "
+        "Board views allow creating filtered perspectives (max 50 per board): each view has a name and filters array with column, operation (=, !=, >, >=, <, <=, contains, starts-with, ends-with, exists, in), and value. "
+        "Preset filters (max 5) allow dynamic filtering of board data: each has column and alias properties. "
         "Layout defaults to auto-layout (Honeycomb arranges panels) but supports manual layout with explicit positioning. "
-        "The tool orchestrates: creating all inline queries with annotations, assembling panel configurations, and creating the board with all panels in one API call."
+        "The tool orchestrates: creating all inline queries with annotations, inline SLOs, board views, preset filters, assembling panel configurations, and creating the board with all components in one API call."
     ),
     "honeycomb_update_board": (
         "Updates an existing board's name, description, or panel configuration. "
@@ -460,6 +518,113 @@ SERVICE_MAP_DESCRIPTIONS = {
     ),
 }
 
+# ==============================================================================
+# Auth
+# ==============================================================================
+
+AUTH_DESCRIPTIONS = {
+    "honeycomb_get_auth": (
+        "Returns metadata about the API key used to authenticate with Honeycomb. "
+        "Use this to verify authentication is working correctly, check key permissions and scopes, "
+        "or discover which team and environment the key belongs to. "
+        "Automatically detects whether to use the v1 endpoint (for regular API keys) or "
+        "v2 endpoint (for management keys) based on the configured credentials. "
+        "Set use_v2=true to explicitly request management key information, which includes "
+        "scopes and team details. Returns an error if use_v2=true but management credentials "
+        "are not configured."
+    ),
+}
+
+# ==============================================================================
+# API Keys (v2)
+# ==============================================================================
+
+API_KEY_DESCRIPTIONS = {
+    "honeycomb_list_api_keys": (
+        "Lists all API keys for your authenticated team with optional filtering by key type. "
+        "Use this to audit team API keys, discover existing keys before creating new ones, or when migrating to management key authentication. "
+        "Requires management key authentication. The team is automatically detected from your credentials (no team parameter needed). "
+        "Optional key_type parameter filters by 'ingest' or 'configuration' keys. "
+        "Returns a list of API key objects with their IDs, names, types, environment associations, and disabled status. "
+        "Note: The key secrets are not included in list responses for security."
+    ),
+    "honeycomb_get_api_key": (
+        "Retrieves detailed information about a specific API key by ID. "
+        "Use this to inspect an API key's configuration including name, type, environment, and disabled status before updating or deleting it. "
+        "Requires only the key ID parameter - the team is automatically detected from your management key credentials. "
+        "Returns the complete API key configuration but does NOT include the secret (only shown on creation). "
+        "Use this to verify key settings or check which environment a key is associated with."
+    ),
+    "honeycomb_create_api_key": (
+        "Creates a new API key for your authenticated team in a specific environment. "
+        "Use this when provisioning new services, creating separate keys for different applications, or rotating credentials. "
+        "Requires key name, key_type ('ingest' for data sending or 'configuration' for API access), and environment_id. The team is automatically detected from your management key. "
+        "CRITICAL FOR CONFIGURATION KEYS: Must include permissions object with boolean properties. Available permissions: 'create_datasets', 'send_events', 'manage_markers', 'manage_triggers', 'manage_boards', 'run_queries', 'manage_columns', 'manage_slos', 'manage_recipients', 'manage_privateBoards'. "
+        "Example for full access: {'create_datasets': true, 'send_events': true, 'manage_markers': true, 'manage_triggers': true, 'manage_boards': true, 'run_queries': true, 'manage_columns': true, 'manage_slos': true, 'manage_recipients': true, 'manage_privateBoards': true}. "
+        "Without permissions object, configuration keys will have NO permissions and cannot perform any actions. "
+        "Ingest keys have fixed permissions (send events, optionally create datasets implicitly) and don't use permissions object. "
+        "CRITICAL: The secret is only returned once during creation - save it immediately. "
+        "Returns the created API key object including the secret field (only time it's available)."
+    ),
+    "honeycomb_update_api_key": (
+        "Updates an existing API key's name or disabled status. "
+        "Use this to rename keys for clarity, disable compromised keys, or re-enable previously disabled keys. "
+        "Requires key ID and at least one of: name (new display name) or disabled (true/false). The team is automatically detected from your credentials. "
+        "Note: You cannot change the key type or environment after creation. To rotate the secret, delete and create a new key. "
+        "The secret value is never returned in update responses."
+    ),
+    "honeycomb_delete_api_key": (
+        "Permanently deletes an API key from Honeycomb. "
+        "Use this when decommissioning services, rotating credentials after a security incident, or cleaning up unused keys. "
+        "Requires only the key ID parameter - the team is automatically detected from your management key. "
+        "Warning: This action cannot be undone. Any services using this key will immediately lose API access. "
+        "The secret cannot be recovered - you must create a new key if deleted accidentally."
+    ),
+}
+
+# ==============================================================================
+# Environments (v2)
+# ==============================================================================
+
+ENVIRONMENT_DESCRIPTIONS = {
+    "honeycomb_list_environments": (
+        "Lists all environments for your authenticated team. "
+        "Use this to discover existing environments, understand team organization, or before creating API keys scoped to specific environments. "
+        "Requires management key authentication. The team is automatically detected from your credentials (no team parameter needed). "
+        "Returns a list of environment objects with their IDs, names, slugs, colors, descriptions, and delete protection status. "
+        "Environments help organize your telemetry data by separating production, staging, development, etc."
+    ),
+    "honeycomb_get_environment": (
+        "Retrieves detailed information about a specific environment by ID, optionally including its datasets. "
+        "Use this to inspect an environment's configuration or get a complete view of an environment including all its datasets. "
+        "Requires only the environment ID parameter - the team is automatically detected from your management key. "
+        "Set with_datasets=true to also return the list of datasets in this environment (useful for understanding environment contents). "
+        "Returns the environment configuration plus optionally a datasets array with all datasets in this environment."
+    ),
+    "honeycomb_create_environment": (
+        "Creates a new environment for organizing telemetry data within your authenticated team. "
+        "Use this when setting up new deployment stages (production, staging, dev), isolating customer data, or creating test environments. "
+        "Requires only the environment name - the team is automatically detected from your management key. "
+        "Optional parameters include description for documentation and color (blue, green, gold, red, purple, or light variants) for visual distinction. "
+        "The environment slug is auto-generated from the name and used for API operations and dataset scoping. "
+        "New environments are delete-protected by default to prevent accidental deletion."
+    ),
+    "honeycomb_update_environment": (
+        "Updates an existing environment's description, color, or delete protection status. "
+        "Use this to add documentation, change visual appearance, or enable/disable delete protection for production environments. "
+        "Requires environment ID and at least one of: description, color, or delete_protected (boolean). The team is automatically detected from your credentials. "
+        "Note: The name and slug cannot be changed after creation. Setting delete_protected=true prevents accidental deletion. "
+        "To delete a protected environment, you must first update it with delete_protected=false."
+    ),
+    "honeycomb_delete_environment": (
+        "Permanently deletes an environment from Honeycomb. "
+        "Use this when decommissioning deployment stages, cleaning up test environments, or reorganizing team structure. "
+        "Requires only the environment ID parameter - the team is automatically detected from your management key. "
+        "Warning: This action cannot be undone. The environment must not be delete-protected. "
+        "All API keys scoped to this environment will become invalid. Datasets are NOT deleted automatically."
+    ),
+}
+
 # Combined mapping of all descriptions
 ALL_DESCRIPTIONS = {
     **TRIGGER_DESCRIPTIONS,
@@ -475,6 +640,9 @@ ALL_DESCRIPTIONS = {
     **MARKER_SETTING_DESCRIPTIONS,
     **EVENT_DESCRIPTIONS,
     **SERVICE_MAP_DESCRIPTIONS,
+    **AUTH_DESCRIPTIONS,
+    **API_KEY_DESCRIPTIONS,
+    **ENVIRONMENT_DESCRIPTIONS,
 }
 
 

honeycomb/tools/executor.py

@@ -13,6 +13,7 @@ from honeycomb.models import (
     BurnAlertRecipient,
     ColumnCreate,
     DatasetCreate,
+    DatasetUpdate,
     DerivedColumnCreate,
     MarkerCreate,
     MarkerSettingCreate,
@@ -27,6 +28,28 @@ if TYPE_CHECKING:
     from honeycomb import HoneycombClient
 
 
+# ==============================================================================
+# Metadata Field Handling
+# ==============================================================================
+
+
+def strip_metadata_fields(tool_input: dict[str, Any]) -> dict[str, Any]:
+    """Strip Claude reasoning metadata fields before API execution.
+
+    These fields are for downstream applications to observe Claude's reasoning
+    and are NOT sent to the Honeycomb API.
+
+    Args:
+        tool_input: The tool input dict (will be modified in place)
+
+    Returns:
+        The same dict with metadata fields removed
+    """
+    tool_input.pop("confidence", None)
+    tool_input.pop("notes", None)
+    return tool_input
+
+
 # ==============================================================================
 # Main Executor
 # ==============================================================================
@@ -63,8 +86,36 @@ async def execute_tool(
         ...     )
         ...     print(result)  # JSON string
     """
+    # Strip metadata fields before processing
+    # These are for Claude's reasoning and not sent to Honeycomb API
+    strip_metadata_fields(tool_input)
+
     # Route to appropriate handler
-    if tool_name == "honeycomb_list_triggers":
+    if tool_name == "honeycomb_get_auth":
+        return await _execute_get_auth(client, tool_input)
+    # API Keys (v2)
+    elif tool_name == "honeycomb_list_api_keys":
+        return await _execute_list_api_keys(client, tool_input)
+    elif tool_name == "honeycomb_get_api_key":
+        return await _execute_get_api_key(client, tool_input)
+    elif tool_name == "honeycomb_create_api_key":
+        return await _execute_create_api_key(client, tool_input)
+    elif tool_name == "honeycomb_update_api_key":
+        return await _execute_update_api_key(client, tool_input)
+    elif tool_name == "honeycomb_delete_api_key":
+        return await _execute_delete_api_key(client, tool_input)
+    # Environments (v2)
+    elif tool_name == "honeycomb_list_environments":
+        return await _execute_list_environments(client, tool_input)
+    elif tool_name == "honeycomb_get_environment":
+        return await _execute_get_environment(client, tool_input)
+    elif tool_name == "honeycomb_create_environment":
+        return await _execute_create_environment(client, tool_input)
+    elif tool_name == "honeycomb_update_environment":
+        return await _execute_update_environment(client, tool_input)
+    elif tool_name == "honeycomb_delete_environment":
+        return await _execute_delete_environment(client, tool_input)
+    elif tool_name == "honeycomb_list_triggers":
         return await _execute_list_triggers(client, tool_input)
     elif tool_name == "honeycomb_get_trigger":
         return await _execute_get_trigger(client, tool_input)
@@ -195,6 +246,155 @@ async def execute_tool(
         )
 
 
+# ==============================================================================
+# Auth
+# ==============================================================================
+
+
+async def _execute_get_auth(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_get_auth tool."""
+    use_v2 = tool_input.get("use_v2")
+    result = await client.auth.get_async(use_v2=use_v2)
+    return json.dumps(result.model_dump(), default=str)
+
+
+# ==============================================================================
+# API Keys (v2)
+# ==============================================================================
+
+
+async def _execute_list_api_keys(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_list_api_keys tool."""
+    key_type = tool_input.get("key_type")
+    keys = await client.api_keys.list_async(key_type=key_type)
+    return json.dumps([k.model_dump() for k in keys], default=str)
+
+
+async def _execute_get_api_key(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_get_api_key tool."""
+    key = await client.api_keys.get_async(key_id=tool_input["key_id"])
+    return json.dumps(key.model_dump(), default=str)
+
+
+async def _execute_create_api_key(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_create_api_key tool."""
+    from honeycomb.models.api_keys import ApiKeyCreate, ApiKeyType
+
+    api_key = ApiKeyCreate(
+        name=tool_input["name"],
+        key_type=ApiKeyType(tool_input["key_type"]),
+        environment_id=tool_input["environment_id"],
+        permissions=tool_input.get("permissions"),
+    )
+    created = await client.api_keys.create_async(api_key=api_key)
+    return json.dumps(created.model_dump(), default=str)
+
+
+async def _execute_update_api_key(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_update_api_key tool."""
+    from honeycomb.models.api_keys import ApiKeyUpdate
+
+    update = ApiKeyUpdate(
+        name=tool_input.get("name"),
+        disabled=tool_input.get("disabled"),
+    )
+    updated = await client.api_keys.update_async(
+        key_id=tool_input["key_id"],
+        api_key=update,
+    )
+    return json.dumps(updated.model_dump(), default=str)
+
+
+async def _execute_delete_api_key(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_delete_api_key tool."""
+    await client.api_keys.delete_async(key_id=tool_input["key_id"])
+    return json.dumps({"status": "deleted", "key_id": tool_input["key_id"]})
+
+
+# ==============================================================================
+# Environments (v2)
+# ==============================================================================
+
+
+async def _execute_list_environments(client: "HoneycombClient", _tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_list_environments tool."""
+    envs = await client.environments.list_async()
+    return json.dumps([e.model_dump() for e in envs], default=str)
+
+
+async def _execute_get_environment(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_get_environment tool."""
+    env = await client.environments.get_async(env_id=tool_input["env_id"])
+    result = env.model_dump()
+
+    # Optionally include datasets (requires environment-scoped API key)
+    if tool_input.get("with_datasets"):
+        import os
+
+        api_key = os.environ.get("HONEYCOMB_API_KEY")
+        if not api_key:
+            result["datasets_error"] = (
+                "Cannot list datasets: No HONEYCOMB_API_KEY found. "
+                "Datasets require an environment-scoped API key."
+            )
+        else:
+            # Create temporary client to verify environment match
+            from honeycomb import HoneycombClient
+
+            async with HoneycombClient(api_key=api_key) as api_key_client:
+                # Verify the API key is for this environment (force v1 for environment_slug)
+                from honeycomb.models.auth import AuthInfo
+
+                auth_info = await api_key_client.auth.get_async(use_v2=False)
+                assert isinstance(auth_info, AuthInfo)  # use_v2=False always returns AuthInfo
+                if auth_info.environment_slug != result["slug"]:
+                    result["datasets_error"] = (
+                        f"Cannot list datasets: HONEYCOMB_API_KEY is for environment "
+                        f"'{auth_info.environment_slug}' but requested '{result['slug']}'"
+                    )
+                else:
+                    # Environment matches - list datasets
+                    datasets = await api_key_client.datasets.list_async()
+                    result["datasets"] = [d.model_dump() for d in datasets]
+
+    return json.dumps(result, default=str)
+
+
+async def _execute_create_environment(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_create_environment tool."""
+    from honeycomb.models.environments import EnvironmentColor, EnvironmentCreate
+
+    environment = EnvironmentCreate(
+        name=tool_input["name"],
+        description=tool_input.get("description"),
+        color=EnvironmentColor(tool_input["color"]) if tool_input.get("color") else None,
+    )
+    created = await client.environments.create_async(environment=environment)
+    return json.dumps(created.model_dump(), default=str)
+
+
+async def _execute_update_environment(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_update_environment tool."""
+    from honeycomb.models.environments import EnvironmentColor, EnvironmentUpdate
+
+    environment = EnvironmentUpdate(
+        description=tool_input.get("description"),
+        color=EnvironmentColor(tool_input["color"]) if tool_input.get("color") else None,
+        delete_protected=tool_input.get("delete_protected"),
+    )
+    updated = await client.environments.update_async(
+        env_id=tool_input["env_id"],
+        environment=environment,
+    )
+    return json.dumps(updated.model_dump(), default=str)
+
+
+async def _execute_delete_environment(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
+    """Execute honeycomb_delete_environment tool."""
+    await client.environments.delete_async(env_id=tool_input["env_id"])
+    return json.dumps({"status": "deleted", "env_id": tool_input["env_id"]})
+
+
 # ==============================================================================
 # Triggers
 # ==============================================================================
@@ -281,32 +481,19 @@ async def _execute_get_slo(client: "HoneycombClient", tool_input: dict[str, Any]
 async def _execute_create_slo(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
     """Execute honeycomb_create_slo.
 
-    If SLI expression is provided or burn_alerts are included, uses SLOBuilder
-    and create_from_bundle_async for orchestration. Otherwise uses simple create_async.
+    Always uses SLOBuilder with Pydantic validation and automatic target_percentage conversion.
+    Handles SLI expressions, burn alerts, and dataset list processing transparently.
     """
-    dataset = tool_input.pop("dataset")
-
-    # Check if we need bundle orchestration
-    sli = tool_input.get("sli", {})
-    burn_alerts = tool_input.get("burn_alerts", [])
-    needs_bundle = ("expression" in sli) or burn_alerts
-
-    if needs_bundle:
-        # Use builder for orchestration
-        builder = _build_slo({"dataset": dataset, **tool_input})
-        bundle = builder.build()
+    # Validate and build (tool_input now contains datasets: list[str])
+    builder = _build_slo(tool_input)
+    bundle = builder.build()
 
-        # Create via bundle (handles derived columns + burn alerts)
-        created_slos = await client.slos.create_from_bundle_async(bundle)
+    # Create via bundle (handles derived columns, burn alerts, and all conversions)
+    created_slos = await client.slos.create_from_bundle_async(bundle)
 
-        # Return the main SLO (first one created)
-        main_slo = list(created_slos.values())[0]
-        return json.dumps(main_slo.model_dump(), default=str)
-    else:
-        # Simple SLO creation (existing derived column, no burn alerts)
-        slo = SLOCreate(**tool_input)
-        created = await client.slos.create_async(dataset=dataset, slo=slo)
-        return json.dumps(created.model_dump(), default=str)
+    # Return the main SLO (first one created)
+    main_slo = list(created_slos.values())[0]
+    return json.dumps(main_slo.model_dump(), default=str)
 
 
 async def _execute_update_slo(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
@@ -431,7 +618,7 @@ async def _execute_create_dataset(client: "HoneycombClient", tool_input: dict[st
 async def _execute_update_dataset(client: "HoneycombClient", tool_input: dict[str, Any]) -> str:
     """Execute honeycomb_update_dataset."""
     slug = tool_input.pop("slug")
-    dataset = DatasetCreate(**tool_input)
+    dataset = DatasetUpdate(**tool_input)
     updated = await client.datasets.update_async(slug=slug, dataset=dataset)
     return json.dumps(updated.model_dump(), default=str)