Major upgrade globby (#1716)

* Major upgrade globby

* Fix

Author: firecow

package-lock.json

@@ -35,7 +35,7 @@
     "dotenv": "17.x",
     "execa": "5.x.x",
     "fs-extra": "11.x.x",
-    "globby": "11.x.x",
+    "globby": "^16.0.0",
     "js-yaml": "4.x.x",
     "jsonpointer": "5.x.x",
     "micromatch": "4.x.x",

package.json

@@ -16,7 +16,7 @@ import {handler} from "./handler.js";
 import * as yaml from "js-yaml";
 import {Parser} from "./parser.js";
 import {resolveIncludeLocal, validateIncludeLocal} from "./parser-includes.js";
-import globby from "globby";
+import {globbySync} from "globby";
 import terminalLink from "terminal-link";
 import * as crypto from "crypto";
 import * as path from "path";
@@ -63,6 +63,8 @@ export interface Need {
     project?: string;
 }
 
+const isGlob = (str: string) => /[*?{}(|)[\]]/.test(str);
+
 const dateFormatter = new Intl.DateTimeFormat(undefined, {
     year: undefined,
     month: undefined,
@@ -1325,11 +1327,11 @@ If you know what you're doing and would like to suppress this warning, use one o
                 }
 
                 let path = _path;
-                if (globby.hasMagic(path) && !path.endsWith("*")) {
+                if (isGlob(path) && !path.endsWith("*")) {
                     path = `${path}/**`;
                 }
 
-                let numOfFiles = globby.sync(path, {
+                let numOfFiles = globbySync(path, {
                     dot: true,
                     onlyFiles: false,
                     cwd: `${this.argv.cwd}/${stateDir}/cache/${cacheName}`,
@@ -1340,7 +1342,7 @@ If you know what you're doing and would like to suppress this warning, use one o
                     continue;
                 }
 
-                if (!globby.hasMagic(path)) numOfFiles++; // add one because the pattern itself is a folder
+                if (!isGlob(path)) numOfFiles++; // add one because the pattern itself is a folder
 
                 writeStreams.stdout(`${_path}: found ${numOfFiles} artifact files and directories\n`);
             }

src/job.ts

@@ -9,7 +9,7 @@ import execa from "execa";
 import assert from "assert";
 import {CICDVariable} from "./variables-from-files.js";
 import {GitData} from "./git-data.js";
-import globby from "globby";
+import {globbySync} from "globby";
 import micromatch from "micromatch";
 import {AxiosRequestConfig} from "axios";
 import path from "path";
@@ -329,7 +329,7 @@ export class Utils {
             if (pattern == "") {
                 continue;
             }
-            if (globby.sync(pattern, {dot: true, cwd}).length > 0) {
+            if (globbySync(pattern, {dot: true, cwd}).length > 0) {
                 return true;
             }
         }