From f7892159223f9778baf78d16c9fa740795cd94ed Mon Sep 17 00:00:00 2001
From: John Wallace <wallace.john.w@gmail.com>
Date: Sat, 5 Sep 2026 16:57:30 -0400
Subject: [PATCH] fix(macos): bundle everything under data/, not just locale

The script copied data/locale and nothing else, so data/effects never reached
the bundle and the Tiles feature was dead on every macOS install. It said so
in the log and nothing was listening:

    Tiles effect not found: effects/corevideo-tiles.effect is missing from
    the plugin's data directory

Caught live 2026-09-05 on the first properly-signed macOS test build.

Same shape as the missing-engine bug fixed the same day: the script knew what
it needed and enumerated it BY HAND rather than copying what is actually there,
so anything added to data/ afterwards was silently dropped. Copying the
directory's contents means the next addition cannot repeat this.

obs_module_file() resolves to Contents/Resources on macOS, so the layout under
data/ carries over verbatim.

Verified: Resources/effects/corevideo-tiles.effect is present in a rebuilt
bundle, alongside Resources/locale/en-US.ini.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 scripts/make-macos-bundle.sh | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/scripts/make-macos-bundle.sh b/scripts/make-macos-bundle.sh
index 693d228..d5471d7 100755
--- a/scripts/make-macos-bundle.sh
+++ b/scripts/make-macos-bundle.sh
@@ -128,7 +128,22 @@ cat > "$BUNDLE/Contents/Info.plist" <<PLIST
 PLIST
 
 cp "$MODULE" "$BUNDLE/Contents/MacOS/obs-zoom-plugin"
-[ -d "$SRC_DIR/data/locale" ] && cp -R "$SRC_DIR/data/locale" "$BUNDLE/Contents/Resources/"
+
+# EVERYTHING under data/, not a hand-listed subset. This used to copy only
+# data/locale, so data/effects never reached the bundle and the Tiles feature
+# was dead on every macOS install with one line in the log to say so:
+#   "Tiles effect not found: effects/corevideo-tiles.effect is missing from the
+#    plugin's data directory"
+# Caught live 2026-09-05. A hand-listed subset silently drops whatever is added
+# to data/ next, which is the same shape as the missing-engine bug fixed the
+# same day: the script knew what it needed and enumerated it by hand instead of
+# copying what is there. obs_module_file() resolves to Contents/Resources on
+# macOS, so the layout under data/ carries over verbatim.
+if [ -d "$SRC_DIR/data" ]; then
+    for d in "$SRC_DIR/data"/*; do
+        [ -e "$d" ] && cp -R "$d" "$BUNDLE/Contents/Resources/"
+    done
+fi
 
 # The engine and the OAuth helper must sit BESIDE the module: both are resolved
 # relative to the loaded module (dladdr) at runtime.