Option to disable tabtab

[tough-griff]

Between using nodenv to manage multiple versions of node and npx to execute and install packages rather than global installations, tabtab has trouble managing the installation locations of packages and clutters my shell configuration by adding completions for scripts I may only run once or twice.

I would like the option to disable tabtab globally in some manner, for example a TABTAB_DISABLE environment variable or similar.

[mklabs]

Hi I understand your issue and will take a good look at it. This is indeed a genuine issue.

Thanks for your feedback. Meanwhile I would recommand uninstalling any tatab completion package you might have active on your system.

[tough-griff]

Excellent, thanks!

I'd be happy to work on a PR, but a cursory glance at the code didn't give me a very good idea on where to implement such a check. If ya can point me in the right direction I can give it a shot!

[Gerst20051]

I don't understand why or how dependencies of packages I'm installing for a project can manage to modify my shell profiles. I'm requiring serverless in my projects and when I run npm i for the first time in one of the projects it updates either .<shell>rc such as .bashrc or .zshrc file (depending on the default shell). I would love if it didn't do that.

Here is the output from running npm i:

# [tabtab] Adding source line to load ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/serverless.zsh
# in ~/.zshrc

# [tabtab] Adding source line to load ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/sls.zsh
# in ~/.zshrc

# [tabtab] Adding source line to load ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/slss.zsh
in ~/.zshrc

Here is the content that was added to .zshrc:

# tabtab source for serverless package
# uninstall by removing these lines or running `tabtab uninstall serverless`
[[ -f ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/serverless.zsh ]] && . ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/serverless.zsh
# tabtab source for sls package
# uninstall by removing these lines or running `tabtab uninstall sls`
[[ -f ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/sls.zsh ]] && . ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/sls.zsh
# tabtab source for slss package
# uninstall by removing these lines or running `tabtab uninstall slss`
[[ -f ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/slss.zsh ]] && . ~/Code/GitlabRepos/my-serverless-project/node_modules/tabtab/.completions/slss.zsh

[Gerst20051]

This is similar to the following issues:

serverless/serverless#3792
serverless/serverless#4069
serverless/serverless#4672
serverless/serverless#5124

Here is a link to the postinstall script that calls tabtab:

https://github.com/serverless/serverless/blob/master/scripts/postinstall.js

Here is the commit that initially added calls to tabtab:

serverless/serverless@671049a

Here is a commit on a forked repo that removes the code that calls tabtab:

axelson/serverless@0e4ddf3

Here is a MR that removes the postinstall script from serverless:

serverless/serverless#5706

Here is a previous MR that fixed an issue calling tabtab:

https://github.com/serverless/serverless/pull/3870/files

Here is a workaround to "short circuit" installing tabtab:

ahawkins/dotfiles@bee5e3c

The slss autocompletion was added in this merge request:

serverless/serverless#4712

[jeffbski]

Another issue is that this old version of tabtab ends up using dependencies that include an old version of spawn-sync that requires a binary compile. Doing npm rebuild with docker lambda causes a postinstall error from spawn-sync.

Node 8+ no longer needs this polyfill, so newer versions of tabtab do not need spawn-sync, so migrating that that would help.

However it would be nice to make this totally optional. I can't even get it to work as is now.

So make autocomplete a plugin and let people choose to install it only if they need/want it.

(I just realized this issue is in the tabtab repo, so I will post the suggestion in serverless repo)
Added comment to serverless/serverless#4069

[bsamuel-ui]

Workaround: leave the # tabtab source for X package lines in your .bashrc

[hraban]

I wish there were some kind of global export NPM_DONT_TOUCH_MY_COMPUTER=noconsent var I could set, supported by all these "smart" tools. Between this, and tools like husky, running npm install on a project is a gamble in "what are well meaning overzealous developers going to break in my setup, this time?"