Delete the out-node build

Gruntfile.js → Gruntfile.cjs

@@ -14,7 +14,7 @@ module.exports = function (grunt) {
     pkg: grunt.file.readJSON('package.json'),
 
     clean: {
-      out: ['gen/', 'out/', 'out-wpt/', 'out-node/'],
+      out: ['gen/', 'out/', 'out-wpt/'],
     },
 
     run: {
@@ -89,14 +89,6 @@ module.exports = function (grunt) {
           '--ignore=src/common/tools',
         ],
       },
-      'build-out-node': {
-        cmd: 'node',
-        args: [
-          'node_modules/typescript/lib/tsc.js',
-          '--project', 'node.tsconfig.json',
-          '--outDir', 'out-node/',
-        ],
-      },
       'copy-assets': {
         cmd: 'node',
         args: [
@@ -188,7 +180,6 @@ module.exports = function (grunt) {
         tasks: [
           'build-standalone',
           'build-wpt',
-          'run:build-out-node',
         ],
       },
       'all-checks': {

babel.config.js

@@ -1,4 +1,4 @@
-module.exports = function (api) {
+export default function (api) {
   api.cache(true);
   return {
     presets: ['@babel/preset-typescript'],

cts.code-workspace

@@ -26,7 +26,6 @@
       "deploy_key.enc": true,
       "node_modules": true,
       "out": true,
-      "out-node": true,
       "out-wpt": true,
       "docs/tsdoc": true,
       "package-lock.json": true

package.json

@@ -2,6 +2,7 @@
   "name": "@webgpu/cts",
   "version": "0.1.0",
   "description": "WebGPU Conformance Test Suite",
+  "type": "module",
   "scripts": {
     "test": "grunt all",
     "all": "grunt all",

src/common/runtime/helper/sys.ts

@@ -1,8 +1,8 @@
 /* eslint no-process-exit: "off" */
 /* eslint @typescript-eslint/no-namespace: "off" */
 
-function node() {
-  const { existsSync } = require('fs');
+async function node() {
+  const { existsSync } = await import('fs');
 
   return {
     type: 'node',
@@ -41,6 +41,6 @@ function deno() {
   };
 }
 
-const sys = typeof globalThis.process !== 'undefined' ? node() : deno();
+const sys = typeof globalThis.process !== 'undefined' ? await node() : deno();
 
 export default sys;

src/common/tools/commonjs/package.json

@@ -0,0 +1,3 @@
+{
+  "type": "commonjs"
+}

src/common/tools/setup-ts-in-node.js → src/common/tools/commonjs/setup-ts-in-node.js

@@ -3,7 +3,7 @@ const path = require('path');
 // Automatically transpile .ts imports
 require('ts-node').register({
   // Specify the project file so ts-node doesn't try to find it itself based on the CWD.
-  project: path.resolve(__dirname, '../../../tsconfig.json'),
+  project: path.resolve(__dirname, '../../../../tsconfig.json'),
   compilerOptions: {
     module: 'commonjs',
   },

src/common/tools/version.ts

@@ -1,4 +1,4 @@
-export const version = require('child_process')
+export const version = (await import('child_process'))
   .execSync('git describe --always --abbrev=0 --dirty')
   .toString()
   .trim();

src/webgpu/web_platform/worker/worker.spec.ts

@@ -27,7 +27,6 @@ g.test('worker')
     // is using commonjs which doesn't support import.meta. Further,
     // we need to put the url in a string add pass the string to import
     // otherwise typescript tries to parse the file which again, fails.
-    // worker_launcher.js is excluded in node.tsconfig.json.
     const url = './worker_launcher.js';
     const { launchWorker } = await import(url);
     const result = await launchWorker();

tools/checklist

@@ -7,5 +7,5 @@
 // and every query in it is valid (e.g. renames have been applied, and new tests added
 // to the spreadsheet have also been added to the CTS).
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 require('../src/common/tools/checklist.ts');

tools/dev_server

@@ -1,4 +1,4 @@
 #!/usr/bin/env node
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 require('../src/common/tools/dev_server.ts');

tools/gen_cache

@@ -1,4 +1,4 @@
 #!/usr/bin/env node
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 require('../src/common/tools/gen_cache.ts');

tools/gen_listings

@@ -3,5 +3,5 @@
 // Crawl a suite directory (e.g. src/webgpu/) to generate a listing.js containing
 // the listing of test files in the suite.
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 require('../src/common/tools/gen_listings.ts');

tools/gen_version

@@ -5,7 +5,7 @@
 
 /* eslint-disable no-console */
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 const fs = require('fs');
 
 const myself = 'tools/gen_version';
@@ -14,7 +14,7 @@ if (!fs.existsSync(myself)) {
   process.exit(1);
 }
 
-const { version } = require('../src/common/tools/version.ts');
+const { version } = require('../src/common/tools/version.ts'); // FIXME doesn't work because in order to require the placeholder version.ts it needs to be commonjs, but we can't configure that for a single file
 
 fs.mkdirSync('./gen/common/internal', { recursive: true });
 // This will be copied into the various other build directories.

tools/gen_wpt_cts_html

@@ -35,5 +35,5 @@
 //   ?q=webgpu:a/bar:bar1={"x":3} <- [ Mac ]
 //   ?q=webgpu:a/bar:bar2~
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 require('../src/common/tools/gen_wpt_cts_html.ts');

tools/merge_listing_times

@@ -3,7 +3,7 @@
 // See `docs/adding_timing_metadata.md` for an explanation of listing times, and
 // a walkthrough on adding entries for new tests.
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 
 // See the help message in this file for info on how to use the tool.
 require('../src/common/tools/merge_listing_times.ts');

tools/package.json

@@ -0,0 +1,3 @@
+{
+  "type": "commonjs"
+}

tools/run_node

@@ -2,5 +2,5 @@
 
 // Run test suites under node.
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 require('../src/common/runtime/cmdline.ts');

tools/run_wpt_ref_tests

@@ -1,4 +1,4 @@
 #!/usr/bin/env node
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 require('../src/common/tools/run_wpt_ref_tests.ts');

tools/validate

@@ -2,5 +2,5 @@
 
 // Validate several properties of test files and the tests in them.
 
-require('../src/common/tools/setup-ts-in-node.js');
+require('../src/common/tools/commonjs/setup-ts-in-node.js');
 require('../src/common/tools/validate.ts');