fix(deps): upgrade glob from v8 to v10

Author: serhalp

package-lock.json

@@ -55,7 +55,7 @@
     "fast-glob": "^3.3.2",
     "filter-obj": "^5.0.0",
     "find-up": "^6.0.0",
-    "glob": "^8.0.3",
+    "glob": "^10.4.5",
     "is-builtin-module": "^3.1.0",
     "is-path-inside": "^4.0.0",
     "junk": "^4.0.0",
@@ -78,7 +78,6 @@
   },
   "devDependencies": {
     "@types/archiver": "6.0.3",
-    "@types/glob": "8.1.0",
     "@types/is-ci": "3.0.4",
     "@types/node": "20.12.11",
     "@types/normalize-path": "3.0.2",

packages/zip-it-and-ship-it/package.json

@@ -62,6 +62,7 @@ const includedFilesToEsbuildExternals = async (includedFiles: string[], baseDir:
       const resolved = await glob(pattern, {
         noglobstar: true,
         cwd: baseDir,
+        dotRelative: true,
       })
 
       result.push(...resolved)

packages/zip-it-and-ship-it/src/runtimes/node/bundlers/esbuild/bundler.ts

@@ -1,28 +1,30 @@
-import { promisify } from 'util'
-
-import globFunction from 'glob'
-import { minimatch as minimatchFunction, MinimatchOptions } from 'minimatch'
+import { glob as originalGlob, type GlobOptions } from 'glob'
+import { minimatch as minimatchFunction, type MinimatchOptions } from 'minimatch'
 import normalizePath from 'normalize-path'
 
-const pGlob = promisify(globFunction)
+// We don't use this. Specifying that we don't makes typing easier, since otherwise `glob` returns
+// either `string[]` or `Path[]` depending on this passed option.
+type Options = Omit<GlobOptions, 'withFileTypes'>
 
 /**
  * Both glob and minimatch only support unix style slashes in patterns
- * For this reason we wrap them and ensure all patters are always unixified
+ * For this reason we wrap them and ensure all patterns are always unixified
  * We use `normalize-path` here instead of `unixify` because we do not want to remove drive letters
  */
-
-export const glob = function (pattern: string, options: globFunction.IOptions): Promise<string[]> {
-  let normalizedIgnore
+export const glob = function (pattern: string, options: Options): Promise<string[]> {
+  let normalizedIgnore: string | undefined
 
   if (options.ignore) {
-    normalizedIgnore =
-      typeof options.ignore === 'string'
-        ? normalizePath(options.ignore)
-        : options.ignore.map((expression) => normalizePath(expression))
+    if (typeof options.ignore === 'string') {
+      normalizedIgnore = normalizePath(options.ignore)
+    } else if (Array.isArray(options.ignore)) {
+      options.ignore.map((expression) => normalizePath(expression))
+    } else {
+      throw new Error('Custom glob ignore is not supported')
+    }
   }
 
-  return pGlob(normalizePath(pattern), { ...options, ignore: normalizedIgnore })
+  return originalGlob(normalizePath(pattern), { ...options, ignore: normalizedIgnore })
 }
 
 export const minimatch = function (target: string, pattern: string, options?: MinimatchOptions): boolean {