fix(transcription): allow endpointing 0 + prod webhook logging

Address Copilot review on #275:
- endpointing settings field & URL param now accept 0 (disable
  endpointing); previously the falsy `|| fallback` and the num()
  `> 0` guard made 0 impossible to set.
- move GO-reminder scheduling error from request.log to console.*
  so it reaches Cloud Logging in production (Fastify logger is
  disabled outside dev/test).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: HugoGresse

functions/src/api/routes/whatsapp/whatsapp.ts

@@ -290,7 +290,9 @@ export const whatsappRoutes = (fastify: FastifyInstance, options: any, done: ()
                     )
                 )
             } catch (err) {
-                request.log?.error({ err }, 'failed to schedule whatsapp panel reminders')
+                // console.* (not request.log) so this reaches Cloud Logging in production, where the
+                // Fastify logger is disabled.
+                console.error('[whatsapp go] failed to schedule panel reminders', err)
             }
 
             reply.status(200).send({ sent: true })

src/public/transcription/TranscriptionSettingsPanel.tsx

@@ -126,7 +126,14 @@ export const TranscriptionSettingsPanel = ({ settings, onChange }: Transcription
                 label="Endpointing (s)"
                 inputProps={{ step: 0.05, min: 0 }}
                 value={settings.endpointing}
-                onChange={(e) => set('endpointing', Number(e.target.value) || settings.endpointing)}
+                onChange={(e) => {
+                    const parsed = Number(e.target.value)
+                    // Keep 0 (valid = disable endpointing); only fall back on empty/invalid input.
+                    set(
+                        'endpointing',
+                        e.target.value === '' || Number.isNaN(parsed) || parsed < 0 ? settings.endpointing : parsed
+                    )
+                }}
                 sx={fieldSx}
             />
             <Button variant="contained" size="small" onClick={() => set('hideSettings', true)} sx={{ ml: 'auto' }}>

src/public/transcription/transcriptionSettings.ts

@@ -44,9 +44,12 @@ export const toHexColor = (value: string): string => (value.startsWith('#') ? va
 // Read settings from a URL query string, falling back to defaults for anything missing/invalid.
 export const settingsFromQuery = (search: string): TranscriptionSettings => {
     const q = new URLSearchParams(search)
-    const num = (key: string, fallback: number) => {
-        const parsed = Number(q.get(key))
-        return Number.isFinite(parsed) && parsed > 0 ? parsed : fallback
+    const num = (key: string, fallback: number, allowZero = false) => {
+        const raw = q.get(key)
+        if (raw === null) return fallback
+        const parsed = Number(raw)
+        if (!Number.isFinite(parsed)) return fallback
+        return parsed > 0 || (allowZero && parsed === 0) ? parsed : fallback
     }
     const csv = (raw: string | null): string[] | null =>
         raw
@@ -69,7 +72,7 @@ export const settingsFromQuery = (search: string): TranscriptionSettings => {
         alignment: alignment === 'left' || alignment === 'right' ? alignment : DEFAULT_SETTINGS.alignment,
         languages: languages?.length ? languages : DEFAULT_SETTINGS.languages,
         customVocabulary: csv(q.get('custom_vocabulary')) ?? DEFAULT_SETTINGS.customVocabulary,
-        endpointing: num('endpointing', DEFAULT_SETTINGS.endpointing),
+        endpointing: num('endpointing', DEFAULT_SETTINGS.endpointing, true),
         hideSettings: q.has('hide_settings') ? q.get('hide_settings') === 'true' : DEFAULT_SETTINGS.hideSettings,
     }
 }