nushell breaking changes

[fdncred]

Heads-up @ajeetdsouza, With nushell/nushell#10566 we will deprecate the def-env syntax in nushell. Our next release 0.86.0 on Oct-17 will allow both old and new syntax but the next release, 4 weeks later, will produce a warning. The new syntax is def --env.

[marcelarie]

Thanks this was really helpful!

[lpnh]

After updating to the new nushell 0.87.0 release, I encountered the following error when opening a new terminal:

Error:   × Deprecated command
    ╭─[/home/user/.zoxide.nu:25:1]
 25 │ # Jump to a directory using only keywords.
 26 │ def-env __zoxide_z [...rest:string] {
    · ───┬───
    ·    ╰── `def-env` is deprecated and will be removed in 0.88.
 27 │   let arg0 = ($rest | append '~').0
    ╰────
  help: Use `def --env` instead


Error:   × Deprecated command
    ╭─[/home/user/.zoxide.nu:36:1]
 36 │ # Jump to a directory using interactive search.
 37 │ def-env __zoxide_zi  [...rest:string] {
    · ───┬───
    ·    ╰── `def-env` is deprecated and will be removed in 0.88.
 38 │   cd $'(zoxide query --interactive -- $rest | str trim -r -c "\n")'
    ╰────
  help: Use `def --env` instead


Error:   × Config record contains invalid values or unknown settings

Error:   × Error while applying config changes
     ╭─[/home/user/.config/nushell/config.nu:152:1]
 152 │
 153 │ ╭─▶     cd: {
 154 │ │           abbreviations: false # allows `cd s/o/f` to expand to `cd some/other/folder`
 155 │ ├─▶     }
     · ╰──── $env.config.cd is an unknown config setting
 156 │
     ╰────
  help: This value will not appear in your $env.config record.

I noticed that there are already two PRs that address this issue: #632 and #642.

[fdncred]

@lpnh Just FYI - This last part has nothing to do with zoxide.

Error:   × Config record contains invalid values or unknown settings

Error:   × Error while applying config changes
     ╭─[/home/user/.config/nushell/config.nu:152:1]
 152 │
 153 │ ╭─▶     cd: {
 154 │ │           abbreviations: false # allows `cd s/o/f` to expand to `cd some/other/folder`
 155 │ ├─▶     }
     · ╰──── $env.config.cd is an unknown config setting
 156 │
     ╰────
  help: This value will not appear in your $env.config record.

$env.config.cd.abbreviations was removed from nushell in favor of using our path completions that work similarly now. You just need to comment that out of your config.nu (or remove that part).

[acidghost]

As far as I can see the only two places where zoxide needs to be updated are

zoxide/templates/nushell.txt

Lines 42 to 61 in 073500c

	def-env __zoxide_z [...rest:string] {
	let arg0 = ($rest | append '~').0
	let path = if (($rest | length) <= 1) and ($arg0 == '-' or ($arg0 | path expand | path type) == dir) {
	$arg0
	} else {
	(zoxide query --exclude $env.PWD -- $rest | str trim -r -c "\n")
	}
	cd $path
	{%- if echo %}
	echo $env.PWD
	{%- endif %}
	}
	# Jump to a directory using interactive search.
	def-env __zoxide_zi [...rest:string] {
	cd $'(zoxide query --interactive -- $rest | str trim -r -c "\n")'
	{%- if echo %}
	echo $env.PWD
	{%- endif %}
	}

I think it's just a matter of replacing def-env with def --env.

[Mickychen00]

Hello. I have notice there is a fix commit "f537a4e6d2f8c2eb84c63f79e290a6d1b16eeb71" in zoxide. How can I install this dev version of zoxide?

[marcelarie]

Not sure but it might be brew install --HEAD zoxide in macOS

$: brew install --HEAD zoxide
Error: No head is defined for zoxide

[Slushee-a]

When will this be pushed to crates.io? Currently having to install from git.

[Aarkon]

Nushell has just published their update, removing def-env altogether. As far as I understand, this breaks zoxide entirely in nu. Providing a fixed release on crates.rs anytime soon has become somewhat urgent now, I’d say.

[dtscherer]

For now I'm just using this in my env.nu

zoxide init nushell | str replace "def-env" "def --env" --all | save -f ~/.config/nushell/modules/.zoxide.nu

[dc740]

Hey, the list of incompatibilities grew longer.

❯ z 3d_designs
Error:   × Automatically spreading lists is deprecated
    ╭─[/home/user1/.zoxide.nu:30:1]
 30 │   } else {
 31 │     (zoxide query --exclude $env.PWD -- $rest | str trim -r -c "\n")
    ·                                         ──┬──
    ·                                           ╰── Spreading lists automatically when calling external commands is deprecated and will be removed in 0.91.
 32 │   }
    ╰────
  help: Use the spread operator (put a '...' before the argument)


[WARN] - (starship::utils): Executing command "/usr/bin/git" timed out.
[WARN] - (starship::utils): You can set command_timeout in your config to a higher value to allow longer-running commands to keep executing.

Which can also be added as a workaround because there are two places where -- $rest is used

zoxide init nushell | str replace "-- $rest" "-- ...$rest" --all | str replace "def-env" "def --env" --all | save -f ~/.zoxide.nu

---

nevermind. I found the PR
#663