refactor: use depseek for deps extraction (google#746)

* refactor: use depseek for deps parsing

* style: linting

* test: extend deps capture test

Author: antongolub

package-lock.json

@@ -67,6 +67,7 @@
     "@webpod/ps": "^0.0.0-beta.2",
     "c8": "^9.1.0",
     "chalk": "^5.3.0",
+    "depseek": "^0.4.1",
     "dts-bundle-generator": "^9.3.1",
     "esbuild": "^0.20.2",
     "esbuild-node-externals": "^1.13.0",

package.json

@@ -37,6 +37,7 @@ const entry = {
       '@types/which',
       'zurk',
       '@webpod/ps',
+      'depseek',
     ], // args['external-inlines'],
   },
   output: {

scripts/build-dts.mjs

@@ -13,7 +13,8 @@
 // limitations under the License.
 
 import { $ } from './core.js'
-import { spinner } from './experimental.js'
+import { spinner } from './goods.js'
+import { depseek } from './vendor.js'
 
 export async function installDeps(
   dependencies: Record<string, string>,
@@ -88,39 +89,25 @@ const builtins = new Set([
   'worker_threads',
   'zlib',
 ])
-const importRe = [
-  /\bimport\s+['"](?<path>[^'"]+)['"]/,
-  /\bimport\(['"](?<path>[^'"]+)['"]\)/,
-  /\brequire\(['"](?<path>[^'"]+)['"]\)/,
-  /\bfrom\s+['"](?<path>[^'"]+)['"]/,
-]
+
 const nameRe =
   /^(?<name>(@[a-z0-9-~][a-z0-9-._~]*\/)?[a-z0-9-~][a-z0-9-._~]*)\/?.*$/i
-const versionRe =
-  /(\/\/|\/\*)\s*@(?<version>[~^]?(v?[\dx*]+([-.][\d*a-z-]+)*))/i
+const versionRe = /^@(?<version>[~^]?(v?[\dx*]+([-.][\d*a-z-]+)*))/i
 
 export function parseDeps(content: Buffer): Record<string, string> {
-  const deps: Record<string, string> = {}
-  const lines = content.toString().split('\n')
-  for (let line of lines) {
-    const tuple = parseImports(line)
-    if (tuple) {
-      deps[tuple.name] = tuple.version
-    }
-  }
-  return deps
-}
-
-function parseImports(
-  line: string
-): { name: string; version: string } | undefined {
-  for (let re of importRe) {
-    const name = parsePackageName(re.exec(line)?.groups?.path)
-    const version = parseVersion(line)
-    if (name) {
-      return { name, version }
+  return depseek(content.toString() + '\n', { comments: true }).reduce<
+    Record<string, string>
+  >((m, { type, value }, i, list) => {
+    if (type === 'dep') {
+      const meta = list[i + 1]
+      const name = parsePackageName(value)
+      const version =
+        (meta?.type === 'comment' && parseVersion(meta?.value.trim())) ||
+        'latest'
+      if (name) m[name] = version
     }
-  }
+    return m
+  }, {})
 }
 
 function parsePackageName(path?: string): string | undefined {

src/deps.ts

@@ -50,6 +50,7 @@ export const YAML: {
 
 export const fs: typeof import('fs-extra') = _fs
 
+export { depseekSync as depseek } from 'depseek'
 export { type Options as GlobbyOptions } from 'globby'
 export { default as chalk, type ChalkInstance } from 'chalk'
 export { default as which } from 'which'

src/vendor.ts

@@ -95,6 +95,8 @@ describe('deps', () => {
   import fs from 'fs'
   import path from 'path'
   import foo from "foo"
+  // import aaa from 'a'
+  /* import bbb from 'b' */ 
   import bar from "bar" /* @1.0.0 */
   import baz from "baz" //    @^2.0
   import qux from "@qux/pkg/entry" //    @^3.0