Lou Acresti

Spoofing Warning

6

Granted that nowhere in your README do you use the words "authentication", it's still probably a good idea to note in the README that this isn't meant to be a...

Default Keyboard Shortcuts

3

It might be nice to use all the default keyboard shortcuts in the official Idris book (for consistency with the Atom plugin): Shortcut | Command | Description -- | --...

`@enable 'serve *'` does nothing -- documentation disparity?

2

Without digging too deeply into recent changelists, it looks like using `@enable 'serve jquery'` or similar does nothing. The "PicoChat" example still works, but only because it runs the `@client`...

Babel Transform

5

Since you've gone to the trouble of writing JSCodeShift transforms, have you considered using a babel transform instead? They're actually quite similar. Using babel would make it easier to transparently...

Use workspace TS version in VSCode

1

By default, VSCode uses its built-in TypeScript version, which is currently 5.x, however langchainjs is using 4.x for its build, so it's possible for contributors to see errors in their...

Update V8 version

3

The version of V8 JV8 uses is some random spot in the commit history. I should aim for a major version release, instead. Related: #12

Proper V8 debugger support.

6

[V8 has built-in support](https://code.google.com/p/v8/wiki/AddDebuggerSupport) for communicating with a remote debugger. The [Chrome Dev Tools](https://code.google.com/p/chromedevtools/) project even has an [Eclipse plugin](https://code.google.com/p/chromedevtools/wiki/EclipseDebugger); it looks shamefully bare-bones, but it will still beat the...

Function Passing

1

For JV8 to truly integrate with JS, it needs the ability to accept functions as parameters to mapped methods in addition to the primitives that are available currently.

Sub-issue of #4

Windows Desktop Build

1

Sub-issue of #4