Run lint locally before tests; fix parameter-property lint error (#250)

CI runs `npm run lint` but the local `npm test` did not, so lint
failures (like the eslint parameter-property warning in malloySQL.ts)
only showed up at publish time. Add `pretest`/`pretest-silent` hooks so
lint gates the local test run, and fix the existing violation by
collapsing the VirtualURIFileHandler constructor into a parameter
property.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: mtoy-googly-moogly

package.json

@@ -21,7 +21,9 @@
   },
   "scripts": {
     "lint": "tsc --noEmit && eslint .",
+    "pretest": "npm run lint",
     "test": "NODE_ENV=test jest --projects jest.config.js --runInBand",
+    "pretest-silent": "npm run lint",
     "test-silent": "NODE_ENV=test JEST_SILENT_REPORTER_SHOW_PATHS=true jest  --projects jest.config.js --runInBand --reporters jest-silent-reporter",
     "test-e2e": "npm run package-npm test > /dev/null 2>&1 && NODE_ENV=test jest --projects=jest-e2e.config.js --runInBand",
     "test-silent-e2e": "npm run package-npm test > /dev/null 2>&1 && NODE_ENV=test JEST_SILENT_REPORTER_SHOW_PATHS=true jest --projects=jest-e2e.config.js --runInBand --reporters jest-silent-reporter",

src/malloy/malloySQL.ts

@@ -17,13 +17,10 @@ import {
 } from './util';
 
 class VirtualURIFileHandler implements URLReader {
-  private uriReader: URLReader;
   private url: URL = new URL('malloy://internal');
   private contents: string = '';
 
-  constructor(uriReader: URLReader) {
-    this.uriReader = uriReader;
-  }
+  constructor(private uriReader: URLReader) {}
 
   public setVirtualFile(url: URL, contents: string): void {
     this.url = url;