use crate::mir::builder::MirBuilder;
use crate::mir::definitions::call_unified::Callee;
use crate::mir::ValueId;

/// Finalize call operands (receiver/args) using LocalSSA; thin wrapper to centralize usage.
pub fn finalize_call_operands(
    builder: &mut MirBuilder,
    callee: &mut Callee,
    args: &mut Vec<ValueId>,
) {
    // Step 1: Receiver materialization (pin slot + LocalSSA) in a dedicated box
    crate::mir::builder::receiver::finalize_method_receiver(builder, callee);

    // Step 2: LocalSSA materialization for args only
    crate::mir::builder::ssa::local::finalize_args(builder, args);

    // Step 3: Disabled - BlockScheduleBox insert-after-phis doesn't work correctly
    // The Copy instructions are being inserted but then lost when blocks are finalized.
    // Instead, rely solely on LocalSSA which uses emit_instruction (the normal path).
    //
    // TODO: Fix BlockScheduleBox or remove it entirely if LocalSSA is sufficient.

    /* DISABLED - causes ValueId(22) undefined error
    if let Callee::Method { box_name, method, receiver: Some(r_local), certainty } = callee.clone() {
        if let Ok(r_after) = crate::mir::builder::schedule::block::BlockScheduleBox::ensure_after_phis_copy(builder, r_local) {
            if let Ok(r_tail) = crate::mir::builder::schedule::block::BlockScheduleBox::emit_before_call_copy(builder, r_after) {
                *callee = Callee::Method { box_name, method, receiver: Some(r_tail), certainty };
            } else {
                *callee = Callee::Method { box_name, method, receiver: Some(r_after), certainty };
            }
        }
    }
    */
}

/// Verify block schedule invariants after emitting a call (dev-only WARNs inside).
pub fn verify_after_call(builder: &mut MirBuilder) {
    crate::mir::builder::schedule::block::BlockScheduleBox::verify_order(builder);
}
