fix external deps

Author: jhheider

.github/deno-to-node.ts

@@ -41,7 +41,7 @@ await build({
   },
   importMap: "deno.json",
   mappings: {
-    "https://deno.land/x/is_what@v4.1.8/src/index.ts": "is-what",
+    "https://deno.land/x/is_what@v4.1.13/src/index.ts": "is-what",
     "https://deno.land/x/outdent@v0.8.0/mod.ts": "outdent",
     "./src/utils/flock.deno.ts": "./src/utils/flock.node.ts"
   },

.github/workflows/cd.yml

@@ -15,33 +15,50 @@ concurrency:
 
 jobs:
   check:
+    if: ${{ github.event.release.prerelease }}
     uses: ./.github/workflows/ci.yml
 
-  build:
+  dnt:
     needs: [check]
-    if: ${{ github.event.release.prerelease }}
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os:
+          - [self-hosted, macOS, X64]
+          - ubuntu-latest  # ∵ resilience: PRs verify dnt with this env
+          - [self-hosted, macOS, ARM64]
+          - [self-hosted, linux, ARM64]
     steps:
       - uses: actions/checkout@v3
       - uses: teaxyz/setup@v0
+      - run: deno task dnt ${{ github.event.release.tag_name }}
+
+      - uses: actions/upload-artifact@v3
+        if: ${{ matrix.os == 'ubuntu-latest' }}
+        with:
+          path: dist
+
+  publish-npm:
+    needs: [dnt]
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/download-artifact@v3
 
       # if we don’t do this the npm publish step doesn’t work
       - uses: actions/setup-node@v2
         with:
           registry-url: https://registry.npmjs.org
 
-      - name: build
-        run: .github/deno-to-node.ts ${{ github.event.release.tag_name }}
-
       - run: npm publish --access public
         working-directory: dist
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
 
-  publish:
+  publish-github:
     permissions:
       contents: write
-    needs: [build]
+    needs: [publish-npm]
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3

.github/workflows/ci.yml

@@ -24,34 +24,45 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - uses: teaxyz/setup@v0
-      - run: deno cache $(find . -name \*.ts)
-      - run: deno task test --coverage=cov_profile --no-check
-      # ^^ no check as we have a separate type-checking step and it’s noise here
+      - run: deno cache mod.ts
+
+      - run: deno task test
+          --coverage=cov_profile
+          --no-check # ⬆signal∶noise & ∵ we have `jobs.typecheck`
+
       - run: deno coverage cov_profile --lcov --exclude=tests/ --output=cov_profile.lcov
+        if: ${{ github.event_name != 'workflow_call' }}
+
       - uses: coverallsapp/github-action@v1
+        if: ${{ github.event_name != 'workflow_call' }}
         with:
           path-to-lcov: cov_profile.lcov
           parallel: true
           flag-name: ${{ matrix.platform.id }}
 
   upload-coverage:
+    if: ${{ github.event_name != 'workflow_call' }}
     needs: tests
     runs-on: ubuntu-latest
     steps:
     - uses: coverallsapp/github-action@v1
       with:
         parallel-finished: true
 
+  verify-usage-as-deno-lib:
+    # we’re checking no import-map type imports snuck in
+    runs-on: ubuntu-latest
+    if: ${{ github.event_name != 'workflow_call' }}
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          path: src
+      - uses: denoland/setup-deno@v1
+      - run: deno run --no-config --unstable src/mod.ts
+
   dnt:
-    runs-on: ${{ matrix.os }}
-    strategy:
-      fail-fast: false
-      matrix:
-        os:
-          - [self-hosted, macOS, X64]
-          - [self-hosted, linux, X64]
-          - [self-hosted, macOS, ARM64]
-          - [self-hosted, linux, ARM64]
+    runs-on: ubuntu-latest
+    if: ${{ github.event_name != 'workflow_call' }}
     steps:
       - uses: actions/checkout@v3
       - uses: teaxyz/setup@v0

deno.json

@@ -29,7 +29,6 @@
   },
   "imports": {
     "is-what": "https://deno.land/x/is_what@v4.1.8/src/index.ts",
-    "outdent": "https://deno.land/x/outdent@v0.8.0/mod.ts",
     "deno/": "https://deno.land/std@0.189.0/"
   }
 }

mod.ts

@@ -1,12 +1,13 @@
+import "./src/utils/misc.ts"
+import { flatmap, validate } from "./src/utils/misc.ts"
+
 import host from "./src/utils/host.ts"
 import SemVer, * as semver from "./src/utils/semver.ts"
 import Path from "./src/utils/Path.ts"
-export * as types from "./src/types.ts"
-
-import "./src/utils/misc.ts"
-import { flatmap, validate } from "./src/utils/misc.ts"
 
+export * as types from "./src/types.ts"
 import * as pkg from "./src/utils/pkg.ts"
+
 import TeaError, { panic } from "./src/utils/error.ts"
 import useConfig from "./src/hooks/useConfig.ts"
 import useOffLicense  from "./src/hooks/useOffLicense.ts"

src/deps.ts

@@ -0,0 +1,22 @@
+import * as is_what from "https://deno.land/x/is_what@v4.1.13/src/index.ts"
+export { is_what }
+
+import { type PlainObject } from "https://deno.land/x/is_what@v4.1.13/src/index.ts"
+export type { PlainObject }
+
+import * as outdent from "https://deno.land/x/outdent@v0.8.0/mod.ts"
+export { outdent }
+
+// importing super specifically to reduce final npm bundle size
+import * as crypto from "https://deno.land/std@0.190.0/crypto/mod.ts"
+import { moveSync } from "https://deno.land/std@0.190.0/fs/move.ts"
+import { readLines } from "https://deno.land/std@0.190.0/io/read_lines.ts"
+import { writeAll } from "https://deno.land/std@0.190.0/streams/write_all.ts"
+import { parse as parseYaml } from "https://deno.land/std@0.190.0/yaml/parse.ts"
+
+const streams = { writeAll }
+const io = { readLines }
+const fs = { moveSync }
+const deno = { readLines, crypto, fs, io, streams, parseYaml }
+
+export { deno }

src/hooks/useConfig.ts

@@ -1,5 +1,5 @@
+import { flatmap } from "../utils/misc.ts"
 import host from "../utils/host.ts"
-import { flatmap } from "../utils/misc.ts";
 import Path from "../utils/Path.ts"
 
 export interface Config {

src/hooks/useDownload.ts

@@ -1,5 +1,6 @@
-import { crypto, toHashString } from "deno/crypto/mod.ts"
-import { writeAll } from "deno/streams/write_all.ts"
+import { deno } from "../deps.ts"
+const { crypto: crypto_, streams: { writeAll } } = deno
+const { toHashString, crypto } = crypto_
 import TeaError, { panic } from "../utils/error.ts"
 import useConfig from "./useConfig.ts"
 import useFetch from "./useFetch.ts"

src/hooks/usePantry.ts

@@ -1,4 +1,5 @@
-import { isNumber, isPlainObject, isString, isArray, isPrimitive, PlainObject, isBoolean } from "is-what"
+import { is_what, PlainObject } from "../deps.ts"
+const { isNumber, isPlainObject, isString, isArray, isPrimitive, isBoolean } = is_what
 import { validatePackageRequirement } from "../utils/hacks.ts"
 import { Package, Installation } from "../types.ts"
 import useMoustaches from "./useMoustaches.ts"

src/hooks/useSync.ts

@@ -1,6 +1,7 @@
 // deno-lint-ignore-file no-deprecated-deno-api
 // ^^ Deno.Command is not implemented for dnt shims yet
-import { writeAll } from "deno/streams/write_all.ts"
+import { deno } from "../deps.ts"
+const { streams: { writeAll } } = deno
 import { flock } from "../utils/flock.deno.ts"
 import useDownload from "./useDownload.ts"
 import usePantry from "./usePantry.ts"