Don't automatically include all types from dependencies (#3704)

* Add npm run lint

* Don't automatically include all types from dependencies

This produces an error if you try to use something from a global scope
accidentally. It's only needed for "ambient" definitions; if the thing
is included directly (like for all of the Node libraries like `fs`) then
its types come along with it automatically.

Also:
- Replace `/// <reference />` usage because it does the same thing.
- Remove unused `@types/node` dependency (TypeScript has that built in).

Author: kainino0x

Gruntfile.js

@@ -311,6 +311,7 @@ module.exports = function (grunt) {
   ]);
 
   registerTaskAndAddToHelp('serve', 'Serve out/ (without building anything)', ['run:serve']);
+  registerTaskAndAddToHelp('lint', 'Check lint and formatting', ['run:lint']);
   registerTaskAndAddToHelp('fix', 'Fix lint and formatting', ['run:fix']);
 
   addExistingTaskToHelp('clean', 'Delete built and generated files');

package-lock.json

@@ -11,6 +11,7 @@
     "checks": "grunt checks",
     "unittest": "grunt unittest",
     "typecheck": "grunt typecheck",
+    "lint": "grunt run:lint",
     "fix": "grunt fix",
     "gen_wpt_cts_html": "node tools/gen_wpt_cts_html",
     "gen_cache": "node tools/gen_cache",
@@ -43,7 +44,6 @@
     "@types/express": "^4.17.20",
     "@types/jquery": "^3.5.25",
     "@types/morgan": "^1.9.7",
-    "@types/node": "^20.8.10",
     "@types/offscreencanvas": "^2019.7.2",
     "@types/pngjs": "^6.0.3",
     "@types/serve-index": "^1.9.3",

package.json

@@ -1,4 +1,5 @@
 // Implements the standalone test runner (see also: /standalone/index.html).
+
 /* eslint no-console: "off" */
 
 import { dataCache } from '../framework/data_cache.js';

src/common/runtime/standalone.ts

@@ -1,5 +1,3 @@
-/// <reference types="@webgpu/types" />
-
 import { TestCaseRecorder } from '../framework/fixture.js';
 
 import { ErrorWithExtra, assert, objectEquals } from './util.js';

src/common/util/navigator_gpu.ts

@@ -3,6 +3,15 @@
   "compilerOptions": {
     "lib": ["dom", "es2020"],
     "module": "esnext",
+    // Use types only from these packages (not everything in package.json, which is the default).
+    // Note that TypeScript's `/// <reference />` is the same as adding an entry here - it's global
+    // to the whole project.
+    "types": [
+      "@types/dom-mediacapture-transform",
+      "@webgpu/types",
+      // Required for standalone.ts
+      "@types/jquery",
+    ],
     /* Output options */
     "noEmit": true,
     /* Strict type-checking options */