Add node_modules as a default exclude and add a skipDefaultExcludes option.

Author: lgarron

c.json

@@ -0,0 +1,6 @@
+{
+  "$schema": "./config-schema.json",
+  "gatton.net": {
+    "skipDefaultExcludes": true
+  }
+}

config-schema.json

@@ -19,11 +19,20 @@
       "properties": {
         "username": {
           "description": "SSH username for the deployment",
-          "type": ["string"]
+          "type": "string"
         },
         "fromLocalDir": {
           "description": "Deploy from this directory instead of the default. (Default example: https://example.com/path/to/dir → ./dist/web/example.com/path/to/dir).",
-          "type": ["string"]
+          "type": "string"
+        },
+        "skipDefaultExcludes": {
+          "description": "Skip all default excludes, or a subset. If any paths are included in `additionalExcludes`, that overrides this setting.",
+          "anyOf": [
+            {
+              "enum": [true]
+            },
+            { "type": "array", "items": { "type": "string" } }
+          ]
         },
         "additionalExcludes": {
           "description": "A list of additional paths to exclude. (The following are always excluded: `.git`, `.DS_Store`.)",

src/lib/deploy.ts

@@ -1,15 +1,19 @@
 import { PrintableShellCommand } from "printable-shell-command";
 import type { Args } from "./args";
 
+const DEFAULT_EXCLUDES = [".DS_Store", ".git", ".jj", "node_modules"];
+
 export interface TargetOptions {
   username?: string;
   fromLocalDir?: string;
+  skipDefaultExcludes?: true | string[];
   additionalExcludes?: string[];
 }
 
 export const targetOptionsFields = {
   username: true,
   fromLocalDir: true,
+  skipDefaultExcludes: true,
   additionalExcludes: true,
 }; // TODO: make this more DRY
 
@@ -83,9 +87,14 @@ export async function deploy(
     //   rsyncCommandArgs.push("--mkpath");
     // }
 
-    rsyncCommandArgs.push(["--exclude", ".git"]);
-    rsyncCommandArgs.push(["--exclude", ".jj"]);
-    rsyncCommandArgs.push(["--exclude", ".DS_Store"]);
+    if (targetOptions?.skipDefaultExcludes !== true) {
+      const toSkip = new Set(targetOptions?.skipDefaultExcludes ?? []);
+      for (const pattern of DEFAULT_EXCLUDES) {
+        if (!toSkip.has(pattern)) {
+          rsyncCommandArgs.push(["--exclude", pattern]);
+        }
+      }
+    }
     for (const additionalExclude of targetOptions?.additionalExcludes ?? []) {
       rsyncCommandArgs.push(["--exclude", additionalExclude]);
     }