chore: re-enable browser compat (#5731)

* chore: re-enable browser compat

* fix

.github/workflows/ci.yml

@@ -51,10 +51,8 @@ jobs:
           TZ=America/Toronto deno test datetime
         if: matrix.os == 'ubuntu-latest'
 
-      # TODO(kt3k): Rewrite test:browser task and enable this check
+      - name: Type check browser compatible modules
-      #- name: Type check browser compatible modules
+        run: deno task test:browser
-      #  shell: bash
-      #  run: deno task test:browser
 
       - name: Generate lcov
         run: deno task cov:gen

_tools/check_import_map.ts

@@ -1,6 +1,6 @@
 // Copyright 2018-2024 the Deno authors. All rights reserved. MIT license.
 
-import denoJson from "../deno.json" with { type: "json" };
+import denoJson from "../import_map.json" with { type: "json" };
 
 const invalidEntries = [];
 

browser-compat.tsconfig.json

@@ -6,6 +6,7 @@
       "ESNext"
     ]
   },
+  "importMap": "./import_map.json",
   "workspace": [
     "./archive",
     "./assert",

deno.json

@@ -5,54 +5,7 @@
     "noImplicitOverride": true,
     "noUncheckedIndexedAccess": true
   },
-  "imports": {
+  "importMap": "./import_map.json",
-    "@deno/graph": "jsr:@deno/graph@^0.74",
-    "@deno/doc": "jsr:@deno/doc@0.137",
-    "npm:/typescript": "npm:typescript@5.4.4",
-    "automation/": "https://raw.githubusercontent.com/denoland/automation/0.10.0/",
-    "graphviz": "npm:node-graphviz@^0.1.1",
-
-    "@std/archive": "jsr:@std/archive@^0.225.0",
-    "@std/assert": "jsr:@std/assert@^1.0.2",
-    "@std/async": "jsr:@std/async@^1.0.3",
-    "@std/bytes": "jsr:@std/bytes@^1.0.2",
-    "@std/cache": "jsr:@std/cache@^0.1.0",
-    "@std/cli": "jsr:@std/cli@^1.0.3",
-    "@std/collections": "jsr:@std/collections@^1.0.5",
-    "@std/crypto": "jsr:@std/crypto@^1.0.2",
-    "@std/csv": "jsr:@std/csv@^1.0.1",
-    "@std/data-structures": "jsr:@std/data-structures@^1.0.1",
-    "@std/datetime": "jsr:@std/datetime@^0.225.0",
-    "@std/dotenv": "jsr:@std/dotenv@^0.225.0",
-    "@std/encoding": "jsr:@std/encoding@^1.0.2",
-    "@std/expect": "jsr:@std/expect@^1.0.0",
-    "@std/fmt": "jsr:@std/fmt@^1.0.0",
-    "@std/front-matter": "jsr:@std/front-matter@^1.0.2",
-    "@std/fs": "jsr:@std/fs@^1.0.1",
-    "@std/html": "jsr:@std/html@^1.0.1",
-    "@std/http": "jsr:@std/http@^1.0.3",
-    "@std/ini": "jsr:@std/ini@^1.0.0-rc.3",
-    "@std/internal": "jsr:@std/internal@^1.0.1",
-    "@std/io": "jsr:@std/io@^0.224.5",
-    "@std/json": "jsr:@std/json@^1.0.0",
-    "@std/jsonc": "jsr:@std/jsonc@^1.0.0",
-    "@std/log": "jsr:@std/log@^0.224.5",
-    "@std/media-types": "jsr:@std/media-types@^1.0.2",
-    "@std/msgpack": "jsr:@std/msgpack@^1.0.0",
-    "@std/net": "jsr:@std/net@^1.0.0",
-    "@std/path": "jsr:@std/path@^1.0.2",
-    "@std/regexp": "jsr:@std/regexp@^1.0.0",
-    "@std/semver": "jsr:@std/semver@^1.0.1",
-    "@std/streams": "jsr:@std/streams@^1.0.2",
-    "@std/testing": "jsr:@std/testing@^1.0.0",
-    "@std/text": "jsr:@std/text@^1.0.3",
-    "@std/toml": "jsr:@std/toml@^1.0.0",
-    "@std/ulid": "jsr:@std/ulid@^1.0.0",
-    "@std/url": "jsr:@std/url@^0.225.0",
-    "@std/uuid": "jsr:@std/uuid@^1.0.1",
-    "@std/webgpu": "jsr:@std/webgpu@^0.224.5",
-    "@std/yaml": "jsr:@std/yaml@^1.0.3"
-  },
   "tasks": {
     "test": "deno test --unstable-http --unstable-webgpu --doc --allow-all --parallel --coverage --trace-leaks --clean",
     "test:browser": "git grep --name-only \"This module is browser compatible.\" | grep -v deno.json | grep -v .github/workflows | grep -v _tools | grep -v encoding/README.md | grep -v media_types/vendor/update.ts | xargs deno check --config browser-compat.tsconfig.json",

import_map.json

@@ -0,0 +1,50 @@
+{
+  "imports": {
+    "@deno/graph": "jsr:@deno/graph@^0.74",
+    "@deno/doc": "jsr:@deno/doc@0.137",
+    "npm:/typescript": "npm:typescript@5.4.4",
+    "automation/": "https://raw.githubusercontent.com/denoland/automation/0.10.0/",
+    "graphviz": "npm:node-graphviz@^0.1.1",
+
+    "@std/archive": "jsr:@std/archive@^0.225.0",
+    "@std/assert": "jsr:@std/assert@^1.0.2",
+    "@std/async": "jsr:@std/async@^1.0.3",
+    "@std/bytes": "jsr:@std/bytes@^1.0.2",
+    "@std/cache": "jsr:@std/cache@^0.1.0",
+    "@std/cli": "jsr:@std/cli@^1.0.3",
+    "@std/collections": "jsr:@std/collections@^1.0.5",
+    "@std/crypto": "jsr:@std/crypto@^1.0.2",
+    "@std/csv": "jsr:@std/csv@^1.0.1",
+    "@std/data-structures": "jsr:@std/data-structures@^1.0.1",
+    "@std/datetime": "jsr:@std/datetime@^0.225.0",
+    "@std/dotenv": "jsr:@std/dotenv@^0.225.0",
+    "@std/encoding": "jsr:@std/encoding@^1.0.2",
+    "@std/expect": "jsr:@std/expect@^1.0.0",
+    "@std/fmt": "jsr:@std/fmt@^1.0.0",
+    "@std/front-matter": "jsr:@std/front-matter@^1.0.2",
+    "@std/fs": "jsr:@std/fs@^1.0.1",
+    "@std/html": "jsr:@std/html@^1.0.1",
+    "@std/http": "jsr:@std/http@^1.0.3",
+    "@std/ini": "jsr:@std/ini@^1.0.0-rc.3",
+    "@std/internal": "jsr:@std/internal@^1.0.1",
+    "@std/io": "jsr:@std/io@^0.224.5",
+    "@std/json": "jsr:@std/json@^1.0.0",
+    "@std/jsonc": "jsr:@std/jsonc@^1.0.0",
+    "@std/log": "jsr:@std/log@^0.224.5",
+    "@std/media-types": "jsr:@std/media-types@^1.0.2",
+    "@std/msgpack": "jsr:@std/msgpack@^1.0.0",
+    "@std/net": "jsr:@std/net@^1.0.0",
+    "@std/path": "jsr:@std/path@^1.0.2",
+    "@std/regexp": "jsr:@std/regexp@^1.0.0",
+    "@std/semver": "jsr:@std/semver@^1.0.1",
+    "@std/streams": "jsr:@std/streams@^1.0.2",
+    "@std/testing": "jsr:@std/testing@^1.0.0",
+    "@std/text": "jsr:@std/text@^1.0.3",
+    "@std/toml": "jsr:@std/toml@^1.0.0",
+    "@std/ulid": "jsr:@std/ulid@^1.0.0",
+    "@std/url": "jsr:@std/url@^0.225.0",
+    "@std/uuid": "jsr:@std/uuid@^1.0.1",
+    "@std/webgpu": "jsr:@std/webgpu@^0.224.5",
+    "@std/yaml": "jsr:@std/yaml@^1.0.3"
+  }
+}