"last week at 04:00" | into datetime panics

[Schweber]

Describe the bug

"last week at 04:00" | into datetime                                                                                                    
Error:   × Main thread panicked.
  ├─▶ at /build/nushell-0.92.1-vendor.tar.gz/human-date-parser/src/lib.rs:436:13
  ╰─▶ Tried to convert Week to a weekday, which is not possible. This is a bug.
  help: set the `RUST_BACKTRACE=1` environment variable to display a backtrace.

How to reproduce

Run "last week at 04:00" | into datetime

Expected behavior

"last week" could be interpreted as "7 days ago" (without more specific input) or "sometime last week" (with more specific input like "Friday last week").

In any case, nushell should not panic but show an error message if it doesn't understand "last week" or if "last week" without more specific input is deemed too vague and rejected on purpose.

Configuration

key	value
version	0.92.1
branch
commit_hash
build_os	linux-x86_64
build_target	x86_64-unknown-linux-gnu
rust_version	rustc 1.77.1 (7cf61ebde 2024-03-27) (built from a source tarball)
cargo_version	cargo 1.77.1
build_time	1980-01-01 00:00:00 +00:00
build_rust_channel	release
allocator	mimalloc
features	default, sqlite, trash, which
installed_plugins

[fdncred]

Probably another upstream issue

[horasal]

actually I can't reproduce this panic on latest version

and the upstream library

I think it's may be solved at sometime (maybe technologicalMayhem/human-date-parser@e61ab92 )

Could you try the latest version of nushell?

[Schweber]

I cant' compile it right now because the latest version of nushell requires rustc 1.77.2 or newer and nixos-unstable is presently at 1.77.1.

But i trust your results and close this issue.

[Schweber]

I guess this #11710 can be closed as well if it checks out