check gitignore before copying to tmpDir

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 into the temporary folder</p>
+<li><p>Copies all code (excluding files in .gitignore) into the temporary folder</p>
 </li>
 <li><p>Adds an entry point: <code>zapierwrapper.js</code></p>
 </li>

docs/cli.md

@@ -34,7 +34,7 @@ This command does the following:
 
 * Creates a temporary folder
 
-* Copies all code into the temporary folder
+* Copies all code (excluding files in .gitignore) into the temporary folder
 
 * Adds an entry point: `zapierwrapper.js`
 

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 into the temporary folder</p>
+<li><p>Copies all code (excluding files in .gitignore) into the temporary folder</p>
 </li>
 <li><p>Adds an entry point: <code>zapierwrapper.js</code></p>
 </li>

packages/cli/docs/cli.md

@@ -34,7 +34,7 @@ This command does the following:
 
 * Creates a temporary folder
 
-* Copies all code into the temporary folder
+* Copies all code (excluding files in .gitignore) into the temporary folder
 
 * Adds an entry point: `zapierwrapper.js`
 

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 into the temporary folder
+* Copies all code (excluding files in .gitignore) into the temporary folder
 * Adds an entry point: \`zapierwrapper.js\`
 * Generates and validates app definition.
 * Detects dependencies via browserify (optional, on by default)

packages/cli/src/utils/build.js

@@ -318,6 +318,29 @@ 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,
@@ -563,4 +586,5 @@ module.exports = {
   listFiles,
   requiredFiles,
   maybeRunBuildScript,
+  buildCopyDirFilter,
 };

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