feat: eliminate "semver" dependency #313

This dependency is unnecessary.

Author: justinmk

README.md

@@ -24,7 +24,16 @@ The `neovim` package exposes these functions:
 
 ### Quickstart: connect to Nvim
 
-Following is a complete example. Paste it into `demo.mjs` file and run it with `node demo.mjs`:
+Following is a complete, working example.
+
+1. Install the `neovim` package _locally_ (i.e. without `-g`. Node will fail with `ERR_MODULE_NOT_FOUND` if a script tries to import a _globally_ installed package).
+   ```bash
+   npm install neovim
+   ```
+2. Paste the script below into a `demo.mjs` file and run it!
+   ```
+   node demo.mjs
+   ```
 
 ```js
 import * as child_process from 'node:child_process'

package-lock.json

@@ -2,7 +2,6 @@
 const { Host } = require('../lib/host');
 const { logger } = require('../lib/utils/logger');
 const { spawnSync } = require('child_process');
-const semver = require('semver');
 
 // node <current script> <rest of args>
 const [, , ...args] = process.argv;
@@ -15,9 +14,13 @@ if (args[0] === '--version') {
   process.exit(0);
 }
 
+// "21.6.1" => "21"
+const nodeMajorVersionStr = process.versions.node.replace(/\..*/, '')
+const nodeMajorVersion = Number.parseInt(nodeMajorVersionStr ?? '0')
+
 if (
   process.env.NVIM_NODE_HOST_DEBUG &&
-  semver.satisfies(process.version, '>=7.6.0 || >=6.12.0 <7.0.0') &&
+  nodeMajorVersion >= 8 &&
   process.execArgv.every(token => token !== '--inspect-brk')
 ) {
   const childHost = spawnSync(

packages/neovim/bin/cli.js

@@ -50,7 +50,6 @@
   },
   "dependencies": {
     "@msgpack/msgpack": "^2.8.0",
-    "semver": "^7.5.4",
     "winston": "3.11.0"
   },
   "devDependencies": {