build(deps): use @flex-development/when

- https://github.com/flex-development/when/tree/1.0.0

Signed-off-by: Lexus Drumgold <unicornware@flexdevelopment.llc>

Author: unicornware

eslint.config.mjs

@@ -33,15 +33,8 @@ const config = [
       'unicorn/text-encoding-identifier-case': 0
     }
   },
-  {
-    files: ['src/internal/__tests__/is-promise.spec.mts'],
-    rules: {
-      'unicorn/no-thenable': 0
-    }
-  },
   {
     files: [
-      'src/internal/chain-or-call.mts',
       'src/lib/get-source.mts',
       'src/lib/is-directory.mts',
       'src/lib/is-file.mts',

package.json

@@ -165,6 +165,7 @@
     "@flex-development/is-builtin": "3.2.0",
     "@flex-development/pathe": "4.0.2",
     "@flex-development/pkg-types": "4.1.0",
+    "@flex-development/when": "1.0.0",
     "devlop": "1.1.0"
   },
   "devDependencies": {

src/internal/__tests__/is-promise.spec.mts

@@ -1,6 +1,6 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
-exports[`unit:lib/resolveModule > fs (default) > should return resolved URL ("#internal/chain-or-call") 1`] = `file://\${process.cwd()}/src/internal/chain-or-call.mjs`;
+exports[`unit:lib/resolveModule > fs (default) > should return resolved URL ("#internal/identity") 1`] = `file://\${process.cwd()}/src/internal/identity.mjs`;
 
 exports[`unit:lib/resolveModule > fs (default) > should return resolved URL ("../../lib") 1`] = `file://\${process.cwd()}/src/lib/index.mjs`;
 
@@ -30,7 +30,7 @@ exports[`unit:lib/resolveModule > fs (default) > should return resolved URL ("su
 
 exports[`unit:lib/resolveModule > fs (default) > should return resolved URL ("unist") 1`] = `file://\${process.cwd()}/node_modules/@types/unist/index.d.ts`;
 
-exports[`unit:lib/resolveModule > fs (only async) > should return resolved URL ("#internal/chain-or-call") 1`] = `file://\${process.cwd()}/src/internal/chain-or-call.mjs`;
+exports[`unit:lib/resolveModule > fs (only async) > should return resolved URL ("#internal/identity") 1`] = `file://\${process.cwd()}/src/internal/identity.mjs`;
 
 exports[`unit:lib/resolveModule > fs (only async) > should return resolved URL ("../../lib") 1`] = `file://\${process.cwd()}/src/lib/index.mjs`;
 

src/internal/chain-or-call.mts

@@ -4,7 +4,6 @@
  */
 
 import fsa from '#fixtures/fsa'
-import isPromise from '#internal/is-promise'
 import testSubject from '#lib/get-source'
 import fsCaseType, { type FileSystemCaseType } from '#tests/utils/fs-case-type'
 import {
@@ -14,6 +13,7 @@ import {
 } from '@flex-development/errnode'
 import type { FileSystem } from '@flex-development/mlly'
 import pathe from '@flex-development/pathe'
+import { isThenable } from '@flex-development/when'
 
 describe('unit:lib/getSource', () => {
   describe.each<[fst: FileSystemCaseType, fs?: FileSystem | null | undefined]>([
@@ -76,7 +76,7 @@ describe('unit:lib/getSource', () => {
         const result = testSubject(id, { ...options, fs })
 
         // Expect
-        expect(result).to.not.satisfy(isPromise)
+        expect(result).to.not.satisfy(isThenable)
         expect(result).to.not.be.a('string')
         expect(String(result)).toMatchSnapshot()
       })
@@ -116,7 +116,7 @@ describe('unit:lib/getSource', () => {
         const result = testSubject(id, { ...options, fs })
 
         // Expect
-        expect(result).to.satisfy(isPromise)
+        expect(result).to.satisfy(isThenable)
         expect(await result).toMatchSnapshot()
       })
     })

src/internal/is-promise.mts

@@ -38,7 +38,7 @@ describe('unit:lib/resolveModule', () => {
     })
 
     it.each<Case>([
-      ['#internal/chain-or-call', import.meta.url, { conditions, ext: 'mjs' }],
+      ['#internal/identity', import.meta.url, { conditions, ext: 'mjs' }],
       ['../../lib', import.meta.url, { ext: () => 'mjs' }],
       ['@types/unist', import.meta.url],
       ['data:text/javascript,', import.meta.url],

src/lib/__snapshots__/node/resolve-module.snap

@@ -3,9 +3,8 @@
  * @module mlly/lib/getSource
  */
 
-import chainOrCall from '#internal/chain-or-call'
 import fs from '#internal/fs'
-import isPromise from '#internal/is-promise'
+import identity from '#internal/identity'
 import process from '#internal/process'
 import isFile from '#lib/is-file'
 import isModuleId from '#lib/is-module-id'
@@ -24,6 +23,7 @@ import type {
   ModuleId,
   Protocol
 } from '@flex-development/mlly'
+import when from '@flex-development/when'
 import { ok } from 'devlop'
 
 export default getSource
@@ -157,21 +157,7 @@ function getSource(
     )
   }
 
-  /**
-   * The source code.
-   *
-   * @var {ReturnType<GetSourceHandler>} code
-   */
-  let code: ReturnType<GetSourceHandler> = handle.call(context, url)
-
-  // resolve source code.
-  if (isPromise(code)) {
-    void code.then(resolved => (code = resolved))
-  }
-
-  return chainOrCall(code, () => {
-    return ok(!isPromise(code), 'expected `code` to be resolved'), code
-  })
+  return when(handle.call(context, url), identity)
 }
 
 /**
@@ -208,15 +194,8 @@ function data(this: GetSourceContext, url: URL): Buffer {
 function file(this: GetSourceContext, url: URL): Awaitable<FileContent | null> {
   ok(url.protocol === 'file:', 'expected `file:` URL')
 
-  /**
-   * Whether the file exists.
-   *
-   * @const {Awaitable<boolean>} exists
-   */
-  const exists: Awaitable<boolean> = isFile(url, this.fs)
-
-  return chainOrCall(exists, isFile => {
-    return isFile ?? exists ? this.fs.readFile(url, this.encoding) : null
+  return when(isFile(url, this.fs), exists => {
+    return exists ? this.fs.readFile(url, this.encoding) : null
   })
 }
 

src/lib/__tests__/get-source.spec.mts

@@ -3,18 +3,11 @@
  * @module mlly/lib/isDirectory
  */
 
-import chainOrCall from '#internal/chain-or-call'
 import constant from '#internal/constant'
 import dfs from '#internal/fs'
-import identity from '#internal/identity'
-import isPromise from '#internal/is-promise'
 import isModuleId from '#lib/is-module-id'
-import type {
-  Awaitable,
-  FileSystem,
-  ModuleId,
-  Stats
-} from '@flex-development/mlly'
+import type { Awaitable, FileSystem } from '@flex-development/mlly'
+import when from '@flex-development/when'
 
 export default isDirectory
 
@@ -66,28 +59,14 @@ function isDirectory(
   id: unknown,
   fs?: FileSystem | null | undefined
 ): Awaitable<boolean> {
-  if (isModuleId(id)) {
-    try {
-      if (typeof id === 'string' && id.startsWith('file:')) id = new URL(id)
+  if (typeof id === 'string' && id.startsWith('file:')) id = new URL(id)
+  if (!isModuleId(id)) return false
 
-      /**
-       * The stats object.
-       *
-       * @var {Awaitable<Stats | null>} stats
-       */
-      let stats: Awaitable<Stats | null> = (fs ?? dfs).stat(id as ModuleId)
+  fs ??= dfs
 
-      // resolve the stats object.
-      if (isPromise<Stats>(stats)) stats = stats.then(identity, constant(null))
-
-      return chainOrCall(stats, (): Awaitable<boolean> => {
-        if (!isPromise(stats)) return stats.isDirectory()
-        return stats.then(stats => stats?.isDirectory() ?? false)
-      })
-    } catch {
-      // swallow error.
-    }
+  try {
+    return when(fs.stat(id), stats => stats.isDirectory(), constant(false))
+  } catch {
+    return false // swallow error.
   }
-
-  return false
 }