nfischer/n_shell

A NodeJS REPL with ShellJS loaded by default

n_shell's Issues

Add an option to show the shelljs version in the prompt

Let's add an option to modify the prompt to look like:

shelljs v0.6.0 $

Also, we can have an option to do a custom prompt (like Bash's PS1 variable). One of the options could be version number. So it would mean that the default prompt looks something like --prompt='shelljs $ ' and the version-number prompt like --prompt='shelljs %v $ '

Add unit tests and linting

It'd be nice to reuse some of the tests written for n_, which uses the mocha framework and jshint.

If anyone is interested in taking up a PR for this (which would be awesome!), please ping this thread to give me a heads up. This is pretty low on my priority list, but if you'd like to see unit tests to improve project stability, please let me know.

--noinspect still imports shelljs-plugin-inspect

I noticed that n_shell --noinspect appears to still load shelljs-plugin-inspect even though the switch is documented as not doing that. This should probably be fixed.

Update shelljs-plugin deps

The shelljs plugins are outdated, and they cause a conflict with newer shelljs versions.

Also, shelljs probably shouldn't be a * dependency

[ feature ] Tab completion

This is a feature idea I have right now, for how to make this more usable. I'd like to see tab completion, not only for variables, but also for strings representing paths on the file system (similar to how Bash works by default).

$ ls()
foo.txt
bar.txt
baz.txt
$ cat('b➡️
$ cat('bar.txt'➡️
$ cat('baz.txt');
this is the context of baz.txt

The ➡️ denotes when I would press tab. Tab completion would happen in-place, not on a new line, but it was easier to show my idea this way.

It'd be even better if tab completion worked on other paths, such as cat('~/.bas➡️ ➡️ cat('~/.bashrc');.

[DEP0124] DeprecationWarning: REPLServer.rli is deprecated

Looks like this was deprecated in v12 and removed in v16. https://nodejs.org/api/deprecations.html#DEP0124. I think this usage is coming from tmpvar/repl.history#15.

Node v12 REPL's function preview interacts poorly with shelljs

It looks like node v12's REPL will run javascript code as you type it into the console. This has some annoying interactions with shelljs commands though because:

1. these have side effects (ex. shell.clear() clears the console)
2. n_shell has a feature to execute commands without typing parentheses (ex. ls<CR> is the same as ls()<CR>)

These combine to make for a really unpleasant user experience. Ex. typing touch (without pressing enter) will cause touch() to be eagerly executed by the REPL preview, which outputs an error (touch: no files given).

Maybe we should just disable the feature referenced in (2) and require parentheses. For some reason the node REPL won't show the preview for ls(), only for ls.