From dff25dd726d6ea0a9b54904ec6aa91f5556df07e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ra=C3=BAl=20Negr=C3=B3n?= <raul.negron@zapier.com>
Date: Wed, 31 Jan 2024 14:45:33 -0400
Subject: [PATCH] Revert "check gitignore before copying to tmpDir"

This reverts commit 4ff1ba9a13ddba4097da71a108606d7bfba30ea4.
---
 docs/cli.html                            |  2 +-
 docs/cli.md                              |  2 +-
 packages/cli/docs/cli.html               |  2 +-
 packages/cli/docs/cli.md                 |  2 +-
 packages/cli/src/oclif/commands/build.js |  2 +-
 packages/cli/src/utils/build.js          | 24 ------------------------
 packages/cli/src/utils/files.js          |  2 +-
 7 files changed, 6 insertions(+), 30 deletions(-)

diff --git a/docs/cli.html b/docs/cli.html
index 18bec1537..5573e7415 100644
--- a/docs/cli.html
+++ b/docs/cli.html
@@ -345,7 +345,7 @@ <h2 id="build">build</h2>
 </blockquote><p><strong>Usage</strong>: <code>zapier build</code></p><p>This command does the following:</p><ul>
 <li><p>Creates a temporary folder</p>
 </li>
-<li><p>Copies all code (excluding files in .gitignore) into the temporary folder</p>
+<li><p>Copies all code into the temporary folder</p>
 </li>
 <li><p>Adds an entry point: <code>zapierwrapper.js</code></p>
 </li>
diff --git a/docs/cli.md b/docs/cli.md
index 5388fe535..434fafd2f 100644
--- a/docs/cli.md
+++ b/docs/cli.md
@@ -34,7 +34,7 @@ This command does the following:
 
 * Creates a temporary folder
 
-* Copies all code (excluding files in .gitignore) into the temporary folder
+* Copies all code into the temporary folder
 
 * Adds an entry point: `zapierwrapper.js`
 
diff --git a/packages/cli/docs/cli.html b/packages/cli/docs/cli.html
index 18bec1537..5573e7415 100644
--- a/packages/cli/docs/cli.html
+++ b/packages/cli/docs/cli.html
@@ -345,7 +345,7 @@ <h2 id="build">build</h2>
 </blockquote><p><strong>Usage</strong>: <code>zapier build</code></p><p>This command does the following:</p><ul>
 <li><p>Creates a temporary folder</p>
 </li>
-<li><p>Copies all code (excluding files in .gitignore) into the temporary folder</p>
+<li><p>Copies all code into the temporary folder</p>
 </li>
 <li><p>Adds an entry point: <code>zapierwrapper.js</code></p>
 </li>
diff --git a/packages/cli/docs/cli.md b/packages/cli/docs/cli.md
index 5388fe535..434fafd2f 100644
--- a/packages/cli/docs/cli.md
+++ b/packages/cli/docs/cli.md
@@ -34,7 +34,7 @@ This command does the following:
 
 * Creates a temporary folder
 
-* Copies all code (excluding files in .gitignore) into the temporary folder
+* Copies all code into the temporary folder
 
 * Adds an entry point: `zapierwrapper.js`
 
diff --git a/packages/cli/src/oclif/commands/build.js b/packages/cli/src/oclif/commands/build.js
index a43bc8c65..1fbbf0e0e 100644
--- a/packages/cli/src/oclif/commands/build.js
+++ b/packages/cli/src/oclif/commands/build.js
@@ -48,7 +48,7 @@ BuildCommand.description = `Build a pushable zip from the current directory.
 This command does the following:
 
 * Creates a temporary folder
-* Copies all code (excluding files in .gitignore) into the temporary folder
+* Copies all code into the temporary folder
 * Adds an entry point: \`zapierwrapper.js\`
 * Generates and validates app definition.
 * Detects dependencies via browserify (optional, on by default)
diff --git a/packages/cli/src/utils/build.js b/packages/cli/src/utils/build.js
index 2b4bedffe..ecfbfe351 100644
--- a/packages/cli/src/utils/build.js
+++ b/packages/cli/src/utils/build.js
@@ -318,29 +318,6 @@ const maybeRunBuildScript = async (options = {}) => {
   }
 };
 
-const buildCopyDirFilter = ({ wdir, skipNpmInstall = false }) => {
-  // read and parse .gitignore
-  const gitIgnorePath = path.join(wdir, '.gitignore');
-  const gitIgnoreFilter = ignore();
-
-  // create an ignore filter from .gitignore, if it exists
-  if (fs.existsSync(gitIgnorePath)) {
-    const gitIgnoredPaths = gitIgnore(gitIgnorePath);
-    const validGitIgnorePaths = gitIgnoredPaths.filter(ignore.isPathValid);
-    gitIgnoreFilter.add(validGitIgnorePaths);
-  }
-
-  return (file) => {
-    // exclude any files defined in .gitignore
-    if (gitIgnoreFilter.ignores(path.relative(wdir, file))) {
-      return false;
-    }
-
-    // exclude '.zip' files only if skipNpmInstall is true
-    return !(skipNpmInstall && file.includes('.zip'));
-  };
-};
-
 const _buildFunc = async ({
   skipNpmInstall = false,
   disableDependencyDetection = false,
@@ -586,5 +563,4 @@ module.exports = {
   listFiles,
   requiredFiles,
   maybeRunBuildScript,
-  buildCopyDirFilter,
 };
diff --git a/packages/cli/src/utils/files.js b/packages/cli/src/utils/files.js
index 997f89798..b749feb53 100644
--- a/packages/cli/src/utils/files.js
+++ b/packages/cli/src/utils/files.js
@@ -87,7 +87,7 @@ const copyFile = (src, dest, mode) => {
   Returns a promise that copies a directory recursively.
 
 	Options:
-
+		
 	- clobber: Overwrite existing files? Default is false.
 	- filter:
 			A function that returns true if the file should be copied. By default, it