fix: update Firestore collection configuration and enhance logging for better environment variable management

Author: danieldanielecki

src/server.ts

@@ -304,14 +304,30 @@ async function onCreateSendEmail(snap: FirebaseFirestore.DocumentSnapshot, _cont
   }
 }
 
+/**
+ * Must be the same collection the Angular client writes to (`getRuntimeEnv('FIRESTORE_COLLECTION_MESSAGES')`
+ * from `src/assets/env.js`, generated by `generate-browser-env.mjs` from `.env`).
+ * If unset in the Functions runtime, Cloud Functions previously defaulted to `testing`, which caused 0 invocations
+ * when the app wrote elsewhere. Set `FIRESTORE_COLLECTION_MESSAGES` in Firebase (or `.env` for emulators) if you use a different name.
+ */
 const firestoreMessagesCollection =
-  String(process.env['FIRESTORE_COLLECTION_MESSAGES'] ?? '').trim() || 'testing';
+  String(process.env['FIRESTORE_COLLECTION_MESSAGES'] ?? '').trim() ||
+  'messages_new_after_migration';
+
+functions.logger.info('contactFormFunction: configured Firestore trigger', {
+  documentPath: `${firestoreMessagesCollection}/{messageId}`,
+  envOverride: !!String(process.env['FIRESTORE_COLLECTION_MESSAGES'] ?? '').trim(),
+});
 
 export const angularUniversalFunction = functions
   .region('europe-west3')
   .https.onRequest(app);
 
-/** Any document ID — client uses `.add()` (auto IDs), not email as document ID. */
+/**
+ * Region must match your **Firestore database** location (not necessarily the same as the HTTPS SSR function).
+ * Default US Firestore (nam5) → `us-central1` triggers. EU DB (eur3) → use an EU region from Firebase’s table.
+ * Keeping SSR in `europe-west3` and this trigger in `us-central1` is valid when the DB is US and you want EU edge for HTTP only.
+ */
 export const contactFormFunction = functions
   .region('us-central1')
   .firestore