Browse Source

node: Revamp and optimize 'node' module

Changes:
- nodenv and nvm now honors (and prioritizes) `$XDG_CONFIG_HOME` over
  `$HOME` to lookup local nodenv/nvm installation.
- Make `nvm` loading lazy (via `--no-use` argument).
- Remove redundant NODENV_ROOT or NVM_DIR, respective script already
  set them up.
- Adhere to more idiomatic Zsh operation and minimize external command
  usage (like `sed`).
master
Indrajit Raychaudhuri 4 years ago committed by Indrajit Raychaudhuri
parent
commit
b68b7d8f42
  1. 7
      modules/node/README.md
  2. 2
      modules/node/functions/node-doc
  3. 37
      modules/node/init.zsh

7
modules/node/README.md

@ -9,7 +9,7 @@ enables [_npm_][2] completion.
home directory. home directory.
This will be loaded automatically if nvm is installed in `$NVM_DIR`, This will be loaded automatically if nvm is installed in `$NVM_DIR`,
_`~/.nvm`_, or nvm is installed with Homebrew. _`$XDG_CONFIG_HOME/nvm`_, _`~/.nvm`_, or is installed with homebrew.
## nodenv ## nodenv
@ -18,7 +18,7 @@ Node versions. It is simple and predictable, Just Works, and is rock solid in
production. nodenv is forked from the popular [_rbenv_][7]. production. nodenv is forked from the popular [_rbenv_][7].
This will be loaded automatically if nodenv is installed in `$NODENV_ROOT`, This will be loaded automatically if nodenv is installed in `$NODENV_ROOT`,
_`~/.nodenv`_, or `nodenv` is on the path. _`$XDG_CONFIG_HOME/nodenv`_, _`~/.nodenv`_, or `nodenv` is on the path.
## Functions ## Functions
@ -46,11 +46,12 @@ _The authors of this module should be contacted via the [issue tracker][4]._
- [Sorin Ionescu](https://github.com/sorin-ionescu) - [Sorin Ionescu](https://github.com/sorin-ionescu)
- [Zeh Rizzatti](https://github.com/zehrizzatti) - [Zeh Rizzatti](https://github.com/zehrizzatti)
- [Indrajit Raychaudhuri](https://github.com/indrajitr)
[1]: http://nodejs.org [1]: http://nodejs.org
[2]: http://npmjs.org [2]: http://npmjs.org
[3]: http://nodejs.org/api [3]: http://nodejs.org/api
[4]: https://github.com/sorin-ionescu/prezto/issues [4]: https://github.com/sorin-ionescu/prezto/issues
[5]: https://github.com/creationix/nvm [5]: https://github.com/nvm-sh/nvm
[6]: https://github.com/nodenv/nodenv [6]: https://github.com/nodenv/nodenv
[7]: https://github.com/sstephenson/rbenv [7]: https://github.com/sstephenson/rbenv

2
modules/node/functions/node-doc

@ -13,6 +13,6 @@ if [[ -z "$BROWSER" ]]; then
fi fi
# TODO: Make the sections easier to use. # TODO: Make the sections easier to use.
"$BROWSER" "http://nodejs.org/docs/$(node --version | sed 's/-.*//')/api/all.html#${1}" "$BROWSER" "https://nodejs.org/docs/${$(node --version 2> /dev/null)/%-*}/api/all.html#${1}"
# } # }

37
modules/node/init.zsh

@ -1,25 +1,32 @@
# #
# Loads the Node Version Manager and enables npm completion. # Configures Node local installation, loads version managers, and defines
# variables and aliases.
# #
# Authors: # Authors:
# Sorin Ionescu <sorin.ionescu@gmail.com> # Sorin Ionescu <sorin.ionescu@gmail.com>
# Zeh Rizzatti <zehrizzatti@gmail.com> # Zeh Rizzatti <zehrizzatti@gmail.com>
# #
# Possible lookup locations.
local_nvm_paths=({$NVM_DIR,{$XDG_CONFIG_HOME/,$HOME/.}nvm}/nvm.sh(N))
local_nodenv_paths=({$NODENV_ROOT,{$XDG_CONFIG_HOME/,$HOME/.}nodenv}/bin/nodenv(N))
# Load manually installed NVM into the shell session. # Load manually installed NVM into the shell session.
if [[ -s "${NVM_DIR:=$HOME/.nvm}/nvm.sh" ]]; then if [[ -s ${local_nvm::=$local_nvm_paths[1]} ]]; then
source "${NVM_DIR}/nvm.sh" source "$local_nvm --no-use"
unset local_nvm{,_paths}
# Load package manager installed NVM into the shell session. # Load package manager installed NVM into the shell session.
elif (( $+commands[brew] )) && \ elif (( $+commands[brew] )) \
[[ -d "${nvm_prefix::="$(brew --prefix nvm 2> /dev/null)"}" ]]; then && [[ -d "${nvm_prefix::="$(brew --prefix nvm 2> /dev/null)"}" ]]; then
source "${nvm_prefix}/nvm.sh" source "$nvm_prefix/nvm.sh --no-use"
unset nvm_prefix unset nvm_prefix
# Load manually installed nodenv into the shell session. # Load manually installed nodenv into the shell session.
elif [[ -s "${NODENV_ROOT:=$HOME/.nodenv}/bin/nodenv" ]]; then elif [[ -s ${local_nodenv::=$local_nodenv_paths[1]} ]]; then
path=("${NODENV_ROOT}/bin" $path) path=("$local_nodenv:h" $path)
eval "$(nodenv init - --no-rehash zsh)" eval "$(nodenv init - --no-rehash zsh)"
unset local_nodenv{,_paths}
# Load package manager installed nodenv into the shell session. # Load package manager installed nodenv into the shell session.
elif (( $+commands[nodenv] )); then elif (( $+commands[nodenv] )); then
@ -31,22 +38,22 @@ elif (( ! $+commands[node] )); then
fi fi
# Load NPM and known helper completions. # Load NPM and known helper completions.
typeset -A compl_commands=( typeset -A _compl_commands=(
npm 'npm completion' npm 'npm completion'
grunt 'grunt --completion=zsh' grunt 'grunt --completion=zsh'
gulp 'gulp --completion=zsh' gulp 'gulp --completion=zsh'
) )
for compl_command in "${(k)compl_commands[@]}"; do for _compl_command in "${(k)_compl_commands[@]}"; do
if (( $+commands[$compl_command] )); then if (( $+commands[$_compl_command] )); then
cache_file="${XDG_CACHE_HOME:-$HOME/.cache}/prezto/$compl_command-cache.zsh" cache_file="${XDG_CACHE_HOME:-$HOME/.cache}/prezto/${_compl_command}-cache.zsh"
# Completion commands are slow; cache their output if old or missing. # Completion commands are slow; cache their output if old or missing.
if [[ "$commands[$compl_command]" -nt "$cache_file" \ if [[ "$commands[$_compl_command]" -nt "$cache_file" \
|| "${ZDOTDIR:-$HOME}/.zpreztorc" -nt "$cache_file" \ || "${ZDOTDIR:-$HOME}/.zpreztorc" -nt "$cache_file" \
|| ! -s "$cache_file" ]]; then || ! -s "$cache_file" ]]; then
mkdir -p "$cache_file:h" mkdir -p "$cache_file:h"
command ${=compl_commands[$compl_command]} >! "$cache_file" 2> /dev/null command ${=_compl_commands[$_compl_command]} >! "$cache_file" 2> /dev/null
fi fi
source "$cache_file" source "$cache_file"
@ -55,4 +62,4 @@ for compl_command in "${(k)compl_commands[@]}"; do
fi fi
done done
unset compl_command{s,} unset _compl_command{s,}

Loading…
Cancel
Save