Refactor various small stylistic things

* Change to `fileMatch` an array for JSON validation
* Add remark rule `lint-no-emphasis-as-heading`, as it’s no longer violated
* Use ESM for test script
* Add XO rule `unicorn/prefer-node-protocol`
* Refactor to combine build script into single `esbuild` call
* Refactor to tiny `.vscodeignore`

Closes GH-111.

Reviewed-by: Christian Murphy <christian.murphy.42@gmail.com>
Reviewed-by: Titus Wormer <tituswormer@gmail.com>

Author: remcohaszing

.vscodeignore

@@ -1,9 +1,6 @@
 .*
 .*/
-node_modules/
 src/
 test/
-test.js
-renovate.json
-package-lock.json
+test.mjs
 tsconfig.json

package.json

@@ -37,11 +37,10 @@
   "contributes": {
     "jsonValidation": [
       {
-        "fileMatch": ".remarkrc",
-        "url": "https://json.schemastore.org/remarkrc"
-      },
-      {
-        "fileMatch": ".remarkrc.json",
+        "fileMatch": [
+          ".remarkrc",
+          ".remarkrc.json"
+        ],
         "url": "https://json.schemastore.org/remarkrc"
       },
       {
@@ -67,10 +66,6 @@
   "remarkConfig": {
     "plugins": [
       "preset-wooorm",
-      [
-        "lint-no-emphasis-as-heading",
-        false
-      ],
       [
         "lint-no-html",
         false
@@ -88,8 +83,7 @@
   "xo": {
     "prettier": true,
     "rules": {
-      "unicorn/prefer-module": "off",
-      "unicorn/prefer-node-protocol": "off"
+      "unicorn/prefer-module": "off"
     }
   },
   "devDependencies": {
@@ -100,19 +94,16 @@
     "remark-cli": "^11.0.0",
     "remark-language-server": "^2.0.0",
     "remark-preset-wooorm": "^9.0.0",
-    "rimraf": "^3.0.0",
     "typescript": "^4.0.0",
     "vsce": "^2.0.0",
     "vscode-languageclient": "^8.0.0",
     "xo": "^0.50.0"
   },
   "scripts": {
-    "build:language-server": "esbuild node_modules/remark-language-server/index.js --bundle --platform=node --target=node16 --format=cjs --outfile=out/remark-language-server.js",
-    "build:extension": "esbuild src/extension.js --bundle --platform=node --target=node16 --format=cjs --external:vscode --external:./remark-language-server --outfile=out/extension.js",
     "lint": "xo",
-    "build": "rimraf out && npm run build:language-server && npm run build:extension",
+    "build": "esbuild extension=src/extension.js remark-language-server --bundle --platform=node --target=node16 --format=cjs --external:vscode --outdir=out --minify",
     "pretest": "npm run build && npm run lint",
-    "test": "node test.js",
+    "test": "node test.mjs",
     "vscode:prepublish": "npm run build"
   }
 }

readme.md

@@ -85,7 +85,7 @@ your `settings.json` (which you can open with <kbd>Ctrl</kbd> + <kbd>,</kbd>):
 ```json
 {
   "[markdown]": {
-    "editor.defaultFormatter": "unifiedjs.vscode-remark",
+    "editor.defaultFormatter": "unifiedjs.vscode-remark"
   }
 }
 ```

test.js

@@ -0,0 +1,7 @@
+#!/usr/bin/env node
+import {runTests} from '@vscode/test-electron'
+
+await runTests({
+  extensionDevelopmentPath: new URL('.', import.meta.url).pathname,
+  extensionTestsPath: new URL('test/index.test.js', import.meta.url).pathname
+})

test.mjs

@@ -1,6 +1,6 @@
-const assert = require('assert')
-const fs = require('fs/promises')
-const path = require('path')
+const assert = require('node:assert')
+const fs = require('node:fs/promises')
+const path = require('node:path')
 
 const {commands, extensions, Uri, window, workspace} = require('vscode')