noetl-server 3.15.4

//! Playbook YAML parser (Canonical Format).
//!
//! Parses YAML playbook definitions into Playbook structures.
//! Validates canonical format:
//! - step.when for transition enable guards
//! - next[].when for conditional routing
//! - loop.spec.mode for iteration configuration
//! - tool.eval for per-task flow control
//! - No case/when/then blocks (deprecated)

use crate::error::{AppError, AppResult};
use crate::playbook::types::{Playbook, ToolDefinition};

/// Parse a YAML string into a Playbook.
pub fn parse_playbook(yaml_content: &str) -> AppResult<Playbook> {
    // First check for deprecated case blocks before full parse
    validate_no_case_blocks(yaml_content)?;

    let mut playbook: Playbook =
        serde_yaml::from_str(yaml_content).map_err(|e| AppError::Parse(e.to_string()))?;

    // Resolve `tool.kind: workbook` references before validation —
    // any workflow step whose tool's kind is `workbook` is rewritten
    // in-place to use the inline action defined in the playbook's
    // top-level `workbook:` list.  The rest of the pipeline (parser
    // validation, orchestrator command-build, worker dispatch) then
    // sees a regular tool spec and never has to know about workbook
    // references.  See noetl/ai-meta#59 for the e2e finding.
    resolve_workbook_references(&mut playbook)?;

    // Validate the playbook
    validate_playbook(&playbook)?;

    Ok(playbook)
}

/// Substitute every `tool.kind: workbook` reference with the
/// resolved inline action from `playbook.workbook`.
///
/// A reference has shape:
///
/// ```yaml
/// tool:
///   kind: workbook
///   name: compute_flag    # action label in playbook.workbook[]
///   input: { ... }        # overrides workbook task's `input:` defaults
/// ```
///
/// After substitution the step's tool is the workbook task's tool
/// spec with `input` keys merged (step input wins over workbook
/// input defaults).
fn resolve_workbook_references(playbook: &mut crate::playbook::types::Playbook) -> AppResult<()> {
    use crate::playbook::types::{ToolDefinition, ToolKind};

    // Snapshot the workbook list once so we can index without holding
    // a mutable borrow on `playbook` while iterating workflow steps.
    let workbook_index: std::collections::HashMap<String, _> = playbook
        .workbook
        .as_ref()
        .map(|tasks| {
            tasks
                .iter()
                .map(|t| (t.name.clone(), t.tool.clone()))
                .collect()
        })
        .unwrap_or_default();

    for step in playbook.workflow.iter_mut() {
        let ToolDefinition::Single(spec) = &mut step.tool else {
            continue;
        };
        if spec.kind != ToolKind::Workbook {
            continue;
        }

        // Pull the action label out of `extra["name"]` (the field
        // doesn't have a dedicated slot on ToolSpec — the v10 YAML
        // writes `name:` on the tool block alongside `kind: workbook`,
        // which lands in the catch-all `extra:` map).
        let action_name = spec
            .extra
            .get("name")
            .and_then(|v| v.as_str())
            .map(|s| s.to_string())
            .ok_or_else(|| {
                AppError::Validation(format!(
                    "Step '{}': tool.kind=workbook requires a 'name' field referencing a workbook action",
                    step.step
                ))
            })?;

        let resolved_tool = workbook_index.get(&action_name).cloned().ok_or_else(|| {
            AppError::Validation(format!(
                "Step '{}': workbook action '{}' not found in playbook.workbook[] \
                 (available: [{}])",
                step.step,
                action_name,
                workbook_index
                    .keys()
                    .map(|s| s.as_str())
                    .collect::<Vec<_>>()
                    .join(", ")
            ))
        })?;

        // Merge the step's input over the workbook task's defaults.
        // The step's input — already aliased into spec.args by the
        // serde alias on ToolSpec.args — is the override layer.
        // Both shapes are JSON objects in practice (Jinja templates,
        // literals, etc.); if either side isn't an object, the step
        // input replaces wholesale.
        let merged_args = match (&resolved_tool.args, &spec.args) {
            (Some(serde_json::Value::Object(base)), Some(serde_json::Value::Object(over))) => {
                let mut merged = base.clone();
                for (k, v) in over {
                    merged.insert(k.clone(), v.clone());
                }
                Some(serde_json::Value::Object(merged))
            }
            (_, Some(step_input)) => Some(step_input.clone()),
            (base, None) => base.clone(),
        };

        // Substitute the resolved tool with merged args.  Keep the
        // step's tool kind from the workbook (e.g. python), drop the
        // `name:` field from extra so downstream serialization
        // doesn't carry the now-resolved reference.
        let mut substituted = resolved_tool;
        substituted.args = merged_args;
        step.tool = ToolDefinition::Single(Box::new(substituted));
    }

    Ok(())
}

/// Check for deprecated case blocks in YAML content.
fn validate_no_case_blocks(yaml_content: &str) -> AppResult<()> {
    let value: serde_yaml::Value =
        serde_yaml::from_str(yaml_content).map_err(|e| AppError::Parse(e.to_string()))?;

    if let Some(workflow) = value.get("workflow").and_then(|w| w.as_sequence()) {
        for (idx, step) in workflow.iter().enumerate() {
            if step.get("case").is_some() {
                let fallback = format!("workflow[{}]", idx);
                let step_name = step
                    .get("step")
                    .and_then(|s| s.as_str())
                    .unwrap_or(&fallback);

                return Err(AppError::Validation(format!(
                    "Step '{}': 'case' blocks are not allowed in canonical format. \
                     Use 'step.when' for enable guards and 'next[].when' for conditional routing. \
                     For pipelines, use 'tool: [- label: {{kind: ...}}]' directly on the step.",
                    step_name
                )));
            }
        }
    }

    Ok(())
}

/// Validate a parsed playbook.
pub fn validate_playbook(playbook: &Playbook) -> AppResult<()> {
    // Check API version
    if playbook.api_version != "noetl.io/v2" {
        return Err(AppError::Validation(format!(
            "Unsupported API version: {}. Expected noetl.io/v2",
            playbook.api_version
        )));
    }

    // Check kind
    if playbook.kind != "Playbook" {
        return Err(AppError::Validation(format!(
            "Invalid kind: {}. Expected Playbook",
            playbook.kind
        )));
    }

    // Check for start step
    if !playbook.has_start_step() {
        return Err(AppError::Validation(
            "Workflow must have a step named 'start'".to_string(),
        ));
    }

    // Check for duplicate step names
    let mut seen_steps = std::collections::HashSet::new();
    for step in &playbook.workflow {
        if !seen_steps.insert(&step.step) {
            return Err(AppError::Validation(format!(
                "Duplicate step name: {}",
                step.step
            )));
        }
    }

    // Validate step transitions and canonical format
    let step_names: std::collections::HashSet<&str> =
        playbook.workflow.iter().map(|s| s.step.as_str()).collect();

    for step in &playbook.workflow {
        // Validate tool definition
        validate_tool_definition(&step.tool, &step.step)?;

        // Validate loop spec if present
        if let Some(ref loop_config) = step.r#loop {
            validate_loop_config(loop_config, &step.step)?;
        }

        // Validate step.when is a valid expression (basic check)
        if let Some(ref when) = step.when {
            if !is_valid_jinja_expression(when) {
                return Err(AppError::Validation(format!(
                    "Step '{}': invalid 'when' expression: {}",
                    step.step, when
                )));
            }
        }

        // Check next step references with when conditions
        if let Some(ref next) = step.next {
            validate_next_refs(next, &step_names, &step.step)?;
        }
    }

    Ok(())
}

/// Validate tool definition (single or pipeline).
fn validate_tool_definition(tool: &ToolDefinition, step_name: &str) -> AppResult<()> {
    match tool {
        ToolDefinition::Single(spec) => {
            // Validate eval conditions if present
            if let Some(ref eval) = spec.eval {
                validate_eval_conditions(eval, step_name)?;
            }
        }
        ToolDefinition::Pipeline(tasks) => {
            if tasks.is_empty() {
                return Err(AppError::Validation(format!(
                    "Step '{}': tool pipeline must have at least one task",
                    step_name
                )));
            }

            for (idx, task) in tasks.iter().enumerate() {
                // Normalize both YAML pipeline shapes to a (label, spec)
                // tuple list for validation.  Flat form uses the
                // `name` field on the spec as the label; nested form
                // uses the single map key.
                let entries: Vec<(String, &crate::playbook::types::ToolSpec)> = match task {
                    crate::playbook::types::PipelineItem::Flat(spec) => {
                        let label = spec
                            .extra
                            .get("name")
                            .and_then(|v| v.as_str())
                            .unwrap_or("")
                            .to_string();
                        vec![(label, spec)]
                    }
                    crate::playbook::types::PipelineItem::Nested(map) => {
                        if map.len() != 1 {
                            return Err(AppError::Validation(format!(
                                "Step '{}': pipeline task[{}] must have exactly one labeled entry (got {})",
                                step_name,
                                idx,
                                map.len()
                            )));
                        }
                        map.iter().map(|(k, v)| (k.clone(), v)).collect()
                    }
                };

                // Validate eval conditions in each task
                for (label, spec) in entries {
                    if let Some(ref eval) = spec.eval {
                        validate_eval_conditions_for_task(eval, step_name, &label)?;
                    }
                }
            }
        }
    }
    Ok(())
}

/// Validate eval conditions list.
fn validate_eval_conditions(
    eval: &[crate::playbook::types::EvalEntry],
    step_name: &str,
) -> AppResult<()> {
    for (idx, entry) in eval.iter().enumerate() {
        match entry {
            crate::playbook::types::EvalEntry::Condition(cond) => {
                // Validate expr if present
                if let Some(ref expr) = cond.expr {
                    if !is_valid_jinja_expression(expr) {
                        return Err(AppError::Validation(format!(
                            "Step '{}': eval[{}] has invalid expression: {}",
                            step_name, idx, expr
                        )));
                    }
                }

                // Validate action
                let valid_actions = ["continue", "retry", "break", "jump", "fail"];
                if !valid_actions.contains(&cond.action.as_str()) {
                    return Err(AppError::Validation(format!(
                        "Step '{}': eval[{}] has invalid action '{}'. Valid: {:?}",
                        step_name, idx, cond.action, valid_actions
                    )));
                }
            }
            crate::playbook::types::EvalEntry::Else { r#else } => {
                // Validate action in else clause
                let valid_actions = ["continue", "retry", "break", "jump", "fail"];
                if !valid_actions.contains(&r#else.action.as_str()) {
                    return Err(AppError::Validation(format!(
                        "Step '{}': eval[{}] else has invalid action '{}'. Valid: {:?}",
                        step_name, idx, r#else.action, valid_actions
                    )));
                }
            }
        }
    }
    Ok(())
}

/// Validate eval conditions for a pipeline task.
fn validate_eval_conditions_for_task(
    eval: &[crate::playbook::types::EvalEntry],
    step_name: &str,
    task_label: &str,
) -> AppResult<()> {
    for (idx, entry) in eval.iter().enumerate() {
        match entry {
            crate::playbook::types::EvalEntry::Condition(cond) => {
                // Validate expr if present
                if let Some(ref expr) = cond.expr {
                    if !is_valid_jinja_expression(expr) {
                        return Err(AppError::Validation(format!(
                            "Step '{}': tool[].{}.eval[{}] has invalid expression: {}",
                            step_name, task_label, idx, expr
                        )));
                    }
                }

                // Validate action
                let valid_actions = ["continue", "retry", "break", "jump", "fail"];
                if !valid_actions.contains(&cond.action.as_str()) {
                    return Err(AppError::Validation(format!(
                        "Step '{}': tool[].{}.eval[{}] has invalid action '{}'. Valid: {:?}",
                        step_name, task_label, idx, cond.action, valid_actions
                    )));
                }
            }
            crate::playbook::types::EvalEntry::Else { r#else } => {
                // Validate action in else clause
                let valid_actions = ["continue", "retry", "break", "jump", "fail"];
                if !valid_actions.contains(&r#else.action.as_str()) {
                    return Err(AppError::Validation(format!(
                        "Step '{}': tool[].{}.eval[{}] else has invalid action '{}'. Valid: {:?}",
                        step_name, task_label, idx, r#else.action, valid_actions
                    )));
                }
            }
        }
    }
    Ok(())
}

/// Validate loop configuration.
fn validate_loop_config(
    loop_config: &crate::playbook::types::Loop,
    step_name: &str,
) -> AppResult<()> {
    // Validate in expression (only when present — cursor loops have no `in:`)
    if let Some(in_expr) = loop_config.in_expr.as_deref() {
        if !is_valid_jinja_expression(in_expr) {
            return Err(AppError::Validation(format!(
                "Step '{}': loop.in has invalid expression: {}",
                step_name, in_expr
            )));
        }
    }

    // Validate iterator name
    if loop_config.iterator.is_empty() {
        return Err(AppError::Validation(format!(
            "Step '{}': loop.iterator must not be empty",
            step_name
        )));
    }

    // mode is validated by serde (LoopMode enum: sequential / parallel / cursor).
    // Shape requirements differ by mode:
    //   - cursor: requires `loop.cursor.claim`; `in:` is not used.
    //   - sequential / parallel: require `in:`; `cursor:` is not used.
    let is_cursor = loop_config.mode() == crate::playbook::types::LoopMode::Cursor;
    if is_cursor {
        if loop_config.cursor.is_none() {
            return Err(AppError::Validation(format!(
                "Step '{}': loop.spec.mode=cursor requires a loop.cursor block with a claim query",
                step_name
            )));
        }
    } else if loop_config.in_expr.as_deref().unwrap_or("").trim().is_empty() {
        return Err(AppError::Validation(format!(
            "Step '{}': loop requires `in:` (the collection expression) for mode={:?}",
            step_name,
            loop_config.mode()
        )));
    }

    Ok(())
}

/// Basic validation that a string looks like a Jinja expression.
fn is_valid_jinja_expression(expr: &str) -> bool {
    // Basic check: should contain {{ }} or be a simple expression
    // More sophisticated validation would require a Jinja parser
    !expr.is_empty() && (expr.contains("{{") || !expr.contains('{'))
}

/// Validate next step references (canonical format).
fn validate_next_refs(
    next: &crate::playbook::types::NextSpec,
    valid_steps: &std::collections::HashSet<&str>,
    current_step: &str,
) -> AppResult<()> {
    match next {
        crate::playbook::types::NextSpec::Single(name) => {
            if !valid_steps.contains(name.as_str()) {
                return Err(AppError::Validation(format!(
                    "Step '{}' references unknown step '{}' in next",
                    current_step, name
                )));
            }
        }
        crate::playbook::types::NextSpec::List(names) => {
            for name in names {
                if !valid_steps.contains(name.as_str()) {
                    return Err(AppError::Validation(format!(
                        "Step '{}' references unknown step '{}' in next",
                        current_step, name
                    )));
                }
            }
        }
        crate::playbook::types::NextSpec::Router(router) => {
            // Canonical v10 format: validate arcs
            for arc in &router.arcs {
                if !valid_steps.contains(arc.step.as_str()) {
                    return Err(AppError::Validation(format!(
                        "Step '{}' references unknown step '{}' in next.arcs",
                        current_step, arc.step
                    )));
                }

                // Validate when condition if present
                if let Some(ref when) = arc.when {
                    if !is_valid_jinja_expression(when) {
                        return Err(AppError::Validation(format!(
                            "Step '{}': next.arcs[].when has invalid expression: {}",
                            current_step, when
                        )));
                    }
                }
            }
        }
        crate::playbook::types::NextSpec::Targets(targets) => {
            // Legacy canonical format
            for target in targets {
                if !valid_steps.contains(target.step.as_str()) {
                    return Err(AppError::Validation(format!(
                        "Step '{}' references unknown step '{}' in next",
                        current_step, target.step
                    )));
                }

                // Validate when condition if present
                if let Some(ref when) = target.when {
                    if !is_valid_jinja_expression(when) {
                        return Err(AppError::Validation(format!(
                            "Step '{}': next[].when has invalid expression: {}",
                            current_step, when
                        )));
                    }
                }
            }
        }
    }
    Ok(())
}

/// Extract kind from YAML content without full parsing.
/// Useful for determining resource type before full parse.
pub fn extract_kind(yaml_content: &str) -> AppResult<String> {
    // Quick parse to get just the kind field
    let value: serde_yaml::Value =
        serde_yaml::from_str(yaml_content).map_err(|e| AppError::Parse(e.to_string()))?;

    value
        .get("kind")
        .and_then(|k| k.as_str())
        .map(|s| s.to_string())
        .ok_or_else(|| AppError::Validation("Missing 'kind' field".to_string()))
}

/// Extract metadata from YAML content.
pub fn extract_metadata(yaml_content: &str) -> AppResult<(String, Option<String>, Option<String>)> {
    let value: serde_yaml::Value =
        serde_yaml::from_str(yaml_content).map_err(|e| AppError::Parse(e.to_string()))?;

    let metadata = value
        .get("metadata")
        .ok_or_else(|| AppError::Validation("Missing 'metadata' field".to_string()))?;

    let name = metadata
        .get("name")
        .and_then(|n| n.as_str())
        .map(|s| s.to_string())
        .ok_or_else(|| AppError::Validation("Missing 'metadata.name' field".to_string()))?;

    let path = metadata
        .get("path")
        .and_then(|p| p.as_str())
        .map(|s| s.to_string());

    let description = metadata
        .get("description")
        .and_then(|d| d.as_str())
        .map(|s| s.to_string());

    Ok((name, path, description))
}

#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn test_workbook_reference_resolves_to_inline_action() {
        // noetl/ai-meta#59 — `tool.kind: workbook` with a `name:`
        // referring to a playbook.workbook[] entry must rewrite
        // the step's tool to the inline action's tool spec, merging
        // the step's input over the workbook's defaults.  Without
        // this resolution the worker errors with "Tool not found:
        // workbook" and the orchestrator stalls.
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: workbook_ref_test
workload:
  temperature_c: 30
workbook:
  - name: compute_flag
    tool:
      kind: python
      input:
        temperature_c: null
      code: "result = {'is_hot': float(temperature_c) >= 25.0}"
workflow:
  - step: start
    tool:
      kind: workbook
      name: compute_flag
      input:
        temperature_c: "{{ temperature_c }}"
"#;
        let playbook = parse_playbook(yaml).expect("workbook ref must parse");
        let start = playbook.get_step("start").expect("start step present");
        match &start.tool {
            crate::playbook::types::ToolDefinition::Single(spec) => {
                assert_eq!(spec.kind, crate::playbook::types::ToolKind::Python,
                    "workbook ref should be rewritten to the inline action's tool kind");
                assert_eq!(spec.code.as_deref(), Some("result = {'is_hot': float(temperature_c) >= 25.0}"),
                    "workbook ref should carry the inline action's code");
                let args = spec.args.as_ref().expect("merged input present");
                let temp = args.get("temperature_c").and_then(|v| v.as_str());
                assert_eq!(temp, Some("{{ temperature_c }}"),
                    "step's input overrides workbook's default");
            }
            other => panic!("expected ToolDefinition::Single, got {:?}", other),
        }
    }

    #[test]
    fn test_workbook_reference_unknown_action_errors() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: workbook_unknown
workbook:
  - name: compute_flag
    tool:
      kind: python
      code: "result = {}"
workflow:
  - step: start
    tool:
      kind: workbook
      name: missing_action
"#;
        let err = parse_playbook(yaml).expect_err("missing workbook action should be rejected");
        let msg = format!("{:?}", err);
        assert!(msg.contains("missing_action"), "error must name the missing action: {}", msg);
        assert!(msg.contains("compute_flag"), "error must list available actions: {}", msg);
    }

    #[test]
    fn test_workbook_reference_missing_name_errors() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: workbook_no_name
workbook:
  - name: x
    tool:
      kind: python
      code: "result = {}"
workflow:
  - step: start
    tool:
      kind: workbook
"#;
        let err = parse_playbook(yaml).expect_err("workbook ref without name must error");
        assert!(format!("{:?}", err).contains("requires a 'name'"));
    }

    #[test]
    fn test_parse_valid_playbook() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    tool:
      kind: python
      code: return {}
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_ok());
    }

    #[test]
    fn test_parse_invalid_api_version() {
        let yaml = r#"
apiVersion: noetl.io/v1
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    tool:
      kind: python
      code: return {}
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_err());
        assert!(result
            .unwrap_err()
            .to_string()
            .contains("Unsupported API version"));
    }

    #[test]
    fn test_parse_missing_start_step() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: process
    tool:
      kind: python
      code: return {}
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_err());
        assert!(result.unwrap_err().to_string().contains("start"));
    }

    #[test]
    fn test_parse_duplicate_step_names() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    tool:
      kind: python
      code: return {}
  - step: start
    tool:
      kind: python
      code: return {}
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_err());
        assert!(result.unwrap_err().to_string().contains("Duplicate"));
    }

    #[test]
    fn test_parse_invalid_next_reference() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    tool:
      kind: python
      code: return {}
    next:
      - step: nonexistent
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_err());
        assert!(result.unwrap_err().to_string().contains("unknown step"));
    }

    #[test]
    fn test_reject_case_blocks() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    tool:
      kind: python
      code: return {}
    case:
      - when: "{{ result.value > 5 }}"
        then:
          - next:
              step: high
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_err());
        let err_msg = result.unwrap_err().to_string();
        assert!(err_msg.contains("case"));
        assert!(err_msg.contains("not allowed"));
    }

    #[test]
    fn test_parse_canonical_next_when() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    tool:
      kind: python
      code: |
        result = {"value": 10}
    next:
      - step: high
        when: "{{ start.value > 5 }}"
      - step: low
        when: "{{ start.value <= 5 }}"
  - step: high
    tool:
      kind: noop
  - step: low
    tool:
      kind: noop
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_ok());
    }

    #[test]
    fn test_parse_step_when_guard() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    when: "{{ workload.enabled }}"
    tool:
      kind: python
      code: return {}
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_ok());
    }

    #[test]
    fn test_parse_pipeline_format() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    tool:
      - fetch:
          kind: http
          url: "https://api.example.com"
          method: GET
      - transform:
          kind: python
          code: |
            result = {"processed": True}
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_ok());
    }

    #[test]
    fn test_parse_loop_spec_mode() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
workflow:
  - step: start
    loop:
      in: "{{ workload.items }}"
      iterator: item
      spec:
        mode: parallel
        max_in_flight: 5
    tool:
      kind: python
      code: return {}
"#;

        let result = parse_playbook(yaml);
        assert!(result.is_ok());
    }

    #[test]
    fn test_extract_kind() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: test
"#;

        let kind = extract_kind(yaml).unwrap();
        assert_eq!(kind, "Playbook");
    }

    #[test]
    fn test_extract_metadata() {
        let yaml = r#"
apiVersion: noetl.io/v2
kind: Playbook
metadata:
  name: my_playbook
  path: test/path
  description: A test playbook
"#;

        let (name, path, desc) = extract_metadata(yaml).unwrap();
        assert_eq!(name, "my_playbook");
        assert_eq!(path, Some("test/path".to_string()));
        assert_eq!(desc, Some("A test playbook".to_string()));
    }
}