diff --git a/crates/nu-command/tests/commands/def.rs b/crates/nu-command/tests/commands/def.rs index 65d5d58db3f9..e149c159fca4 100644 --- a/crates/nu-command/tests/commands/def.rs +++ b/crates/nu-command/tests/commands/def.rs @@ -164,3 +164,19 @@ fn extern_with_block() { assert_eq!(actual.out, "--bar,baz,--,-q,-u,-x"); } + +#[test] +fn def_default_value_shouldnt_restrict_explicit_type() { + let actual = nu!("def foo [x: any = null] { $x }; foo 1"); + assert_eq!(actual.out, "1"); + let actual2 = nu!("def foo [--x: any = null] { $x }; foo --x 1"); + assert_eq!(actual2.out, "1"); +} + +#[test] +fn def_default_value_should_restrict_implicit_type() { + let actual = nu!("def foo [x = 3] { $x }; foo 3.0"); + assert!(actual.err.contains("expected int")); + let actual2 = nu!("def foo2 [--x = 3] { $x }; foo2 --x 3.0"); + assert!(actual2.err.contains("expected int")); +} diff --git a/crates/nu-parser/src/parser.rs b/crates/nu-parser/src/parser.rs index b56c60022996..507c61269441 100644 --- a/crates/nu-parser/src/parser.rs +++ b/crates/nu-parser/src/parser.rs @@ -3374,6 +3374,7 @@ pub fn parse_signature_helper(working_set: &mut StateWorkingSet, span: Span) -> let mut args: Vec = vec![]; let mut parse_mode = ParseMode::ArgMode; + let mut arg_explicit_type = false; for token in &output { match token { @@ -3428,6 +3429,7 @@ pub fn parse_signature_helper(working_set: &mut StateWorkingSet, span: Span) -> working_set.error(ParseError::Expected("default value", span)); } } + arg_explicit_type = false; } else { match parse_mode { ParseMode::ArgMode | ParseMode::AfterCommaArgMode => { @@ -3710,6 +3712,7 @@ pub fn parse_signature_helper(working_set: &mut StateWorkingSet, span: Span) -> } } } + arg_explicit_type = true; } parse_mode = ParseMode::ArgMode; } @@ -3732,10 +3735,12 @@ pub fn parse_signature_helper(working_set: &mut StateWorkingSet, span: Span) -> let var_type = &working_set.get_variable(var_id).ty; match var_type { Type::Any => { - working_set.set_variable_type( - var_id, - expression.ty.clone(), - ); + if !arg_explicit_type { + working_set.set_variable_type( + var_id, + expression.ty.clone(), + ); + } } _ => { if !type_compatible(var_type, &expression.ty) { @@ -3763,7 +3768,9 @@ pub fn parse_signature_helper(working_set: &mut StateWorkingSet, span: Span) -> None }; - *shape = expression.ty.to_shape(); + if !arg_explicit_type { + *shape = expression.ty.to_shape(); + } *required = false; } Arg::RestPositional(..) => { @@ -3800,9 +3807,13 @@ pub fn parse_signature_helper(working_set: &mut StateWorkingSet, span: Span) -> if var_type != &Type::Bool { match var_type { Type::Any => { - *arg = Some(expression_ty.to_shape()); - working_set - .set_variable_type(var_id, expression_ty); + if !arg_explicit_type { + *arg = Some(expression_ty.to_shape()); + working_set.set_variable_type( + var_id, + expression_ty, + ); + } } t => { if t != &expression_ty {