Fix subprocess connector setup/teardown for connectors without those commands

The error check for "unknown command" was case-sensitive and expected
quotes around the command name, but citty outputs "Unknown command setup"
(capital U, no quotes). Use case-insensitive regex instead.

This fixes Google Sheets (and any destination that doesn't implement
setup/teardown) working via the engine's subprocess connector mode.

Also enables Sheets sync in the e2e shell script (no longer gated
behind SHEETS_ENABLED).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Committed-By-Agent: claude

Author: tonyxiao

apps/engine/src/lib/destination-exec.ts

@@ -74,10 +74,7 @@ export function createDestinationFromExec(cmd: string): Destination {
           JSON.stringify(params.catalog),
         ])
       } catch (err) {
-        if (String(err).includes("unknown command 'setup'")) {
-          console.error('setup: not applicable')
-          return
-        }
+        if (/unknown command.*setup/i.test(String(err))) return
         throw err
       }
     },
@@ -91,10 +88,7 @@ export function createDestinationFromExec(cmd: string): Destination {
           JSON.stringify(params.config),
         ])
       } catch (err) {
-        if (String(err).includes("unknown command 'teardown'")) {
-          console.error('teardown: not applicable')
-          return
-        }
+        if (/unknown command.*teardown/i.test(String(err))) return
         throw err
       }
     },

apps/engine/src/lib/source-exec.ts

@@ -96,10 +96,7 @@ export function createSourceFromExec(cmd: string): Source {
           JSON.stringify(params.catalog),
         ])
       } catch (err) {
-        if (String(err).includes("unknown command 'setup'")) {
-          console.error('setup: not applicable')
-          return
-        }
+        if (/unknown command.*setup/i.test(String(err))) return
         throw err
       }
     },
@@ -113,10 +110,7 @@ export function createSourceFromExec(cmd: string): Source {
           JSON.stringify(params.config),
         ])
       } catch (err) {
-        if (String(err).includes("unknown command 'teardown'")) {
-          console.error('teardown: not applicable')
-          return
-        }
+        if (/unknown command.*teardown/i.test(String(err))) return
         throw err
       }
     },

e2e/temporal.test.sh

@@ -208,11 +208,7 @@ fi
 
 # ── Sync 2: Stripe → Google Sheets (optional) ─────────────────────
 
-# Google Sheets connector doesn't support subprocess mode (setup/teardown commands).
-# The engine uses subprocess mode when running connectors as binaries.
-# Sheets e2e is tested in vitest (temporal.test.ts) where connectors run in-process.
-SHEETS_ENABLED="${SHEETS_ENABLED:-}"
-if [ -n "$SHEETS_ENABLED" ] && [ -n "${GOOGLE_CLIENT_ID:-}" ] && [ -n "${GOOGLE_CLIENT_SECRET:-}" ] && \
+if [ -n "${GOOGLE_CLIENT_ID:-}" ] && [ -n "${GOOGLE_CLIENT_SECRET:-}" ] && \
    [ -n "${GOOGLE_REFRESH_TOKEN:-}" ] && [ -n "${GOOGLE_SPREADSHEET_ID:-}" ]; then
 
   echo ""
@@ -268,8 +264,7 @@ print(len(rows) - 1 if len(rows) > 1 else 0)  # minus header
   run_sync_cycle "Stripe → Google Sheets" "$SHEETS_SYNC_ID" verify_sheets
 else
   echo ""
-  echo "--- Skipping Google Sheets sync (set SHEETS_ENABLED=1 + Google env vars) ---"
-  echo "    Note: Sheets connector requires in-process mode; use vitest (temporal.test.ts) for full Sheets e2e"
+  echo "--- Skipping Google Sheets sync (set GOOGLE_CLIENT_ID, GOOGLE_CLIENT_SECRET, GOOGLE_REFRESH_TOKEN, GOOGLE_SPREADSHEET_ID) ---"
 fi
 
 echo ""