add support for nushell

[KiaraGrouwstra]

in addition to support for command-not-found support for bash/zsh/fish, it would be nice to see support for nushell as well.

c.f. fish thread: #126

[ameertaweel]

I wrote this Nushell command_not_found hook to get nix-index support on Nushell:

Code

$env.config.hooks.command_not_found = { |cmd_name|
  let install = { |pkgs|
    $pkgs | each {|pkg| $"  nix shell nixpkgs#($pkg)" }
  }
  let run_once = { |pkgs|
    $pkgs | each {|pkg| $"  nix shell nixpkgs#($pkg) --command '($cmd_name) ...'" }
  }
  let single_pkg = { |pkg|
    let lines = [
      $"The program '($cmd_name)' is currently not installed."
      ""
      "You can install it by typing:"
      (do $install [$pkg] | get 0)
      ""
      "Or run it once with:"
      (do $run_once [$pkg] | get 0)
    ]
    $lines | str join "\n"
  }
  let multiple_pkgs = { |pkgs|
    let lines = [
      $"The program '($cmd_name)' is currently not installed. It is provided by several packages."
      ""
      "You can install it by typing one of the following:"
      (do $install $pkgs | str join "\n")
      ""
      "Or run it once with:"
      (do $run_once $pkgs | str join "\n")
    ]
    $lines | str join "\n"
  }
  let pkgs = (nix-locate --minimal --no-group --type x --type s --top-level --whole-name --at-root $"/bin/($cmd_name)" | lines)
  let len = ($pkgs | length)
  let ret = match $len {
    0 => null,
    1 => (do $single_pkg ($pkgs | get 0)),
    _ => (do $multiple_pkgs $pkgs),
  }
  return $ret
}

Demo

Binary provided by several packags:

Binary provided by a single package:

Binary is not provided by any package (default Nushell behavior):

[KiaraGrouwstra]

@ameertaweel it works great :), would you mind making a PR for this?

[ameertaweel]

Thanks for your feedback. I created #271 with the hook and instructions on how to use it in the README.

[tknawara]

Hey @ameertaweel Thanks for working on this, I was trying it in nushell, but I'm getting some errors around how arguments are passed to nix-locate

[ameertaweel]

Hey @tknawara can you give a minimal config that reproduces the issue? I will be happy to help but I can't do much if I can't reproduce the issue...

[tknawara]

@ameertaweel Thanks for replying so quickly, I've tried to include the relevant parts from my config, these are the flake changes

    nix-index = {
      url = "github:nix-community/nix-index";
      inputs.nixpkgs.follows = "nixpkgs";
    };

and in nushell config, this is what I added

let nix-index-pkg = inputs.nix-index.packages.${pkgs.system}.default;
in {
  programs.nushell = {
    enable = true;
    configFile.source = ./config.nu;
    extraConfig = # nu
      ''
        $env.config.hooks.command_not_found = source ${nix-index-pkg}/etc/profile.d/command-not-found.nu
      '';
  }

and I can confirm that the generated nushell config includes the correct sourcing and everything

 $env.config.hooks.command_not_found = source /nix/store/1qkaf7kp12dh9iw8p84amg9c1fnfi7vr-nix-index-0.1.9/etc/profile.d/command-not-found.nu

I've starship integration with nushell (if that is relevant), everything is done via home-manager

[ameertaweel]

@tknawara
I will try to work with this. Even though it's still not guaranteed I will be able to reproduce the issue.

Ideally, you would create a standalone minimal config that I can directly test in a VM.

I will be a bit busy for a while, but I will do my best to take a look in a couple of days. I just wanted to let you know that I saw your comment and that I will keep it in mind.

[ameertaweel]

I was able to reproduce the issue. Created #275 to track it separate from here.

[nixos-discourse]

This issue has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/nushell-as-default-shell/68609/4