refactor: simplify re-init of exclude filter

core/src/vcs/git-repo.ts

@@ -31,17 +31,14 @@ const getIncludeExcludeFiles: IncludeExcludeFilesHandler<GitRepoGetFilesParams,
   params
 ) => {
   const { include, path, scanFromProjectRoot } = params
-  let { exclude } = params
-
-  if (!exclude) {
-    exclude = []
-  }
 
   // Make sure action config is not mutated.
+  let exclude = !params.exclude ? [] : [...params.exclude]
+
   // Do the same normalization of the excluded paths like in GitHandler.
   // This might be redundant because the non-normalized paths will be handled by augmentGlobs below.
   // But this brings no harm and makes the implementation more clear.
-  exclude = [...exclude.map(normalize)]
+  exclude = exclude.map(normalize)
 
   // We allow just passing a path like `foo` as include and exclude params
   // Those need to be converted to globs, but we don't want to touch existing globs

core/src/vcs/git.ts

@@ -81,13 +81,11 @@ interface GitSubTreeIncludeExcludeFiles extends BaseIncludeExcludeFiles {
 const getIncludeExcludeFiles: IncludeExcludeFilesHandler<GetFilesParams, GitSubTreeIncludeExcludeFiles> = async (
   params: GetFilesParams
 ) => {
-  let { include, exclude } = params
-
-  if (!exclude) {
-    exclude = []
-  }
+  let include = params.include
 
   // Make sure action config is not mutated.
+  let exclude = !params.exclude ? [] : [...params.exclude]
+
   // It looks like paths with redundant '.' and '..' parts
   // do not work well along with --exclude and --glob-pathspecs flags.
   exclude = [...exclude.map(normalize), "**/.garden/**/*"]