refactor: use ts exts for relative paths (#1111)

* refactor: use ts exts for relative paths

* test:  fix coverage

Author: antongolub

.github/workflows/test.yml

@@ -215,7 +215,7 @@ jobs:
       - name: Install deps
         run: npm ci
       - name: Install TypeScript ${{ matrix.ts }}
-        run: npm i --force --no-package-lock typescript@${{ matrix.ts }}
+        run: npm i --force --no-package-lock --no-fund typescript@${{ matrix.ts }}
       - uses: actions/download-artifact@v4
         with:
           name: build

.nycrc

@@ -2,5 +2,14 @@
   "reporter": ["html", "text"],
   "lines": 98,
   "branches": "90",
-  "statements": "98"
+  "statements": "98",
+  "exclude": [
+    "build/deno.js",
+    "build/vendor-extra.cjs",
+    "build/vendor-core.cjs",
+    "build/esblib.cjs",
+    "test/**",
+    "scripts",
+    "src/util.ts"
+  ]
 }

.size-limit.json

@@ -30,7 +30,7 @@
   {
     "name": "all",
     "path": "build/*",
-    "limit": "850.2 kB",
+    "limit": "850.25 kB",
     "brotli": false,
     "gzip": false
   }

package.json

@@ -63,7 +63,7 @@
     "fmt": "prettier --write .",
     "fmt:check": "prettier --check .",
     "build": "npm run build:js && npm run build:dts && npm run build:tests",
-    "build:js": "node scripts/build-js.mjs --format=cjs --hybrid --entry=src/*.ts:!src/error.ts:!src/repl.ts && npm run build:vendor",
+    "build:js": "node scripts/build-js.mjs --format=cjs --hybrid --entry=src/*.ts:!src/error.ts:!src/repl.ts:!src/md.ts && npm run build:vendor",
     "build:vendor": "node scripts/build-js.mjs --format=cjs --entry=src/vendor-*.ts --bundle=all",
     "build:tests": "node scripts/build-tests.mjs",
     "build:dts": "tsc --project tsconfig.json && rm build/error.d.ts build/repl.d.ts && node scripts/build-dts.mjs",
@@ -75,7 +75,7 @@
     "test:it": "node ./test/it/build.test.js",
     "test:jsr": "node ./test/it/build-jsr.test.js",
     "test:unit": "node --experimental-strip-types ./test/all.test.js",
-    "test:coverage": "c8 -x build/deno.js -x build/vendor-extra.cjs -x build/vendor-core.cjs -x build/esblib.cjs -x 'test/**' -x scripts --check-coverage npm run test:unit",
+    "test:coverage": "c8 -c .nycrc --check-coverage npm run test:unit",
     "test:circular": "madge --circular src/*",
     "test:types": "tsd",
     "test:license": "node ./test/extra.test.js",
@@ -84,7 +84,7 @@
     "test:smoke:strip-types": "node --experimental-strip-types test/smoke/ts.test.ts",
     "test:smoke:tsx": "tsx test/smoke/ts.test.ts",
     "test:smoke:tsc": "cd test/smoke && mkdir -p node_modules && ln -s ../../../  ./node_modules/zx; ../../node_modules/typescript/bin/tsc -v && ../../node_modules/typescript/bin/tsc --esModuleInterop --module node16 --rootDir . --outdir ./temp ts.test.ts && node ./temp/ts.test.js",
-    "test:smoke:ts-node": "node --loader ts-node/esm test/smoke/ts.test.ts",
+    "test:smoke:ts-node": "cd test/smoke && node --loader ts-node/esm ts.test.ts",
     "test:smoke:bun": "bun test ./test/smoke/bun.test.js && bun ./test/smoke/node.test.mjs",
     "test:smoke:win32": "node ./test/smoke/win32.test.js",
     "test:smoke:cjs": "node ./test/smoke/node.test.cjs",

scripts/build-dts.mjs

@@ -110,6 +110,7 @@ for (const dts of await glob(['build/**/*.d.ts', '!build/vendor-*.d.ts'])) {
   const contents =
     (pkgEntries.some((e) => dts.includes(e)) ? prefix : '') +
     (await fs.readFile(dts, 'utf8'))
+      .replaceAll(".ts';", ".js';")
       .split('\n')
       .filter((line) => !line.startsWith('/// <reference types'))
       .join('\n')

src/cli.ts

@@ -28,11 +28,12 @@ import {
   path,
   stdin,
   VERSION,
-} from './index.js'
-import { installDeps, parseDeps } from './deps.js'
-import { startRepl } from './repl.js'
-import { randomId, bufToString } from './util.js'
-import { createRequire, type minimist } from './vendor.js'
+} from './index.ts'
+import { installDeps, parseDeps } from './deps.ts'
+import { startRepl } from './repl.ts'
+import { randomId, bufToString } from './util.ts'
+import { transformMarkdown } from './md.ts'
+import { createRequire, type minimist } from './vendor.ts'
 
 const EXT = '.mjs'
 const EXT_RE = /^\.[mc]?[jt]sx?$/
@@ -206,86 +207,15 @@ async function readScriptFromHttp(remote: string): Promise<string> {
   return res.text()
 }
 
+export { transformMarkdown }
+
 export function injectGlobalRequire(origin: string): void {
   const __filename = path.resolve(origin)
   const __dirname = path.dirname(__filename)
   const require = createRequire(origin)
   Object.assign(globalThis, { __filename, __dirname, require })
 }
 
-export function transformMarkdown(buf: Buffer | string): string {
-  const output = []
-  const tabRe = /^(  +|\t)/
-  const codeBlockRe =
-    /^(?<fence>(`{3,20}|~{3,20}))(?:(?<js>(js|javascript|ts|typescript))|(?<bash>(sh|shell|bash))|.*)$/
-  let state = 'root'
-  let codeBlockEnd = ''
-  let prevLineIsEmpty = true
-  for (const line of bufToString(buf).split(/\r?\n/)) {
-    switch (state) {
-      case 'root':
-        if (tabRe.test(line) && prevLineIsEmpty) {
-          output.push(line)
-          state = 'tab'
-          continue
-        }
-        const { fence, js, bash } = line.match(codeBlockRe)?.groups || {}
-        if (!fence) {
-          prevLineIsEmpty = line === ''
-          output.push('// ' + line)
-          continue
-        }
-        codeBlockEnd = fence
-        if (js) {
-          state = 'js'
-          output.push('')
-        } else if (bash) {
-          state = 'bash'
-          output.push('await $`')
-        } else {
-          state = 'other'
-          output.push('')
-        }
-        break
-      case 'tab':
-        if (line === '') {
-          output.push('')
-        } else if (tabRe.test(line)) {
-          output.push(line)
-        } else {
-          output.push('// ' + line)
-          state = 'root'
-        }
-        break
-      case 'js':
-        if (line === codeBlockEnd) {
-          output.push('')
-          state = 'root'
-        } else {
-          output.push(line)
-        }
-        break
-      case 'bash':
-        if (line === codeBlockEnd) {
-          output.push('`')
-          state = 'root'
-        } else {
-          output.push(line)
-        }
-        break
-      case 'other':
-        if (line === codeBlockEnd) {
-          output.push('')
-          state = 'root'
-        } else {
-          output.push('// ' + line)
-        }
-        break
-    }
-  }
-  return output.join('\n')
-}
-
 export function isMain(
   metaurl: string = import.meta.url,
   scriptpath: string = process.argv[1]

src/core.ts

@@ -31,7 +31,7 @@ import {
   formatExitMessage,
   getCallerLocation,
   getExitCodeInfo,
-} from './error.js'
+} from './error.ts'
 import {
   exec,
   buildCmd,
@@ -41,7 +41,7 @@ import {
   VoidStream,
   type ChalkInstance,
   type TSpawnStore,
-} from './vendor-core.js'
+} from './vendor-core.ts'
 import {
   type Duration,
   log,
@@ -60,9 +60,9 @@ import {
   toCamelCase,
   randomId,
   bufArrJoin,
-} from './util.js'
+} from './util.ts'
 
-export { log, type LogEntry } from './util.js'
+export { log, type LogEntry } from './util.ts'
 
 const CWD = Symbol('processCwd')
 const SYNC = Symbol('syncExec')

src/deps.ts

@@ -12,9 +12,9 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-import { $ } from './core.js'
-import { spinner } from './goods.js'
-import { depseek } from './vendor.js'
+import { $ } from './core.ts'
+import { spinner } from './goods.ts'
+import { depseek } from './vendor.ts'
 
 /**
  * Install npm dependencies

src/globals.ts

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-import * as _ from './index.js'
+import * as _ from './index.ts'
 
 Object.assign(globalThis, _)
 // TODO: global types not working with jsr.io

src/goods.ts

@@ -14,21 +14,21 @@
 
 import assert from 'node:assert'
 import { createInterface } from 'node:readline'
-import { $, within, ProcessOutput } from './core.js'
+import { $, within, ProcessOutput } from './core.ts'
 import {
   type Duration,
   identity,
   isStringLiteral,
   parseBool,
   parseDuration,
   toCamelCase,
-} from './util.js'
+} from './util.ts'
 import {
   type RequestInfo,
   type RequestInit,
   nodeFetch,
   minimist,
-} from './vendor.js'
+} from './vendor.ts'
 
 export { default as path } from 'node:path'
 export * as os from 'node:os'